Content
56%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 well-organized, code-forward skill with a genuinely executable flagship workflow and thoughtful validation/troubleshooting sections. Its weaknesses are the non-functional cylinder placeholder, plotting boilerplate, and the total absence of progressive disclosure — everything is inlined in one long file.
Suggestions
Replace the cylinder placeholder with a working penalty/immersed-boundary implementation (or move it to a reference file until it works) so every advertised core workflow is executable.
Split the plotting blocks and the flow-regime/troubleshooting tables into a reference file (e.g. references/visualization.md) and link to it, shrinking SKILL.md to an overview plus the core solver.
Add a short feedback loop for the validation checklist: what to change (dt, grid, Poisson iterations) when the divergence or CFL check fails.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly tight, functional code, but it carries unnecessary weight: ~50 lines of matplotlib boilerplate for the cavity plots and a 'cylinder_flow_simple' placeholder whose loop body is literally '# ... (full NS solver needed for proper simulation)'. This fits the 'mostly efficient but includes some unnecessary explanation or could be tightened' anchor rather than the minor-trimming level 4 anchor. | 3 / 5 |
Actionability | The lid-driven cavity workflow is genuinely executable and complete, but the second advertised workflow (flow past a cylinder) is a placeholder that returns a field forced to zero inside the mask — effectively pseudocode. With one of three core workflows non-functional, this sits at 'some concrete guidance but incomplete' rather than 'mostly executable with minor gaps'. | 3 / 5 |
Workflow Clarity | Steps are clearly organized (When to Use / Do NOT Use / workflows / Validation Checklist / Troubleshooting) with real checkpoints: a CFL/diffusion-number stability check before iteration, in-loop divergence monitoring, and the Ghia et al. (1982) benchmark comparison. It is not a 5 because there is no fix-and-retry feedback loop telling Claude what to do when validation fails (e.g., how to adapt dt or grid when the divergence check fails). | 4 / 5 |
Progressive Disclosure | The body has clear section headers but is a single ~220-line monolith with three large inline code blocks and no references to any separate files (no references/, scripts/, or assets/ exist in the bundle). This matches 'some structure but could be better organized; content that should be separate is inline' rather than the level 4 anchor, where most bulk content would live in appropriately placed separate files. | 3 / 5 |
Total | 13 / 20 Passed |