Content
86%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.
Excellent actionability and token efficiency: nearly every line carries non-obvious internal-framework knowledge delivered as executable commands, exact signatures, and failure-mode-specific fixes. The weaknesses are structural — a 230-line single-file layout that should split the cheat-sheet and gotchas into reference files, and a Step 4 that promises verification but only documents invocation.
Suggestions
Move the composite-ops cheat-sheet table (section 2) and the architecture gotchas (section 3) into one-level-deep reference files (e.g. references/composites.md and references/gotchas.md), keeping SKILL.md as a concise workflow overview with clearly signaled links.
Add explicit post-export verification steps to Step 4 — the heading says "Full Export & litert_lm Verification" but the bullets only cover how to invoke the export; state what to check on the exported artifact and how.
Add a short error-recovery loop in Step 3 (e.g. "if assert_v2_weights_resolvable fails, check gotcha F items 1–3 and re-run") so the fast verification step explicitly routes failures to their fixes.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with internal-API specifics Claude cannot know (exact call signatures like "requires 4 positional arguments (note the 4th ones tensor)", flag couplings, failure modes) and explains no generally known concepts. Every section earns its tokens through non-obvious detail delivered in tight bullets, tables, and commands, matching the "lean and efficient... every token earns its place" anchor. | 5 / 5 |
Actionability | Commands are copy-paste ready with real paths and flags (bazel run inspector, litert convert with --extension-modules, --target=npu:qualcomm), the cheat-sheet gives exact import paths and call signatures, and code snippets like the _find_modeling_module loop and attn_output reshape are executable. This matches the fully-executable, common-cases-covered anchor. | 5 / 5 |
Workflow Clarity | The 4-step workflow is clearly sequenced with strong explicit validation in Step 3 (three named testing.assert_* checks plus dry_run, with timing budgets like "<2s on CPU") and error-recovery detail in the gotchas. However, Step 4 is titled "Full Export & litert_lm Verification" yet its bullets only show export invocation commands and never state how to verify the exported artifact — a minor validation gap that fits anchor 4 rather than the explicit-checkpoint-everywhere anchor 5. | 4 / 5 |
Progressive Disclosure | The body has good internal structure (numbered sections, headers, a signature table), but it is a single ~230-line monolith with no bundle files at all: the composite-ops cheat-sheet and the six architecture gotchas are exactly the kind of deep reference content that belongs in separate one-level-deep files. This matches anchor 3 ("content that should be separate is inline"), not anchor 4's "bulk in separate file". | 3 / 5 |
Total | 17 / 20 Passed |