cameri / nostream / 39131056656

74%

main: 74%

push

github

docs: add a CLAUDE.md for the Rust rewrite (#883)

* docs: add agent instructions for the Rust rewrite branch

* docs: keep the rewrite CLAUDE.md portable across contributor machines

* docs: drop the builder-image wrapper from the rewrite CLAUDE.md

* ci(rust): carry the strict clippy lints in the workspace configuration

checks.yml passed -D clippy::unwrap_used -D clippy::expect_used -D clippy::panic only on
the command line. A crate inheriting [lints] workspace = true therefore passed a local
cargo clippy and was rejected by the gate.

The three lints move into [workspace.lints.clippy], and the 17 integration-test targets
that assert by unwrapping or panicking carry an explicit allow at the top of the file. The
allowance names the file rather than exempting test targets workspace-wide, so a new test
file has to ask.

Measured from a clean target directory: fmt clean; all-targets clippy clean; strict
lib+bins pass clean; 157 tests passed, 0 failed; release build succeeds. Proven to fire:
Some(1).unwrap() added to nostream-config/src/lib.rs fails the all-targets pass.

* docs: correct the error-model rules Greptile flagged

Two contradictions in the CLAUDE.md:
- 'everything outside the two binaries returns CliError' read as 'all libraries depend on
  nostream-cli'. It now says each library module returns its own crate's type, with the CLI's
  library modules returning CliError.
- The anyhow check searched only crates/*/src/lib.rs, so a module file using anyhow would
  pass. It now scans every .rs file, excludes the two allowed main.rs, and skips comment
  lines that merely name the crate. Re-proven by adding an anyhow::Result probe to
  nostream-config/src/load.rs, which the check printed.

3588 of 5362 branches covered (66.92%)

Branch coverage included in aggregate %.

7992 of 10291 relevant lines covered (77.66%)

870.32 hits per line