chore(clippy): resolve -D warnings across the workspace #8

Merged
hugooconnor merged 0 commits from refs/pull/8/head into main 2026-06-21 13:40:47 +00:00
hugooconnor commented 2026-06-21 13:40:29 +00:00 (Migrated from codeberg.org)

Clears the pre-existing cargo clippy --all -- -D warnings failures so the CI lint step (added in #5) is green.

Changes

  • Inline format args (uninlined_format_args) across ar-edit and ar-edit-core — the bulk of the diff, mechanical (format!("{}", x)format!("{x}")), applied via cargo clippy --fix.
  • ar-edit-collab/pairing.rs: separate a trailing paragraph from a doc bullet list (doc_lazy_continuation).
  • ar-edit-collab/shell/daemon.rs: rewrite the accept() loop as while let (while_let_loop).
  • ar-edit/main.rs: remove a stale doc comment + duplicate #[cfg(unix)] left above enum HostStatus by an earlier refactor (duplicated_attributes).

Verification

  • cargo clippy --all -- -D warnings → clean (this is the CI command).
  • cargo fmt --all -- --check → clean.
  • cargo test --workspace1399 passed.

Out of scope: test-target lints under --all-targets (deprecated assert_cmd::cargo_bin, type_complexity) are pre-existing and not in the CI path; left as-is.

🤖 Generated with Claude Code

Clears the pre-existing `cargo clippy --all -- -D warnings` failures so the CI lint step (added in #5) is green. ## Changes - **Inline format args** (`uninlined_format_args`) across `ar-edit` and `ar-edit-core` — the bulk of the diff, mechanical (`format!("{}", x)` → `format!("{x}")`), applied via `cargo clippy --fix`. - **ar-edit-collab/pairing.rs**: separate a trailing paragraph from a doc bullet list (`doc_lazy_continuation`). - **ar-edit-collab/shell/daemon.rs**: rewrite the `accept()` loop as `while let` (`while_let_loop`). - **ar-edit/main.rs**: remove a stale doc comment + duplicate `#[cfg(unix)]` left above `enum HostStatus` by an earlier refactor (`duplicated_attributes`). ## Verification - `cargo clippy --all -- -D warnings` → clean (this is the CI command). - `cargo fmt --all -- --check` → clean. - `cargo test --workspace` → **1399 passed**. Out of scope: test-target lints under `--all-targets` (deprecated `assert_cmd::cargo_bin`, `type_complexity`) are pre-existing and not in the CI path; left as-is. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
anuna-research/ar-edit!8
No description provided.