Content
81%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 tightly written routing skill: dense with real pitfalls and concrete MCP tool calls, a validated workflow with error-recovery paths, and disciplined deferral of detail to per-job reference files. The main weaknesses are navigational redundancy between its two index sections and a path inconsistency (file-vs-directory references) in the Reference Index.
Suggestions
Merge the Section index and Reference Index into a single routing table — they cover the same jobs, and `scope-fqn-reference` is currently pointed to from three separate places.
Fix the path inconsistency in the Reference Index: it names `references/property-constraints.md` and `references/referent-constraints.md` as files, while the Section index shows these are directories containing multiple files.
Inline one minimal `ConceptConstraints` root JSON blueprint in the workflow (step 3 currently defers entirely to `references/concept-roots.md`) so the single most common case is executable without a file read.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Efficient and assumes domain competence ("The **constraints** aspect controls *runtime* rules about nodes" — no tutorial padding; no explaining what MPS is). Not 5: the Section index and Reference Index overlap in purpose, and `scope-fqn-reference` is pointed to in three places (directives, workflow step 2, index), which could be tightened. Not 3: nearly every line is routing, a pitfall, or a concrete directive — there is no unnecessary explanation. | 4 / 5 |
Actionability | Concrete executable guidance: named MCP tools with parameters ("create it with `mps_mcp_create_model` (`moduleName: "<lang>", `modelName: "<lang>.constraints"` — aspect ID `constraints`, case-sensitive, no `@` suffix")"), exact FQNs, and hard rules ("**must explicitly assign** `node.<prop> = propertyValue`"). Not 5: the executable JSON blueprints are entirely deferred to reference files, so inline coverage of common cases has minor gaps. Not 3: what is inline is specific and directly executable, not pseudocode or vague direction. | 4 / 5 |
Workflow Clarity | The Common-Path Workflow is a clearly sequenced 6-step procedure ending in "Validate with `mps_mcp_check_root_node_problems` and rebuild the language", with explicit error-recovery routing ("if the new constraint is invisible at runtime, run `mps_mcp_reload_all`" and `references/common-failures.md` for symptom-driven fixes). Explicit validation plus feedback loops matches anchor 5, not 4. | 5 / 5 |
Progressive Disclosure | Good hub structure: a "one row, one read" Section index table, a "Start here" pointer for the most common case, and one-level-deep references. Not 5: the Reference Index refers to `references/property-constraints.md` and `references/referent-constraints.md` as single files while the Section index shows those are directories (`references/property-constraints/nodepropertyconstraint-shape.md`), and the two indexes are partially redundant; also no bundle files were present to verify the referenced paths. Not 3: references are clearly signaled, shallow, and content is appropriately split rather than inlined. | 4 / 5 |
Total | 17 / 20 Passed |