]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
ci: Update claude review prompt to insist on valid lines
authorDaan De Meyer <daan@amutable.com>
Tue, 10 Mar 2026 18:56:09 +0000 (19:56 +0100)
committerDaan De Meyer <daan@amutable.com>
Tue, 10 Mar 2026 18:56:09 +0000 (19:56 +0100)
Hopefully fixes the failure in
https://github.com/systemd/systemd/actions/runs/22904601370/job/66461528144.

.github/workflows/claude-review.yml

index 4b2a0dc5d638f2132b82968f50ff91fdab276976..0f1a52a30542bb9206563dfb228fbf5c6986b5f4 100644 (file)
@@ -187,8 +187,10 @@ jobs:
               Each subagent must return a JSON array of issues:
               `[{"file": "path", "line": <number>, "severity": "must-fix|suggestion|nit", "body": "..."}]`
 
-              `line` must be a line number from the NEW side of the diff (i.e. where the comment
-              should appear in the changed file after the patch is applied).
+              `line` must be a line number from the NEW side of the diff **that appears inside
+              a diff hunk** (i.e. a line that is shown in the patch output). GitHub's review
+              comment API rejects lines outside the diff context, so never reference lines
+              that are not visible in the patch.
 
               Each subagent MUST verify its findings before returning them:
               - For style/convention claims, check at least 3 existing examples in the codebase to confirm