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.5 of 8 (Phase 4): 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/ci-poller.js
lib/stack-detection.js
lib/self-review-context.js
lib/deny-hooks.js
lib/policy-doc-cache.js
lib/pr-outcome.js
lib/attempt-log.js
lib/coding-task-spec.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.5 of 8 (Phase 4): 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/ci-poller.jslib/stack-detection.jslib/self-review-context.jslib/deny-hooks.jslib/policy-doc-cache.jslib/pr-outcome.jslib/attempt-log.jslib/coding-task-spec.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