test(desktop): remove redundant keyring module-load test

[t3-code]

summary

remove the redundant import-only test from ChromiumKeys.module.test.ts.

under the same throwing @napi-rs/keyring mock, the remaining macOS keychain-unavailable test imports ChromiumKeys.ts, invokes resolveChromiumKeys, and verifies the concrete typed failure. the removed test only checked that the identical cached import resolved, so it covered no additional implementation, branch, or failure mode.

verification

• before: vp test run apps/desktop/src/ipc/methods/preview.test.ts apps/desktop/src/preview/BrowserImport/ChromiumKeys.module.test.ts (2 files, 10 tests passed)
• after: vp test run apps/desktop/src/ipc/methods/preview.test.ts apps/desktop/src/preview/BrowserImport/ChromiumKeys.module.test.ts (2 files, 9 tests passed)
• vp run knip:check (passed)
• vp run --filter @t3tools/desktop typecheck (passed, existing suggestions only)
• vp lint apps/desktop/src/ipc/methods/preview.test.ts apps/desktop/src/preview/BrowserImport/ChromiumKeys.module.test.ts --report-unused-disable-directives (passed)
• vp fmt --check apps/desktop/src/ipc/methods/preview.test.ts apps/desktop/src/preview/BrowserImport/ChromiumKeys.module.test.ts (passed)
• git diff --check (passed)
• focused diff secret scan (passed)

model: gpt-5.6-sol
harness: hermes agent

[macroscopeapp]

Approvability

Verdict: Approved at 8a5ad6b

Macroscope's review found this PR approvable — This is a four-line deletion from a single ignored test path that removes redundant test coverage without changing product runtime behavior, defaults, or static-analysis configuration.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

^{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.5 KiB	13.5 KiB	−1 B (−0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.1 KiB	7.1 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.4 KiB	−4 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	−3 B (−0.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.1 KiB	7.1 KiB	+3 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	−6 B (−0.1%)	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: b21c545 · PR result: 8a5ad6b · 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.

Dismissing prior approval to re-evaluate 8a5ad6b