fix(copilot): stop the special-tag parser discarding text it cannot resolve - #5952

Merged

j15z merged 34 commits into

fix/unclosed-special-tags

Jul 29, 2026

Commits on Jul 29, 2026

fix(copilot): render unclosed special tags as text on completed messages
j15z
committed
improvement(copilot): show text as soon as an unclosed tag cannot resolve
j15z
committed
fix(copilot): ignore tag syntax quoted inside a JSON tag body
j15z
committed
fix(copilot): keep prose a mispaired tag would swallow, and bail on dead JSON
j15z
committed
fix(copilot): keep prose a tag wrapped instead of a JSON payload
j15z
committed
refactor(copilot): resolve each special tag through four named outcomes
j15z
committed
test(copilot): pin the no-closing-tag case as lossless
j15z
committed
fix(copilot): stop the literal path flashing payloads and rescanning the buffer
j15z
committed
refactor(copilot): drop the JSON-prefix wrapper the split orphaned

j15z
and
claude
committed
refactor(copilot): drop the nested-marker scan the JSON rule already covers

j15z
and
claude
committed
fix(copilot): stop the rescan deleting quoted text, and bound its cost

j15z
and
claude
committed
test(copilot): assert what a reader sees, and pin the unclosed-thinking trade

j15z
and
claude
committed
docs(copilot): name the scan window's blind spot instead of glossing it

j15z
and
claude
committed
fix(copilot): make the indexOf cache correct for any call order

j15z
and
claude
committed
fix(copilot): resume at the window edge, not past the close, on a truncated body

j15z
and
claude
committed
fix(copilot): only discard a body that actually parsed as JSON

j15z
and
claude
committed
fix(copilot): apply the nesting rule to a prose body on the matched-pair path too

j15z
and
claude
committed
test(copilot): assert parser invariants over generated messages

j15z
and
claude
committed
fix(copilot): strip a stray null byte from the test file

j15z
and
claude
committed
refactor(copilot): split resolveTagAt into budget, classification, and resume

j15z
and
claude
committed
fix(copilot): stop backtick stripping from unbalancing a whole message

j15z
and
claude
committed
fix(copilot): apply the same backtick guard to the balanced unwrap

j15z
and
claude
committed
fix(copilot): pair backticks the way markdown does, and bound the scan

j15z
and
claude
committed
fix(copilot): address the confirmed review findings

j15z
and
claude
committed
refactor(copilot): quality pass over the parser and its tests

j15z
and
claude
committed
docs(copilot): reattach an orphaned docstring and cut comment noise

j15z
and
claude
committed
fix(copilot): sanitize backticks in one pass so a flush code span survives

j15z
and
claude
committed
fix(copilot): rewind the scan-window resume so a straddling opener still parses

j15z
and
claude
committed
fix(copilot): trust the raw body for markers once it is known not to be JSON

j15z
and
claude
committed
refactor(copilot): share the scaling-ratio harness between the two perf tests

j15z
and
claude
committed
docs(copilot): correct a resume comment the rewind fix left inaccurate

j15z
and
claude
committed
test(copilot): pin the blanking rule with a body staging cannot recover

j15z
and
claude
committed
test(copilot): cover every tag in the property invariants, and pin that it stays that way

j15z
and
claude
committed
docs(copilot): state the actual bound on the thinking nesting rule

j15z
and
claude
committed