Content
50%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-structured and provides concrete, mostly-complete examples, but it falls short of fully executable guidance (undefined helpers, an API inconsistent with its own bundle), repeats value-range guidance, omits validation checkpoints, and never navigates to its existing reference files. It is a competent reference that needs tightening and bundle integration.
Suggestions
Make the code examples copy-paste ready by defining `BASE` and `get_headers()` (and the `requests` import) once, and reconcile the body's `camera_control`/`type`/`config` API with the different `camera_motion`/`camera_intensity` API in the reference bundle files so the skill is internally consistent.
Consolidate the redundant value-range guidance (inline config comments, Shot Type Quick Reference, and Intensity Guidelines overlap) and move scattered model/version numbers into a single reference or deprecated section to reduce tokens.
Add explicit validation checkpoints (e.g., verify only one config field is non-zero before posting, and check the response) and link the bundle files from the body (e.g., 'See references/cinematic-presets.md for presets') so the existing references are discoverable rather than orphaned.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean and reference-style (no padded explanations of concepts Claude knows), but value-range guidance is repeated three ways (inline config comments, the Shot Type Quick Reference table, and the Intensity Guidelines table) and scattered model/version numbers ('kling-v2-6', 'v1.6+') are inline rather than in a deprecated section, so it could be tightened. | 2 / 3 |
Actionability | The five code examples have complete, realistic request bodies, but they are not copy-paste ready: `BASE` and `get_headers()` are undefined and `requests` is not imported, and the body's `camera_control`/`type`/`config` API is inconsistent with the `camera_motion`/`camera_intensity` API used in the reference bundle files, leaving key details missing. | 2 / 3 |
Workflow Clarity | The single-action flow (pick a shot, set one config field non-zero, POST) is present and the 'only ONE field non-zero' rule plus the error table give some guidance, but there is no explicit validation checkpoint before/after the call and no feedback loop, and the async result-retrieval step is absent, matching 'checkpoints missing or implicit'. | 2 / 3 |
Progressive Disclosure | The body is internally well-organized with clear sections, but the five reference bundle files are never linked or signaled from the body (the Resources section points only to external docs), and stubs like examples.md and the duplicate errors.md are orphaned, so references are present but not clearly signaled. | 2 / 3 |
Total | 8 / 12 Passed |