Content
88%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 an actionable, well-sequenced runbook with real validation checkpoints and error-recovery loops, at minimal token cost. Its only weaknesses are the unpinned-into-a-section version check, the dangling referenced bundle paths in this bundle, and the indirect npm-root pointer for canonical references.
Suggestions
Move the version pin (R=0.20.0) into the referenced CLI skill's own versioning or a dedicated compatibility section, so the body stays free of time-sensitive constants.
Bundle the referenced paths (scripts/is-hybrid-app.sh, flows/README.md) or inline the minimal selector/recording rules they hold, so navigation doesn't depend on an npm-installed directory that may not match the local bundle.
Consolidate the repeated "fresh snapshot before/after acting" directives in steps 8-9 into one interaction-safety checklist to trim the duplicated guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes competence ("If `metro prepare` fails, **STOP** and surface the error verbatim"), but the pre-flight embeds a pinned version ("R=0.20.0") and dense inline shell, and a few directives are repeated across the safety section — mostly anchor 4 with minor trims available. | 4 / 5 |
Actionability | Every step gives a copy-paste-ready command ("agent-device open <bundle-id> --platform <p> --device \"<name>\"", "agent-device react-native dismiss-overlay") plus a bundle-ID/build-command table, covering the common bring-up and interaction cases. | 5 / 5 |
Workflow Clarity | The bring-up is a numbered sequence with explicit validation gates ("If any line shows `FAIL`, stop and surface the fix", "If the resolved bundle ID is missing from the list, **STOP**") and a retry feedback loop (dismiss overlay -> fresh snapshot -> retry via selector), matching the top anchor. | 5 / 5 |
Progressive Disclosure | References are one level deep and clearly signaled ("[Agent decision loop](flows/README.md)", "Do not add measurement flows here"), but the referenced paths (flows/README.md, scripts/is-hybrid-app.sh) are not present in the bundle and the "Canonical skill references" section routes through an npm-installed directory, adding an indirection hop. | 4 / 5 |
Total | 18 / 20 Passed |