Content
42%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The skill provides highly actionable, executable code examples covering the full breadth of ArcGIS popup template configuration. However, it is far too verbose for a SKILL.md file—it reads as comprehensive API documentation rather than a focused skill guide. The content desperately needs progressive disclosure, splitting detailed references (chart types, date formats, layer-specific patterns) into separate files while keeping SKILL.md as a concise overview with the most common patterns.
Suggestions
Extract detailed content type references (all chart types, date formats, FieldInfo options, layer-specific examples) into separate reference files like CONTENT_TYPES.md, CHARTS.md, and EXPRESSIONS.md, keeping only the overview table and one basic example per type in SKILL.md.
Add a quick-start workflow section at the top showing the minimal steps to create a popup: 1) define outFields, 2) set title with field substitution, 3) add content array, with a note about the outFields pitfall inline.
Remove redundant examples—e.g., the individual bar/pie/line/column chart sections repeat the same pattern with different type strings; a single example with a comment listing valid types would suffice.
Add a validation/debugging section or checklist: verify field name casing matches layer schema, confirm outFields includes all referenced fields, test expression syntax in Arcade playground.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is extremely verbose at ~500+ lines, serving as an exhaustive API reference rather than a focused skill. It includes many variations and patterns (every chart type individually, multiple attachment display types, TypeScript usage, clustering popups, etc.) that could be in separate reference files. Much of this is documentation Claude could infer from the ArcGIS API. | 1 / 3 |
Actionability | All code examples are concrete, executable JavaScript/TypeScript snippets that are copy-paste ready. Field formats, chart configurations, action handlers, and expression definitions are all fully specified with real property names and values. | 3 / 3 |
Workflow Clarity | The content is organized as a reference catalog of content types rather than a workflow. There's no clear sequence for building a popup template from scratch, no validation steps (e.g., verifying outFields match content references), and no guidance on debugging common issues beyond the pitfalls list at the end. | 2 / 3 |
Progressive Disclosure | This is a monolithic wall of content with everything inline. The exhaustive chart type examples, date format lists, FieldInfo configurations, and layer-specific patterns should be split into separate reference files. The single cross-reference to arcgis-core-maps is good but insufficient given the volume of content that should be externalized. | 1 / 3 |
Total | 7 / 12 Passed |