@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.
Files changed (49) hide show
  1. package/CHANGELOG.md +41 -0
  2. package/README.md +62 -16
  3. package/app/agents/backend-specialist.md +8 -0
  4. package/app/agents/code-reviewer.md +9 -0
  5. package/app/agents/database-architect.md +8 -0
  6. package/app/agents/debugger.md +8 -0
  7. package/app/agents/devops-implementer.md +8 -0
  8. package/app/agents/documenter.md +8 -0
  9. package/app/agents/frontend-specialist.md +8 -0
  10. package/app/agents/performance-optimizer.md +8 -0
  11. package/app/agents/security-auditor.md +9 -0
  12. package/app/agents/test-engineer.md +9 -0
  13. package/app/skills/analyze/SKILL.md +15 -0
  14. package/app/skills/api-patterns/SKILL.md +10 -0
  15. package/app/skills/ci-cd-patterns/SKILL.md +10 -0
  16. package/app/skills/clean-code/SKILL.md +10 -0
  17. package/app/skills/database-patterns/SKILL.md +10 -0
  18. package/app/skills/debug/SKILL.md +16 -0
  19. package/app/skills/docs/SKILL.md +16 -0
  20. package/app/skills/git-mastery/SKILL.md +10 -0
  21. package/app/skills/onboard/SKILL.md +15 -0
  22. package/app/skills/performance-profiling/SKILL.md +10 -0
  23. package/app/skills/plan/SKILL.md +16 -0
  24. package/app/skills/refactor/SKILL.md +16 -0
  25. package/app/skills/review/SKILL.md +58 -3
  26. package/app/skills/security-patterns/SKILL.md +10 -0
  27. package/app/skills/tdd/SKILL.md +6 -0
  28. package/app/skills/testing-patterns/SKILL.md +10 -0
  29. package/bin/ai-toolkit.js +33 -5
  30. package/kb/procedures/release-verification-sop.md +283 -0
  31. package/kb/reference/architecture-overview.md +36 -7
  32. package/kb/reference/competitive-features-implementation.md +51 -52
  33. package/kb/reference/language-rules.md +18 -4
  34. package/kb/reference/skills-catalog.md +57 -1
  35. package/llms-full.txt +451 -64
  36. package/llms.txt +1 -0
  37. package/manifest.json +1 -1
  38. package/package.json +4 -2
  39. package/scripts/dir_rules_shared.py +441 -0
  40. package/scripts/generate_antigravity.py +36 -0
  41. package/scripts/generate_augment_rules.py +107 -0
  42. package/scripts/generate_cline_rules.py +31 -0
  43. package/scripts/generate_conventions.py +37 -0
  44. package/scripts/generate_cursor_mdc.py +127 -0
  45. package/scripts/generate_roo_rules.py +30 -0
  46. package/scripts/generate_windsurf_rules.py +31 -0
  47. package/scripts/install.py +26 -1
  48. package/scripts/install_steps/ai_tools.py +149 -31
  49. 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 Augment, ready in 60 seconds.
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
  [![CI](https://github.com/softspark/ai-toolkit/actions/workflows/ci.yml/badge.svg)](https://github.com/softspark/ai-toolkit/actions/workflows/ci.yml)
6
6
  [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
7
7
  [![Skills](https://img.shields.io/badge/skills-91-brightgreen)](app/skills/)
8
8
  [![Agents](https://img.shields.io/badge/agents-44-blue)](app/agents/)
9
- [![Tests](https://img.shields.io/badge/tests-377%20passing-success)](tests/)
9
+ [![Tests](https://img.shields.io/badge/tests-408%20passing-success)](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 File | How | Scope |
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/windsurf/memories/global_rules.md` | `ai-toolkit install` | global |
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.md` | `ai-toolkit install --local` | project |
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 remains the primary platform with full feature support (agents, lifecycle hooks, safety constitution). Other platforms receive a distilled ruleset generated from the same source. For editors lacking native bash lifecycle hooks, `--local` installs a Git hooks fallback (`.git/hooks/pre-commit`) to enforce quality gates pre-commit.
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 (via package.json, go.mod, Cargo.toml, etc.) no need for `--auto-detect`. Rules are injected into `CLAUDE.md` and auto-updated on `ai-toolkit update --local`.
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
- | `augment-rules` | Generate `.augment/rules/ai-toolkit.md` in current dir |
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 # also set up project-local configs (CLAUDE.md, settings, constitution, Copilot, Cline, Roo, Aider, Git Hooks, MCP Defaults)
739
- ai-toolkit update --local # re-apply + refresh project-local configs
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:
@@ -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`
@@ -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
@@ -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 |
@@ -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