Skip to content

refactor(cli): name the MCP launch-command transform after what it does #824

Description

@blafourcade

Problem

Once #816 lands, cli/src/contexts/tools/domain/mcp-exclusion.ts holds only transformFor, the Windows transform of an MCP server's launch command. The MCP exclusion it was named for is deleted. The file name now points a reader at a feature that no longer exists. The repository's own naming rule is to name by intention, not by history.

Scope

  • Rename the module after what it does: a transform of the MCP launch command for the target platform.
  • Update its imports (install-config-use-case.ts and any others), the module list in tests/architecture/context-boundary.arch.test.ts, and the test file name.

Acceptance criteria

  • No module under cli/src is named after the MCP exclusion.
  • Typecheck, test:arch and the tools suite pass unchanged. It is a rename with no behaviour change.

Prior art in this repository

Out of scope

  • Any change to what the transform does.

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

    No labels
    No labels

    Type

    Fields

    Priority

    None yet

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions