[plan] Remove zombie skipped test functions from pkg/cli/ test files #19242

Objective

Remove permanently-skipped test functions from four pkg/cli/ test files. Unlike the pkg/workflow/ zombie files where entire files can be deleted, these files contain a mix of active and skipped functions — so only the skipped functions should be removed, not the entire files.

Context

Discussion #19223 identified 7 permanently-skipped test functions across 4 files in pkg/cli/. All skips use the require() migration message.

Files to Edit

Approach

For each file:

1. Open the file and identify all test functions that contain t.Skip(...) with the message "scripts now use require() pattern" (or similar require() migration messages)
2. Remove those test functions entirely (including any helper functions that exist solely to support the skipped tests)
3. Do NOT remove functions that are either:
   • Not skipped
   • Skipped for a different reason (e.g., // FIXME skips)

Search pattern to find skipped functions:

grep -n "t.Skip" pkg/cli/audit_test.go pkg/cli/logs_firewall_parse_test.go pkg/cli/logs_parse_test.go pkg/cli/mcp_inspect_safe_inputs_integration_test.go

Validation

go build ./...
go vet ./...
make fmt
make lint
make test-unit

Acceptance Criteria

• All require()-migration-skipped test functions removed from the 4 files
• No non-skipped test functions accidentally removed
• Files still compile and pass go vet
• make test-unit passes with no regressions
• make lint passes (no unused helper functions left behind)

Generated by Plan Command for issue #discussion #19223

• expires on Mar 4, 2026, 2:57 PM UTC