Skip to content

fix(skills): diff installed skills against bundled template by default - #514

Merged
solomonneas merged 2 commits into
mainfrom
issue/484-skills-diff-bundled-template
Jul 24, 2026
Merged

fix(skills): diff installed skills against bundled template by default#514
solomonneas merged 2 commits into
mainfrom
issue/484-skills-diff-bundled-template

Conversation

@solomonneas

Copy link
Copy Markdown
Collaborator

Summary

  • Default brigade skills diff to the bundled Brigade package template when one exists, so stale .brigade/skills/registry/ copies cannot mask drift from newer bundled skills.
  • Add --against registry to keep the previous registry baseline when that comparison is intentional.
  • JSON output now includes against and baseline_skill for easier debugging.

Closes #484.

Test plan

  • pytest tests/test_skills_cmd.py tests/test_skill_templates_metadata.py
  • ./scripts/verify (4191 passed, 82.60% coverage)
  • Stale registry install + default diff reports changed: true
  • Fresh bundled install + default diff reports changed: false
  • --against registry reports clean when installed matches registry

Made with Cursor

solomonneas and others added 2 commits July 24, 2026 00:07
Update overview and technical guide links, finish phase-4a checklist item
for product/doc migration, and align stations/notify README with setup flow.

Co-authored-by: Cursor <cursoragent@cursor.com>
Default `brigade skills diff` to the Brigade package template so stale
registry copies cannot mask drift from newer bundled skills. Add
`--against registry` for the previous registry baseline. Fixes #484.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • coderabbit-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: escoffier-labs/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a741a370-e36d-4dad-92bb-c82c765c69ce

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch issue/484-skills-diff-bundled-template

Comment @coderabbitai help to get the list of available commands.

@solomonneas solomonneas self-assigned this Jul 24, 2026
@solomonneas
solomonneas merged commit e8a1766 into main Jul 24, 2026
28 checks passed
@solomonneas
solomonneas deleted the issue/484-skills-diff-bundled-template branch July 26, 2026 20:38
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.

skills diff: compare installed skills against the bundled template, not the target registry

1 participant