[deep-report] Migrate 251 assert.NoError calls to require.NoError in pkg/cli/ tests

[github-actions]

Description

The pkg/cli/ test suite contains 251 assert.NoError calls where require.NoError is the correct choice. With assert.NoError, a failed error check does not halt the test, so subsequent assertions run against invalid state and produce noisy cascading failures that obscure the real fault. Mechanically replace assert.NoError(t, err ...) with require.NoError(t, err ...) (adding the require import where missing).

Expected Impact

Cleaner, fail-fast test diagnostics; the first error is reported instead of a cascade. Low-risk mechanical change verifiable by a passing test run.

Suggested Agent

Code Simplifier / ESLint Refiner-style mechanical refactor agent, or Copilot SWE Agent.

Estimated Effort

Quick (< 1 hour)

Data Source

DeepReport 2026-07-22 analysis of Repository Quality report #47310 (251 assert.NoError metric).

Generated by 🔬 Deep Report · age00 213.6 AIC · ⌖ 11 AIC · ⊞ 10.3K · ◷

• expires on Jul 24, 2026, 7:26 AM UTC-08:00

[github-actions]

🍪 Issue Monster selected this for Copilot

I've identified this issue as a good candidate for automated resolution and requested assignment to the Copilot coding agent.

If assignment succeeds, the Copilot coding agent will analyze the issue and create a pull request with the fix.

Om nom nom! 🍪

🍪 Om nom nom by Issue Monster · gpt54 2.57 AIC · ⌖ 10.8 AIC · ⊞ 8K · ◷