From 94801283cc0527dcef918c690d4d1718cf6fb87c Mon Sep 17 00:00:00 2001 From: Koen van der Veen Date: Wed, 2 Sep 2026 15:03:08 +0200 Subject: [PATCH] chore: keep .claude out of the reviewed file set in prefetch.sh A skill edit landed on a working branch before it lands on the base rides along in that branch's PR diff, so the review would describe the reviewer's own tooling change as the PR author's work. Classify .claude/ as skippable, alongside lock files and generated data. --- .claude/skills/syft-pr-review/prefetch.sh | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.claude/skills/syft-pr-review/prefetch.sh b/.claude/skills/syft-pr-review/prefetch.sh index 5b2af4ce14f..eecbb765285 100755 --- a/.claude/skills/syft-pr-review/prefetch.sh +++ b/.claude/skills/syft-pr-review/prefetch.sh @@ -37,7 +37,10 @@ gh pr view "$PR" --json files \ | sort -rn >"$OUT/files.txt" # Lock files, notebooks and generated data are counted and skipped, not read. -grep -E '(\.lock|lock\.json|\.ipynb|\.min\.(js|css)|\.svg|\.png|\.csv|\.parquet)$' \ +# `.claude/` too: agent tooling is not the PR author's work, and it rides along +# in the diff whenever a skill edit is landed on the branch before it lands on +# the base. Reviewing it would attribute it to the wrong person. +grep -E '(\.lock|lock\.json|\.ipynb|\.min\.(js|css)|\.svg|\.png|\.csv|\.parquet)$|(^| )\.claude/' \ "$OUT/files.txt" >"$OUT/files-skippable.txt" || true # ----------------------------------------------------------------- history ---