Content
62%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is well-organized with executable main code and genuine validation checkpoints, but it is weakened by a non-functional placeholder 'Core Workflow' and a monolithic single-file structure with no progressive disclosure.
Suggestions
Replace or remove the '### 3 Flow Past a Cylinder' placeholder with either a complete executable example or move it out of the numbered 'Core Workflows' so it is not presented as a working workflow.
Move the full lid-driven cavity solver into a scripts/ file (e.g., scripts/lid_driven_cavity.py) and keep a concise invocation snippet plus results-interpretation in SKILL.md to improve progressive disclosure.
Trim the plotting boilerplate to the essential lines, or relocate it to a reference, to tighten conciseness.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean code with no concept lectures, but the 'cylinder_flow_simple' block is a low-value placeholder ('This is a simplified placeholder — real cylinder flows need proper immersed boundary') and the plotting boilerplate is lengthy; not a 1 because there is no padded explanation of concepts Claude knows, not a 3 because of the placeholder and boilerplate that could be tightened. | 2 / 3 |
Actionability | The lid-driven cavity solver and centerline code are fully executable and copy-paste ready, but '### 3 Flow Past a Cylinder' is a Core Workflow whose code does not actually simulate anything ('... (full NS solver needed for proper simulation)'); not a 3 because a presented core workflow is incomplete placeholder code, not a 1 because the primary workflow is concrete and runnable. | 2 / 3 |
Workflow Clarity | Workflows are numbered, the cavity solver includes inline validation (CFL/diffusion stability check, periodic divergence monitoring), and a 'Validation Checklist' plus 'Troubleshooting' table provide explicit checkpoints and error-recovery guidance; not a 2 because validation is explicit rather than implicit. | 3 / 3 |
Progressive Disclosure | The file is well-sectioned (Overview, When to Use, Do NOT Use, Core Workflows, Reference, Validation, Troubleshooting) with no nested references, but there are no bundle files at all and a ~110-line full solver is inlined in a ~220-line SKILL.md that could be offloaded to a script; not a 1 because organization is good, not a 3 because content that should be separate is inline and there are no one-level-deep references. | 2 / 3 |
Total | 9 / 12 Passed |