[diagrams] skip_green (default true) draws nothing for a GREEN result;
config/werkrevisor.toml switches the diagrams on with Sonnet 5.5.
pr_lens.mjs ends a failure with one line that names the cause.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-code
marked the pull request as ready for review 2026-10-20 12:18:00 +02:00
YELLOW Weakened merge gate according to Jev: werkrevisor/config.py (0.75); a human checks it, unless you revert the change because the task does not need it
hint writing rules violated in the changed docs: README.md:1060: paragraph with 6 sentences, more than 5
hint 6 changed file(s) that neither weaken the checks, touch security nor show a risk category of the repository config according to Jev
hint 1 build change(s) of low risk according to Jev: statistics/build.gradle.kts (weakens checks 0.02, risk 0.00, security 0.02)
hint 10 changed file(s) agree with the PR document according to Jev
hint 1 changed file was given to the AI as excerpt of the selected lines and 5 lines around each changed line, not in full (each larger than [review] max_full_file_chars = 80000): werkrevisor/cli.py
hint 1 finding(s) of low severity or rated minor by Jev: README.md:1062-1063
hint model recommends merge with caution
Findings
[medium] Documentation Contradiction at README.md:1062-1063
hint; Jev: real 0.81, severity 2.0 of 3
The README states "Sonnet 5.5 did on 5 of 5 PRs, GPT-6 Luna too but with rough boxes only, MiniMax M3 on none". This contradicts the PR document at docs/prs/2026-10-20-PR#125-diagrams-on.md and the comment in config/werkrevisor.toml, both of which list 4 measured PRs (PR#117, PR#118, PR#122, PR#123) with Sonnet 5.5 and GPT-6 Luna at 4 of 4 and MiniMax M3 at 0 of 1. A reader who consults the README and then the PR document will see different sample sizes for the same model evaluation, which weakens trust in the choice of model.
Classifier
8 of 8 units passed all questions (1 PR document(s)).
2 answers ignored because of a confidence below [classifier] min_confidence.
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 40550, calls 1, tokens in 39093 / out 27160 (reasoning included in out), duration 165.4 s, cost 0.0443 USD (reported), classifier `jev-1.13.0` calls 8, tokens in 6254, cost 0.0003 USD, finding judge `jev-1.13.0` calls 1, tokens in 2114, cost 0.0001 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 23226 / out 253, cost 0.0030 USD (reported), PR document check `jev-1.13.0` calls 10, tokens in 23557, cost 0.0010 USD, build file judge `jev-1.13.0` calls 8, tokens in 7066, cost 0.0003 USD, phases: path_judge 0.5 s, classifier 0.6 s, doc_check 0.7 s, context 5.2 s, model_review 157.5 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main db2fba4).
Reviewed head `eeda61bc3b3332ba27cf252582306f5109aa4e17` against base `db2fba45e738880d6b09a0ab2eba56cced6e7b31`, werkrevisor 0.52.0.
<!-- werkrevisor:eeda61bc3b3332ba27cf252582306f5109aa4e17 -->
<!-- werkrevisor-verdict:YELLOW -->
<!-- werkrevisor-blocks:04cb0b7d13f841288b55a2255c7da0ca,64ee59d14d23c60833672f2eb378669e,bf978d07fd88cf68c2401cb1865aeb7f,98834fe83858b86053f19d6bb1f68763,9daeb8c97aa5ae9f08393a6a396b5f33,d944a358bc7dca7322d85e2315032953,371794da7a3e8af4df7958ed28d4290a,8b1242ae84ecd9cd38dd78a008211247,cf3314d6fa7917e6e5d936aa63e2d07c,8246b345bbea119f36fc4a45682d6b3b,e47b6a6fdd33aa76fa816f47dd993220,fec09e2ac21aacef1d2e05d32adf5992,bc3fb04d7eb6b4ed98b645211a93f443,230929668134f79355aa0499c320a5bf,6349ad1da12830384973aade32db5627,d47100acd03f99054cb986ac14cbeef0,24a5fda583f3f7dd51a6c3160250b2e7,aff003b72561664332ca81d8acdca862,5a9745a0285612b9764bd530abc592b5 -->
<!-- werkrevisor-folded -->
<details><summary>🟡 werkrevisor YELLOW for <code>eeda61b</code>, superseded by <a href="http://git.javagil.de/mi/werkrevisor/pulls/125#issuecomment-6503">the result for <code>9911267</code></a></summary>
## 🟡 werkrevisor: YELLOW / HUMAN_REVIEW
Change kind: `mixed`.
### Reasons
- **YELLOW** Weakened merge gate according to Jev: werkrevisor/config.py (0.75); a human checks it, unless you revert the change because the task does not need it
- **hint** writing rules violated in the changed docs: README.md:1060: paragraph with 6 sentences, more than 5
- **hint** 6 changed file(s) that neither weaken the checks, touch security nor show a risk category of the repository config according to Jev
- **hint** 1 build change(s) of low risk according to Jev: statistics/build.gradle.kts (weakens checks 0.02, risk 0.00, security 0.02)
- **hint** 10 changed file(s) agree with the PR document according to Jev
- **hint** 1 changed file was given to the AI as excerpt of the selected lines and 5 lines around each changed line, not in full (each larger than [review] max_full_file_chars = 80000): werkrevisor/cli.py
- **hint** 1 finding(s) of low severity or rated minor by Jev: README.md:1062-1063
- **hint** model recommends merge with caution
### Findings
1. **[medium] Documentation Contradiction** at `README.md:1062-1063`
hint; Jev: real 0.81, severity 2.0 of 3
The README states "Sonnet 5.5 did on 5 of 5 PRs, GPT-6 Luna too but with rough boxes only, MiniMax M3 on none". This contradicts the PR document at `docs/prs/2026-10-20-PR#125-diagrams-on.md` and the comment in `config/werkrevisor.toml`, both of which list 4 measured PRs (PR#117, PR#118, PR#122, PR#123) with Sonnet 5.5 and GPT-6 Luna at 4 of 4 and MiniMax M3 at 0 of 1. A reader who consults the README and then the PR document will see different sample sizes for the same model evaluation, which weakens trust in the choice of model.
### Classifier
8 of 8 units passed all questions (1 PR document(s)).
2 answers ignored because of a confidence below [classifier] min_confidence.
<sub>
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 40550, calls 1, tokens in 39093 / out 27160 (reasoning included in out), duration 165.4 s, cost 0.0443 USD (reported), classifier `jev-1.13.0` calls 8, tokens in 6254, cost 0.0003 USD, finding judge `jev-1.13.0` calls 1, tokens in 2114, cost 0.0001 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 23226 / out 253, cost 0.0030 USD (reported), PR document check `jev-1.13.0` calls 10, tokens in 23557, cost 0.0010 USD, build file judge `jev-1.13.0` calls 8, tokens in 7066, cost 0.0003 USD, phases: path_judge 0.5 s, classifier 0.6 s, doc_check 0.7 s, context 5.2 s, model_review 157.5 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main db2fba4).<br>
Reviewed head `eeda61bc3b3332ba27cf252582306f5109aa4e17` against base `db2fba45e738880d6b09a0ab2eba56cced6e7b31`, werkrevisor 0.52.0.
</sub>
</details>
werkrevisor: YELLOW / HUMAN_REVIEW for eeda61bc3b3332ba27cf252582306f5109aa4e17.
The result comment of werkrevisor holds the details.
werkrevisor: YELLOW / HUMAN_REVIEW for `eeda61bc3b3332ba27cf252582306f5109aa4e17`.
The result comment of werkrevisor holds the details.
<!-- werkrevisor-review:eeda61bc3b3332ba27cf252582306f5109aa4e17 -->
ai-review/assessment AI review passed; approved by werkrevisor; merge waits for 1 open hint(s): reply to each line comment and resolve it, then restart the review or push a commit
Finding README.md:1062-1063 (5 of 5 vs. 4 of 4): fixed in 9911267. Both counts held, the README counted hsadmin-ng PR#319 as well; it now names the measured PRs.
Writing rule, paragraph with 6 sentences: fixed in 9911267.
YELLOW "weakened merge gate" for werkrevisor/config.py (Jev 0.75): not changed. skip_green only decides whether a GREEN result gets diagrams; the diagrams never change verdict, status, approval or exit code (Scenario#121.01, #121.03). No check or gate gets weaker. This reason is left to the human.
Answer to the review of `eeda61b`:
- **Finding README.md:1062-1063 (5 of 5 vs. 4 of 4):** fixed in `9911267`. Both counts held, the README counted hsadmin-ng PR#319 as well; it now names the measured PRs.
- **Writing rule, paragraph with 6 sentences:** fixed in `9911267`.
- **YELLOW "weakened merge gate" for `werkrevisor/config.py` (Jev 0.75):** not changed. `skip_green` only decides whether a GREEN result gets diagrams; the diagrams never change verdict, status, approval or exit code (Scenario#121.01, #121.03). No check or gate gets weaker. This reason is left to the human.
The change enables informational diagrams, skips their generation for GREEN results, and improves error reporting. It preserves the merge-gate decisions, secret-scan protections, and isolation of diagram failures from review outcomes.
No findings.
The merge waits for the status ai-review/codex: every line comment resolved, and the approval of a human for a critical PR or for a finding that is only here.
Model `gpt-6-astra`, effort high, tokens in 278458 (cached 221440) / out 1128, 55.3 s, 10 commands; reviewed head `eeda61bc3b3332ba27cf252582306f5109aa4e17`.
<!-- werkrevisor-codex:eeda61bc3b3332ba27cf252582306f5109aa4e17 -->
<!-- werkrevisor-codex-critical:false -->
<!-- werkrevisor-codex-unattached:0 -->
<!-- werkrevisor-codex -->
<!-- werkrevisor-folded -->
<details><summary>⚠️ werkrevisor ERROR for <code>?</code>, superseded by <a href="http://git.javagil.de/mi/werkrevisor/pulls/125#issuecomment-6503">the result for <code>9911267</code></a></summary>
## Codex review: not critical
The change enables informational diagrams, skips their generation for GREEN results, and improves error reporting. It preserves the merge-gate decisions, secret-scan protections, and isolation of diagram failures from review outcomes.
No findings.
The merge waits for the status `ai-review/codex`: every line comment resolved, and the approval of a human for a critical PR or for a finding that is only here.
<sub>
Model `gpt-6-astra`, effort high, tokens in 278458 (cached 221440) / out 1128, 55.3 s, 10 commands; reviewed head `eeda61bc3b3332ba27cf252582306f5109aa4e17`.
</sub>
</details>
Gate: approved by werkrevisor; merge waits for 1 open hint(s): reply to each line comment and resolve it, then restart the review or push a commit.
Change kind: mixed.
Reasons
hint 7 changed file(s) that neither weaken the checks, touch security nor show a risk category of the repository config according to Jev
hint 1 build change(s) of low risk according to Jev: statistics/build.gradle.kts (weakens checks 0.02, risk 0.00, security 0.01)
hint 10 changed file(s) agree with the PR document according to Jev
hint 1 changed file was given to the AI as excerpt of the selected lines and 5 lines around each changed line, not in full (each larger than [review] max_full_file_chars = 80000): werkrevisor/cli.py
hint 1 finding(s) possibly severe, but Jev doubts that they are real: config/werkrevisor.toml:79-80 (advisory: Jev judged every changed file)
hint 1 finding(s) of low severity or rated minor by Jev: docs/prs/2026-10-20-PR#125-diagrams-on.md:47
hint model recommends changes before merge
Findings
[medium] Build timeout at config/werkrevisor.toml:79-80, YELLOW, as a line comment
[medium] Secret exception at docs/prs/2026-10-20-PR#125-diagrams-on.md:47
hint; Jev: real 0.51, severity 2.2 of 3
The scenario promises diagrams for every YELLOW result, but may_draw still rejects every result with a secrets reason; the existing secret-scan test confirms that a possible-secret YELLOW invokes no PR Lens and attaches nothing. Document this intentional security exception so the stated behavior matches the implementation.
Classifier
8 of 8 units passed all questions (1 PR document(s)).
2 answers ignored because of a confidence below [classifier] min_confidence.
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 39255, calls 1, tokens in 37855 / out 17796 (reasoning included in out), duration 173.7 s, cost 0.0327 USD (reported), classifier `jev-1.13.0` calls 8, tokens in 6254, cost 0.0003 USD, finding judge `jev-1.13.0` calls 2, tokens in 2708, cost 0.0001 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 23226 / out 268, cost 0.0004 USD (reported), PR document check `jev-1.13.0` calls 10, tokens in 23583, cost 0.0010 USD, build file judge `jev-1.13.0` calls 8, tokens in 7066, cost 0.0003 USD, phases: path_judge 0.5 s, classifier 0.5 s, doc_check 0.9 s, context 5.2 s, model_review 165.8 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main db2fba4).
Reviewed head `9911267cd9f574dae29a9f0f5a42481de1d0ef23` against base `db2fba45e738880d6b09a0ab2eba56cced6e7b31`, werkrevisor 0.52.0.
<!-- werkrevisor:9911267cd9f574dae29a9f0f5a42481de1d0ef23 -->
<!-- werkrevisor-verdict:GREEN -->
<!-- werkrevisor-blocks:04cb0b7d13f841288b55a2255c7da0ca,babbea6ec33c1e16762df9c50746f63e,bf978d07fd88cf68c2401cb1865aeb7f,98834fe83858b86053f19d6bb1f68763,9daeb8c97aa5ae9f08393a6a396b5f33,d944a358bc7dca7322d85e2315032953,371794da7a3e8af4df7958ed28d4290a,8b1242ae84ecd9cd38dd78a008211247,cf3314d6fa7917e6e5d936aa63e2d07c,8246b345bbea119f36fc4a45682d6b3b,e47b6a6fdd33aa76fa816f47dd993220,fec09e2ac21aacef1d2e05d32adf5992,bc3fb04d7eb6b4ed98b645211a93f443,230929668134f79355aa0499c320a5bf,6349ad1da12830384973aade32db5627,d47100acd03f99054cb986ac14cbeef0,24a5fda583f3f7dd51a6c3160250b2e7,aff003b72561664332ca81d8acdca862,5a9745a0285612b9764bd530abc592b5 -->
<!-- werkrevisor-folded -->
<details><summary>🟢 werkrevisor GREEN for <code>9911267</code>, superseded by <a href="http://git.javagil.de/mi/werkrevisor/pulls/125#issuecomment-6511">the result for <code>69224e9</code></a></summary>
## 🟢 werkrevisor: GREEN / PASS
Gate: approved by werkrevisor; merge waits for 1 open hint(s): reply to each line comment and resolve it, then restart the review or push a commit.
Change kind: `mixed`.
### Reasons
- **hint** 7 changed file(s) that neither weaken the checks, touch security nor show a risk category of the repository config according to Jev
- **hint** 1 build change(s) of low risk according to Jev: statistics/build.gradle.kts (weakens checks 0.02, risk 0.00, security 0.01)
- **hint** 10 changed file(s) agree with the PR document according to Jev
- **hint** 1 changed file was given to the AI as excerpt of the selected lines and 5 lines around each changed line, not in full (each larger than [review] max_full_file_chars = 80000): werkrevisor/cli.py
- **hint** 1 finding(s) possibly severe, but Jev doubts that they are real: config/werkrevisor.toml:79-80 (advisory: Jev judged every changed file)
- **hint** 1 finding(s) of low severity or rated minor by Jev: docs/prs/2026-10-20-PR#125-diagrams-on.md:47
- **hint** model recommends changes before merge
### Findings
1. **[medium] Build timeout** at `config/werkrevisor.toml:79-80`, YELLOW, as a line comment
2. **[medium] Secret exception** at `docs/prs/2026-10-20-PR#125-diagrams-on.md:47`
hint; Jev: real 0.51, severity 2.2 of 3
The scenario promises diagrams for every YELLOW result, but `may_draw` still rejects every result with a `secrets` reason; the existing secret-scan test confirms that a possible-secret YELLOW invokes no PR Lens and attaches nothing. Document this intentional security exception so the stated behavior matches the implementation.
### Classifier
8 of 8 units passed all questions (1 PR document(s)).
2 answers ignored because of a confidence below [classifier] min_confidence.
<sub>
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 39255, calls 1, tokens in 37855 / out 17796 (reasoning included in out), duration 173.7 s, cost 0.0327 USD (reported), classifier `jev-1.13.0` calls 8, tokens in 6254, cost 0.0003 USD, finding judge `jev-1.13.0` calls 2, tokens in 2708, cost 0.0001 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 23226 / out 268, cost 0.0004 USD (reported), PR document check `jev-1.13.0` calls 10, tokens in 23583, cost 0.0010 USD, build file judge `jev-1.13.0` calls 8, tokens in 7066, cost 0.0003 USD, phases: path_judge 0.5 s, classifier 0.5 s, doc_check 0.9 s, context 5.2 s, model_review 165.8 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main db2fba4).<br>
Reviewed head `9911267cd9f574dae29a9f0f5a42481de1d0ef23` against base `db2fba45e738880d6b09a0ab2eba56cced6e7b31`, werkrevisor 0.52.0.
</sub>
</details>
werkrevisor: GREEN / PASS for 9911267cd9f574dae29a9f0f5a42481de1d0ef23.
The result comment of werkrevisor holds the details.
werkrevisor: GREEN / PASS for `9911267cd9f574dae29a9f0f5a42481de1d0ef23`.
The result comment of werkrevisor holds the details.
<!-- werkrevisor-review:9911267cd9f574dae29a9f0f5a42481de1d0ef23 -->
Enabling diagrams adds a PR Lens subprocess that may run for 300 seconds after the review. The file documents a 15-minute Werkator allowance for two 300-second review attempts, so a slow two-attempt review followed by a hanging diagram run can exceed the build limit and terminate before attachment completes. Configure a shorter diagram timeout or increase the build allowance.
**[medium] Build timeout** (werkrevisor)
YELLOW; Jev: real 0.48, severity 2.9 of 3
Enabling diagrams adds a PR Lens subprocess that may run for 300 seconds after the review. The file documents a 15-minute Werkator allowance for two 300-second review attempts, so a slow two-attempt review followed by a hanging diagram run can exceed the build limit and terminate before attachment completes. Configure a shorter diagram timeout or increase the build allowance.
The change enables advisory diagrams, skips their generation for GREEN verdicts, and improves error reporting. It preserves the existing secret checks and failure isolation without changing authorization, merge gates, or stored data.
No findings.
The merge waits for the status ai-review/codex: every line comment resolved, and the approval of a human for a critical PR or for a finding that is only here.
Model `gpt-6-astra`, effort high, tokens in 215328 (cached 161408) / out 1115, 46.9 s, 11 commands; reviewed head `9911267cd9f574dae29a9f0f5a42481de1d0ef23`.
<!-- werkrevisor-codex:9911267cd9f574dae29a9f0f5a42481de1d0ef23 -->
<!-- werkrevisor-codex-critical:false -->
<!-- werkrevisor-codex-unattached:0 -->
<!-- werkrevisor-codex -->
<!-- werkrevisor-folded -->
<details><summary>⚠️ werkrevisor ERROR for <code>?</code>, superseded by <a href="http://git.javagil.de/mi/werkrevisor/pulls/125#issuecomment-6511">the result for <code>69224e9</code></a></summary>
## Codex review: not critical
The change enables advisory diagrams, skips their generation for GREEN verdicts, and improves error reporting. It preserves the existing secret checks and failure isolation without changing authorization, merge gates, or stored data.
No findings.
The merge waits for the status `ai-review/codex`: every line comment resolved, and the approval of a human for a critical PR or for a finding that is only here.
<sub>
Model `gpt-6-astra`, effort high, tokens in 215328 (cached 161408) / out 1115, 46.9 s, 11 commands; reviewed head `9911267cd9f574dae29a9f0f5a42481de1d0ef23`.
</sub>
</details>
hint approval of werkrevisor for 9911267 not carried over: 2 approved block(s) are missing
hint 7 changed file(s) that neither weaken the checks, touch security nor show a risk category of the repository config according to Jev
hint 1 build change(s) of low risk according to Jev: statistics/build.gradle.kts (weakens checks 0.02, risk 0.00, security 0.02)
hint 10 changed file(s) agree with the PR document according to Jev
hint 1 changed file was given to the AI as excerpt of the selected lines and 5 lines around each changed line, not in full (each larger than [review] max_full_file_chars = 80000): werkrevisor/cli.py
Classifier
8 of 8 units passed all questions (1 PR document(s)).
2 answers ignored because of a confidence below [classifier] min_confidence.
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 40171, calls 1, tokens in 38748 / out 26795 (reasoning included in out), duration 695.6 s, cost 0.0345 USD (reported), classifier `jev-1.13.0` calls 8, tokens in 6337, cost 0.0003 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 23226 / out 258, cost 0.0004 USD (reported), PR document check `jev-1.13.0` calls 10, tokens in 24283, cost 0.0010 USD, build file judge `jev-1.13.0` calls 8, tokens in 7116, cost 0.0003 USD, phases: path_judge 0.4 s, classifier 0.4 s, doc_check 0.7 s, context 5.1 s, model_review 688.5 s, config: defaults + host + repository (.werkrevisor.toml at main db2fba4).
Reviewed head `69224e98c433f0ede40e29b2508f3cc4f236d6c5` against base `db2fba45e738880d6b09a0ab2eba56cced6e7b31`, werkrevisor 0.52.0.
<!-- werkrevisor:69224e98c433f0ede40e29b2508f3cc4f236d6c5 -->
<!-- werkrevisor-verdict:GREEN -->
<!-- werkrevisor-blocks:04cb0b7d13f841288b55a2255c7da0ca,babbea6ec33c1e16762df9c50746f63e,c4771c5eef00717c28eb09ec50d7e294,62eb22a832908298347b1186c3fe43ca,9daeb8c97aa5ae9f08393a6a396b5f33,d944a358bc7dca7322d85e2315032953,371794da7a3e8af4df7958ed28d4290a,8b1242ae84ecd9cd38dd78a008211247,cf3314d6fa7917e6e5d936aa63e2d07c,8246b345bbea119f36fc4a45682d6b3b,e47b6a6fdd33aa76fa816f47dd993220,fec09e2ac21aacef1d2e05d32adf5992,bc3fb04d7eb6b4ed98b645211a93f443,230929668134f79355aa0499c320a5bf,6349ad1da12830384973aade32db5627,d47100acd03f99054cb986ac14cbeef0,24a5fda583f3f7dd51a6c3160250b2e7,aff003b72561664332ca81d8acdca862,5a9745a0285612b9764bd530abc592b5 -->
## 🟢 werkrevisor: GREEN / PASS
Change kind: `mixed`.
### Reasons
- **hint** approval of werkrevisor for `9911267` not carried over: 2 approved block(s) are missing
- **hint** 7 changed file(s) that neither weaken the checks, touch security nor show a risk category of the repository config according to Jev
- **hint** 1 build change(s) of low risk according to Jev: statistics/build.gradle.kts (weakens checks 0.02, risk 0.00, security 0.02)
- **hint** 10 changed file(s) agree with the PR document according to Jev
- **hint** 1 changed file was given to the AI as excerpt of the selected lines and 5 lines around each changed line, not in full (each larger than [review] max_full_file_chars = 80000): werkrevisor/cli.py
### Classifier
8 of 8 units passed all questions (1 PR document(s)).
2 answers ignored because of a confidence below [classifier] min_confidence.
<sub>
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 40171, calls 1, tokens in 38748 / out 26795 (reasoning included in out), duration 695.6 s, cost 0.0345 USD (reported), classifier `jev-1.13.0` calls 8, tokens in 6337, cost 0.0003 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 23226 / out 258, cost 0.0004 USD (reported), PR document check `jev-1.13.0` calls 10, tokens in 24283, cost 0.0010 USD, build file judge `jev-1.13.0` calls 8, tokens in 7116, cost 0.0003 USD, phases: path_judge 0.4 s, classifier 0.4 s, doc_check 0.7 s, context 5.1 s, model_review 688.5 s, config: defaults + host + repository (.werkrevisor.toml at main db2fba4).<br>
Reviewed head `69224e98c433f0ede40e29b2508f3cc4f236d6c5` against base `db2fba45e738880d6b09a0ab2eba56cced6e7b31`, werkrevisor 0.52.0.
</sub>
werkrevisor: GREEN / PASS for 69224e98c433f0ede40e29b2508f3cc4f236d6c5.
The result comment of werkrevisor holds the details.
werkrevisor: GREEN / PASS for `69224e98c433f0ede40e29b2508f3cc4f236d6c5`.
The result comment of werkrevisor holds the details.
<!-- werkrevisor-review:69224e98c433f0ede40e29b2508f3cc4f236d6c5 -->
ai-reviewer
scheduled this pull request to auto merge when all checks succeed 2026-10-20 12:39:00 +02:00
ai-reviewer
removed review request for mi 2026-10-20 12:39:00 +02:00
The change enables optional diagrams, skips generation for GREEN results, and improves error reporting. It preserves the secret-scan checks and does not weaken review gates, alter permissions, or modify existing data irreversibly.
No findings.
The merge waits for the status ai-review/codex: every line comment resolved, and the approval of a human for a critical PR or for a finding that is only here.
Model `gpt-6-astra`, effort high, tokens in 273462 (cached 218496) / out 1248, 55.3 s, 10 commands; reviewed head `69224e98c433f0ede40e29b2508f3cc4f236d6c5`.
<!-- werkrevisor-codex:69224e98c433f0ede40e29b2508f3cc4f236d6c5 -->
<!-- werkrevisor-codex-critical:false -->
<!-- werkrevisor-codex-unattached:0 -->
<!-- werkrevisor-codex -->
## Codex review: not critical
The change enables optional diagrams, skips generation for GREEN results, and improves error reporting. It preserves the secret-scan checks and does not weaken review gates, alter permissions, or modify existing data irreversibly.
No findings.
The merge waits for the status `ai-review/codex`: every line comment resolved, and the approval of a human for a critical PR or for a finding that is only here.
<sub>
Model `gpt-6-astra`, effort high, tokens in 273462 (cached 218496) / out 1248, 55.3 s, 10 commands; reviewed head `69224e98c433f0ede40e29b2508f3cc4f236d6c5`.
</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
See docs/prs for the PR document.
🤖 Generated with Claude Code
🟡 werkrevisor YELLOW for
eeda61b, superseded by the result for9911267🟡 werkrevisor: YELLOW / HUMAN_REVIEW
Change kind:
mixed.Reasons
Findings
[medium] Documentation Contradiction at
README.md:1062-1063hint; Jev: real 0.81, severity 2.0 of 3
The README states "Sonnet 5.5 did on 5 of 5 PRs, GPT-6 Luna too but with rough boxes only, MiniMax M3 on none". This contradicts the PR document at
docs/prs/2026-10-20-PR#125-diagrams-on.mdand the comment inconfig/werkrevisor.toml, both of which list 4 measured PRs (PR#117, PR#118, PR#122, PR#123) with Sonnet 5.5 and GPT-6 Luna at 4 of 4 and MiniMax M3 at 0 of 1. A reader who consults the README and then the PR document will see different sample sizes for the same model evaluation, which weakens trust in the choice of model.Classifier
8 of 8 units passed all questions (1 PR document(s)).
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 40550, calls 1, tokens in 39093 / out 27160 (reasoning included in out), duration 165.4 s, cost 0.0443 USD (reported), classifier `jev-1.13.0` calls 8, tokens in 6254, cost 0.0003 USD, finding judge `jev-1.13.0` calls 1, tokens in 2114, cost 0.0001 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 23226 / out 253, cost 0.0030 USD (reported), PR document check `jev-1.13.0` calls 10, tokens in 23557, cost 0.0010 USD, build file judge `jev-1.13.0` calls 8, tokens in 7066, cost 0.0003 USD, phases: path_judge 0.5 s, classifier 0.6 s, doc_check 0.7 s, context 5.2 s, model_review 157.5 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main2 answers ignored because of a confidence below [classifier] min_confidence.
db2fba4).Reviewed head `eeda61bc3b3332ba27cf252582306f5109aa4e17` against base `db2fba45e738880d6b09a0ab2eba56cced6e7b31`, werkrevisor 0.52.0.
werkrevisor: YELLOW / HUMAN_REVIEW for
eeda61bc3b3332ba27cf252582306f5109aa4e17.The result comment of werkrevisor holds the details.
@@ -1057,3 +1057,4 @@| `max_diagrams` | `5` | Gitea takes at most 5 attachments per comment by default (`[attachment] MAX_FILES`). || `skip_green` | `true` | No diagrams for a GREEN result: it merges without a human, so nobody would look at them. |Only the host config sets the section, since the diagrams cost a second model call:Writing rule: paragraph with 6 sentences, more than 5 (werkrevisor writing rule)
Fix the line, or reply why it stays, and resolve this comment.
Fixed in
9911267: two paragraphs, 4 and 5 sentences.Answer to the review of
eeda61b:9911267. Both counts held, the README counted hsadmin-ng PR#319 as well; it now names the measured PRs.9911267.werkrevisor/config.py(Jev 0.75): not changed.skip_greenonly decides whether a GREEN result gets diagrams; the diagrams never change verdict, status, approval or exit code (Scenario#121.01, #121.03). No check or gate gets weaker. This reason is left to the human.⚠️ werkrevisor ERROR for
?, superseded by the result for9911267Codex review: not critical
The change enables informational diagrams, skips their generation for GREEN results, and improves error reporting. It preserves the merge-gate decisions, secret-scan protections, and isolation of diagram failures from review outcomes.
No findings.
The merge waits for the status
Model `gpt-6-astra`, effort high, tokens in 278458 (cached 221440) / out 1128, 55.3 s, 10 commands; reviewed head `eeda61bc3b3332ba27cf252582306f5109aa4e17`.ai-review/codex: every line comment resolved, and the approval of a human for a critical PR or for a finding that is only here.🟢 werkrevisor GREEN for
9911267, superseded by the result for69224e9🟢 werkrevisor: GREEN / PASS
Gate: approved by werkrevisor; merge waits for 1 open hint(s): reply to each line comment and resolve it, then restart the review or push a commit.
Change kind:
mixed.Reasons
Findings
[medium] Build timeout at
config/werkrevisor.toml:79-80, YELLOW, as a line comment[medium] Secret exception at
docs/prs/2026-10-20-PR#125-diagrams-on.md:47hint; Jev: real 0.51, severity 2.2 of 3
The scenario promises diagrams for every YELLOW result, but
may_drawstill rejects every result with asecretsreason; the existing secret-scan test confirms that a possible-secret YELLOW invokes no PR Lens and attaches nothing. Document this intentional security exception so the stated behavior matches the implementation.Classifier
8 of 8 units passed all questions (1 PR document(s)).
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 39255, calls 1, tokens in 37855 / out 17796 (reasoning included in out), duration 173.7 s, cost 0.0327 USD (reported), classifier `jev-1.13.0` calls 8, tokens in 6254, cost 0.0003 USD, finding judge `jev-1.13.0` calls 2, tokens in 2708, cost 0.0001 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 23226 / out 268, cost 0.0004 USD (reported), PR document check `jev-1.13.0` calls 10, tokens in 23583, cost 0.0010 USD, build file judge `jev-1.13.0` calls 8, tokens in 7066, cost 0.0003 USD, phases: path_judge 0.5 s, classifier 0.5 s, doc_check 0.9 s, context 5.2 s, model_review 165.8 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main2 answers ignored because of a confidence below [classifier] min_confidence.
db2fba4).Reviewed head `9911267cd9f574dae29a9f0f5a42481de1d0ef23` against base `db2fba45e738880d6b09a0ab2eba56cced6e7b31`, werkrevisor 0.52.0.
werkrevisor: GREEN / PASS for
9911267cd9f574dae29a9f0f5a42481de1d0ef23.The result comment of werkrevisor holds the details.
@@ -74,0 +76,4 @@# Measured on PR#117, PR#118, PR#122 and PR#123: Sonnet names the changed functions, GPT-6 Luna only rough boxes,# and the graph documents of MiniMax M3 do not validate.# Only PRs that a human reviews get them (skip_green), at about 0.10 to 0.20 USD per head.enabled = true[medium] Build timeout (werkrevisor)
YELLOW; Jev: real 0.48, severity 2.9 of 3
Enabling diagrams adds a PR Lens subprocess that may run for 300 seconds after the review. The file documents a 15-minute Werkator allowance for two 300-second review attempts, so a slow two-attempt review followed by a hanging diagram run can exceed the build limit and terminate before attachment completes. Configure a shorter diagram timeout or increase the build allowance.
Fixed in
69224e9:timeout_seconds = 120for werkrevisor; Sonnet needed 18 to 43 s, and 2 × 300 s + 120 s stay within the 15 minutes of the build.New commits pushed, approval review dismissed automatically according to repository settings
⚠️ werkrevisor ERROR for
?, superseded by the result for69224e9Codex review: not critical
The change enables advisory diagrams, skips their generation for GREEN verdicts, and improves error reporting. It preserves the existing secret checks and failure isolation without changing authorization, merge gates, or stored data.
No findings.
The merge waits for the status
Model `gpt-6-astra`, effort high, tokens in 215328 (cached 161408) / out 1115, 46.9 s, 11 commands; reviewed head `9911267cd9f574dae29a9f0f5a42481de1d0ef23`.ai-review/codex: every line comment resolved, and the approval of a human for a critical PR or for a finding that is only here.🟢 werkrevisor: GREEN / PASS
Change kind:
mixed.Reasons
9911267not carried over: 2 approved block(s) are missingClassifier
8 of 8 units passed all questions (1 PR document(s)).
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 40171, calls 1, tokens in 38748 / out 26795 (reasoning included in out), duration 695.6 s, cost 0.0345 USD (reported), classifier `jev-1.13.0` calls 8, tokens in 6337, cost 0.0003 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 23226 / out 258, cost 0.0004 USD (reported), PR document check `jev-1.13.0` calls 10, tokens in 24283, cost 0.0010 USD, build file judge `jev-1.13.0` calls 8, tokens in 7116, cost 0.0003 USD, phases: path_judge 0.4 s, classifier 0.4 s, doc_check 0.7 s, context 5.1 s, model_review 688.5 s, config: defaults + host + repository (.werkrevisor.toml at main2 answers ignored because of a confidence below [classifier] min_confidence.
db2fba4).Reviewed head `69224e98c433f0ede40e29b2508f3cc4f236d6c5` against base `db2fba45e738880d6b09a0ab2eba56cced6e7b31`, werkrevisor 0.52.0.
werkrevisor: GREEN / PASS for
69224e98c433f0ede40e29b2508f3cc4f236d6c5.The result comment of werkrevisor holds the details.
Codex review: not critical
The change enables optional diagrams, skips generation for GREEN results, and improves error reporting. It preserves the secret-scan checks and does not weaken review gates, alter permissions, or modify existing data irreversibly.
No findings.
The merge waits for the status
Model `gpt-6-astra`, effort high, tokens in 273462 (cached 218496) / out 1248, 55.3 s, 10 commands; reviewed head `69224e98c433f0ede40e29b2508f3cc4f236d6c5`.ai-review/codex: every line comment resolved, and the approval of a human for a critical PR or for a finding that is only here.