[codex] Remove unused LitzRouteRule public API - #198
Conversation
Remove the unused LitzRouteRule interface from the Vite type surface and stop exporting it from litzjs/vite. This avoids stabilizing route-level cache, header, redirect, prerender, and proxy type hints that are not implemented for 1.0. Refs #191
Greptile SummaryThis PR removes the
Confidence Score: 5/5This is a straightforward dead-code removal — the type had no runtime consumers and is fully excised from imports, exports, and the type definitions file. The change is a clean, surgical deletion confirmed by a codebase-wide search showing zero remaining references to LitzRouteRule. The changeset correctly flags it as a minor (breaking) bump so downstream consumers are warned before upgrading. No files require special attention.
|
| Filename | Overview |
|---|---|
| src/vite/types.ts | Removes the entire LitzRouteRule interface — clean deletion with no remaining references in the codebase. |
| src/vite.ts | Removes the LitzRouteRule import and drops it from the public re-export; LitzPluginOptions export is unchanged. |
| .changeset/remove-litz-route-rule.md | Changeset correctly uses a minor bump to signal the backward-incompatible type removal. |
Reviews (2): Last reviewed commit: "fix: mark route rule removal as minor" | Re-trigger Greptile
Update the changeset for the LitzRouteRule public API removal from patch to minor so consumers get the correct semver signal for the breaking type-level change. Refs #191
Summary
Removes the unused
LitzRouteRuletype before the 1.0 API surface is stabilized.Root cause
LitzRouteRulewas exported fromlitzjs/vite, but no runtime path consumed it. Keeping it public would imply support for route-level cache, headers, redirects, prerendering, and proxy behavior that Litz does not implement for 1.0.Changes
LitzRouteRuleinterface from the Vite type definitions.LitzRouteRulefrom thelitzjs/vitepublic type export.Validation
bun fmtbun lint:fixbun run testbun typecheckbun lintCloses #191