---
id: TASK-11
title: Fix clippy errors/warnings introduced in recent tests
status: Done
assignee:
  - codex
created_date: '2026-02-26 13:12'
updated_date: '2026-02-26 13:16'
labels: []
dependencies: []
references:
  - src/core/cmd.rs
  - src/core/cli.rs
  - src/core/config.rs
  - src/config/load.rs
  - tests/integration_tests.rs
---

## Description

<!-- SECTION:DESCRIPTION:BEGIN -->
Resolve `just check` (clippy) errors and warnings in test code without changing CLI/config behavior.
<!-- SECTION:DESCRIPTION:END -->

## Acceptance Criteria
<!-- AC:BEGIN -->
- [x] #1 `just check` passes with no clippy errors; warnings are resolved or explicitly justified.
- [x] #2 Test-only changes; no CLI/config behavior changes.
- [x] #3 `just test` passes.
<!-- AC:END -->

## Implementation Plan

<!-- SECTION:PLAN:BEGIN -->
Plan:
1) Update tests to satisfy clippy: rename shadowed bindings, avoid `panic!` in tests by using `assert!`/`unwrap`/`expect`, and remove redundant clones.
2) Resolve `similar_names` by renaming `runner_cmd`/`runners_cmd` variables.
3) Run `just check`, then `just test`.
<!-- SECTION:PLAN:END -->

## Implementation Notes

<!-- SECTION:NOTES:BEGIN -->
Resolved clippy issues in test code: removed shadowing, replaced panic-style matches with assertions, removed redundant clones, adjusted CLI test pattern matches, and cleaned error matching in config/load tests. `just check` now passes cleanly; `just test` passes (64 tests).
<!-- SECTION:NOTES:END -->

## Final Summary

<!-- SECTION:FINAL_SUMMARY:BEGIN -->
Fixed clippy errors/warnings in recent tests by renaming shadowed bindings, avoiding panic-style matches, removing redundant clones, and adjusting CLI pattern matches. `just check` and `just test` both pass.

Tests: `just check`, `just test`
<!-- SECTION:FINAL_SUMMARY:END -->
