Skip to content

test(server): stabilize daemon and Git fixture timing - #855

Merged
rynfar merged 3 commits into
pylonfrom
fix/prime-daemon-postkill-test-timeout-20260925
Sep 25, 2026
Merged

rynfar merged 3 commits into
pylonfrom
fix/prime-daemon-postkill-test-timeout-20260925

Conversation

@rynfar

@rynfar rynfar commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Two unrelated timing hazards were exposed by catch-up CI. The Prime daemon post-kill receipt test gave private-socket cleanup only 5 ms before it could reach the intended assertion. The PR-template fixture allowed Git automatic maintenance to write object packs while its scoped temporary repository was being removed.

Give the Prime test 250 ms for socket cleanup while still bounding the hung receipt, and disable auto-GC inside the PR-template test repository. Production daemon behavior and application Git settings are unchanged. The dedicated socket-cleanup timeout test remains at 5 ms.

Validation: 49 focused tests across both suites passed; server typecheck passed. The original failures are in #852 Test Server 2 and this PR's first Test Server 3 run. Fresh exact-head CI is running.

GPT-6 Sol via Pylon Codex harness.

@vercel

vercel Bot commented Sep 25, 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 25, 2026 5:20am UTC

@github-actions

github-actions Bot commented Sep 25, 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.9 KiB 14.0 KiB +60 B (+0.4%) 15.1 KiB ✅
Codex Thread snapshot wire 7.2 KiB 7.2 KiB +12 B (+0.2%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.7 KiB 6.7 KiB +48 B (+0.7%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 58.0 KiB 58.0 KiB +44 B (+0.1%) 66.4 KiB ✅
Codex Live turn messages 8 9 +1 (+12.5%) 21 ✅
Claude Total thread wire 13.9 KiB 14.0 KiB +69 B (+0.5%) 15.1 KiB ✅
Claude Thread snapshot wire 7.2 KiB 7.2 KiB +9 B (+0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.7 KiB 6.8 KiB +60 B (+0.9%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.8 KiB 58.9 KiB +44 B (+0.1%) 66.4 KiB ✅
Claude Live turn messages 8 9 +1 (+12.5%) 21 ✅

Baseline: 36ce2f8 · PR result: d026156 · 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 changed the title test(server): stabilize post-kill exit timeout assertion test(server): stabilize daemon and Git fixture timing Sep 25, 2026
@rynfar
rynfar merged commit 0a4fa62 into pylon Sep 25, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 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