Content
71%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A well-structured, information-dense skill body with concrete MCP tool guidance, a sequenced validated workflow, and strong reference signaling. Its main weaknesses are the missing copy-paste code examples and, critically, an entire references/ bundle that is referenced but not present.
Suggestions
Ship the references/ bundle: create the 13 referenced .md files (terminology.md, actions.md, action-groups.md, tools.md, keymaps.md, preferences.md, editor-tabs.md, interface-group.md, non-dumb-aware.md, idea-configuration-xml.md, lifecycle-plugins.md, common-failures.md, concept-catalog.md) so the signaled navigation resolves to real content.
Add at least one copy-paste-ready JSON example of an mps_mcp_insert_root_node_from_json call (e.g. a minimal ActionDeclaration or ActionGroupDeclaration payload) in the Common-Path Workflow.
Add an inline validate→fix→retry loop after step 8 (e.g. "If mps_mcp_check_root_node_problems reports problems, fix and re-check before reloading") instead of deferring all error recovery to common-failures.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense, directive bullets that assume Claude's competence ("A plugin solution is NOT an IntelliJ plugin jar", "ModificationStatement.modifiedGroup is a node reference (r:...), not a concept reference (c:...)") with no basic-concept padding; the only verbosity is the Reference Index enumerating every concept field inline, which could be trimmed since those details belong in the reference files. | 4 / 5 |
Actionability | Concrete, executable guidance throughout — specific MCP tool names ("mps_mcp_insert_root_node_from_json", "mps_mcp_update_node", "mps_mcp_search_root_node_by_name", "mps_mcp_check_root_node_problems") and named example nodes/models ("ShowHelpForNode", "TodoViewer", "jetbrains.mps.ide.actions"); the gap is the absence of any copy-paste JSON example showing an actual node insert payload. | 4 / 5 |
Workflow Clarity | A clearly sequenced 8-step Common-Path Workflow with an explicit validation checkpoint (step 8: "Validate with mps_mcp_check_root_node_problems") and rebuild/reload step; falls short of 5 because the validate→fix→retry feedback loop is only implied via a pointer to references/common-failures.md rather than stated inline. | 4 / 5 |
Progressive Disclosure | The overview is excellently signaled with one-level-deep references ("Open references/terminology.md when starting any plugin task", "Open references/actions.md when declaring..."), but none of the referenced bundle files (references/terminology.md, actions.md, action-groups.md, tools.md, keymaps.md, preferences.md, editor-tabs.md, interface-group.md, non-dumb-aware.md, idea-configuration-xml.md, lifecycle-plugins.md, common-failures.md, concept-catalog.md) actually exist in the bundle, so the progressive disclosure is promised but undeliverable. | 3 / 5 |
Total | 15 / 20 Passed |