docs: capture Morpheus parity findings; de-flake dark-mode e2e

The dark-mode macronizer test clicked straight after goto(), racing the
module script that top-level-awaits __wasmReady — intermittently failed on
a 60s timeout and passed on retry. Wait for #macronize_btn like the other
tests do. Suite drops 1.6m -> 29s with no flake.

CLAUDE.md: new 'Macronizer / Morpheus traps' section recording the four
diagnostics that cost this session (comma in the Morpheus accented form,
the entriesCache staleness, the cruncher.js-vs-.data manifest that makes
byte-grepping the blob useless, and the GREEK language default that makes
a working cruncher look dead). Annotated the existing 'E2E must wait'
lesson as enforced rather than duplicating it.

docs/ISSUES.md: new, with stable IDs for the r/latin feedback and
CodeRabbit leftovers so they stop being re-derived from the thread.
7 open, 2 fixed.

NEXT.md: handoff, with the ruled-out dead ends spelled out.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: hellpanderrr

wiktionary_pron/CLAUDE.md

@@ -134,10 +134,17 @@ Each of these cost real debugging time in past sessions. Check this list before
 - **Never shim `document` in `scripts/tests/setup.cjs`** — wasmoon's Emscripten glue uses `typeof document` for environment detection and dies with "Invalid URL" under Node. Shim it per-test-file only in suites that don't load wasmoon (see `unit/lexicon_decode.test.js`).
 - **Lexicon test words must be letters only.** Every lookup path strips `[^\p{Letter}\p{Mark}-]` — synthetic words like `word000042` contain digits, get cleaned to `word`, and silently never match. Cost a failed "language isolation" test until spotted.
 - **Macronizer results have empty `textContent`** — words render via `<span class="ipa" content="...">` painted with CSS `attr(content)`. Assert the `content` attribute, not text. (Same idiom in the transcriber's line mode.)
-- **E2E must wait for `#lang` to be enabled before any interaction** — `main.js` top-level-awaits the wasmoon engine; clicking earlier hits elements with no listeners attached. Symptom: clear/dark-mode/persistence tests fail with stale values.
+- **E2E must wait for `#lang` to be enabled before any interaction** — `main.js` top-level-awaits the wasmoon engine; clicking earlier hits elements with no listeners attached. Symptom: clear/dark-mode/persistence tests fail with stale values. 2026-08-05: recurred on **macronizer.html**, where the gate is `#macronize_btn` being enabled — the dark-mode test clicked straight after `goto()` and was intermittently flaky (60s timeout, passed on retry). ✅ enforced by the readiness wait in `e2e/macronizer.spec.js`.
 - **Browser Latin ≠ Node Latin**: the browser flow macronizes input first (provinciarum → prōvinciārum), so e2e IPA expectations carry length marks that the Node suite's don't.
 - **RU/UK stress-transfer skips multi-form dictionary entries by design** — test it with a single-form word (голова → голова́), not вода (record "во́да, вода́" has a comma → skipped).
 
+**Macronizer / Morpheus traps** (2026-08-05)
+- **Morpheus returns `"accented_stem,lemma"`** — e.g. `currito_,curro`. The comma must be stripped from the *accented* form (Python does `accented.split(",")[0]`, `postags.py:434-436`). Keeping it corrupts `accentedUnderscore`, breaks DP alignment, and silently mis-macronizes every out-of-wordlist word. Fixed in `MorpheusAnalyzer.parseAnalysisLine`.
+- **`WordlistEngine.addEntry` must invalidate `entriesCache`** — `ensureAnalyzed` caches the *empty* lookup for a missing word, then Morpheus writes the row; without invalidation `getAccents` re-reads the stale empty cache and falls through to the `tag_to_endings` guess. This is why `currito` came out `currītō` on first visit.
+- **Don't byte-grep `cruncher.data` for filenames** — the preload manifest lives in **`cruncher.js`** (`{"files":[...]}` with offsets), not in the `.data` blob. Grepping the blob returns 0 hits for files that *are* packed, which sent a whole session down a false "missing stemlib" trail. Verify by reading the manifest and `cmp`-ing byte ranges.
+- **The cruncher defaults to GREEK.** `morpheus_init()` only sets `MORPHLIB`; you must call `morpheus_set_language(32768)` (LATIN) before `morpheus_analyze`, or it looks in `stemlib/Greek/` and returns 0 analyses for *every* word — including trivial ones like `aqua`. Native equivalent: the `-L` flag. A probe that skips this "proves" Morpheus is broken when it isn't.
+- **Don't run `build-morpheus-wasm.sh` against a read-write mount of the repo** — it does `rm -rf stemlib/Greek` and will delete 711 tracked files from your working tree. Mount read-only or copy first.
+
 **Code traps**
 - **Regex char classes with `-` between Unicode literals form ranges**: `[^\p{L}\p{M}'’-‿]` parsed `’-‿` as U+2019–U+203F and stripped ASCII hyphens from every word. Put `-` last in the class. (Was a live bug in `sanitize()` for years.)
 - **`#header > a > i` selects the HOME link's icon** — the dark-mode toggle on macronizer.html restyled the wrong button for this reason. Use `#dark_mode i`.

wiktionary_pron/NEXT.md

@@ -0,0 +1,48 @@
+# Next
+
+_Updated 2026-08-05 — branch `main`_
+
+## State
+Fixed the macronizer parity bug reported on r/latin: `currito` → `curritō` and
+`diffregit` → `diffrēgit` now match Winge's original. Two interacting port bugs
+(M-001 Morpheus `,lemma` suffix never stripped; M-002 `addEntry` left a stale
+`entriesCache`). Verified first-visit *and* return-visit in a real browser;
+81 node tests + 3 macronizer e2e green on exit code. Site `main` = `1cb1771`
+(live on Pages), engine `macronizer-ui-support` = `797f779`. Both pushed, both
+working trees clean.
+
+## Open threads
+- **M-003, the obvious next one**: scansion feet disappear when a line number
+  trails the line. Display-only — start at `scannedFeet[lineIdx]` in
+  `macronizer.html` (~line 1150). Reproduce with Catullus 1 in hendecasyllable
+  mode; line 4 shows no foot.
+- **Reply to the r/latin thread.** Both macronization bugs are genuinely fixed
+  and were real bugs in the port — say so. See `docs/ISSUES.md` M-003..M-007
+  for the rest of that feedback.
+- Remaining findings have IDs in `docs/ISSUES.md` (7 open) — read it rather
+  than re-deriving from the thread.
+
+## Running / unfinished
+Nothing running. A local static server may still be on :8993
+(`curl -s -o/dev/null -w "%{http_code}" http://127.0.0.1:8993/wiktionary_pron/macronizer.html`);
+harmless, and `npm run test:e2e` starts its own.
+
+## Don't redo
+- **Morpheus is NOT broken.** A long stretch of last session chased "missing
+  Latin stemlib indices in `cruncher.data`". False. Two independent agents
+  refuted it and the shipped WASM analyzes `currito`/`diffregit`/`aqua`
+  correctly. The byte-grep that "proved" it was meaningless — filenames live in
+  `cruncher.js`'s manifest, not the `.data` blob.
+- **Always `morpheus_set_language(32768)` before analyzing.** Skipping it makes
+  the cruncher read `stemlib/Greek/` and return 0 for every word, which looks
+  exactly like a broken build.
+- **`macrons.txt`, `tag_to_endings`, the RFTagger model and `toAscii` are all
+  byte-identical to upstream Alatius.** Already diffed. Not the problem.
+- **The `macronizer-py-compare` Docker image is not a usable reference** — its
+  cruncher defaults to Greek and the build strips `stemlib/Greek`, so it fails
+  on every word and "agrees" with any wrong output.
+- **The scansion of `iam tum, cum ausus es unus Italorum` is not a parity bug.**
+  Prose and scan modes give identical output in both engines.
+- **Never run `build-morpheus-wasm.sh` against a read-write mount of the engine
+  repo** — its `rm -rf stemlib/Greek` deleted 711 tracked files last session
+  (restored).

wiktionary_pron/docs/ISSUES.md

@@ -0,0 +1,64 @@
+# Issues
+
+Findings that outlived the session in which they were discovered. IDs are
+stable and never renumbered; fixed rows stay, with `Status: FIXED` and the
+evidence that closed them.
+
+Totals: 7 open, 2 fixed (9 total).
+
+---
+
+## M-001 — Morpheus accented form keeps the `,lemma` suffix
+**Status: FIXED** (2026-08-05, engine `797f779` / site `1cb1771`)
+`parseAnalysisLine` stored `currito_,curro` as the accented form instead of
+`currito_`. Python reference strips it (`postags.py:434-436`). Corrupted DP
+alignment for every out-of-wordlist word on return visits.
+
+## M-002 — `addEntry` left a stale `entriesCache`
+**Status: FIXED** (2026-08-05, same commits)
+`ensureAnalyzed` cached the empty lookup for a missing word; the Morpheus row
+written immediately after was invisible to `getAccents`, which fell through to
+the `tag_to_endings` heuristic. Broke first visits: `currito` → `currītō`.
+
+## M-003 — Scansion feet vanish when a line number trails the line
+**Status: OPEN.** Reported on r/latin. Reproduced on Catullus 1: line 4
+renders no metrical notation while the macronized words stay correct.
+Cause is display-side — `scannedFeet[lineIdx]` in `macronizer.html` (~line
+1150) misaligns when a trailing line-number token shifts the line index.
+Engine macronization is *not* affected. Not a parity bug.
+
+## M-004 — Output is not editable
+**Status: OPEN.** Adoption blocker per r/latin feedback: users macronize to
+catch typos and normalize spellings (`-īs` → `-ēs`) and cannot correct the
+result. Winge's original site made the output `contenteditable` in May 2017
+plus per-vowel click-to-toggle. Substantial build: contenteditable +
+re-macronize + re-export + surviving the cycle/click handlers.
+
+## M-005 — Word popup shows no dictionary definition
+**Status: OPEN.** Users cannot tell *populus* (people) from *populus* (poplar),
+or the two *malus* lemmas, from the readings list alone. Add a dictionary link
+per reading — `accentedSources` already carries lemma+tag per row.
+
+## M-006 — Popup buries the useful section under debug detail
+**Status: OPEN.** r/latin feedback: "Possible readings" is the only part users
+want; the RFTagger/Morpheus detail reads as debug output. Move readings to the
+top, collapse the rest behind `<details>`. Also the heading is misleading — it
+lists *distinct macronizations*, not all morphological readings (readings that
+differ only in a short vowel collapse into one row).
+
+## M-007 — Individual words/lines cannot be selected or copied
+**Status: OPEN.** Only the bulk "copy" button works. Structural: words render
+via `<span class="ipa" content="…">` painted with CSS `attr(content)`, so the
+selectable text is empty, and the popup/cycle handlers own the click.
+
+## M-008 — `rftagger.js` ships an assertions (debug) build
+**Status: OPEN.** CodeRabbit finding on PR #7. `assert()` bodies,
+`checkStackCookie`, `runtimeDebug` and the missing/unexported symbol tables are
+all present, unlike the release-style `cruncher.js` beside it. Inflates the
+one-time download and keeps assertions on hot paths. Fix belongs in the engine
+repo's build, then re-sync `dist/`.
+
+## M-009 — Deployed WASM assets are untracked
+**Status: OPEN.** `macronizer/wasm/cruncher.*` is untracked (not ignored), so
+the bytes actually served have no verifiable provenance. Commit them, or record
+the build SHA that produced them.

wiktionary_pron/e2e/macronizer.spec.js

@@ -41,6 +41,12 @@ test.describe("macronizer", () => {
     // Regression: the toggle used to target '#header > a > i', which is the
     // HOME link's icon — the sun appeared on the wrong side of the header.
     await page.goto(PAGE);
+    // The toggle's listener is attached by the module script that top-level-awaits
+    // __wasmReady, so clicking straight after goto() is a race: the click lands on
+    // a button with no handler and body never gets the class. Wait for readiness.
+    await expect(page.locator("#macronize_btn")).toBeEnabled({
+      timeout: 240_000,
+    });
     await page.click("#dark_mode");
     await expect(page.locator("body")).toHaveClass(/dark_mode/);
     await expect(page.locator("#dark_mode i")).toHaveClass(/icon-sun/);