Content
78%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 highly actionable with concrete commands and clear conditional workflows, and well-structured for a single-file skill. Its main weakness is conciseness: the runtime-contract separation and the do-not-init-to-view points are stated multiple times and could be consolidated.
Suggestions
Consolidate the runtime-contract disclaimer: state once that in-project read/write guidance ships as the separate project-local skill, then have later sections reference it instead of re-explaining.
Merge the repeated 'Never run ok init just to view or open a file' guidance into a single prominent callout in the Opening-a-file section rather than restating it.
Add a brief post-`ok init` verification cue (e.g. 'ok init prints which agents it wired; re-run to refresh') so the mutating setup step has an explicit checkpoint.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with commands and short imperatives, but the runtime-contract disclaimer ('does not carry the in-project read/write contract / ships separately') is restated in the intro, install section, and a dedicated Working-inside-a-project section, and the 'never run ok init just to view a file' point is repeated — more than minor tightening needed. | 3 / 5 |
Actionability | Fully executable guidance throughout — copy-paste commands (npx @inkeep/open-knowledge init, ok init, ok cowork, ok open /abs/path), the preview_url MCP tool with its file parameter, --project <dir> variants, and a concrete decision branch by viewing surface covering the common cases. | 5 / 5 |
Workflow Clarity | Clear sequences (numbered share steps 1-3, conditional open-a-file branch by surface) with feedback guidance ('exits non-zero and says why, so you never need a second command'); minor validation gaps since the mutating ok init setup step has no explicit post-run verify checkpoint. | 4 / 5 |
Progressive Disclosure | Single-file skill with well-organized section headers and one-level-deep pointers to docs/source links; no bundle files exist and none are needed, so well-organized sections satisfy the no-external-references exception. | 5 / 5 |
Total | 17 / 20 Passed |