Skip to content

gut: eliminate callers and delete ~18 EXCLUDE-STUB files in src/agents/ #2367

@alexey-pelykh

Description

@alexey-pelykh

Summary

Eliminate callers and delete ~18 EXCLUDE-STUB stubs in src/agents/. Most have 1-2 callers; 4 have 5-6.

Files (current caller count)

File Callers Notes
pi-embedded-runner/model.ts 1 tts/tts-core.ts
pi-embedded-runner/run/params.ts 1 commands/agent/types.ts or gateway
pi-embedded-runner/runs.ts 1 gateway/server.impl.ts
pi-embedded.ts 6 tts, auto-reply, gateway, commands
pi-embedded-runner.ts 1 tts, commands, auto-reply
auth-profiles/session-override.ts 1 auto-reply/get-reply-run.ts
bash-tools.ts 1 ExecToolDefaults alias
bootstrap-budget.ts 1 returns empty Set
bootstrap-cache.ts 6 snapshot clear functions
cli-runner.ts 3 returns undefined
context-window-guard.ts 1 min-tokens constant
failover-error.ts 2 failover reason types
identity-file.ts 1 returns null
model-fallback.ts 4 returns undefined
sandbox.ts 1 minimal mode logic
skills-status.ts 2 returns loaded:false
skills.ts 1 SkillCommandSpec type
stream-message-shared.ts 2 returns empty
system-prompt.ts 4 returns ""
test-helpers/usage-fixtures.ts 1 zero-usage constant

AC

  • All listed files deleted
  • All callers eliminated (import removed, call site simplified or deleted)
  • Disposition entries updated: EXCLUDE-STUB → EXCLUDE-GUT
  • pnpm check passes
  • pnpm test passes
  • If pi-embedded-runner/ directory is empty after deletions, remove directory

Strategy

Start from leaf stubs (0-1 callers) and work up. Many callers are other stubs — deleting one may cascade-eliminate callers of another. Error cascades from NOT creating replacement stubs reveal dead code chains per fork feedback.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions