@k-l-lambda/lilylet 0.1.56 → 0.1.57

package/lib/lilylet/lilypondDecoder.js

@@ -587,10 +587,6 @@ const parseLilyDocument = (lilyDocument) => {
                             duration: convertDuration(term.durationValue),
                             invisible: term.isSpacer || undefined,
                         };
-                        // Positioned rest
-                        if (!term.isSpacer && context.pitch) {
-                            restEvent.pitch = convertPitch(context.pitch.phonetStep, 0, context.pitch.octave);
-                        }
                         voice.events.push(restEvent);
                     }
                 }
@@ -982,7 +978,7 @@ const parsedMeasuresToDoc = (parsedMeasures, metadata) => {
     const measures = parsedMeasures.map(pm => {
         // Filter out voices that only contain spacer rests and context changes
         const filteredVoices = pm.voices.filter(v => hasRealContent(v.events));
-        // Group voices by partIndex, then merge voices on the same staff
+        // Group voices by partIndex, then collect voice arrays per staff
         const partMap = new Map();
         for (const v of filteredVoices) {
             const pi = v.partIndex || 1;
@@ -990,23 +986,23 @@ const parsedMeasuresToDoc = (parsedMeasures, metadata) => {
                 partMap.set(pi, new Map());
             }
             const staffMap = partMap.get(pi);
-            // Merge events from voices on the same staff
+            // Preserve each voice as a separate array
             if (!staffMap.has(v.staff)) {
                 staffMap.set(v.staff, []);
             }
-            staffMap.get(v.staff).push(...v.events);
+            staffMap.get(v.staff).push(v.events);
         }
         // Convert to parts array (sorted by part index, then by staff)
-        // Apply deduplication to merged events
+        // Apply deduplication to each voice's events
         const partIndices = Array.from(partMap.keys()).sort((a, b) => a - b);
         const parts = partIndices.map(pi => {
             const staffMap = partMap.get(pi);
             const staffNums = Array.from(staffMap.keys()).sort((a, b) => a - b);
             return {
-                voices: staffNums.map(staff => ({
+                voices: staffNums.flatMap(staff => staffMap.get(staff).map(events => ({
                     staff,
-                    events: dedupeContextEvents(staffMap.get(staff)),
-                })),
+                    events: dedupeContextEvents(events),
+                }))),
             };
         });
         // Fallback to single empty part if no voices

package/lib/lilylet/lilypondEncoder.js

@@ -294,11 +294,19 @@ const encodeNoteEvent = (event, env, lastDuration) => {
     if (event.pitches.length > 1) {
         result += '<';
         const pitchStrs = [];
-        for (const pitch of event.pitches) {
-            const { str, newEnv: ne } = encodePitch(pitch, newEnv);
+        let firstPitchEnv;
+        for (let i = 0; i < event.pitches.length; i++) {
+            // In LilyPond relative mode, each pitch in a chord is relative
+            // to the previous pitch in the chord (cascading).
+            // After the chord, env becomes the first pitch.
+            const { str, newEnv: ne } = encodePitch(event.pitches[i], newEnv);
             pitchStrs.push(str);
             newEnv = ne;
+            if (i === 0)
+                firstPitchEnv = ne;
         }
+        // After chord, reference resets to first pitch
+        newEnv = firstPitchEnv;
         result += pitchStrs.join(' ');
         result += '>';
     }
@@ -347,12 +355,14 @@ const encodeRestEvent = (event, env, lastDuration) => {
         result += encodeDuration(event.duration);
     }
     // Positioned rest
+    let newEnv = env;
     if (event.pitch && !event.fullMeasure && !event.invisible) {
-        const { str } = encodePitch(event.pitch, env);
+        const { str, newEnv: ne } = encodePitch(event.pitch, env);
         result = str + result.slice(1); // Replace 'r' with pitch
         result += '\\rest';
+        newEnv = ne;
     }
-    return { str: result, newEnv: env, newDuration: event.duration };
+    return { str: result, newEnv, newDuration: event.duration };
 };
 /**
  * Encode a context change event
@@ -397,8 +407,9 @@ const encodeTupletEvent = (event, env, lastDuration) => {
             newDuration = nd;
         }
         else if (subEvent.type === 'rest') {
-            const { str, newDuration: nd } = encodeRestEvent(subEvent, newEnv, newDuration);
+            const { str, newEnv: ne, newDuration: nd } = encodeRestEvent(subEvent, newEnv, newDuration);
             result += str + ' ';
+            newEnv = ne;
             newDuration = nd;
         }
     }
@@ -415,11 +426,15 @@ const encodeTremoloEvent = (event, env) => {
     if (event.pitchA.length > 1) {
         pitchA += '<';
         const pitchStrs = [];
-        for (const pitch of event.pitchA) {
-            const { str, newEnv: ne } = encodePitch(pitch, newEnv);
+        let firstPitchEnv;
+        for (let i = 0; i < event.pitchA.length; i++) {
+            const { str, newEnv: ne } = encodePitch(event.pitchA[i], newEnv);
             pitchStrs.push(str);
             newEnv = ne;
+            if (i === 0)
+                firstPitchEnv = ne;
         }
+        newEnv = firstPitchEnv;
         pitchA += pitchStrs.join(' ');
         pitchA += '>';
     }
@@ -433,11 +448,15 @@ const encodeTremoloEvent = (event, env) => {
     if (event.pitchB.length > 1) {
         pitchB += '<';
         const pitchStrs = [];
-        for (const pitch of event.pitchB) {
-            const { str, newEnv: ne } = encodePitch(pitch, newEnv);
+        let firstPitchEnv;
+        for (let i = 0; i < event.pitchB.length; i++) {
+            const { str, newEnv: ne } = encodePitch(event.pitchB[i], newEnv);
             pitchStrs.push(str);
             newEnv = ne;
+            if (i === 0)
+                firstPitchEnv = ne;
         }
+        newEnv = firstPitchEnv;
         pitchB += pitchStrs.join(' ');
         pitchB += '>';
     }
@@ -490,8 +509,9 @@ const encodeVoice = (voice, measureContext, voiceIndex) => {
                 break;
             }
             case 'rest': {
-                const { str, newDuration } = encodeRestEvent(event, env, lastDuration);
+                const { str, newEnv, newDuration } = encodeRestEvent(event, env, lastDuration);
                 result += str + ' ';
+                env = newEnv;
                 lastDuration = newDuration;
                 break;
             }
@@ -528,7 +548,8 @@ const encodeVoice = (voice, measureContext, voiceIndex) => {
                 break;
             }
             case 'pitchReset': {
-                env = { step: 0, octave: 0 };
+                // Ignore: each measure already gets its own \relative c' block,
+                // and within a measure the LilyPond reference pitch is not reset.
                 break;
             }
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@k-l-lambda/lilylet",
-  "version": "0.1.56",
+  "version": "0.1.57",
   "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",

package/source/lilylet/lilypondDecoder.ts

@@ -711,15 +711,6 @@ const parseLilyDocument = (lilyDocument: lilyParser.LilyDocument): ParsedMeasure
 							invisible: term.isSpacer || undefined,
 						};
 
-						// Positioned rest
-						if (!term.isSpacer && context.pitch) {
-							restEvent.pitch = convertPitch(
-								context.pitch.phonetStep,
-								0,
-								context.pitch.octave
-							);
-						}
-
 						voice.events.push(restEvent);
 					}
 				}
@@ -1145,8 +1136,8 @@ const parsedMeasuresToDoc = (parsedMeasures: ParsedMeasure[], metadata?: Metadat
 		// Filter out voices that only contain spacer rests and context changes
 		const filteredVoices = pm.voices.filter(v => hasRealContent(v.events));
 
-		// Group voices by partIndex, then merge voices on the same staff
-		const partMap = new Map<number, Map<number, Event[]>>();
+		// Group voices by partIndex, then collect voice arrays per staff
+		const partMap = new Map<number, Map<number, Event[][]>>();
 		for (const v of filteredVoices) {
 			const pi = v.partIndex || 1;
 			if (!partMap.has(pi)) {
@@ -1154,24 +1145,26 @@ const parsedMeasuresToDoc = (parsedMeasures: ParsedMeasure[], metadata?: Metadat
 			}
 			const staffMap = partMap.get(pi)!;
 
-			// Merge events from voices on the same staff
+			// Preserve each voice as a separate array
 			if (!staffMap.has(v.staff)) {
 				staffMap.set(v.staff, []);
 			}
-			staffMap.get(v.staff)!.push(...v.events);
+			staffMap.get(v.staff)!.push(v.events);
 		}
 
 		// Convert to parts array (sorted by part index, then by staff)
-		// Apply deduplication to merged events
+		// Apply deduplication to each voice's events
 		const partIndices = Array.from(partMap.keys()).sort((a, b) => a - b);
 		const parts = partIndices.map(pi => {
 			const staffMap = partMap.get(pi)!;
 			const staffNums = Array.from(staffMap.keys()).sort((a, b) => a - b);
 			return {
-				voices: staffNums.map(staff => ({
-					staff,
-					events: dedupeContextEvents(staffMap.get(staff)!),
-				})),
+				voices: staffNums.flatMap(staff =>
+					staffMap.get(staff)!.map(events => ({
+						staff,
+						events: dedupeContextEvents(events),
+					}))
+				),
 			};
 		});
 

package/source/lilylet/lilypondEncoder.ts

@@ -394,11 +394,18 @@ const encodeNoteEvent = (event: NoteEvent, env: PitchEnv, lastDuration: Duration
 	if (event.pitches.length > 1) {
 		result += '<';
 		const pitchStrs: string[] = [];
-		for (const pitch of event.pitches) {
-			const { str, newEnv: ne } = encodePitch(pitch, newEnv);
+		let firstPitchEnv: PitchEnv | undefined;
+		for (let i = 0; i < event.pitches.length; i++) {
+			// In LilyPond relative mode, each pitch in a chord is relative
+			// to the previous pitch in the chord (cascading).
+			// After the chord, env becomes the first pitch.
+			const { str, newEnv: ne } = encodePitch(event.pitches[i], newEnv);
 			pitchStrs.push(str);
 			newEnv = ne;
+			if (i === 0) firstPitchEnv = ne;
 		}
+		// After chord, reference resets to first pitch
+		newEnv = firstPitchEnv!;
 		result += pitchStrs.join(' ');
 		result += '>';
 	} else if (event.pitches.length === 1) {
@@ -455,13 +462,15 @@ const encodeRestEvent = (event: RestEvent, env: PitchEnv, lastDuration: Duration
 	}
 
 	// Positioned rest
+	let newEnv = env;
 	if (event.pitch && !event.fullMeasure && !event.invisible) {
-		const { str } = encodePitch(event.pitch, env);
+		const { str, newEnv: ne } = encodePitch(event.pitch, env);
 		result = str + result.slice(1);  // Replace 'r' with pitch
 		result += '\\rest';
+		newEnv = ne;
 	}
 
-	return { str: result, newEnv: env, newDuration: event.duration };
+	return { str: result, newEnv, newDuration: event.duration };
 };
 
 
@@ -512,8 +521,9 @@ const encodeTupletEvent = (event: TupletEvent, env: PitchEnv, lastDuration: Dura
 			newEnv = ne;
 			newDuration = nd;
 		} else if (subEvent.type === 'rest') {
-			const { str, newDuration: nd } = encodeRestEvent(subEvent, newEnv, newDuration);
+			const { str, newEnv: ne, newDuration: nd } = encodeRestEvent(subEvent, newEnv, newDuration);
 			result += str + ' ';
+			newEnv = ne;
 			newDuration = nd;
 		}
 	}
@@ -535,11 +545,14 @@ const encodeTremoloEvent = (event: TremoloEvent, env: PitchEnv): { str: string;
 	if (event.pitchA.length > 1) {
 		pitchA += '<';
 		const pitchStrs: string[] = [];
-		for (const pitch of event.pitchA) {
-			const { str, newEnv: ne } = encodePitch(pitch, newEnv);
+		let firstPitchEnv: PitchEnv | undefined;
+		for (let i = 0; i < event.pitchA.length; i++) {
+			const { str, newEnv: ne } = encodePitch(event.pitchA[i], newEnv);
 			pitchStrs.push(str);
 			newEnv = ne;
+			if (i === 0) firstPitchEnv = ne;
 		}
+		newEnv = firstPitchEnv!;
 		pitchA += pitchStrs.join(' ');
 		pitchA += '>';
 	} else if (event.pitchA.length === 1) {
@@ -553,11 +566,14 @@ const encodeTremoloEvent = (event: TremoloEvent, env: PitchEnv): { str: string;
 	if (event.pitchB.length > 1) {
 		pitchB += '<';
 		const pitchStrs: string[] = [];
-		for (const pitch of event.pitchB) {
-			const { str, newEnv: ne } = encodePitch(pitch, newEnv);
+		let firstPitchEnv: PitchEnv | undefined;
+		for (let i = 0; i < event.pitchB.length; i++) {
+			const { str, newEnv: ne } = encodePitch(event.pitchB[i], newEnv);
 			pitchStrs.push(str);
 			newEnv = ne;
+			if (i === 0) firstPitchEnv = ne;
 		}
+		newEnv = firstPitchEnv!;
 		pitchB += pitchStrs.join(' ');
 		pitchB += '>';
 	} else if (event.pitchB.length === 1) {
@@ -624,8 +640,9 @@ const encodeVoice = (
 				break;
 			}
 			case 'rest': {
-				const { str, newDuration } = encodeRestEvent(event, env, lastDuration);
+				const { str, newEnv, newDuration } = encodeRestEvent(event, env, lastDuration);
 				result += str + ' ';
+				env = newEnv;
 				lastDuration = newDuration;
 				break;
 			}
@@ -662,7 +679,8 @@ const encodeVoice = (
 				break;
 			}
 			case 'pitchReset': {
-				env = { step: 0, octave: 0 };
+				// Ignore: each measure already gets its own \relative c' block,
+				// and within a measure the LilyPond reference pitch is not reset.
 				break;
 			}
 		}