Skip to content

Use the Markdown collector as the streaming source of truth - #34197

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/0aca912c53080267a2d12ad397e548641171942a
Jul 19, 2026
Merged

Use the Markdown collector as the streaming source of truth#34197
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/0aca912c53080267a2d12ad397e548641171942a

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 19, 2026

Copy link
Copy Markdown

Use the Markdown collector as the streaming source of truth

What changed

  • Return committed byte ranges from MarkdownStreamCollector so the streaming controller can render and scan the collector's source directly.
  • Transfer the complete source out of the collector during finalization, preserving the final unterminated line while removing the controller's duplicate source buffer.
  • Cover finalization of an empty collector and update streaming tests to validate the collector-backed source.

## What changed

- Return committed byte ranges from `MarkdownStreamCollector` so the streaming controller can render and scan the collector's source directly.
- Transfer the complete source out of the collector during finalization, preserving the final unterminated line while removing the controller's duplicate source buffer.
- Cover finalization of an empty collector and update streaming tests to validate the collector-backed source.

GitOrigin-RevId: 0aca912c53080267a2d12ad397e548641171942a
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/0aca912c53080267a2d12ad397e548641171942a branch from 7ea2d6c to 3dd3c5d Compare July 19, 2026 16:52
@copyberry
copyberry Bot merged commit 3dd3c5d into main Jul 19, 2026
32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/0aca912c53080267a2d12ad397e548641171942a branch July 19, 2026 16:53
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant