Skip to content

feat: add dynamic ai-helpers skill fetching from GitHub#187

Closed
jpuzz0 wants to merge 1 commit intopatternfly:mainfrom
jpuzz0:skill-mcp-resources
Closed

feat: add dynamic ai-helpers skill fetching from GitHub#187
jpuzz0 wants to merge 1 commit intopatternfly:mainfrom
jpuzz0:skill-mcp-resources

Conversation

@jpuzz0
Copy link
Copy Markdown

@jpuzz0 jpuzz0 commented May 6, 2026

Summary

  • Adds useAiHelpersSkill MCP tool — call with no args to list all skills, pass a name to get full skill content
  • Adds aihelpers://skills/index resource and aihelpers://skills/{name} resource template for MCP client browsing
  • Fetches skills dynamically from GitHub at runtime with disk cache fallback — no bundled JSON, no manual sync
  • 5-minute in-memory TTL, 10s fetch timeout, graceful degradation if GitHub is unreachable

Companion PR: patternfly/ai-helpers#72 (adds dist/skills.json generation + CI auto-update)

Ref: PF-4034

How it works

An agent in a Claude Code session asks about skills → the MCP server calls useAiHelpersSkill → fetches dist/skills.json from GitHub → returns skill content to the agent. The agent then follows the skill instructions. No config, no install — just the MCP connection.

Test plan

End-to-end verification requires patternfly/ai-helpers#72 merged first. Once landed: start a Claude Code session with the PF MCP connected, ask "what ai-helpers skills are available?" → agent lists skills via the tool. Ask it to use a specific skill by name → agent retrieves and follows the skill instructions.

@jpuzz0 jpuzz0 requested a review from cdcabrera as a code owner May 6, 2026 16:02
@jpuzz0 jpuzz0 force-pushed the skill-mcp-resources branch from 4a714d9 to 1040b76 Compare May 6, 2026 16:03
Add useAiHelpersSkill MCP tool and resource templates that fetch skills
from ai-helpers at runtime. Disk cache fallback when GitHub is unreachable.

Ref: PF-4034
@jpuzz0 jpuzz0 force-pushed the skill-mcp-resources branch from cf5d2f1 to d5fc18b Compare May 6, 2026 16:38
@cdcabrera cdcabrera added the hold review needed label May 6, 2026
@cdcabrera
Copy link
Copy Markdown
Member

Closing this down @jpuzz0 this work was still being discussed.

@cdcabrera cdcabrera closed this May 6, 2026
@cdcabrera cdcabrera added the planning needed Work may already be in progress or POC but aspects of it are still in the planning stage. label May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold review needed planning needed Work may already be in progress or POC but aspects of it are still in the planning stage.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants