werkrevisor 0.32.7: a plus in the text is no added line #80

Merged
ai-reviewer merged 3 commits from claude-code/diff-marker-note into main 2026-10-11 08:55:10 +02:00
Collaborator

A note for the model when an unchanged diff line starts with + or -, e.g. an optional Werkbaum node.

See docs/prs/2026-10-11-PR#80-diff-marker-note.md.

🤖 Generated with Claude Code

A note for the model when an unchanged diff line starts with + or -, e.g. an optional Werkbaum node. See docs/prs/2026-10-11-PR#80-diff-marker-note.md. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
claude-code added 1 commit 2026-10-11 08:46:16 +02:00
PR document: a plus in the text is no added line
werkator build succeeded after 00:13
werkator/werkrevisor-statistics build succeeded after 00:41
ba204ca817
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi added 1 commit 2026-10-11 08:47:28 +02:00
A note for the model when an unchanged diff line starts with + or -
werkator build succeeded after 00:09
werkator/werkrevisor-statistics build succeeded after 00:43
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
ai-review/gate build succeeded after 01:18
f355f3e323
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-code marked the pull request as ready for review 2026-10-11 08:47:43 +02:00
Collaborator
🟢 werkrevisor GREEN for f355f3e, superseded by the result for e72687a

🟢 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 6 section(s) of the PR document agree with the change summary of openrouter/minimax/minimax-m3 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: docs/prs/2026-10-11-PR#80-diff-marker-note.md:28-47

Findings

  1. Undocumented Markdown scenario at docs/prs/2026-10-11-PR#80-diff-marker-note.md:28-47, severity unknown, counted as medium, hint, as a line comment

Classifier

6 of 6 units passed all questions (1 PR document(s)).
1 answer ignored because of a confidence below [classifier] min_confidence.

Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 15047, calls 1, tokens in 14550 / out 8715 (reasoning included in out), duration 75.1 s, cost 0.0148 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4480, cost 0.0002 USD, finding judge `jev-1.13.0` calls 1, tokens in 1382, cost 0.0001 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 21115 / out 277, cost 0.0028 USD (reported), change summary `openrouter/minimax/minimax-m3` via Minimax calls 1, tokens in 1556 / out 853, cost 0.0015 USD (reported), PR document check `jev-1.13.0` calls 6, tokens in 6892, cost 0.0003 USD, build file judge `jev-1.13.0` calls 5, tokens in 3138, cost 0.0001 USD, phases: path_judge 0.4 s, classifier 0.4 s, change_summary 5.5 s, doc_check 0.5 s, context 5.5 s, model_review 62.1 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main 54fe179).
Reviewed head `f355f3e32302c4957232f7e60464421bfefeadf1` against base `54fe179f4d2460ae400ea3fd1dd41bb23a4916c5`, werkrevisor 0.32.6.
<!-- werkrevisor:f355f3e32302c4957232f7e60464421bfefeadf1 --> <!-- werkrevisor-verdict:GREEN --> <!-- werkrevisor-blocks:3eba7e3699de5dd5a6b0984bac2b09c3,63f19d926c5fb5039b765d2b5809ae8c,4f1cfa8d373f6a9c89422aef8827af15,57ece0ab34c7d79d700f6b1a42f26614,ae37885e6a0f516aa8b6bfdcc9139451,902b78ba936bfbe07baf8eaab281213d,70e59913d42a4a4ff32e74f16b86b233,244495aaae870ef781d318f685f0b5bb --> <!-- werkrevisor-folded --> <details><summary>🟢 werkrevisor GREEN for <code>f355f3e</code>, superseded by <a href="http://git.javagil.de/mi/werkrevisor/pulls/80#issuecomment-3637">the result for <code>e72687a</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** 6 section(s) of the PR document agree with the change summary of openrouter/minimax/minimax-m3 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: docs/prs/2026-10-11-PR#80-diff-marker-note.md:28-47 ### Findings 1. **Undocumented Markdown scenario** at `docs/prs/2026-10-11-PR#80-diff-marker-note.md:28-47`, severity unknown, counted as medium, hint, as a line comment ### Classifier 6 of 6 units passed all questions (1 PR document(s)). 1 answer ignored because of a confidence below [classifier] min_confidence. <sub> Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 15047, calls 1, tokens in 14550 / out 8715 (reasoning included in out), duration 75.1 s, cost 0.0148 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4480, cost 0.0002 USD, finding judge `jev-1.13.0` calls 1, tokens in 1382, cost 0.0001 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 21115 / out 277, cost 0.0028 USD (reported), change summary `openrouter/minimax/minimax-m3` via Minimax calls 1, tokens in 1556 / out 853, cost 0.0015 USD (reported), PR document check `jev-1.13.0` calls 6, tokens in 6892, cost 0.0003 USD, build file judge `jev-1.13.0` calls 5, tokens in 3138, cost 0.0001 USD, phases: path_judge 0.4 s, classifier 0.4 s, change_summary 5.5 s, doc_check 0.5 s, context 5.5 s, model_review 62.1 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main 54fe179).<br> Reviewed head `f355f3e32302c4957232f7e60464421bfefeadf1` against base `54fe179f4d2460ae400ea3fd1dd41bb23a4916c5`, werkrevisor 0.32.6. </sub> </details>
ai-reviewer approved these changes 2026-10-11 08:48:59 +02:00
Dismissed
ai-reviewer left a comment
Collaborator

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

werkrevisor: GREEN / PASS for `f355f3e32302c4957232f7e60464421bfefeadf1`. The result comment of werkrevisor holds the details. <!-- werkrevisor-review:f355f3e32302c4957232f7e60464421bfefeadf1 -->
@@ -0,0 +25,4 @@
### Feature: the model tells the text from the marker
#### Scenario#80.01: an unchanged line whose text starts with a plus gets a note
Collaborator

Undocumented Markdown scenario, severity unknown, counted as medium (werkrevisor)

hint; Jev: real 0.76, severity 2.0 of 3

The test file adds three test methods, but the PR document specifies only two scenarios. The third test, test_an_unchanged_markdown_item_gets_the_note, checks that an unchanged Markdown list item starting with - triggers the note; it has no scenario in the document, no Scenario#80.0x docstring, and Scenario#80.01's title mentions only "a plus". The Markdown case is explicitly called out in the Problem section and warrants its own scenario so the feature's contract is fully documented.

**Undocumented Markdown scenario**, severity unknown, counted as medium (werkrevisor) hint; Jev: real 0.76, severity 2.0 of 3 The test file adds three test methods, but the PR document specifies only two scenarios. The third test, `test_an_unchanged_markdown_item_gets_the_note`, checks that an unchanged Markdown list item starting with `-` triggers the note; it has no scenario in the document, no `Scenario#80.0x` docstring, and Scenario#80.01's title mentions only "a plus". The Markdown case is explicitly called out in the Problem section and warrants its own scenario so the feature's contract is fully documented.
Author
Collaborator

Fixed: Scenario#80.01 now covers a plus or a minus and lists test_an_unchanged_markdown_item_gets_the_note.

Fixed: Scenario#80.01 now covers a plus or a minus and lists test_an_unchanged_markdown_item_gets_the_note.
claude-code marked this conversation as resolved
mi added 1 commit 2026-10-11 08:53:43 +02:00
Scenario#80.01 covers the Markdown list item too
werkator build succeeded after 00:23
ai-review/assessment AI review passed; approved by werkrevisor; merge scheduled for when the checks succeed
ai-review/gate build succeeded after 01:22
werkator/werkrevisor-statistics build succeeded after 01:04
e72687afe3
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi dismissed ai-reviewer's review 2026-10-11 08:53:43 +02:00
Reason:

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

Collaborator

🟢 werkrevisor: GREEN / PASS

Change kind: mixed.

Reasons

  • hint approval of werkrevisor for f355f3e not carried over: 2 approved block(s) are missing
  • 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 6 section(s) of the PR document agree with the change summary of openrouter/minimax/minimax-m3 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

6 of 6 units passed all questions (1 PR document(s)).
1 answer ignored because of a confidence below [classifier] min_confidence.

Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 16548, calls 1, tokens in 16011 / out 8534 (reasoning included in out), duration 72.8 s, cost 0.0150 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4486, cost 0.0002 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 21115 / out 274, cost 0.0003 USD (reported), change summary `openrouter/minimax/minimax-m3` via Minimax calls 1, tokens in 1565 / out 1013, cost 0.0017 USD (reported), PR document check `jev-1.13.0` calls 6, tokens in 8116, cost 0.0003 USD, build file judge `jev-1.13.0` calls 5, tokens in 3149, cost 0.0001 USD, phases: path_judge 0.5 s, classifier 0.4 s, change_summary 6.7 s, doc_check 0.6 s, context 4.1 s, model_review 59.9 s, config: defaults + host + repository (.werkrevisor.toml at main 54fe179).
Reviewed head `e72687afe39323b191cb9f3db10291a2a30c7ef6` against base `54fe179f4d2460ae400ea3fd1dd41bb23a4916c5`, werkrevisor 0.32.6.
<!-- werkrevisor:e72687afe39323b191cb9f3db10291a2a30c7ef6 --> <!-- werkrevisor-verdict:GREEN --> <!-- werkrevisor-blocks:d9fd23b1ac25387034a9b035c3afdcd5,63f19d926c5fb5039b765d2b5809ae8c,4f1cfa8d373f6a9c89422aef8827af15,c570516be053c88eb07a1fd33c20dc62,ae37885e6a0f516aa8b6bfdcc9139451,902b78ba936bfbe07baf8eaab281213d,70e59913d42a4a4ff32e74f16b86b233,244495aaae870ef781d318f685f0b5bb --> ## 🟢 werkrevisor: GREEN / PASS Change kind: `mixed`. ### Reasons - **hint** approval of werkrevisor for `f355f3e` not carried over: 2 approved block(s) are missing - **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** 6 section(s) of the PR document agree with the change summary of openrouter/minimax/minimax-m3 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 6 of 6 units passed all questions (1 PR document(s)). 1 answer ignored because of a confidence below [classifier] min_confidence. <sub> Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 16548, calls 1, tokens in 16011 / out 8534 (reasoning included in out), duration 72.8 s, cost 0.0150 USD (reported), classifier `jev-1.13.0` calls 6, tokens in 4486, cost 0.0002 USD, excerpt selection `openrouter/openai/gpt-6-luna` calls 1, tokens in 21115 / out 274, cost 0.0003 USD (reported), change summary `openrouter/minimax/minimax-m3` via Minimax calls 1, tokens in 1565 / out 1013, cost 0.0017 USD (reported), PR document check `jev-1.13.0` calls 6, tokens in 8116, cost 0.0003 USD, build file judge `jev-1.13.0` calls 5, tokens in 3149, cost 0.0001 USD, phases: path_judge 0.5 s, classifier 0.4 s, change_summary 6.7 s, doc_check 0.6 s, context 4.1 s, model_review 59.9 s, config: defaults + host + repository (.werkrevisor.toml at main 54fe179).<br> Reviewed head `e72687afe39323b191cb9f3db10291a2a30c7ef6` against base `54fe179f4d2460ae400ea3fd1dd41bb23a4916c5`, werkrevisor 0.32.6. </sub>
ai-reviewer left a comment
Collaborator

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

werkrevisor: GREEN / PASS for `e72687afe39323b191cb9f3db10291a2a30c7ef6`. The result comment of werkrevisor holds the details. <!-- werkrevisor-review:e72687afe39323b191cb9f3db10291a2a30c7ef6 -->
ai-reviewer scheduled this pull request to auto merge when all checks succeed 2026-10-11 08:55:01 +02:00
ai-reviewer merged commit 934a0bdc78 into main 2026-10-11 08:55:10 +02:00
ai-reviewer deleted branch claude-code/diff-marker-note 2026-10-11 08:55:10 +02:00
Sign in to join this conversation.
No Reviewers
No labels
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mi/werkrevisor#80