[code-simplifier] Simplify duplicate alt-path blocks in fetch.go; fix add_command test broken by #20375

[github-actions]

This PR simplifies recently modified code (from §22919210941) to improve clarity and fix a broken test.

Files Changed

• pkg/cli/fetch.go — Merge two identical fallback-path lookup blocks into a loop
• pkg/cli/add_command_test.go — Fix assertion broken by PR fix(cli): standardize --help flag descriptions and "agentic workflow" terminology #20375's terminology change

Improvements Made

1. Reduced Duplication in fetch.go

fetchRemoteWorkflow had two near-identical blocks for retrying a short workflow name under common directory prefixes (workflows/ and .github/workflows/). Both blocks performed the same steps: build the alt path, ensure .md suffix, log, attempt download, return on success. The only difference was the prefix string.

Consolidated into a single loop over []string{"workflows/", ".github/workflows/"}, reducing ~25 lines to ~13 with no behavioral change:

// Before: two separate identical blocks
altPath := "workflows/" + spec.WorkflowPath
...
if altContent, altErr := parser.DownloadFileFromGitHub(...); altErr == nil { return ... }

altPath = ".github/workflows/" + spec.WorkflowPath
...
if altContent, altErr := parser.DownloadFileFromGitHub(...); altErr == nil { return ... }

// After: one loop
for _, prefix := range []string{"workflows/", ".github/workflows/"} {
    altPath := prefix + spec.WorkflowPath
    ...
    if altContent, altErr := parser.DownloadFileFromGitHub(...); altErr == nil { return ... }
}

2. Fixed Broken Test in add_command_test.go

PR #20375 changed the add command's long description from "Add one or more workflows" to "Add one or more agentic workflows", but the corresponding test assertion was not updated. This caused TestNewAddCommand to fail on every run.

Changes Based On

Recent changes from:

• refactor: split pkg/cli/remote_workflow.go into 5 focused modules #20377 — refactor: split pkg/cli/remote_workflow.go into 5 focused modules
• fix(cli): standardize --help flag descriptions and "agentic workflow" terminology #20375 — fix(cli): standardize --help flag descriptions and "agentic workflow" terminology

Testing

• ✅ TestNewAddCommand now passes
• ✅ Build succeeds (make build)
• ✅ go test ./pkg/cli/ passes
• ✅ TestWasmGolden_CompileFixtures failures are pre-existing (unrelated to these changes)

References:

• §22919210941

Generated by Code Simplifier · ◷

Warning

⚠️ Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• releaseassets.githubusercontent.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "releaseassets.githubusercontent.com"

See Network Configuration for more information.

• expires on Mar 11, 2026, 7:09 PM UTC