Content
85%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.
An exceptionally actionable, project-specific reference with complete executable examples, clear guardrails, and a consolidating checklist — but it is a monolithic single-file skill. Splitting the field catalog, auto-save details, and mutation-typing rules into bundled reference files would cut context-window cost without losing usability.
Suggestions
Move the field-component catalog (Input through BaseField), the auto-save sections, and the mutation-typing rules into references/ files (e.g., references/fields.md, references/autosave.md, references/mutations.md), keeping SKILL.md as a concise overview with one-level-deep, clearly signaled links.
Deduplicate the reset-after-save guidance (currently both 'Resetting After Save' and 'Form Reset After Save' sections) and trim the confirm-prop examples from five to the two most illustrative cases.
Replace the '../../../static/app/components/core/form/fields.mdx#checkbox-group' link with a bundled reference file (or inline the checkbox-group example), since the relative repo path is not resolvable from the skill bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly all content is project-specific API knowledge Claude cannot know (useScrapsForm, defaultFormOptions semantics, FieldErrors contract, auto-save triggers per field type), so tokens mostly earn their place. It sits at score 4 rather than 5 because of noticeable duplication: form-reset-after-save guidance appears twice as full sections ('Resetting After Save' and 'Form Reset After Save'), defaultFormOptions importance is explained three times, and the confirm-prop section gives five overlapping examples that could be trimmed. It is well above the score-3 anchor since padding is the exception, not the rule. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready TSX examples cover the common cases end to end: complete useScrapsForm usage, all nine field types (input, number, select single/multiple, switch, textarea, range, radio, custom BaseField), layouts, Zod schema patterns including the nullable-refine z.input/z.output flow, server error mapping with setFieldErrors, AutoSaveForm with mutationOptions, and confirm dialogs. Paired ❌/✅ do-and-don't pairs make the correct path unambiguous. | 5 / 5 |
Workflow Clarity | The 'Quick Reference Checklist' provides an explicit ordered sequence for both form creation and auto-save fields, and the surrounding sections supply the validation checkpoints it references: schema validation via validators:{onDynamic}, error recovery via narrowing to RequestError and setFieldErrors, conditional field-level validators, and reset-after-save. The feedback loops (validate → show field errors, mutate → setFieldErrors, save → form.reset) match the score-5 anchor including its checklist criterion. | 5 / 5 |
Progressive Disclosure | No bundle files (references/, scripts/, assets/) exist, and this ~1,100-line SKILL.md inlines everything: a full field-component catalog, layout reference, auto-save behavior tables, and an extensive do's-and-don'ts section — material that belongs in one-level-deep reference files per the rubric rationale. Internal section structure and the closing File References table are good, which keeps it above the score-2 'minimal structure' anchor, but it matches score 3's 'content that should be separate is inline'. The one delegation attempt — a relative link to '../../../static/app/components/core/form/fields.mdx#checkbox-group' — points outside the skill bundle to a path that cannot be verified and is unlikely to resolve from the skill's directory. | 3 / 5 |
Total | 17 / 20 Passed |