[t3-code-bot]

summary

remove the complete leaves short text alone test from packages/shared/src/String.test.ts. it repeats the same no-truncation branch already checked by stronger retained cases. no production changes.

removal evidence

removed	retained coverage	implementation
truncate("hello", 50) === "hello"	truncate("👍👍👍", 5) === "👍👍👍" covers a strictly shorter input, including unicode code-point handling; truncate(" hello ", 5) === "hello" covers the inclusive boundary and trimming	String.ts:2-5 trims, builds code points, and returns the trimmed input when characters.length <= maxLength

this is not the default-argument test: the removed call explicitly supplies 50. the retained over-limit emoji case still checks actual truncation. the deleted test adds no distinct branch, edge case, integration boundary, or production path.

broader audit included mobile/web stack navigation, terminal synchronization, shared/re-exported endpoint and project helpers, codex auth callbacks, diff paging/highlighting, composer layout, oauth scopes, and cached-test tooling. similar-looking stack tests exercise separate implementations; callback and re-export tests without canonical duplicates remain. no entire file was proven valueless, so none was removed merely to enlarge the diff.

verification

• started from clean current origin/main at 0a9abddaee00 on a new branch.
• vp test run packages/shared/src/String.test.ts: before 3 passed; after 2 passed. both initial runs reported unrelated missing workspace-package pre-transform errors despite exiting 0.
• repaired the stale dependency installation with vp install --frozen-lockfile (exit 0).
• vp run check:changed: final exit 0. formatting, lint, both knip passes, affected/dependent typechecks (35 packages), and selected tests (2 files, 4 tests) passed. typecheck emitted non-failing schema-brand suggestions.
• the initial check:changed failed on stale dependencies, missing effect-tsgo, and license-generation errors; all resolved after installation and the final run above passed.
• git diff --check: passed. reviewed the full diff: only one complete test and its separator removed, no secrets or production edits.

scheduled useless-test audit automation; bot-only authorship, no human co-author.

model: gpt-6.1-sol. harness: hermes agent.

[macroscopeapp]

Approvability

Verdict: Approved at 01f86a3

Macroscope's review found this PR approvable — This removes a redundant truncation test within the repository’s ignored scope. No production behavior, product defaults, or diagnostic-suppression directives are changed.

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	5.0 KiB	5.0 KiB	0 B (0.0%)	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.2 KiB	0 B (0.0%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	21.5 KiB	21.5 KiB	0 B (0.0%)	24.4 KiB	✅
Codex	Live turn messages	1	1	0 (0.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	0 B (0.0%)	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.2 KiB	1.2 KiB	0 B (0.0%)	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	1	1	0 (0.0%)	8	✅

Baseline: 41443e5 · PR result: 01f86a3 · 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.