Stop vendoring the bitrefill skill; install from upstream - #8
Conversation
The bitrefill skill ships as an upstream plugin (bitrefill/agents) that registers both the skill and its eCommerce MCP. Vendoring a copy under skills/bitrefill/ drifts from upstream and isn't a supported install path. Remove the vendored skill and point qupick's docs at the plugin install (/plugin install bitrefill@bitrefill-skills). qupick already requires the Bitrefill MCP tools, which the plugin provides.
qupick calls the mcp__bitrefill__* tools directly; it never uses the bitrefill skill's own mechanics. Lead the docs with connecting the Bitrefill MCP (claude mcp add) and present the upstream plugin as an optional superset that also bundles the skill.
🤖 Augment PR SummarySummary: This PR stops vendoring the Bitrefill skill in-repo and switches the docs to installing Bitrefill from its upstream plugin to avoid drift. Changes:
Technical Notes: qupick still relies on Bitrefill’s eCommerce MCP ( 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
| 1. **bitrefill (upstream plugin).** Registers the skill and its eCommerce MCP in one step: | ||
|
|
||
| ``` | ||
| /plugin marketplace add bitrefill/agents |
There was a problem hiding this comment.
README.md: the /plugin ... / /reload-plugins lines read like Claude Code slash-commands; consider explicitly stating they’re meant to be run inside Claude Code (not a terminal), otherwise operators may try to run them in a shell and hit errors.
Severity: low
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
There was a problem hiding this comment.
Addressed in 1cb5c44 — the optional plugin step now states the /plugin lines are slash-commands run inside Claude Code, not a shell. The required step (claude mcp add) is already a shell command.
1 participant
The repo vendored a full copy of the bitrefill skill (
skills/bitrefill/—SKILL.md+ 10 reference files). qupick never used that skill's own mechanics: its purchase flow calls the Bitrefill MCP tools (mcp__bitrefill__*) directly. Vendoring the skill duplicated content qupick doesn't need and drifts from upstream (Bitrefill ships the skill as a plugin and doesn't support installing it by copying files).Changes
skills/bitrefill/entirely./plugin install bitrefill@bitrefill-skills, https://github.com/bitrefill/agents) is presented as an optional superset that also bundles the skill.README.md— rewrite "Install the skill", the Bitrefill prerequisite bullet, the line-7 delegation note, and the safeguards pointer (the vendored path is gone; the README's own Safeguards section is self-contained).AGENTS.md— replace the mirrored "pick a path / safeguards / reference table" section (which linked the deleted files) with an MCP-first prerequisite note.skills/qupick/SKILL.md— frontmatter + delegation note + the internalbuy-products/search-productsreferences now point at the Bitrefill MCP; the broken../bitrefill/...links are removed.Notes
skills/bitrefillreferences remain in tracked files.