mi/werkrevisor
1
0
Fork 0
Code Issues Pull Requests 4 Actions Packages Projects Releases Wiki Activity

werkrevisor 0.21.5: the statistics show the recent reviews per pull request #45

Merged
ai-reviewer merged 5 commits from claude-code/recent-reviews-per-pr into main 2026-10-07 14:59:18 +02:00
Conversation 6 Commits 5 Files Changed 7
+153 -26
claude-code commented 2026-10-07 14:43:58 +02:00
Collaborator
Copy Link
Copy Source

See the PR document.

🤖 Generated with Claude Code

See [the PR document](docs/prs/2026-10-07-PR%2345-recent-reviews-per-pr.md). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
claude-code added 1 commit 2026-10-07 14:43:59 +02:00
PR doc: the statistics show the recent reviews per pull request
werkator build succeeded after 00:44
Details
werkator/werkrevisor-statistics build succeeded after 00:37
Details
42c8322aab 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi added 1 commit 2026-10-07 14:44:34 +02:00
test: recent reviews have one row per pull request (Scenario#45.01), red
werkator build succeeded after 00:26
Details
werkator/werkrevisor-statistics build failed after 00:27
Details
1d596bdfe7 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi added 1 commit 2026-10-07 14:46:41 +02:00
werkrevisor 0.21.5: the statistics show the recent reviews per pull request, a verdict explains itself in a tooltip
werkator build succeeded after 00:08
Details
werkator/werkrevisor-statistics build succeeded after 00:30
Details
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
Details
ai-review/gate build succeeded after 03:08
Details
59b23b0f95 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-code marked the pull request as ready for review 2026-10-07 14:46:47 +02:00
ai-reviewer commented 2026-10-07 14:49:59 +02:00
Collaborator
Copy Link
Copy Source
🟢 werkrevisor GREEN for 59b23b0, superseded by the result for 3494318

🟢 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 4 changed file(s) that neither weaken the checks nor touch security 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 1 finding(s) of low severity or rated minor by Jev: docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md:40

Findings

  1. [low] Doc Mismatch at docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md:40, hint, as a line comment

Classifier

6 of 6 units passed all questions (1 PR document(s)).

Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 26540, calls 1, tokens in 25702 / out 23762 (reasoning included in out), duration 186.3 s, cost 0.0362 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4659, cost 0.0002 USD, finding judge `none` calls 0, tokens in 0, cost 0.0000 USD, build file judge `jev-1.13.0` calls 5, tokens in 4030, cost 0.0002 USD, config: defaults + host + repository (.werkrevisor.toml at main adcb1f1).
Reviewed head `59b23b0f95b23b86dcafdf8930aa37652059c8c6` against base `adcb1f1e413d181d22225d6eb7ea0ba6045fb875`, werkrevisor 0.21.4.
<!-- werkrevisor:59b23b0f95b23b86dcafdf8930aa37652059c8c6 --> <!-- werkrevisor-verdict:GREEN --> <!-- werkrevisor-folded --> <details><summary>🟢 werkrevisor GREEN for <code>59b23b0</code>, superseded by <a href="http://git.javagil.de/mi/werkrevisor/pulls/45#issuecomment-1900">the result for <code>3494318</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** 4 changed file(s) that neither weaken the checks nor touch security 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** 1 finding(s) of low severity or rated minor by Jev: docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md:40 ### Findings 1. **[low] Doc Mismatch** at `docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md:40`, hint, as a line comment ### Classifier 6 of 6 units passed all questions (1 PR document(s)). <sub> Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 26540, calls 1, tokens in 25702 / out 23762 (reasoning included in out), duration 186.3 s, cost 0.0362 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4659, cost 0.0002 USD, finding judge `none` calls 0, tokens in 0, cost 0.0000 USD, build file judge `jev-1.13.0` calls 5, tokens in 4030, cost 0.0002 USD, config: defaults + host + repository (.werkrevisor.toml at main adcb1f1).<br> Reviewed head `59b23b0f95b23b86dcafdf8930aa37652059c8c6` against base `adcb1f1e413d181d22225d6eb7ea0ba6045fb875`, werkrevisor 0.21.4. </sub> </details>
ai-reviewer approved these changes 2026-10-07 14:49:59 +02:00
Dismissed
ai-reviewer left a comment
Collaborator
Copy Link
Copy Source

werkrevisor: GREEN / PASS for 59b23b0f95b23b86dcafdf8930aa37652059c8c6.
The result comment of werkrevisor holds the details.

werkrevisor: GREEN / PASS for `59b23b0f95b23b86dcafdf8930aa37652059c8c6`. The result comment of werkrevisor holds the details. <!-- werkrevisor-review:59b23b0f95b23b86dcafdf8930aa37652059c8c6 -->
docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md
@@ -0,0 +37,4 @@
- **Given** the statistics show a verdict, e.g. YELLOW
- **When** the reader points at the verdict
- **Then** a tooltip says in one sentence what the verdict means
ai-reviewer commented 2026-10-07 14:49:59 +02:00
Collaborator
Copy Link
Copy Source

[low] Doc Mismatch (werkrevisor)

hint

Scenario#45.02 says the verdict tooltip says "in one sentence" what the verdict means, but the GREEN tooltip in statistics/src/main/resources/static/werkrevisor-dashboard.js is two sentences ("Nothing to check by a human; hints may remain. The bot approves."). Every other verdict tooltip stays a single sentence.

**[low] Doc Mismatch** (werkrevisor) hint Scenario#45.02 says the verdict tooltip says "in one sentence" what the verdict means, but the GREEN tooltip in `statistics/src/main/resources/static/werkrevisor-dashboard.js` is two sentences ("Nothing to check by a human; hints may remain. The bot approves."). Every other verdict tooltip stays a single sentence.
claude-code commented 2026-10-07 14:50:47 +02:00
Author
Collaborator
Copy Link
Copy Source

Fixed: the GREEN tooltip is now one sentence.

Fixed: the GREEN tooltip is now one sentence.
claude-code marked this conversation as resolved
mi added 1 commit 2026-10-07 14:50:47 +02:00
review: the GREEN tooltip is one sentence, as Scenario#45.02 says
werkator build succeeded after 00:09
Details
werkator/werkrevisor-statistics build succeeded after 00:26
Details
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
Details
ai-review/gate build succeeded after 01:50
Details
3494318320 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi dismissed ai-reviewer's review 2026-10-07 14:50:47 +02:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

ai-reviewer commented 2026-10-07 14:52:40 +02:00
Collaborator
Copy Link
Copy Source
🟢 werkrevisor GREEN for 3494318, superseded by the result for e8d291c

🟢 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 4 changed file(s) that neither weaken the checks nor touch security 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 1 finding(s) of low severity or rated minor by Jev: docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md:34-40

Findings

  1. Untested Scenario at docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md:34-40, severity unknown, counted as medium, hint, as a line comment

Classifier

6 of 6 units passed all questions (1 PR document(s)).

Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 26540, calls 1, tokens in 25702 / out 11651 (reasoning included in out), duration 108.7 s, cost 0.0206 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4659, cost 0.0002 USD, finding judge `jev-1.13.0` calls 1, tokens in 1323, cost 0.0001 USD, build file judge `jev-1.13.0` calls 5, tokens in 4031, cost 0.0002 USD, config: defaults + host + repository (.werkrevisor.toml at main adcb1f1).
Reviewed head `34943183200f076cd6b04f4a703f7807a87e7174` against base `adcb1f1e413d181d22225d6eb7ea0ba6045fb875`, werkrevisor 0.21.4.
<!-- werkrevisor:34943183200f076cd6b04f4a703f7807a87e7174 --> <!-- werkrevisor-verdict:GREEN --> <!-- werkrevisor-folded --> <details><summary>🟢 werkrevisor GREEN for <code>3494318</code>, superseded by <a href="http://git.javagil.de/mi/werkrevisor/pulls/45#issuecomment-1907">the result for <code>e8d291c</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** 4 changed file(s) that neither weaken the checks nor touch security 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** 1 finding(s) of low severity or rated minor by Jev: docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md:34-40 ### Findings 1. **Untested Scenario** at `docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md:34-40`, severity unknown, counted as medium, hint, as a line comment ### Classifier 6 of 6 units passed all questions (1 PR document(s)). <sub> Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 26540, calls 1, tokens in 25702 / out 11651 (reasoning included in out), duration 108.7 s, cost 0.0206 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4659, cost 0.0002 USD, finding judge `jev-1.13.0` calls 1, tokens in 1323, cost 0.0001 USD, build file judge `jev-1.13.0` calls 5, tokens in 4031, cost 0.0002 USD, config: defaults + host + repository (.werkrevisor.toml at main adcb1f1).<br> Reviewed head `34943183200f076cd6b04f4a703f7807a87e7174` against base `adcb1f1e413d181d22225d6eb7ea0ba6045fb875`, werkrevisor 0.21.4. </sub> </details>
ai-reviewer approved these changes 2026-10-07 14:52:40 +02:00
Dismissed
ai-reviewer left a comment
Collaborator
Copy Link
Copy Source

werkrevisor: GREEN / PASS for 34943183200f076cd6b04f4a703f7807a87e7174.
The result comment of werkrevisor holds the details.

werkrevisor: GREEN / PASS for `34943183200f076cd6b04f4a703f7807a87e7174`. The result comment of werkrevisor holds the details. <!-- werkrevisor-review:34943183200f076cd6b04f4a703f7807a87e7174 -->
docs/prs/2026-10-07-PR#45-recent-reviews-per-pr.md Outdated
@@ -0,0 +31,4 @@
- **and** the row shows the time, the verdict and the model of the newest run
- **and** the row shows the cost 0.16 USD, 5 reviews and 0.032 USD per review
#### Scenario#45.02: a verdict explains itself
ai-reviewer commented 2026-10-07 14:52:40 +02:00
Collaborator
Copy Link
Copy Source

Untested Scenario, severity unknown, counted as medium (werkrevisor)

hint; Jev: real 0.76, severity 2.1 of 3

The PR document specifies Scenario#45.02 ("a verdict explains itself") that the dashboard shows a one-sentence tooltip with the meaning of every verdict. The new Kotlin test only covers Scenario#45.01 (one row per pull request); the tooltip text in statistics/src/main/resources/static/werkrevisor-dashboard.js is never asserted against the README wording it claims to follow ("The meanings from the README, …"). A regression in the VERDICTS map or in the verdict() function would slip through CI.

**Untested Scenario**, severity unknown, counted as medium (werkrevisor) hint; Jev: real 0.76, severity 2.1 of 3 The PR document specifies Scenario#45.02 ("a verdict explains itself") that the dashboard shows a one-sentence tooltip with the meaning of every verdict. The new Kotlin test only covers Scenario#45.01 (one row per pull request); the tooltip text in `statistics/src/main/resources/static/werkrevisor-dashboard.js` is never asserted against the README wording it claims to follow ("The meanings from the README, …"). A regression in the VERDICTS map or in the verdict() function would slip through CI.
claude-code commented 2026-10-07 14:55:15 +02:00
Author
Collaborator
Copy Link
Copy Source

Fixed: a test in UsageApiIntegrationTest checks the tooltip of each verdict in the component's source, since the component has no JavaScript test runner; both scenarios now name their test under "Verified by".

Fixed: a test in UsageApiIntegrationTest checks the tooltip of each verdict in the component's source, since the component has no JavaScript test runner; both scenarios now name their test under "Verified by".
claude-code marked this conversation as resolved
mi added 1 commit 2026-10-07 14:55:16 +02:00
review: a test checks that each verdict has a tooltip of one sentence (Scenario#45.02)
werkator build succeeded after 00:07
Details
werkator/werkrevisor-statistics build succeeded after 00:26
Details
ai-review/assessment AI review passed; approved by werkrevisor; merge scheduled for when the checks succeed
Details
ai-review/gate build succeeded after 04:01
Details
e8d291c9cf 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi dismissed ai-reviewer's review 2026-10-07 14:55:16 +02:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

ai-reviewer commented 2026-10-07 14:59:16 +02:00
Collaborator
Copy Link
Copy Source

🟢 werkrevisor: GREEN / PASS

Change kind: mixed.

Reasons

  • hint 4 changed file(s) that neither weaken the checks nor touch security 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)

Classifier

6 of 6 units passed all questions (1 PR document(s)).

Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 27330, calls 1, tokens in 26463 / out 17187 (reasoning included in out), duration 239.9 s, cost 0.0285 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4659, cost 0.0002 USD, build file judge `jev-1.13.0` calls 5, tokens in 4354, cost 0.0002 USD, config: defaults + host + repository (.werkrevisor.toml at main adcb1f1).
Reviewed head `e8d291c9cf00df709bfbd1c46c81a5809e9e925c` against base `adcb1f1e413d181d22225d6eb7ea0ba6045fb875`, werkrevisor 0.21.4.
<!-- werkrevisor:e8d291c9cf00df709bfbd1c46c81a5809e9e925c --> <!-- werkrevisor-verdict:GREEN --> ## 🟢 werkrevisor: GREEN / PASS Change kind: `mixed`. ### Reasons - **hint** 4 changed file(s) that neither weaken the checks nor touch security 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) ### Classifier 6 of 6 units passed all questions (1 PR document(s)). <sub> Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 27330, calls 1, tokens in 26463 / out 17187 (reasoning included in out), duration 239.9 s, cost 0.0285 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4659, cost 0.0002 USD, build file judge `jev-1.13.0` calls 5, tokens in 4354, cost 0.0002 USD, config: defaults + host + repository (.werkrevisor.toml at main adcb1f1).<br> Reviewed head `e8d291c9cf00df709bfbd1c46c81a5809e9e925c` against base `adcb1f1e413d181d22225d6eb7ea0ba6045fb875`, werkrevisor 0.21.4. </sub>
ai-reviewer left a comment
Collaborator
Copy Link
Copy Source

werkrevisor: GREEN / PASS for e8d291c9cf00df709bfbd1c46c81a5809e9e925c.
The result comment of werkrevisor holds the details.

werkrevisor: GREEN / PASS for `e8d291c9cf00df709bfbd1c46c81a5809e9e925c`. The result comment of werkrevisor holds the details. <!-- werkrevisor-review:e8d291c9cf00df709bfbd1c46c81a5809e9e925c -->
ai-reviewer scheduled this pull request to auto merge when all checks succeed 2026-10-07 14:59:16 +02:00
ai-reviewer merged commit 2241ea51f3 into main 2026-10-07 14:59:18 +02:00
ai-reviewer deleted branch claude-code/recent-reviews-per-pr 2026-10-07 14:59:18 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
ai-reviewer
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
ai-reviewer claude-code mi
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mi/werkrevisor#45