-
-
Notifications
You must be signed in to change notification settings - Fork 0
feat: Auto-generate SKILL.md and extract version bump script #105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Add script/generate-skill.ts to generate SKILL.md from Stricli metadata - Add script/check-skill.ts to verify SKILL.md is up-to-date in CI - Add script/bump-version.ts to simplify .craft.yml release commands - Add check-skill CI job that posts PR comments when SKILL.md is stale - Add symlink at .cursor/skills/ for Cursor users The SKILL.md file is intentionally not regenerated to test the CI check.
Semver Impact of This PR🟡 Minor (new features) 📋 Changelog PreviewThis is how your changes will appear in the changelog. New Features ✨
Bug Fixes 🐛
🤖 This preview updates automatically when you update the PR. |
Codecov Results 📊✅ Patch coverage is 100.00%. Project has 1756 uncovered lines. Files with missing lines (24)
Coverage diff@@ Coverage Diff @@
## main #PR +/-##
==========================================
+ Coverage 66.66% 66.66% —%
==========================================
Files 45 45 —
Lines 5267 5267 —
Branches 0 0 —
==========================================
+ Hits 3511 3511 —
- Misses 1756 1756 —
- Partials 0 0 —Generated by Codecov Action |
Changed from issue comment to PR review comment with suggestion block, which enables the 1-click 'Apply suggestion' button in GitHub UI.
- Fix lint errors in bump-version.ts, check-skill.ts, generate-skill.ts - Add generate-skill.yml workflow with manual trigger - Simplify check-skill job to output instructions in job summary - Link to workflow dispatch with pre-filled branch name - Remove complex PR comment generation (not needed anymore)
External contributors won't have permissions to trigger workflows, so we only show the local command instructions for fork PRs.
- Add explicit 'contents: write' permission to satisfy security scanning - Revert SKILL.md to main branch version to test the staleness check flow
|
|
||
| ### Issues | ||
| **Flags:** | ||
| - `--token <value> - Authenticate using an API token instead of OAuth` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should find a way to generate these examples back. I'm okay if those examples live in the original help text as that's also very useful.
- Remove git config from generate-skill.yml (use git defaults) - Source prerequisites from docs/getting-started.mdx - Source command examples from docs/commands/*.md files - Remove hardcoded sections (context, monorepo, workflows, etc.) - Add comment explaining why we use simplified types for Stricli introspection - Fix lint errors (top-level regex, variable shadowing)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
- Re-add git config user.name/email (required for commits) - Fix loadPrerequisites to use extracted code blocks from docs - Parse PackageManagerCode Astro component to get npm install command - Extract generateInstallSection/generateAuthSection helpers
Summary
Addresses two issues from #69:
.craft.ymlversion bump scripts were unwieldy inline Node.js codeChanges
Version Bump Script (
script/bump-version.ts).craft.ymlinto a standalone Bun script--pre) and post-release (--post) scenariospackage.jsonandplugin.jsonversions in syncSkill Generation (
script/generate-skill.ts)SKILL.mdby introspecting the Stricli route treeCI Integration
Check Job (
check-skillinci.yml):SKILL.mdis staleGenerate Workflow (
.github/workflows/generate-skill.yml):workflow_dispatchtrigger with branch inputCursor Support
.cursor/skills/sentry-cli/SKILL.mdpointing to the plugins locationNew Scripts