Content
88%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 high-quality, information-dense skill body: a five-step workflow with real validation and retry loops, runnable code for the main paths, and disciplined avoidance of padding. The two gaps are minor: undefined variables in the table example, and reference-grade table/frame detail inlined in the main file rather than split out.
Suggestions
In the TableBlueprint example, show how `path` and `dataset` are obtained (e.g., a temp file path and the dataset handle) so the snippet is copy-paste ready.
Consider moving the table-blueprint path-escaping rules and the named-coordinate-frame troubleshooting into a references/ file, keeping SKILL.md as a leaner overview with clearly signaled one-level-deep links.
Add a short code or command example for component overrides (colors/colormaps) since the section currently points only to the external MCP docs.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence — 'The constructors are discoverable (dir(rrb), help(rrb.Spatial3DView)); this skill covers what you can't guess' — with no padding or explanation of concepts Claude already knows. Nearly every line carries non-obvious, project-specific detail (mode semantics, reference-integrity rules, store-identity binding), matching the 'every token earns its place' anchor. | 5 / 5 |
Actionability | Mostly executable: complete, runnable examples for blueprint construction, the ViewerClient screenshot loop, dataset schema enumeration, and concrete MCP call shapes ('rerun_set_blueprint with {"table": {}, "mode": "partial"}'). It falls short of 5 because the TableBlueprint snippet references undefined variables ('blueprint.save("my_app", path)', 'dataset.register_blueprint(path.as_uri(), ...)') without showing how path or dataset are obtained — a minor but real copy-paste gap. | 4 / 5 |
Workflow Clarity | A clear numbered sequence (1. Read the data → 5. Iterate from screenshots) with explicit validation checkpoints — 'verify state before assessing screenshots', 'After each write, check the views and their reports in rerun_get_state, plus logs' — and an error-recovery feedback loop ('If the structure differs, inspect rerun_get_blueprint before retrying') plus a final screenshot check before reporting completion. This matches the top anchor including the retry loop. | 5 / 5 |
Progressive Disclosure | Good structure with clearly signaled one-level external references (MCP docs for component overrides and table blueprints, the table-layout howto, and a 'See also' section for sibling skills), and no bundle files exist so nothing is misfiled. It is not 5 because a single ~240-line file inlines fairly deep reference-grade material — table column path-escaping rules and the named-coordinate-frame troubleshooting — that would fit a reference file under a leaner overview. | 4 / 5 |
Total | 18 / 20 Passed |