Skip to content

refactor(installer): installViaFileCopy is 227 lines #335

Description

@dean0x

Finding

installViaFileCopy in src/targets/claude-code/installer.ts (around lines 370-597, ~227 lines) is a single function handling file-copy installation. Pre-existing before this PR's changes.

Why deferred

Flagged during code review of PR #334 (Tracker Phase 1 — MDS plumbing). Triage disposition: Fix Separately — pre-existing complexity issue, reviewer prescribes a separate PR. Extracting installAgents/installCommands would rewrite the install path that every devflow init flows through, which is out of scope for a plumbing-only phase PR.

Suggested fix

Extract installAgents() and installCommands() as separate functions from installViaFileCopy, reducing the parent function's responsibility and length.

References

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    tech-debtTechnical debt items to address

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions