Skip to content

Fix code review suggestions#2

Open
pyyush wants to merge 1 commit intomainfrom
fix/code-review-suggestions
Open

Fix code review suggestions#2
pyyush wants to merge 1 commit intomainfrom
fix/code-review-suggestions

Conversation

@pyyush
Copy link
Contributor

@pyyush pyyush commented Feb 22, 2026

Summary

  • Add missing CHANGELOG 0.2.0 entry documenting the jq-to-node migration, plugin.json cleanup, and marketplace submission
  • Remove references to nonexistent /skill-tools:validate and /skill-tools:score slash commands from skill-quality SKILL.md
  • Remove hardcoded "9 rules" count from hook success message so it stays accurate when rules are added/removed
  • Pin skill-tools version in skill-quality examples to @0.2.2 instead of @latest to match the version pinning strategy used everywhere else

Test plan

  • Verify CHANGELOG reads correctly and entries are in reverse chronological order
  • Verify skill-quality SKILL.md only references existing slash commands (/skill-tools:lint, /skill-tools:check)
  • Verify hook success message no longer hardcodes a rule count
  • Verify all npx examples in skill-quality use @0.2.2, not @latest
  • Run tests/test-hook.sh to confirm no regressions

…unpinned version

- Add missing CHANGELOG 0.2.0 entry documenting jq-to-node migration and marketplace submission
- Remove references to nonexistent /skill-tools:validate and /skill-tools:score slash commands
- Remove hardcoded "9 rules" count from hook success message to avoid staleness on rule additions
- Pin skill-tools version in skill-quality examples to @0.2.2 instead of @latest
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.

1 participant