Tighten content skills: remove no-ops and redundancy#7981
Conversation
Surgical pass over the merged content skills to remove ceremony and duplicated instructions without losing safety rails or inline warnings: - content-write-blog: drop the "keep the boundary clear" no-op; tighten the structured-question guidance. - content-create-hero-image: fold the "Read first" navigation note into one line (the per-reference "read before…" directives already live in the References list); trim the wordy Pre-conditions; drop five Anti-patterns bullets that only restate the Output contract and Validation checklist (format, filenames, temp dir), keeping the creative/brand traps a checklist can't catch. - docs-writer: drop the "This skill is opinionated" filler, keep the house-style precedence rule. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughThe PR rewrites guidance in three skill docs: the hero-image skill now points operators to ChangesSkill instruction wording updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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.
Inline comments:
In @.claude/skills/content-create-hero-image/SKILL.md:
- Around line 75-77: The precondition in `SKILL.md` is too restrictive because
it only allows a blog post slug or title, which conflicts with the
README-supported PR URL entry point. Update the precondition to include PR URL
as an accepted input for the `content-create-hero-image` flow, or revise the
precondition text to match the documented contract in `README.md` so the entry
points stay consistent.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 1b441a0c-8509-4db9-a85d-a40011a65d56
📒 Files selected for processing (3)
.claude/skills/content-create-hero-image/SKILL.md.claude/skills/content-write-blog/SKILL.md.claude/skills/docs-writer/SKILL.md
Summary
Follow-up to #7975 (merged). A surgical editing pass over the three content skills to remove no-ops and within-file redundancy and make the instructions tighter for both an agent and a human reader. No behavioral rules were dropped — only ceremony and restatement.
Changes
content-write-blog/SKILL.mdcontent-create-hero-image/SKILL.mddocs-writer/SKILL.mdNotes
🤖 Generated with Claude Code
Summary by CodeRabbit