Skip to content

perf(server): stream static files and keep deployed HTML current - #334

Merged
rynfar merged 3 commits into
pylonfrom
upstream/2026-09-06-static-response-caching
Sep 6, 2026
Merged

perf(server): stream static files and keep deployed HTML current#334
rynfar merged 3 commits into
pylonfrom
upstream/2026-09-06-static-response-caching

Conversation

@rynfar

@rynfar rynfar commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Static requests buffered entire files and sent reusable bundles again on every request. Stream bounded chunks from a request-scoped file handle, cache only hashed assets listed in the build manifest, and support conditional requests for mutable assets. HTML always returns the current app shell so deployments preserving file sizes and timestamps cannot strand clients on obsolete bundle URLs.

Adopts upstream #9669 and the HTML correction from #9799. Pylon auth, routing, compression, remote origins, and desktop identity stay intact. The live-stream portion of #9799 landed separately in #333; the source commit is counted only once.

Validation: the HTML regression reproduced a stale 304 before the correction; all 174 server routing tests pass afterward, including atomic replacement and handle cleanup after GET, HEAD, 304, and cancellation. Server and web typechecks, targeted lint, formatting, and diff checks pass.

Model: GPT-6 Astra. Harness: Codex.


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

t3dotgg and others added 3 commits September 6, 2026 12:59
(cherry picked from commit 27e6cc27fe0f3cff53a44905e40615b7db99c80c)
Selectively ports the HTML validator correction from pingdotgg/t3code commit ce4712d5b04fb998f79fe132245289191147e5d5 (#9799). Restores the Queue test import required after Pylon cleanup.
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Sep 6, 2026
@github-actions

github-actions Bot commented Sep 6, 2026

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 13.8 KiB 13.7 KiB −76 B (−0.5%) 15.1 KiB
Codex Thread snapshot wire 7.1 KiB 7.1 KiB −4 B (−0.1%) 7.3 KiB
Codex Live turn WebSocket wire 6.7 KiB 6.6 KiB −72 B (−1.0%) 7.8 KiB
Codex Live turn WebSocket decoded 57.4 KiB 57.3 KiB −88 B (−0.1%) 66.4 KiB
Codex Live turn messages 10 8 −2 (−20.0%) 21
Claude Total thread wire 13.8 KiB 13.7 KiB −137 B (−1.0%) 15.1 KiB
Claude Thread snapshot wire 7.1 KiB 7.1 KiB −5 B (−0.1%) 7.3 KiB
Claude Live turn WebSocket wire 6.7 KiB 6.6 KiB −132 B (−1.9%) 7.8 KiB
Claude Live turn WebSocket decoded 58.2 KiB 57.4 KiB −884 B (−1.5%) 66.4 KiB
Claude Live turn messages 10 9 −1 (−10.0%) 21

Baseline: 1e7ffff · PR result: 07cd530 · 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: 111.2 KiB
  • Claude decoded thread snapshot: 111.9 KiB

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

@rynfar
rynfar merged commit 89d36dd into pylon Sep 6, 2026
17 checks passed
@rynfar
rynfar deleted the upstream/2026-09-06-static-response-caching branch September 6, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 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.

2 participants