Content
71%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.
The body is a dense, accurate, and highly actionable reference with excellent practical warnings in Best Practices and Common Pitfalls, and nearly all code is executable. Its main weaknesses are the absence of any progressive-disclosure structure (everything inlined in a ~370-line SKILL.md) and a few fragmentary code examples that leave the reader mid-implementation.
Suggestions
Move the exhaustive Virtualizer Options table, Virtualizer API, and VirtualItem interface into a references/api.md file, keeping SKILL.md to the core pattern, dynamic sizing, and pointers.
Complete the fragmentary examples: finish the Infinite Scrolling render block (including the loader row) and give the Sticky Items section a real render snippet instead of a trailing comment.
Add per-framework entry points (@tanstack/vue-virtual, @tanstack/solid-virtual, etc.) or a short note on where to find them, since the description promises six frameworks but every example is React-only.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is code-first and dense with no explanations of concepts Claude already knows (no "what is virtualization" padding), and inline comments carry high-value warnings like "// REQUIRED for measurement" and "// Do NOT set fixed height". It is not 5 because the exhaustive 17-row options table and full Virtualizer API listing are bulk reference material that pushes the file past a lean token budget and could be trimmed or offloaded. | 4 / 5 |
Actionability | The Core Pattern, Dynamic Heights, Grid, Window Scrolling, and Sticky sections contain complete copy-paste-ready TSX with all required options shown. It is not 5 because a few snippets are fragments: the Infinite Scrolling example ends at a comment ("// Render virtual items, show loader row..."), the Smooth Scrolling example references an undeclared parentRef, and the Sticky section's rendering is only a comment. | 4 / 5 |
Workflow Clarity | The Core Pattern is an unambiguous single recipe and the Best Practices / Common Pitfalls sections act as an error-avoidance checklist (e.g. "Forgetting to subtract scrollMargin from translateY in window scrolling"). It is not 5 because there are no explicit validation checkpoints or feedback loops for the trickier multi-step setups (dynamic measurement, infinite scroll fetch triggering), and not 3 because steps that do exist are concrete and correctly ordered rather than implicit. | 4 / 5 |
Progressive Disclosure | Sections are well-labeled and logically ordered, but no references/ bundle exists and roughly 200 lines of API material (the exhaustive options table, Virtualizer API, and VirtualItem interface) that clearly belongs in a separate reference file is inlined in SKILL.md. This matches anchor 3 'some structure but could be better organized; content that should be separate is inline', and does not qualify for the under-50-lines simple-skill exception. | 3 / 5 |
Total | 15 / 20 Passed |