Repository navigation
README: document asking an AI agent to install the skill (easiest path) - #10
Merged
Merged
Conversation
4 tasks done
MRagabMob
force-pushed
the
readme-agent-install
branch
from
August 13, 2026 16:16
c5f8b97 to
4db5180
Compare
Verified on both Codex (installs via $skill-installer to ~/.codex/skills/) and Claude Code (installs as a personal skill plus the slash commands in ~/.claude/commands/). Becomes item 1 and takes the "easiest, recommended" badge from the AGENTS.md item; items renumber to 1-7. The documented prompt carries the /tree/main/skills/paymob-integration subdirectory rather than the bare repo root. Codex resolved the subdirectory unaided, but only after searching the web and inspecting the layout, and an agent that guesses wrong installs the multi-agent plugin package as though it were a standalone skill. States the two limits this path has, both found by testing rather than reading: it does not register the MCP server, so there is no live account access, and it does not namespace the skill, so installing it alongside the plugin yields two identically-described skills. Also documents that a fresh session is required, which is the most common reason the install looks like it failed. Separately: the Claude Code plugin block now notes that a shell "claude plugin install" needs /reload-plugins or a restart to take effect, and marketplace.json gains the description that "claude plugin validate --strict" requires; validate.py now enforces it.
The intro reads as though the plugin ships through Codex, Claude, and Lovable marketplaces. It does not: this repository is the only distribution channel today, and the Paymob MCP server is in no vendor connector registry either. A user who searches a marketplace for "Paymob", finds nothing, and concludes the install is broken is the failure this note prevents. The Claude Code path registers this repo as a custom marketplace, which needs no vendor listing, so every documented method works as written.
MRagabMob
force-pushed
the
readme-agent-install
branch
from
August 13, 2026 16:27
d257276 to
0f04f23
Compare
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
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.
Adds "just ask your agent to install it" as README item 1, with the
(easiest, recommended)badge moved off theAGENTS.mditem. Items renumber to 1–7.Verified, not assumed
Tested on both agents before documenting:
$skill-installer, installs 17 files to~/.codex/skills/paymob-integration~/.claude/skills/paymob-integration/and the three slash commands into~/.claude/commands/(creating that directory)Three things testing surfaced that reading the docs would not have
The prompt has to carry the subdirectory. Codex did resolve
skills/paymob-integrationunaided — but only after searching the web and inspecting the repo layout, and its target directory already existed. An agent that guesses wrong installs the multi-agent plugin package as if it were a standalone skill. The documented prompt therefore includes/tree/main/skills/paymob-integrationso nothing is left to inference.A fresh session is required. Skills and commands are read at session start, so they do not appear in the session that installed them. This is the single most likely reason a user concludes the install failed, and it was undocumented.
This path does not give you the MCP server. Claude Code deliberately declined to register it, correctly — it is a persistent config change pointing at a live financial service. That is the real trade-off against the plugin install, so item 1 states it plainly and routes anyone wanting live account access to item 7 rather than leaving them to discover the gap later. Item 1 also notes the name collision you get from installing both ways: two
paymob-integrationskills with identical descriptions.Also in here
/reload-pluginsis now documented. A shellclaude plugin installdoes not take effect in a running session. This was missing and is a predictable support burden.marketplace.jsongains adescription.claude plugin validate . --strict— Anthropic's own validator, run against this repo — failed withNo marketplace description provided. It now passes.validate.pyenforces the field so it cannot regress.On
claude plugin marketplace addI flagged in #8 that this might not exist as a shell command: both the plugins-reference and cli-reference pages enumerate
claude pluginsubcommands and neither listsmarketplace.That was wrong, and the README is correct as-is. I resolved it by fetching the published CLI and asking it directly:
addalso takes--scopeand--sparse. The docs are simply incomplete. No change made to the install commands or to thevalidate.pyrequired string — recording it here so the question doesn't get reopened.Test plan
python scripts/validate.pypassesclaude plugin validate . --strictpasses (failed before thedescriptionfix)python scripts/package_skill.pybuildsvalidate_install_docsrequired substrings intact through the renumberingeasiest, recommended;#live-access--paymob-mcp-serverand#installationanchors resolve to real headings (validate.pydoes not check#anchortargets, so this was checked by hand)Sequencing
Stacked behind two open PRs, both of which should land first:
${CLAUDE_PLUGIN_ROOT}is never expanded, so without Fix command reference paths so they resolve outside a plugin install #9 the commands it installs cannot read their reference files.Both touch
README.md, but in different sections from this PR, so conflicts are unlikely. Happy to rebase whichever lands last.