Skip to content

fix(skills): handle standalone GitHub skill URLs#409

Merged
christso merged 1 commit into
mainfrom
fix/standalone-skill-source
May 20, 2026
Merged

fix(skills): handle standalone GitHub skill URLs#409
christso merged 1 commit into
mainfrom
fix/standalone-skill-source

Conversation

@christso
Copy link
Copy Markdown
Contributor

Summary

  • fix standalone allagents skill add <GitHub skill URL> so the direct-install path uses the fetched source path only once
  • keep the standalone GitHub URL flow on the direct-plugin path instead of falsely reporting Available skills: none
  • add regression coverage for both URL path resolution and the standalone install flow

Root cause

installSkillFromSource() resolved the GitHub subpath before handing off to installSkillDirect(), and installSkillDirect() resolved the same subpath again. For URLs like https://github.com/NousResearch/hermes-agent/tree/main/skills/research/llm-wiki, that produced a non-existent path under the cache and made skill discovery return no skills.

Manual E2E

Red

cd /tmp/allagents-standalone-red-RBtPTs
node /home/entity/projects/EntityProcess/allagents.worktrees/fix-standalone-skill-source/dist/index.js skill add https://github.com/NousResearch/hermes-agent/tree/main/skills/research/llm-wiki
# before fix: Error: Skill 'llm-wiki' not found ... Available skills: none

Green

cd /tmp/allagents-standalone-red-RBtPTs
node /home/entity/projects/EntityProcess/allagents.worktrees/fix-standalone-skill-source/dist/index.js skill add https://github.com/NousResearch/hermes-agent/tree/main/skills/research/llm-wiki
# after fix: ✓ Enabled skill: llm-wiki (llm-wiki)
find .agents/skills -maxdepth 1 -mindepth 1
# shows .agents/skills/llm-wiki

Validation

bun test tests/unit/cli/skills-add-url-detection.test.ts tests/unit/cli/skills-add-standalone-install.test.ts
bun run lint
bun test
bun run build

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying allagents with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8aa3f0c
Status: ✅  Deploy successful!
Preview URL: https://efe3ca68.allagents.pages.dev
Branch Preview URL: https://fix-standalone-skill-source.allagents.pages.dev

View logs

@christso christso merged commit 4b8954b into main May 20, 2026
1 check passed
@christso christso deleted the fix/standalone-skill-source branch May 20, 2026 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant