Represent mojo skill in docs + refresh v0.2.0 description - #361
Conversation
The mojo plugin was registered but never represented in generated docs: - STATIC_SKILLS lacked the mojo skill, so the architecture table counted mojo as 0 skills (AGENTS.md said 45 total while README said 46). - CATEGORY_MAP had no mojo entry and the skills-table categoryOrder omitted 'Languages', so even a mapped Languages group would not render. Fixes: - Add mojo skill to STATIC_SKILLS and map it to a new 'Languages' category; add 'Languages' to categoryOrder so it renders. - Regenerate AGENTS.md / site/content.json (mojo 0 -> 1 skill, total 45 -> 46). - List mojo in the hand-maintained README External skill plugins table. - Refresh the marketplace description to match mojo v0.2.0 (memory/copy optimization and Mojo/Python interop, not just SIMD + GPU). Pin unchanged (v0.2.0 / 4d6f5fe). All marketplace/generate-docs/readme tests pass (190); generate-docs --check is fresh.
There was a problem hiding this comment.
Pull request overview
Ensures the externally pinned mojo plugin is fully represented in the repo’s generated documentation (skills tables + site stats) and updates its marketplace listing text to match the v0.2.0 capabilities.
Changes:
- Add
mojotoSTATIC_SKILLS, map it to a newLanguagescategory, and includeLanguagesin the skills table render order. - Regenerate derived docs/artifacts so skill totals align across AGENTS/README/site (45 → 46).
- Refresh the
mojomarketplace description to reflect v0.2.0 features (memory optimization + Python interop).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
scripts/generate-docs.js |
Adds mojo to static skills, categorizes it under Languages, and ensures the category renders in generated skills tables. |
AGENTS.md |
Updates generated architecture counts and mojo’s skill count to reflect the newly represented skill. |
site/content.json |
Updates the displayed skill total to 46 in the site stats content. |
README.md |
Adds mojo to the hand-maintained external skill plugins table under Languages. |
.claude-plugin/marketplace.json |
Updates the marketplace description for mojo to match v0.2.0 messaging. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | can-i-help | 1 | 1 | Contributor guidance | | ||
| | zig-lsp | 0 | 0 | | | ||
| | mojo | 0 | 0 | | | ||
| | mojo | 0 | 1 | | |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request integrates the 'mojo' plugin and skill into the project's documentation and metadata. Key changes include updating the plugin description in marketplace.json, incrementing the global skill count across several files, and adding 'mojo' to the documentation generation scripts and README under a new 'Languages' category. Feedback highlights an inconsistency in the manual skill count within AGENTS.md and the omission of the plugin from the PURPOSE_MAP in the documentation script, which results in empty cells in the generated architecture tables.
| ``` | ||
| lib/ → Shared library (vendored to plugins) | ||
| plugins/ → 25 plugins, 50 agents (40 file-based + 10 role-based), 45 skills | ||
| plugins/ → 25 plugins, 50 agents (40 file-based + 10 role-based), 46 skills |
There was a problem hiding this comment.
The skill count was updated to 46 here, but the manual section on line 194 still references '45 skills'. Additionally, the mojo skill should be added to the table in that section (likely under a new 'Languages' category) to maintain consistency across the document.
| 'audit-project': 'Code Review', | ||
| 'glidemq': 'Message Queues' | ||
| 'glidemq': 'Message Queues', | ||
| 'mojo': 'Languages' |
There was a problem hiding this comment.
When adding a new plugin to CATEGORY_MAP, it should typically also be added to PURPOSE_MAP (around line 127) to ensure the 'Purpose' column in the generated architecture tables is populated. Currently, mojo is missing from PURPOSE_MAP, which results in an empty cell in the AGENTS.md table.
Follow-up to the mojo v0.2.0 pin bump (#360): mojo was registered but not actually represented in generated docs.
Problems
STATIC_SKILLSomitted mojo's skill -> architecture table counted mojo as 0 skills; AGENTS.md total said 45 while README said 46.CATEGORY_MAPhad no mojo entry and the skills-tablecategoryOrderomittedLanguages, so the group never rendered (count included it, row dropped - caught by the generate-docs 'all skills are represented' test once mojo was added).Fixes
STATIC_SKILLS, map to newLanguagescategory, addLanguagestocategoryOrder.Scope / tests
Pin unchanged (v0.2.0 / 4d6f5fe). 190 marketplace/generate-docs/readme tests pass;
generate-docs --checkfresh. Did not touch zig representation (separate pre-existing gap).