Skip to content

Keep parsed Markdown in transcript ranges; restore the reader's place - #588

Merged
trissim merged 11 commits into
mainfrom
fix/native-participant-capture-20261008
Oct 9, 2026
Merged

trissim merged 11 commits into
mainfrom
fix/native-participant-capture-20261008

Conversation

@trissim

@trissim trissim commented Oct 9, 2026 •

Copy link
Copy Markdown

Codex work, mostly replaced. 582 of 1930 code lines it added are still in main.

Parsed Markdown source is now carried through the transcript range and fragment, and message views that are thrown away are rebuilt from that source. A missing document now fails where it is supplied, and the retry on interaction refresh and the quiet reparse of a broken stored parse are removed. The reader's position is captured before focus or hiding changes the scene, so an offscreen reader can be restored. The change spans about 30 files, mostly prepared_markdown.py, viewport_body.py, history_anchor.py, streaming_markdown.py and transcript_history.py.

trissim added 11 commits October 8, 2026 19:55
Default conversation syntax now uses the original native acquired document producer and typed rendering tasks. Source ranges keep real prefixes and acquire native controls only for interaction. Body resources own intrinsic extent, displayed source membership, width invalidation and retained rows; membership changes rematch original CSS and only equal acquired values admit retained paint. Resize/style preparation borrows original parser/link data rather than reacquiring it.

Four changed modules compile; complete source/tests/tools parse: 744 modules, zero omissions. Actual saved ACP source App reached process transport but failed on native speedup Spacing pickling before wheel/reader checks. Native geometry owner is fixing that boundary. Source checkpoint only; no installed gain or completed interaction/reader acceptance claim.
Integrate Erdos session/page lifetime handoff. The operational session retains the original prepared page objects, source scope and admitted ranges while disposable native publishers detach. View-owned transcripts still close on detach through their existing base behavior; the session declaration supplies retention polymorphically. Reader intent binds the actual source fragment and its row before retirement awaits, and restores only after committed placement. New edge/End acquisition chooses its requested range independently of parked restoration.

Eight handoff modules compile; all frontend source compiles and diff check passes. Prior genuine reader-return row mismatch remains negative; latest matched combined saved App reader was unreached due native geometry transport, fixed separately by the native owner. Filtered-prefix, rewrap character anchoring and rendered-paint retention remain unqualified. No test, install or provider attempt repeated for this source checkpoint.
Copilot AI balanced review requested due to automatic review settings October 9, 2026 04:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@trissim
trissim merged commit defe8bb into main Oct 9, 2026
1 of 2 checks passed
@trissim trissim changed the title Retain acquired document source and restore its original reader Keep parsed Markdown in transcript ranges; restore the reader's place Oct 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants