Content
77%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 strong, highly actionable skill body with an exemplary verification-gated workflow, weakened by redundant provenance/privacy passages and by a promised references/pi-data-format.md that is missing from the bundle while its would-be content sits inline. Tightening the duplicates and either shipping the reference file or removing the pointer would lift it substantially.
Suggestions
Merge the "Add provenance markers" bullets with the "**Mark provenance**" block — they state the same ^[inferred]/^[ambiguous] rules twice within Step 4; keep one authoritative list.
Create references/pi-data-format.md (moving the entry-type table and message-role schemas there) or delete the dangling "See references/pi-data-format.md" pointer; as written the reference does not exist in the bundle.
Fold the "Privacy and Compliance" section into the "Critical privacy filter" in Step 2 (or vice versa) — both cover redaction of secrets and sensitive identifiers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly dense, high-signal instruction with no tutorial padding, but it repeats itself: the provenance marker rules (^[inferred], ^[ambiguous]) are stated in full in both the "Add provenance markers" bullets and the following "**Mark provenance**" block, and privacy guidance appears in both "Critical privacy filter" and "Privacy and Compliance". It fits the 'mostly efficient but could be tightened' anchor rather than the 4-anchor, where redundancy would be only minor. | 3 / 5 |
Actionability | Guidance is copy-paste executable throughout: `find ~/.pi/agent/sessions -name "*.jsonl"`, an entry-type ingest table, the exact `pi:<file>#<entry-id>` reference format, manifest field lists, the locked `obsidian-wiki memory sync PI_HISTORY_INGEST …` command, and the full `qmd update`/`embed`/`ls` sequence with fallback handling. Specific examples cover the common cases, matching the fully-executable anchor. | 5 / 5 |
Workflow Clarity | Steps 1–5 are clearly sequenced with explicit validation checkpoints appropriate to a batch ingest: a delta summary reported before deep parsing, an evidence ledger built before writing, a five-point source verification gate with explicit failure handling (delete the claim or re-mark it), and QMD failure isolation ("do not roll back… report the QMD status separately"). This is the checklist-plus-feedback-loop anchor, not the 4-anchor with minor validation gaps. | 5 / 5 |
Progressive Disclosure | Sections are well organized and the one reference is clearly signaled, but `references/pi-data-format.md` does not exist in the bundle (there is no references/ directory), so the pointer dangles. Meanwhile field-level format detail (the entry-type table and message-role schemas) that the rubric expects in a separate file is inlined. This matches the 'some structure; content that should be separate is inline' anchor; it is not the 1–2 anchors since navigation is not broken and structure is genuinely good, and not 4–5 because the split is not actually realized. | 3 / 5 |
Total | 16 / 20 Passed |