[t3dotgg]

The v0.1.5 release failed at "Finalize release". Its git commit ran our pre-commit hook. Since #26167, that hook runs knip when a package.json is staged. The finalize job installs only two workspaces, so knip reported most of the repo as unused and the commit failed (run).

The CI bot commits now use --no-verify. The release step already formats the files it commits. The mobile EAS version bump commits a staged .ts file, so it had the same problem. It gets the same fix.

For v0.1.5, I pushed the version bump to main manually and reran the failed jobs.

🤖 Generated with Claude Code (Claude Opus 5.5)

[macroscopeapp]

Approvability

Verdict: Needs Human Approval

Macroscope's review found this PR requires human approval — Release and mobile version-bump commits now bypass the pre-commit hook, including its lint and knip checks. Human review is required to authorize that static-analysis bypass.

No code changes detected at 613e497. Prior analysis still applies.

^{You can add or adjust custom eligibility rules. Learn more.}

[github-actions]

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	+6 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.3 KiB	+6 B (+0.5%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	21.6 KiB	21.6 KiB	0 B (0.0%)	24.4 KiB	✅
Codex	Live turn messages	2	2	0 (0.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.1 KiB	+6 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.3 KiB	1.3 KiB	+6 B (+0.5%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.9 KiB	21.9 KiB	0 B (0.0%)	24.4 KiB	✅
Claude	Live turn messages	2	2	0 (0.0%)	8	✅

Baseline: bf8e132 · PR result: 613e497 · 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.

[coderabbitai]

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: fix/release-commit-skips-hooks
• File: .coderabbit.config.ts
• Commit: 613e497
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Team
• Run ID: 945169ce-9e47-481b-be5d-48d3d9e74700

Commits

Reviewing files that changed from the base of the PR and between fd0bb50 and 613e497.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• Trigger review

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: fix/release-commit-skips-hooks
• File: .coderabbit.config.ts
• Commit: fd0bb50
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Team
• Run ID: f0eb8f4f-c171-4b60-8e1c-60a0e1b6cd07

Commits

Reviewing files that changed from the base of the PR and between 37fbefd and fd0bb50.

Files selected for processing: 2

• .github/workflows/mobile-eas-production.yml
• .github/workflows/release.yml

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.

Walkthrough

Both production workflows now add --no-verify to commits that record version changes.

Changes: Version commit workflows

Layer / File(s)	Summary
Skip hooks for version commits .github/workflows/mobile-eas-production.yml, .github/workflows/release.yml	Both workflows use git commit --no-verify for their version-change commits.

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: Minimal · previous review fd0bb

The automated version commits have no remaining merge-blocking risk.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly identifies the release version-bump CI failure caused by the pre-commit hook and matches the changes in both workflows.
Description check		The description clearly explains the release failure, the cause, the --no-verify fix, and the related mobile EAS change. It also references the failed run and manual recovery. It does not use the temp…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description clearly explains the release failure, the cause, the --no-verify fix, and the related mobile EAS change. It also references the failed run and manual recovery. It does not use the template headings or provide explicit verification results and approval details, but the core information is present.

Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts