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 highly actionable, template-driven skill whose description-side discipline carries into concrete commands, tables, and a self-review gate. Its weaknesses are repetition of the same offline-sync/tenant-scoping rules across four sections and reliance on undefined external references (project-context, plugin refs) with the full spec template inlined rather than split out.
Suggestions
Consolidate the tenant/org-scoped offline-read rule into a single authoritative statement (e.g., in the Offline Sync section) and have the checklist and anti-patterns reference it, rather than restating the full rule four times.
Move the ~170-line spec template to a template file under references/ (or scripts/) and point to it with one clearly signaled link, keeping SKILL.md as the workflow overview and rules.
Resolve the dangling 'project-context' references: state once where project-context lives (a file path or skill) or inline the workspace→path table it keeps deferring to.
Present the operating sequence as an explicit ordered list (fetch ticket → announce → resolve design refs → fill template → run gate → save → branch/PR → post-merge comment) so the workflow order doesn't have to be inferred from section layout.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly dense, project-specific instruction with no padding about concepts Claude already knows, but several rules are stated repeatedly: the tenant/org-scoped offline-read filter appears in Mobile UI, Offline Sync, Project-Specific Rules, the Self-Review Checklist, and Anti-Patterns; the 'never edit a generated sync schema file' rule appears twice; role-derivation rules appear three times. This matches 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than 4, where over-explanation would only be minor. | 3 / 5 |
Actionability | Fully copy-paste ready: exact save paths ('docs/superpowers/specs/<STORY-KEY>.md'), a branch name, a PR title, an executable bash block ('bash ${CLAUDE_PLUGIN_ROOT}/scripts/jira-site-guard.sh || exit 1', 'acli jira workitem comment create ...'), a complete markdown spec template with tables and fully-typed TypeScript interface examples, and a machine-checkable self-review gate. This matches 'fully executable; copy-paste ready code or commands; specific examples cover the common cases'. | 5 / 5 |
Workflow Clarity | The flow is discernible and guarded: a hard precondition ('If you did not successfully fetch the Jira ticket before calling this skill, STOP'), an announced start, a template to fill, a self-review gate ('before saving -> every item in gate must hold'), and post-merge steps. It is not 5 because the sequence is never presented as an ordered step list — the reader must infer the order of Guard → template → checklist → save → PR → comment from section layout — and there is no explicit retry/recovery loop if the gate fails. | 4 / 5 |
Progressive Disclosure | Headings are clear, but no bundle files exist (no references/, scripts/, or assets/ directories) while the body leans on external pointers that are neither verifiable nor clearly signaled as navigation: 'plugins/sdlc/refs/artifact-encoding.md' is cited in one prose line, '${CLAUDE_PLUGIN_ROOT}/refs/design-references.md' in another, and 'project-context' is invoked repeatedly without ever being resolved to a file. The ~170-line inlined spec template is content that could live in a separate template file. This matches 'some structure but could be better organized; references present but not clearly signaled'. | 3 / 5 |
Total | 15 / 20 Passed |