Repository navigation
docs: lead with the MCP server and specify skill packaging - #10
Merged
Merged
Conversation
juandresrodca
added this pull request to stack #11
September 22, 2026 10:44
The MCP server is what flat-list assignment tools lack, yet it sat below the demo GIF. It now follows the demo command, the tagline and description name it, the viewer's zero-network-call design is stated as a feature, and docs/skill.md specifies the Claude Code plugin - manifest, skill and the same six tools - as not yet shipped.
juandresrodca
force-pushed
the
docs/mcp-first-readme
branch
from
September 22, 2026 10:46
3bc12e2 to
ed612b9
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
1 participant
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.
Recovered from an unpushed local branch (
3bc12e2, 13 Sep) and pushed under a clean branch name. Unchanged from what was committed then.README restructure — the MCP server now follows the 30-second demo command instead of sitting below the GIF.
New:is dropped from its heading, and the GIF moves into a new One HTML file, zero network calls section that states the inlined-renderer design as a feature: no CDN, no web fonts, no telemetry, with a Pester test failing the build if the generated report links to anything external. It also says plainly that a report from a real tenant carries the data inside it and should be treated like thegraph.jsonit came from.Canonical description now leads with the MCP server, in
docs/discoverability.mdand the.psd1manifest together, per the rule in that file that the two change in the same commit:The reasoning is recorded alongside it: search results truncate, so the thing that separates this from flat-list assignment tools belongs in the first sentence rather than the tail.
docs/skill.md(new, 211 lines) specifies shipping the six read-only tools as a Claude Code plugin — manifest, marketplace entry, and aSKILL.mdcarrying the method the tools alone do not convey: check the snapshot age before trusting it, pin an exact name before querying, quote theViapath rather than paraphrase it, run a what-if before touching a group, never ask for tenant credentials.Status: specified, not shipped. Nothing here installs as a plugin; the README says so and the checklist at the end of the file is the acceptance criteria for the change that would.
Docs and manifest metadata only; no code paths touched.
#9 is stacked on this branch and will retarget to
mainonce this merges.