Skip to content

chore: add syft-pr-review skill - #9498

Merged
koenvanderveen merged 8 commits into
devfrom
koen/pr-review-skill
Sep 2, 2026
Merged

chore: add syft-pr-review skill#9498
koenvanderveen merged 8 commits into
devfrom
koen/pr-review-skill

Conversation

@koenvanderveen

Copy link
Copy Markdown
Collaborator

Summary

Adds a Claude Code skill that writes a review document for a syft PR, so the format is shared
rather than re-invented per reviewer.

The output is one markdown document, structured as tickable checkboxes so a reviewer can work
through it and leave a record of what they read:

  1. Flows — first, because it is why you opened the review. What calls what, and for a changed
    flow, what it did before.
  2. What is new — additions only.
  3. Changes — everything changed or deleted, grouped by theme, each as
    "we previously did Z, now we do A".
  4. Open questions — only decisions a reviewer has to make.

Notes

  • Verbosity is meant to follow the size of the code: a ten-line class gets a few words, and small
    related additions are grouped under one bullet.
  • Sections 2 and 3 do not describe the same thing twice — section 3 refers to new code by name
    instead of re-explaining it.
  • .gitignore un-ignores .claude/skills/syft-pr-review/ only. Other local skills under
    .claude/ stay ignored.

Testing

Run on #9487 and #9497; both reviews were posted as PR comments.

Adds a Claude Code skill that turns a syft PR into a short review document:
the flows that changed first, then what was added, then every individual
change, all as tickable checkboxes so a reviewer can work through it.

Also un-ignores .claude/skills/syft-pr-review/ so the skill is shared,
while other local skills under .claude/ stay ignored.
@github-actions github-actions Bot added the chore label Sep 2, 2026
@koenvanderveen
koenvanderveen merged commit 5212484 into dev Sep 2, 2026
21 checks passed
@koenvanderveen
koenvanderveen deleted the koen/pr-review-skill branch September 2, 2026 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant