Example reports with every state of a method (0.26.10) #65

2026-10-11T17:35:45+02:00

claude-code commented

2026-10-11 17:35:45 +02:00

Example reports with every state of a method in docs/examples, as section 8.4 asks; a test keeps them equal to what the writers write.

See docs/prs/2026-10-11-PR#65-example-reports.md.

🤖 Generated with Claude Code

Example reports with every state of a method in `docs/examples`, as section 8.4 asks; a test keeps them equal to what the writers write. See `docs/prs/2026-10-11-PR#65-example-reports.md`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claude-code added 3 commits 2026-10-11 17:35:46 +02:00

PR document for the example reports 3c08a0ac2c

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Example reports with every state of a method, kept current by ExampleReportsTest ff1f09fedc

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Final report, README and plan: section 8.4 complete (0.26.10)

werkator build succeeded after 02:53

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 02:00

Details

9737a8d83a

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

claude-code marked the pull request as ready for review 2026-10-11 17:38:58 +02:00

ai-reviewer commented

2026-10-11 17:40:57 +02:00

🟢 werkrevisor GREEN for 9737a8d, superseded by the result for d518977

🟢 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 writing rules violated in the changed docs: docs/examples/mutation-crap.md:3: line longer than 128 characters (195)
hint 1 changed file(s) that neither weaken the checks, touch security nor show a risk category of jvm-spring-backend according to Jev
hint 1 build change(s) of low risk according to Jev: build.gradle.kts (weakens checks 0.03, risk 0.00, security 0.01)
hint 1 finding(s) of low severity or rated minor by Jev: docs/examples/README.md:18-19

Findings

Documentation inconsistency at docs/examples/README.md:18-19, severity unknown, counted as medium

hint; Jev: real 0.81, severity 1.5 of 3

The README states that the class demo.Classifier "has two groups of methods, LCOM4 2, and a change risk", but the JSON example and the Markdown example both show changeRisk: null for that class (the Change Risk table in mutation-crap.md is empty, with reason method-not-scored:retry(I)I). A reader looking for the change risk value won't find one. Either rephrase to reflect that the change risk is null because of an inconclusive method, or drop the "and a change risk" claim.

Classifier

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

_{Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 17360, calls 1, tokens in 16759 / out 11094 (reasoning included in out), duration 115.3 s, cost 0.0183 USD (reported), classifier `jev-1.13.0` calls 4, tokens in 2682, cost 0.0001 USD, finding judge `jev-1.13.0` calls 1, tokens in 1406, cost 0.0001 USD, build file judge `jev-1.13.0` calls 2, tokens in 2727, cost 0.0001 USD, phases: path_judge 0.3 s, classifier 0.3 s, context 1.1 s, model_review 112.9 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main 1577d2f).

Reviewed head `9737a8d83af060570e7c45a05b0dde161ad9d298` against base `1577d2f442136b889e9b20d3da1b43b4929c1974`, werkrevisor 0.32.7.}

<details><summary>🟢 werkrevisor GREEN for <code>9737a8d</code>, superseded by <a href="http://git.javagil.de/mi/werkcrappit/pulls/65#issuecomment-4050">the result for <code>d518977</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** writing rules violated in the changed docs: docs/examples/mutation-crap.md:3: line longer than 128 characters (195) - **hint** 1 changed file(s) that neither weaken the checks, touch security nor show a risk category of jvm-spring-backend according to Jev - **hint** 1 build change(s) of low risk according to Jev: build.gradle.kts (weakens checks 0.03, risk 0.00, security 0.01) - **hint** 1 finding(s) of low severity or rated minor by Jev: docs/examples/README.md:18-19 ### Findings 1. **Documentation inconsistency** at `docs/examples/README.md:18-19`, severity unknown, counted as medium hint; Jev: real 0.81, severity 1.5 of 3 The README states that the class `demo.Classifier` "has two groups of methods, LCOM4 2, and a change risk", but the JSON example and the Markdown example both show `changeRisk: null` for that class (the `Change Risk` table in `mutation-crap.md` is empty, with reason `method-not-scored:retry(I)I`). A reader looking for the change risk value won't find one. Either rephrase to reflect that the change risk is null because of an inconclusive method, or drop the "and a change risk" claim. ### Classifier 4 of 4 units passed all questions (1 PR document(s)). Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 17360, calls 1, tokens in 16759 / out 11094 (reasoning included in out), duration 115.3 s, cost 0.0183 USD (reported), classifier `jev-1.13.0` calls 4, tokens in 2682, cost 0.0001 USD, finding judge `jev-1.13.0` calls 1, tokens in 1406, cost 0.0001 USD, build file judge `jev-1.13.0` calls 2, tokens in 2727, cost 0.0001 USD, phases: path_judge 0.3 s, classifier 0.3 s, context 1.1 s, model_review 112.9 s, finding_judge 0.2 s, config: defaults + host + repository (.werkrevisor.toml at main 1577d2f). Reviewed head `9737a8d83af060570e7c45a05b0dde161ad9d298` against base `1577d2f442136b889e9b20d3da1b43b4929c1974`, werkrevisor 0.32.7. </details>

ai-reviewer approved these changes

2026-10-11 17:40:57 +02:00

Dismissed

ai-reviewer left a comment

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

werkrevisor: GREEN / PASS for `9737a8d83af060570e7c45a05b0dde161ad9d298`. The result comment of werkrevisor holds the details.

docs/examples/mutation-crap.md

						
				@@ -0,0 +1,69 @@

				# Mutation-CRAP

				Experimental metric `mutation-crap` version 1, status policy `timeouts-killed-v1`: `C² × (1 − MS)³ + C` with C the JaCoCo complexity and MS the share of killed mutants, as the policy counts them.

ai-reviewer commented

2026-10-11 17:40:57 +02:00

Writing rule: line longer than 128 characters (195) (werkrevisor writing rule)

Fix the line, or reply why it stays, and resolve this comment.

**Writing rule: line longer than 128 characters (195)** (werkrevisor writing rule) Fix the line, or reply why it stays, and resolve this comment.

claude-code commented

2026-10-11 17:42:22 +02:00

No change: docs/examples/mutation-crap.md is the output of MarkdownReport, and ExampleReportsTest fails when the file differs from it; the long line is the report's first paragraph as every user gets it. Shortening it means changing the report itself, which is out of scope here.

No change: `docs/examples/mutation-crap.md` is the output of `MarkdownReport`, and `ExampleReportsTest` fails when the file differs from it; the long line is the report's first paragraph as every user gets it. Shortening it means changing the report itself, which is out of scope here.

claude-code marked this conversation as resolved

claude-code commented

2026-10-11 17:42:23 +02:00

Round 1:

"Documentation inconsistency", fixed in d518977: the README says that demo.Classifier has no change risk, and why.
Writing rule in docs/examples/mutation-crap.md: no change, answered in its thread; the file is the generated report, which the test pins.

Round 1: - "Documentation inconsistency", fixed in d518977: the README says that `demo.Classifier` has no change risk, and why. - Writing rule in `docs/examples/mutation-crap.md`: no change, answered in its thread; the file is the generated report, which the test pins.

mi added 1 commit 2026-10-11 17:42:23 +02:00

Examples README: the class has no change risk, and why

werkator build succeeded after 02:50

Details

ai-review/assessment AI review passed; approved by werkrevisor; merge scheduled for when the checks succeed

Details

ai-review/gate build succeeded after 00:02

Details

d518977879

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

mi dismissed ai-reviewer's review 2026-10-11 17:42:23 +02:00

Reason:

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

ai-reviewer commented

2026-10-11 17:43:40 +02:00

🟢 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 approval of werkrevisor for 9737a8d not carried over: 1 approved block(s) are missing
hint writing rules violated in the changed docs: docs/examples/mutation-crap.md:3: line longer than 128 characters (195)
hint 1 changed file(s) that neither weaken the checks, touch security nor show a risk category of jvm-spring-backend according to Jev
hint 1 build change(s) of low risk according to Jev: build.gradle.kts (weakens checks 0.03, risk 0.00, security 0.01)

Classifier

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

_{Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 17382, calls 1, tokens in 16778 / out 9794 (reasoning included in out), duration 74.4 s, cost 0.0168 USD (reported), classifier `jev-1.13.0` calls 4, tokens in 2682, cost 0.0001 USD, build file judge `jev-1.13.0` calls 2, tokens in 2727, cost 0.0001 USD, phases: path_judge 0.2 s, classifier 0.2 s, context 0.2 s, model_review 73.3 s, config: defaults + host + repository (.werkrevisor.toml at main 1577d2f).

Reviewed head `d51897787933f3594bc2f3d095f8619704b858a0` against base `1577d2f442136b889e9b20d3da1b43b4929c1974`, werkrevisor 0.32.7.}

## 🟢 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** approval of werkrevisor for `9737a8d` not carried over: 1 approved block(s) are missing - **hint** writing rules violated in the changed docs: docs/examples/mutation-crap.md:3: line longer than 128 characters (195) - **hint** 1 changed file(s) that neither weaken the checks, touch security nor show a risk category of jvm-spring-backend according to Jev - **hint** 1 build change(s) of low risk according to Jev: build.gradle.kts (weakens checks 0.03, risk 0.00, security 0.01) ### Classifier 4 of 4 units passed all questions (1 PR document(s)). Usage: model `openrouter/minimax/minimax-m3`, LLM done, tokens counted before the call 17382, calls 1, tokens in 16778 / out 9794 (reasoning included in out), duration 74.4 s, cost 0.0168 USD (reported), classifier `jev-1.13.0` calls 4, tokens in 2682, cost 0.0001 USD, build file judge `jev-1.13.0` calls 2, tokens in 2727, cost 0.0001 USD, phases: path_judge 0.2 s, classifier 0.2 s, context 0.2 s, model_review 73.3 s, config: defaults + host + repository (.werkrevisor.toml at main 1577d2f). Reviewed head `d51897787933f3594bc2f3d095f8619704b858a0` against base `1577d2f442136b889e9b20d3da1b43b4929c1974`, werkrevisor 0.32.7.

ai-reviewer approved these changes

2026-10-11 17:43:41 +02:00

ai-reviewer left a comment

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

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

docs/examples/mutation-crap.md

						
				@@ -0,0 +1,69 @@

				# Mutation-CRAP

				Experimental metric `mutation-crap` version 1, status policy `timeouts-killed-v1`: `C² × (1 − MS)³ + C` with C the JaCoCo complexity and MS the share of killed mutants, as the policy counts them.

ai-reviewer commented

2026-10-11 17:43:41 +02:00

Writing rule: line longer than 128 characters (195) (werkrevisor writing rule)

Fix the line, or reply why it stays, and resolve this comment.

**Writing rule: line longer than 128 characters (195)** (werkrevisor writing rule) Fix the line, or reply why it stays, and resolve this comment.

claude-code commented

2026-10-11 17:47:20 +02:00

No change, as in round 1: the file is the generated report, pinned by ExampleReportsTest; the long line is the report's first paragraph.

No change, as in round 1: the file is the generated report, pinned by `ExampleReportsTest`; the long line is the report's first paragraph.

claude-code marked this conversation as resolved

ai-reviewer scheduled this pull request to auto merge when all checks succeed 2026-10-11 18:02:46 +02:00

ai-reviewer commented

2026-10-11 18:02:46 +02:00

werkrevisor, review restarted for d51897787933f3594bc2f3d095f8619704b858a0: approved by werkrevisor; merge scheduled for when the checks succeed.

werkrevisor, review restarted for `d51897787933f3594bc2f3d095f8619704b858a0`: approved by werkrevisor; merge scheduled for when the checks succeed.

ai-reviewer merged commit d441b059af into main

2026-10-11 18:02:49 +02:00

ai-reviewer deleted branch claude-code/example-reports

2026-10-11 18:02:49 +02:00

Sign in to join this conversation.

3 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: mi/werkcrappit#65