Repository navigation
fix(echidnabot): move the pin to the upstream repair, and mark 1 fleet test value - #660
Conversation
…t test value 4 checks are red on `main` and on each pull request: `scan / gitleaks`, `scan / rust-secrets`, and the 2 UUID conformance checks. The lines that they report are in the copy of echidnabot and in 1 fleet test: - The Rust rule of the secret scan: 2 names of environment variables in `bots/echidnabot/src/api/auth.rs`, 2 test values in `bots/echidnabot/tests/smoke.rs`, and 1 test value in `tests/echidnabot/api_gate_regressions.rs`. - The strict UUID rule: 3 nil UUID literals in the echidnabot tests. The source of the first 4 lines and of the 3 literals is the echidnabot repository. hyperpolymath/echidnabot#197 repaired them there (merged, 998d306). Changes: - `bots/echidnabot/`: `scripts/sync-vendored-bot.sh echidnabot --sync` moves the pin from 1443adf to a045184 (upstream `main`). 4 files change: `src/api/auth.rs`, `tests/protocol_contract.rs`, `tests/smoke.rs` (echidnabot#197) and `Cargo.lock` (echidnabot#198: `event-listener`, `chacha20`, `spin`). - `tests/echidnabot/api_gate_regressions.rs`: the pragma `// scanner-allow: rust-secrets` on the 1 test value that the fleet owns. The value is a test credential, not a secret. Not in this change: the sqlx repair for RUSTSEC-2026-0459 (hyperpolymath/echidnabot#199). It is not merged upstream at this time. A later pin move carries it. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0136P7ZmiFSCoB3UR9Kk56rK
|
Caution Review failedThe pull request is closed.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
|
Codex Review Summary: ❌ FailedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
This security review couldn’t start because the paying account has exhausted its Codex allowance and has insufficient available credits. The payer can check their balance and add credits in the Codex usage dashboard, then try again. If you do not manage the paying account, contact this repository's admins. |



Summary
4 checks are red on
mainand on each pull request (for example #658 and #659):scan / gitleaks,scan / rust-secrets, and the 2 UUID conformance checks. This pull request repairs their 2 causes.bots/echidnabot/src/api/auth.rs, 2 test values inbots/echidnabot/tests/smoke.rstests/echidnabot/api_gate_regressions.rsThe echidnabot lines are repaired at their source: hyperpolymath/echidnabot#197 (merged,
998d306). No line here is a secret. The values are names of environment variables and test credentials.Changes
bots/echidnabot/:scripts/sync-vendored-bot.sh echidnabot --sync --rev a045184...moves the pin from1443adftoa045184, the head of upstreammain. 4 files change in the copy:src/api/auth.rs,tests/protocol_contract.rs,tests/smoke.rs: the changes of echidnabot#197 (the pragma on 4 lines; the nil UUID comes fromuuid::Uuid::nil()and is not a literal).Cargo.lock: the changes of echidnabot#198 (event-listener5.4.2,chacha200.10.2,spin0.9.9).tests/echidnabot/api_gate_regressions.rs: the pragma// scanner-allow: rust-secretson the 1 test value that this repository owns, with a comment of 2 lines.The 2 dated notes in
docs/migrations/secret-scanner-cicd-suite/name the old pin. They are records of 2026-11-09 and do not change.Not in this change
The sqlx repair for RUSTSEC-2026-0459 (hyperpolymath/echidnabot#199) is not merged upstream at this time. A later pin move carries it.
Testing
bash scripts/sync-vendored-bot.sh echidnabot --check:bots/echidnabot matches ...@a045184... (90 entries), exit 0.proven-cicd03e858b, and the strict UUID rule), run on the whole tree: 0 hits in 270 Rust files. Before this change the same copy gave 5 lines and 3 literals.998d306.What to expect in CI
scan / gitleaks,scan / rust-secretsand the 2 UUID conformance checks pass. If 1 of them does not, the result is added here.Generated by Claude Code