@k-l-lambda/lilylet 0.1.71 → 0.1.73

package/lib/tests/unit/tupletWithBaseDuration.test.d.ts

@@ -1,17 +0,0 @@
-/**
- * Unit test: \tuplet N/M D { notes } with base-duration argument must
- * preserve the tuplet wrapper in the decoded doc.
- *
- * Bug: The LilyPond decoder accessed args[1].body to find the tuplet body,
- * but \tuplet 3/2 4 { ... } has args = ["3/2", "4", {body}].
- * args[1] is "4" (the base duration), not the music block, so body was
- * empty and all notes were silently decoded as plain notes outside any tuplet.
- *
- * Real-world case: chopin-28-14.ly — every measure is wrapped in
- *   \tuplet 3/2 4 { | notes }
- * producing 12 eighth notes in the time of 8, but the decoded lyl had
- * no \times wrapper at all.
- *
- * Usage: npx tsx tests/unit/tupletWithBaseDuration.test.ts
- */
-export {};

package/lib/tests/unit/tupletWithBaseDuration.test.js

@@ -1,139 +0,0 @@
-/**
- * Unit test: \tuplet N/M D { notes } with base-duration argument must
- * preserve the tuplet wrapper in the decoded doc.
- *
- * Bug: The LilyPond decoder accessed args[1].body to find the tuplet body,
- * but \tuplet 3/2 4 { ... } has args = ["3/2", "4", {body}].
- * args[1] is "4" (the base duration), not the music block, so body was
- * empty and all notes were silently decoded as plain notes outside any tuplet.
- *
- * Real-world case: chopin-28-14.ly — every measure is wrapped in
- *   \tuplet 3/2 4 { | notes }
- * producing 12 eighth notes in the time of 8, but the decoded lyl had
- * no \times wrapper at all.
- *
- * Usage: npx tsx tests/unit/tupletWithBaseDuration.test.ts
- */
-import { decode } from "../../source/lilylet/lilypondDecoder.js";
-import { serializeLilyletDoc } from "../../source/lilylet/serializer.js";
-let passed = 0;
-let failed = 0;
-function assert(condition, message) {
-    if (condition) {
-        console.log(`  ✓ ${message}`);
-        passed++;
-    }
-    else {
-        console.error(`  ✗ FAIL: ${message}`);
-        failed++;
-    }
-}
-const LY_BOILERPLATE = `
-\\version "2.22.0"
-\\language "english"
-\\header { tagline = ##f }
-\\layout { \\context { \\Score autoBeaming = ##f } }
-`;
-// Warm-up
-{
-    const w = console.warn, a = console.assert;
-    console.warn = () => { };
-    console.assert = () => { };
-    try {
-        await decode('{ c }');
-    }
-    catch { }
-    console.warn = w;
-    console.assert = a;
-}
-// ─── Test 1: \tuplet 3/2 4 { notes } preserves tuplet ─────────────────────
-// Chopin 28-14 pattern: \tuplet 3/2 4 wraps 12 eighth notes per measure.
-// Without base-duration: \tuplet 3/2 { ef8 bf gf ... } = 12 notes
-// With base-duration:    \tuplet 3/2 4 { | ef8 bf gf ... } = same semantics
-console.log('\nTest 1: \\tuplet 3/2 4 { notes } — tuplet wrapper preserved');
-await (async () => {
-    const LY = LY_BOILERPLATE + `
-\\score {
-  \\new Staff {
-    \\new Voice {
-      \\relative ef, {
-        \\time 4/4
-        \\tuplet 3/2 4 {
-          ef8 [ bf' gf ] cf [ ef, cf' ] d, [ cf' f, ] bf [ d, bf']
-        }
-      }
-    }
-  }
-  \\layout {}
-}
-`;
-    const doc = await decode(LY);
-    assert(doc.measures.length >= 1, `decoded 1 measure (got ${doc.measures.length})`);
-    if (!doc.measures[0])
-        return;
-    const voice = doc.measures[0].parts[0]?.voices[0];
-    assert(!!voice, 'voice exists');
-    if (!voice)
-        return;
-    // The voice should have a tuplet event, NOT 12 bare note events
-    const tuplets = voice.events.filter(e => e.type === 'tuplet' || e.type === 'times');
-    const bareNotes = voice.events.filter(e => e.type === 'note');
-    assert(tuplets.length >= 1, `voice contains at least 1 tuplet/times event (got ${tuplets.length}) — not bare notes`);
-    assert(bareNotes.length === 0, `no bare notes at top level (got ${bareNotes.length}) — all should be inside tuplet`);
-    if (tuplets.length >= 1) {
-        const t = tuplets[0];
-        const innerNotes = t.events.filter(e => e.type === 'note');
-        assert(innerNotes.length === 12, `tuplet contains 12 inner notes (got ${innerNotes.length})`);
-        assert(t.ratio.numerator === 2 && t.ratio.denominator === 3, `ratio is 2/3 (lilylet: play 12 eighth notes in time of 8) — got ${t.ratio.numerator}/${t.ratio.denominator}`);
-    }
-})();
-// ─── Test 2: \tuplet 3/2 (no base duration) — should still work ────────────
-console.log('\nTest 2: \\tuplet 3/2 { notes } without base duration (baseline)');
-await (async () => {
-    const LY = LY_BOILERPLATE + `
-\\score {
-  \\new Staff {
-    \\new Voice {
-      \\relative ef, {
-        \\time 4/4
-        \\tuplet 3/2 {
-          ef8 [ bf' gf ] cf [ ef, cf' ] d, [ cf' f, ] bf [ d, bf']
-        }
-      }
-    }
-  }
-  \\layout {}
-}
-`;
-    const doc = await decode(LY);
-    const voice = doc.measures[0]?.parts[0]?.voices[0];
-    const tuplets = voice?.events.filter(e => e.type === 'tuplet' || e.type === 'times') ?? [];
-    assert(tuplets.length >= 1, `\\tuplet 3/2 (no base dur) also produces tuplet event (got ${tuplets.length})`);
-})();
-// ─── Test 3: serialized lyl contains \tuplet or \times wrapper ─────────────
-console.log('\nTest 3: serialized lyl contains tuplet notation');
-await (async () => {
-    const LY = LY_BOILERPLATE + `
-\\score {
-  \\new Staff {
-    \\new Voice {
-      \\relative ef, {
-        \\time 4/4
-        \\tuplet 3/2 4 {
-          ef8 [ bf' gf ] cf [ ef, cf' ] d, [ cf' f, ] bf [ d, bf']
-        }
-      }
-    }
-  }
-  \\layout {}
-}
-`;
-    const doc = await decode(LY);
-    const lyl = serializeLilyletDoc(doc);
-    console.log(`  lyl: ${lyl.trim().split('\n').find(l => l.includes('time') || l.includes('tuplet') || l.includes('times')) ?? '(not found)'}`);
-    assert(lyl.includes('\\tuplet') || lyl.includes('\\times'), `serialized lyl contains \\tuplet or \\times notation (got no tuplet wrapper)`);
-})();
-// ─── summary ─────────────────────────────────────────────────────────────────
-console.log(`\n${'═'.repeat(50)}`);
-console.log(`Total: ${passed + failed}  Passed: ${passed}  Failed: ${failed}`);
-process.exit(failed > 0 ? 1 : 0);

package/lib/tests/unit/voiceStaffParsing.test.d.ts

@@ -1,13 +0,0 @@
-/**
- * Unit tests for voice.staff parsing edge cases from GPT review of the
- * leadingStaff fix (commit c0763a4, 47fcf61).
- *
- * Cases:
- *   A. non-staff context before \staff "N" in same voice line
- *   B. second voice after \\ with no explicit \staff (fallback)
- *   C. trailing | + newline → spurious empty measure filtered
- *   D. measure with only \bar → filtered as empty
- *
- * Usage: npx tsx tests/unit/voiceStaffParsing.test.ts
- */
-export {};

package/lib/tests/unit/voiceStaffParsing.test.js

@@ -1,118 +0,0 @@
-/**
- * Unit tests for voice.staff parsing edge cases from GPT review of the
- * leadingStaff fix (commit c0763a4, 47fcf61).
- *
- * Cases:
- *   A. non-staff context before \staff "N" in same voice line
- *   B. second voice after \\ with no explicit \staff (fallback)
- *   C. trailing | + newline → spurious empty measure filtered
- *   D. measure with only \bar → filtered as empty
- *
- * Usage: npx tsx tests/unit/voiceStaffParsing.test.ts
- */
-import { parseCode } from '../../source/lilylet/parser.js';
-let passed = 0;
-let failed = 0;
-function assert(condition, message) {
-    if (condition) {
-        console.log(`  ✓ ${message}`);
-        passed++;
-    }
-    else {
-        console.error(`  ✗ FAIL: ${message}`);
-        failed++;
-    }
-}
-function voiceStaff(lyl, measureIdx = 0, voiceIdx = 0) {
-    const doc = parseCode(lyl);
-    return doc.measures[measureIdx]?.parts[0]?.voices[voiceIdx]?.staff ?? -1;
-}
-function measureCount(lyl) {
-    return parseCode(lyl).measures.length;
-}
-// ─── Case A1: \clef before \staff "2" ────────────────────────────────────────
-console.log('\nCase A1: \\clef before \\staff "2" — staff must be 2');
-{
-    const lyl = `\\clef "bass" \\staff "2" c4 d e f |`;
-    assert(voiceStaff(lyl) === 2, `\\clef "bass" \\staff "2" c4 → voice.staff=2 (got ${voiceStaff(lyl)})`);
-}
-// ─── Case A2: \key before \staff "2" ─────────────────────────────────────────
-console.log('\nCase A2: \\key before \\staff "2" — staff must be 2');
-{
-    const lyl = `\\key g \\major \\staff "2" c4 d e f |`;
-    assert(voiceStaff(lyl) === 2, `\\key g \\major \\staff "2" → voice.staff=2 (got ${voiceStaff(lyl)})`);
-}
-// ─── Case A3: \time \clef \key before \staff "2" \times ──────────────────────
-console.log('\nCase A3: multiple context events before \\staff "2" \\times');
-{
-    const lyl = `\\time 4/4 \\clef "treble" \\key c \\major \\staff "2" \\times 2/3 { c8 d e } r2. |`;
-    const doc = parseCode(lyl);
-    const voice = doc.measures[0]?.parts[0]?.voices[0];
-    assert(!!voice, 'voice exists');
-    if (voice) {
-        assert(voice.staff === 2, `context chain before \\staff "2" → voice.staff=2 (got ${voice.staff})`);
-        // Verify the event stream: leading context events (non-staff) then staff=2 context before tuplet
-        const staffCtxIdx = voice.events.findIndex((e) => e.type === 'context' && e.staff === 2);
-        const tupletIdx = voice.events.findIndex((e) => e.type === 'tuplet' || e.type === 'times');
-        assert(staffCtxIdx >= 0, `context{staff:2} event exists in events`);
-        assert(tupletIdx >= 0, `tuplet event exists in events`);
-        assert(staffCtxIdx < tupletIdx, `context{staff:2} (idx=${staffCtxIdx}) appears before tuplet (idx=${tupletIdx})`);
-    }
-}
-// ─── Case A4: \staff "1" first, cross-staff switch — staff must be 1 ─────────
-console.log('\nCase A4: \\staff "1" then \\staff "2" — leading staff is 1');
-{
-    const lyl = `\\staff "1" c4 d \\staff "2" e f |`;
-    assert(voiceStaff(lyl) === 1, `\\staff "1" c4 d \\staff "2" e f → voice.staff=1 (got ${voiceStaff(lyl)})`);
-}
-// ─── Case B1: second voice — stale currentStaff=2 must NOT leak ──────────────
-// Voice 0 ends after switching to \staff "2" (currentStaff=2 is stale).
-// Voice 1 has no explicit \staff — should default to staff=1 (part_start reset),
-// NOT inherit the stale currentStaff=2 from voice 0.
-console.log('\nCase B1: second voice (no \\staff) must NOT inherit stale currentStaff=2');
-{
-    const lyl = `\\staff "1" c4 \\staff "2" d e f \\\\ g4 a b c |`;
-    const doc = parseCode(lyl);
-    const voices = doc.measures[0]?.parts[0]?.voices ?? [];
-    assert(voices.length === 2, `two voices parsed (got ${voices.length})`);
-    assert(voices[0].staff === 1, `voice 0 leading \\staff "1" (got ${voices[0]?.staff})`);
-    // part_start resets currentStaff=1 for each part_voices attempt;
-    // if stale currentStaff=2 leaked, voice 1 would wrongly get staff=2
-    assert(voices[1].staff === 1, `voice 1 no \\staff → defaults to 1, not stale 2 (got ${voices[1]?.staff})`);
-}
-// ─── Case B2: \\ then \staff "2" ─────────────────────────────────────────────
-console.log('\nCase B2: second voice starts with \\staff "2"');
-{
-    const lyl = `\\staff "1" c4 d e f \\\\ \\staff "2" c4 d e f |`;
-    const doc = parseCode(lyl);
-    const voices = doc.measures[0]?.parts[0]?.voices ?? [];
-    assert(voices.length === 2, `two voices parsed (got ${voices.length})`);
-    assert(voices[1].staff === 2, `voice 1 \\staff "2" → staff=2 (got ${voices[1]?.staff})`);
-}
-// ─── Case C: trailing newline after last | — no spurious measure ─────────────
-console.log('\nCase C: trailing newline after last | — no spurious empty measure');
-{
-    // lyl with explicit trailing newline
-    const lyl = `\\staff "1" c4 d e f |\n`;
-    const count = measureCount(lyl);
-    assert(count === 1, `exactly 1 measure (got ${count})`);
-}
-// ─── Case D: measure with only \bar — filtered as empty ──────────────────────
-console.log('\nCase D: measure with only \\bar "|." — filtered as empty');
-{
-    const lyl = `\\staff "1" c4 d e f | \\bar "|." |`;
-    const count = measureCount(lyl);
-    // The real measure (c4 d e f) is kept; the barline-only measure is filtered
-    assert(count === 1, `barline-only measure filtered, 1 real measure remains (got ${count})`);
-}
-// ─── Case E: measure with notes then trailing empty measure ──────────────────
-console.log('\nCase E: two real measures, no spurious third');
-{
-    const lyl = `\\staff "1" c4 d e f | g4 a b c |\n`;
-    const count = measureCount(lyl);
-    assert(count === 2, `exactly 2 measures (got ${count})`);
-}
-// ─── Summary ─────────────────────────────────────────────────────────────────
-console.log(`\n${'═'.repeat(50)}`);
-console.log(`Total: ${passed + failed}  Passed: ${passed}  Failed: ${failed}`);
-process.exit(failed > 0 ? 1 : 0);