API Surface Issue
Category
Unused export
Summary
- File:
src/commands/predownload.ts
- Symbol:
resolveImages
- Issue:
resolveImages is exported but is never imported by any module other than its own test file (predownload.test.ts). The only production call site is internal — predownloadCommand calls resolveImages directly within the same file.
Evidence
$ grep -rw "resolveImages" src --include="*.ts" | grep -v ".test.ts" | grep -v "predownload.ts"
(no output — zero external import sites)
src/commands/predownload.ts:30:export function resolveImages(options: PredownloadOptions): string[] {
src/commands/predownload.ts:66: const images = resolveImages(options); // internal call only
A prior issue (#2816) tracked resolveImages alongside related command-module exports and was closed as completed, but the export was not removed.
Recommended Fix
Remove the export keyword from resolveImages, making it a module-private helper. Update src/commands/predownload.test.ts to test it indirectly via predownloadCommand if direct unit coverage of resolveImages is still desired, or keep a narrow unit test with the function unexported (test the module's internal logic via the public command).
Impact
- Dead code risk: Medium — callers could invoke
resolveImages with arbitrary inputs, bypassing any future validation added to predownloadCommand.
- Maintenance burden: Low
Detected by Export Audit workflow. Triggered by push to main on 2026-05-19. See also prior issue #2816.
Generated by API Surface & Export Audit · ● 10.8M · ◷
API Surface Issue
Category
Unused export
Summary
src/commands/predownload.tsresolveImagesresolveImagesis exported but is never imported by any module other than its own test file (predownload.test.ts). The only production call site is internal —predownloadCommandcallsresolveImagesdirectly within the same file.Evidence
A prior issue (#2816) tracked
resolveImagesalongside related command-module exports and was closed as completed, but the export was not removed.Recommended Fix
Remove the
exportkeyword fromresolveImages, making it a module-private helper. Updatesrc/commands/predownload.test.tsto test it indirectly viapredownloadCommandif direct unit coverage ofresolveImagesis still desired, or keep a narrow unit test with the function unexported (test the module's internal logic via the public command).Impact
resolveImageswith arbitrary inputs, bypassing any future validation added topredownloadCommand.Detected by Export Audit workflow. Triggered by push to main on 2026-05-19. See also prior issue #2816.