solid-reviewer
Trigger
Use this skill for PR/diff audits where findings must be prioritized and mapped to concrete remediation.
Required Inputs
- Diff or changed files.
- Runtime context (client, SSR, SolidStart server usage if applicable).
- Known risk constraints (performance budget, accessibility requirements, release urgency).
Workflow
- Triage by severity in strict order: correctness, performance, maintainability, accessibility.
- Validate reactive boundaries first: derivations vs effects, dependency explosion, stale closures.
- Evaluate control flow and async completeness (
loading,empty,error,success). - Check SSR/hydration and browser-only assumptions in render paths.
- Ensure every finding includes file reference and concrete fix direction.
- Return validation commands relevant to touched surface.
Failure Modes
- Missing diff context: fail and request files or patch.
- Findings without reproducible impact: downgrade to question or remove.
- Suggestions without file-level anchor: invalid until anchored.
- No findings detected: explicitly state "No findings" and list residual test gaps.
Output Contract
Return output matching ReviewOutput schema at ../../skills/contracts/review-output.schema.json with:
summary,findings,open_questions,validation_commands.findingsordered by severity and containingfile(andlinewhen available).citations: every standards claim must include normalizeddoc_id+claim.
Validation
node tools/scripts/validate-skills.mjs --skill solid-reviewernode tools/scripts/validate-output-contracts.mjs
References
../../references/solidjs-normalized/manifest.jsonl../../references/solidjs-normalized/taxonomy.json../../references/solidjs/review-checklist.md../../references/solidjs/control-flow.md../../references/solidjs/performance-ssr.md
Converted and distributed by TomeVault — claim your Tome and manage your conversions.