@softspark/ai-toolkit 1.3.14 → 1.4.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/CHANGELOG.md +41 -0
- package/README.md +62 -16
- package/app/agents/backend-specialist.md +8 -0
- package/app/agents/code-reviewer.md +9 -0
- package/app/agents/database-architect.md +8 -0
- package/app/agents/debugger.md +8 -0
- package/app/agents/devops-implementer.md +8 -0
- package/app/agents/documenter.md +8 -0
- package/app/agents/frontend-specialist.md +8 -0
- package/app/agents/performance-optimizer.md +8 -0
- package/app/agents/security-auditor.md +9 -0
- package/app/agents/test-engineer.md +9 -0
- package/app/skills/analyze/SKILL.md +15 -0
- package/app/skills/api-patterns/SKILL.md +10 -0
- package/app/skills/ci-cd-patterns/SKILL.md +10 -0
- package/app/skills/clean-code/SKILL.md +10 -0
- package/app/skills/database-patterns/SKILL.md +10 -0
- package/app/skills/debug/SKILL.md +16 -0
- package/app/skills/docs/SKILL.md +16 -0
- package/app/skills/git-mastery/SKILL.md +10 -0
- package/app/skills/onboard/SKILL.md +15 -0
- package/app/skills/performance-profiling/SKILL.md +10 -0
- package/app/skills/plan/SKILL.md +16 -0
- package/app/skills/refactor/SKILL.md +16 -0
- package/app/skills/review/SKILL.md +58 -3
- package/app/skills/security-patterns/SKILL.md +10 -0
- package/app/skills/tdd/SKILL.md +6 -0
- package/app/skills/testing-patterns/SKILL.md +10 -0
- package/bin/ai-toolkit.js +33 -5
- package/kb/procedures/release-verification-sop.md +283 -0
- package/kb/reference/architecture-overview.md +36 -7
- package/kb/reference/competitive-features-implementation.md +51 -52
- package/kb/reference/language-rules.md +18 -4
- package/kb/reference/skills-catalog.md +57 -1
- package/llms-full.txt +451 -64
- package/llms.txt +1 -0
- package/manifest.json +1 -1
- package/package.json +4 -2
- package/scripts/dir_rules_shared.py +441 -0
- package/scripts/generate_antigravity.py +36 -0
- package/scripts/generate_augment_rules.py +107 -0
- package/scripts/generate_cline_rules.py +31 -0
- package/scripts/generate_conventions.py +37 -0
- package/scripts/generate_cursor_mdc.py +127 -0
- package/scripts/generate_roo_rules.py +30 -0
- package/scripts/generate_windsurf_rules.py +31 -0
- package/scripts/install.py +26 -1
- package/scripts/install_steps/ai_tools.py +149 -31
- package/scripts/install_steps/detect_language.py +68 -5
package/CHANGELOG.md
CHANGED
|
@@ -7,6 +7,47 @@ Versioning follows [Semantic Versioning](https://semver.org/).
|
|
|
7
7
|
|
|
8
8
|
---
|
|
9
9
|
|
|
10
|
+
## v1.4.0 — Full Platform Parity: 11 Editors, Directory-Based Rules, --editors Flag (2026-04-09)
|
|
11
|
+
|
|
12
|
+
### Added
|
|
13
|
+
- **Google Antigravity support** — new editor integration with `.agent/rules/` (6 rule files) and `.agent/workflows/` (13 workflow templates with YAML frontmatter). Full agent/skill catalog parity with other platforms.
|
|
14
|
+
- **Directory-based rules for all editors** — every platform now gets modern directory-based configs in addition to legacy single-file formats:
|
|
15
|
+
- Cursor: `.cursor/rules/*.mdc` with YAML frontmatter (`alwaysApply`, `globs`, `description`)
|
|
16
|
+
- Windsurf: `.windsurf/rules/*.md`
|
|
17
|
+
- Cline: `.cline/rules/*.md`
|
|
18
|
+
- Roo Code: `.roo/rules/*.md` (shared rules for all modes)
|
|
19
|
+
- Augment: `.augment/rules/ai-toolkit-*.md` with `auto_attached` globs per file type
|
|
20
|
+
- Aider: `CONVENTIONS.md` (auto-loaded as read-only context)
|
|
21
|
+
- **`--editors` flag** for `install --local` — selective editor installation:
|
|
22
|
+
- `--editors all` — install all 8 editors
|
|
23
|
+
- `--editors cursor,aider` — install only selected
|
|
24
|
+
- (no flag) — auto-detect from existing project files
|
|
25
|
+
- `update --local` auto-detects editors from existing configs
|
|
26
|
+
- **`--lang` flag** — explicit language selection for rules (`--lang typescript`, `--lang go,python`) with aliases (`go`→`golang`, `c++`→`cpp`, `cs`→`csharp`)
|
|
27
|
+
- **Two-phase language detection** — marker files (package.json, go.mod, etc.) + source file extension scanning (.py, .ts, .go, etc.)
|
|
28
|
+
- **Shared rule content module** (`dir_rules_shared.py`) — all platforms get identical agent/skill catalog, guidelines, and rules from a single source of truth
|
|
29
|
+
- **7 new CLI commands**: `cursor-mdc`, `windsurf-dir-rules`, `cline-dir-rules`, `roo-dir-rules`, `augment-dir-rules`, `conventions-md`, `antigravity-rules`
|
|
30
|
+
- **71 generator tests** — file existence, content verification, user file preservation, idempotency, stale cleanup, cross-platform parity check
|
|
31
|
+
|
|
32
|
+
### Changed
|
|
33
|
+
- `install --local` now installs only Claude Code configs by default (no editor bloat); editors require `--editors` flag or auto-detect from existing files
|
|
34
|
+
- All directory-based generators use `ai-toolkit-` prefix to prevent overwriting user files
|
|
35
|
+
- Total test count: 377 → 408
|
|
36
|
+
|
|
37
|
+
---
|
|
38
|
+
|
|
39
|
+
## v1.3.15 — Quality Guardrails: Anti-Rationalization, Confidence Scoring, Verification Checklists (2026-04-08)
|
|
40
|
+
|
|
41
|
+
### Added
|
|
42
|
+
- **Anti-rationalization tables** — 15 core skills now include `## Common Rationalizations` sections with domain-specific excuse/rebuttal tables that prevent agent drift and shortcut-taking. Inspired by [addyosmani/agent-skills](https://github.com/addyosmani/agent-skills).
|
|
43
|
+
- **Confidence scoring** (`/review`) — review findings now include per-issue confidence scores (1-10) and severity classification (critical/major/minor/nit) with a calibration guide.
|
|
44
|
+
- **LLM-as-Judge self-evaluation** (`/review`) — structured self-check after review: blind spot detection, anchoring bias check, and confidence calibration.
|
|
45
|
+
- **Agent verification checklists** — 10 key agents (`code-reviewer`, `test-engineer`, `security-auditor`, `debugger`, `backend-specialist`, `frontend-specialist`, `database-architect`, `performance-optimizer`, `devops-implementer`, `documenter`) now include `## Verification Checklist` exit criteria.
|
|
46
|
+
- **Skill reference routing** — 7 core skills (`/review`, `/debug`, `/plan`, `/refactor`, `/tdd`, `/docs`, `/analyze`) include `## Related Skills` sections for follow-up discoverability.
|
|
47
|
+
- **Intent Capture Interview** (`/onboard`) — Step 0 interview phase with 5 targeted questions to capture undocumented project intent before setup.
|
|
48
|
+
|
|
49
|
+
---
|
|
50
|
+
|
|
10
51
|
## v1.3.14 — CVE Scanner + Open Contributions (2026-04-08)
|
|
11
52
|
|
|
12
53
|
### Added
|
package/README.md
CHANGED
|
@@ -1,12 +1,12 @@
|
|
|
1
1
|
# ai-toolkit
|
|
2
2
|
|
|
3
|
-
> Professional-grade AI coding toolkit with multi-platform support. Machine-enforced safety, 91 skills, 44 agents, expanded lifecycle hooks, persona presets, experimental opt-in plugin packs, and benchmark tooling — works with Claude, Cursor, Windsurf, Copilot, Gemini, Cline, Roo Code, Aider, and
|
|
3
|
+
> Professional-grade AI coding toolkit with multi-platform support. Machine-enforced safety, 91 skills, 44 agents, expanded lifecycle hooks, persona presets, experimental opt-in plugin packs, and benchmark tooling — works with Claude, Cursor, Windsurf, Copilot, Gemini, Cline, Roo Code, Aider, Augment, and Google Antigravity, ready in 60 seconds.
|
|
4
4
|
|
|
5
5
|
[](https://github.com/softspark/ai-toolkit/actions/workflows/ci.yml)
|
|
6
6
|
[](LICENSE)
|
|
7
7
|
[](app/skills/)
|
|
8
8
|
[](app/agents/)
|
|
9
|
-
[](tests/)
|
|
10
10
|
|
|
11
11
|
---
|
|
12
12
|
|
|
@@ -112,20 +112,21 @@ Replaces all symlinks with real files, inlines rules into CLAUDE.md, copies cons
|
|
|
112
112
|
|
|
113
113
|
## Platform Support
|
|
114
114
|
|
|
115
|
-
| Platform | Config
|
|
116
|
-
|
|
115
|
+
| Platform | Config Files | How | Scope |
|
|
116
|
+
|----------|-------------|-----|-------|
|
|
117
117
|
| Claude Code | `~/.claude/` | `ai-toolkit install` | global |
|
|
118
|
-
| Cursor | `~/.cursor/rules` | `ai-toolkit install` | global |
|
|
119
|
-
| Windsurf | `~/.codeium
|
|
118
|
+
| Cursor | `~/.cursor/rules` + `.cursor/rules/*.mdc` | `ai-toolkit install` / `--local` | global + project |
|
|
119
|
+
| Windsurf | `~/.codeium/.../global_rules.md` + `.windsurf/rules/*.md` | `ai-toolkit install` / `--local` | global + project |
|
|
120
120
|
| Gemini CLI | `~/.gemini/GEMINI.md` | `ai-toolkit install` | global |
|
|
121
121
|
| GitHub Copilot | `.github/copilot-instructions.md` | `ai-toolkit install --local` | project |
|
|
122
|
-
| Cline | `.clinerules` | `ai-toolkit install --local` | project |
|
|
123
|
-
| Roo Code | `.roomodes` | `ai-toolkit install --local` | project |
|
|
124
|
-
| Aider | `.aider.conf.yml` | `ai-toolkit install --local` | project |
|
|
125
|
-
| Augment | `.augment/rules/ai-toolkit
|
|
122
|
+
| Cline | `.clinerules` + `.cline/rules/*.md` | `ai-toolkit install --local` | project |
|
|
123
|
+
| Roo Code | `.roomodes` + `.roo/rules/*.md` | `ai-toolkit install --local` | project |
|
|
124
|
+
| Aider | `.aider.conf.yml` + `CONVENTIONS.md` | `ai-toolkit install --local` | project |
|
|
125
|
+
| Augment | `.augment/rules/ai-toolkit-*.md` | `ai-toolkit install --local` | project |
|
|
126
|
+
| Google Antigravity | `.agent/rules/*.md` + `.agent/workflows/*.md` | `ai-toolkit install --local` | project |
|
|
126
127
|
| Codex / OpenCode | `AGENTS.md` | `ai-toolkit agents-md` | project |
|
|
127
128
|
|
|
128
|
-
> **Note:** Claude Code
|
|
129
|
+
> **Note:** Claude Code is always installed (primary platform with full feature support). Other editors are installed on demand with `--editors <list>` or auto-detected from existing project files. All platforms receive the same agent/skill catalog, guidelines, and rules. For editors lacking native bash lifecycle hooks, `--local` installs a Git hooks fallback (`.git/hooks/pre-commit`) to enforce quality gates pre-commit.
|
|
129
130
|
|
|
130
131
|
---
|
|
131
132
|
|
|
@@ -385,6 +386,43 @@ Three skills enforce non-negotiable quality gates with anti-rationalization tabl
|
|
|
385
386
|
| `debugging-tactics` | `NO FIXES WITHOUT ROOT CAUSE INVESTIGATION FIRST` | 4-phase debugging: root cause → pattern → hypothesis → fix. 3+ failed fixes → question architecture. |
|
|
386
387
|
| `verification-before-completion` | `NO COMPLETION CLAIMS WITHOUT FRESH VERIFICATION EVIDENCE` | Gate function: IDENTIFY → RUN → READ → VERIFY → CLAIM. "Should work now" is not evidence. |
|
|
387
388
|
|
|
389
|
+
Additionally, **15 core skills** include `## Common Rationalizations` tables — domain-specific excuses with rebuttals that prevent agent drift and shortcut-taking. Skills with rationalization tables: `/review`, `/debug`, `/refactor`, `/tdd`, `/plan`, `/docs`, `/analyze`, `security-patterns`, `testing-patterns`, `api-patterns`, `ci-cd-patterns`, `clean-code`, `performance-profiling`, `git-mastery`, `database-patterns`.
|
|
390
|
+
|
|
391
|
+
### Confidence Scoring & Self-Evaluation (`/review`)
|
|
392
|
+
|
|
393
|
+
The `/review` skill outputs findings with per-issue confidence scores (1-10) and severity classification (critical/major/minor/nit). After completing a review, an LLM-as-Judge self-evaluation pass checks for blind spots: anchoring bias, assumption vs verification, missing unhappy paths, and calibrates confidence scores.
|
|
394
|
+
|
|
395
|
+
### Agent Verification Checklists
|
|
396
|
+
|
|
397
|
+
10 key agents include `## Verification Checklist` — exit criteria that MUST be met before presenting results. Each checklist is domain-specific:
|
|
398
|
+
|
|
399
|
+
| Agent | Key exit criteria |
|
|
400
|
+
|-------|------------------|
|
|
401
|
+
| `code-reviewer` | Every finding has file:line + evidence, not just opinion |
|
|
402
|
+
| `security-auditor` | Each finding includes proof-of-concept or exploit path |
|
|
403
|
+
| `test-engineer` | No empty/placeholder tests, mocks only at boundaries |
|
|
404
|
+
| `debugger` | Root cause identified, regression test added |
|
|
405
|
+
| `backend-specialist` | Input validation, error format, query optimization |
|
|
406
|
+
| `frontend-specialist` | Empty/loading/error states, accessibility, responsive |
|
|
407
|
+
| `database-architect` | Migration tested on prod-like volume, rollback tested |
|
|
408
|
+
| `performance-optimizer` | Baseline measured, profiler evidence attached |
|
|
409
|
+
| `devops-implementer` | Dry run passed, rollback documented, no hardcoded secrets |
|
|
410
|
+
| `documenter` | Code examples runnable, no placeholders, valid links |
|
|
411
|
+
|
|
412
|
+
### Skill Reference Routing
|
|
413
|
+
|
|
414
|
+
7 core skills include `## Related Skills` sections that suggest logical follow-up skills, improving discoverability:
|
|
415
|
+
|
|
416
|
+
```
|
|
417
|
+
/review → found issues? → /debug, /tdd, /cve-scan, /analyze
|
|
418
|
+
/debug → bug fixed? → /review, /tdd, /workflow incident-response
|
|
419
|
+
/plan → approved? → /orchestrate, /write-a-prd, /grill-me
|
|
420
|
+
```
|
|
421
|
+
|
|
422
|
+
### Intent Capture Interview (`/onboard`)
|
|
423
|
+
|
|
424
|
+
The `/onboard` skill now includes a Step 0 interview phase before setup — asking 5 targeted questions to capture undocumented project intent (common contributor mistakes, protected files, deployment model, non-obvious constraints, review culture). Answers customize the generated `CLAUDE.md`.
|
|
425
|
+
|
|
388
426
|
### 7. Two-Stage Review (`/subagent-development`)
|
|
389
427
|
|
|
390
428
|
Per-task review pipeline inspired by [obra/superpowers](https://github.com/obra/superpowers):
|
|
@@ -486,7 +524,7 @@ ai-toolkit install --local --lang typescript # explicit language selection
|
|
|
486
524
|
ai-toolkit install --local --lang go,python # multiple languages
|
|
487
525
|
```
|
|
488
526
|
|
|
489
|
-
`--local` automatically detects languages (
|
|
527
|
+
`--local` automatically detects languages using two-phase detection: config markers (package.json, go.mod, Cargo.toml, etc.) plus source file extension scanning (.py, .ts, .go, etc.). `--lang` accepts aliases (`go`, `c++`, `cs`). Rules are injected into `CLAUDE.md` and auto-updated on `ai-toolkit update --local`.
|
|
490
528
|
|
|
491
529
|
---
|
|
492
530
|
|
|
@@ -722,7 +760,14 @@ Usage: ai-toolkit <command> [options]
|
|
|
722
760
|
| `cline-rules` | Generate `.clinerules` in current dir |
|
|
723
761
|
| `roo-modes` | Generate `.roomodes` in current dir |
|
|
724
762
|
| `aider-conf` | Generate `.aider.conf.yml` in current dir |
|
|
725
|
-
| `
|
|
763
|
+
| `conventions-md` | Generate `CONVENTIONS.md` for Aider (auto-loaded) |
|
|
764
|
+
| `augment-rules` | Generate `.augment/rules/ai-toolkit.md` (legacy single file) |
|
|
765
|
+
| `augment-dir-rules` | Generate `.augment/rules/ai-toolkit-*.md` (recommended) |
|
|
766
|
+
| `cursor-mdc` | Generate `.cursor/rules/*.mdc` for Cursor (recommended) |
|
|
767
|
+
| `windsurf-dir-rules` | Generate `.windsurf/rules/*.md` for Windsurf |
|
|
768
|
+
| `cline-dir-rules` | Generate `.cline/rules/*.md` for Cline |
|
|
769
|
+
| `roo-dir-rules` | Generate `.roo/rules/*.md` for Roo Code |
|
|
770
|
+
| `antigravity-rules` | Generate `.agent/rules/` and `.agent/workflows/` for Google Antigravity |
|
|
726
771
|
| `agents-md` | Regenerate `AGENTS.md` from agent definitions |
|
|
727
772
|
| `llms-txt` | Generate `llms.txt` and `llms-full.txt` |
|
|
728
773
|
| `generate-all` | Generate all platform configs at once |
|
|
@@ -735,11 +780,12 @@ ai-toolkit install --only agents,hooks # apply only listed components
|
|
|
735
780
|
ai-toolkit install --skip hooks # skip listed components
|
|
736
781
|
ai-toolkit install --profile minimal # profile preset: minimal | standard | strict
|
|
737
782
|
ai-toolkit install --persona backend-lead # persona preset: backend-lead | frontend-lead | devops-eng | junior-dev
|
|
738
|
-
ai-toolkit install --local #
|
|
739
|
-
ai-toolkit
|
|
783
|
+
ai-toolkit install --local # Claude Code only (CLAUDE.md, settings, constitution, language rules)
|
|
784
|
+
ai-toolkit install --local --editors all # Claude Code + all editors (Cursor, Windsurf, Cline, Roo, Aider, Augment, Copilot, Antigravity)
|
|
785
|
+
ai-toolkit install --local --editors cursor,aider # Claude Code + specific editors
|
|
786
|
+
ai-toolkit update --local # re-apply; auto-detects editors from existing project files
|
|
740
787
|
ai-toolkit install --list # dry-run: show what would be applied
|
|
741
788
|
ai-toolkit install --modules core,agents,rules-typescript # selective module install
|
|
742
|
-
ai-toolkit install --local # auto-detects language, installs matching rules
|
|
743
789
|
ai-toolkit install --lang typescript # explicit language for rules install
|
|
744
790
|
```
|
|
745
791
|
|
|
@@ -184,6 +184,14 @@ After implementing significant changes, update documentation:
|
|
|
184
184
|
### Delegation
|
|
185
185
|
For large documentation tasks, hand off to `documenter` agent.
|
|
186
186
|
|
|
187
|
+
## Verification Checklist
|
|
188
|
+
Before presenting implementation:
|
|
189
|
+
- [ ] All new endpoints have input validation
|
|
190
|
+
- [ ] Error responses follow the project's error format
|
|
191
|
+
- [ ] Database queries are optimized (checked with EXPLAIN if applicable)
|
|
192
|
+
- [ ] New dependencies are justified and audited
|
|
193
|
+
- [ ] Migration is reversible
|
|
194
|
+
|
|
187
195
|
## KB Integration
|
|
188
196
|
|
|
189
197
|
Before coding, search knowledge base:
|
|
@@ -177,6 +177,15 @@ After significant reviews, update documentation:
|
|
|
177
177
|
### Delegation
|
|
178
178
|
For large documentation tasks, hand off to `documenter` agent.
|
|
179
179
|
|
|
180
|
+
## Verification Checklist
|
|
181
|
+
Before presenting review results:
|
|
182
|
+
- [ ] Every finding includes file:line reference
|
|
183
|
+
- [ ] Each finding has evidence (code snippet or reasoning), not just opinion
|
|
184
|
+
- [ ] Severity rating reflects actual impact, not gut feeling
|
|
185
|
+
- [ ] "No findings" includes list of specific checks performed
|
|
186
|
+
- [ ] Security-sensitive files received deeper scrutiny
|
|
187
|
+
- [ ] Test coverage gaps are flagged, not assumed covered
|
|
188
|
+
|
|
180
189
|
## Limitations
|
|
181
190
|
|
|
182
191
|
- **Security penetration testing** → Use `security-auditor`
|
|
@@ -308,6 +308,14 @@ After schema/database changes, update documentation:
|
|
|
308
308
|
### Delegation
|
|
309
309
|
For large documentation tasks, hand off to `documenter` agent.
|
|
310
310
|
|
|
311
|
+
## Verification Checklist
|
|
312
|
+
Before presenting schema changes:
|
|
313
|
+
- [ ] Migration tested on production-like data volume
|
|
314
|
+
- [ ] Rollback script exists and was tested
|
|
315
|
+
- [ ] Indexes cover the expected query patterns
|
|
316
|
+
- [ ] No long-running locks on large tables
|
|
317
|
+
- [ ] Application handles both old and new schema during migration
|
|
318
|
+
|
|
311
319
|
## KB Integration
|
|
312
320
|
|
|
313
321
|
Before designing, search knowledge base:
|
package/app/agents/debugger.md
CHANGED
|
@@ -231,6 +231,14 @@ After fixing significant bugs, update documentation:
|
|
|
231
231
|
### Delegation
|
|
232
232
|
For large documentation tasks, hand off to `documenter` agent.
|
|
233
233
|
|
|
234
|
+
## Verification Checklist
|
|
235
|
+
Before claiming a bug is fixed:
|
|
236
|
+
- [ ] Root cause identified, not just symptoms addressed
|
|
237
|
+
- [ ] Fix was verified by reproducing the original failure first
|
|
238
|
+
- [ ] Regression test added to prevent recurrence
|
|
239
|
+
- [ ] Related code paths checked for similar issues
|
|
240
|
+
- [ ] Fix doesn't introduce new side effects
|
|
241
|
+
|
|
234
242
|
## Limitations
|
|
235
243
|
|
|
236
244
|
- **Performance profiling** → Use `performance-optimizer`
|
|
@@ -187,6 +187,14 @@ instructions: |
|
|
|
187
187
|
---
|
|
188
188
|
```
|
|
189
189
|
|
|
190
|
+
## Verification Checklist
|
|
191
|
+
Before presenting infrastructure changes:
|
|
192
|
+
- [ ] Dry run completed successfully
|
|
193
|
+
- [ ] Rollback procedure documented and tested
|
|
194
|
+
- [ ] Secrets are not hardcoded or logged
|
|
195
|
+
- [ ] Health checks configured for new services
|
|
196
|
+
- [ ] Resource limits set to prevent runaway costs
|
|
197
|
+
|
|
190
198
|
## Limitations
|
|
191
199
|
|
|
192
200
|
- **Architectural decisions** → Use `infrastructure-architect`
|
package/app/agents/documenter.md
CHANGED
|
@@ -357,6 +357,14 @@ last_updated: "YYYY-MM-DD"
|
|
|
357
357
|
- Add diagrams where helpful
|
|
358
358
|
- Version documentation with code
|
|
359
359
|
|
|
360
|
+
## Verification Checklist
|
|
361
|
+
Before presenting documentation:
|
|
362
|
+
- [ ] Code examples are tested and runnable
|
|
363
|
+
- [ ] No placeholder text or TODO markers remain
|
|
364
|
+
- [ ] Links and references are valid
|
|
365
|
+
- [ ] Architecture decisions include rationale (not just "what")
|
|
366
|
+
- [ ] KB documents have complete YAML frontmatter
|
|
367
|
+
|
|
360
368
|
## Limitations
|
|
361
369
|
|
|
362
370
|
- **Code implementation** → Use `devops-implementer`
|
|
@@ -200,6 +200,14 @@ After implementing significant changes, update documentation:
|
|
|
200
200
|
### Delegation
|
|
201
201
|
For large documentation tasks, hand off to `documenter` agent.
|
|
202
202
|
|
|
203
|
+
## Verification Checklist
|
|
204
|
+
Before presenting implementation:
|
|
205
|
+
- [ ] Components render correctly with empty/loading/error states
|
|
206
|
+
- [ ] Accessibility basics checked (keyboard nav, aria labels, contrast)
|
|
207
|
+
- [ ] No console errors or warnings in dev tools
|
|
208
|
+
- [ ] Responsive behavior verified at mobile/tablet/desktop breakpoints
|
|
209
|
+
- [ ] Bundle size impact assessed for new dependencies
|
|
210
|
+
|
|
203
211
|
## KB Integration
|
|
204
212
|
|
|
205
213
|
Before coding, search knowledge base:
|
|
@@ -247,6 +247,14 @@ After performance optimizations, update documentation:
|
|
|
247
247
|
### Delegation
|
|
248
248
|
For large documentation tasks, hand off to `documenter` agent.
|
|
249
249
|
|
|
250
|
+
## Verification Checklist
|
|
251
|
+
Before presenting optimization:
|
|
252
|
+
- [ ] Baseline measurement taken before changes
|
|
253
|
+
- [ ] Improvement measured with realistic data, not synthetic benchmarks
|
|
254
|
+
- [ ] No regressions in other metrics (memory, latency, correctness)
|
|
255
|
+
- [ ] Optimization targets the actual bottleneck (profiler evidence attached)
|
|
256
|
+
- [ ] Cache invalidation strategy documented if caching was added
|
|
257
|
+
|
|
250
258
|
## Limitations
|
|
251
259
|
|
|
252
260
|
- **Production incidents** → Use `incident-responder`
|
|
@@ -301,6 +301,15 @@ For large documentation tasks, hand off to `documenter` agent.
|
|
|
301
301
|
- No destructive testing without explicit approval
|
|
302
302
|
- Report findings responsibly
|
|
303
303
|
|
|
304
|
+
## Verification Checklist
|
|
305
|
+
Before presenting security findings:
|
|
306
|
+
- [ ] Each finding includes proof-of-concept or exploit path
|
|
307
|
+
- [ ] Severity ratings reference actual impact, not theoretical risk
|
|
308
|
+
- [ ] "No findings" includes the specific checks performed
|
|
309
|
+
- [ ] Dependencies were scanned, not just application code
|
|
310
|
+
- [ ] Secrets scan covered all file types (not just source code)
|
|
311
|
+
- [ ] Auth flows were traced end-to-end, not spot-checked
|
|
312
|
+
|
|
304
313
|
## Limitations
|
|
305
314
|
|
|
306
315
|
- **Code implementation** → Use `devops-implementer`
|
|
@@ -257,6 +257,15 @@ After writing significant tests, update documentation:
|
|
|
257
257
|
### Delegation
|
|
258
258
|
For large documentation tasks, hand off to `documenter` agent.
|
|
259
259
|
|
|
260
|
+
## Verification Checklist
|
|
261
|
+
Before presenting test results:
|
|
262
|
+
- [ ] Every test has a clear assertion (no empty or placeholder tests)
|
|
263
|
+
- [ ] Edge cases are explicitly tested, not assumed
|
|
264
|
+
- [ ] Mocks are only at system boundaries, not internal collaborators
|
|
265
|
+
- [ ] Test names describe behavior, not implementation
|
|
266
|
+
- [ ] Flaky test patterns (time, network, order-dependent) are flagged
|
|
267
|
+
- [ ] Coverage gaps are reported with specific uncovered paths
|
|
268
|
+
|
|
260
269
|
## Limitations
|
|
261
270
|
|
|
262
271
|
- **Code implementation** → Use `devops-implementer`
|
|
@@ -82,6 +82,15 @@ python3 ${CLAUDE_SKILL_DIR}/scripts/complexity.py .
|
|
|
82
82
|
|
|
83
83
|
Reports file counts by type, largest files, TODO/FIXME counts, and total code lines.
|
|
84
84
|
|
|
85
|
+
## Common Rationalizations
|
|
86
|
+
|
|
87
|
+
| Excuse | Why It's Wrong |
|
|
88
|
+
|--------|----------------|
|
|
89
|
+
| "The linter is green, the code is fine" | Linters catch syntax, not design flaws — analysis covers architecture and patterns |
|
|
90
|
+
| "We know where the problems are" | Intuition misses systemic issues — data-driven analysis reveals hidden hotspots |
|
|
91
|
+
| "Analysis takes too long" | A 5-minute scan prevents weeks of debugging — front-load the investment |
|
|
92
|
+
| "It's legacy code, analysis won't help" | Legacy code benefits most — find the critical paths before they break |
|
|
93
|
+
|
|
85
94
|
## Tools Used
|
|
86
95
|
|
|
87
96
|
| Language | Tools |
|
|
@@ -90,3 +99,9 @@ Reports file counts by type, largest files, TODO/FIXME counts, and total code li
|
|
|
90
99
|
| JavaScript | eslint, tsc |
|
|
91
100
|
| Go | golangci-lint |
|
|
92
101
|
| Rust | clippy |
|
|
102
|
+
|
|
103
|
+
## Related Skills
|
|
104
|
+
- Found quality issues? → `/refactor` to fix them systematically
|
|
105
|
+
- Security issues detected? → `/cve-scan` for dependency audit
|
|
106
|
+
- Want deeper architecture review? → `/architecture-audit` for friction discovery
|
|
107
|
+
- Performance hotspots found? → `/workflow performance-optimization`
|
|
@@ -292,6 +292,16 @@ Accept: application/vnd.myapi.v1+json
|
|
|
292
292
|
|
|
293
293
|
---
|
|
294
294
|
|
|
295
|
+
## Common Rationalizations
|
|
296
|
+
|
|
297
|
+
| Excuse | Why It's Wrong |
|
|
298
|
+
|--------|----------------|
|
|
299
|
+
| "We'll version the API later" | Unversioned APIs break clients on every change — version from day one |
|
|
300
|
+
| "Retries are the client's problem" | Server-side idempotency prevents data corruption — design for at-least-once delivery |
|
|
301
|
+
| "We'll add rate limiting later" | Unprotected endpoints get abused within hours of deployment |
|
|
302
|
+
| "Error messages are just for debugging" | Error responses are your API's UX — clients depend on consistent, parseable errors |
|
|
303
|
+
| "PATCH and PUT are the same thing" | PUT replaces the resource, PATCH modifies it — wrong semantics cause data loss |
|
|
304
|
+
|
|
295
305
|
## Best Practices
|
|
296
306
|
|
|
297
307
|
- [ ] Use HTTPS only
|
|
@@ -291,6 +291,16 @@ env:
|
|
|
291
291
|
| `feat:` | Minor (0.x.0) | `feat: add user search endpoint` |
|
|
292
292
|
| `feat!:` / `BREAKING CHANGE:` | Major (x.0.0) | `feat!: change API response format` |
|
|
293
293
|
|
|
294
|
+
## Common Rationalizations
|
|
295
|
+
|
|
296
|
+
| Excuse | Why It's Wrong |
|
|
297
|
+
|--------|----------------|
|
|
298
|
+
| "CI is green, ship it" | CI tests the happy path — verify edge cases, security, and performance separately |
|
|
299
|
+
| "Manual deploys give us more control" | Manual deploys give you more human error — automate the repeatable parts |
|
|
300
|
+
| "We'll set up CI when the project is bigger" | Small projects grow fast — CI debt compounds and retrofitting is painful |
|
|
301
|
+
| "Caching isn't worth the complexity" | Uncached builds waste developer time daily — caching pays for itself in a week |
|
|
302
|
+
| "Feature flags are over-engineering" | Feature flags decouple deploy from release — they're the cheapest safety net |
|
|
303
|
+
|
|
294
304
|
## Anti-Patterns
|
|
295
305
|
- Secrets in pipeline logs or environment dumps
|
|
296
306
|
- No caching (slow builds)
|
|
@@ -100,6 +100,16 @@ Keep modules focused. Order contents consistently: imports (stdlib, third-party,
|
|
|
100
100
|
|
|
101
101
|
---
|
|
102
102
|
|
|
103
|
+
## Common Rationalizations
|
|
104
|
+
|
|
105
|
+
| Excuse | Why It's Wrong |
|
|
106
|
+
|--------|----------------|
|
|
107
|
+
| "It's readable enough" | "Enough" means someone will misread it eventually — clarity prevents incidents |
|
|
108
|
+
| "Refactoring for readability is gold-plating" | Readability is maintainability — future you will thank present you |
|
|
109
|
+
| "Short variable names are faster to type" | You type it once, readers parse it hundreds of times — optimize for reading |
|
|
110
|
+
| "DRY means never repeat anything" | Wrong DRY creates coupling — duplicate until you see the real abstraction |
|
|
111
|
+
| "More abstractions = cleaner code" | Premature abstraction is worse than duplication — wait for the third use |
|
|
112
|
+
|
|
103
113
|
## Language-Specific References
|
|
104
114
|
|
|
105
115
|
For detailed patterns, type hints, linting configuration, and idiomatic code per language:
|
|
@@ -296,3 +296,13 @@ client.create_payload_index(
|
|
|
296
296
|
| COSINE | Text embeddings | Yes |
|
|
297
297
|
| EUCLID | Image embeddings | No |
|
|
298
298
|
| DOT | When vectors pre-normalized | Yes |
|
|
299
|
+
|
|
300
|
+
## Common Rationalizations
|
|
301
|
+
|
|
302
|
+
| Excuse | Why It's Wrong |
|
|
303
|
+
|--------|----------------|
|
|
304
|
+
| "We'll add indexes later when it's slow" | Missing indexes on production tables cause outages, not slowdowns — index from design |
|
|
305
|
+
| "The ORM handles performance" | ORMs generate queries, they don't optimize them — always check the query plan |
|
|
306
|
+
| "NoSQL is faster" | NoSQL trades consistency for speed — if you need joins, use a relational DB |
|
|
307
|
+
| "We don't need migrations, we'll update the schema directly" | Direct schema changes are irreversible and untestable — migrations are the safety net |
|
|
308
|
+
| "One big table is simpler" | Denormalization without measurement creates update anomalies — normalize first, denormalize with data |
|
|
@@ -144,6 +144,16 @@ Have them talk to each other to challenge each other's theories.
|
|
|
144
144
|
Report consensus when done.
|
|
145
145
|
```
|
|
146
146
|
|
|
147
|
+
## Common Rationalizations
|
|
148
|
+
|
|
149
|
+
| Excuse | Why It's Wrong |
|
|
150
|
+
|--------|----------------|
|
|
151
|
+
| "It works on my machine" | Environment differences are the #1 cause of production bugs — reproduce in prod-like env |
|
|
152
|
+
| "It must be a library bug" | 95% of the time it's your code — exhaust local hypotheses first |
|
|
153
|
+
| "I'll just add more logging and wait" | Passive debugging wastes hours — form a hypothesis and test it actively |
|
|
154
|
+
| "The error message says X, so it must be X" | Error messages often describe symptoms, not root causes — trace the full chain |
|
|
155
|
+
| "It only happens sometimes, probably a fluke" | Intermittent bugs are race conditions or state leaks — they get worse, not better |
|
|
156
|
+
|
|
147
157
|
## Debug Checklist
|
|
148
158
|
|
|
149
159
|
- [ ] Identified error/symptom
|
|
@@ -152,3 +162,9 @@ Report consensus when done.
|
|
|
152
162
|
- [ ] Reproduced issue
|
|
153
163
|
- [ ] Formed hypothesis
|
|
154
164
|
- [ ] Tested fix
|
|
165
|
+
|
|
166
|
+
## Related Skills
|
|
167
|
+
- Bug fixed? → `/review` to verify the fix quality
|
|
168
|
+
- Need a regression test? → `/tdd` to write it test-first
|
|
169
|
+
- Performance issue? → `/analyze --type=complexity` for hotspot analysis
|
|
170
|
+
- Incident in production? → `/workflow incident-response` for full response
|
package/app/skills/docs/SKILL.md
CHANGED
|
@@ -113,6 +113,16 @@ Proposed
|
|
|
113
113
|
- [ ] Commit changes
|
|
114
114
|
```
|
|
115
115
|
|
|
116
|
+
## Common Rationalizations
|
|
117
|
+
|
|
118
|
+
| Excuse | Why It's Wrong |
|
|
119
|
+
|--------|----------------|
|
|
120
|
+
| "The code is self-documenting" | Code shows how, not why — decisions, constraints, and context need prose |
|
|
121
|
+
| "Nobody reads docs anyway" | People don't read bad docs — good docs are the first thing consulted |
|
|
122
|
+
| "I'll document it when it's stable" | Unstable code needs docs most — document intent so others can contribute |
|
|
123
|
+
| "Comments get stale" | That's an argument for maintaining docs, not skipping them |
|
|
124
|
+
| "The tests are the documentation" | Tests verify behavior but don't explain architecture, trade-offs, or setup |
|
|
125
|
+
|
|
116
126
|
## Configuration
|
|
117
127
|
|
|
118
128
|
Documentation settings in:
|
|
@@ -143,3 +153,9 @@ Create an agent team for documentation:
|
|
|
143
153
|
- Teammate 3 (documenter): "Generate README sections: installation, usage, API reference." Use Opus.
|
|
144
154
|
Teammates should NOT overlap — each owns their assigned scope.
|
|
145
155
|
```
|
|
156
|
+
|
|
157
|
+
## Related Skills
|
|
158
|
+
- Documenting an architecture decision? → `/council` for multi-perspective analysis first
|
|
159
|
+
- Need to explore the codebase? → `/explore` to understand structure before documenting
|
|
160
|
+
- Writing a PRD? → `/write-a-prd` for structured product requirements
|
|
161
|
+
- Auditing existing docs? → `/analyze` for coverage gaps
|
|
@@ -68,3 +68,13 @@ git cherry-pick --continue
|
|
|
68
68
|
- `perf:` Performance improvement
|
|
69
69
|
- `test:` Adding missing tests
|
|
70
70
|
- `chore:` Build process/auxiliary tools
|
|
71
|
+
|
|
72
|
+
## Common Rationalizations
|
|
73
|
+
|
|
74
|
+
| Excuse | Why It's Wrong |
|
|
75
|
+
|--------|----------------|
|
|
76
|
+
| "I'll clean up commits later" | Later means never — write clean commits as you go |
|
|
77
|
+
| "Force push is fine on my branch" | Others may have fetched your branch — use --force-with-lease |
|
|
78
|
+
| "One big commit is simpler" | Big commits are impossible to review, bisect, or revert — keep them atomic |
|
|
79
|
+
| "Merge conflicts mean someone else's problem" | Conflicts mean you diverged too long — rebase frequently to stay aligned |
|
|
80
|
+
| "Commit messages don't matter" | Messages are documentation — future you needs to understand why, not just what |
|
|
@@ -17,6 +17,21 @@ Guide the user through setting up the ai-toolkit in their project, including con
|
|
|
17
17
|
|
|
18
18
|
## Setup Steps
|
|
19
19
|
|
|
20
|
+
### Step 0: Intent Capture Interview
|
|
21
|
+
|
|
22
|
+
Before setting up tooling, understand the project's undocumented context. Ask the developer these questions (adapt based on what the codebase scan reveals):
|
|
23
|
+
|
|
24
|
+
1. **What's the one thing a new contributor always gets wrong?** — This reveals the biggest documentation gap
|
|
25
|
+
2. **Are there files or directories that should NOT be modified?** — Identifies protected areas (legacy, generated, vendor)
|
|
26
|
+
3. **What's the deployment model?** — Monolith, microservices, serverless, edge — shapes which agents and skills are most relevant
|
|
27
|
+
4. **Are there non-obvious constraints?** — Compliance requirements, performance budgets, browser support matrix
|
|
28
|
+
5. **What's the team's review culture?** — Strict PR reviews, trunk-based, pair programming — configures `/review` behavior
|
|
29
|
+
|
|
30
|
+
Use answers to:
|
|
31
|
+
- Customize the generated `CLAUDE.md` with project-specific warnings and conventions
|
|
32
|
+
- Select the right `--profile` (minimal/standard/strict) automatically
|
|
33
|
+
- Pre-configure relevant language rules
|
|
34
|
+
|
|
20
35
|
### Step 1: Prerequisites Check
|
|
21
36
|
- [ ] Claude Code CLI installed
|
|
22
37
|
- [ ] ai-toolkit repository cloned
|
|
@@ -57,3 +57,13 @@ Always measure -> change -> measure.
|
|
|
57
57
|
2. **Algorithm**: (O(n²) -> O(n log n))
|
|
58
58
|
3. **Memory**: (Allocation churn, GC pressure)
|
|
59
59
|
4. **Micro-optimization**: (Loop unrolling, etc.) - *Smallest Gains*
|
|
60
|
+
|
|
61
|
+
## Common Rationalizations
|
|
62
|
+
|
|
63
|
+
| Excuse | Why It's Wrong |
|
|
64
|
+
|--------|----------------|
|
|
65
|
+
| "It feels slow, let me optimize this function" | Feelings aren't data — profile first, then optimize the actual bottleneck |
|
|
66
|
+
| "We should optimize everything" | Premature optimization is the root of all evil — focus on the critical path |
|
|
67
|
+
| "Caching will fix it" | Caching masks problems and adds complexity — fix the root cause first |
|
|
68
|
+
| "It's fast enough in dev" | Dev has 1 user — production has thousands and cold caches |
|
|
69
|
+
| "We'll optimize later" | Performance debt compounds — a 100ms regression per sprint = 5s in a year |
|
package/app/skills/plan/SKILL.md
CHANGED
|
@@ -95,6 +95,16 @@ During planning:
|
|
|
95
95
|
- NO code writing
|
|
96
96
|
- NO file creation (except plan)
|
|
97
97
|
|
|
98
|
+
## Common Rationalizations
|
|
99
|
+
|
|
100
|
+
| Excuse | Why It's Wrong |
|
|
101
|
+
|--------|----------------|
|
|
102
|
+
| "We already know what to build" | Assumed requirements lead to rework — validate assumptions explicitly |
|
|
103
|
+
| "Planning is wasted time, just start coding" | Unplanned work has 3-5x more rework — 30 min planning saves days |
|
|
104
|
+
| "The requirements will change anyway" | Plans adapt — without one, you can't assess impact of changes |
|
|
105
|
+
| "It's a small feature, no plan needed" | Small features in complex systems have hidden dependencies — map them |
|
|
106
|
+
| "We'll figure it out as we go" | Discovery without structure leads to scope creep and missed edge cases |
|
|
107
|
+
|
|
98
108
|
## Next Steps
|
|
99
109
|
|
|
100
110
|
After plan approval:
|
|
@@ -108,3 +118,9 @@ Before planning:
|
|
|
108
118
|
smart_query("project template: {type}")
|
|
109
119
|
hybrid_search_kb("architecture {pattern}")
|
|
110
120
|
```
|
|
121
|
+
|
|
122
|
+
## Related Skills
|
|
123
|
+
- Plan approved? → `/orchestrate` or `/workflow` to execute with agents
|
|
124
|
+
- Need requirements first? → `/write-a-prd` for structured product requirements
|
|
125
|
+
- Want to stress-test the plan? → `/grill-me` for Socratic questioning
|
|
126
|
+
- Ready to break into issues? → `/prd-to-plan` → `/triage-issue`
|
|
@@ -81,6 +81,16 @@ Before executing:
|
|
|
81
81
|
- [ ] Tests passing
|
|
82
82
|
- [ ] Backup created
|
|
83
83
|
|
|
84
|
+
## Common Rationalizations
|
|
85
|
+
|
|
86
|
+
| Excuse | Why It's Wrong |
|
|
87
|
+
|--------|----------------|
|
|
88
|
+
| "It works, don't touch it" | Working code that's hard to maintain slows every future change |
|
|
89
|
+
| "We'll refactor it later" | Later never comes — refactor when the pain is fresh and context is loaded |
|
|
90
|
+
| "It's too risky to change" | That's exactly why it needs refactoring — risk compounds with complexity |
|
|
91
|
+
| "Just one more hack won't hurt" | Each hack makes the next one easier to justify — break the cycle now |
|
|
92
|
+
| "We need to rewrite from scratch" | Incremental refactoring is safer and delivers value continuously |
|
|
93
|
+
|
|
84
94
|
## READ BEFORE WRITE
|
|
85
95
|
|
|
86
96
|
This command analyzes and plans first.
|
|
@@ -122,3 +132,9 @@ Create an agent team for refactoring:
|
|
|
122
132
|
- Teammate 2 (backend-specialist): "Implement the refactoring changes identified by the reviewer." Use Opus.
|
|
123
133
|
Teammate 1 completes first, then Teammate 2 acts on the plan.
|
|
124
134
|
```
|
|
135
|
+
|
|
136
|
+
## Related Skills
|
|
137
|
+
- Need a safe refactor plan? → `/refactor-plan` for incremental steps as GitHub RFC
|
|
138
|
+
- Want to validate architecture? → `/analyze` for code quality metrics
|
|
139
|
+
- Need tests before refactoring? → `/tdd` to build safety net first
|
|
140
|
+
- Architecture decision needed? → `/council` for multi-perspective evaluation
|