Problem
#7291 proposed migrating packages/loopover-mcp to TypeScript, phased by file size after a real
line-count check showed the six files are wildly uneven (27 to 6,594 lines). This is Phase 1:
prove the build pipeline against the four smallest, lowest-risk files before touching anything
larger.
Proposal
- Add
packages/loopover-mcp/tsconfig.json and a real tsc-based build/typecheck script,
replacing the current node --check-only build.
- Convert these four files (no existing
.d.ts to remove -- this package has none):
lib/cli-error.js (27 lines)
lib/format-table.js (58 lines)
lib/redact-local-path.js (58 lines)
lib/telemetry.js (68 lines)
Deliverables
Test Coverage Requirements
No behavior change from this migration alone. Existing tests must continue to pass unmodified.
Expected Outcome
The build pipeline is proven and these four small utilities are real TypeScript, clearing the way
for #7291's Phase 2 (local-branch.js) and Phase 3 (bin/loopover-mcp.js).
Links & Resources
Problem
#7291 proposed migrating packages/loopover-mcp to TypeScript, phased by file size after a real
line-count check showed the six files are wildly uneven (27 to 6,594 lines). This is Phase 1:
prove the build pipeline against the four smallest, lowest-risk files before touching anything
larger.
Proposal
packages/loopover-mcp/tsconfig.jsonand a realtsc-basedbuild/typecheckscript,replacing the current
node --check-only build..d.tsto remove -- this package has none):lib/cli-error.js(27 lines)lib/format-table.js(58 lines)lib/redact-local-path.js(58 lines)lib/telemetry.js(68 lines)Deliverables
packages/loopover-mcp/tsconfig.jsonaddedbuild/typecheckscripts updated to a realtsccompile.jsto.tsTest Coverage Requirements
No behavior change from this migration alone. Existing tests must continue to pass unmodified.
Expected Outcome
The build pipeline is proven and these four small utilities are real TypeScript, clearing the way
for #7291's Phase 2 (
local-branch.js) and Phase 3 (bin/loopover-mcp.js).Links & Resources
packages/loopover-miner