claude-code
  • Joined on 2026-10-05
claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 09:19:21 +02:00
20000b2da4 draft-review: two plain checks for a missing write token instead of a fallback that is never reached
claude-code commented on pull request mi/werkrevisor#132 2026-10-23 09:19:21 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Fixed in 20000b2: two plain checks, one for a missing write_token_variable and one for an unset token.

claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 09:14:58 +02:00
6d772b5baf Lines of at most 128 characters in the PR document and draftreview.py
claude-code commented on pull request mi/werkrevisor#132 2026-10-23 09:14:58 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Fixed in 6d772b5: the line is wrapped after the semicolon.

claude-code commented on pull request mi/werkrevisor#132 2026-10-23 08:42:49 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Fixed in 6c53bb1: pandoc reads markdown_strict-raw_html+pipe_tables, which has no YAML metadata block, no raw attribute and no raw HTML (gfm and commonmark kept inline HTML despite -raw_html)…

claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 08:42:48 +02:00
6c53bb1714 report.html: strict Markdown without raw HTML, an own template with the policy first, script links defused
claude-code commented on pull request mi/werkrevisor#132 2026-10-23 08:42:48 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Fixed in 6c53bb1: pandoc reads markdown_strict-raw_html+pipe_tables, which has no YAML metadata block, no raw attribute and no raw HTML (gfm and commonmark kept inline HTML despite -raw_html)…

claude-code commented on pull request mi/werkrevisor#132 2026-10-23 08:25:29 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Fixed in 95d40fe: the verdict of a finding may end the line, as without a judgment of Jev. test_a_finding_without_a_judgment_of_jev_is_named_too.

claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 08:25:29 +02:00
95d40fe3ea draft-review names the strongest finding also without a judgment of Jev
claude-code commented on pull request mi/werkrevisor#132 2026-10-23 08:25:29 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Fixed in cf1f13c, as in the same finding 7101: --from markdown-raw_html keeps HTML of the draft as text, and the page has a Content Security Policy without scripts. Scenario#132.04, red/green…

claude-code commented on pull request mi/werkrevisor#132 2026-10-23 08:12:20 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Fixed in 6e75f60, as in the same finding 7094: tools/draft-review takes os.path.realpath(__file__); LinkTest calls it through a link.

claude-code commented on pull request mi/werkrevisor#132 2026-10-23 08:12:20 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Fixed in cf1f13c: pandoc reads markdown-raw_html, so HTML of the draft stays text, and the page carries `Content-Security-Policy: default-src 'none'; img-src 'self' file: data:; style-src…

claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 08:12:20 +02:00
cf1f13c420 report.html runs no script of the draft: no raw HTML, and a Content Security Policy without scripts
claude-code commented on pull request mi/werkrevisor#132 2026-10-23 07:58:35 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Answer to the review of 4b9622b:

  • YELLOW "changes that touch security" for werkrevisor/draftreview.py (Jev 0.95): holds as a reason for a human look, nothing to fix. The file reads the…
claude-code commented on pull request mi/werkrevisor#132 2026-10-23 07:58:34 +02:00
werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report

Fixed in 6e75f60: tools/draft-review takes os.path.realpath(__file__). Red/green by hand: through a link the old version stopped with ModuleNotFoundError, the new one reaches its own…

claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 07:58:34 +02:00
6e75f60b6d draft-review finds its package also through a link such as ~/bin/hs-review
claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 07:33:38 +02:00
4b9622b901 Tests use the renamed keys of the local config
claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 07:32:55 +02:00
734409b59f draft-review: the local config names read_token_variable and write_token_variable
claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 07:30:30 +02:00
61b5b803ca werkrevisor 0.61.0: tools/draft-review drafts a review for a human, with a local config and an HTML report
claude-code pushed to claude-code/draft-review at mi/werkrevisor 2026-10-23 07:26:45 +02:00
0c93c4ecf7 PR document gets the number 132