claude-code
  • Joined on 2026-10-05
claude-code pushed to claude-code/diagram-cost at mi/werkrevisor 2026-10-23 18:25:39 +02:00
02703cbc85 PR document: number 140
claude-code created pull request mi/werkrevisor#140 2026-10-23 18:25:31 +02:00
WIP: werkrevisor 0.66.0: the cost of the PR Lens diagrams in the usage record, the result comment and the statistics
claude-code pushed to claude-code/diagram-cost at mi/werkrevisor 2026-10-23 18:25:30 +02:00
97b41904d9 PR document: the cost of the PR Lens diagrams
claude-code created branch claude-code/diagram-cost in mi/werkrevisor 2026-10-23 18:25:30 +02:00
claude-code pushed to claude-code/assessment-after-report at mi/werkrevisor 2026-10-23 17:13:40 +02:00
5c507ceefb --attach keeps an old SVG that the new text still links
claude-code commented on pull request mi/werkrevisor#139 2026-10-23 17:13:39 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

Fixed in 5c507ce: a replacement deletes only the old SVGs that the new text does not link; a kept one stays in hs-review-assets, so a later replacement can still delete it. `test_a_replacement_…

claude-code commented on pull request mi/werkrevisor#139 2026-10-23 17:12:05 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

Fixed in bc40aad: the item is split in two, the post or replacement and the fold.

claude-code pushed to claude-code/assessment-after-report at mi/werkrevisor 2026-10-23 17:12:05 +02:00
bc40aada4b Docs: --attach folds the own reports of all other heads; a hand edit of the report text is replaced
claude-code commented on pull request mi/werkrevisor#139 2026-10-23 17:12:04 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

By design, documented in bc40aad: the text of the report is what attach wrote, so a replacement replaces it as a whole; merging hand edits into a new report has no safe rule. A note belongs…

claude-code commented on pull request mi/werkrevisor#139 2026-10-23 17:12:04 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

Fixed in bc40aad: README, both help texts, the Werkbaum and the PR document now say "the own reports of all other heads", also a newer one when the head went back to an earlier commit; the…

claude-code pushed to claude-code/assessment-after-report at mi/werkrevisor 2026-10-23 16:59:45 +02:00
fe7a622e49 --attach tracks the IDs of its SVGs and folds the current body of an earlier report
claude-code commented on pull request mi/werkrevisor#139 2026-10-23 16:59:45 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

Fixed in fe7a622: Scenario#139.03 is now "a failed replacement leaves no broken links": the comment holds the old or the new report, and each SVG that it links is attached; the When names both…

claude-code commented on pull request mi/werkrevisor#139 2026-10-23 16:59:44 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

Fixed in fe7a622: the comment names the IDs of the SVGs that attach uploaded in <!-- hs-review-assets:<ids> -->; a replacement deletes only those. `test_a_replacement_keeps_an_svg_linked_by_h…

claude-code commented on pull request mi/werkrevisor#139 2026-10-23 16:59:44 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

Fixed in fe7a622: attach reads each earlier report again right before it folds it. test_a_fold_keeps_an_edit_made_meanwhile failed before the fix.

claude-code commented on pull request mi/werkrevisor#139 2026-10-23 16:46:06 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

No change: a rejected upload is already the safe fallback. _upload deletes the SVGs it uploaded, the comment keeps its old body and old SVGs, and --attach exits with the error (`test_a_failed_r…

claude-code pushed to claude-code/assessment-after-report at mi/werkrevisor 2026-10-23 16:46:06 +02:00
79591ad69b PR document: one aspect per scenario; earlier reports are only folded
claude-code commented on pull request mi/werkrevisor#139 2026-10-23 16:46:06 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

Does not hold: the comprehension in line 82 reads header after the draft line is filtered out in line 81, so header[i - 1] is the previous retained line and the blank line after the removed…

claude-code commented on pull request mi/werkrevisor#139 2026-10-23 16:27:40 +02:00
werkrevisor 0.66.0: draft-review --attach puts the assessment after the report and replaces the report of the same commit

Fixed in 77bb9d9: a replacement deletes only the SVGs of PR Lens that the old report linked and the new one does not; a file that someone added to the comment stays. `test_a_replacement_keeps_a_f…

claude-code pushed to claude-code/assessment-after-report at mi/werkrevisor 2026-10-23 16:27:40 +02:00
77bb9d9361 --attach deletes only the SVGs of the replaced report
claude-code commented on pull request mi/werkrevisor#137 2026-10-23 16:27:25 +02:00
werkrevisor 0.74.1: hsadmin-ng findings that cannot occur are hints, not ORANGE

Round 6 (a048f264): no model findings and no open line comments. The only YELLOW reasons are the implementation_details scores of Scenario#137.01 and #137.02 (0.68, 0.61), left to the human as…