refactor(cargo-wdk): share project directory iteration

[ITSMESB (ITSMERNB)]

Summary

• add a shared CargoProjectIterator for immediate subdirectories containing Cargo.toml
• use the same project discovery logic in both build and clean actions
• avoid the build action's previous duplicate Cargo.toml probing
• add focused iterator tests for filtering files and non-Rust directories

Fixes #644

Testing

• cargo test -p cargo-wdk --bin cargo-wdk --locked (104/104 pass)
• cargo clippy -p cargo-wdk --locked --all-targets -- -D warnings
• git diff --check
• the full package test command reaches integration tests, but the AURON service test session does not expose cargo on child-process PATH; the unit/action coverage above is green.

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR refactors “emulated workspace” detection/iteration logic by introducing a shared iterator that yields immediate subdirectories containing a Cargo.toml, and then reuses it in both the build and clean actions.

Changes:

• Added a shared CargoProjectIterator for enumerating Rust package directories under a working directory.
• Updated build and clean actions to use the iterator for validation and per-package processing.
• Added unit tests for the new iterator.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
crates/cargo-wdk/src/actions/mod.rs	Registers the new internal cargo_project_iterator module.
crates/cargo-wdk/src/actions/clean/mod.rs	Replaces manual directory scanning with CargoProjectIterator for emulated workspace clean.
crates/cargo-wdk/src/actions/cargo_project_iterator.rs	Implements and tests a reusable iterator over Cargo project subdirectories.
crates/cargo-wdk/src/actions/build/mod.rs	Replaces manual directory scanning with CargoProjectIterator for emulated workspace build.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🟢 Approval recommended

The refactor is localized, removes duplicated logic, preserves existing behavior, and is backed by targeted unit tests for the new iterator.

Review details

• Files reviewed: 4/4 changed files
• Comments generated: 0 new
• Review effort level: Lite

[ITSMESB (ITSMERNB)]

Rebased this onto the current main and resolved the clean-action overlap while preserving the newer logging behavior. The branch is mergeable again at 5199e09. Locally, the 109 cargo-wdk unit tests pass and cargo clippy -p cargo-wdk --locked --all-features --all-targets -- -D warnings is clean. The GitHub workflow runs are currently waiting for maintainer approval.