Skip to content

fix(ci): main passes the fork stack and unused-code checks again - #80

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

lukemaj merged 1 commit into
mainfrom
fix/main-ci-fork-check

Conversation

@lukemaj

@lukemaj lukemaj commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

What: main passes the Fork Stack Model and Check (knip) jobs again.
Why: #70 and #77 edited upstream files (AGENTS.md, ThreadPullRequestsPanel.tsx, t3.json) without allowlisting them, and #76 exported LINKED_ISSUE_STATES_REFRESH_MS, which nothing imports.
So what: Merges once CI is green; no follow-up needed.

Changes

  • scripts/fork-upstream-edits.txt and docs/fork-features.md: t3.json -> branding, AGENTS.md -> fork-maintenance, ThreadPullRequestsPanel.tsx -> issues-links.
  • apps/web/src/state/issues.ts: the constant is no longer exported.

Proof

  • bash scripts/fork-check.sh: fork-features: OK (18 features). fork-check: OK.
  • knip runs in CI.

Elon record

Model: Claude Opus 5.5, Claude Code in T3 Code.

🤖 Generated with Claude Code

#70 and #77 edited AGENTS.md, ThreadPullRequestsPanel.tsx and t3.json
without allowlisting them, and #76 exported a constant nothing imports.
The three files are listed with their feature owners and the constant is
module-local.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S labels Sep 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.5 KiB 13.5 KiB +3 B (+0.0%) 15.1 KiB ✅
Codex Thread snapshot wire 7.1 KiB 7.1 KiB +9 B (+0.1%) 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 +30 B (+0.2%) 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.4 KiB 6.4 KiB +35 B (+0.5%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 57.0 KiB 57.0 KiB +44 B (+0.1%) 66.4 KiB ✅
Claude Live turn messages 8 9 +1 (+12.5%) 21 ✅

Baseline: f8a22e9 · PR result: bec20a5 · 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.

@lukemaj
lukemaj merged commit cd93d8a into main Sep 29, 2026
18 checks passed
@lukemaj
lukemaj deleted the fix/main-ci-fork-check branch September 29, 2026 17:07
lukemaj added a commit that referenced this pull request Sep 29, 2026
#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 added a commit that referenced this pull request Sep 29, 2026
#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>
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