werkrevisor 0.40.1: the README and the diagrams describe the current review #99
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: mi/werkrevisor#99
Reference in New Issue
Block a user
Corrects the README and the diagrams to the review of 0.40.0.
See the PR document under
docs/prs/.🤖 Generated with Claude Code
🟢 werkrevisor GREEN for
3e8d250, superseded by the result fore0fb03b🟢 werkrevisor: GREEN / PASS
Gate: approved by werkrevisor; merge waits for 2 open hint(s): reply to each line comment and resolve it, then restart the review or push a commit.
Change kind:
mixed.Reasons
Findings
Mermaid chart claim contradicts README at
docs/prs/2026-10-13-PR#99-readme-workflow.md:38-39, severity unknown, counted as medium, hint, as a line commentPR workflow diagram change unverifiable at
docs/prs/2026-10-13-PR#99-readme-workflow.md:40-41, severity unknown, counted as medium, hint, as a line commentClassifier
6 of 6 units passed all questions (1 PR document(s)).
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 19411, calls 1, tokens in 18685 / out 8324 (reasoning included in out), duration 111.9 s, cost 0.0156 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4258, cost 0.0002 USD, finding judge `jev-1.13.0` calls 2, tokens in 2571, cost 0.0001 USD, change summary `openrouter/minimax/minimax-m3` via Minimax calls 1, tokens in 19897 / out 1537, cost 0.0078 USD (reported), PR document check `jev-1.13.0` calls 6, tokens in 10870, cost 0.0005 USD, build file judge `jev-1.13.0` calls 4, tokens in 3117, cost 0.0001 USD, phases: path_judge 0.3 s, classifier 0.5 s, change_summary 27.5 s, doc_check 0.6 s, context 1.2 s, model_review 81.2 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main1 answer ignored because of a confidence below [classifier] min_confidence.
fb0c20a).Reviewed head `3e8d250135032f3999ab5694377541ae81d2a3ec` against base `fb0c20af25b996c7ac0a07779b6dbeea7f137f09`, werkrevisor 0.40.0.
werkrevisor: GREEN / PASS for
3e8d250135032f3999ab5694377541ae81d2a3ec.The result comment of werkrevisor holds the details.
@@ -0,0 +35,4 @@step 2 the secret scan and the shortcut, step 10 the escalation and every write to Gitea.- The decisions diagram asks for a secret before the shortcut; the shortcut ends in "Verdict of the rules alone",and a new decision shows the escalation after `max_red_rounds`.- The Mermaid chart, the inside diagram and the PR workflow diagram say "advises or decides";Mermaid chart claim contradicts README, severity unknown, counted as medium (werkrevisor)
hint; Jev: real 0.80, severity 1.3 of 3
The PR document states "The Mermaid chart, the inside diagram and the PR workflow diagram say 'advises or decides'". However, the Mermaid chart in the README (line 223) only says "Model review by PR-Agent, with the hints of Jev" — the word "advises" was removed and "advises or decides" was never added. The inside diagram and the decisions diagram do say "advises or decides", so the PR doc is correct about those, but wrong about the Mermaid chart. Either the Mermaid chart needs to be updated to match, or the PR doc must be corrected.
Fixed in
e0fb03b: the PR document now says that the Mermaid chart no longer claims that the model advises; only the inside and the PR workflow diagram say "advises or decides".@@ -0,0 +37,4 @@and a new decision shows the escalation after `max_red_rounds`.- The Mermaid chart, the inside diagram and the PR workflow diagram say "advises or decides";the inside diagram names no model of a host in its steps.- The PR workflow diagram gives 2 to 6 minutes, names the images, the carry-over of an approval,PR workflow diagram change unverifiable, severity unknown, counted as medium (werkrevisor)
hint; Jev: real 0.73, severity 1.3 of 3
The PR document claims changes to "the PR workflow diagram" (2 to 6 minutes, images, carry-over of an approval, arrow from a requested change to the rework), but no PR workflow diagram appears in the diff. Only the decisions diagram and the inside diagram were updated. If the PR workflow diagram is a separate file not included in this PR, the PR doc overstates what this PR does; if it should have been updated, the change is missing.
Not changed: the PR workflow diagram is in this PR, commit
2d29aa3changesdocs/werkrevisor-pr-workflow.svgand its dark variant. The model does not see SVG files; the result comment lists them as files the model cannot review.New commits pushed, approval review dismissed automatically according to repository settings
🟢 werkrevisor: GREEN / PASS
Change kind:
mixed.Reasons
3e8d250not carried over: 1 approved block(s) are missingClassifier
6 of 6 units passed all questions (1 PR document(s)).
Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 19424, calls 1, tokens in 18698 / out 16110 (reasoning included in out), duration 142.6 s, cost 0.0229 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4267, cost 0.0002 USD, change summary `openrouter/minimax/minimax-m3` via Minimax calls 1, tokens in 19897 / out 1553, cost 0.0078 USD (reported), PR document check `jev-1.13.0` calls 6, tokens in 9985, cost 0.0004 USD, build file judge `jev-1.13.0` calls 4, tokens in 3117, cost 0.0001 USD, phases: path_judge 0.2 s, classifier 0.5 s, change_summary 12.3 s, doc_check 0.6 s, context 0.3 s, model_review 128.2 s, config: defaults + host + repository (.werkrevisor.toml at main1 answer ignored because of a confidence below [classifier] min_confidence.
fb0c20a).Reviewed head `e0fb03bf0fb703f67b4e6eebb68d3be3176fb52c` against base `fb0c20af25b996c7ac0a07779b6dbeea7f137f09`, werkrevisor 0.40.0.
werkrevisor: GREEN / PASS for
e0fb03bf0fb703f67b4e6eebb68d3be3176fb52c.The result comment of werkrevisor holds the details.