Skip to content

ci: only main pushes start the fork nightly - #130

Merged
jayleaton merged 1 commit into
mainfrom
ci/nightly-main-only
Oct 24, 2026
Merged

jayleaton merged 1 commit into
mainfrom
ci/nightly-main-only

Conversation

@jayleaton

Copy link
Copy Markdown
Owner

Every push to any branch started a fork nightly run. Its jobs skipped, but the run still took the single pending slot of the fork-nightly-release concurrency group, so each PR-branch push cancelled main's queued nightly. Today the nightly carrying #123/#125/#126 was cancelled twice (runs 40072672650, 40072695260).

Fix: trigger on pushes to main only. workflow_dispatch is unchanged.

Claude Opus 5.5 via Claude Code (T3 Captain).

🤖 Generated with Claude Code

Pushes to any branch started a fork nightly run. Its jobs skipped, but the run
still took the single pending slot of the fork-nightly-release concurrency
group, so every PR-branch push cancelled main's queued nightly. On 2026-10-24
the nightly carrying #123/#125/#126 was cancelled twice this way.

Trigger on pushes to main only; workflow_dispatch still works for other refs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS labels Oct 24, 2026
@github-actions

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 5.0 KiB 5.0 KiB +7 B (+0.1%) 6.8 KiB ✅
Codex Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Codex Live turn WebSocket wire 1.2 KiB 1.2 KiB +7 B (+0.6%) 2.0 KiB ✅
Codex Live turn WebSocket decoded 20.9 KiB 20.9 KiB 0 B (0.0%) 29.3 KiB ✅
Codex Live turn messages 2 2 0 (0.0%) 8 ✅
Claude Total thread wire 5.0 KiB 5.0 KiB +7 B (+0.1%) 6.8 KiB ✅
Claude Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Claude Live turn WebSocket wire 1.2 KiB 1.2 KiB +7 B (+0.6%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 21.2 KiB 21.2 KiB 0 B (0.0%) 29.3 KiB ✅
Claude Live turn messages 2 2 0 (0.0%) 8 ✅

Baseline: 5b6bff2 · PR result: e92ce23 · 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.

@jayleaton
jayleaton merged commit 130e9ed into main Oct 24, 2026
25 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