Use virtual subdirectory install to avoid pulling entire repo via apm#357
Use virtual subdirectory install to avoid pulling entire repo via apm#357
Conversation
When users run `apm install arika0093/Linqraft`, the entire .NET repository (source code, tests, etc.) was cloned to apm_modules/. Only the files under .apm/ are needed. Following the APM documentation's virtual subdirectory mechanism: - Add .apm/apm.yml (package manifest for virtual subdirectory install) - Create .apm/.apm/ nested structure with the same primitives (instructions, prompts, skills) so that APM's package validation passes when only the .apm/ directory is installed - Update README.md: apm install arika0093/Linqraft/.apm Uses git sparse-checkout to install only the .apm/ directory - Fix path reference in prompt file (use skill name, not repo path) Agent-Logs-Url: https://github.com/arika0093/Linqraft/sessions/4ecf7d75-dc2d-4d81-a265-6bd248305309 Co-authored-by: arika0093 <4524647+arika0093@users.noreply.github.com>
… and agentskills compat - skills/apm.yml: package manifest enabling APM virtual subdirectory install - skills/linqraft/SKILL.md: SKILL.md at agentskills.io standard path for npx skills etc. - skills/.apm/skills/linqraft/SKILL.md: APM sub-skill promotion path - README.md: update install command and curl URL to use skills/ path Agent-Logs-Url: https://github.com/arika0093/Linqraft/sessions/b9fc72ff-1cb7-4308-83d9-be829708e68b Co-authored-by: arika0093 <4524647+arika0093@users.noreply.github.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b57d11403d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| apm install arika0093/Linqraft/skills | ||
| # or install manually | ||
| curl https://raw.githubusercontent.com/arika0093/Linqraft/main/.apm/skills/linqraft/SKILL.md -o your/skills/linqraft/SKILL.md | ||
| curl https://raw.githubusercontent.com/arika0093/Linqraft/main/skills/linqraft/SKILL.md -o your/skills/linqraft/SKILL.md |
There was a problem hiding this comment.
The updated manual install command now downloads main/skills/linqraft/SKILL.md, but this commit does not add that file (the skill file was moved to skills/.apm/skills/linqraft/SKILL.md), so the documented curl path resolves to a missing file and manual installation fails immediately for users following the README.
Useful? React with 👍 / 👎.
skills/apm.yml— package manifest forapm install arika0093/Linqraft/skillsskills/linqraft/SKILL.md— direct skill access for agentskills.io /npx skillsskills/.apm/skills/linqraft/SKILL.md— APM sub-skill promotion inside the installed packageREADME.md— change install command fromarika0093/Linqraft/.apmtoarika0093/Linqraft/skills