Content
86%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 tight, actionable skill body: the module layout, interaction rules, and verification gate are all stated concretely with no padding. The remaining gaps are an undefined-template-placeholder risk, a missing minimal example of the module test class, and no guidance on recovering from a failed verify run.
Suggestions
Briefly define or resolve the placeholders — state that {{BASE_PACKAGE}}/{{BASE_PATH}} are substituted from project configuration (or show one concrete expanded example like com.acme.billing) so the paths are unambiguous at execution time.
Include a 5-8 line skeleton of <Module>ModuleTests.java showing the @ApplicationModuleTest annotation and one public-API-driven test, making step 4 copy-paste ready.
Add a short error-recovery step after "Run `./mvnw verify`": e.g., "If ModuleBoundariesTests fails, the report names the offending references — fix the caller or move the type to internal, then re-run."
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — e.g., "Each direct subpackage of `{{BASE_PACKAGE}}` is a module. `./mvnw verify` fails if another module reaches into its nested packages or if modules form a cycle" states only project-specific facts, and annotations like `@ApplicationModuleListener` are used without tutorial explanation. Every sentence instructs; nothing re-teaches Spring Modulith concepts, matching anchor 5. | 5 / 5 |
Actionability | Concrete and mostly executable: exact layout ("{{BASE_PACKAGE}}.<module>.internal"), exact test path ("src/test/java/{{BASE_PATH}}/<module>/<Module>ModuleTests.java"), annotations, and the command "Run `./mvnw verify`". However, no code snippet is given for the test class or the public API surface, and the placeholders {{BASE_PACKAGE}}/{{BASE_PATH}} are never defined or explained in the skill — minor gaps that fit anchor 4 rather than 5's fully copy-paste-ready coverage. | 4 / 5 |
Workflow Clarity | A clear, well-ordered six-step sequence with checkpoints: step 1 verifies name availability ("Check that no existing module owns it") and step 6 runs the gate ("Run `./mvnw verify`"). It falls short of anchor 5 because there is no error-recovery loop telling Claude what to do when verify fails (e.g., read the ModuleBoundariesTests report, fix the offending reference, re-run), though it comfortably exceeds anchor 3's implicit-checkpoint level. The destructive/batch cap does not apply since adding a module is an additive operation with verification present. | 4 / 5 |
Progressive Disclosure | At roughly 20 lines with a single heading, a two-line framing sentence, and a numbered procedure, the skill is well-organized and needs no external files — the simple-skill guideline allows a 5 here. Its only external pointer ("the `Modules` section of `AGENTS.md`") is clearly signaled and one level deep, and no bundle files exist to reference. | 5 / 5 |
Total | 18 / 20 Passed |