Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable and well-sequenced with strong validation checkpoints across three repos, but it is long and repeats the OAuth/DynamicSourceSetup guidance several times. Splitting the context-mill and created_via sections into reference files would reduce token cost and improve organization.
Suggestions
De-duplicate the DynamicSourceSetup / OAuth guidance: state the 'route the switch case to DynamicSourceSetup, no bespoke form' rule once and remove the restatements in the 'OAuth plumbing' and 'Setup-form specifics' sections.
Move the context-mill wizard surface and the created_via attribution table into one-level-deep reference files (e.g. references/wizard-surface.md and references/created-via.md) to shrink the main SKILL.md overview.
The posthog/code and posthog/posthog per-source checklists are the core reusable material — consider extracting them to reference files that the overview points to, keeping SKILL.md as a concise routing + deploy-ordering guide.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~250-line body is mostly efficient and assumes Claude's competence (no 'what is a warehouse' preamble), but repeats the 'route to DynamicSourceSetup / no bespoke OAuth form' guidance across the setup-form, OAuth-plumbing, and Setup-form-specifics sections, which could be tightened; matching the score-2 anchor of mostly efficient but with some unnecessary restatement. | 2 / 3 |
Actionability | Provides copy-paste-ready, executable guidance — exact paths and symbols ('packages/shared/src/inbox-types.ts — add "jira" to the SourceProduct union'), commands ('git worktree add ...', 'python manage.py makemigrations signals', 'pnpm typecheck', 'biome lint packages/core packages/ui', 'node scripts/build.js') — matching the score-3 anchor for fully executable, specific examples. | 3 / 3 |
Workflow Clarity | The multi-repo process is clearly sequenced with explicit deploy ordering ('land the posthog/posthog PR(s) first ... then the posthog/code PR'), per-surface numbered checklists, and Verify checkpoints after each phase (makemigrations --check clean, pnpm typecheck, biome lint zero violations, build validates structure), matching the score-3 anchor for clear sequence with explicit validation steps and checklists. | 3 / 3 |
Progressive Disclosure | No bundle files exist and the skill is monolithic at ~250 lines; the context-mill wizard surface and the created_via attribution table are inline content that would be better split into one-level-deep reference files, matching the score-2 anchor of 'content that should be separate is inline' despite otherwise clear headers and tables. | 2 / 3 |
Total | 10 / 12 Passed |