adf-codec/docs/comprehension-history.md

Comprehension history

2026-10-09T00:41:03Z, PR #19 at 859c952, against 9519606

Ruling: better

Seat	Vote
Mid A	better
Inherited architect	better

Mid A, decided by:

• src/markdown/portable/task-ids.ts:7 mintTaskIds: one walk, no carried set filled by the parser as a side effect
• src/markdown/parse/markdown-to-adf.ts:26 Reading: plain data, codeBlockNode no longer mutates carried
• src/markdown/portable/conventions.ts:8 flavourClaims: one table states what each reader claims; branches name the claim they test
• src/markdown/parse/markdown-to-adf.ts:161 quoteNode: the title is read in the reader's own flavour, no hop through inlineLeaves
• src/index.ts:5 exports: each call named by its flavour; "plain" no longer means two things

Inherited architect, decided by:

• src/markdown/parse/markdown-to-adf.ts:26 Reading: the hidden carried set and the retrying mintTaskIds are gone
• src/markdown/parse/markdown-to-adf.ts:161 quoteNode: parser and emitter no longer correct only together
• src/markdown/parse/markdown-to-adf.ts:113 readBlock: branches name the claim (claims.alerts, claims.taskMarkers)
• src/index.ts:5 exports: flavour names say which file to open
• src/markdown/portable/conventions.ts:3 flavourClaims: cost, all three flavours' rows live under portable/

2026-10-09T00:56:04Z, PR #19 at 5fc877e, against 9519606

Ruling: better

Seat	Vote
Mid A	better
Inherited architect	better

Mid A, decided by:

• src/markdown/portable/conventions.ts:8 flavourClaims: one table says what each flavour reads; every gate reads a named field
• src/markdown/portable/task-ids.ts:7 mintTaskIds: one pass, no carried set crossing modules
• src/index.ts:5 exports: each name says which markdown it means
• src/markdown/parse/markdown-to-adf.ts:161 quoteNode: no reach into the portable writer's reduction
• src/markdown/portable/conventions.ts:20 ReadClaims: cost, directiveNode leans on a type in another file

Inherited architect, decided by:

• src/markdown/portable/task-ids.ts:6 mintTaskIds: the retry loop and the parser-filled set are gone
• src/markdown/parse/markdown-to-adf.ts:26 Reading: no hidden state; branches name the feature they gate
• src/markdown/portable/conventions.ts:8 flavourClaims: cost, the table for all three flavours lives under portable/
• src/markdown/emit/line-escaping.ts:92 escapeClaims: directive escaping under the flag the parser checks
• src/markdown/emit/adf-to-markdown.ts:146 spellPortableBlock: no hard-coded lossless title

2026-10-09T01:43:11Z, PR #21 at c45320b, against 9a242c7

Ruling: better

Seat	Vote
Mid A	better
Inherited architect	better

Mid A, decided by:

• src/markdown/parse/markdown-to-adf.ts:23 MarkerParagraph: the block kind says a marker's paragraph holds no lone image, where readMarked scanned its output afterwards
• src/markdown/emit/image.ts:7 tryImage: builds what the reader would build and compares, where imageShape restated the shape by hand
• src/markdown/parse/inline-content.ts:25 InlineContent: one rule replaces five image refusal sites over two files
• src/markdown/external-image.ts:4 centeredImage: one builder for the shape three literals held
• src/markdown/parse/inline-content.ts:550 pieceNodes: costs more — its image arm relies on assemble having linked every top-level image

Inherited architect, decided by:

• src/markdown/external-image.ts:4 centeredImage: the image's ADF shape is defined once for reader, emitter and portable reducer
• src/markdown/parse/markdown-to-adf.ts:23 MarkerParagraph: replaces readMarked's after-the-fact search and its caller-computed flag
• src/result.ts:1 ConvertErrorCode: unmappable-image and its refusal paths are gone
• src/markdown/parse/inline-content.ts:43 Image: costs more — four functions must agree on the image's link
• src/markdown/parse/inline-content.ts:26 InlineContent: costs more — no longer image xor nodes

2026-10-09T08:38:28Z, PR #21 at 1098b5e, against 9a242c7

Ruling: better

Seat	Vote
Mid A	better
Inherited architect	better

Mid A, decided by:

• src/markdown/external-image.ts:4 centeredImage: one builder where three places built the image node by hand
• src/markdown/emit/image.ts:7 tryImage: compares against what the reader builds, where imageShape restated the shape
• src/markdown/parse/markdown-to-adf.ts:26 TaskParagraph: a named block kind replaces readMarked's after-the-fact index search
• src/markdown/parse/inline-content.ts:302 assemble: costs more — which of three readings an image takes hangs on a one-line condition and the inLink flag
• src/markdown/parse/inline-content.ts:26 InlineContent: costs more — no longer image xor nodes

Inherited architect, decided by:

• src/markdown/external-image.ts:4 centeredImage: the image shape is defined once for reader and both writers
• src/markdown/emit/image.ts:7 tryImage: the emitter cannot drift from the reader
• src/markdown/parse/markdown-to-adf.ts:136 readBlock: the task paragraph is routed where the block is classified
• src/markdown/parse/inline-content.ts:302 assemble: costs more — the most expensive line to decode
• src/markdown/parse/markdown-to-adf.ts:154 paragraphsOf: costs more — the name does not say it splits the marker line

2026-10-09T08:46:45Z, PR #21 at e7029c1, against 9a242c7

Ruling: better

Seat	Vote
Mid A	better
Inherited architect	better

Mid A, decided by:

• src/markdown/emit/image.ts:7 tryImage: builds what the reader would build and compares, where imageShape restated the shape
• src/markdown/external-image.ts:4 centeredImage: the image shape lives in one named file
• src/markdown/parse/markdown-to-adf.ts:26 TaskParagraph: a typed block kind replaces readMarked's after-the-fact search
• src/markdown/parse/inline-content.ts:302 assemble: costs more — one dense ternary over inLink
• src/markdown/parse/inline-content.ts:26 InlineContent: costs more — no longer a discriminated union

Inherited architect, decided by:

• src/markdown/external-image.ts:4 centeredImage: one builder where three had to stay in step
• src/markdown/parse/markdown-to-adf.ts:136 readBlock: the task paragraph is routed by a named block kind
• src/result.ts:1 ConvertErrorCode: one rule no longer enforced at five sites
• src/markdown/parse/inline-content.ts:26 InlineContent: costs more — contentNode drops image silently
• src/markdown/parse/inline-content.ts:302 assemble: costs more — three routes from an image piece to text

2026-10-09T09:00:13Z, PR #22 at fe5c731, against 78655bc

Ruling: same

Seat	Vote
Mid A	same
Inherited architect	same

2026-10-09T09:52:41Z, PR #22 at 6f1a16b, against 78655bc

Ruling: worse

Seat	Vote
Mid A	worse
Inherited architect	worse

Mid A, decided by:

• src/markdown/emit/inline-line.ts:106 takeFallback: a range may part joined breaks or carry, decided by state an earlier pass built
• src/markdown/emit/inline-line.ts:316 emitMarkedRun: a carry that signals parting to takeFallback two levels up
• src/markdown/emit/inline-line.ts:233 joinsRun: a lookahead whose shortcut holds only by inlineRuns' call order
• src/markdown/emit/inline-line.ts:58 portableLineFallback: a second retry loop sharing the parting
• src/adf/inline-nodes.ts:5 InlineNodeModel.marks: reads as "has marks", not "takes marks"
• src/markdown/parse/inline-content.ts:328 linkTo: the comment hides that the breaks stay after the destination

Inherited architect, decided by:

• src/markdown/emit/inline-line.ts:331 emitMarkedRun: the newline check returns a carry that is a parting signal, one fact over three functions
• src/markdown/emit/inline-line.ts:106 takeFallback: two growing sets whose order matters
• src/markdown/emit/inline-line.ts:58 portableLineFallback: a second retry loop with its own exit rule
• src/markdown/emit/inline-line.ts:233 joinsRun: lookahead, shortcut and three conditions per break; inlineRuns takes the whole context
• src/adf/inline-nodes.ts:5 InlineNodeModel.marks: the name hides "takes marks"
• src/markdown/parse/inline-content.ts:328 linkTo: the comment no longer says an empty destination reads as nothing

2026-10-09T09:57:25Z, PR #22 at 45b5446, against 78655bc

Ruling: worse

Seat	Vote
Mid A	worse
Inherited architect	worse

Mid A, decided by:

• src/markdown/emit/inline-line.ts:100 takeFallback: the first fallback is discarded to stop joining, known only from a comment
• src/markdown/emit/inline-line.ts:200 inlineRuns: an indexed loop moving its own counter by joinedBreaks' result
• src/markdown/emit/inline-line.ts:226 joinedBreaks: six positional parameters and two jobs

Inherited architect, decided by:

• src/markdown/emit/inline-line.ts:101 takeFallback: a hidden first step that takes no fallback
• src/markdown/emit/inline-line.ts:200 inlineRuns: runs built by a forward scan the reader steps through by hand
• src/markdown/emit/inline-line.ts:56 portableLineFallback: checks an unjoined line while the final emit starts joined
• src/markdown/emit/inline-line.ts:40 LineFallbacks.joinsBreaks: a fallbacks field whose fallback value is false

2026-10-09T10:02:23Z, PR #22 at f195ebd, against 78655bc

Ruling: same

Seat	Vote
Mid A	same
Inherited architect	same

2026-10-09T10:07:15Z, PR #22 at 37629e4, against 78655bc

Ruling: same

Seat	Vote
Mid A	same
Inherited architect	same

2026-10-09T10:19:03Z, PR #22 at 239218f, against 78655bc

Ruling: same

Seat	Vote
Mid A	same
Inherited architect	same

2026-10-09T12:13:22Z, PR #24 at d109199, against 78655bc

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	same	same	better	better
Inherited architect	same	same	same	worse	same
Maintainability senior	same	better	worse	same	same
Junior A	better	better	same	same	better

Mid A, decided by:

• Navigation, Self-sufficiency: src/markdown/portable/task-list.ts:5 splitTaskList: the split rule is one named unit both directions import, where it sat in bulletNode and reduceTaskList/nestIn
• Self-sufficiency, Navigation: src/adf/inline-nodes.ts:12 inlineNodes: "a hard break takes no marks" is the field takesMarks: false, read by parser and emitter
• Shape: src/markdown/portable/adf-to-portable-markdown.ts:289 reduceTaskList: the loop lost its nested accumulator and nestIn; line 303 packs the split and a merge into one expression
• Shape, Locality: src/markdown/portable/task-list.ts:8 splitTaskList: a standing flag across iterations and stands overwritten mid-iteration; heldTask says little
• Locality: src/markdown/parse/markdown-to-adf.ts:86 readBlocks: bulletNode returns a schema-breaking task list that its caller splits, and neither says so
• Self-sufficiency: src/markdown/parse/inline-content.ts:328 linkTo: the comment covers the hard-break case

Inherited architect, decided by:

• Self-sufficiency: src/markdown/portable/task-list.ts:5 splitTaskList: the comment states the schema constraint, not the rule; the loop must be traced by hand
• Locality: src/markdown/parse/markdown-to-adf.ts:201 bulletNode: builds a schema-breaking intermediate that only readBlocks repairs under an unexplained guard
• Locality: src/markdown/portable/adf-to-portable-markdown.ts:311 reduceTask: dropping blankedCode rests on a guarantee neither unit states
• Locality: src/markdown/portable/adf-to-portable-markdown.ts:317 nestIn: two hand-mirrored halves of one rule became one function
• Self-sufficiency, Locality: src/adf/inline-nodes.ts:12 inlineNodes: takesMarks: false makes the hard-break rule checkable at its source
• Self-sufficiency: src/markdown/portable/adf-to-portable-markdown.ts:288 reduceTaskList: the comment no longer prepares the reader for line 303
• Shape: src/markdown/portable/task-list.ts:12 splitTaskList: standing/stands reuse a word reduceStanding uses for another meaning

Maintainability senior, decided by:

• Locality: src/markdown/portable/task-list.ts:5 splitTaskList: the stay-or-stand rule lives once, where the reader's bulletNode and the writer's nestIn mirrored it
• Locality: src/markdown/parse/markdown-to-adf.ts:86 readBlocks: bulletNode returns a shape the caller quietly replaces, behind an unexplained block.kind guard
• Shape: src/markdown/portable/task-list.ts:9 splitTaskList: two cross-iteration accumulators, a negated compound ternary, near-synonym names, and heldTask hides the taskItem conversion
• Shape: src/markdown/portable/adf-to-portable-markdown.ts:303 reduceTaskList: one 180-character line maps the split and re-merges
• Shape: AGENTS.md:28 decision index: a title grew into a two-clause sentence

Junior A, decided by:

• Locality, Navigation: src/markdown/portable/task-list.ts:5 splitTaskList: one function decides for reader and writer, where each had its own rule
• Locality: src/markdown/parse/markdown-to-adf.ts:86 readBlocks: bulletNode returns an unfinished shape only the caller completes
• Locality: src/adf/inline-nodes.ts:3 InlineNodeModel.takesMarks: the hard break's rule lives in one row both directions read
• Shape: src/markdown/portable/task-list.ts:5 splitTaskList: a carried standing flag, a reassigned stands, and heldTask's name hides the conversion
• Navigation: src/markdown/parse/markdown-to-adf.ts:22 imports: both converters lead to the same named file

2026-10-09T12:35:05Z, PR #24 at e487fa4, against 78655bc

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	same	same	better
Inherited architect	better	better	same	better	better

Mid A, decided by:

• Locality, Overall: src/markdown/portable/adf-to-portable-markdown.ts:289 reduceTaskList: a plain collect loop handing the list to splitTasks, where the base carried a nested accumulator and nestIn mutated tasks
• Locality: src/markdown/portable/task-list.ts:6 splitTaskList: reader and writer share one rule, where bulletNode and nestIn each encoded it
• Locality, Shape: src/markdown/parse/markdown-to-adf.ts:84 readBlocks: the portable task list is caught before readBlock, which depends on that order
• Shape: src/markdown/portable/task-list.ts:33 itemParts: paragraphs and nested are indices named like collections, and the demotion to taskItem is unnamed
• Navigation, Shape: src/adf/block-nodes.ts:92 isTaskItem: one findable predicate, against one more hop from a parse symptom

Inherited architect, decided by:

• Locality, Navigation, Overall: src/markdown/portable/task-list.ts:6 splitTaskList: one unit decides where a task item's blocks go, for reader and writer
• Locality, Shape: src/markdown/portable/adf-to-portable-markdown.ts:289 reduceTaskList: one pass then one call, no state carried across iterations
• Self-sufficiency, Locality: src/adf/inline-nodes.ts:5 takesMarks: the hard-break rule is a field both directions read
• Shape: src/markdown/portable/task-list.ts:33 itemParts: an index over content used over run, and nested running into the siblings
• Shape, Locality: src/markdown/parse/markdown-to-adf.ts:84 readBlocks: a third special case ahead of the dispatch
• Shape: src/adf/block-nodes.ts:92 isTaskItem: one predicate replaces two copies
• Self-sufficiency: src/markdown/portable/task-list.ts:5 comment: states the schema limit and what follows from it

2026-10-09T13:03:22Z, PR #25 at 2b211e9, against 875c0e5

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	better	same	better
Inherited architect	same	better	better	same	better

Mid A, decided by:

• Locality, Shape, Overall: src/markdown/portable/adf-to-portable-markdown.ts:118 portableSequence: a plain splice with index -= 1, where the base merged neighbours after every splice through spliced, mergedLists, joinedLists and splitTasks
• Locality, Navigation: src/markdown/parse/markdown-to-adf.ts:98 listBreakFault: gone, with the base's look at blocks[index - 1] and blocks[index + 1] and the separator spelled in four places
• Navigation, Shape: src/markdown/parse/blocks.ts:232 continuesList: one named predicate on kind and delimiter that itemStart and openedList share
• Locality: src/markdown/emit/adf-to-markdown.ts:73 joinBlocks: a toggled alternated flag decides each list's text from the one before it, and isOrdered counts a task list as a bullet list unsaid
• Shape: src/markdown/emit/adf-to-markdown.ts:334 spelledList: tryList builds both spellings then falls back once, and tryTaskList shares the helper

Inherited architect, decided by:

• Locality, Shape, Overall: src/markdown/portable/adf-to-portable-markdown.ts:118 portableSequence: a filter and a loop ending in index -= 1, where the base kept the splice window, the merge rules and the renumbering-as-text rule together
• Locality, Shape: src/markdown/parse/markdown-to-adf.ts:79 readBlocks: reads one block at a time, where the base judged a separator by its siblings
• Locality: src/markdown/emit/adf-to-markdown.ts:73 joinBlocks: a toggle carried across iterations sends a third adjacent list back to -, seen only by tracing
• Shape: src/markdown/emit/adf-to-markdown.ts:334 spelledList: the both-or-neither rule is one uncommented expression
• Self-sufficiency: src/markdown/emit/adf-to-markdown.ts:369 tryListItemLines: inner.startsWith('*') depends on the alternate bullet being *, and the comment needs the inline escaping worked out
• Self-sufficiency: src/markdown/parse/blocks.ts:232 continuesList: states the rule the base sent the reader to the spec for

2026-10-09T13:37:16Z, PR #25 at ca3d703, against 875c0e5

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	better	better	better
Inherited architect	better	better	better	same	better

Mid A, decided by:

• Locality, Shape, Overall: src/markdown/portable/adf-to-portable-markdown.ts:118 portableSequence: a worklist that keeps a block that spells or pushes its items back, where the base spliced, merged neighbours and stepped the index back by hand
• Navigation, Locality: src/markdown/parse/markdown-to-adf.ts:70 readBlocks: handles each block alone, where the base judged a separator by its neighbours across five places
• Shape, Self-sufficiency: src/markdown/parse/blocks.ts:232 continuesList: one named predicate on kind and delimiter, its rule stated in place
• Self-sufficiency, Locality: src/markdown/emit/adf-to-markdown.ts:27 SpelledBlock: its comment states why alternateText exists, though joinBlocks now carries a toggle across siblings
• Shape: src/markdown/emit/adf-to-markdown.ts:334 spelledList: line building in small units, the fallback once after the loop, shared by tryTaskList
• Self-sufficiency: src/markdown/emit/adf-to-markdown.ts:369 tryListItemLines: the * branch's comment needs a reread

Inherited architect, decided by:

• Locality, Shape, Overall: src/markdown/portable/adf-to-portable-markdown.ts:118 portableSequence: a work stack replacing six helpers and the base's cursor rewind
• Locality, Navigation: src/markdown/parse/markdown-to-adf.ts:70 readBlocks: each block reads alone, and two merged lists lead to one emitter and one parser unit
• Locality, Navigation: src/markdown/parse/blocks.ts:232 continuesList: a list continues exactly where CommonMark says, checkable without the spec
• Locality: src/markdown/emit/adf-to-markdown.ts:72 joinBlocks: a block's spelling depends on the run before it, where the base's separation was a pure function of a pair
• Shape: src/markdown/emit/adf-to-markdown.ts:334 spelledList: one exit to the directive form, though bullets pass () => '' as the number
• Self-sufficiency: src/markdown/emit/adf-to-markdown.ts:369 tryListItemLines: the * branch needs the spec open, cancelling the spec pointers removed

2026-10-09T14:50:52Z, PR #27 at 2378782, against a9fedb3

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	better	better	better
Inherited architect	better	better	better	same	better

Mid A, decided by:

• Locality, Self-sufficiency: src/markdown/inline-tokens.ts:20 readInlineToken: the parser and the escaper ask one function what the reader takes at a character, where each held its own copy of the rules
• Locality, Shape: src/markdown/emit/line-escaping.ts:102 escapeClaims: the order-dependent afterEscape and pairsEquals flags are gone; runs go whole to escapeRun and fixed inputs ride one EscapeWalk record
• Shape, Navigation: src/markdown/parse/inline-content.ts:110 scanInline: a two-level character dispatch and eight helpers became one switch over token kinds
• Shape: src/markdown/emit/line-escaping.ts:158 claimsCharacter: switches on token.kind in place of a chain of character tests

Inherited architect, decided by:

• Navigation, Shape: src/markdown/inline-tokens.ts:20 readInlineToken: one tokenizer answers what the reader sees at an index, where two parallel implementations had to agree
• Locality: src/markdown/emit/line-escaping.ts:112 escapeClaims: each step reads on its own, no flag carried from the previous character
• Shape: src/markdown/emit/line-escaping.ts:158 claimsCharacter: switches on the token kind and takes the shared record; the heading closer is worked out once in closingHashes
• Shape, Navigation: src/markdown/parse/inline-content.ts:121 readToken: one exhaustive switch over InlineToken['kind'] replaces a two-level dispatch
• Locality: src/markdown/emit/line-escaping.ts:173 claimsCharacter: default: return true claims every kind it does not name, so a new kind is escaped without its author deciding

2026-10-09T17:14:17Z, PR #27 at 65d2a26, against a9fedb3

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	better	same	better
Inherited architect	better	better	better	same	better

Mid A, decided by:

• Navigation, Locality: src/markdown/inline-tokens.ts:21 readInlineToken: the one place that says what a character opens, which the reader and the escaper both dispatch on, where each held a copy that could drift
• Locality, Shape: src/markdown/emit/line-escaping.ts:156 claimsCharacter: a flat switch on the token kind over a readonly EscapeWalk, where one character's escape hung on flags the previous iteration set
• Shape: src/markdown/parse/inline-content.ts:121 readToken: one flat switch, recognition in the tokenizer and action here, where a two-level character switch fanned out to eight helpers
• Locality: src/markdown/directive-syntax.ts:14 ClaimedPrefix: the directive read can no longer answer that no prefix stands there
• Shape: src/markdown/emit/line-escaping.ts:147 claimsRunRest: re-reading each suffix of a run is a step the name does not say

Inherited architect, decided by:

• Navigation, Locality: src/markdown/inline-tokens.ts:21 readInlineToken: the 3am symptom of an unescaped character leads to one file both directions ask
• Locality, Shape: src/markdown/emit/line-escaping.ts:156 claimsCharacter: an exhaustive switch read one case at a time, where the decision needed replaying the loop for pairsEquals
• Shape, Locality: src/markdown/parse/inline-content.ts:121 readToken: one pure tokenizer and one switch that applies tokens
• Shape: src/markdown/directive-syntax.ts:14 ClaimedPrefix: the directive read needs proof of its prefix, and the misnamed opensConstruct is gone
• Locality: src/markdown/emit/line-escaping.ts:112 escapeClaims: the escaper steps one character at a time while the reader steps by token, which only the == comment explains

2026-10-10T09:16:56Z, PR #30 at 6182ba6, against c11aa91

Ruling: worse

Fell: Locality. Under the floor of 7.0: Locality 5.63, so the same ruling counts as worse.

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	same	same	same
Inherited architect	better	worse	same	same	same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Junior A	7	5.5	6.5	5.5	6
Mid A	7	5.5	6.5	5.5	6
Maintainability senior	7	6	6.5	6	6
Inherited architect	6.5	5.5	6	6	6
Mean	6.88	5.63	6.38	5.75	6.00

Mid A, decided by:

• Locality: src/markdown/parse/blocks.ts:79 Walk.endedByDefinitions: written in closeLeaf at :469 and read in openedList at :257, never cleared, correct only by identity with blocks.at(-1); closeLeaf takes on recording where definitions ended a list

Inherited architect, decided by:

• Locality: src/markdown/parse/blocks.ts:79 Walk.endedByDefinitions: a never-reset field closeLeaf writes and openedList reads, right only by identity and by openContainers closing before it opens; reordering those steps breaks the split silently
• Locality: src/markdown/parse/blocks.ts:432 readLineBlock: a second path that can find a paragraph emptied by definitions records nothing, which a reader must prove harmless by hand
• Navigation: src/markdown/parse/blocks.ts:252 openedList: the comment names the exception, so a reader of two lists merged or not lands on the right unit
• Navigation: README.md:242 the CommonMark exceptions bullet: each exception is named rather than counted by position
• Navigation: CHANGELOG.md:5 ## Unreleased: breaking entries come first, other changes last

2026-10-11T10:33:32Z, PR #32 at be23835, against c11aa91

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	same	worse	worse
Inherited architect	same	worse	worse	worse	worse

Mid A, decided by:

• Overall, Locality, Self-sufficiency: src/markdown/parse/markdown-to-adf.ts:229 splitPath: builds stand-in nodes and replays splitTaskList to place a refused block's path, checkable only by knowing how the split moves blocks
• Locality, Self-sufficiency: src/markdown/portable/task-list.ts:23 splitPlace: finds the stand-in by object identity, which holds only while taskOfParagraphs keeps the original children, and nothing there says so
• Locality: src/markdown/parse/blocks.ts:262 endList: closeLeaf fills Walk.endedLists and openedList reads it, where openedList looked only at the previous block

Inherited architect, decided by:

• Locality, Self-sufficiency, Overall: src/markdown/parse/markdown-to-adf.ts:229 splitPath: a fake task list replayed through splitTaskList and found again by identity; if taskOfParagraphs ever copies nodes the old wrong path returns silently
• Shape, Locality: src/markdown/parse/markdown-to-adf.ts:203 bulletBlocks: one readBlocks call became a per-block loop with a hand-built failure, a second depth check and a fan-out through taskItems, splitPath, childType and splitPlace; Refusal names a context bag
• Locality: src/markdown/parse/blocks.ts:78 Walk.endedLists: identity-keyed walk state that closeLeaf writes and openedList reads
• Shape: src/markdown/portable/adf-to-portable-markdown.ts:354 reduceMedia: the callback's held shadows the alt's, and the link goes mark to definition to mark

2026-10-11T10:33:35Z, PR #32 at be23835

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Junior A	7	6	6.5	6	6.5
Mid A	7	6	6.5	5.5	6
Maintainability senior	7	5.5	6.5	5.5	6
Inherited architect	6.5	5.5	6	6	6
Mean	6.88	5.75	6.38	5.75	6.13

Junior A, hardest first:

• Locality: src/markdown/parse/markdown-to-adf.ts:229 splitPath: replays splitTaskList's internals to name a path
• Self-sufficiency: src/markdown/emit/line-escaping.ts:189 escapeClosedRuns: the escaping rule lives in spec/flavour.md

Mid A, hardest first:

• Locality: src/markdown/emit/adf-to-markdown.ts:30 SpellingMemo: reference-keyed memo shared by parse and emit
• Self-sufficiency: src/markdown/parse/inline-content.ts:62 Scan: rules read with spec/flavour.md open alongside

Maintainability senior, hardest first:

• Locality: src/markdown/parse/markdown-to-adf.ts:257 directiveNode: the parser asks the emitter whether input was legal
• Self-sufficiency: src/markdown/emit/line-escaping.ts:189 escapeClosedRuns: needs spec/flavour.md Escaping open

Inherited architect, would restructure first:

• Navigation: src/markdown/portable/conventions.ts:23 flavourClaims: every flavour's switchboard sits in portable/
• Locality: src/markdown/emit/adf-to-markdown.ts:117 readableBlock: reader acceptance hangs on emitter verdicts

2026-10-11T11:29:00Z, PR #32 at 27c8f9b, against c11aa91

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	same	same	worse
Inherited architect	same	worse	same	same	worse

Mid A, decided by:

• Locality, Overall: src/markdown/parse/markdown-to-adf.ts:229 splitPath: rewrites a refused path by replaying splitTaskList on stand-in nodes, so a change to the split moves error paths quietly
• Locality: src/markdown/parse/blocks.ts:262 endList: closeLeaf writes Walk.endedLists and openedList reads it later

Inherited architect, decided by:

• Locality, Overall: src/markdown/parse/markdown-to-adf.ts:229 splitPath: four facts held at once, the mutated task.content, the path slice, the stand-ins and a second parse in childType
• Locality: src/markdown/portable/task-list.ts:23 splitPlace: finds the refused node by object identity, right only while taskOfParagraphs keeps its children

2026-10-11T11:29:02Z, PR #32 at 27c8f9b

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Junior A	7	5.5	6.5	5.5	6
Mid A	7	5.5	6.5	5.5	6
Maintainability senior	7	5.5	6.5	5.5	6
Inherited architect	6.5	6	6	6.5	6.5
Mean	6.88	5.63	6.38	5.75	6.13

Junior A, hardest first:

• Locality: src/markdown/parse/inline-content.ts:62 Scan: a mutable bag many functions change
• Self-sufficiency: src/markdown/emit/line-escaping.ts:189 escapeClosedRuns: the rule lives in spec/flavour.md

Mid A, hardest first:

• Locality: src/markdown/parse/markdown-to-adf.ts:257 directiveNode: the parser asks the emitter for verdicts
• Self-sufficiency: src/markdown/emit/line-escaping.ts:100 escapeClaims: needs the spec open alongside

Maintainability senior, hardest first:

• Locality: src/markdown/parse/inline-content.ts:62 Scan: flags that act at a distance between escaper, emitter and parser
• Self-sufficiency: src/markdown/emit/line-escaping.ts:189 escapeClosedRuns: no local statement of the escaping contract

Inherited architect, would restructure first:

• Navigation: src/markdown/portable/conventions.ts:23 flavourClaims: the table for all three flavours sits in portable/
• Shape: src/markdown/emit/adf-to-markdown.ts:131 spellReadableBlock: the flavour concern spread over three directories

2026-10-11T18:44:18Z, PR #35 at 0227c0b, against 4f00e60

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	same	same	same	better
Inherited architect	same	worse	worse	same	same
Maintainability senior	better	worse	same	same	same
Junior A	better	same	same	same	better

Mid A, decided by:

• Navigation, Overall: README.md:211 "The calls": each public call sits in one row with the codes it refuses, where the base split that knowledge across prose bullets and the feature list
• Navigation: README.md:384 "The errors": parsing, emitting and either direction have their own sub-headings, and the error shape is a bulleted list
• Navigation: src/result.ts:49 withinEngineLimits: the refusal for output too long is visible at both public entry points, its message in one place

Inherited architect, decided by:

• Locality, Shape: src/result.ts:49 withinEngineLimits: one try/catch wraps each writer's whole call and matches four exact engine messages; a concatenation deep in the emitter does not show it is caught at the top, a reworded engine message turns the refusal back into a throw, and engineLimitFaults borrows "fault" from ConvertFault while holding plain strings
• Shape: src/result.ts:54 withinEngineLimits: an engine overflow is reported as unsupported-node-shape at an empty path, though the node shapes are fine and the output is what is too big
• Locality: src/canonical-json.ts:30 writeCanonicalJson: one walker serves two callers through a stop-returning write callback, one more contract to hold when editing schedule

Maintainability senior, decided by:

• Locality: src/result.ts:49 withinEngineLimits: the guard sits only at the two public writers while writeMarkdown stays exported and unguarded and the readers rely on another mechanism; nothing states which paths can outgrow the engine's longest string
• Locality: src/result.ts:46 engineLimitFaults: errors match exact engine strings, and why a stack overflow stays a throw is not said
• Navigation: README.md:211 "The calls": one table maps each call to its refusal codes, where the base spread them over prose that was wrong for adfToPortableMarkdown
• Navigation: src/markdown/opaque-carry.ts:78 readCarriedJson: spellsCanonicalJson reads as plainly at the call site, with its reason beside it

Junior A, decided by:

• Navigation, Overall: README.md:193 "The calls": a table pairs each call with what it reads or writes and what it refuses, and a node-to-spelling table turns a symptom into a concept to grep for

2026-10-11T19:02:20Z, PR #35 at 90de6cb, against 4f00e60

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	same	same	same	better
Inherited architect	better	same	worse	better	better

Mid A, decided by:

• Navigation, Overall: README.md:211 "The calls": each call's row says what it reads or writes and what it refuses, and the errors split by direction, where the base spread the codes across a prose list and paragraphs
• Navigation: src/result.ts:49 withinEngineLimits: the engine-limit refusal lives in one wrapper, called once at each public writer

Inherited architect, decided by:

• Navigation, Self-sufficiency: README.md:211 "The calls": one row per call gives what it reads or writes and its refusal codes, where the base split them and left out unsupported-node-shape for the portable writer
• Self-sufficiency: README.md:241 "The lossless flavour": two tables map ADF types to their spellings, so the 3am case reads without the spec open
• Shape: src/result.ts:55 withinEngineLimits: a length failure is reported under unsupported-node-shape at an empty path, and the README cell explaining it holds four cases
• Shape: src/result.ts:45 engineLimitThrows: matching exact engine message text rots silently when an engine rewords, and JavaScriptCore's Out of memory blurs a heap shortage with length

2026-10-12T15:56:53Z, PR #36 at 584b214, against 7a27f6b

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	worse	worse	same	worse
Inherited architect	better	worse	worse	same	worse

Mid A, decided by:

• Locality, Shape, Overall: src/markdown/emit/adf-to-markdown.ts:61 walkBlocks: a running forecast depends on Writing.indent, which each container must pass correctly, tryList deliberately not indenting
• Shape, Locality: src/markdown/emit/adf-to-markdown.ts:398 spelledList: template strings became joinedWithin piped through nested boundedThen callbacks, and joinedItems repeats the counting loop
• Locality: src/markdown/emit/adf-to-markdown.ts:147 readableBlock: the memo is filled in commonMarkSpelling and emptied on read, so when an entry is present depends on call order across two functions
• Shape: src/markdown/directive-syntax.ts:114 spellAttributes: each spelling exists twice, as a string and as a generator of pieces
• Locality: src/markdown/portable/editor-normal.ts:26 toEditorNormal: editor normalization imports the markdown length bound
• Locality: src/markdown/portable/adf-to-portable-markdown.ts:126 portableSequence: each block carries its source path, rebuilt by hand for a replacement
• Navigation: src/markdown/length-bound.ts:1 withinEngineLimits: the bound, its refusal and the catch share one module named for them

Inherited architect, decided by:

• Locality, Shape: src/markdown/emit/adf-to-markdown.ts:61 walkBlocks: the forecast is a second length bound beside joinedWithin, whose never-overcount rule fixes how every container walks
• Locality: src/markdown/emit/adf-to-markdown.ts:377 tryList: whether items walk indented depends on what spelledOrDirectiveList may decide later
• Locality: src/markdown/parse/markdown-to-adf.ts:266 directiveNode: the emitter's indent leaks into the parser and the portable reduction
• Locality: src/markdown/emit/adf-to-markdown.ts:147 readableBlock: filling and emptying the memo split across two functions
• Shape: src/markdown/emit/adf-to-markdown.ts:194 quotedBlock: nested boundedThen reads as continuation passing where a template stood
• Shape, Locality: src/markdown/emit/pipe-table.ts:9 tryPipeTable: the sum-then-check rule appears in seven shapes, and a reader cannot tell which is the real bound
• Navigation, Shape: src/markdown/length-bound.ts:41 withinEngineLimits: a too-long symptom leads straight to the module named for it
• Shape: src/markdown/directive-syntax.ts:133 spellDirectiveOpener: each spelling is a string function and a pieces generator
• Shape: src/markdown/opaque-carry.ts:64 carriedJson: the lambda parameter spelled shadows the function's own

2026-10-12T16:40:52Z, PR #36 at acd22aa, against 7a27f6b

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	worse	worse	same	worse
Inherited architect	better	worse	worse	same	worse

Mid A, decided by:

• Locality, Shape: src/markdown/emit/adf-to-markdown.ts:374 spelledList: a builder's undefined now means either no list spelling or past the length bound, and every caller of joinBlocks must remember which it holds
• Shape: src/markdown/emit/adf-to-markdown.ts:383 listLines: split, map and join became generators yielding pieces into separatedBy and joinedWithin, so a reader rebuilds the string in their head
• Locality: src/markdown/emit/adf-to-markdown.ts:125 readableBlock: the memo is filled in commonMarkSpelling and deleted on read, a one-shot cache whose hits depend on which walk came first
• Navigation: src/markdown/length-bound.ts:41 withinEngineLimits: a too-long refusal leads to a module named for it

Inherited architect, decided by:

• Locality, Shape: src/markdown/emit/adf-to-markdown.ts:374 spelledList: undefined means two things a few lines apart, told apart only by the side of a nested boundedThen callback
• Locality: src/markdown/emit/adf-to-markdown.ts:125 readableBlock: reading the memo is a mutation, and a second read spells again
• Shape: src/markdown/pipe-table-syntax.ts:43 pipeRowPieces: one-line templates became generators whose output cannot be read off one line
• Locality, Shape: src/markdown/emit/block-directive-spelling.ts:27 spellBlockDirectiveOpener: the opener's layout is known here and in spellDirectiveOpener
• Shape: src/markdown/portable/adf-to-portable-markdown.ts:231 reduceCodeBlock: an ADF-to-ADF reducer carries a markdown length bound through a callback
• Navigation: src/markdown/length-bound.ts:41 withinEngineLimits: a too-long refusal leads straight to one small, well-named module

2026-10-12T19:45:45Z, PR #36 at 1f4551a, against 7a27f6b

Ruling: worse

Fell: Locality. Under the floor of 7.0: Locality 5.13, so the same ruling counts as worse.

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	worse	better	same	same
Inherited architect	better	worse	better	same	same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Junior A	6.5	5	6	5.5	5.5
Mid A	7	5	6	5	5.5
Maintainability senior	7	5.5	6.5	6	6
Inherited architect	6.5	5	5.5	6.5	5.5
Mean	6.75	5.13	6.00	5.75	5.63

Mid A, decided by:

• Locality: src/markdown/emit/adf-to-markdown.ts:114 commonMarkSpelling: the memo is filled here and read-and-deleted in readableBlock, a write-once, read-once protocol whose order is set in another module
• Locality: src/markdown/emit/adf-to-markdown.ts:52 writeMarkdown: one rule, output past longestMarkdown is refused, is checked in six places with different triggers
• Navigation, Shape: src/markdown/length-bound.ts:1 tooLong: a too-long refusal leads to one small module holding the bound, its message and the engine guard
• Shape: src/canonical-json.ts:23 canonicalJsonPieces: a generator replaces a boolean-returning callback, and both readers become plain loops
• Shape, Locality: src/markdown/portable/adf-to-portable-markdown.ts:128 portableSequence: a refused spelling is returned where it is produced, and each block carries its own path
• Shape: src/markdown/emit/inline-line.ts:235 emitLeaf: one content-or-text check, and the path is built only to refuse

Inherited architect, decided by:

• Locality: src/markdown/emit/adf-to-markdown.ts:114 commonMarkSpelling: filling and consuming the memo split across an exported predicate and a private reader, a consume-once protocol among three callers
• Locality: src/markdown/emit/adf-to-markdown.ts:52 writeMarkdown: one bound enforced at five sites in four files, and a new builder has nothing nearby saying whether it needs a check
• Navigation, Shape: src/markdown/length-bound.ts:1 withinEngineLimits: an output-too-long symptom leads straight to a module named for it
• Navigation: src/markdown/parse/markdown-to-adf.ts:47 readDocument: the parse takes the same guard as both writers
• Shape: src/canonical-json.ts:23 canonicalJsonPieces: plain loops replace a boolean-returning callback
• Shape: src/markdown/emit/inline-line.ts:235 emitLeaf: both leaf emitters return a plain emission, the path built only on refusal

Junior A, hardest first:

• Locality: src/markdown/emit/adf-to-markdown.ts:114 commonMarkSpelling: a mutable memo shared by three walks, deleted on read and rebased by depth

Mid A, hardest first:

• Locality: src/markdown/emit/adf-to-markdown.ts:125 readableBlock: shared mutable state with no single owner, parse and emit coupled both ways

Maintainability senior, hardest first:

• Locality: src/markdown/emit/adf-to-markdown.ts:114 commonMarkSpelling: the parser and the portable reducer call into the emitter and share a reference-keyed, delete-on-read memo

Inherited architect, would restructure first:

• Locality: src/markdown/emit/adf-to-markdown.ts:126 readableBlock: the memo is mutated and deleted across parse, emit and portable, and the length bound is applied at five sites

2026-10-13T14:48:38Z, PR #37 at ecd695a, against 7a27f6b

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	worse	better	worse	worse
Inherited architect	better	worse	same	worse	worse

Mid A, decided by:

• Locality, Self-sufficiency: src/markdown/emit/adf-to-markdown.ts:116 commonMarkSpelling / readableBlock / takeKeptSpelling: only commonMarkSpelling fills the memo and a read empties it, so whether an entry exists depends on which caller ran first; the reason lives only behind a pointer to docs/decisions.md
• Locality: src/markdown/emit/line-escaping.ts:229 escapeClosedRuns: both steps write one shared Uint8Array, and correctness rests on the scan order a second comment line must defend
• Navigation, Shape: src/markdown/length-bound.ts:30 withinEngineLimits: the engine-limit catch moves out of result.ts into one well-named file, with string-length and engine limits apart and engine-limit a code of its own
• Locality: src/markdown/emit/adf-to-markdown.ts:51 writeMarkdown / walkBlocks / commonMarkSpelling: the length bound is enforced in layers a reader changing it must hold together
• Shape: src/markdown/parse/markdown-to-adf.ts:266 directiveNode: undefined is wrapped into a Result to pass the engine guard, then unpicked
• Shape: src/markdown/emit/line-escaping.ts:35 Escapings: the type is a lookup function, its comment describes the Uint8Array behind it; backslashed adds 4096-piece chunking
• Shape, Locality: src/canonical-json.ts:23 canonicalJsonPieces: a generator replaces a boolean-continue callback
• Shape: src/markdown/emit/inline-line.ts:235 emitLeaf: the content-or-text refusal moves into one place
• Locality: src/markdown/portable/adf-to-portable-markdown.ts:128 portableSequence: a spelling refusal propagates where it was swallowed, at the cost of Sourced path plumbing

Inherited architect, decided by:

• Locality, Self-sufficiency: src/markdown/emit/adf-to-markdown.ts:139 takeKeptSpelling: reading deletes the entry and only commonMarkSpelling fills it, so readableBlock is right only if another file's caller asked first, once per node; explained only by a pointer to docs/decisions.md
• Locality, Shape: src/markdown/parse/markdown-to-adf.ts:266 directiveNode: a second withinEngineLimits inside readDocument's, fed ?? success(undefined), so null and undefined carry meaning the reader must remember
• Locality: src/markdown/emit/adf-to-markdown.ts:58 walkBlocks: the length bound is enforced at about six sites across three files
• Navigation: src/markdown/length-bound.ts:30 withinEngineLimits: engine-limit handling sits in a file named for it, and the engine-limit code parts memory from the length bound
• Shape, Locality: src/markdown/emit/line-escaping.ts:229 escapeClosedRuns: mutates its input in place and needs a two-line comment arguing it is safe; output joined in batches of 4096
• Locality, Shape: src/markdown/portable/adf-to-portable-markdown.ts:128 portableSequence: a refusal propagates with its source path, where it was swallowed
• Shape: src/markdown/emit/inline-line.ts:235 emitLeaf: one place refuses content and text, and a generator replaces the boolean-callback JSON writer

2026-10-13T15:35:32Z, PR #37 at 025ccc0, against 7a27f6b

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	worse	same	same	worse
Inherited architect	better	worse	same	worse	worse

Mid A, decided by:

• Locality: src/markdown/emit/adf-to-markdown.ts:58 walkBlocks: the length bound is enforced at six sites across four files, each comment giving only its own reason, where one wrapper covered the conversion
• Locality: src/markdown/emit/adf-to-markdown.ts:126 readableBlock: the memo is emptied on read and filled only when asked, so whether a spelling is cached depends on call order and caller
• Navigation: src/markdown/length-bound.ts:1 withinEngineLimits: a refused long document leads to a file named for the problem, holding the bound, both throw lists and both messages
• Locality: src/markdown/parse/markdown-to-adf.ts:266 directiveNode: success(undefined) and success(null) encode the ask's answer, decoded only by tracing commonMarkSpelling
• Shape: src/canonical-json.ts:23 canonicalJsonPieces: a generator replaces a boolean-continue callback
• Shape: src/markdown/emit/line-escaping.ts:38 PlacedSegment: segment placements drop the cursor arithmetic two functions repeated
• Shape: src/markdown/emit/line-escaping.ts:113 backslashed: byte flags, codes and 4096-piece joins read harder than a character loop
• Locality: src/markdown/portable/adf-to-portable-markdown.ts:128 portableSequence: a refused spelling propagates where it was silently swallowed

Inherited architect, decided by:

• Locality: src/markdown/emit/adf-to-markdown.ts:58 walkBlocks: one rule, no markdown past longestMarkdown, is checked at about seven places across four files
• Locality, Shape: src/markdown/emit/adf-to-markdown.ts:126 readableBlock: a read deletes the entry and a bare positional asked boolean decides what is stored, so results depend on call order
• Locality, Self-sufficiency: src/markdown/emit/line-escaping.ts:113 backslashed: flags, byte codes, a lookup closure and batched joins take more passes to read as putting a backslash before characters
• Self-sufficiency: src/markdown/emit/adf-to-markdown.ts:65 walkBlocks: the refusal's comment makes sense only once you know V8's limit applies on every engine, which lives in docs/decisions.md; tooLong carries path [] though the walk knows the path
• Shape: src/markdown/length-bound.ts:17 tooLong: the length refusal keeps unsupported-node-shape while engine-limit got a name of its own; directiveNode folds three outcomes into one Result to fit the wrapper
• Navigation: src/markdown/length-bound.ts:37 withinEngineLimits: the engine guard moves out of the Result module, and engine-limit leads from the symptom to it
• Shape, Locality: src/markdown/portable/adf-to-portable-markdown.ts:128 portableSequence: a refusal propagates with its block's path, where it was silently swallowed
• Shape: src/canonical-json.ts:23 canonicalJsonPieces: a generator, emitLeaf's one refusal and PlacedSegment each read simpler than before

2026-10-13T15:41:57Z, PR #37 at f1ea9fd, against 7a27f6b

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	worse	better	same	same
Inherited architect	better	worse	same	same	worse
Maintainability senior	better	worse	better	same	same
Junior A	better	worse	better	same	same

Mid A, decided by:

• Locality: src/markdown/parse/markdown-to-adf.ts:266 directiveNode: a second engine guard inside the parse's own, explained by how each engine overflows
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: a caller flag decides the fill and every read deletes, so what a later call sees depends on who called first
• Locality: src/markdown/emit/adf-to-markdown.ts:66 walkBlocks: the length bound is checked at four points, each relying on the others to cover what it skips
• Navigation, Shape: src/markdown/length-bound.ts:37 withinEngineLimits: engine and length refusals have their own file and code, and result.ts is back to types
• Shape: src/markdown/length-bound.ts:17 tooLong: a string overflow keeps unsupported-node-shape while array and memory take engine-limit, explained only by a comment
• Shape, Locality: src/markdown/portable/adf-to-portable-markdown.ts:128 portableSequence: a failed spelling returns early with each block's own path, where it was swallowed
• Shape: src/markdown/emit/adf-to-markdown.ts:117 commonMarkSpelling: returns Result, which says what it means, where undefined and success(null) encoded the answer
• Shape: src/canonical-json.ts:23 canonicalJsonPieces: a generator replaces a boolean early-stop callback
• Shape: src/markdown/emit/line-escaping.ts:113 backslashed: PlacedSegment drops repeated cursor arithmetic, at the cost of byte codes, chunked joins and RunEscapes

Inherited architect, decided by:

• Locality: src/markdown/emit/adf-to-markdown.ts:126 readableBlock: a read deletes and only an asked caller fills, so whether a node is spelled again depends on call order; the first place to rot
• Locality, Navigation: src/markdown/length-bound.ts:36 withinEngineLimits: wrap-every-builder is a project-wide duty, and six sites refuse with one message and path [], so a report cannot say which fired
• Navigation: src/markdown/length-bound.ts:1 withinEngineLimits: the size concern has a file named for it and engine refusals their own code
• Locality: src/markdown/parse/markdown-to-adf.ts:265 directiveNode: a nested engine guard mid-walk, there only to place V8's throw at this directive
• Shape: src/markdown/emit/adf-to-markdown.ts:117 commonMarkSpelling: Result tells the truth and a refusal now propagates with its path; line 134's conditional is dense
• Shape: src/markdown/emit/line-escaping.ts:40 RunEscapes: byte flags, codes and batched joins read less directly, and escapable reads close to backwards; PlacedSegment is a real gain

Maintainability senior, decided by:

• Navigation, Shape: src/markdown/length-bound.ts:37 withinEngineLimits: guard, bound and refusals share one file, and result.ts is back to describing results
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: deletes on every read and writes only when asked, so a hit depends on call order across the parser, the portable reduction and the walk
• Locality, Self-sufficiency: src/markdown/parse/markdown-to-adf.ts:266 directiveNode: a check's place is justified by where V8 would throw, with a nested engine guard, so moving one needs V8's behaviour reconstructed
• Locality: src/markdown/portable/adf-to-portable-markdown.ts:128 portableSequence: a refused spelling comes back as a value with its block's path, where it was swallowed
• Shape: src/markdown/emit/adf-to-markdown.ts:117 commonMarkSpelling: Result where absence coded both answers
• Shape: src/markdown/length-bound.ts:17 tooLong: a length refusal under unsupported-node-shape with path [], and the README row grown into a catch-all
• Shape: src/markdown/emit/line-escaping.ts:113 backslashed: byte codes and chunked joins against PlacedSegment removing duplicated cursor arithmetic, about even
• Shape: src/canonical-json.ts:23 canonicalJsonPieces: a generator replaces a boolean-returning callback
• Locality: src/markdown/emit/inline-line.ts:235 emitLeaf: the content-or-text refusal is hoisted once, a small gain

Junior A, decided by:

• Navigation, Shape: src/markdown/length-bound.ts:37 withinEngineLimits: the engine-overflow catch moves out of result.ts into a file named for the job, with the two message lists saying which throw means what
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: a read deletes and a caller flag decides the store, an order dependence across three files
• Locality: src/markdown/parse/markdown-to-adf.ts:266 directiveNode: a nested engine guard to pin V8's throw, and the bound checked in five places
• Shape, Locality: src/markdown/emit/adf-to-markdown.ts:117 commonMarkSpelling: Result matches its name, and portableSequence propagates a refusal with each block's path
• Shape: src/markdown/emit/line-escaping.ts:38 PlacedSegment: replaces a parallel placements array and hand-advanced cursor, against byte codes and chunked joins
• Shape: src/canonical-json.ts:23 canonicalJsonPieces: a generator replaces a write callback threading a keep-going flag
• Shape: src/markdown/emit/inline-line.ts:175 heldContentOrText: the content-or-text refusal checked once in emitLeaf

2026-10-13T18:05:39Z, PR #37 at 3ba3a82, against 7caa823

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	better	same	better
Inherited architect	better	better	better	better	better

Mid A, decided by:

• Shape: src/markdown/emit/adf-to-markdown.ts:117 commonMarkSpelling: returns Result, where undefined and success(null) had to be decoded from the callers
• Shape, Locality: src/markdown/parse/markdown-to-adf.ts:266 directiveNode: one boolean test where a typed guard, a fallback and an undefined check decoded the tri-state again
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: every memo read, delete and write sits in one function, where three held its life cycle
• Navigation: src/markdown/length-bound.ts:52 engineLimit: an engine-limit refusal has its own code and constructor, where it shared unsupported-node-shape with shape refusals
• Shape, Locality: src/markdown/portable/adf-to-portable-markdown.ts:128 portableSequence: one check on spelled.value, and a flatMap replaces listItemBlocks
• Locality: src/markdown/emit/line-escaping.ts:38 PlacedSegment: computed once and passed on, where three functions each re-derived starts with their own cursor
• Shape: src/markdown/emit/line-escaping.ts:52 escapingCodes: byte codes, flags and a 4096-piece join, and the RunEscapes union, read harder than a plain list

Inherited architect, decided by:

• Navigation, Self-sufficiency: src/result.ts:2 ConvertErrorCode: engine-limit names its cause and points at one function, where the most overloaded code hid whether the document or the runtime failed
• Shape, Locality: src/markdown/emit/adf-to-markdown.ts:117 commonMarkSpelling: Result, so directiveNode tests one boolean where it decoded a tri-state
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: the memo's read, delete and write sit in one function under one comment, at the cost of a caller flag
• Shape, Self-sufficiency: src/markdown/portable/adf-to-portable-markdown.ts:134 portableSequence: one boolean test, though the list's own path for its items is now stated only in the README
• Locality: src/markdown/emit/line-escaping.ts:38 PlacedSegment: each segment arrives with its start, so two functions no longer walk in step; backslashed itself reads costlier
• Shape: src/markdown/emit/line-escaping.ts:41 RunEscapes: its field collides with isEscapable and the call site handles false first, the slowest spot to read
• Self-sufficiency: src/markdown/length-bound.ts:43 withinEngineLimits: the stack-overflow comment sits on its line, the carry comment says why the join is bounded

2026-10-13T22:23:11Z, PR #40 at 8d14bbf, against 341280d

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	same	better	same	same
Inherited architect	same	better	better	same	better
Maintainability senior	worse	same	better	same	same
Junior A	same	same	better	better	better

Mid A, decided by:

• Shape: src/markdown/directive-syntax.ts:109 slotFault: both branches return unsupported-node-shape through its constructor, where one small function returned two codes and the old name sounded like a writer's failure

Inherited architect, decided by:

• Shape, Locality: src/markdown/directive-syntax.ts:109 slotFault: one rule with one code, where two branches gave the same kind of problem different codes and the reason lay outside the function
• Shape: src/result.ts:1 ConvertErrorCode: the list loses a code named for writing that only the parser raised, so every name matches its producer

Maintainability senior, decided by:

• Shape: src/markdown/directive-syntax.ts:109 slotFault: the two checks raise one code, where sibling refusals of one slot took different codes under a name untrue twice over
• Navigation: README.md:451 unsupported-node-shape: a symptom arrives as a code many sites raise, and its longest cell grows, so a reader searches on the message text where one row and one producer led straight there

Junior A, decided by:

• Shape: src/result.ts:1 ConvertErrorCode: the list loses a writer-named code only the parser raised
• Shape, Self-sufficiency: src/markdown/directive-syntax.ts:109 slotFault: one code for both branches, and the message states the rule and the fix that lived only in the README

2026-10-13T23:14:24Z, PR #41 at d1dc155, against e0ead52

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	same	same	better	same
Inherited architect	same	worse	worse	better	worse
Maintainability senior	same	same	worse	same	same
Junior A	same	same	same	better	same

Mid A, decided by:

• Self-sufficiency: src/markdown/directive-syntax.ts:108 slotFault: the comment now names the node the emitter carries, where "such" pointed at nothing in view
• Self-sufficiency: src/markdown/portable/conventions.ts:13 Claims.pipeTables: the three-valued claim's comment says what each value accepts, so every check in blocks.ts reads without the spec

Inherited architect, decided by:

• Locality, Overall: src/markdown/parse/blocks.ts:381 readBlockLine / openLeaf / openGfmTable / pipeTableRow / pipeTableBlock / bareTableFault: a three-valued claim tested in six places with four comparisons, and no single place states what each mode does to a table
• Locality: src/markdown/parse/blocks.ts:497 openGfmTable: cuts the open paragraph through walk.leaf, closes it, then sets walk.leaf again, an order on shared state; its call must precede the pipeCells branch and nothing says so
• Shape: src/markdown/portable/conventions.ts:14 Claims.pipeTables: 'flavour' names the lossless rule though every reader is a flavour, so === 'flavour' needs the field's comment to decode
• Shape: src/markdown/parse/blocks.ts:514 pipeTableRow: one if merges three reasons into "return cells", two of which mean "end the table"; the indentation guard is duplicated at two call sites
• Self-sufficiency: src/markdown/parse/blocks.ts:496 openGfmTable, pipeTableRow: each opens with a "GFM, Tables:" line stating the rule it enforces

Maintainability senior, decided by:

• Shape: src/markdown/portable/conventions.ts:14 Claims.pipeTables: 'flavour' names only the lossless reader's strict rule though portable is a flavour too, and blocks.ts tests the three states three ways
• Locality: src/markdown/parse/blocks.ts:497 openGfmTable: rewrites walk.leaf, closes it, then installs the table, and its call must precede the pipeCells claim and the setext check with no comment saying so
• Locality: src/markdown/parse/blocks.ts:63 ParagraphLine: each line keeps its position, where two parallel arrays had to stay the same length

Junior A, decided by:

• Self-sufficiency: src/markdown/directive-syntax.ts:108 slotFault: the comment says on its own which node the emitter carries
• Self-sufficiency: src/markdown/portable/conventions.ts:13 Claims.pipeTables: the comment says what each mode reads, so the branches in blocks.ts read without the spec

2026-10-14T08:45:14Z, PR #43 at b83adc4, against e193536

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	worse	same	worse
Inherited architect	same	worse	worse	same	worse

Mid A, decided by:

• Locality: src/markdown/parse/directive-nodes.ts:140 readNamedMark: words its error by deleting ; ${directiveEscape} from a fault built in directive-syntax.ts, so rewording that message silently changes this one
• Shape, Locality: src/markdown/parse/directive-nodes.ts:100 readMarks / readMarkArray / readNamedMarks / readNamedMark: one attribute's parse now spans four units and three modules, a cursor-driven list parser in a file about directive nodes
• Shape: src/markdown/directive-syntax.ts:299 readAttributes / readDirectiveName: exported but left among the private helpers, below the file's export block
• Locality: src/markdown/directive-syntax.ts:69 attributeValue: the numbers branch reads a JSON array only for a respelling check that lives in another file

Inherited architect, decided by:

• Locality, Overall: src/markdown/parse/directive-nodes.ts:140 readNamedMark: strips the escape hint with a string replace tied to the wording of pairFault and prefixFault in directive-syntax.ts, which nothing there warns of
• Locality, Shape: src/markdown/parse/directive-nodes.ts:117 readNamedMarks / readNamedMark: a grammar inside a quoted attribute value, two levels of escaping held at once, the reader in a file otherwise about building nodes and the writer in block-directive.ts
• Shape: src/adf/mark-attributes.ts:22 markVocabularies / markAttributes / isMarkType: markAttributes silently means inline marks beside markVocabularies meaning all, and isMarkType('alignment') is false

2026-10-14T08:45:16Z, PR #43 at b83adc4

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Junior A	6.5	5	6	5	5.5
Mid A	6	5	5.5	5	5.5
Maintainability senior	7	5.5	6.5	5.5	6
Inherited architect	7	6	6	6.5	6.5
Mean	6.63	5.38	6.00	5.50	5.88

Junior A, hardest first:

• Locality, Overall: src/markdown/parse/markdown-to-adf.ts:237 splitPath: builds stand-in nodes and re-runs splitTaskList to find an error path, the comments naming the trick but not why it is correct
• Locality, Self-sufficiency: src/markdown/emit/line-escaping.ts:230 escapeClosedRuns / escapeClaims: per-character flags in Uint8Arrays and a backward-filled set of backtick lengths, opaque without the spec open
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: an identity-keyed memo deleted on read and filled by the parser and the portable writer, action at a distance
• Locality: src/markdown/parse/inline-content.ts:330 closeBracket: mutable Scan flags set mid-scan and read later, then a call back into the emitter to decide a parse error
• Shape: src/markdown/emit/inline-line.ts:82 emitLine: a retry loop widening a carried index set threaded through nested emitRun calls
• Shape: src/markdown/portable/inline-reduction.ts:194 trimmedEdges: two nested fixpoint loops and a number-or-undefined edge depth

Mid A, hardest first:

• Locality, Self-sufficiency: src/markdown/emit/line-escaping.ts:142 escapeClaims / escapeClosedRuns: three passes over a Uint8Array whose conditions tie escaping kind, token kind and position, unverifiable against the spec's prose
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: filled only on an asked call, deleted on any read, so correctness depends on the order two directions call it
• Locality: src/markdown/parse/markdown-to-adf.ts:237 splitPath: stand-in nodes and a re-run split to report an error path
• Locality: src/markdown/emit/inline-line.ts:82 emitLine: a fixpoint loop the parser re-enters through openingLinkTakesDirective
• Shape: src/markdown/portable/conventions.ts:24 flavourClaims: the switch every reader and writer uses lives under portable/
• Shape: src/markdown/directive-syntax.ts:32 Read: the general reader-failure type defined in the directive grammar file

Maintainability senior, hardest first:

• Locality, Overall: src/markdown/parse/markdown-to-adf.ts:237 splitPath: rebuilds a fake task list to place a refusal, correct only by knowing how splitTaskList and splitPlace move children
• Self-sufficiency: src/markdown/emit/line-escaping.ts:142 escapeClaims / escapeClosedRuns: four escaping kinds and three special cases at once, complete only against spec §Canonical form
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: a reference-keyed memo with delete-on-read and depth rebasing, called from the parser and the portable reducer
• Shape: src/markdown/parse/inline-content.ts:443 resolveNodes / pairedHighlights: parallel pieces and nodes arrays, a cursor advanced inside an index expression, and a seven-way Piece union
• Shape: src/markdown/portable/conventions.ts:8 Claims: sets all three readers' behaviour yet lives under portable/

Inherited architect, would restructure first:

• Shape, Navigation: src/markdown/portable/conventions.ts:8 Claims / flavourClaims: the switch for all three flavours filed under the portable directory
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: the parse-to-emit dependency runs through a reference-keyed SpellingMemo shared across calls
• Shape: src/markdown/parse/markdown-to-adf.ts:143 quoteNode / splitPath: the portable reader lives in parse/ and reaches back into portable/
• Shape: src/markdown/emit/adf-to-markdown.ts:157 spellPortableBlock: portable-only writing in a file whose name matches neither flavour
• Locality: src/markdown/emit/line-escaping.ts:260 unspellableRuns: the 3am unit, concentrated escaping that needs spec §Canonical form and §Marks open to change safely

2026-10-14T13:01:12Z, PR #46 at 1e8fa26, against 06739b40b7

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	better	same	better
Inherited architect	better	better	same	better	better

Mid A, decided by:

• Locality, Shape: src/adf/inline-nodes.ts:37 takesMark: the per-node mark types say which marks each node takes, and one predicate serves the parser and the emitter where each worked it out differently
• Locality: src/markdown/parse/directive-nodes.ts:138 readNamedMark: the caller passes the message it wants instead of cutting text out of another module's message
• Locality: src/markdown/parse/markdown-to-adf.ts:280 commonMarkSpellings: a new hand-kept table the parser must keep in step with the emitter's commonMarkSpelling
• Navigation: src/markdown/emit/inline-line.ts:224 carries: "withheld-mark" names the carry the same way as the corpus fixture and the decision index
• Shape: src/markdown/parse/inline-content.ts:405 linkPiece: refusals name the fix, though marksItem returning true for a non-node and the hard-break carve-out read a little harder

Inherited architect, decided by:

• Navigation, Self-sufficiency: README.md:455 unsupported-node-shape: each cause sits beside its own remedy, grouped by direction, where one cell held them all
• Locality, Self-sufficiency: src/adf/inline-nodes.ts:37 takesMark: one predicate and a per-type mark list replace a boolean read two ways by the parser and the emitter
• Locality: src/markdown/parse/directive-nodes.ts:137 readNamedMark: the caller passes its message rather than editing another module's text
• Locality: src/markdown/parse/markdown-to-adf.ts:279 commonMarkSpellings: a hand-kept list of the emitter's CommonMark spellings in another file, where the new version rots first
• Shape: src/markdown/parse/inline-content.ts:294 marksItem: reads as a verb phrase and returns true for items no mark lands on
• Locality: src/markdown/parse/inline-content.ts:400 linkPiece: linkTo's destination-ahead-of-hard-breaks branch is right only because linkPiece refused every other unlinkable node first, and neither says so
• Navigation: MIGRATION.md:3 From 0.2.0 to 0.3.0: a reader starting from a mark lost after upgrading finds a table and a fixture named after the cause

2026-10-14T13:02:45Z, PR #46 at 1e8fa26

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Junior A	7	5.5	6.5	5.5	6
Mid A	7	5	6	5	5.5
Maintainability senior	7	5.5	6.5	5.5	6
Inherited architect	6.5	6	5.5	7	6
Mean	6.88	5.50	6.13	5.75	5.88

Junior A, hardest first:

• Self-sufficiency, Overall: src/markdown/emit/line-escaping.ts:230 escapeClosedRuns / escapeClaims: three parallel per-character arrays and a backward-built set of backtick lengths, opaque without spec §Canonical form open
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: a memo filled by the parser and the portable reducer and drained by the walk, deleted on read
• Locality: src/markdown/parse/markdown-to-adf.ts:237 splitPath: stand-in nodes and a re-run split to place an error path
• Locality, Shape: src/markdown/parse/inline-content.ts:449 resolveNodes / pairedHighlights: placeholder text nodes until a highlight pairs, counters and in-place overwrites
• Shape: src/markdown/emit/inline-line.ts:82 emitLine: a retry loop whose outcome also flows back into the parser

Mid A, hardest first:

• Locality, Overall: src/markdown/parse/markdown-to-adf.ts:237 splitPath: fake taskList and bulletList stand-ins and a probe node re-run through the split to name a path
• Self-sufficiency: src/markdown/emit/line-escaping.ts:230 escapeClosedRuns: why a right-to-left walk suffices lives only in the spec
• Locality: src/markdown/parse/inline-content.ts:332 closeBracket / truncatePieces: four sites mutate scan.pieces and deactivatedBefore
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: filled from the parser and the portable reducer, the clearest action at a distance
• Shape: src/markdown/portable/inline-reduction.ts:194 trimmedEdges: two fixpoint loops with no termination argument and a number-or-undefined edge depth

Maintainability senior, hardest first:

• Self-sufficiency, Overall: src/markdown/emit/line-escaping.ts:142 escapeClaims / unspellableRuns: three passes whose sufficiency no statement gives, and an opening-link signal read from segment structure
• Locality: src/markdown/parse/inline-content.ts:332 closeBracket / resolveNodes: Scan mutated from about ten functions over a three-way Inline union
• Locality: src/markdown/parse/markdown-to-adf.ts:237 splitPath: a local simulation of the task-list split, hard to convince oneself is right
• Locality: src/markdown/emit/adf-to-markdown.ts:127 readableBlock: a reference-keyed memo filled across three directories
• Shape: src/markdown/portable/adf-to-portable-markdown.ts:59 reduceToPortable: reruns the reduction on the raw document to get a refusal's path

Inherited architect, would restructure first:

• Navigation, Shape: src/markdown/portable/conventions.ts:8 Claims / flavourClaims: the switch for all three flavours filed under portable/
• Navigation: src/markdown/parse/markdown-to-adf.ts:161 quoteNode / splitPath: the portable reader lives in parse/ and the portable writer's spellings in emit/adf-to-markdown.ts:157 spellPortableBlock
• Locality: src/markdown/parse/markdown-to-adf.ts:266 directiveNode: the reader asks the writer's commonMarkSpelling, and the writer reuses the reader's tokenizer, so neither side changes alone
• Shape: src/markdown/directive-syntax.ts:32 Read: the general reader-failure type housed in the directive grammar file, among 15 loose files in src/markdown/
• Shape: src/markdown/emit/adf-to-markdown.ts:45 writeMarkdown: writes both flavours in a file named for the lossless one

2026-10-14T19:06:43Z, PR #49 at d8fbff6fde, against 696aca1c71

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	same	same	better
Inherited architect	same	same	worse	same	same
Maintainability senior	same	better	same	same	better
Junior A	better	same	worse	same	better

Mid A, decided by:

• Navigation, Locality: src/adf/inline-nodes.ts:41 takenMarkTypes / takesMark: the code-text rule lives in one predicate beside the mark table, reached by the parser, the emitter's carry and the portable reducer
• Locality: src/markdown/parse/inline-content.ts:584 highlighted: asks marksItem instead of a hand-kept code check
• Locality: src/markdown/portable/inline-reduction.ts:136 portableMarks: drops exactly the marks the reader refuses, by the shared name
• Locality: src/markdown/emit/inline-line.ts:103 freshFallbacks: carries no longer shows every reason a node is carried; the set starts pre-filled
• Shape, Self-sufficiency: src/adf/inline-nodes.ts:40 takenMarkTypes: undefined means every mark, explained only by its comment; spec says code text takes only link

Inherited architect, decided by:

• Shape, Locality: src/markdown/emit/inline-line.ts:102 freshFallbacks: seeds schema carries into a set named for fallbacks, and carries no longer says why a node is carried
• Locality: src/adf/inline-nodes.ts:5 codeTextMarkTypes: holds code with nothing saying why, and spec/flavour.md says code text takes only link
• Locality: src/adf/inline-nodes.ts:46 takesMark: the answer depends on the node's marks at call time, right only because a code span is built with code first
• Locality: src/markdown/parse/inline-content.ts:584 highlighted: one table now answers what two hand-written checks did

Maintainability senior, decided by:

• Locality, Overall: src/adf/inline-nodes.ts:41 takenMarkTypes / takesMark: one function states the code-text rule the parser and emitter both ask; undefined meaning every mark is visible only in its comment
• Locality: src/markdown/parse/inline-content.ts:587 highlighted: routes through marksItem instead of a private copy of a schema rule
• Locality: src/markdown/portable/inline-reduction.ts:146 portableMarks: filters by the shared list, but checks code itself rather than asking takesMark
• Locality: src/markdown/emit/inline-line.ts:103 freshFallbacks: carries is right only if the caller seeded the set, and its comment no longer names withheld marks

Junior A, decided by:

• Navigation, Overall: src/adf/inline-nodes.ts:5 codeTextMarkTypes: the code-text rule lives in one table beside the other inline nodes' table
• Shape: src/adf/inline-nodes.ts:41 takenMarkTypes: "taken" reads as accepted or as already held, and undefined means every mark
• Shape: src/markdown/emit/inline-line.ts:103 freshFallbacks: named fresh but seeds the carried set, and carries's comment no longer names the withheld-mark carry

2026-10-14T20:46:20Z, PR #53 at 787d909, against 498e874

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	better	same	better
Inherited architect	better	better	better	same	better

Mid A, decided by:

• Shape: src/markdown/emit/inline-line.ts:103 withheldMarkFallbacks: renamed from freshFallbacks, it now says the fallbacks start seeded with every node holding a withheld mark
• Locality: src/markdown/emit/inline-line.ts:234 carries: the comment now names withheldMarkFallbacks as where the withheld-mark carries are decided, linking the two units

Inherited architect, decided by:

• Locality, Navigation: src/markdown/emit/inline-line.ts:234 carries: the comment now points at withheldMarkFallbacks for the third kind of carry, a hidden dependence made followable
• Shape, Navigation: src/markdown/emit/inline-line.ts:103 withheldMarkFallbacks: the old name freshFallbacks hid a pre-seeded set; the new one says what it holds
• Shape: src/markdown/portable/inline-reduction.ts:84 mediaLeaves: three branches, two writing [alt](url) by different routes, seen only by reading portableMarks
• Self-sufficiency: src/markdown/portable/inline-reduction.ts:83 mediaLeaves: the comment says why an inline image becomes a link, not why the other target goes in parentheses as image

2026-10-22T19:35:21Z, PR #80 at ff220e6, against 2e06141

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	better	same	better
Inherited architect	better	better	better	same	better

Mid A, decided by:

• Navigation, Shape: src/markdown/portable/portable-markdown-to-adf.ts:22 portableMarkdownToAdf: the portable reading has its own file, and parse/markdown-to-adf.ts shrinks from 394 to 267 lines
• Navigation, Shape: src/markdown/directive/syntax.ts:1 directive/: ten directive files once spread over three directories share one folder
• Navigation, Shape: .oxlintrc.json:5 overrides: lint enforces the layering, and parse/ reaches emit/ only through spelling-questions.ts
• Navigation, Shape: src/markdown/flavour.ts:6 Claims: the shared claims left a portable-named file for a neutral home
• Locality: src/markdown/portable/portable-markdown-to-adf.ts:102 bulletBlocks / readTaskBlock: a refusal's path is found by one marker after the one split, not a replayed split on stand-ins
• Locality: src/markdown/emit/adf-to-markdown.ts:35 SpelledBlock: a list's interruption and kind are fixed when spelled, not re-derived with a taskList special case
• Locality: src/markdown/commonmark/backtick-runs.ts:11 backtickCloser: per-length cursors that need from rising add hidden state (counterweight)
• Shape: src/markdown/emit/adf-to-markdown.ts:26 FlavourBlockWriters / FlavourBlockReaders: the core now exports internals and a reader follows an injected function (counterweight)

Inherited architect, decided by:

• Locality, Shape: src/markdown/parse/markdown-to-adf.ts:23 FlavourBlockReaders / ReadAs: the core reader is flavour-neutral but for two hooks; the callback costs a search to find its supplier
• Locality: src/markdown/portable/portable-markdown-to-adf.ts:147 readTaskBlock / bulletBlocks: one marker found by identity replaces splitPath's fake list; steps.slice(0, -2) still costs a re-read
• Navigation, Shape: src/markdown/flavour.ts:1 Flavour / Claims / flavourClaims: what any flavour claims no longer lives in a portable-named file
• Navigation, Shape: src/markdown/directive/syntax.ts:1 directive/: one folder answers where the directive grammar is
• Navigation, Shape: src/markdown/emit/spelling-questions.ts:1 spelling-questions: the one parse→emit dependency is a named seam
• Locality, Shape: src/markdown/portable/block-spelling.ts:12 portableBlockWriters: the emitter loses the portable spellings and re-derives no list facts; the portable file imports five emitter internals back
• Locality: src/markdown/commonmark/backtick-runs.ts:11 backtickCloser: a closure correct only for rising from, stated not enforced (against)
• Locality: src/markdown/parse/blocks.ts:67 Line.spaces / matchBlankLine: tab remainders in a separate field every site must remember (against)
• Shape: src/markdown/portable/portable-markdown-to-adf.ts:19 expandBlockReaders / portableBlockReaders: a swapped reader object replaces the inExpand flag, less obvious on first read

2026-10-23T01:53:14Z, PR #82 at 8e7da0d, against 33e2a86

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	worse	same	worse
Inherited architect	same	worse	same	same	worse

Mid A, decided by:

• Locality, Shape: src/markdown/parse/inline-content.ts:264 assemble: an (image) leaves a slot through Scanned.imageLinks, is dropped by linkedAlt, regathered by linkedImages and kept apart by resolvedPieces; five functions and two recursion levels to follow one image
• Locality: src/markdown/parse/inline-content.ts:209 directivePieces: inLink holds only through mark slots, so the mark is read before the slot and a node directive silently resets it
• Shape: src/markdown/parse/inline-content.ts:58 Piece: gains ImageLink, Scanned becomes { imageLinks, pieces }, and linkedAlt now also discards imageLink pieces, which its name does not say
• Shape: src/markdown/mark-spellings.ts:58 commonMarkLink: a sixth positional parameter, escapes, which the parser always passes as 'lossless'
• Locality: src/markdown/portable/inline-reduction.ts:154 portableLink: reads easier; it keeps the title and leaves escaping to spellTitle, and imageAside builds the (image) once

Inherited architect, decided by:

• Locality: src/markdown/parse/inline-content.ts:209 directivePieces: each slot returns its nodes and a list of (image)s still to place, tracked through eight units, and inLink's narrowed meaning lives in one boolean expression
• Locality: src/markdown/emit/inline-line.ts:69 tryPipeCell: the portable flavour skips the pipe check, its safety resting on three guards elsewhere, and the comment names the wrong file for half of it
• Locality: src/markdown/portable/inline-reduction.ts:154 portableLink: reads easier; the reducer only normalizes and escaping stays in spellTitle

2026-10-26T12:16:45Z, PR #86 at fa00f09e18, against 61a5c073e9

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	same	same	better
Inherited architect	same	better	same	same	better

Mid A, decided by:

• Locality, Navigation: src/adf/shown-text.ts:60 shownText: one switch per question says what each node shows, shared by the portable writer and the HTML carry, where nodeLeaves worked it out inline
• Locality: src/adf/document.ts:53 writableContent: every writer takes the shape, version and length checks from one function instead of repeating them
• Navigation, Locality: src/adf/plain-image.ts:19 plainImage: one named predicate answers "is this the plain image" for both formats; tryImage shrinks to a lookup
• Navigation: src/length-bound.ts:4 longestOutput: the file and the name now say the bound covers every writer
• Navigation: src/html/emit/adf-to-html.ts:21 adfToHtml: a cold-findable html/emit/ of small units named for one block kind each, its contract in spec/html.md
• Shape: src/markdown/mark-spellings.ts:40 linkMark: seven small converters between three representations of one link; shownLink and linkAttrs differ only on a non-string title
• Shape: src/html/emit/carry-element.ts:13 carriedBlock: reuses the names src/markdown/opaque-carry.ts exports, with other return types
• Shape, Self-sufficiency: src/html/emit/carry-element.ts:30 shownBlocks: a stack walk with null sentinels and a threaded code field needs its comment and a second read

Inherited architect, decided by:

• Locality, Overall: src/adf/shown-text.ts:60 shownText: what each node shows has one switch per question, shared by the portable writer and the HTML carry, where it was worked out inline in nodeLeaves, cardLeaves and mediaLeaves
• Locality, Overall: src/adf/document.ts:53 writableContent: every writer opens with one call instead of repeating the prologue
• Locality: .oxlintrc.json:6 the src/adf/** and src/html/** import bans: adf/ importing no format, and HTML and markdown not importing each other, is now machine-checked
• Locality: src/markdown/parse/inline-content.ts:387 closeBracket: an explicit bracket stack matches CommonMark's algorithm where a backward scan and deactivatedBefore did
• Shape: src/markdown/mark-spellings.ts:40 linkMark: two shapes for one link with converters both ways, and shownLink reading a non-string title differently from linkAttrs
• Shape: src/adf/shown-text.ts:7 isBlockNodeType: neither it nor oneLine is about shown text; the renames to longestOutput and plain-image.ts balance it
• Navigation: src/html/emit/carry-element.ts:13 carriedBlock: its name, tryImage and linkedText collide with units in src/markdown/, so a symbol search finds two homes
• Self-sufficiency: src/html/emit/carry-element.ts:30 shownBlocks: the densest new unit, a stack with null block ends and an inherited code; its comments send a reader to the decision log where spec/html.md explains it

2026-10-28T10:33:35Z, PR #88 at 207bf01, against 240c39a

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	worse	same	same
Inherited architect	better	worse	better	same	same

Mid A, decided by:

• Locality: src/html/block-elements.ts:27 readListStart: each element's spelling now sits in a small file beside its reader (anchor-element.ts, image-element.ts, language-class.ts), where 240c39a kept them inside the emitter, so changing one spelling no longer means hunting for its inverse
• Locality: src/adf/carried-json.ts:24 carriedValue: the markdown carry and the HTML carry share three named steps where readCarriedJson parsed, bounded and validated inline; notAString likewise leaves markdown-to-adf.ts
• Shape: src/html/parse/html-to-adf.ts:248 readItemLine: an item's line spreads across readLineParagraph, partedLine, asFirstParagraph and formRefusals, with a nested ternary picking the refusal, in a file importing 20 modules
• Shape, Self-sufficiency: src/html/escaping.ts:14 misspelling: four plain replaceAll calls became regexes built from an entity map, with a negative lookahead to rebuild in one's head
• Self-sufficiency: src/html/parse/carry-element.ts:14 readCarry: the coupling to carryContent is stated where it lives, so spec/html.md need not be open
• Shape: src/html/data-attributes.ts:26 readDataAttributes: an optional leading parameter and a previous sentinel only nameFault's comment explains, and a respell default added to spellsCanonicalJson

Inherited architect, decided by:

• Locality: src/html/parse/html-to-adf.ts:234 readItem: the writer's list-item layout is copied as string literals into the reader, to change in lockstep with writeItem in another directory; readParagraph finds an item's first paragraph by order, and line 256 is one ternary of about 300 characters
• Navigation, Shape: src/html/anchor-element.ts:4 anchorOpener: each element's spelling has a file named after the element with its inverse beside it, where 240c39a kept them in a file named for escaping
• Shape: src/html/emit/adf-to-html.ts:90 listElement: tag names and the order check come from block-elements.ts instead of inline literals, and readListStart is defined by re-running the writer
• Locality: src/html/carry-content.ts:13 carryContent: the writer and the reader's edit check share one function, its comment warning that a change breaks reading
• Locality, Self-sufficiency: src/html/data-attributes.ts:26 readDataAttributes: a leading parameter only <ol start> uses, with the hardest comments in the change to parse
• Locality: src/html/carry-attributes.ts:11 carryJsonOffset: rebuilds the writer's start-tag prefix to place an error, correct only while carryAttributes writes class first
• Shape: src/adf/carried-json.ts:12 BoundedJson: a type whose only job is to order three steps, a respell hook, and ten small root files in src/html/

2026-10-28T18:08:02Z, PR #99 at fcc24ad, against 1530666

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	same	same	better
Inherited architect	same	better	same	better	better

Mid A, decided by:

• Locality, Overall: src/markdown/emit/line-escaping.ts:97 readBack: the writer reads its escaped line back through the real reader instead of imitating its backtick, emphasis and bracket rules by hand in escapeClosedRuns, unspellableRuns and lastLinkClose
• Locality: src/markdown/inline-content.ts:103 SharedScan: what a directive's content shares with the outer content is one named object, where the base spread it through Scan and picked it with Pick
• Locality: src/markdown/commonmark/link-syntax.ts:55 readInlineTarget: a mutable memo shared by every link target, filled lazily, where the base read a target as a pure function of its arguments
• Self-sufficiency: src/markdown/commonmark/link-syntax.ts:76 destinationEnds: a stack-based table followed only through its long comment, though the base's equally opaque escapeClosedRuns is gone
• Shape, Navigation: src/markdown/inline-content.ts:121 parseInlineContent: moved out of parse/ now that emit/ uses it, and takes openingLinkTakesDirective as an argument every caller passes the same

Inherited architect, decided by:

• Locality, Self-sufficiency: src/markdown/emit/line-escaping.ts:97 readBack: the writer checks its line through readLineBack, so a change to the reader's matching needs no mirror in emit/; the faulted fallback still leans on its comment
• Self-sufficiency: src/markdown/emit/line-escaping.ts:136 withTextEscapes: with textOpeners, states the whole rule, where the base's escapeClosedRuns and its formed set needed the spec paragraph beside it
• Locality: src/markdown/emit/line-escaping.ts:271 unspellableRuns: asks the reader's emphasis pairing instead of rebuilding delimiter runs and re-running matchEmphasis
• Locality: src/markdown/inline-content.ts:461 resolveTarget: one target memo keyed by outermost offsets, shared through scan.shared, brings offset arithmetic and a bounds check that depends on how the slot was cut
• Self-sufficiency: src/markdown/commonmark/link-syntax.ts:76 destinationEnds: the densest new code, understandable only through its five-clause comment
• Shape: src/markdown/inline-content.ts:768 parseInlineContent: takes the writer's openingLinkTakesDirective as a sixth parameter, and moved out of parse/ where a reader would look for it

2026-10-28T19:34:20Z, PR #100 at 222ad24, against 0003675

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	same	same	same	same
Inherited architect	better	same	same	same	same

Inherited architect, decided by:

• Navigation: README.md:730 "Changing the source": names ./ci.sh and ./mutation-tests/run.sh, what they need and what fails a run, where the base's README named neither

2026-10-29T15:41:57Z, PR #106 at 4246600, against c7bbb04

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	worse	worse	worse
Inherited architect	same	worse	worse	same	worse

Mid A, decided by:

• Locality, Self-sufficiency: src/markdown/emit/block-text.ts:56 writtenText: prefixes recorded in a tree and applied later by a stack walk over open, quotes, started and 'leave' sentinels, where the base's quoted and tryListItemLines applied them in place with split, map and join
• Locality, Shape: src/markdown/emit/block-text.ts:12 Shape: eight cached measures kept in step by hand across four constructors, their arithmetic shown equal to the written text only in a test
• Locality: src/markdown/commonmark/grammar.ts:207 holdsOnlyRuleCharacters: a grammar helper carries a contract owned by BlockText, standing in front of isThematicBreak
• Locality: src/markdown/portable/block-spelling.ts:15 claimsItem: a markerOpening fast path that must agree with two regexes, only a comment saying so
• Shape, Locality: src/markdown/emit/inline-line.ts:31 FallbacksPerPass: a switch whose only second user is a test, threaded through five production functions and changing droppedRuns
• Self-sufficiency, Shape: src/markdown/emit/line-escaping.ts:274 unspellableRuns: the cascade through givesUp and mayFlank needs CommonMark's flanking rules open beside it, where the base returned the first misread pair
• Shape: src/markdown/emit/inline-line.ts:94 emitLine: linePass, takenThisPass and checkedCarry add two concepts to the fallback loop

Inherited architect, decided by:

• Locality, Shape: src/markdown/emit/block-text.ts:56 writtenText: a stack walk with a 'leave' sentinel and eight hand-kept counters replaces string operations readable where they stood; prefixedText with an empty indent stretches its name
• Locality: src/markdown/portable/inline-reduction.ts:254 droppedRuns: which runs drop in one pass is decided twice, by givesUp in line-escaping and again by dropsAlone, in the code the 3am question lands in
• Shape: src/markdown/emit/inline-line.ts:31 FallbacksPerPass: a test-only mode threaded through production signatures, its 'first' reading not holding in inline-reduction's highlight drops; checkedCarry adds an ordering rule
• Locality: src/markdown/emit/adf-to-markdown.ts:310 tryListItemLines: reads four measures defined in block-text.ts where the base checked strings in place
• Locality: src/markdown/commonmark/grammar.ts:206 holdsOnlyRuleCharacters: the grammar module holds a rule that exists only for the emitter's BlockText
• Locality: src/markdown/portable/conventions.ts:5 markerOpening: repeats the [ the marker regexes encode

2026-10-30T11:10:42Z, PR #107 at 430310d, against 41fd106

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	worse	worse	worse
Inherited architect	same	worse	worse	worse	worse

Mid A, decided by:

• Locality, Shape: src/markdown/emit/line-escaping.ts:294 unspellableRuns: a sweep over the shared PairSweep.fallen, each verdict reading those settled before it, through helpers that call each other in a web, where the base's one loop called givesUp
• Locality: src/markdown/emit/line-escaping.ts:32 RunFallback: the escaper models how each caller takes a run back, the portable reducer's dropsAlone reaching it as the alone callback, where the base's reducer chose its drops beside its leaves
• Self-sufficiency: src/markdown/emit/line-escaping.ts:344 holdsAlike: its delimiter condition needs the delimiters' spellings known, and the sweep's rule rests on matching one run per pass, which only a property test shows
• Shape: src/markdown/emit/line-escaping.ts:65 Standing: fallen, merging, pairAt and either are defined only by one-line comments that use further new terms

Inherited architect, decided by:

• Locality, Shape: src/markdown/emit/line-escaping.ts:294 unspellableRuns: settles pairs outward from the first misread pair, each verdict reading the fallen flags settled before it, so one pair's verdict needs the whole sweep's state, where the base's rule was one loop plus givesUp and a four-line mayFlank
• Shape, Locality: src/markdown/emit/line-escaping.ts:337 fallsWithFirst: calls run about five levels deep through mayFlank, runEdge, pairSpellings, wordBeside and pairStanding, each taking the whole sweep; holdsAlike, holdsEarlier, spansAlike and Standing need their comments, and pairStanding's rule is asserted with no derivation
• Locality: src/markdown/portable/inline-reduction.ts:223 spellableLine: passes dropsAlone as a callback through four functions into the escaper's merging, splitting one contract across two files
• Locality: src/markdown/portable/inline-reduction.ts:244 withoutMarks: runs may now nest, for a reason that lives in the escaper's sweep
• Self-sufficiency: src/markdown/emit/line-escaping.ts:336 fallsWithFirst: its rule is equivalence to the 'first' mode, checked only by a property test, and runEdge and wordBeside need CommonMark's delimiter-run rules beside the code

2026-10-31T01:31:04Z, PR #110 at ceebcfb, against e743868

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	better	worse	better
Inherited architect	better	better	better	worse	better

Mid A, decided by:

• Navigation, Shape: src/markdown/emit/unspellable-runs.ts:33 unspellableRuns: the emphasis-run sweep has its own file; line-escaping.ts holds only escaping
• Locality, Shape: src/markdown/emit/unspellable-runs.ts:9 RunFallback: the drop fallback carries nodes, and dropsAlone sits beside its only consumer
• Shape: src/markdown/emit/unspellable-runs.ts:30 Standing: four named states and shared touching replace a bare undefined and two hand-rolled walks
• Locality, Shape: src/markdown/emit/inline-line.ts:268 emitLeaf: leaf spellers return segments, and one place builds and checks every carry
• Locality: src/markdown/emit/block-text.ts:58 holdsBlankLine: callers ask BlockText instead of combining flat fields
• Self-sufficiency: src/markdown/emit/unspellable-runs.ts:208 pairStanding: the reasoning moved to the decision log, so its branches cannot be checked without it open
• Self-sufficiency: src/markdown/emit/inline-line.ts:48 readBackPasses: the 32-pass cap and its carry-all fallback are justified only by a pointer to the decision log

Inherited architect, decided by:

• Navigation, Shape: src/markdown/emit/unspellable-runs.ts:33 unspellableRuns: a mark gone after a round trip now leads to a file whose name says so
• Locality: src/markdown/portable/inline-reduction.ts:228 spellableLine: no closure into the portable reducer's dropsAlone from deep in the emitter
• Locality, Shape: src/markdown/emit/inline-line.ts:53 emitInlineLine: the test-only perPass knob is gone from production signatures, at the price of exports only a test uses
• Shape: src/markdown/emit/unspellable-runs.ts:191 touching: one shared walk, and truer names such as holdsPairBefore and mergingEnds
• Shape: src/markdown/emit/block-text.ts:13 FirstLine: eight flat fields became two grouped records with named predicates
• Self-sufficiency: src/markdown/emit/unspellable-runs.ts:208 pairStanding: five branches whose proof now lives in the decision log
• Self-sufficiency, Locality: src/markdown/emit/inline-line.ts:48 readBackPasses: a magic cap of 32 shared by two retry loops that must agree on giving up

2026-10-31T02:06:38Z, PR #111 at b067b88, against de0033c

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	same	better	better
Inherited architect	same	same	better	same	better

Mid A, decided by:

• Self-sufficiency, Overall: src/adf/inline-nodes.ts:42 takenMarkTypes: text takes a named list, so the marks it takes read without the schema open
• Locality: src/markdown/emit/inline-line.ts:65 portableLineFallback: one guard covers both the seeded carries and the emission's

Inherited architect, decided by:

• Shape, Overall: src/adf/inline-nodes.ts:42 takenMarkTypes: undefined now means only a node type outside the tables
• Shape: src/markdown/inline-content.ts:261 directiveMarkPieces: this caller overrides marksItem's rule that a non-node is markable, with nothing at the call site saying why

2026-11-03T21:09:33Z, PR #116 at 9e8df99, against 8bcf4a5

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	worse	better	better
Inherited architect	same	better	worse	better	better

Mid A, decided by:

• Locality, Overall: src/markdown/mark-spellings.ts:48 insideRun. What counts as "in the same run" is now one function, used by the emitter and the reducer, and its comment names the footgun; at the base the two compared marks separately and nothing said they had to agree.
• Locality: src/markdown/portable/inline-reduction.ts:221 sharedRuns. The reducer no longer depends on a comparator exported from the editor-normal module, which made a reader check whether normalization mattered here.
• Self-sufficiency, Overall: src/markdown/portable/inline-reduction.ts:231 spellableLine and src/markdown/portable/inline-reduction.ts:165 trimmedEdges. Both loops now state in one line why they terminate.
• Self-sufficiency: src/markdown/portable/inline-reduction.ts:262 withoutMarks with markCount. A pass that takes no mark returns undefined and withoutFallback refuses it, so the argument is enforced in the code.
• Shape: src/markdown/mark-spellings.ts:21 RunNode, threaded through src/markdown/emit/inline-line.ts:213 emitRun and src/markdown/emit/inline-line.ts:232 inlineRuns. outer and depth always hold the same value, and a reader has to trace the recursion to answer why both exist.
• Shape: src/markdown/emit/inline-line.ts:384 tryLink against src/markdown/emit/inline-line.ts:364 codeSpans. The same count is spelled two ways a few lines apart.
• Shape: src/markdown/emit/inline-line.ts:288 carriedRun with lineNodes in InlineContext. Nodes and range can no longer disagree, but the context gains a field only this function uses.
• Locality: src/markdown/portable/inline-reduction.ts:250 withoutOpeningLinkCode. A findIndex over the run says what it means, where the base wrapped one mark in an array to reuse a helper.

Inherited architect, decided by:

• Locality, Self-sufficiency: src/markdown/portable/inline-reduction.ts:262 withoutMarks, with markCount at :278 and the comment at :231 in spellableLine. A pass that takes no mark returns undefined and withoutFallback refuses it, so the termination claim can be checked in this file.
• Locality: src/markdown/mark-spellings.ts:48 insideRun. The rule for what stays in a run has one home, used by the writer's inlineRuns and by the reducer, where the base left a reader to prove two comparators agreed on reduced marks.
• Shape, Locality: src/markdown/portable/editor-normal.ts:11 joinsWhenEditorNormal. sameMarkWhenEditorNormal is gone, so the file now only answers what the editor joins and normalizes.
• Shape: src/markdown/emit/inline-line.ts:213 emitRun. It carries depth and a per-node RunNode.outer that are always equal, and a reader has to trace every path to learn that, because nothing in this version needs a per-node count.
• Shape: src/markdown/emit/inline-line.ts:384 tryLink, against :368 in codeSpans. Two sources for one number sit in the same file, where the base used depth + 1 in both.
• Locality: src/markdown/emit/inline-line.ts:288 carriedRun, with lineNodes at :37. The nodes are sliced from a new context field by range, so range must be line-absolute; a small cost against the Locality gains.
• Shape: src/markdown/portable/inline-reduction.ts:247 withoutOpeningLinkCode. findIndex with insideRun and an end bound reads directly as the opening link's run; a real gain, smaller than the depth and outer cost.
• Shape: src/markdown/portable/inline-reduction.ts:221 sharedRuns. A seven-line walk threads a RunNode where the depth is already the loop index, replacing a two-line while; it uses the shared rule but reads no easier.
• Self-sufficiency: src/markdown/portable/inline-reduction.ts:165 trimmedEdges. The new comment names the termination measure: whitespace or marks per character, which merging never adds.
• Self-sufficiency: src/markdown/mark-spellings.ts:20 RunNode and :47 insideRun. The comments pay for the file's extra concept, though nodeMarks(node)[outer] at inline-line.ts:238 now needs a jump to another file, and src/markdown/portable/inline-reduction.test.ts:39 has a stray comment above an unrelated test.

2026-10-31T03:20:46Z, PR #112 at 66ad0be, against 8bcf4a5

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	same	same	worse
Inherited architect	same	worse	same	same	worse

Mid A, decided by:

• Locality: src/markdown/emit/skipped-nodes.ts:15 spanRuns: a second copy of inlineRuns' run grouping, kept in step only by a comment, editing spanned in place while it recurses
• Locality: src/markdown/portable/inline-reduction.ts:169 trimmedEdges: reads each leaf's neighbours from a parallel spanned array, so the portable edge rule depends on the writer's spanning
• Locality: src/markdown/emit/inline-line.ts:119 spannedLine: every pass emits a second, spanned line first and throws it away on a fallback or a portable !adf:
• Locality: src/markdown/emit/skipped-nodes.ts:54 holdsEmptyMarks: a one-line exception at the end of a predicate decides whether a node is lent a mark

Inherited architect, decided by:

• Locality: src/markdown/emit/skipped-nodes.ts:16 spanRuns: run grouping in two places that must agree, linked only by a comment naming inlineRuns
• Locality: src/markdown/portable/inline-reduction.ts:169 trimmedEdges: models the writer as always spanning, which spannedLine does only where the line reads back
• Locality: src/markdown/emit/inline-line.ts:119 spannedLine: refuses a portable span by the !adf: prefix of a segment's text, and turns every emission failure into the parted line
• Locality: src/markdown/emit/skipped-nodes.ts:11 spanSkippedNodes: signals nothing spanned by returning the same array, and hides the empty-marks exception from the caller

2026-10-31T07:18:17Z, PR #112 at d554e22, against 8bcf4a5

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	worse	same	worse
Inherited architect	same	worse	worse	same	worse

Mid A, decided by:

• Locality, Shape: src/markdown/emit/inline-line.ts:246 inlineRuns: a spans flag switches it between two modes, and a mutable spanEnd set by spannedTo lets one iteration steer later ones
• Shape: src/markdown/emit/inline-line.ts:230 emitRun: picks the behaviour with a bare positional false, as lendMarks does with true
• Locality: src/markdown/emit/inline-line.ts:291 spannedNodes: decides whether anything spanned by object identity, which holds only because lendMarks pushes originals and only withMark copies
• Locality: src/markdown/emit/inline-line.ts:108 linePass: tries the spanned line first and falls back to the parted line, and spannedLine turns every refusal into a quiet undefined
• Locality: src/markdown/portable/inline-reduction.ts:168 trimmedEdges: judges edge whitespace against the emitter's spannedNodes, so the portable trimming depends on the writer's span rule
• Shape: src/markdown/emit/inline-line.ts:270 spannedTo: one of four new units in a file grown from 390 to 451 lines

Inherited architect, decided by:

• Locality: src/markdown/emit/inline-line.ts:108 linePass: each pass emits two candidate lines, and nothing records which one was written
• Locality: src/markdown/portable/inline-reduction.ts:168 trimmedEdges: imports the emitter's spannedNodes with a fresh carried set, so its verdict and the written line can be judged on different node lists
• Shape: src/markdown/emit/inline-line.ts:246 inlineRuns: one function serves two callers through a boolean that emitRun always passes as false, plus a lookahead cursor
• Shape: src/markdown/emit/inline-line.ts:271 spannedTo: returns start to mean "spans nothing", a sentinel its name does not state, beside spansNode's five-condition expression

2026-11-03T14:18:32Z, PR #112 at 63e3eb2, against 8bcf4a5

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	worse	worse	worse
Inherited architect	same	worse	same	same	worse

Mid A, decided by:

• Locality, Self-sufficiency: src/markdown/portable/inline-reduction.ts:164 trimmedEdges: indexes the emitter's spanned nodes beside merged, relying on one output node per input in order, and on the emitter writing the spanned line
• Locality, Shape: src/markdown/emit/inline-line.ts:288 spannedRuns: reports a span through the mutable spanned.spans, set deep in the recursion, while splicing and rewriting open.nodes in place
• Shape, Locality: src/markdown/emit/inline-line.ts:113 linePass: tries the spanned line then the parted one, and spannedLineRefusal's reasons are a distinction its caller never uses
• Self-sufficiency: src/markdown/emit/inline-line.ts:315 spansNode: maps onto takesMark only after a hop to takenMarkTypes

Inherited architect, decided by:

• Locality: src/markdown/portable/inline-reduction.ts:168 trimmedEdges: reads spanned[index ± 1] beside merged[index], a same-length contract neither end states
• Locality: src/markdown/emit/inline-line.ts:69 portableLineFallback: the reducer judges the parted line while linePass writes the spanned one, so the line judged is no longer the line written
• Locality: src/markdown/emit/inline-line.ts:113 linePass: each pass builds, emits and may discard a rewritten node list, a second path to suspect for a mark gone

2026-11-03T21:46:31Z, PR #112 at f7b8e45, against 0166a71

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	worse	same	worse
Inherited architect	same	worse	worse	worse	worse

Mid A, decided by:

• Locality, Overall: src/markdown/emit/line-escaping.ts:22 MarkRun. depth used to be an unconditional index into each node's marks. It now means that only "on a parted line", so withoutMarks (src/markdown/portable/inline-reduction.ts:257) and endsAlone (src/markdown/emit/unspellable-runs.ts:94) are correct only because of which line they were handed, and neither shows that at its use.
• Locality: src/markdown/emit/inline-line.ts:71 portableLineFallback. The bare false argument is what upholds that depth condition, and the reason sits in another file. linePass tries true first and the reducer's counts use spans: true, so a reader must keep track of which walk each caller gets.
• Locality, Shape, Overall: src/markdown/emit/inline-line.ts:113 linePass. One straight walk became two attempts with a preference. Its helper writesSpanned infers edge whitespace from a !adf: prefix on a syntax segment, so the reader has to work out why that prefix check means whitespace.
• Locality, Shape: src/markdown/emit/inline-line.ts:282 runsSharedWithNext. With countSharedRuns, this is a second recursive walk that the reducer must trust to mirror emitRun. It uses an empty carried set, returns silently past largestNesting, and adds to the emit file's exports.
• Shape: src/markdown/emit/inline-line.ts:247 inlineRuns. The base's simple append loop became an offset-skipping loop plus nodesInRun, which walks past holders and then slices back to them. It reads as correct, but it takes more to follow.
• Shape: src/markdown/portable/inline-reduction.ts:199 leafEdges. The neighbour nodes gave way to number | undefined counts, where undefined means a line edge. They come from the shared array, which trimmedEdges indexes by position, and this is less direct than asking about the neighbour itself.
• Shape: src/markdown/emit/inline-line.ts:40 InlineContext.spans. spans is a boolean here, but in emitMarkedRun it holds the result of codeSpans(nodes), so the name tells a reader less than it should.
• Locality: src/markdown/portable/inline-reduction.ts:168 trimmedEdges. This one reads better. The reducer asks the emitter's own grouping instead of the base's parallel sharedRuns and its warning comment, so the two can't disagree on what a run is. It offsets part of the cost but doesn't flip the ruling.
• Shape: src/markdown/emit/inline-line.ts:430 tryLink. This also reads better, since nodes[0]?.outer ?? 0 says what the link's nodes hold where the base's depth + 1 assumed it. It is too small to move the ruling.

Inherited architect, decided by:

• Locality, Self-sufficiency: src/markdown/emit/inline-line.ts:71 portableLineFallback, together with src/markdown/emit/inline-line.ts:115 linePass and src/markdown/portable/inline-reduction.ts:168 trimmedEdges. The reducer now trims edges by the spanned grouping and judges fallbacks on the parted line, and the writer then prefers the spanned line, so three steps read two different lines. For **Run `npm ci` first** the parted line traces as misflanked and strong should drop, and only adf-to-portable-markdown.test.ts:426 showed otherwise: it survives because emitEmphasis spells the kept edge space as !adf:text, in a portable verdict line whose directive the portable writer would never emit, and no comment states this.
• Locality: src/markdown/emit/inline-line.ts:267 nodesInRun, with src/markdown/emit/line-escaping.ts:22 MarkRun. In the base, a RunNode's outer always equalled the walk's depth, and depth was always an index into a node's marks; now MarkRun.depth is a mark index only on a parted line. withoutMarks trusts that index, and it holds only because linePass and portableLineFallback both route fallbacks through the parted attempt, in another file. tryLink's nodes[0]?.outer ?? 0 is safe only if the first node is always a holder, which the reader has to derive.
• Locality, Shape: src/markdown/portable/inline-reduction.ts:218 edgeDepth, with src/markdown/emit/inline-line.ts:282 runsSharedWithNext. The base's local sharedRuns is gone, a real gain because the grouping now lives in one place. The replacement is a second recursive walk in the emitter, countSharedRuns, which returns silently past largestNesting where emitRun refuses; its result is a count of runs, yet edgeDepth uses it as an index into the leaf's marks, and nothing says why the two agree on a spanned line.
• Shape: src/markdown/emit/inline-line.ts:40 InlineContext. spans and spansLines sit on adjacent lines and mean unrelated things: a run across nodes, and a hard break across lines. spans also means code spans at emitMarkedRun and at inline-reduction.ts:248, while runSpans and codeSpans read as siblings, and the new boolean is passed bare at its call sites.
• Shape: src/markdown/emit/inline-line.ts:247 inlineRuns. The base grew the previous run in place in one loop. The new version uses a next skip cursor, a helper that overshoots, and inside.slice(0, holders) to take back trailing spanned nodes; the rule is sound and a comment explains it, but it takes longer to verify. RunGrouping is satisfied structurally by passing the whole InlineContext.
• Self-sufficiency: spec/flavour.md:568 section Marks, read against src/markdown/emit/inline-line.ts:115 linePass and writesSpanned. The spec says that where the spanned line does not read back "the run breaks there", while the code parts every run on the line. The code's test is "no fallback, and no directive in a flavour without directives", not a read-back to the same nodes, so a reader holding the spec is misled about which one applies.
• Self-sufficiency: src/markdown/mark-spellings.ts:49 insideRun. The base comment gave the footgun: the reducer must ask this grouping or it trims by runs the writers do not spell. The new comment is a bare spec pointer, and no comment on runsSharedWithNext carries that warning forward, so the reason the reducer imports the emitter's walk is unstated in the code.

2026-11-03T22:54:25Z, PR #112 at 8e1ba5d, against 0166a71

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	worse	better	same	worse
Inherited architect	better	worse	better	same	worse

Mid A, decided by:

• Locality: src/markdown/emit/inline-line.ts:129 takeFallback. The first call ignores the fallback it was handed and only flips parted, so what a pass means now depends on hidden mutable state. emitLine (line 97) starts at pass 0 to compensate, and its comment has to explain why. At base (src/markdown/emit/inline-line.ts:130) each call did what its argument said.
• Locality: src/markdown/portable/inline-reduction.ts:264 judgedLine. The reducer now mirrors the writer's spanned-then-parted order and judges two differently trimmed lines. It returns the parted leaves because the fallback's indexes only fit that line, so spellableLine and withoutFallback must keep the two paired. At base spellableLine (line 230) judged and fixed the one current line.
• Locality: src/markdown/portable/inline-reduction.ts:202 runsSharedWithNext, read with leafEdges at line 222 and edgeDepth at line 241. Trimming a leaf's whitespace now needs a run walk over the whole line, delivered through an index-parallel shared array read as shared[index - 1] and shared[index]. At base sharedRuns (line 221) compared a leaf with its neighbour and could be read alone. The new way is the more honest source of truth, but the unit can't be understood without inlineRuns.
• Locality: src/markdown/emit/inline-runs.ts:32 inlineRuns, read with runEnd at line 38. A run is now found by lookahead with an end skip, and crossed nodes keep their outer through insideRun(...) ?? runNode. So outer no longer tracks depth in emitRun, and tryLink must read nodes[0]?.outer. At base (src/markdown/emit/inline-line.ts:232) a run was built by appending to the previous one, and outer was always depth + 1 inside a run.
• Navigation, Shape: src/markdown/emit/inline-runs.ts:19 inlineRuns. Run grouping, RunNode, insideRun and carries now share one file named for what it holds, and writer and reducer both ask it. At base they were split between a private function in inline-line.ts and mark-spellings.ts, which held run concepts in a file about spellings, plus the reducer's own sharedRuns.
• Shape: src/markdown/emit/line-escaping.ts:23 MarkRun. A run now names its mark where it named depth, a positional index into each node's marks. withoutMarks (src/markdown/portable/inline-reduction.ts:294) and endsAlone (src/markdown/emit/unspellable-runs.ts:94) therefore say which mark they mean, with no index to keep aligned.

Inherited architect, decided by:

• Locality, Overall: src/markdown/emit/inline-line.ts:129 takeFallback: the first fallback it is handed is thrown away and only sets parted, so a reader must already know that a verdict on the spanned line says nothing about what the parted line needs. In 0166a71 each call took what it was given. The loop in emitLine now also starts at pass 0 (:97), and readBackPasses counts past it, which adds more order to keep in mind.
• Locality, Overall: src/markdown/portable/inline-reduction.ts:264 judgedLine: it restates the writer's order, spanned then parted, outside the writer, and returns different leaves from the ones it received, so the two orders stay in step only by hand. spellableLine (:250) retries the spanned line on each pass while emitLine keeps parted set once flipped, an asymmetry the reader has to reconcile. In 0166a71 spellableLine made one fallback call on the current line.
• Locality: src/markdown/portable/inline-reduction.ts:241 edgeDepth, with runsSharedWithNext (:202) and countSharedRuns (:209): in 0166a71 sharedRuns answered from a leaf and its neighbour alone. Now the number comes from a recursive walk of the whole line's runs, under a grouping threaded through trimmedEdges. Reading it as "outermost marks" rests on the invariant stated in the comment at edgeDepth.
• Navigation, Shape: src/markdown/emit/inline-runs.ts:19 inlineRuns: "why did this run group so" now has one named file, shared by the writer and the reducer. In 0166a71 the grouping was private to inline-line.ts, and insideRun sat in mark-spellings.ts, a file about spellings. inline-line.ts is lighter for it, though carries at :61 stretches the new file's name a little.
• Shape: src/markdown/emit/line-escaping.ts:23 MarkRun: it holds the mark itself where 0166a71 held a depth index, and that index would be wrong once a run crosses nodes. withoutMarks (inline-reduction.ts:294) and endsAlone (unspellable-runs.ts:94) now read as what they do. This outweighs a smaller cost: depth in emitRun and outer in RunNode now mean different things.

2026-11-04T12:16:56Z, PR #112 at eb92f94, against 0166a71

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	same	better	same	better
Inherited architect	better	same	better	same	better

Mid A, decided by:

• Navigation, Shape, Overall: src/markdown/emit/inline-runs.ts:18 inlineRuns. Run grouping now has its own truthfully named file, holding the walk, the crossing rule and the shared-run count. At base the walk sat inside inline-line.ts, and insideRun and RunNode sat in mark-spellings.ts, a module about spellings, so a run-grouping symptom now leads to one file.
• Shape, Overall: src/markdown/emit/line-escaping.ts:23 MarkRun. A run now names the mark it spells, so withoutMarks (src/markdown/portable/inline-reduction.ts:278) and endsAlone (src/markdown/emit/unspellable-runs.ts:94) read directly. The base's depth index needed a comment and a matching index filter to say the same thing.
• Shape, Overall: src/markdown/emit/inline-line.ts:262 emitLeaf. It takes a PlainRun whose carried is decided once per node in lineSegments. The base asked carries(node, carried, index) from three sites and threaded carried through InlineContext.
• Shape: src/markdown/portable/inline-reduction.ts:248 judgedLine. The counterweight, narrowing the gain without cancelling it: it repeats emitLine's spanned-then-parted order, its fallback indexes name the parted line, which only a comment explains, and runsSharedWithNext lives in emit/ but serves only the reducer.

Inherited architect, decided by:

• Navigation, Shape, Overall: src/markdown/emit/inline-runs.ts:18 inlineRuns. Run grouping now has one home, named for what it holds, so a symptom about a mark split or opened twice leads straight there. At base the same logic sat at inline-line.ts:232, in the middle of the line writer, with RunNode and insideRun in a file about spellings.
• Shape: src/markdown/mark-spellings.ts:32 markSpelling. The file now holds only spellings, so its name tells the truth. The base also kept the run type and insideRun there.
• Shape: src/markdown/emit/line-escaping.ts:23 MarkRun. A run now names its mark instead of an array index. This reads truthfully at withoutMarks (src/markdown/portable/inline-reduction.ts:278), and a reader no longer has to work out what a depth indexes into.

2026-11-04T13:11:07Z, PR #117 at 283cf91, against d5d594b

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	same	same	same	same
Inherited architect	same	same	worse	same	same

Inherited architect, decided by:

• Shape: src/html/node-element.ts:9 NodeElement.textAttribute. Leaving it undefined marks the row as a date, which the type doesn't say; emit and parse both read undefined as "derive the text from timestamp", and the four rows are built three ways.
• Shape: src/html/parse/node-element.ts:36 mismatch. It picks its message from model.leading (datetime for a date, href for a card) in place of the node type, and readNodeElement checks all four kinds in one condition, so a reader works out which checks apply to which kind.

2026-11-04T15:59:34Z, PR #119 at 65f1c2a, against 13a2689

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	same	better	better
Inherited architect	better	better	same	better	better

Mid A, decided by:

• Locality: src/html/node-element.ts:19 nodeElements: each inline node's element, shown text, leading attribute and mismatch message sit in one entry, where the base spread the date, text and card cases over three places
• Locality: src/html/parse/node-element.ts:24 readNodeElement: one generic check runs the writer's own shown and leading.value over the node it built, where the base special-cased the date and branched on the leading name for its message
• Self-sufficiency: src/html/node-element.ts:7 NodeElement: the type's comment states the contract, so a cold reader sees why the reader is safe without opening the spec
• Shape: src/html/node-element.ts:50 card: dense early returns, href is undefined exactly when a card holds both data and url; the type grew from five fields to eight
• Shape: src/html/node-element.ts:62 textElement: hardcodes 'text' while inlineNodes in src/adf/inline-nodes.ts:16 still declares textAttribute: 'text', a second source for one fact

Inherited architect, decided by:

• Locality, Navigation: src/html/parse/node-element.ts:24 readNodeElement: one check built on model.build, model.shown and model.leading.value holds for every node type, where the base's mismatch switched on the leading attribute's kind
• Locality, Self-sufficiency: src/html/node-element.ts:31 inlineCard: the card takes ADF's own vocabulary and card at :50 says why a card holding both data and url rides the carry, where the base pointed at todo.md item 133
• Locality, Navigation: src/html/emit/node-element.ts:7 nodeElement: the date, card and text rules live in one table, so a symptom about one node type leads to one entry
• Shape: src/html/node-element.ts:8 NodeElement: the record grew from five fields to eight, three of them functions, behind a dense one-line contract comment, and the card's build adds url only when data is absent

2026-11-04T18:58:37Z, PR #120 at 755241d, against 81d741d

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	same	same	same	same
Inherited architect	same	worse	same	worse	worse
Maintainability senior	same	worse	same	same	same
Junior A	same	worse	same	same	same

Inherited architect, decided by:

• Locality: src/html/emit/inline.ts:15 inlineHtml: whether a linked card is refused depends on whether its enclosing block rides the carry, which src/html/emit/adf-to-html.ts:123 tryLeafBlock decides far from this file
• Self-sufficiency: src/html/emit/inline.ts:15 inlineHtml: the comment gave "HTML nests no link in a link" as the reason, which the carry's single link refutes; since fixed to name the schema
• Self-sufficiency, Locality: spec/html.md:130 the carry rule: the exception sits beside the lossless markdown writer, which carries the same shape

Maintainability senior, decided by:

• Locality: src/html/emit/inline.ts:15 inlineHtml: whether a linked card is refused or carried depends on tryLeafBlock and inlineElement in src/html/emit/adf-to-html.ts:122, so changing which paragraphs ride the carry changes which documents are refused

Junior A, decided by:

• Locality: src/html/emit/inline.ts:15 inlineHtml: the card refusal fires only where the block around the card is written as an element, and inlineElement at src/html/emit/adf-to-html.ts:131 decides that first, where the base's loop decided each node from the node alone

2026-11-04T19:35:07Z, PR #121 at 78ca66b, against 30f45b7

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	same	same	better
Inherited architect	same	better	same	same	better

Mid A, decided by:

• Locality: src/adf/document.ts:151 refusedShape: one walk refuses the card for every writer and, through carriedNode, both carry readers, where the base held the rule in the HTML emitter alone
• Locality: src/html/emit/inline.ts:11 inlineHtml: its loop no longer opens with a check whose comment relied on which callers reach it

Inherited architect, decided by:

• Locality: src/adf/document.ts:151 refusedShape: the rule against a link-marked card lives in one walk that every writer and both carry readers pass through
• Locality: src/html/emit/inline.ts:15 inlineHtml: the base held the check at the top of the HTML writer's inline loop, with a comment about callers elsewhere

2026-11-08T21:12:40Z, PR #139 at 75606ad, against acc6df4

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	better	better	better	same	better
Inherited architect	same	better	better	same	better

Mid A, decided by:

• Locality, Shape, Overall: src/html/carry-content.ts:21 shownBlocks: one BlockPart compared by reference replaces null sentinels and a mutable current record
• Locality, Shape, Overall: src/markdown/emit/adf-to-markdown.ts:135 spellingMemo: ask and take hold the keep-or-drop rule in one closure, and the caller flag is gone
• Locality, Navigation, Overall: src/adf/document.ts:174 shapeRefusal: one walk refuses every bad shape, where three files did
• Locality, Overall: src/html/data-attributes.ts:42 readDataAttributesAfter: the leading-attribute case leaves nameFault and its hidden invariant
• Locality, Shape: src/html/parse/html-to-adf.ts:105 readBlocks: table state threads through the block loop, which reads harder than before
• Shape: src/markdown/emit/unspellable-runs.ts:30 Verdict: one named verdict per pair replaces a Uint8Array and a separate set
• Shape: src/markdown/mark-spellings.ts:70 commonMarkLink: a named LinkSurroundings object replaces three positional parameters
• Navigation, Shape: src/html/cell-element.ts:22 cellAttributes: table-cell attributes are written and read in one file named for them

Inherited architect, decided by:

• Locality, Shape, Overall: src/markdown/portable/adf-to-portable-markdown.ts:61 reduceToPortable: a refusal's path comes from a source map, so no second run on the raw document
• Locality, Overall: src/adf/document.ts:174 shapeRefusal: schema shape refusals live in one function in the first step
• Locality, Shape, Overall: src/html/carry-content.ts:21 shownBlocks: no sentinel to hold in mind
• Shape: src/markdown/portable/inline-reduction.ts:242 edgeOutcome: a tagged union says what the result means, where undefined did not
• Locality, Shape: src/html/data-attributes.ts:42 readDataAttributesAfter: its own entry point for the leading attribute, chosen openly by the callers
• Shape: src/adf/block-nodes.ts:1 isHeadingLevel: one source for the 1 to 6 range
• Locality: src/html/parse/html-to-adf.ts:95 readBlocks: RowStep, closerOf and the close loop have to be pieced together to follow table state
• Self-sufficiency: src/html/parse/html-to-adf.ts:273 readTable: the thead and tbody branches advance by one length that holds only because both strings are equally long

2026-11-08T22:07:58Z, PR #140 at 27e212c, against b3d136d

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	same	same	same	same
Inherited architect	same	same	same	same	same

2026-11-08T23:20:43Z, PR #141 at 883fe80, against 2766a0a

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	same	same	same	same
Inherited architect	same	same	worse	better	same

Inherited architect, decided by:

• Shape: src/markdown/portable/portable-markdown-to-adf.test.ts:245 refuses what a callout title or body holds that no node spells, at the path of the paragraph or the callout: a third local refused helper, in two variants, beside the ones at :184 and :268
• Shape: src/markdown/portable/adf-to-portable-markdown.test.ts:514 writes a task list nested in the task before it, and more than one beside each other in that task: repeats task-list spellings the existing test at :141 pins, in two places that do not point to each other
• Shape: src/markdown/portable/portable-markdown-to-adf.test.ts:278 reads a task whose marker line holds only breaks and spaces by the lines after it: its first assertion has text on the marker line, so the name does not describe it
• Self-sufficiency: src/markdown/portable/portable-markdown-to-adf.test.ts:232 keeps the target of a link in a title whichever marks sit beside the link: worked input-to-output examples where a reader needed spec/flavour.md open beside the code
• Self-sufficiency: src/markdown/portable/adf-to-portable-markdown.test.ts:495 writes a task list nested to the deepest level the reader carries, and refuses one more at the item holding it: shows in one place that the writer's depth limit lines up with the reader's

2026-11-09T00:23:10Z, PR #143 at 9a8f8b3, against 47b95fb

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	better	better	better
Inherited architect	same	better	better	better	better

Mid A, decided by:

• Locality, Shape: src/markdown/inline-content.ts:518 markedNodes: the steps are calculations now; one array no longer rewritten in place by three steps that each rely on the shape the last left
• Self-sufficiency, Locality: src/markdown/inline-content.ts:551 pieceNodes: == is its literal text at once, where the placeholder's life cycle spanned three functions and a comment
• Shape: src/markdown/inline-content.ts:595 pairingMarks: the name says it returns marks, where markPairings mutated nodes and returned a boolean
• Locality: src/markdown/inline-content.ts:626 highlightDelimiters: reads marks[index] directly and counts every piece's text alike, with no holder special case

Inherited architect, decided by:

• Locality, Self-sufficiency: src/markdown/inline-content.ts:518 markedNodes: marks are computed first as plain data and highlights pair on them, so no placeholder's life cycle is held in mind across three steps
• Locality, Shape: src/markdown/inline-content.ts:595 pairingMarks: pure, returns each piece's marks or undefined on the refusal, where markPairings rewrote nodes and returned a boolean
• Shape, Self-sufficiency: src/markdown/inline-content.ts:626 highlightDelimiters: holder becomes marks and the position special case is gone
• Self-sufficiency: src/markdown/inline-content.ts:550 pieceNodes: its comment says what the delimiter is, where it explained a trick

2026-11-09T00:53:30Z, PR #144 at f822c57519, against 19c1a53dcc

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	better	better	same	better
Inherited architect	same	better	same	same	better

Mid A, decided by:

• Locality, Shape: src/markdown/inline-content.ts:425 closeBracket: it reads the scan and returns a Closing; the base changed the scan at four points as it went
• Locality: src/markdown/inline-content.ts:438 applyClosing: every change a ] makes to the scan sits in one function, and linkOpenersFrom is one expression where the base wrote it twice
• Shape: src/markdown/inline-content.ts:421 Closing: a two-arm type replaces popBracket, literalClose and unopened

Inherited architect, decided by:

• Locality: src/markdown/inline-content.ts:438 applyClosing: every change a ] makes to the scan happens here, linkOpenersFrom one expression keyed on closesLink
• Locality: src/markdown/inline-content.ts:425 closeBracket: it only reads the scan and returns a Closing listing the two outcomes
• Locality: src/markdown/inline-content.ts:179 readToken: the bracket-close case flushes like its neighbours

2026-11-09T04:00:13Z, PR #145 at 3c07509b73, against 1939359cb8

Ruling: better

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	same	same	better	better
Inherited architect	better	same	same	same	better

Mid A, decided by:

• Self-sufficiency, Overall: src/markdown/emit/unspellable-runs.test.ts:11 lines: twelve worked lines give the exact runs a carry or a drop takes, where the base had no direct test of unspellableRuns
• Self-sufficiency: src/markdown/emit/unspellable-runs.test.ts:87 carryRuns: it shows the two call paths that reach unspellableRuns, which the base left to be traced across inline-line.ts and portable/inline-reduction.ts

Inherited architect, decided by:

• Navigation, Overall: src/markdown/emit/unspellable-runs.test.ts:80 lines: a sibling test beside unspellable-runs.ts holds twelve concrete lines under carry and drop, where the base had none

2026-11-09T04:44:01Z, PR #146 at 6f5087c, against 59453f6

Ruling: worse

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	worse	same	same	worse
Inherited architect	same	worse	same	same	worse

Mid A, decided by:

• Locality, Overall: src/markdown/portable/block-spelling.ts:45 tryTaskList: the loop pushes into a placeholder item array that is dropped if the first child is not a task, so blocks vanish silently and whether that can happen is decided in task-list.ts itemParts; the base handled it locally
• Locality: src/markdown/portable/block-spelling.ts:46 tryTaskList: an item alias pushed into items and changed through item.push is more state across loop turns than the base's items.at(-1)

Inherited architect, decided by:

• Locality, Overall: src/markdown/portable/block-spelling.ts:44 tryTaskList: a stand-in item array holds blocks pushed before the first task and loses them without a failure, so its safety rests on reduceTaskList and splitTaskList in other files, where the base never lost a block
• Locality: src/markdown/portable/task-list.ts:42 itemsOf: groups a task and its children up to the next task as the base tryTaskList did, and the new loop breaks that match between the two places

2026-11-09T04:45:44Z, PR #146 at 6030d12, against 59453f6

Ruling: same

Seat	Navigation	Locality	Shape	Self-sufficiency	Overall
Mid A	same	same	same	same	same
Inherited architect	same	same	same	same	same