Problem
#7290 proposed migrating packages/loopover-miner/lib/** to TypeScript, phased for gate safety.
Phase 1 (#7299) landed the tsc build pipeline. This issue is batch 4.8 of 8 (Phase 4, convert last -- highest fan-in in the whole package): convert this batch's
files, using the same in-place-emit pattern Phase 1 already wired up.
Proposal
Convert these files (and their hand-maintained .d.ts siblings) from .js to real .ts:
lib/governor-ledger.js
lib/event-ledger.js
lib/portfolio-queue.js
lib/cli-error.js
No tsconfig.json changes are needed -- its existing include glob (bin//*.ts, lib//*.ts)
picks up newly-converted files automatically.
Deliverables
Test Coverage Requirements
No behavior change from this migration alone. Existing tests for these modules must continue to
pass unmodified (only import paths/build output shape might need adjustment).
Expected Outcome
These files are real, compiler-verified TypeScript with zero behavior change.
Links & Resources
Problem
#7290 proposed migrating packages/loopover-miner/lib/** to TypeScript, phased for gate safety.
Phase 1 (#7299) landed the tsc build pipeline. This issue is batch 4.8 of 8 (Phase 4, convert last -- highest fan-in in the whole package): convert this batch's
files, using the same in-place-emit pattern Phase 1 already wired up.
Proposal
Convert these files (and their hand-maintained .d.ts siblings) from .js to real .ts:
lib/governor-ledger.jslib/event-ledger.jslib/portfolio-queue.jslib/cli-error.jsNo tsconfig.json changes are needed -- its existing include glob (bin//*.ts, lib//*.ts)
picks up newly-converted files automatically.
Deliverables
Test Coverage Requirements
No behavior change from this migration alone. Existing tests for these modules must continue to
pass unmodified (only import paths/build output shape might need adjustment).
Expected Outcome
These files are real, compiler-verified TypeScript with zero behavior change.
Links & Resources