feat: bulk resolve 13 open issues#23
Merged
Merged
Conversation
… new tool targets, onboarding, authoring docs Closes #3: Version-pinned disk cache with LRU eviction (5MB cap, core skills exempt) Closes #8: CONTRIBUTING.md, SKILL-TEMPLATE.md, and PR template for skill submissions Closes #10: Context-aware skill budgeting (15% of detected context window, 2k-50k range) Closes #14: Windsurf, Aider, and Continue added to tool detector and setup client registry Closes #15: `superskill-cli onboard` command for automated setup of all detected AI tools Also closed as already resolved: #2, #4, #5, #16, #17 Also closed as superseded/deferred: #7, #11, #12 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…reliability - fitSkillsToBudget: stop after first exclusion to respect priority order - onboard: wrap scanInstalledSkills in try/catch for graceful error accumulation - skill-cache: add error logging in cacheSkill, clean up temp files on failure - skill-cache: use mtime instead of atime for LRU (reliable on Linux/containers) - context-budget.test: add missing SPDX header - CONTRIBUTING.md: fix stale SPDX reference, remove nonexistent LICENSE-COMMERCIAL.md - SKILL-TEMPLATE.md: add triggers field to frontmatter Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
1 participant
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
Triaged all 13 open issues against the current skills.sh first-class integration layer. Closed 8 as resolved/superseded/deferred, implemented fixes for the remaining 5.
Issues closed as already resolved
registry/index.jsonwith 87 skills ships since v0.3.0trigger-matcher.tsfully implemented in v0.3.0skill-session.tswith remember/recall in v0.3.0Issues closed as superseded/deferred
skill add owner/repoIssues fixed in this PR
name@1.0.0.md), LRU eviction at 5MB, core skills exemptdocs/SKILL-TEMPLATE.md,.github/PULL_REQUEST_TEMPLATE/skill_submission.mdcontext-budget.ts— 15% of detected context window (2k-50k range), integrated into activationsuperskill-cli onboard— auto-detects tools, configures MCP, scans skillsStats
Test plan
npm run build— clean compilationnpm test— 906/906 tests passsuperskill-cli onboardon clean machineCloses #3 #8 #10 #14 #15