@k-l-lambda/lilylet 0.1.50 → 0.1.51

package/lib/lilylet/serializer.js

@@ -190,6 +190,9 @@ const serializeMarks = (marks) => {
                     parts.push(pedalStr);
                 break;
             }
+            case 'fingering':
+                parts.push('-' + mark.finger);
+                break;
         }
     }
     return parts.join('');
@@ -449,8 +452,9 @@ const findVoiceClef = (voice) => {
 // Takes currentStaff (what parser thinks staff is) and returns { str, newStaff }
 // If isGrandStaff is true, always output \staff command for clarity
 // measureContext provides key/time for first voice
-// staffClef is the clef for this voice's staff (tracked across measures)
-const serializeVoice = (voice, currentStaff, isGrandStaff = false, measureContext, isFirstVoice = false, staffClef) => {
+// allStaffClefs is the clef map for all staves (tracked across measures)
+// emittedClefs tracks which clefs have already been output (avoids duplicates)
+const serializeVoice = (voice, currentStaff, isGrandStaff = false, measureContext, isFirstVoice = false, allStaffClefs, emittedClefs) => {
     const parts = [];
     let prevDuration;
     // Each voice starts fresh from middle C (step=0, octave=0)
@@ -480,22 +484,61 @@ const serializeVoice = (voice, currentStaff, isGrandStaff = false, measureContex
             parts.push('\\time ' + numerator + '/' + denominator);
         }
     }
-    // Output clef for every voice (use staff clef tracked across measures, or find from voice events)
-    const voiceClef = staffClef || findVoiceClef(voice);
-    if (voiceClef) {
+    // Output clef only if not yet emitted or changed for this staff
+    const voiceClef = allStaffClefs?.[voice.staff] || findVoiceClef(voice);
+    const clefAlreadyEmitted = voiceClef && emittedClefs?.[voice.staff] === voiceClef;
+    if (voiceClef && !clefAlreadyEmitted) {
         parts.push('\\clef "' + CLEF_MAP[voiceClef] + '"');
+        if (emittedClefs)
+            emittedClefs[voice.staff] = voiceClef;
     }
-    // Track if we've already output the clef to avoid duplication
-    let clefOutputted = !!voiceClef;
+    // Skip redundant clef context events if this staff's clef is already established
+    const clefOutputted = !!voiceClef && !!emittedClefs?.[voice.staff];
+    let activeStaff = voice.staff;
+    let activeStemDir;
     for (const event of voice.events) {
-        // Skip clef context events if we've already output the clef at the beginning
-        if (clefOutputted && event.type === 'context') {
+        if (event.type === 'context') {
             const ctx = event;
-            if (ctx.clef && !ctx.key && !ctx.time && !ctx.ottava && !ctx.stemDirection && !ctx.tempo && !ctx.staff) {
-                // This is a clef-only context event, skip it
+            // Skip context events that belong to a different staff (cross-staff clef/ottava)
+            if (ctx.staff && ctx.staff !== voice.staff) {
+                continue;
+            }
+            // Skip clef-only context events if clef already established for this staff
+            if (clefOutputted && ctx.clef && !ctx.key && !ctx.time && !ctx.ottava && !ctx.stemDirection && !ctx.tempo) {
                 continue;
             }
         }
+        if (event.type === 'note') {
+            const noteEvt = event;
+            // Cross-staff: emit \staff when note's effective staff differs from active
+            const effectiveStaff = noteEvt.staff || voice.staff;
+            if (effectiveStaff !== activeStaff) {
+                activeStaff = effectiveStaff;
+                parts.push('\\staff "' + activeStaff + '"');
+                // Emit the target staff's clef if it differs from what was last emitted for this staff
+                const targetClef = allStaffClefs?.[activeStaff];
+                if (targetClef && emittedClefs?.[activeStaff] !== targetClef) {
+                    parts.push('\\clef "' + CLEF_MAP[targetClef] + '"');
+                    if (emittedClefs)
+                        emittedClefs[activeStaff] = targetClef;
+                }
+            }
+            // Stem direction: emit \stemUp/\stemDown/\stemNeutral on change
+            const stemDir = noteEvt.stemDirection;
+            if (stemDir !== activeStemDir) {
+                if (stemDir === StemDirection.up) {
+                    parts.push('\\stemUp');
+                }
+                else if (stemDir === StemDirection.down) {
+                    parts.push('\\stemDown');
+                }
+                else if (activeStemDir) {
+                    // Was set, now undefined → reset to neutral
+                    parts.push('\\stemNeutral');
+                }
+                activeStemDir = stemDir;
+            }
+        }
         const { str: eventStr, newEnv } = serializeEvent(event, pitchEnv, prevDuration);
         pitchEnv = newEnv;
         if (eventStr) {
@@ -508,12 +551,16 @@ const serializeVoice = (voice, currentStaff, isGrandStaff = false, measureContex
         else if (event.type === 'rest') {
             prevDuration = event.duration;
         }
+        else if (event.type === 'context' && event.clef && emittedClefs) {
+            const ctx = event;
+            emittedClefs[ctx.staff || activeStaff] = ctx.clef;
+        }
     }
     return { str: parts.join(' '), newStaff: voice.staff };
 };
 // Serialize a part, tracking staff state across voices
 // measureContext is passed to all voices (for clef), but key/time only to first voice
-const serializePart = (part, currentStaff, isGrandStaff = false, measureContext, isFirstPart = false, clefsByStaff) => {
+const serializePart = (part, currentStaff, isGrandStaff = false, measureContext, isFirstPart = false, clefsByStaff, emittedClefs) => {
     if (part.voices.length === 0) {
         return { str: '', newStaff: currentStaff };
     }
@@ -522,10 +569,8 @@ const serializePart = (part, currentStaff, isGrandStaff = false, measureContext,
     for (let i = 0; i < part.voices.length; i++) {
         const voice = part.voices[i];
         // Pass measureContext to all voices, isFirstVoice for key/time
-        // Pass staff clef from clefsByStaff map
         const isFirstVoice = isFirstPart && i === 0;
-        const staffClef = clefsByStaff?.[voice.staff];
-        const { str, newStaff } = serializeVoice(voice, staff, isGrandStaff, measureContext, isFirstVoice, staffClef);
+        const { str, newStaff } = serializeVoice(voice, staff, isGrandStaff, measureContext, isFirstVoice, clefsByStaff, emittedClefs);
         voiceStrs.push(str);
         staff = newStaff;
     }
@@ -534,7 +579,7 @@ const serializePart = (part, currentStaff, isGrandStaff = false, measureContext,
 };
 // Serialize a measure, tracking staff state across parts
 // Always output key/time at start of each measure
-const serializeMeasure = (measure, _isFirst, currentStaff, isGrandStaff = false, currentKey, currentTime, staffClefs) => {
+const serializeMeasure = (measure, _isFirst, currentStaff, isGrandStaff = false, currentKey, currentTime, staffClefs, emittedClefs) => {
     const parts = [];
     // Build measure context for all voices (key/time)
     // Key and time are written to first voice, clef to all voices based on staff
@@ -548,7 +593,7 @@ const serializeMeasure = (measure, _isFirst, currentStaff, isGrandStaff = false,
     // Parts
     let staff = currentStaff;
     if (measure.parts.length === 1) {
-        const { str: partStr, newStaff } = serializePart(measure.parts[0], staff, isGrandStaff, measureContext, true, clefsByStaff);
+        const { str: partStr, newStaff } = serializePart(measure.parts[0], staff, isGrandStaff, measureContext, true, clefsByStaff, emittedClefs);
         if (partStr) {
             parts.push(partStr);
         }
@@ -560,7 +605,7 @@ const serializeMeasure = (measure, _isFirst, currentStaff, isGrandStaff = false,
         for (let i = 0; i < measure.parts.length; i++) {
             const part = measure.parts[i];
             // Pass measureContext to all parts, isFirstPart to first part only
-            const { str, newStaff } = serializePart(part, staff, isGrandStaff, measureContext, i === 0, clefsByStaff);
+            const { str, newStaff } = serializePart(part, staff, isGrandStaff, measureContext, i === 0, clefsByStaff, emittedClefs);
             if (str) {
                 partStrs.push(str);
             }
@@ -620,6 +665,7 @@ export const serializeLilyletDoc = (doc) => {
     let currentKey;
     let currentTime;
     const staffClefs = {}; // Track clef per staff
+    const emittedClefs = {}; // Track which clefs have been output
     for (let i = 0; i < doc.measures.length; i++) {
         const measure = doc.measures[i];
         // Update current key/time if measure has them
@@ -634,12 +680,15 @@ export const serializeLilyletDoc = (doc) => {
             for (const voice of part.voices) {
                 for (const event of voice.events) {
                     if (event.type === 'context' && event.clef) {
-                        staffClefs[voice.staff] = event.clef;
+                        const ctx = event;
+                        // Use the event's staff if specified (cross-staff), otherwise the voice's staff
+                        const clefStaff = ctx.staff || voice.staff;
+                        staffClefs[clefStaff] = ctx.clef;
                     }
                 }
             }
         }
-        const { str: measureStr, newStaff } = serializeMeasure(measure, i === 0, currentStaff, isGrandStaff, currentKey, currentTime, staffClefs);
+        const { str: measureStr, newStaff } = serializeMeasure(measure, i === 0, currentStaff, isGrandStaff, currentKey, currentTime, staffClefs, emittedClefs);
         // Always include measure, even if empty (use space rest for empty measures)
         measureStrs.push(measureStr || 's1');
         currentStaff = newStaff;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@k-l-lambda/lilylet",
-  "version": "0.1.50",
+  "version": "0.1.51",
   "description": "Lilylet is a lilyopnd-like sheet music language designed for Markdown rendering and symbolic music representation in AIGC applications.",
   "type": "module",
   "main": "lib/index.js",