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.
A highly actionable, well-sequenced skill body: executable commands everywhere, phased workflow, and a rigorous validate-and-reject QA gate system with error recovery. Its weaknesses are mild redundancy around cachedValue staleness and title-width rules, and a monolithic inline layout where the pitfalls table and QA gate scripts would be better split into reference files.
Suggestions
Move the 17-row Known Issues & Pitfalls table (D-1..D-17) into a references/pitfalls.md file, keeping only the 3-4 most frequent traps inline with a one-line pointer per the existing '→ see officecli-xlsx §' pointer style.
Extract the Gate 1-8 bash scripts into a references/qa-gates.md (or scripts/run-gates.sh) and keep in SKILL.md only the gate list, what each verifies, and the COUNT-then-if pattern.
De-duplicate the cachedValue-staleness rule (Requirements bullet, Gate 8 intro, D-16) into a single authoritative statement plus a cross-reference, trimming roughly 15-20 lines.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with CLI-specific knowledge Claude would not know (DeferredAddKeys, raw-set activeTab ordering, SUMPRODUCT array-formula failures) and teaches no generic Excel concepts, so it broadly assumes competence. Minor over-explanation remains: the cachedValue-staleness rule is explained three times (Requirements, Gate 8 intro, D-16) and the chart-title-width rule appears in Design Ideas, Gate 2, and its comment, so it sits just below the lean anchor 5. | 4 / 5 |
Actionability | Fully executable throughout: a phased Quick Start with copy-paste bash, concrete prop values ('--prop highPoint=true --prop highMarkerColor=FF0000'), worked arithmetic ('width ≈ ceil((visible_chars + 2) × 1.3)'), and complete gate scripts including the col_num helper. Common cases (revenue trend dashboard, print delivery, CF rule types) are all covered by specific commands. | 5 / 5 |
Workflow Clarity | Clear sequencing with explicit validation: Quick Start is broken into phases 'so a single failed phase is obvious', Reference states the build order ('raw-set activeTab LAST after all sheets exist'), and the QA section is a REQUIRED delivery gate with eight count-then-if checks, per-failure REJECT messages, an error-recovery loop ('If anything fails, fix at source, re-run the full cycle'), and mandatory fallbacks when view html is blocked. This matches the anchor-5 validate→fix→retry pattern. | 5 / 5 |
Progressive Disclosure | Structure and navigation are good (clear sections, explicit deferral of schemas to 'officecli help' and hard rules to the officecli-xlsx parent skill), but the ~400-line body inlines substantial material that clearly belongs in separate reference files — the 17-row Known Issues table and ~120 lines of copy-paste gate scripts. With no bundle files at all, this is 'some structure, content that should be separate is inline', not above the anchor-3 line; not below 3 because what external deferral exists is clearly signaled and one level deep. | 3 / 5 |
Total | 17 / 20 Passed |