Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
skills-package-manager | c47a080 | Apr 09 2026, 07:05 PM |
There was a problem hiding this comment.
Pull request overview
This PR introduces “self skill” bootstrap support to skills-package-manager by adding a selfSkill manifest flag, implementing manifest expansion via repo-local discovery, and integrating that effective manifest into init, install, add, and update. It also updates docs and tests to cover the new end-to-end behavior and adds a maintainer self-skill document.
Changes:
- Add
selfSkilltoskills.jsonschema/normalization and expand the effective manifest by auto-discovering exactly one repo-authored skill. - Integrate expanded manifests into install/update/add flows and add targeted test coverage.
- Update documentation and examples (CLI, manifest/lockfile, specifiers) to describe
selfSkillbehavior.
Reviewed changes
Copilot reviewed 27 out of 27 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| website/docs/getting-started.mdx | Document selfSkill in the default manifest + behavior description. |
| website/docs/architecture/manifest-and-lockfile.mdx | Add selfSkill to manifest example + field docs. |
| website/docs/architecture/cli-commands.mdx | Document effective-manifest expansion during install/update. |
| website/docs/api/commands.mdx | Update command docs to mention selfSkill defaults + install behavior. |
| website/docs/_pnpm.mdx | Update pnpm docs/examples to include selfSkill. |
| skills.json | Enable selfSkill in the repo’s root manifest. |
| skills-lock.yaml | Add a lock entry for an auto-discovered self skill. |
| README.md | Update local skill specifier docs and add selfSkill explanation. |
| packages/skills-package-manager/test/manifest.test.ts | Add tests for selfSkill defaulting + manifest expansion behavior. |
| packages/skills-package-manager/test/install.test.ts | Add install tests covering self-skill injection on/off. |
| packages/skills-package-manager/test/init.test.ts | Update init tests for selfSkill default field. |
| packages/skills-package-manager/test/github.test.ts | Add tests for self-skill discovery behavior. |
| packages/skills-package-manager/test/add.test.ts | Add test ensuring self skill is locked but not written into skills.json. |
| packages/skills-package-manager/src/install/installSkills.ts | Use expanded/effective manifest for sync/fetch/link and frozen checks. |
| packages/skills-package-manager/src/index.ts | Re-export new manifest helpers and self-skill discovery. |
| packages/skills-package-manager/src/github/listSkills.ts | Add shared discovery helper + self-skill discovery entrypoint. |
| packages/skills-package-manager/src/config/writeSkillsManifest.ts | Normalize manifest on write; include selfSkill. |
| packages/skills-package-manager/src/config/types.ts | Add selfSkill?: boolean to manifest type. |
| packages/skills-package-manager/src/config/skillsManifest.ts | New: manifest normalization + expansion via self-skill discovery. |
| packages/skills-package-manager/src/config/readSkillsManifest.ts | Normalize manifest on read; default selfSkill to false. |
| packages/skills-package-manager/src/commands/update.ts | Use expanded manifest for target selection and lock resolution. |
| packages/skills-package-manager/src/commands/install.ts | Use expanded manifest for total count, frozen checks, fetch/link. |
| packages/skills-package-manager/src/commands/init.ts | Set selfSkill: false in generated manifests. |
| packages/skills-package-manager/src/commands/add.ts | Sync lock based on expanded manifest so self skills are locked. |
| packages/skills-package-manager/skills/skills-package-manager-cli/SKILL.md | Add a bundled maintainer self-skill document. |
| packages/skills-package-manager/README.md | Document selfSkill default and behavior. |
| packages/skills-package-manager/package.json | Publish skills/ directory in the package files list. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
packages/skills-package-manager/skills/skills-package-manager-cli/SKILL.md
Outdated
Show resolved
Hide resolved
selfSkill option
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
This PR adds repo-authored self skill support to
skills-package-manager, including manifest expansion, install and update integration, and a bundled self-maintenance skill for this repository. It also updates the CLI init flow, lockfile behavior, tests, and docs so the newselfSkillsupport is covered end to end.Related Links
None
Checklist