API Surface Issue
Category
Unused export
Summary
- File:
src/services/agent-volumes/home-strategy.ts
- Symbol:
buildToolDirectoryMounts
- Issue: Function is exported but never imported outside its own module
Evidence
$ grep -rw "buildToolDirectoryMounts" src/ --include="*.ts"
src/services/agent-volumes/home-strategy.ts: mounts.push(...buildToolDirectoryMounts(params));
src/services/agent-volumes/home-strategy.ts:export function buildToolDirectoryMounts(params: HomeMountsParams): string[] {
buildToolDirectoryMounts is a private helper called only internally within home-strategy.ts by buildHomeMounts(). No other file imports it. The export keyword unnecessarily exposes this internal helper as a public API, likely a leftover from the refactoring that split agent-volumes.ts into submodules (issue #3630).
Recommended Fix
- Remove the
export keyword — make it a private helper within the module:
// Before
export function buildToolDirectoryMounts(params: HomeMountsParams): string[] {
// After
function buildToolDirectoryMounts(params: HomeMountsParams): string[] {
Impact
- Dead code risk: Low — no callers exist, so no runtime impact
- Maintenance burden: Low — but keeping unnecessary exports inflates the module's API surface and misleads future contributors about intended access patterns
Detected by Export Audit workflow. Triggered by push to main on 2026-05-27
Generated by API Surface & Export Audit · sonnet46 1.7M · ◷
API Surface Issue
Category
Unused export
Summary
src/services/agent-volumes/home-strategy.tsbuildToolDirectoryMountsEvidence
buildToolDirectoryMountsis a private helper called only internally withinhome-strategy.tsbybuildHomeMounts(). No other file imports it. Theexportkeyword unnecessarily exposes this internal helper as a public API, likely a leftover from the refactoring that splitagent-volumes.tsinto submodules (issue #3630).Recommended Fix
exportkeyword — make it a private helper within the module:Impact
Detected by Export Audit workflow. Triggered by push to main on 2026-05-27