Content
70%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 content is a highly actionable, well-sequenced closed-loop debugging workflow with strong validation and feedback loops. Its main weaknesses are repetition of the element-not-found guidance across several sections and a lack of progressive disclosure into separate reference files.
Suggestions
Consolidate the repeated 'Element Not Found' guidance into a single triage block and have the pattern table, fix section, and quick reference point to it rather than restating it.
Split the pattern/fix reference tables and the Phase 7 report template into a references/ file (e.g. FIX-PATTERNS.md) and link to it, reducing the inline wall of text.
Fill or document the <SCHEME_NAME>/<TARGET> placeholders with a concrete example value to push actionability toward fully copy-paste-ready.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and free of concept-explanation fluff, but the 'Element Not Found' material is repeated across the failure-pattern table, a dedicated triage subsection, the Phase 4 fix, the Error Quick Reference, and the decision tree, and the full Phase 7 report template adds length that could be tightened. | 3 / 5 |
Actionability | Executable bash (xcodebuild, xcrun xcresulttool, xcsym) and complete BEFORE/AFTER Swift fix examples cover the common cases, but parameterization placeholders like <SCHEME_NAME>UITests and <path-to-ips>, plus a couple of high-level steps ('Read the test code', 'Search the app source'), leave minor gaps short of fully copy-paste-ready. | 4 / 5 |
Workflow Clarity | A clear closed-loop sequence (RUN -> CAPTURE -> ANALYZE -> SUGGEST -> FIX -> VERIFY -> REPORT) is laid out in numbered phases with explicit validation (Phase 6 re-run), a feedback loop for error recovery, a decision tree, and a 'max 3 iterations then escalate' checkpoint. | 5 / 5 |
Progressive Disclosure | The ~320-line body is well sectioned but entirely inline with no bundle files (references/, scripts/, assets/ are absent); pattern tables and fix recipes that could live in separate reference files are monolithic, and the only signaled reference (skills/xcsym-ref.md) is an external skill rather than this skill's own bundle. | 3 / 5 |
Total | 15 / 20 Passed |