claude-code

claude-code pushed to claude-code/previous-findings at mi/werkrevisor

2026-10-23 22:50:52 +02:00

f71dbfaae1 Review round 7: Scenario#142.03 names the behaviour

claude-code commented on pull request mi/werkrevisor#142

2026-10-23 22:50:51 +02:00

werkrevisor 0.69.0: the model judges the findings of the last posted review

Review round 7:

YELLOW Scenario#142.03 implementation_details: fixed in f71dbfa; the scenario names what the PR author writes and what the model gets, not the block lines and test details. -…

claude-code pushed to claude-code/previous-findings at mi/werkrevisor

2026-10-23 22:31:56 +02:00

0c15d5413f Review round 6: no note with a possible secret, no folded report, Codex findings count

claude-code commented on pull request mi/werkrevisor#142

2026-10-23 22:31:56 +02:00

werkrevisor 0.69.0: the model judges the findings of the last posted review

Review round 6:

werkrevisor/earlierreview.py:113 Secret Exposure (and the security concern): fixed in 0c15d54; a match of SECRET_PATTERNS in the earlier findings, Codex findings or…

claude-code pushed to claude-code/model-facts at mi/werkrevisor

2026-10-23 22:30:03 +02:00

6f047e17a4 Review round 5: the stack only informs the model; the verdict stays as on main

claude-code commented on pull request mi/werkrevisor#141

2026-10-23 22:30:02 +02:00

werkrevisor 0.72.0: the model learns the base branch and which changesets are new

Review round 5 (Codex), fixed in 6f047e1 by a change of design, chosen by the human:

werkrevisor/cli.py:427 outgoing PRs: the stack no longer changes the verdict. A changed or removed…

claude-code pushed to claude-code/previous-findings at mi/werkrevisor

2026-10-23 21:45:19 +02:00

37633af64c Review round 5: a newer review without findings replaces the older one

claude-code commented on pull request mi/werkrevisor#142

2026-10-23 21:45:17 +02:00

werkrevisor 0.69.0: the model judges the findings of the last posted review

Review round 5:

werkrevisor/earlierreview.py:65 newest review without findings (Codex): fixed in 37633af; the newest review counts, and without findings it gives no note. Red/green:…

claude-code pushed to claude-code/previous-findings at mi/werkrevisor

2026-10-23 21:37:40 +02:00

2f7f240b28 Review round 4: the first findings heading counts; tests for the newest review and quoted headings

claude-code commented on pull request mi/werkrevisor#142

2026-10-23 21:37:38 +02:00

werkrevisor 0.69.0: the model judges the findings of the last posted review

Review round 4:

werkrevisor/earlierreview.py:62 Misparsed Findings (and the security concern): does not hold for quoted text. report.render_comment and the hs-review report indent every…

claude-code pushed to claude-code/previous-findings at mi/werkrevisor

2026-10-23 21:19:56 +02:00

1bda450071 Review round 3: the earlier review goes into the user prompt, not the system prompt

e9c3490165 Review round 2: no commit subjects in the note, only their number

31accd0388 Review round 1: commit subjects stay inside the data block of the earlier review

14479e6a21 Shorter test names, so that the PR document keeps its line length

81c19521c2 README, Werkbaum, version 0.69.0 and the solution of the earlier review

Compare 9 commits »

claude-code commented on pull request mi/werkrevisor#141

2026-10-23 21:19:05 +02:00

werkrevisor 0.72.0: the model learns the base branch and which changesets are new

Review round 4 (Codex), fixed in ed0a05c, rebased onto main with PR#139:

werkrevisor/cli.py:422 forward-port PRs: a protected base branch, e.g. a release branch with an open forward port…

claude-code pushed to claude-code/model-facts at mi/werkrevisor

2026-10-23 21:19:05 +02:00

ed0a05c7d8 Review round 4: a protected branch with an open PR gives no stack

d69e44d1ef Review round 3: a branch with two targets gives no stack; duplicate ids are no released change

3fdb1770a3 Review round 2: only the head branch of an open PR makes a stack

95f40e2af2 Review round 1: a repeatable changeset missing on the default branch is not released either

c61d5d0213 README, Werkbaum, version 0.68.0 and the solution of the model facts

Compare 9 commits »

claude-code pushed to claude-code/previous-findings at mi/werkrevisor

2026-10-23 21:11:19 +02:00

cbfff346ba Review round 3: the earlier review goes into the user prompt, not the system prompt

claude-code commented on pull request mi/werkrevisor#142

2026-10-23 21:11:18 +02:00

werkrevisor 0.69.0: the model judges the findings of the last posted review

Review round 3:

werkrevisor/earlierreview.py:42 Prompt injection (and the security concern): fixed in cbfff34. The system prompt gets only an instruction without any text of the earlier…

claude-code pushed to claude-code/model-facts at mi/werkrevisor

2026-10-23 21:00:05 +02:00

0720ccc1d7 Review round 3: a branch with two targets gives no stack; duplicate ids are no released change

claude-code commented on pull request mi/werkrevisor#141

2026-10-23 21:00:04 +02:00

werkrevisor 0.72.0: the model learns the base branch and which changesets are new

Review round 3:

werkrevisor/dbschema.py:165 changelog renamed by the parent branch (Codex, on e385001): fixed in e0c72a6 already. A 404 on the target counts as absence only if a PR of the…

claude-code pushed to claude-code/previous-findings at mi/werkrevisor

2026-10-23 20:49:54 +02:00

1e27580809 Review round 2: no commit subjects in the note, only their number

claude-code commented on pull request mi/werkrevisor#142

2026-10-23 20:49:54 +02:00

werkrevisor 0.69.0: the model judges the findings of the last posted review

Review round 2 (werkrevisor and Codex), all fixed in 1e27580:

werkrevisor/earlierreview.py:101 Prompt Injection (and the security concern): the note holds no commit subjects any more, only…

claude-code commented on pull request mi/werkrevisor#141

2026-10-23 20:44:28 +02:00

werkrevisor 0.72.0: the model learns the base branch and which changesets are new

Review round 2 (werkrevisor and Codex), all fixed in e0c72a6:

werkrevisor/cli.py:177 Non-default base treated as stacked (werkrevisor) and "Do not treat every non-default target as an…