[topcheer]

deadcode r84 - 5 symbols + dedicated-only tests (-213 lines)

Baseline: fresh worktree @ origin/main (d8050d4, #4347+#4350 both in), deadcode -tags goolm ./... + repo-wide grep dual-verify.

Removed (100% confirmed: zero prod refs, tests existed only to test the dead symbol itself)

• internal/a2a: isToolAllowed + TestIsToolAllowed - nil-allow-all helper, zero prod refs
• internal/tool: SetPrecommitGateEnabled + test - config setter never wired; precommitGateEnabled var stays live
• internal/tool: SetPostEditDiagEnabled + test - same class; var stays live
• internal/tool: LoadProtectedPatternsFromFile + test - .ggcode/protect loader never wired; orphaned os import pruned
• internal/tool: readFileRange + read_file_range_test.go (6 tests all dedicated to it) - thin wrapper over live readFileRangeWithOptions

Kept - classification ledger (r83 ledger continuation)

• Test infrastructure: resetPeerBreakers/stateSnapshot (8 tests), NewMemoryBindingStore (4 tui panel tests), ClipboardAvailable (skip guard), validateRefName (3 test files)
• Regression tests on live code: decodeSSE (zz565 BUG-F covers live decodeSSECtx), VerifyAgentCardSignature (8 tests exercise live pipeline via wrapper)
• Revival-precedent family: agent 7-check family (79 fns) - registerAllChecks history ([business-logic] interface_compliance 与 deprecated_api 检测器从未注册到 write_integrity pipeline——注释声称已集成但生产 100% 漏报（测试绕过装配层全绿） #328/[code-review] 4 个检测器（printf_format/suspicious_comparison/dep_major_bump/dependency_vuln）从未注册到 write_integrity 注册表——生产 100% 漏报，#328 同根因扩展批次 #330, [business-logic] checkAppendIgnored is a dead detector — never registered in write_integrity allChecks, the documented write-time discarded-append() detection never fires #499, [code-review] ignored_error + naked_return dead detectors: delete+tombstone (sa-167-verified FP classes); empty_error_check: register as zero-FP Pattern-1 per #507 tombstone reservation #508/[business-logic] R73 census: duplicate_code unordered-Jaccard FP, hardcoded_host localhost semantic inversion, http_timeout set-delta FN + post-assign FP, loop_perf double-count + param FN — 4 dead detectors deleted, hardcoded_path registered live #516 revival) marks zero-wiring detectors as a bug class
• Frozen (wailskit dead-block): agentruntime interactions/desktop_adapters/desktop_stream/sessions/mobile + ClearSessionBindingsGlobal - all referenced by desktop/wailskit/chat.go / sessions.go (api_keys family frozen until wailskit/config.go lands)
• False-positive family: shell_sandbox / atspi / ydo (Linux build-tag conditional callers)
• SDK API face: sdk/plugin, a2a exported options/setters - deferred
• Deferred (big-family): im adapters (~50), auth oauth/token-cache (18), acp/client (68, chore: remove confirmed dead code (deadcode r78, acp AgentLoop fs-delegation face) #4337 overlap risk)

Verification

• go build -tags goolm ./... PASS
• go vet -tags goolm ./internal/a2a/... ./internal/tool/... PASS
• go test -tags goolm ./internal/a2a/ ./internal/tool/ PASS (102s / 63s)
• Pool file-face check: zero overlap with OPEN deadcode PRs (chore: remove confirmed dead code (deadcode r81: a2a/webui/session/cmd x4) #4349/chore: remove confirmed dead code (r79: internal/agent x3) #4343/chore: remove confirmed dead code (deadcode r73) #4333)