Skip to content

test: repair fork CI fixtures and lint violations - #51

Merged
alexcstark merged 2 commits into
mainfrom
stark/fix-fork-ci-regressions
Oct 7, 2026
Merged

alexcstark merged 2 commits into
mainfrom
stark/fix-fork-ci-regressions

Conversation

@alexcstark

@alexcstark alexcstark commented Oct 7, 2026 •

Copy link
Copy Markdown
Owner

Restored fork CI exposed stale test fixtures and six lint violations that prevented validation from completing. This repairs launcher discovery and SSH stop fixtures, restores the real DOM for the empty timeline test, and represents blocking user input as a waiting run in notification fixtures. It also makes the unused process identity helper private, uses the existing 11px font token, and applies the required Effect catchTags form without changing error handling.

Validation: 185 focused tests pass (one Windows-only test skipped on macOS), scoped TypeScript checking passes, and focused lint and formatting checks pass. The failing fixture cases were reproduced before fixing them. CI runs the complete test, typecheck, build, lint, and release smoke checks.

Model: GPT-6. Harness: Codex.

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

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire — 5.0 KiB — 6.8 KiB ✅
Codex Thread snapshot wire — 3.8 KiB — 4.9 KiB ✅
Codex Live turn WebSocket wire — 1.2 KiB — 2.0 KiB ✅
Codex Live turn WebSocket decoded — 20.9 KiB — 29.3 KiB ✅
Codex Live turn messages — 2 — 8 ✅
Claude Total thread wire — 5.0 KiB — 6.8 KiB ✅
Claude Thread snapshot wire — 3.8 KiB — 4.9 KiB ✅
Claude Live turn WebSocket wire — 1.2 KiB — 2.0 KiB ✅
Claude Live turn WebSocket decoded — 21.2 KiB — 29.3 KiB ✅
Claude Live turn messages — 2 — 8 ✅

Baseline: unavailable · PR result: 569b985 · 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: 108.5 KiB
  • Claude decoded thread snapshot: 108.8 KiB

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

@alexcstark alexcstark changed the title test: repair fixtures exposed by restored fork CI test: repair fork CI fixtures and lint violations Oct 7, 2026
@github-actions github-actions Bot added size:S and removed size:XS labels Oct 7, 2026
@alexcstark
alexcstark merged commit c539bf5 into main Oct 7, 2026
26 of 27 checks passed
@alexcstark
alexcstark deleted the stark/fix-fork-ci-regressions branch October 7, 2026 14:23
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