18 - the parse keeps each node's readable spelling, so the directive ask spells it once #105

Merged
lilleman merged 5 commits from directive-spelling-subtree into main 2026-09-19 13:50:09 +02:00
Owner

The commonMarkSpelling ask emitted the whole subtree below every directive block, so a node at depth d was spelled d times. A memo the parse owns keeps each node's readable spelling, rebasing the kept headroom by the depth difference. Behaviour is unchanged; the two depth-boundary tests pin the rebase.

The `commonMarkSpelling` ask emitted the whole subtree below every directive block, so a node at depth d was spelled d times. A memo the parse owns keeps each node's readable spelling, rebasing the kept headroom by the depth difference. Behaviour is unchanged; the two depth-boundary tests pin the rebase.
lilleman added 2 commits 2026-09-19 13:19:19 +02:00
lilleman marked the pull request as work in progress 2026-09-19 13:19:22 +02:00
lilleman added 1 commit 2026-09-19 13:38:00 +02:00
A read below the depth that filled the memo re-spells, so the depth guards still run
CI / gate (push) Successful in 28s
CI / publish (push) Has been skipped
d6756ea5e6
lilleman added 1 commit 2026-09-19 13:43:37 +02:00
lilleman added 1 commit 2026-09-19 13:48:56 +02:00
The memo stops the per-level re-spelling; it does not promise one spelling
CI / gate (push) Successful in 27s
CI / publish (push) Successful in 4s
00e18d66b5
lilleman marked the pull request as ready for review 2026-09-19 13:50:00 +02:00
lilleman merged commit 00e18d66b5 into main 2026-09-19 13:50:09 +02:00
lilleman deleted branch directive-spelling-subtree 2026-09-19 13:50:09 +02:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: larvit/adf-codec#105