Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is lean, well-organized, and packed with concrete rules, but its actionability and workflow clarity are undermined because the workflow and docs files it routes to are missing from the bundle, and there is no in-line executable extension skeleton. Progressive disclosure is partially broken by the same missing references.
Suggestions
Add the missing `workflows/create-extension.md`, `add-capability.md`, and `debug-extension.md` files (or inline minimal step-by-step build/debug procedures) so the routing targets resolve.
Include a copy-paste-ready minimal extension skeleton (default export, `ExtensionAPI` wiring, one tool and one command registration) directly in the body to lift actionability to the top anchor.
Either provide the referenced `docs/extension-sdk/manifest-spec.md`, `testing.md`, and `docs/extension-sdk/` files or remove those entries from the reference_index so all navigation links resolve.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense, token-efficient bullet lists and tables with no concept-explanation padding (no 'what is TypeScript/an extension' filler); every line states a concrete rule or import, matching the 'lean and efficient; every token earns its place' anchor. | 3 / 3 |
Actionability | Concrete directives exist ("Use `StringEnum` from `@gsd/pi-ai`", "Truncate tool output to 50KB / 2000 lines", "Use `pi.exec()` not `child_process`"), but there are no executable code examples or an extension skeleton, and the detailed build steps live in `workflows/create-extension.md` which is not present in the bundle. | 2 / 3 |
Workflow Clarity | Routing maps intents to workflows and `success_criteria` provides a solid validation checklist (compiles, loads on `/reload`, tools appear in system prompt), but the core build/debug sequences are deferred to `workflows/*.md` files that do not exist, leaving the main multi-step sequence absent from the bundle. | 2 / 3 |
Progressive Disclosure | Section organization and one-level-deep signaling are good, but routing points to `workflows/create-extension.md`, `add-capability.md`, and `debug-extension.md` and the reference_index cites `docs/extension-sdk/manifest-spec.md`, `testing.md`, and `docs/extension-sdk/` — none of which exist in the bundle, so a large share of the navigation is dead links rather than resolvable one-level-deep references. | 2 / 3 |
Total | 9 / 12 Passed |