refactor(mobile): keep review reset hashing private

[juliusmarminge]

The native review reset hash was exported and re-exported only for tests of its string format. Those assertions did not verify token invalidation when diff content changes.

Keep the hash private and remove its unused hook re-export. Test reset-key changes through the live builder for same-length content changes, different row counts, and cleared diffs. The hash algorithm and key representation are unchanged.

Verification: 7 focused reset-key and native-view tests passed before, 6 after. Mobile typecheck, targeted lint/format, and diff checks passed. No visual change.

Model: gpt-6 astra. Harness: Codex in T3 Code.

Note

Make hashReviewDiffKey private in reviewDiffBridgeKeys module

Removes hashReviewDiffKey from the public API by dropping its export and the re-export in useNativeReviewDiffBridge.ts. The function itself stays unchanged. Tests in reviewDiffBridgeKeys.test.ts no longer import or test the hash directly; instead they verify that buildNativeReviewTokensResetKey returns a different key when diff contents, row count, or a null diff change.

^{Macroscope summarized bb9fe60.}

[cursor]

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

[macroscopeapp]

Approvability

Verdict: Approved at bb9fe60

Macroscope's review found this PR approvable — This small refactor makes the review-diff hashing helper private while preserving reset-key computation and the public builder used by the hook. The remaining changes are test-only, and no in-repository consumers or production behavior are affected.

^{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	13.6 KiB	13.5 KiB	−54 B (−0.4%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.5 KiB	−56 B (−0.8%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.1 KiB	57.0 KiB	−88 B (−0.2%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.6 KiB	+12 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−13 B (−0.2%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.6 KiB	+25 B (+0.4%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.9 KiB	+44 B (+0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	10	+1 (+11.1%)	21	✅

Baseline: c059d09 · PR result: bb9fe60 · 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.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

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