Skip to content

fix(ci): drop fork allowlist entries duplicated by #78 and #80 - #81

Merged
lukemaj merged 1 commit into
mainfrom
fix/fork-check-t3-json
Sep 29, 2026
Merged

lukemaj merged 1 commit into
mainfrom
fix/fork-check-t3-json

Conversation

@lukemaj

@lukemaj lukemaj commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

What: main's fork check passes again by removing allowlist entries that #78 and #80 both added.
Why: #78 and #80 fixed the same fork-check gaps at the same time, so AGENTS.md and ThreadPullRequestsPanel.tsx were each listed twice and fork-features fails with issues-links: duplicate upstreamFiles.
So what: Nothing further; #80's entries stay as the single owners.

Elon record

Proof

  • bash scripts/fork-check.sh: exit 0.

🤖 Generated with Claude Code

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

github-actions Bot commented Sep 29, 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.5 KiB 13.5 KiB −8 B (−0.1%) 15.1 KiB ✅
Codex Thread snapshot wire 7.1 KiB 7.1 KiB −2 B (−0.0%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.4 KiB 6.4 KiB −6 B (−0.1%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 56.2 KiB 56.2 KiB 0 B (0.0%) 66.4 KiB ✅
Codex Live turn messages 9 9 0 (0.0%) 21 ✅
Claude Total thread wire 13.5 KiB 13.5 KiB −14 B (−0.1%) 15.1 KiB ✅
Claude Thread snapshot wire 7.1 KiB 7.1 KiB +2 B (+0.0%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.4 KiB 6.4 KiB −16 B (−0.2%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 57.0 KiB 57.0 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 9 9 0 (0.0%) 21 ✅

Baseline: cd93d8a · PR result: e97a327 · 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: 113.9 KiB
  • Claude decoded thread snapshot: 114.6 KiB

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

#78 and #80 fixed the same fork-check gaps concurrently; together they listed
AGENTS.md and ThreadPullRequestsPanel.tsx twice, which fails fork-features.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@lukemaj
lukemaj force-pushed the fix/fork-check-t3-json branch from 1e95314 to e97a327 Compare September 29, 2026 17:14
@lukemaj lukemaj changed the title fix(ci): allowlist the t3.json icon edit from #77 under branding fix(ci): drop fork allowlist entries duplicated by #78 and #80 Sep 29, 2026
@github-actions github-actions Bot added size:S and removed size:XS labels Sep 29, 2026
@lukemaj
lukemaj merged commit 7bb6a21 into main Sep 29, 2026
18 of 19 checks passed
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