Improve maintainer skill-authoring DX#190
Merged
Merged
Conversation
|
View your CI Pipeline Execution ↗ for commit c5af4f0
☁️ Nx Cloud last updated this comment at |
commit: |
Contributor
|
Caution Review failedAn error occurred during the review process. Please try again later. ✨ 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 |
Contributor
✨ 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 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Offline-first fixes to the maintainer skill authoring and update loop — the friction around actually updating a shipped skill.
validate --set-version <version>(5ebd44a) — updating the library version a skill targets meant hand-editingmetadata.library_versionacross everySKILL.md, which is tedious and error-prone in a package with many skills. The new verb stamps the version across all matched skills and then re-validates. It reusesvalidate's existing file discovery and YAML-frontmatter surgery, so comments, CRLF line endings, and body bytes are preserved (no naive find/replace). Adds the key when absent, overwrites when present, and is idempotent. Guards reject--set-version --checkand an empty/whitespace value; files whosemetadatais a non-mapping scalar are skipped (validation already errors on those).generate-skillmeta-skill with the validator (0e4668f) — the generator instructed agents that the full slash path (e.g.tanstack-query/core) was the frontmattername, butvalidaterejects anynamecontaining/and requires it to equal the parent directory. Corrected the input definition, the three frontmatter template blocks, and the validation checklist so generated skills validate on the first pass. Docs-only.skill-staleness-checkmeta-skill at the real command (0696d3f) — it referenced ascripts/sync-skills.mjsthat does not exist in the repo. Rewritten to use the realintent stalecommand and to describe the actualsync-state.jsonshape. Docs-only.Impact / compatibility
--set-versionis maintainer/author-side release plumbing (local or CIpostversion), never a runtime or consumer concern. It changes only the version stamp, not skill content.