Content
63%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.
The body is a strongly actionable CLI reference — real commands, a full action catalog, and well-handled connection-state branching with recovery paths. Its weaknesses are a marketing-style intro and other cuttable filler, a dangling "Step 2" reference in an otherwise unnumbered workflow, and no progressive disclosure: everything, including boilerplate and catalog reference material, is inlined in a single file.
Suggestions
Trim the three-sentence product intro to one line and delete filler like "Use action names and parameters as needed." — Claude needs the operational commands, not a description of what Codacy is for.
Extract the Membrane auth/connection boilerplate (and optionally the full 16-action table) into a references/ file, keeping a quick-start and a handful of top actions inline in SKILL.md.
Fix the dangling "skip to Step 2" references by numbering the workflow steps explicitly, and clarify that the table's "Key" value is what gets passed as <actionId> to "membrane action run".
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The intro paragraph ("Codacy is a code analytics platform that helps developers and teams monitor and improve code quality… reduce technical debt") is product-pitch filler, and "Use action names and parameters as needed." adds no instruction. This fits anchor 3 ("Mostly efficient but includes some unnecessary explanation or could be tightened"); it is above 2 because the bulk of the body is dense command and table reference with little padding. | 3 / 5 |
Actionability | Concrete, copy-paste commands appear throughout ("npm install -g @membranehq/cli@latest", "membrane connection ensure \"https://www.codacy.com/\" --json", "--input '{\"key\": \"value\"}'") alongside a 16-action catalog. This matches anchor 4 ("Mostly executable guidance; concrete code or commands with minor gaps"); it falls short of 5 because "membrane action run <actionId>" never shows a concrete action id and the table's "Key" column is never explicitly linked to <actionId>. | 4 / 5 |
Workflow Clarity | The sequence (install → login → connection ensure → poll → search → run) is clear, with explicit state-based checkpoints (READY, CLIENT_ACTION_REQUIRED, CONFIGURATION_ERROR) and recovery guidance ("poll again… to check if the state moved to READY"). This fits anchor 4; it is not 5 because the body twice says "skip to Step 2" yet no section is labeled Step 2 and the steps are not numbered, and it is not 3 because checkpoints are present and explicit. | 4 / 5 |
Progressive Disclosure | There are no bundle files — all 157 lines live inline in SKILL.md — and the 16-row action table plus the Membrane auth/connection boilerplate are reference-grade content that belongs in separate files. This matches anchor 3 ("Some structure but could be better organized… content that should be separate is inline"); it is above 2 because headers are clear and the document is easy to navigate. | 3 / 5 |
Total | 14 / 20 Passed |