Strengthen codemod registry coverage by asserting all registered IDs

[Copilot]

TestGetAllCodemods_ContainsExpectedCodemods covered only 42 IDs while the registry now contains 62 codemods, leaving silent-deletion risk for ~32% of entries. This update closes that gap by asserting the full expected ID set.

• What changed

  • Expanded expectedIDs in pkg/cli/fix_codemods_test.go to include the 20 missing codemod IDs already present in the registry/order contract.
  • Kept scope strictly to the coverage assertion test (no production logic changes).

• Coverage gap removed

  • Added IDs across recent migration families, including:
    • discussion/comment + MCP mode/type + install script URL migrations
    • roles/bots/engine step migrations
    • app/github-app and repo allowlist migrations
    • features CLI/DIFC proxy migrations
    • sandbox MCP/agent removals
    • run-install-scripts runtime migration

expectedIDs := []string{
    // existing IDs...
    "add-comment-discussion-removal",
    "mcp-mode-to-type-migration",
    "install-script-url-migration",
    "bash-anonymous-removal",
    "activation-outputs-to-sanitized-step",
    // ...
    "sandbox-agent-false-removal",
    "run-install-scripts-to-runtimes-node",
}

[github-actions]

Great work closing the codemod registry coverage gap! Expanding expectedIDs in TestGetAllCodemods_ContainsExpectedCodemods from 42 to 62 entries is a clean, targeted improvement that directly prevents silent-deletion regressions across ~32% of the registered codemods.

The PR is tightly scoped to a single test file, carries a clear description of what changed and why, introduces no new dependencies, and the change is purely additive. This looks ready for review.

Generated by ✅ Contribution Check · 228.2 AIC · ⌖ 35.5 AIC · ⊞ 6.3K · ◷

[Copilot]

Pull request overview

This pull request strengthens test coverage for the GetAllCodemods() registry by expanding the TestGetAllCodemods_ContainsExpectedCodemods assertion list so it enumerates all currently-registered codemod IDs, reducing the risk of silent deletions from the registry.

Changes:

• Expanded the expectedIDs list to include 20 codemod IDs that were already present in the registry but previously unasserted by this test.
• Kept changes scoped to test-only coverage assertions (no production registry or codemod logic changes).

Show a summary per file

File	Description
pkg/cli/fix_codemods_test.go	Expands the expected codemod ID assertion list to cover the full codemod registry.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 1/1 changed files
• Comments generated: 1
• Review effort level: Low