@event4u/agent-config 1.32.0 → 1.34.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.agent-src/commands/research/deep.md +149 -0
- package/.agent-src/commands/research/report.md +134 -0
- package/.agent-src/commands/research.md +43 -13
- package/.agent-src/commands/review-changes.md +13 -8
- package/.agent-src/personas/README.md +12 -21
- package/.agent-src/personas/_template-specialist/persona.md +89 -0
- package/.agent-src/personas/backend-architect.md +96 -0
- package/.agent-src/personas/eloquent-tamer.md +96 -0
- package/.agent-src/personas/frontend-engineer.md +100 -0
- package/.agent-src/personas/qa.md +27 -2
- package/.agent-src/personas/security-engineer.md +100 -0
- package/.agent-src/skills/accessibility-auditor/SKILL.md +132 -0
- package/.agent-src/skills/adr-create/SKILL.md +1 -0
- package/.agent-src/skills/adversarial-review/SKILL.md +1 -0
- package/.agent-src/skills/agent-docs-writing/SKILL.md +1 -0
- package/.agent-src/skills/agents-md-thin-root/SKILL.md +1 -0
- package/.agent-src/skills/ai-council/SKILL.md +1 -0
- package/.agent-src/skills/analysis-autonomous-mode/SKILL.md +1 -0
- package/.agent-src/skills/analysis-skill-router/SKILL.md +1 -0
- package/.agent-src/skills/api-design/SKILL.md +3 -0
- package/.agent-src/skills/api-endpoint/SKILL.md +1 -0
- package/.agent-src/skills/api-testing/SKILL.md +1 -0
- package/.agent-src/skills/architecture-review-lens/SKILL.md +137 -0
- package/.agent-src/skills/artisan-commands/SKILL.md +1 -0
- package/.agent-src/skills/async-python-patterns/SKILL.md +1 -0
- package/.agent-src/skills/authz-review/SKILL.md +4 -0
- package/.agent-src/skills/aws-infrastructure/SKILL.md +1 -0
- package/.agent-src/skills/blade-ui/SKILL.md +1 -0
- package/.agent-src/skills/blast-radius-analyzer/SKILL.md +3 -0
- package/.agent-src/skills/bug-analyzer/SKILL.md +1 -0
- package/.agent-src/skills/check-refs/SKILL.md +1 -0
- package/.agent-src/skills/code-refactoring/SKILL.md +1 -0
- package/.agent-src/skills/code-review/SKILL.md +1 -0
- package/.agent-src/skills/command-routing/SKILL.md +1 -0
- package/.agent-src/skills/command-writing/SKILL.md +1 -0
- package/.agent-src/skills/composer-packages/SKILL.md +1 -0
- package/.agent-src/skills/context-authoring/SKILL.md +1 -0
- package/.agent-src/skills/context-document/SKILL.md +1 -0
- package/.agent-src/skills/conventional-commits-writing/SKILL.md +1 -0
- package/.agent-src/skills/copilot-agents-optimization/SKILL.md +1 -0
- package/.agent-src/skills/copilot-config/SKILL.md +1 -0
- package/.agent-src/skills/dashboard-design/SKILL.md +1 -0
- package/.agent-src/skills/data-flow-mapper/SKILL.md +1 -0
- package/.agent-src/skills/database/SKILL.md +3 -0
- package/.agent-src/skills/dcf-modeling/SKILL.md +1 -0
- package/.agent-src/skills/decision-record/SKILL.md +143 -0
- package/.agent-src/skills/deep-reading-analyst/SKILL.md +1 -0
- package/.agent-src/skills/defense-in-depth/SKILL.md +1 -0
- package/.agent-src/skills/dependency-upgrade/SKILL.md +1 -0
- package/.agent-src/skills/description-assist/SKILL.md +1 -0
- package/.agent-src/skills/design-review/SKILL.md +1 -0
- package/.agent-src/skills/devcontainer/SKILL.md +1 -0
- package/.agent-src/skills/developer-like-execution/SKILL.md +1 -0
- package/.agent-src/skills/docker/SKILL.md +1 -0
- package/.agent-src/skills/dto-creator/SKILL.md +1 -0
- package/.agent-src/skills/eloquent/SKILL.md +3 -0
- package/.agent-src/skills/error-handling-patterns/SKILL.md +1 -0
- package/.agent-src/skills/estimate-ticket/SKILL.md +1 -0
- package/.agent-src/skills/existing-ui-audit/SKILL.md +3 -0
- package/.agent-src/skills/fe-design/SKILL.md +4 -1
- package/.agent-src/skills/feature-planning/SKILL.md +1 -0
- package/.agent-src/skills/file-editor/SKILL.md +1 -0
- package/.agent-src/skills/finishing-a-development-branch/SKILL.md +1 -0
- package/.agent-src/skills/flux/SKILL.md +1 -0
- package/.agent-src/skills/form-handler/SKILL.md +145 -0
- package/.agent-src/skills/funnel-analysis/SKILL.md +1 -0
- package/.agent-src/skills/git-workflow/SKILL.md +1 -0
- package/.agent-src/skills/github-ci/SKILL.md +1 -0
- package/.agent-src/skills/grafana/SKILL.md +1 -0
- package/.agent-src/skills/guideline-writing/SKILL.md +1 -0
- package/.agent-src/skills/incident-commander/SKILL.md +140 -0
- package/.agent-src/skills/jira-integration/SKILL.md +1 -0
- package/.agent-src/skills/jobs-events/SKILL.md +1 -0
- package/.agent-src/skills/judge-bug-hunter/SKILL.md +1 -0
- package/.agent-src/skills/judge-code-quality/SKILL.md +1 -0
- package/.agent-src/skills/judge-security-auditor/SKILL.md +3 -0
- package/.agent-src/skills/judge-test-coverage/SKILL.md +1 -0
- package/.agent-src/skills/laravel/SKILL.md +1 -0
- package/.agent-src/skills/laravel-horizon/SKILL.md +1 -0
- package/.agent-src/skills/laravel-mail/SKILL.md +1 -0
- package/.agent-src/skills/laravel-middleware/SKILL.md +1 -0
- package/.agent-src/skills/laravel-notifications/SKILL.md +1 -0
- package/.agent-src/skills/laravel-pennant/SKILL.md +1 -0
- package/.agent-src/skills/laravel-pulse/SKILL.md +1 -0
- package/.agent-src/skills/laravel-reverb/SKILL.md +1 -0
- package/.agent-src/skills/laravel-scheduling/SKILL.md +1 -0
- package/.agent-src/skills/laravel-validation/SKILL.md +1 -0
- package/.agent-src/skills/learning-to-rule-or-skill/SKILL.md +1 -0
- package/.agent-src/skills/lint-skills/SKILL.md +1 -0
- package/.agent-src/skills/livewire/SKILL.md +1 -0
- package/.agent-src/skills/livewire-architect/SKILL.md +158 -0
- package/.agent-src/skills/logging-monitoring/SKILL.md +1 -0
- package/.agent-src/skills/markitdown/SKILL.md +1 -0
- package/.agent-src/skills/mcp/SKILL.md +1 -0
- package/.agent-src/skills/mcp-builder/SKILL.md +1 -0
- package/.agent-src/skills/md-language-check/SKILL.md +1 -0
- package/.agent-src/skills/merge-conflicts/SKILL.md +1 -0
- package/.agent-src/skills/migration-architect/SKILL.md +119 -0
- package/.agent-src/skills/migration-creator/SKILL.md +1 -0
- package/.agent-src/skills/mobile-e2e-strategy/SKILL.md +2 -1
- package/.agent-src/skills/module-management/SKILL.md +1 -0
- package/.agent-src/skills/multi-tenancy/SKILL.md +1 -0
- package/.agent-src/skills/okr-tree-modeling/SKILL.md +1 -0
- package/.agent-src/skills/openapi/SKILL.md +1 -0
- package/.agent-src/skills/override-management/SKILL.md +1 -0
- package/.agent-src/skills/performance/SKILL.md +1 -0
- package/.agent-src/skills/performance-analysis/SKILL.md +1 -0
- package/.agent-src/skills/persona-writing/SKILL.md +1 -0
- package/.agent-src/skills/pest-testing/SKILL.md +1 -0
- package/.agent-src/skills/php-coder/SKILL.md +1 -0
- package/.agent-src/skills/php-debugging/SKILL.md +1 -0
- package/.agent-src/skills/php-service/SKILL.md +1 -0
- package/.agent-src/skills/playwright-architect/SKILL.md +141 -0
- package/.agent-src/skills/playwright-testing/SKILL.md +1 -0
- package/.agent-src/skills/po-discovery/SKILL.md +127 -0
- package/.agent-src/skills/project-analysis-core/SKILL.md +1 -0
- package/.agent-src/skills/project-analysis-hypothesis-driven/SKILL.md +1 -0
- package/.agent-src/skills/project-analysis-laravel/SKILL.md +1 -0
- package/.agent-src/skills/project-analysis-nextjs/SKILL.md +1 -0
- package/.agent-src/skills/project-analysis-node-express/SKILL.md +1 -0
- package/.agent-src/skills/project-analysis-react/SKILL.md +1 -0
- package/.agent-src/skills/project-analysis-symfony/SKILL.md +1 -0
- package/.agent-src/skills/project-analysis-zend-laminas/SKILL.md +1 -0
- package/.agent-src/skills/project-analyzer/SKILL.md +1 -0
- package/.agent-src/skills/project-docs/SKILL.md +1 -0
- package/.agent-src/skills/prompt-engineering-patterns/SKILL.md +1 -0
- package/.agent-src/skills/prompt-optimizer/SKILL.md +1 -0
- package/.agent-src/skills/quality-tools/SKILL.md +1 -0
- package/.agent-src/skills/react-native-setup/SKILL.md +1 -0
- package/.agent-src/skills/react-shadcn-ui/SKILL.md +1 -0
- package/.agent-src/skills/readme-reviewer/SKILL.md +1 -0
- package/.agent-src/skills/readme-writing/SKILL.md +1 -0
- package/.agent-src/skills/readme-writing-package/SKILL.md +1 -0
- package/.agent-src/skills/receiving-code-review/SKILL.md +1 -0
- package/.agent-src/skills/refine-prompt/SKILL.md +1 -0
- package/.agent-src/skills/refine-ticket/SKILL.md +1 -0
- package/.agent-src/skills/repomix-packer/SKILL.md +1 -0
- package/.agent-src/skills/requesting-code-review/SKILL.md +1 -0
- package/.agent-src/skills/review-routing/SKILL.md +1 -0
- package/.agent-src/skills/rice-prioritization/SKILL.md +1 -0
- package/.agent-src/skills/risk-officer/SKILL.md +141 -0
- package/.agent-src/skills/roadmap-management/SKILL.md +1 -0
- package/.agent-src/skills/roadmap-writing/SKILL.md +1 -0
- package/.agent-src/skills/rtk-output-filtering/SKILL.md +1 -0
- package/.agent-src/skills/rule-writing/SKILL.md +1 -0
- package/.agent-src/skills/script-writing/SKILL.md +1 -0
- package/.agent-src/skills/secrets-management/SKILL.md +1 -0
- package/.agent-src/skills/security/SKILL.md +1 -0
- package/.agent-src/skills/security-audit/SKILL.md +1 -0
- package/.agent-src/skills/sentry-integration/SKILL.md +1 -0
- package/.agent-src/skills/sequential-thinking/SKILL.md +1 -0
- package/.agent-src/skills/skill-improvement-pipeline/SKILL.md +1 -0
- package/.agent-src/skills/skill-management/SKILL.md +1 -0
- package/.agent-src/skills/skill-reviewer/SKILL.md +1 -0
- package/.agent-src/skills/skill-writing/SKILL.md +1 -0
- package/.agent-src/skills/sql-writing/SKILL.md +1 -0
- package/.agent-src/skills/stakeholder-tradeoff/SKILL.md +149 -0
- package/.agent-src/skills/subagent-orchestration/SKILL.md +13 -0
- package/.agent-src/skills/systematic-debugging/SKILL.md +1 -0
- package/.agent-src/skills/tailwind-engineer/SKILL.md +130 -0
- package/.agent-src/skills/tech-debt-tracker/SKILL.md +152 -0
- package/.agent-src/skills/technical-specification/SKILL.md +1 -0
- package/.agent-src/skills/terraform/SKILL.md +1 -0
- package/.agent-src/skills/terragrunt/SKILL.md +1 -0
- package/.agent-src/skills/test-driven-development/SKILL.md +1 -0
- package/.agent-src/skills/test-performance/SKILL.md +1 -0
- package/.agent-src/skills/testing-anti-patterns/SKILL.md +1 -0
- package/.agent-src/skills/threat-modeling/SKILL.md +3 -0
- package/.agent-src/skills/token-optimizer/SKILL.md +1 -0
- package/.agent-src/skills/traefik/SKILL.md +1 -0
- package/.agent-src/skills/ui-component-architect/SKILL.md +153 -0
- package/.agent-src/skills/unit-economics-modeling/SKILL.md +1 -0
- package/.agent-src/skills/universal-project-analysis/SKILL.md +1 -0
- package/.agent-src/skills/upstream-contribute/SKILL.md +1 -0
- package/.agent-src/skills/using-git-worktrees/SKILL.md +1 -0
- package/.agent-src/skills/validate-feature-fit/SKILL.md +1 -0
- package/.agent-src/skills/verify-completion-evidence/SKILL.md +1 -0
- package/.agent-src/skills/websocket/SKILL.md +1 -0
- package/.claude-plugin/marketplace.json +17 -1
- package/AGENTS.md +1 -0
- package/CHANGELOG.md +68 -0
- package/README.md +3 -3
- package/docs/architecture.md +3 -3
- package/docs/catalog.md +26 -5
- package/docs/contracts/command-clusters.md +1 -1
- package/docs/contracts/file-ownership-matrix.json +560 -0
- package/docs/contracts/persona-schema.md +136 -0
- package/docs/contracts/skill-domains.md +143 -0
- package/docs/decisions/ADR-005-subagent-worktrees.md +120 -0
- package/docs/decisions/ADR-006-skill-tools-python-pilot.md +114 -0
- package/docs/decisions/INDEX.md +3 -0
- package/docs/getting-started.md +1 -1
- package/docs/guidelines/agent-infra/5w2h-analysis.md +260 -0
- package/docs/guidelines/agent-infra/critical-thinking.md +156 -0
- package/docs/guidelines/agent-infra/first-principles.md +192 -0
- package/docs/guidelines/agent-infra/six-hats.md +353 -0
- package/docs/guidelines/agent-infra/systems-thinking.md +220 -0
- package/docs/personas.md +115 -0
- package/package.json +1 -1
- package/scripts/_backfill_skill_domains.py +140 -0
- package/scripts/_emit_domain_table.py +35 -0
- package/scripts/install-hooks.sh +21 -4
- package/scripts/lint_skill_tools.py +168 -0
- package/scripts/schemas/skill.schema.json +6 -1
- package/scripts/skill_linter.py +19 -4
- package/scripts/skill_tools/__init__.py +22 -0
- package/scripts/skill_tools/audit_persona_coverage.py +147 -0
- package/scripts/skill_tools/run_block_d_eval.py +129 -0
- package/scripts/skill_tools/score_skill_relevance.py +169 -0
- package/scripts/skill_tools/suggest_skill_for_task.py +113 -0
|
@@ -0,0 +1,137 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: architecture-review-lens
|
|
3
|
+
description: "Use when a diff may break system boundaries, dependency direction, or cross-service contracts — fifth judge dispatched by /review-changes alongside the four standard judges."
|
|
4
|
+
personas:
|
|
5
|
+
- backend-architect
|
|
6
|
+
- senior-engineer
|
|
7
|
+
source: package
|
|
8
|
+
domain: quality
|
|
9
|
+
---
|
|
10
|
+
|
|
11
|
+
# architecture-review-lens
|
|
12
|
+
|
|
13
|
+
> Fifth judge in the `/review-changes` family. Reviews a diff for
|
|
14
|
+
> **architectural fit**, not correctness, security, tests, or style.
|
|
15
|
+
> Catches what the other four miss: layer violations, wrong
|
|
16
|
+
> dependency direction, leaking abstractions, and broken cross-service
|
|
17
|
+
> contracts. Sibling of [`judge-bug-hunter`](../judge-bug-hunter/SKILL.md)
|
|
18
|
+
> et al. — never overlaps.
|
|
19
|
+
|
|
20
|
+
## When to use
|
|
21
|
+
|
|
22
|
+
- `/review-changes` dispatches its "architecture" slice to this skill.
|
|
23
|
+
- A reviewer asks "does this belong here?", "should this be in
|
|
24
|
+
the domain layer?", or "is this leaking storage details?".
|
|
25
|
+
- A diff adds a cross-service call, event, or contract.
|
|
26
|
+
|
|
27
|
+
Do NOT use when:
|
|
28
|
+
|
|
29
|
+
- The diff is documentation-only or a formatting-only change.
|
|
30
|
+
- The concern is correctness — route to
|
|
31
|
+
[`judge-bug-hunter`](../judge-bug-hunter/SKILL.md).
|
|
32
|
+
- The concern is security — route to
|
|
33
|
+
[`judge-security-auditor`](../judge-security-auditor/SKILL.md).
|
|
34
|
+
- The concern is naming or DRY — route to
|
|
35
|
+
[`judge-code-quality`](../judge-code-quality/SKILL.md).
|
|
36
|
+
- The concern is *whether to make* the architectural change — route
|
|
37
|
+
to [`decision-record`](../decision-record/SKILL.md) first.
|
|
38
|
+
|
|
39
|
+
## Procedure
|
|
40
|
+
|
|
41
|
+
### 1. Anchor on the system shape
|
|
42
|
+
|
|
43
|
+
Read the codebase's stated architecture (ADRs, AGENTS.md, module
|
|
44
|
+
docs). If no shape is documented, infer it from folder structure
|
|
45
|
+
and surface the gap. You are judging the diff against **the stated
|
|
46
|
+
shape**, not a fantasy ideal.
|
|
47
|
+
|
|
48
|
+
### 2. Inspect each changed file for fit
|
|
49
|
+
|
|
50
|
+
For every changed file, answer:
|
|
51
|
+
|
|
52
|
+
| Question | Smell when "no" |
|
|
53
|
+
|---|---|
|
|
54
|
+
| Does this file live in the right layer? | Layer violation |
|
|
55
|
+
| Are imports flowing in the allowed direction? | Inverted dependency |
|
|
56
|
+
| Does this leak a storage / framework detail? | Leaky abstraction |
|
|
57
|
+
| Is the public API of the module still the same? | Contract drift |
|
|
58
|
+
| Does a cross-service call respect its contract? | Contract break |
|
|
59
|
+
|
|
60
|
+
Each smell is a finding with a file:line citation.
|
|
61
|
+
|
|
62
|
+
### 3. Check the seams
|
|
63
|
+
|
|
64
|
+
Pay special attention to:
|
|
65
|
+
|
|
66
|
+
- New public methods on existing classes.
|
|
67
|
+
- New imports that cross module boundaries.
|
|
68
|
+
- New events, queue messages, or HTTP calls.
|
|
69
|
+
- Removed deprecation warnings or feature flags.
|
|
70
|
+
|
|
71
|
+
Cross-module / cross-service additions are the highest-leverage
|
|
72
|
+
findings — surface them even at low individual severity.
|
|
73
|
+
|
|
74
|
+
### 4. Issue a verdict per the judge contract
|
|
75
|
+
|
|
76
|
+
| Verdict | When |
|
|
77
|
+
|---|---|
|
|
78
|
+
| `apply` | No architectural concerns; diff fits the stated shape |
|
|
79
|
+
| `revise` | Findings exist; diff lands after the listed fixes |
|
|
80
|
+
| `reject` | Architectural shape itself must be reconsidered (rare) |
|
|
81
|
+
|
|
82
|
+
`reject` requires citing which ADR or stated shape would need to
|
|
83
|
+
change — never reject for taste.
|
|
84
|
+
|
|
85
|
+
### 5. Validate the verdict
|
|
86
|
+
|
|
87
|
+
Verify before emitting: every finding has a `file:line` citation and
|
|
88
|
+
a smell label from the taxonomy; the verdict matches the worst
|
|
89
|
+
finding (`revise` if any finding exists, `apply` only when none);
|
|
90
|
+
`reject` cites the ADR or stated shape that would need to change.
|
|
91
|
+
Ensure no finding restates a concern owned by another judge.
|
|
92
|
+
|
|
93
|
+
## Output format
|
|
94
|
+
|
|
95
|
+
The verdict block carries these ordered fields:
|
|
96
|
+
|
|
97
|
+
1. `Judge:` — fixed value `architecture-review-lens`
|
|
98
|
+
2. `Model:` and `Target:` — model id from `.agent-settings.yml` and diff range
|
|
99
|
+
3. `Verdict:` — exactly one of `apply` / `revise` / `reject`
|
|
100
|
+
4. `Issues:` — numbered list, each with `file:line`, smell label, suggested fix
|
|
101
|
+
|
|
102
|
+
```
|
|
103
|
+
Judge: architecture-review-lens
|
|
104
|
+
Model: <model id from .agent-settings.yml>
|
|
105
|
+
Target: <branch / diff range>
|
|
106
|
+
|
|
107
|
+
Verdict: apply | revise | reject
|
|
108
|
+
|
|
109
|
+
Issues:
|
|
110
|
+
1. 🔴 <finding> file:line
|
|
111
|
+
Smell: layer-violation | inverted-dep | leak | contract-drift | contract-break
|
|
112
|
+
Suggested fix: <one sentence>
|
|
113
|
+
2. 🟡 ...
|
|
114
|
+
3. 🟢 ...
|
|
115
|
+
```
|
|
116
|
+
|
|
117
|
+
## Gotcha
|
|
118
|
+
|
|
119
|
+
- The stated shape may be wrong. If the diff has a sound reason to
|
|
120
|
+
break it, do NOT raise a finding — recommend an ADR via
|
|
121
|
+
`decision-record` and approve the diff.
|
|
122
|
+
- "Could be split into more files" is style, not architecture; route
|
|
123
|
+
to `judge-code-quality`.
|
|
124
|
+
- A finding with no file:line citation is a vibe; reject your own
|
|
125
|
+
finding before issuing it.
|
|
126
|
+
|
|
127
|
+
## Do NOT
|
|
128
|
+
|
|
129
|
+
- Do NOT review correctness, security, tests, or style — those are
|
|
130
|
+
other judges.
|
|
131
|
+
- Do NOT issue `reject` without naming the ADR or stated shape that
|
|
132
|
+
would need to change.
|
|
133
|
+
- Do NOT raise findings against the *current* architecture if the
|
|
134
|
+
diff did not introduce them; this is a diff-judge, not a
|
|
135
|
+
codebase-audit.
|
|
136
|
+
- Do NOT merge findings with another judge's output — the user needs
|
|
137
|
+
to see which lens raised each one.
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: async-python-patterns
|
|
3
3
|
description: "Use when writing Python asyncio code — picking between gather / TaskGroup / wait, structured concurrency, timeouts, cancellation, sync-bridging — decision framework only, cookbook externalized."
|
|
4
4
|
source: package
|
|
5
|
+
domain: engineering
|
|
5
6
|
status: active
|
|
6
7
|
refresh_trigger: "Python ships a new structured-concurrency primitive (post-TaskGroup), OR ≥30% of cited upstream cookbook examples become deprecated, OR the cited libraries (aiohttp, httpx, anyio, trio) cut a major version with breaking async surface changes."
|
|
7
8
|
sunset_criterion: "When `https://docs.python.org/3/library/asyncio.html` ships an in-tree decision framework AND consumer projects no longer cite this skill in PR reviews for two consecutive review cycles."
|
|
@@ -1,7 +1,11 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: authz-review
|
|
3
3
|
description: "Use when reviewing authorization end-to-end — route → gate → policy → query scope → response filter — before changes to permissions, tenants, ownership, or admin flows."
|
|
4
|
+
personas:
|
|
5
|
+
- security-engineer
|
|
6
|
+
- backend-architect
|
|
4
7
|
source: package
|
|
8
|
+
domain: quality
|
|
5
9
|
---
|
|
6
10
|
|
|
7
11
|
# authz-review
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: aws-infrastructure
|
|
3
3
|
description: "Use when working with AWS resources — ECS Fargate, ECR, EFS, Secrets Manager, gomplate templates, multi-env deployments — even when the user says 'deploy to staging' without naming AWS."
|
|
4
4
|
source: package
|
|
5
|
+
domain: devops
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# aws-infrastructure
|
|
@@ -1,7 +1,10 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: blast-radius-analyzer
|
|
3
3
|
description: "Use BEFORE editing shared code — enumerates every call site, event consumer, queue worker, API client, migration, and test that a planned change will touch, with a file:line citation per dependency."
|
|
4
|
+
personas:
|
|
5
|
+
- backend-architect
|
|
4
6
|
source: package
|
|
7
|
+
domain: discovery
|
|
5
8
|
---
|
|
6
9
|
|
|
7
10
|
# blast-radius-analyzer
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: bug-analyzer
|
|
3
3
|
description: "Use when the user shares a Sentry error, Jira bug ticket, or error description and wants root cause analysis. Also for proactive bug hunting and code audits for hidden bugs."
|
|
4
4
|
source: package
|
|
5
|
+
domain: discovery
|
|
5
6
|
council_depth: deep
|
|
6
7
|
---
|
|
7
8
|
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: check-refs
|
|
3
3
|
description: "Use when verifying cross-references between skills, rules, commands, guidelines, and context documents are not broken after edits, renames, or deletions."
|
|
4
4
|
source: package
|
|
5
|
+
domain: process
|
|
5
6
|
execution:
|
|
6
7
|
type: assisted
|
|
7
8
|
handler: shell
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: code-refactoring
|
|
3
3
|
description: "Use when the user says "refactor this", "rename class", or "move method". Safely refactors PHP code — finds all callers, updates downstream dependencies, and verifies with quality tools."
|
|
4
4
|
source: package
|
|
5
|
+
domain: engineering
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# refactorer
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: command-routing
|
|
3
3
|
description: "Use when the user invokes a slash command like /create-pr, /commit, /fix-ci, or pastes command file content — routes to the right command with context inference and GitHub API patterns."
|
|
4
4
|
source: package
|
|
5
|
+
domain: process
|
|
5
6
|
execution:
|
|
6
7
|
type: assisted
|
|
7
8
|
handler: internal
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: command-writing
|
|
3
3
|
description: "Use when creating or editing a slash command in .agent-src.uncompressed/commands/ — frontmatter, numbered steps, safety gates — even when the user just says 'add a /command for X'."
|
|
4
4
|
source: package
|
|
5
|
+
domain: process
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
<!-- cloud_safe: degrade -->
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: composer-packages
|
|
3
3
|
description: "Use when building or maintaining a Composer library — versioning, Laravel integration, autoloading, publishing to private registries — even when the user says 'release a new version'."
|
|
4
4
|
source: package
|
|
5
|
+
domain: engineering
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# composer-packages
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: context-authoring
|
|
3
3
|
description: "Use when filling in knowledge-layer context files — auth-model, tenant-boundaries, data-sensitivity, deployment-order, observability — interactive walkthrough that turns templates into reviewer fuel."
|
|
4
4
|
source: package
|
|
5
|
+
domain: process
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# context-authoring
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: conventional-commits-writing
|
|
3
3
|
description: "Use when writing commit messages or squash-merge titles — `feat:`, `fix:`, `chore:`, scopes, breaking changes — even when the user just says 'commit this' without naming Conventional Commits."
|
|
4
4
|
source: package
|
|
5
|
+
domain: process
|
|
5
6
|
execution:
|
|
6
7
|
type: assisted
|
|
7
8
|
handler: internal
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: copilot-agents-optimization
|
|
3
3
|
description: "Use when optimizing AGENTS.md or copilot-instructions.md — deduplicates against .augment/ content, enforces line budgets, and focuses each file on its audience."
|
|
4
4
|
source: package
|
|
5
|
+
domain: process
|
|
5
6
|
execution:
|
|
6
7
|
type: assisted
|
|
7
8
|
handler: internal
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: copilot-config
|
|
3
3
|
description: "Use when configuring GitHub Copilot — copilot-instructions.md, PR review patterns, output optimization — even when the user just says 'tune Copilot' or 'why is Copilot commenting on X'."
|
|
4
4
|
source: package
|
|
5
|
+
domain: process
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# Copilot Skill
|
|
@@ -1,7 +1,10 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: database
|
|
3
3
|
description: "Use when working with database architecture, MariaDB/MySQL tuning, indexing strategies, slow queries, or multi-connection patterns — even when the user just says 'this query is slow'."
|
|
4
|
+
personas:
|
|
5
|
+
- eloquent-tamer
|
|
4
6
|
source: package
|
|
7
|
+
domain: engineering
|
|
5
8
|
---
|
|
6
9
|
|
|
7
10
|
# database
|
|
@@ -0,0 +1,143 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: decision-record
|
|
3
|
+
description: "Use when locking a trade-off, structuring an ADR draft, or wiring supersession chains — frames options · trade-offs · consequences before the file is written by `adr-create`."
|
|
4
|
+
personas:
|
|
5
|
+
- senior-engineer
|
|
6
|
+
- critical-challenger
|
|
7
|
+
- product-owner
|
|
8
|
+
source: package
|
|
9
|
+
domain: process
|
|
10
|
+
---
|
|
11
|
+
|
|
12
|
+
# decision-record
|
|
13
|
+
|
|
14
|
+
> Shape the **thinking** behind an architectural decision: list the
|
|
15
|
+
> options, expose trade-offs, lock the chosen variant, and wire the
|
|
16
|
+
> supersession chain. Hands off to [`adr-create`](../adr-create/SKILL.md)
|
|
17
|
+
> for file mechanics (numbering, index, frontmatter). This skill does
|
|
18
|
+
> not write the ADR file itself.
|
|
19
|
+
|
|
20
|
+
## When to use
|
|
21
|
+
|
|
22
|
+
- A team is about to commit to a non-trivial architectural choice and
|
|
23
|
+
wants the trade-offs on paper first.
|
|
24
|
+
- Two engineers disagree on an approach — surface the options and
|
|
25
|
+
consequences before one wins by attrition.
|
|
26
|
+
- A previous decision needs revisiting; the result must `supersede:`
|
|
27
|
+
the older ADR with explicit rationale.
|
|
28
|
+
- German triggers: "lass uns das festzurren", "Trade-off-Matrix",
|
|
29
|
+
"welche Option ziehen wir?".
|
|
30
|
+
|
|
31
|
+
Do NOT use when:
|
|
32
|
+
|
|
33
|
+
- The file mechanics are the entire ask (numbering, regenerating the
|
|
34
|
+
index) — route directly to [`adr-create`](../adr-create/SKILL.md).
|
|
35
|
+
- The decision is reversible and small (e.g. variable rename) — ADRs
|
|
36
|
+
are for choices that constrain future work.
|
|
37
|
+
- The user wants a feature plan, not a decision — route to the
|
|
38
|
+
planning command instead.
|
|
39
|
+
|
|
40
|
+
## Procedure
|
|
41
|
+
|
|
42
|
+
### 1. State the decision in one sentence
|
|
43
|
+
|
|
44
|
+
*"We need to decide `X` because `Y`."* If you cannot, the decision
|
|
45
|
+
is not ripe — surface that and stop.
|
|
46
|
+
|
|
47
|
+
### 2. Identify and enumerate options (≥ 2, usually ≤ 4)
|
|
48
|
+
|
|
49
|
+
Each option gets:
|
|
50
|
+
|
|
51
|
+
- **Name** — short, descriptive, never "option A / B / C".
|
|
52
|
+
- **Sketch** — what it actually does in one paragraph.
|
|
53
|
+
- **Cost / benefit** — three to five bullets per side, concrete.
|
|
54
|
+
- **Reversibility** — easy / costly / one-way (Bezos two-way / one-way
|
|
55
|
+
door framing).
|
|
56
|
+
|
|
57
|
+
Add an **explicit "do nothing"** option whenever it is plausible —
|
|
58
|
+
forces the user to compare the change against the status quo.
|
|
59
|
+
|
|
60
|
+
### 3. Build the trade-off matrix
|
|
61
|
+
|
|
62
|
+
| Criterion | Option 1 | Option 2 | ... |
|
|
63
|
+
|---|---|---|---|
|
|
64
|
+
| Implementation cost | | | |
|
|
65
|
+
| Operational cost | | | |
|
|
66
|
+
| Reversibility | | | |
|
|
67
|
+
| Blast radius | | | |
|
|
68
|
+
| Time-to-value | | | |
|
|
69
|
+
|
|
70
|
+
Pick criteria that *separate* the options. Criteria where every
|
|
71
|
+
option scores the same are noise — drop them.
|
|
72
|
+
|
|
73
|
+
### 4. Lock the choice + consequences
|
|
74
|
+
|
|
75
|
+
Write:
|
|
76
|
+
|
|
77
|
+
- **Decision:** *"We pick `<Option name>`."*
|
|
78
|
+
- **Consequences:** what becomes true, what becomes harder, what
|
|
79
|
+
becomes impossible. At least one of each, or admit that nothing
|
|
80
|
+
becomes harder (rare; verify).
|
|
81
|
+
|
|
82
|
+
### 5. Wire the supersession chain (if any)
|
|
83
|
+
|
|
84
|
+
If this decision overrides a prior ADR:
|
|
85
|
+
|
|
86
|
+
- Cite the prior ADR id and one-line summary.
|
|
87
|
+
- State **what changed in the world** that justifies overriding —
|
|
88
|
+
not "we now think differently".
|
|
89
|
+
- Hand off the `supersedes:` linkage to `adr-create`.
|
|
90
|
+
|
|
91
|
+
### 6. Hand off to file mechanics
|
|
92
|
+
|
|
93
|
+
Output the structured payload (below). The user — or
|
|
94
|
+
`adr-create` — turns it into the file.
|
|
95
|
+
|
|
96
|
+
## Output format
|
|
97
|
+
|
|
98
|
+
```
|
|
99
|
+
Decision: <one sentence>
|
|
100
|
+
|
|
101
|
+
Options:
|
|
102
|
+
1. <name> — <sketch>
|
|
103
|
+
Pros: ...
|
|
104
|
+
Cons: ...
|
|
105
|
+
Reversibility: easy | costly | one-way
|
|
106
|
+
2. ...
|
|
107
|
+
N. Do nothing — <sketch>
|
|
108
|
+
|
|
109
|
+
Trade-off matrix:
|
|
110
|
+
| Criterion | <opt 1> | <opt 2> | ... |
|
|
111
|
+
| ...
|
|
112
|
+
|
|
113
|
+
Locked: <chosen option name>
|
|
114
|
+
|
|
115
|
+
Consequences:
|
|
116
|
+
+ <becomes true>
|
|
117
|
+
- <becomes harder>
|
|
118
|
+
✗ <becomes impossible>
|
|
119
|
+
|
|
120
|
+
Supersedes: <ADR-XYZ "title"> (if any)
|
|
121
|
+
Trigger: <what changed> (if superseding)
|
|
122
|
+
|
|
123
|
+
Next: /adr-create with the payload above
|
|
124
|
+
```
|
|
125
|
+
|
|
126
|
+
## Gotcha
|
|
127
|
+
|
|
128
|
+
- "We compared the options" is not a comparison. The matrix must
|
|
129
|
+
separate them on at least two criteria.
|
|
130
|
+
- Pros without cons is advocacy, not analysis. If you cannot find
|
|
131
|
+
cons for the chosen option, you have not understood it yet.
|
|
132
|
+
- A supersession with no "what changed in the world" is a vibe
|
|
133
|
+
pivot. Reject and ask.
|
|
134
|
+
|
|
135
|
+
## Do NOT
|
|
136
|
+
|
|
137
|
+
- Do NOT pick the option before listing them — the matrix is the
|
|
138
|
+
point, not the conclusion.
|
|
139
|
+
- Do NOT write the ADR file from this skill — `adr-create` owns
|
|
140
|
+
filenames, numbering, and index regeneration.
|
|
141
|
+
- Do NOT pad option counts to look thorough; two real options beat
|
|
142
|
+
four straw men.
|
|
143
|
+
- Do NOT silently update an old ADR; supersession is explicit.
|
|
@@ -3,6 +3,7 @@ name: deep-reading-analyst
|
|
|
3
3
|
description: "Deep analysis of articles/long-form via thinking frameworks (SCQA, mental models, inversion) — 'analyze article', 'deep dive', 'extract insights', URL/text wanting depth not summary."
|
|
4
4
|
status: active
|
|
5
5
|
source: package
|
|
6
|
+
domain: discovery
|
|
6
7
|
external_source: "https://github.com/ginobefun/deep-reading-analyst-skill/tree/26cd7dc9920e025d39751e396e707399022e49ef/src/deep-reading-analyst"
|
|
7
8
|
refresh_trigger: "Upstream `ginobefun/deep-reading-analyst-skill` major rewrite (new framework added, dispatch table reshaped, or SHA pin invalidated by reference rename)."
|
|
8
9
|
sunset_criterion: "Replace with a 50-line pointer skill if (a) all referenced modules are adopted as project-local guidelines (`docs/guidelines/agent-infra/{framework}.md`) AND (b) the dispatch logic moves into a project-native router."
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: defense-in-depth
|
|
3
3
|
description: "Use when validation needs entry, business-logic, environment, and instrumentation guards so a bad value cannot reach the failure point — turns a local bug fix into a structural one."
|
|
4
4
|
source: package
|
|
5
|
+
domain: quality
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# defense-in-depth
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: dependency-upgrade
|
|
3
3
|
description: "Use when upgrading dependencies — "update Laravel", "bump PHP version", or "upgrade packages". Covers changelog review, breaking change detection, and verification."
|
|
4
4
|
source: package
|
|
5
|
+
domain: engineering
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# dependency-upgrade
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: description-assist
|
|
3
3
|
description: "Use when polishing a skill/rule/command/guideline frontmatter description — pushier phrasing, trigger coverage, undertrigger audit — even if the user just says 'make this pushier'."
|
|
4
4
|
source: package
|
|
5
|
+
domain: process
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
<!-- cloud_safe: degrade -->
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: design-review
|
|
3
3
|
description: "Use when the user says "review the design", "check the UI", or wants a comprehensive UI/UX review. Uses a 7-phase methodology covering interaction, responsiveness, accessibility, and more."
|
|
4
4
|
source: package
|
|
5
|
+
domain: quality
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# design-review
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: devcontainer
|
|
3
3
|
description: "Use when configuring DevContainers or GitHub Codespaces — devcontainer.json, custom images, secrets, VS Code features — even when the user just says 'why does my Codespace not start'."
|
|
4
4
|
source: package
|
|
5
|
+
domain: devops
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# devcontainer
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: developer-like-execution
|
|
3
3
|
description: "Use when implementing, debugging, refactoring, or reviewing code — enforces the think → analyze → verify → execute workflow — even when the user just says 'implement X' without naming it."
|
|
4
4
|
source: package
|
|
5
|
+
domain: process
|
|
5
6
|
execution:
|
|
6
7
|
type: assisted
|
|
7
8
|
handler: internal
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: docker
|
|
3
3
|
description: "Use when working with Docker — Dockerfile edits, docker-compose services, containers, or the dual-container (fast + Xdebug) setup — even when the user just says 'my container won't start'."
|
|
4
4
|
source: package
|
|
5
|
+
domain: devops
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# docker
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: dto-creator
|
|
3
3
|
description: "Use when the user says "create a DTO", "new data transfer object", or needs to convert request/response data into a typed PHP class. Creates DTOs with SimpleDto base class and attribute mapping."
|
|
4
4
|
source: package
|
|
5
|
+
domain: engineering
|
|
5
6
|
---
|
|
6
7
|
|
|
7
8
|
# dto-creator
|
|
@@ -1,7 +1,10 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: eloquent
|
|
3
3
|
description: "Use when writing Eloquent models, relationships, scopes, or queries via Model:: — 'fetch users with their orders'. NOT for PHPStan output, non-Eloquent services, or raw SQL questions."
|
|
4
|
+
personas:
|
|
5
|
+
- eloquent-tamer
|
|
4
6
|
source: package
|
|
7
|
+
domain: engineering
|
|
5
8
|
---
|
|
6
9
|
|
|
7
10
|
# eloquent
|
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
name: error-handling-patterns
|
|
3
3
|
description: "Use when picking a failure-reporting strategy — exceptions vs Result types, recoverable vs not, retry / circuit-breaker / graceful degradation — decision framework only, catalogues externalized."
|
|
4
4
|
source: package
|
|
5
|
+
domain: engineering
|
|
5
6
|
status: active
|
|
6
7
|
refresh_trigger: "≥30% of cited upstream pattern catalogues become deprecated, OR a new top-2 ecosystem (Python/JS/PHP/Go/Rust) ships a paradigm-shifting standard error model"
|
|
7
8
|
sunset_criterion: "When the upstream framework docs (Laravel, FastAPI, Express, Axum, Effect-TS) all carry an equivalent in-tree decision framework AND consumer projects no longer cite this skill in PR reviews for two consecutive review cycles."
|
|
@@ -1,7 +1,10 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: existing-ui-audit
|
|
3
3
|
description: "Use BEFORE writing or editing any non-trivial UI — inventories components, design tokens, shadcn primitives, and reusable patterns into state.ui_audit. Hard gate for the ui directive set."
|
|
4
|
+
personas:
|
|
5
|
+
- frontend-engineer
|
|
4
6
|
source: package
|
|
7
|
+
domain: discovery
|
|
5
8
|
---
|
|
6
9
|
|
|
7
10
|
# existing-ui-audit
|
|
@@ -1,7 +1,10 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: fe-design
|
|
3
3
|
description: "Reference for frontend-design heuristics — component architecture, layout patterns, form/table design, responsive strategy, a11y, UX principles. Stack-agnostic; cited by directives/ui/design.py."
|
|
4
|
+
personas:
|
|
5
|
+
- frontend-engineer
|
|
4
6
|
source: package
|
|
7
|
+
domain: engineering
|
|
5
8
|
---
|
|
6
9
|
|
|
7
10
|
# Frontend Design Skill (Reference)
|
|
@@ -200,7 +203,7 @@ Step indicator (1 — 2 — 3)
|
|
|
200
203
|
|
|
201
204
|
When `directives/ui/design.py` (or any caller) cites this skill:
|
|
202
205
|
|
|
203
|
-
1. **
|
|
206
|
+
1. **Inspect `state.ui_audit` first** — review the audit produced by [`existing-ui-audit`](../existing-ui-audit/SKILL.md); it is mandatory. Stop and request the audit if missing.
|
|
204
207
|
2. **Pick the smallest matching section** — Component Architecture, Form Design, Table Design, Responsive Strategy, Accessibility, or UX Principles. Cite by H2/H3 heading, never paste the whole skill.
|
|
205
208
|
3. **Defer to audit findings** — when the audit pins a project pattern (token, primitive, layout convention), use it. The heuristics here are fallbacks for gaps, not overrides.
|
|
206
209
|
4. **Defer to the stack apply skill** — Blade vs. Livewire vs. Flux vs. React-shadcn choices come from the dispatched impl skill, never from this reference.
|