docs(skills): add tool-design to inventory, reconcile counts to 101#64
Conversation
Add `tool-design` to the Doctrine layer in DOMAIN-SKILLS.md and reconcile the native skill count across all docs to the filesystem-verified total of 101 (was inconsistently 100/72/76 across files). - DOMAIN-SKILLS.md: Doctrine 14->15, total 100->101, primary 72->73; new tool-design row (28 helpers unchanged; 73+28=101). - USER-GUIDE.md: "All 100 skills" -> 101; install-output "Skills: 100" -> 101. - COMMANDS.md: taxonomy "(100 skills)" -> "(101 skills)". - cheatsheets/CLAUDE-skills.md: "total = 76" -> 101 (76 was a stale undercount). - cheatsheets/CLAUDE.md: "100 skills nativas v0.1.0" -> "101 skills nativas". No version bump (stays 0.1.2). README Skills table left untouched (orchestration-core only). .maxvision/ specs untouched (historical). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughDocumentation updates consistent with adding a new ChangesSkill Count Update to 101
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c83d9413e0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ``` | ||
| Installed maxvision@maxvision (v0.1.0) | ||
| Skills: 100 | ||
| Skills: 101 |
There was a problem hiding this comment.
This expected-output block now combines the v0.1.0 install banner with the 101-skill count, but tool-design is documented in CHANGELOG.md as added in 0.1.2 and the plugin metadata currently reports 0.1.2. Users validating a fresh install will never see this exact output: v0.1.0 had the old skill set, while a 101-skill install should be labeled v0.1.2.
Useful? React with 👍 / 👎.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/cheatsheets/CLAUDE-skills.md`:
- Line 23: A frase "Skills nativas total = **101**. Todas invocáveis via
`/maxvision:<skill-name>`" está incorreta; altere esse trecho para indicar que
há 101 skills nativas, mas apenas as marcadas como "user-invocable" podem ser
chamadas diretamente via o padrão `/maxvision:<skill-name>` e que outras skills
internas são não-invocáveis pelo usuário (orchestrate as instala/usa on-demand);
update the line containing "Skills nativas total = **101**. Todas invocáveis via
`/maxvision:<skill-name>`" to explicitly state the distinction and how to
identify/invoke user-invocable skills.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: fe3c043a-63eb-4425-a5ad-a1b9819afd25
📒 Files selected for processing (5)
docs/COMMANDS.mddocs/DOMAIN-SKILLS.mddocs/USER-GUIDE.mddocs/cheatsheets/CLAUDE-skills.mddocs/cheatsheets/CLAUDE.md
|
Both findings addressed in the latest commit:
Copilot errored out on review (twice on prior PRs) — proceeding per maintainer policy. |
|
To use Codex here, create an environment for this repo. |
- CLAUDE-skills.md (CodeRabbit Major): "Todas invocáveis" was false after reconciling to 101 — 28 of the 101 are internal orchestrate helpers, not user-invocable. Clarify: 73 primary invocable + 28 internal helpers. - USER-GUIDE.md (Codex P3): install-example banner showed v0.1.0 alongside the 101-skill count; a 101-skill install is v0.1.2. Bump banner to v0.1.2. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
# List CodeRabbitAI's review comments on PR `#64` to find the one to resolve
gh api repos/produtoramaxvision/maxvision/pulls/64/comments \
--jq '.[] | select(.user.login == "coderabbitai") | {id: .id, body: .body[0:120]}'Length of output: 476 🏁 Script executed: #!/bin/bash
# Check issue-level comments on PR `#64`
gh api repos/produtoramaxvision/maxvision/issues/64/comments \
--jq '.[] | select(.user.login == "coderabbitai") | {id: .id, body: .body[0:150]}'Length of output: 441
The PR looks good to merge. 🐇 |
Summary
Documentation follow-up to the
tool-designskill (shipped in v0.1.2). Adds the skill to the inventory docs and reconciles the native skill count — previously inconsistent across files (100 / 72 / 76) — to the filesystem-verified total of 101.No version bump (stays 0.1.2). Routed via
/orchestrate→maxvision-doc-writer.Changes
docs/DOMAIN-SKILLS.mdtool-designrow (28 helpers unchanged)docs/USER-GUIDE.mddocs/COMMANDS.mddocs/cheatsheets/CLAUDE-skills.mddocs/cheatsheets/CLAUDE.mdVerification
find skills -name SKILL.md -not -path "*/worktrees/*" | wc -l== 101Out of scope (intentionally left)
README.mdSkills table (orchestration-core only — tool-design doesn't fit).maxvision/specs (historical/frozen)COMMERCIAL.md"20–100 skills" (build-estimate range, not a count)@codex @copilot @CodeRabbit — please review.
🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Documentation