Skip to content

Add vulnerability triage brocards skill - #201

Merged
dguido merged 2 commits into
mainfrom
add-vulnerability-triage-brocards
Jul 29, 2026
Merged

Add vulnerability triage brocards skill#201
dguido merged 2 commits into
mainfrom
add-vulnerability-triage-brocards

Conversation

@DarkaMaul

Copy link
Copy Markdown
Contributor

No description provided.

@DarkaMaul
DarkaMaul requested a review from dguido as a code owner July 20, 2026 15:15

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@dguido dguido left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving on the contribution checklist and CI, reviewed with Claude Code.

Verified:

  • Frontmatter parses; name is kebab-case and matches the directory
  • Description is third person and carries concrete trigger phrases
  • All three required sections present: ## When to Use, ## When NOT to Use, ## Rationalizations to Reject
  • Version parity 0.1.0 between plugin.json and marketplace.json
  • Registered in marketplace.json, root README table, and CODEOWNERS
  • SKILL.md is 208 lines; one level of references, no chains
  • No hardcoded absolute paths; skills/ at plugin root
  • All 7 checks green after updating the branch

Not verified: the substance of the 7 brocards as triage guidance. That rests on @DarkaMaul and the linked source.

@dguido
dguido merged commit 101a102 into main Jul 29, 2026
7 checks passed
@dguido
dguido deleted the add-vulnerability-triage-brocards branch July 29, 2026 20:49
This was referenced Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants