Make the agent skills installable via APM - #54
Merged
Conversation
The skills live in `.agents/skills/`, which is the cross-runtime location
agents read directly — but it is not a layout APM discovers in a source
repo. APM expects `apm.yml` + `.apm/skills/` at the package root
(`.agents/skills/` is its deploy *target*, not a source), so pointing a
dependency at this repo failed with "no apm.yml, SKILL.md, hooks, or
plugin structure found".
Adds the two files that make it a package, mirroring how our other skill
bundles are laid out:
* `apm.yml` — name, description, version
* `.apm/skills` — symlink to `../.agents/skills`
`.agents/skills/` stays the single source of truth, so a plain checkout
keeps working without APM, and one dependency now pulls all six skills:
dependencies:
apm:
- git: git@github.com:phpro/http-tools.git
ref: v2.x
Verified against APM 0.29.0: resolves as `package_type: apm_package` and
integrates 6 skills with their `references/` files intact.
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.
Follow-up to #53. The skills are in
.agents/skills/, which agents read directly — but APM does not discover that layout in a source repo, soapm install phpro/http-toolsfails:.agents/skills/is APM's cross-runtime deploy target (--target agent-skills, converged in APM v0.22.0), not a source location. Its source layout isapm.yml+.apm/skills/<name>/SKILL.md.Change
Two files, mirroring how
phpro/hexagonal-skillsand friends are laid out:apm.yml— name, description, version.apm/skills— symlink to../.agents/skills.agents/skills/remains the single source of truth, so a plain checkout still works with no APM involved. The README gains a short install section.Result
One dependency instead of six per-skill paths:
Subsetting also becomes available:
Verification
Installed from this branch with APM 0.29.0 into a scratch project:
Resolves as
package_type: apm_package; all 10 files deploy,references/included. Also confirmed on both 0.14.2 and 0.29.0 that the repo root is not auto-detected without these files, so this is required rather than cosmetic.