Skip to content

fix(server): hold streamed section titles with content - #869

Merged
rynfar merged 1 commit into
pylonfrom
upstream/2026-09-26-stream-section-titles
Sep 27, 2026
Merged

rynfar merged 1 commit into
pylonfrom
upstream/2026-09-26-stream-section-titles

Conversation

@rynfar

@rynfar rynfar commented Sep 27, 2026 •

Copy link
Copy Markdown
Collaborator

Streamed Markdown headings and standalone bold section titles could appear by themselves while the following paragraph, list, or code block was still arriving. Hold those titles until a safe boundary includes content below them. An unindented heading still releases the completed paragraph above it.

Selectively adapts T3 a36af0637a63ebf316beb5db5670be226df01cbc (#13504) to Pylon's existing natural-boundary splitter. Existing final-message flush and bounded-buffer behavior remain in effect. No wire contract or client rendering changes.

Verification: vp test run apps/server/src/orchestration/Layers/ProviderRuntimeIngestion.test.ts (135 passed); pnpm --filter t3 typecheck; scoped vp lint (existing unused ProjectId import warning); scoped vp fmt --check; git diff --check.

Model: GPT-6 Sol (low reasoning), overseen by Codex. Harness: Pylon Codex.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Adapt upstream a36af0637a63ebf316beb5db5670be226df01cbc for Pylon streaming boundaries.
@vercel

vercel Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
pylon-marketing Ignored Ignored Preview Sep 27, 2026 5:32am UTC

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S labels Sep 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 14.0 KiB 14.0 KiB −10 B (−0.1%) 15.1 KiB ✅
Codex Thread snapshot wire 7.2 KiB 7.3 KiB +16 B (+0.2%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.7 KiB 6.7 KiB −26 B (−0.4%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 58.0 KiB 58.0 KiB 0 B (0.0%) 66.4 KiB ✅
Codex Live turn messages 9 9 0 (0.0%) 21 ✅
Claude Total thread wire 14.0 KiB 14.0 KiB −38 B (−0.3%) 15.1 KiB ✅
Claude Thread snapshot wire 7.3 KiB 7.3 KiB −5 B (−0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.8 KiB 6.7 KiB −33 B (−0.5%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.9 KiB 58.9 KiB −44 B (−0.1%) 66.4 KiB ✅
Claude Live turn messages 10 9 −1 (−10.0%) 21 ✅

Baseline: 324d708 · PR result: 00a50ea · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 115.7 KiB
  • Claude decoded thread snapshot: 116.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@rynfar
rynfar merged commit 607cb42 into pylon Sep 27, 2026
19 checks passed
@rynfar
rynfar deleted the upstream/2026-09-26-stream-section-titles branch September 27, 2026 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant