chore: remove unused backend code - #5
Conversation
Greptile SummaryThis PR is a backend-wide cleanup that removes 196 net lines of dead code: unused exports, duplicated helpers, deprecated type aliases, and test-only compatibility wrappers. All consumer call sites are updated to use the canonical APIs.
Confidence Score: 5/5Every removed function was either dead code, a compatibility re-export, or a duplicated helper replaced by an identical canonical version. No runtime behavior changes in production paths. The cleanup is narrow and mechanical: new canonical files replace exact duplicates, mapAgentEventToChat was provably always returning [], validatePreviewAccess never read its token argument, and the shared isRecord matches the stricter local variant it replaced. The typecheck and 972 passing tests leave no suspicious gap. No files require special attention. The projection and preview-manager changes are the most structurally significant but are fully covered by updated tests. Important Files Changed
Reviews (1): Last reviewed commit: "chore: remove unused backend code" | Re-trigger Greptile |
Summary
Why
A backend-wide spring-clean audit found unused exports, test-only compatibility APIs, duplicated helpers, and stale command documentation. Removing them reduces maintenance surface without changing supported runtime behavior. Cleanup already covered by PR #2 was intentionally excluded.
Impact
This removes 196 net lines across the backend and related consumer tests. Private compatibility exports are removed in favor of their existing canonical replacements.
Validation