Content
81%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 well-engineered operational skill: precise commands, explicit gating, verification, and error recovery, with lean domain-specific guidance throughout. The only real costs are verbatim repetition of the root-guard preamble across code blocks, a templated PublishXml body, and reliance on plugin-root files outside the evaluated bundle. These are minor and do not impede correct execution.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational with no explanations of concepts Claude already knows — Dataverse-specific behaviors (metadata serialization lock, MSCRM.MergeLabels, PublishXml vs PublishAllXml) are genuinely non-obvious domain knowledge. Minor over-explanation/trimmable repetition remains: the `cd -- '<working_dir>' || { echo "BLOCKED…"; exit 1; }` guard is repeated verbatim in all six code blocks, and NEEDS_CONTEXT rules are restated across sections — anchor 4, not 5 ('every token earns its place'). | 4 / 5 |
Actionability | Concrete, near copy-paste bash commands with exact API paths, helper scripts, flags, tenant arguments, and expected response codes (204/no-op/403/429) cover the common cases. The one notable gap keeps it below anchor 5: the targeted PublishXml example embeds a `<table>` template placeholder inside the ParameterXml JSON body and instructs manual substitution, so it is a template rather than a fully executable example. | 4 / 5 |
Workflow Clarity | A seven-step sequence is stated up front, then elaborated with explicit checkpoints: auth/environment match (Step 1), an approval gate comparing the status table to the approved delta, a hard IsCustomizable rule, sequential PUT discipline with a concurrency warning, re-GET verification with BLOCKED on mismatch (Step 6), and error-recovery paths (401 refresh, 429 back-off, 0x80071151 fallback) plus a closing status-code contract — matching anchor 5's 'explicit validation steps; feedback loops for error recovery'. | 5 / 5 |
Progressive Disclosure | The file opens with a clearly signaled, one-level-deep reference list (shared-instructions.md 'read first', offline-profile-schema.md with its purpose) and the body splits core workflow from helpers appropriately. It stays at anchor 4 rather than 5 because the referenced files and scripts live under ${PLUGIN_ROOT} and none are present in this skill's bundle to verify, and the long inline root-binding/invocation-context section arguably belongs in the shared instructions it links to. | 4 / 5 |
Total | 17 / 20 Passed |