Skip to content

agent-conformance: clear the base clippy -D warnings errors (lint only, I-1228-f) - #1256

Draft
SashaMIT wants to merge 2 commits into
spike/agent-hostfrom
ultrafast/omp-y33-acclippy
Draft

SashaMIT wants to merge 2 commits into
spike/agent-hostfrom
ultrafast/omp-y33-acclippy

Conversation

@SashaMIT

Copy link
Copy Markdown
Collaborator

Fixes I-1228-f: cargo clippy --all-targets -- -D warnings for lab/agent-conformance failed on spike/agent-host. Lint cleanup only, with no change in behaviour. 3 files, +21/−8.

  • src/report.rs:100: the duplicate-inventory if/else-if becomes one || condition. The short-circuit is the same: an invalid name never inserts.
  • src/report.rs:399: !parse().is_ok() becomes parse().is_err().
  • standin-runtime/mod.rs: adds impl Default for StandIn, which calls new().
  • tests/common/grants.rs:2857,3064,3094: &[x.clone()] becomes std::slice::from_ref(&x).

Results (box, Linux):

  • clippy exits 0.
  • standin_contract 13/0, standin_grants 1/0, fmt clean.
  • --test effects: 44 passed, 19 failed, 12 ignored.
    • All 19 failures are the darwin-only operator-Home prerequisite skip ("probe could not run", probe.rs:139: "operator Home requires darwin-arm64 or darwin-amd64").
    • The three edited tests pass: cancel_refuses_retries_and_unexpected_requests, kill_refuses_requests_from_any_session, kill_sibling_read_must_be_new_in_its_session.

No falsifier, because no guard was added or removed.

Grok Bot slot driver added 2 commits October 30, 2026 20:01
…-1228-f)

`cargo clippy --all-targets -- -D warnings` for lab/agent-conformance failed on
main. This is a behaviour-preserving lint cleanup:
- report.rs:100 collapses the duplicate-inventory if/else-if into one condition
  (same short-circuit: an invalid name never inserts).
- report.rs:399 changes `!parse().is_ok()` to `parse().is_err()`.
- standin-runtime/mod.rs: `impl Default for StandIn` calling `new()`.
- tests/common/grants.rs:2857,3064,3094: `&[x.clone()]` becomes
  `std::slice::from_ref(&x)`.
Clippy exits 0. standin_contract 13/0, standin_grants 1/0, fmt clean.
effects: 44 passed, 19 failed, 12 ignored. All 19 failures are the
darwin-only operator-Home prerequisite skip ("probe could not run" at
probe.rs:139). The three edited tests pass: cancel_refuses_retries_and_unexpected_requests,
kill_refuses_requests_from_any_session and kill_sibling_read_must_be_new_in_its_session.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant