92bfb426ca
All checks were successful
CI / verification (pull_request) Successful in 20s
Three findings of the review, each first measured green on the previous tree (18/18), then red on this one. The proxy test dropped the inherited no_proxy, so a script exporting NO_PROXY alone stayed green, although curl reads the lowercase variable first and an inherited no_proxy would win. The test now sets no_proxy to a list that leaves the loopback out. Red when the script exports NO_PROXY alone, and on the script from main. The column-0 require pin missed a space before the parenthesis: an indented `require (` and an `import (` both passed. The regexes now accept whitespace there. Red on each. The continue-on-error detector required a colon after the word, so a YAML anchor reused as a key (`&coe continue-on-error`, then `*coe : true`) passed, while PyYAML reads it as the key. The detector is now the bare word, with a self-check row for the anchor; red when the colon is required again, three rows red with the original anchored regex. An escaped key still passes, since the pin reads text and not YAML: the comment and CLAUDE.md now claim any literal spelling, no longer any spelling, and list the indirections the require pin cannot follow. ci.test.js 18 -> 19 cases, vitest 346 -> 347. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| agent.test.js | ||
| auth.test.js | ||
| ci.test.js | ||
| findings.test.js | ||
| health.test.js | ||
| hosts.test.js | ||
| state.test.js | ||
| writes.test.js | ||