feat(skills): add before-and-after - #191
Conversation
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe PR adds the ChangesBefore-and-after skill
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
SKILLS.txt (1)
190-191: 🔒 Security & Privacy | 🔵 Trivial | ⚡ Quick winList the skill explicitly instead of installing all repository skills.
Line 1 defines an omitted skill list as install-all. This PR adds only
before-and-after, andskills-lock.jsonlocks only that skill. Use an explicit list so future upstream skills are not selected implicitly.Suggested fix
-# vercel-labs/before-and-after (1 total) - keep all -vercel-labs/before-and-after +# vercel-labs/before-and-after (1 total) +vercel-labs/before-and-after before-and-after🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@SKILLS.txt` around lines 190 - 191, Update the skill configuration around the vercel-labs/before-and-after entry to explicitly list that skill instead of relying on an omitted list being interpreted as install-all. Preserve the existing before-and-after selection and keep it consistent with the single locked skill in skills-lock.json.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@SKILLS.txt`:
- Around line 190-191: Update the skill configuration around the
vercel-labs/before-and-after entry to explicitly list that skill instead of
relying on an omitted list being interpreted as install-all. Preserve the
existing before-and-after selection and keep it consistent with the single
locked skill in skills-lock.json.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 4fcec505-cc0e-47b2-a0fd-a45958577115
📒 Files selected for processing (2)
SKILLS.txtskills-lock.json
Adds Vercels before-and-after screenshot skill to the managed catalog and lockfile.
Validated with make skills-install.
Summary by cubic
Added the
vercel-labs/before-and-afterscreenshot skill to the managed catalog and updatedskills-lock.jsonto lock its source for consistent installs.Written for commit 9589ed3. Summary will update on new commits.