test(web): remove duplicate lookup assertion

[t3-code]

problem

claimWorkspaceBasenameLookup's single-claim test called the same predicate twice in a row and asserted true both times.

change

removed only the second, identical assertion.

evidence

• the predicate is a pure comparison between its captured sequence and the current sequence. calling it twice without creating another claim exercises the exact same branch twice.
• the only production caller invokes the predicate once, after its asynchronous workspace search settles.
• the first assertion remains, so the useful single-claim behavior stays covered.

verification

• before: ./node_modules/.bin/vp test run apps/web/src/workspaceBasenameLookup.test.ts passed, 1 file and 10 tests
• after: ./node_modules/.bin/vp test run apps/web/src/workspaceBasenameLookup.test.ts passed, 1 file and 10 tests
• ./node_modules/.bin/vp lint apps/web/src/workspaceBasenameLookup.test.ts --report-unused-disable-directives passed
• ./node_modules/.bin/vp run --filter @t3tools/web typecheck passed
• ./node_modules/.bin/vp fmt --check apps/web/src/workspaceBasenameLookup.test.ts passed
• git diff --check passed

made with gpt-5.6-sol via hermes agent.

---

Note

Low Risk
Test-only change with no production code or behavior impact.

Overview
Test cleanup in workspaceBasenameLookup.test.ts: the "stays valid while it is the only claim" case no longer asserts the claim predicate twice with the same expectation.

The remaining assertion still verifies that a lone claimWorkspaceBasenameLookup() returns a predicate that evaluates to true. The dropped call duplicated the same branch with no extra claim or state change.

^{Reviewed by Cursor Bugbot for commit 9cd1dc2. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove duplicate assertion in claimWorkspaceBasenameLookup test

Removes a redundant second call to the same claim function in the "stays valid while it is the only claim" test in workspaceBasenameLookup.test.ts. The test now asserts validity once instead of twice.

^{Macroscope summarized 9cd1dc2.}

[macroscopeapp]

Approvability

Verdict: Approved 9cd1dc2

Test-only change that removes a duplicate assertion line. No runtime behavior is affected.

^{You can customize Macroscope's approvability policy. 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	12.6 KiB	12.6 KiB	−24 B (−0.2%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.3 KiB	6.3 KiB	−3 B (−0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.3 KiB	−21 B (−0.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	51.8 KiB	51.8 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	16	16	0 (0.0%)	21	✅
Claude	Total thread wire	12.6 KiB	12.6 KiB	0 B (0.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.3 KiB	6.3 KiB	+8 B (+0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.3 KiB	6.3 KiB	−8 B (−0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	52.7 KiB	52.7 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	16	16	0 (0.0%)	21	✅

Baseline: a4cc136 · PR result: 9cd1dc2 · 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: 99.9 KiB
• Claude decoded thread snapshot: 100.6 KiB

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