@thierrynakoa/fire-flow 10.0.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/.claude-plugin/plugin.json +64 -0
- package/ARCHITECTURE-DIAGRAM.md +440 -0
- package/COMMAND-REFERENCE.md +172 -0
- package/DOMINION-FLOW-OVERVIEW.md +421 -0
- package/LICENSE +21 -0
- package/QUICK-START.md +351 -0
- package/README.md +398 -0
- package/TROUBLESHOOTING.md +264 -0
- package/agents/fire-codebase-mapper.md +484 -0
- package/agents/fire-debugger.md +535 -0
- package/agents/fire-executor.md +949 -0
- package/agents/fire-fact-checker.md +276 -0
- package/agents/fire-learncoding-explainer.md +237 -0
- package/agents/fire-learncoding-walker.md +147 -0
- package/agents/fire-planner.md +675 -0
- package/agents/fire-project-researcher.md +155 -0
- package/agents/fire-research-synthesizer.md +166 -0
- package/agents/fire-researcher.md +723 -0
- package/agents/fire-reviewer.md +499 -0
- package/agents/fire-roadmapper.md +203 -0
- package/agents/fire-verifier.md +880 -0
- package/bin/cli.js +208 -0
- package/commands/fire-0-orient.md +476 -0
- package/commands/fire-1-new.md +281 -0
- package/commands/fire-1a-discuss.md +455 -0
- package/commands/fire-2-plan.md +527 -0
- package/commands/fire-3-execute.md +1303 -0
- package/commands/fire-4-verify.md +845 -0
- package/commands/fire-5-handoff.md +515 -0
- package/commands/fire-6-resume.md +501 -0
- package/commands/fire-7-review.md +409 -0
- package/commands/fire-add-new-skill.md +598 -0
- package/commands/fire-analytics.md +499 -0
- package/commands/fire-assumptions.md +78 -0
- package/commands/fire-autonomous.md +528 -0
- package/commands/fire-brainstorm.md +413 -0
- package/commands/fire-complete-milestone.md +270 -0
- package/commands/fire-dashboard.md +375 -0
- package/commands/fire-debug.md +663 -0
- package/commands/fire-discover.md +616 -0
- package/commands/fire-double-check.md +460 -0
- package/commands/fire-execute-plan.md +182 -0
- package/commands/fire-learncoding.md +242 -0
- package/commands/fire-loop-resume.md +272 -0
- package/commands/fire-loop-stop.md +198 -0
- package/commands/fire-loop.md +1168 -0
- package/commands/fire-map-codebase.md +313 -0
- package/commands/fire-new-milestone.md +356 -0
- package/commands/fire-reflect.md +235 -0
- package/commands/fire-research.md +246 -0
- package/commands/fire-search.md +330 -0
- package/commands/fire-security-audit-repo.md +293 -0
- package/commands/fire-security-scan.md +484 -0
- package/commands/fire-session-summary.md +252 -0
- package/commands/fire-skills-diff.md +506 -0
- package/commands/fire-skills-history.md +388 -0
- package/commands/fire-skills-rollback.md +408 -0
- package/commands/fire-skills-sync.md +470 -0
- package/commands/fire-test.md +520 -0
- package/commands/fire-todos.md +335 -0
- package/commands/fire-transition.md +186 -0
- package/commands/fire-update.md +312 -0
- package/commands/fire-verify-uat.md +146 -0
- package/commands/fire-vuln-scan.md +493 -0
- package/hooks/hooks.json +16 -0
- package/hooks/run-hook.cmd +69 -0
- package/hooks/run-hook.sh +8 -0
- package/hooks/run-session-end.cmd +49 -0
- package/hooks/run-session-end.sh +7 -0
- package/hooks/session-end.sh +90 -0
- package/hooks/session-start.sh +111 -0
- package/package.json +52 -0
- package/plugin.json +7 -0
- package/references/auto-skill-extraction.md +136 -0
- package/references/behavioral-directives.md +365 -0
- package/references/blocker-tracking.md +155 -0
- package/references/checkpoints.md +165 -0
- package/references/circuit-breaker.md +410 -0
- package/references/context-engineering.md +587 -0
- package/references/decision-time-guidance.md +289 -0
- package/references/error-classification.md +326 -0
- package/references/execution-mode-intelligence.md +242 -0
- package/references/git-integration.md +217 -0
- package/references/honesty-protocols.md +304 -0
- package/references/integration-architecture.md +470 -0
- package/references/issue-to-pr-pipeline.md +150 -0
- package/references/metrics-and-trends.md +234 -0
- package/references/playwright-e2e-testing.md +326 -0
- package/references/questioning.md +125 -0
- package/references/research-improvements.md +110 -0
- package/references/skills-usage-guide.md +429 -0
- package/references/tdd.md +131 -0
- package/references/testing-enforcement.md +192 -0
- package/references/ui-brand.md +383 -0
- package/references/validation-checklist.md +456 -0
- package/references/verification-patterns.md +187 -0
- package/references/warrior-principles.md +173 -0
- package/skills-library/SKILLS-INDEX.md +588 -0
- package/skills-library/_general/frontend/html-visual-reports.md +292 -0
- package/skills-library/_general/methodology/debug-swarm-researcher-escape-hatch.md +240 -0
- package/skills-library/_general/methodology/learncoding-agentic-pattern.md +114 -0
- package/skills-library/_general/methodology/shell-autonomous-loop-fixplan.md +238 -0
- package/skills-library/basics/api-rest-basics.md +162 -0
- package/skills-library/basics/env-variables.md +96 -0
- package/skills-library/basics/error-handling-basics.md +125 -0
- package/skills-library/basics/git-commit-conventions.md +106 -0
- package/skills-library/basics/readme-template.md +108 -0
- package/skills-library/common-tasks/async-await-patterns.md +157 -0
- package/skills-library/common-tasks/auth-jwt-basics.md +164 -0
- package/skills-library/common-tasks/database-schema-design.md +166 -0
- package/skills-library/common-tasks/file-upload-basics.md +166 -0
- package/skills-library/common-tasks/form-validation.md +159 -0
- package/skills-library/debugging/FAILURE_TAXONOMY_CLASSIFICATION.md +117 -0
- package/skills-library/debugging/THREE_AGENT_HYPOTHESIS_DEBUGGING.md +86 -0
- package/skills-library/methodology/BREATH_BASED_PARALLEL_EXECUTION.md +678 -0
- package/skills-library/methodology/CONFIDENCE_GATED_EXECUTION.md +243 -0
- package/skills-library/methodology/EVIDENCE_BASED_VALIDATION.md +308 -0
- package/skills-library/methodology/MULTI_PERSPECTIVE_CODE_REVIEW.md +330 -0
- package/skills-library/methodology/PATH_VERIFICATION_GATE.md +211 -0
- package/skills-library/methodology/REFLEXION_MEMORY_PATTERN.md +183 -0
- package/skills-library/methodology/RESEARCH_BACKED_WORKFLOW_UPGRADE.md +263 -0
- package/skills-library/methodology/SABBATH_REST_PATTERN.md +267 -0
- package/skills-library/methodology/STONE_AND_SCAFFOLD.md +220 -0
- package/skills-library/performance/cache-augmented-generation.md +172 -0
- package/skills-library/quality-safety/debugging-steps.md +147 -0
- package/skills-library/quality-safety/deployment-checklist.md +155 -0
- package/skills-library/quality-safety/security-checklist.md +204 -0
- package/skills-library/quality-safety/testing-basics.md +180 -0
- package/skills-library/security/agent-security-scanner.md +445 -0
- package/skills-library/specialists/api-architecture/api-designer.md +49 -0
- package/skills-library/specialists/api-architecture/graphql-architect.md +49 -0
- package/skills-library/specialists/api-architecture/mcp-developer.md +51 -0
- package/skills-library/specialists/api-architecture/microservices-architect.md +50 -0
- package/skills-library/specialists/api-architecture/websocket-engineer.md +48 -0
- package/skills-library/specialists/backend/django-expert.md +52 -0
- package/skills-library/specialists/backend/fastapi-expert.md +52 -0
- package/skills-library/specialists/backend/laravel-specialist.md +52 -0
- package/skills-library/specialists/backend/nestjs-expert.md +51 -0
- package/skills-library/specialists/backend/rails-expert.md +53 -0
- package/skills-library/specialists/backend/spring-boot-engineer.md +56 -0
- package/skills-library/specialists/data-ml/fine-tuning-expert.md +48 -0
- package/skills-library/specialists/data-ml/ml-pipeline.md +47 -0
- package/skills-library/specialists/data-ml/pandas-pro.md +47 -0
- package/skills-library/specialists/data-ml/rag-architect.md +51 -0
- package/skills-library/specialists/data-ml/spark-engineer.md +47 -0
- package/skills-library/specialists/frontend/angular-architect.md +52 -0
- package/skills-library/specialists/frontend/flutter-expert.md +51 -0
- package/skills-library/specialists/frontend/nextjs-developer.md +54 -0
- package/skills-library/specialists/frontend/react-native-expert.md +50 -0
- package/skills-library/specialists/frontend/vue-expert.md +51 -0
- package/skills-library/specialists/infrastructure/chaos-engineer.md +74 -0
- package/skills-library/specialists/infrastructure/cloud-architect.md +70 -0
- package/skills-library/specialists/infrastructure/database-optimizer.md +64 -0
- package/skills-library/specialists/infrastructure/devops-engineer.md +70 -0
- package/skills-library/specialists/infrastructure/kubernetes-specialist.md +52 -0
- package/skills-library/specialists/infrastructure/monitoring-expert.md +70 -0
- package/skills-library/specialists/infrastructure/sre-engineer.md +70 -0
- package/skills-library/specialists/infrastructure/terraform-engineer.md +51 -0
- package/skills-library/specialists/languages/cpp-pro.md +74 -0
- package/skills-library/specialists/languages/csharp-developer.md +69 -0
- package/skills-library/specialists/languages/dotnet-core-expert.md +54 -0
- package/skills-library/specialists/languages/golang-pro.md +51 -0
- package/skills-library/specialists/languages/java-architect.md +49 -0
- package/skills-library/specialists/languages/javascript-pro.md +68 -0
- package/skills-library/specialists/languages/kotlin-specialist.md +68 -0
- package/skills-library/specialists/languages/php-pro.md +49 -0
- package/skills-library/specialists/languages/python-pro.md +52 -0
- package/skills-library/specialists/languages/react-expert.md +51 -0
- package/skills-library/specialists/languages/rust-engineer.md +50 -0
- package/skills-library/specialists/languages/sql-pro.md +56 -0
- package/skills-library/specialists/languages/swift-expert.md +69 -0
- package/skills-library/specialists/languages/typescript-pro.md +51 -0
- package/skills-library/specialists/platform/atlassian-mcp.md +52 -0
- package/skills-library/specialists/platform/embedded-systems.md +53 -0
- package/skills-library/specialists/platform/game-developer.md +53 -0
- package/skills-library/specialists/platform/salesforce-developer.md +53 -0
- package/skills-library/specialists/platform/shopify-expert.md +49 -0
- package/skills-library/specialists/platform/wordpress-pro.md +49 -0
- package/skills-library/specialists/quality/code-documenter.md +51 -0
- package/skills-library/specialists/quality/code-reviewer.md +67 -0
- package/skills-library/specialists/quality/debugging-wizard.md +51 -0
- package/skills-library/specialists/quality/fullstack-guardian.md +51 -0
- package/skills-library/specialists/quality/legacy-modernizer.md +50 -0
- package/skills-library/specialists/quality/playwright-expert.md +65 -0
- package/skills-library/specialists/quality/spec-miner.md +56 -0
- package/skills-library/specialists/quality/test-master.md +65 -0
- package/skills-library/specialists/security/secure-code-guardian.md +55 -0
- package/skills-library/specialists/security/security-reviewer.md +53 -0
- package/skills-library/specialists/workflow/architecture-designer.md +53 -0
- package/skills-library/specialists/workflow/cli-developer.md +70 -0
- package/skills-library/specialists/workflow/feature-forge.md +65 -0
- package/skills-library/specialists/workflow/prompt-engineer.md +54 -0
- package/skills-library/specialists/workflow/the-fool.md +62 -0
- package/templates/ASSUMPTIONS.md +125 -0
- package/templates/BLOCKERS.md +73 -0
- package/templates/DECISION_LOG.md +116 -0
- package/templates/UAT.md +96 -0
- package/templates/blueprint.md +94 -0
- package/templates/brainstorm.md +185 -0
- package/templates/conscience.md +92 -0
- package/templates/fire-handoff.md +159 -0
- package/templates/metrics.md +67 -0
- package/templates/phase-prompt.md +142 -0
- package/templates/record.md +131 -0
- package/templates/review-report.md +117 -0
- package/templates/skills-index.md +157 -0
- package/templates/verification.md +149 -0
- package/templates/vision.md +79 -0
- package/validation-config.yml +793 -0
- package/version.json +7 -0
- package/workflows/execute-phase.md +732 -0
- package/workflows/handoff-session.md +678 -0
- package/workflows/new-project.md +578 -0
- package/workflows/plan-phase.md +592 -0
- package/workflows/verify-phase.md +874 -0
|
@@ -0,0 +1,110 @@
|
|
|
1
|
+
# Research-Backed Improvements Reference
|
|
2
|
+
|
|
3
|
+
> Dominion Flow v3.2 — Research sources, gap analysis findings, and implementation mapping
|
|
4
|
+
>
|
|
5
|
+
> Every v3.2 enhancement cites a tag from this document (e.g., `GAP-1`, `RECITATION-1`).
|
|
6
|
+
> Use this file to trace any inline comment back to its research basis.
|
|
7
|
+
|
|
8
|
+
---
|
|
9
|
+
|
|
10
|
+
## Research Sources (2024–2026)
|
|
11
|
+
|
|
12
|
+
### Academic Papers
|
|
13
|
+
|
|
14
|
+
| ID | Paper | Key Finding | Applied In |
|
|
15
|
+
|----|-------|-------------|------------|
|
|
16
|
+
| PLAN-DEBUG-1 | MapCoder (ACL 2024) — "Multi-Agent Code Generation through Planning" | Achieved 93.9% pass@1 by feeding the Debugging Agent the original plan alongside buggy code. Plan-derived debugging significantly outperforms debugging without plan context because the agent can compare "what was intended" vs "what happened." | `/fire-debug` Step 4: `<plan_context>` block |
|
|
17
|
+
| RECITATION-1 | Manus AI — Context Engineering (2025) | Without task recitation, agents drift off-topic after ~50 tool calls. Rewriting a compressed todo at the end of context keeps the current plan in the model's recent attention span, preventing the "lost-in-the-middle" problem (transformer attention bias toward recent tokens). | `/fire-loop` Step 7 rule 6, iteration context injection |
|
|
18
|
+
| OBSERVE-1 | Mastra — Observational Memory (2025) | Two-agent (Observer/Reflector) compression achieves 5-40x compression with 10x cost reduction over RAG. Observer extracts key facts; Reflector synthesizes patterns. | Future: Tier 2 enhancement |
|
|
19
|
+
| JUDGE-1 | Mason (2026) — "Judge Agent Separation" | Dedicated Judge agent that only evaluates (never generates) prevents self-evaluation bias. Separating generation from evaluation improves quality scoring accuracy. | Future: Tier 3 enhancement |
|
|
20
|
+
| SWE-1 | SWE-Bench Pro (2025) | Single agent + retries outperforms multi-agent swarms for most routine tasks. Multi-agent overhead only justified for genuinely parallel work. | Informs execution mode intelligence (SWARM threshold) |
|
|
21
|
+
| METR-1 | METR Study (2025) — "AI Impact on Developer Productivity" | AI can slow experienced devs 19% on familiar code. AI excels at exploration and unfamiliar codebases, not well-known code. | Informs when to apply AI vs manual work |
|
|
22
|
+
| RLCR-1 | MIT RLCR (2025) — "Confidence-Based Escalation" | Route low-confidence items to human review instead of auto-deciding. Confidence thresholds improve decision quality across automated workflows. | Future: Tier 2 enhancement |
|
|
23
|
+
|
|
24
|
+
### Community Patterns
|
|
25
|
+
|
|
26
|
+
| ID | Source | Pattern | Applied In |
|
|
27
|
+
|----|--------|---------|------------|
|
|
28
|
+
| CNCF-1 | CNCF Four Pillars (2025) | Golden Paths, Guardrails, Safety Nets, Manual Review — taxonomy for organizing validation checklists by enforcement level. | Future: restructure 70-point checklist |
|
|
29
|
+
| BOLT-1 | Bolt.new — .powerignore (2025) | File-level exclusion patterns prevent context pollution from generated files, node_modules, etc. | v3.0: `.powerignore` in context engineering |
|
|
30
|
+
| RALPH-1 | Ralph Wiggum Loops — Circuit Breaker Fork | Stall/spin/degradation detection with automatic recovery. Prevents infinite loops and wasted iterations. | v3.0: Circuit Breaker system |
|
|
31
|
+
| REPLIT-1 | Replit Agent — Decision-Time Guidance (2025) | Micro-inject relevant skills at the moment of decision rather than front-loading all knowledge. Reduces context bloat while maintaining quality. | v3.0: DTG system |
|
|
32
|
+
|
|
33
|
+
---
|
|
34
|
+
|
|
35
|
+
## Gap Analysis Findings
|
|
36
|
+
|
|
37
|
+
> Internal analysis of Dominion Flow v3.1 identified these gaps.
|
|
38
|
+
> Each gap has a severity, the v3.2 fix applied, and the file modified.
|
|
39
|
+
|
|
40
|
+
### GAP Series (Structural Gaps)
|
|
41
|
+
|
|
42
|
+
| ID | Gap Description | Severity | v3.2 Fix | File Modified |
|
|
43
|
+
|----|----------------|----------|----------|---------------|
|
|
44
|
+
| GAP-1 | No explicit decision log — architectural decisions scattered across handoffs and CONSCIENCE.md with no central tracking | HIGH | Created `DECISION_LOG.md` template with DEC-XXX format, cross-phase impact tracking, and integration points | `templates/DECISION_LOG.md` (new) |
|
|
45
|
+
| GAP-2 | Assumptions accumulate across phases without systematic tracking — Phase 6 can contradict Phase 1 assumptions | HIGH | Enhanced `ASSUMPTIONS.md` with cross-phase contradiction detection, phase-gate validation protocol | `templates/ASSUMPTIONS.md` |
|
|
46
|
+
| GAP-3 | No cross-phase contradiction detection for decisions | MEDIUM | Decision Log includes "Affected Phases" and cross-phase validation protocol | `templates/DECISION_LOG.md` |
|
|
47
|
+
| GAP-4 | Deferred items multiply without compound impact tracking | MEDIUM | Added deferred-item impact analysis template (DEFERRED-XXX format) to Assumptions | `templates/ASSUMPTIONS.md` |
|
|
48
|
+
| GAP-5 | Handoff completeness not validated — broken context chains if any session skips proper handoff | HIGH | Added 17-point automated validation checklist as Step 5 in handoff command | `commands/fire-5-handoff.md` |
|
|
49
|
+
| GAP-6 | Task recitation missing — agents drift after many iterations in fire-loop | HIGH | Added recitation pattern to iteration context injection with compressed todo rewrite | `commands/fire-loop.md` |
|
|
50
|
+
| GAP-7 | Debug loops lack plan context — debugger can't compare intended vs actual behavior | MEDIUM | Added `<plan_context>` block to debugger spawn and continuation prompts | `commands/fire-debug.md` |
|
|
51
|
+
| GAP-8 | CONSCIENCE.md doesn't reference Decision Log | LOW | Added Decision Log reference and Assumptions Health section to state template | `templates/state.md` |
|
|
52
|
+
| GAP-9 | Handoff success criteria don't include Decision Log or Assumptions sync | LOW | Added 3 new items to handoff success criteria | `commands/fire-5-handoff.md` |
|
|
53
|
+
| GAP-10 | No executable handoff validation — relies on manual review | HIGH | 17-point checklist with structural, content, and cross-reference checks | `commands/fire-5-handoff.md` |
|
|
54
|
+
|
|
55
|
+
### BLIND-SPOT Series (Discovered During Analysis)
|
|
56
|
+
|
|
57
|
+
| ID | Blind Spot | Impact | v3.2 Fix |
|
|
58
|
+
|----|-----------|--------|----------|
|
|
59
|
+
| BLIND-SPOT-A | Decisions made in one phase can silently conflict with decisions from earlier phases | Cross-phase bugs, wasted work | Decision Log cross-phase validation protocol |
|
|
60
|
+
| BLIND-SPOT-B | Unvalidated assumptions from early phases relied upon in later phases without re-check | Architecture built on unverified foundations | Phase-gate validation: mandatory assumption audit at phase start |
|
|
61
|
+
| BLIND-SPOT-C | Context drift in long fire-loop runs leads to repeated mistakes and circular attempts | Wasted iterations, degraded solutions | Task recitation pattern forces model to re-read compressed state |
|
|
62
|
+
| BLIND-SPOT-D | Handoff chain breaks permanently if one session skips proper handoff | Complete context loss, restart required | Automated completeness validation prevents saving incomplete handoffs |
|
|
63
|
+
|
|
64
|
+
---
|
|
65
|
+
|
|
66
|
+
## Implementation Mapping
|
|
67
|
+
|
|
68
|
+
### Tier 1 — Implemented in v3.2.0
|
|
69
|
+
|
|
70
|
+
| Enhancement | Research Basis | Files Changed |
|
|
71
|
+
|-------------|---------------|---------------|
|
|
72
|
+
| Decision Log template | GAP-1, GAP-3, BLIND-SPOT-A | `templates/DECISION_LOG.md` (new) |
|
|
73
|
+
| Assumptions Registry enhancement | GAP-2, GAP-4, BLIND-SPOT-B | `templates/ASSUMPTIONS.md` |
|
|
74
|
+
| CONSCIENCE.md Decision/Assumptions references | GAP-8 | `templates/state.md` |
|
|
75
|
+
| Handoff Completeness Validator | GAP-5, GAP-9, GAP-10, BLIND-SPOT-D | `commands/fire-5-handoff.md` |
|
|
76
|
+
| Task Recitation Pattern | RECITATION-1, GAP-6, BLIND-SPOT-C | `commands/fire-loop.md` |
|
|
77
|
+
| Plan-Aware Debugging | PLAN-DEBUG-1, GAP-7 | `commands/fire-debug.md` |
|
|
78
|
+
| Plugin version bump + this doc | — | `plugin.json`, `references/research-improvements.md` |
|
|
79
|
+
|
|
80
|
+
### Tier 2 — Planned for v3.3.0
|
|
81
|
+
|
|
82
|
+
| Enhancement | Research Basis | Target |
|
|
83
|
+
|-------------|---------------|--------|
|
|
84
|
+
| Observational Memory (Observer/Reflector agents) | OBSERVE-1 | Handoff compression, cross-session learning |
|
|
85
|
+
| Confidence-Based Escalation | RLCR-1 | Route uncertain decisions to user during execution |
|
|
86
|
+
| CNCF Four Pillars restructure | CNCF-1 | Reorganize 70-point checklist by enforcement level |
|
|
87
|
+
| METR-aware task routing | METR-1 | Route AI to exploration tasks, manual for familiar code |
|
|
88
|
+
|
|
89
|
+
### Tier 3 — Planned for v3.4.0
|
|
90
|
+
|
|
91
|
+
| Enhancement | Research Basis | Target |
|
|
92
|
+
|-------------|---------------|--------|
|
|
93
|
+
| Judge Agent separation | JUDGE-1 | Dedicated evaluation agent for verification |
|
|
94
|
+
| SWE-Bench calibrated execution modes | SWE-1 | Better SWARM vs SEQUENTIAL thresholds |
|
|
95
|
+
| Skills Library auto-pruning | Community patterns | Remove unused skills, version popular ones |
|
|
96
|
+
|
|
97
|
+
---
|
|
98
|
+
|
|
99
|
+
## Version History
|
|
100
|
+
|
|
101
|
+
| Version | Date | Key Additions |
|
|
102
|
+
|---------|------|---------------|
|
|
103
|
+
| v3.0.0 | 2026-02-09 | Dominion Flow consolidation, Circuit Breaker, Error Classification, Context Engineering, DTG |
|
|
104
|
+
| v3.1.0 | 2026-02-10 | Playwright E2E testing (Step 8), 70-point validation, visual regression |
|
|
105
|
+
| v3.2.0 | 2026-02-10 | Research-backed: Task Recitation, Plan-Aware Debug, Decision Log, Assumptions enhancement, Handoff Validator, Code Comments Standard |
|
|
106
|
+
|
|
107
|
+
---
|
|
108
|
+
|
|
109
|
+
*Created: 2026-02-10*
|
|
110
|
+
*Source: 4-agent parallel research sweep (15 papers + 15 community patterns + 10 testing findings + 10 internal gaps)*
|
|
@@ -0,0 +1,429 @@
|
|
|
1
|
+
# Skills Usage Guide
|
|
2
|
+
|
|
3
|
+
> How agents discover, use, and contribute to the skills library
|
|
4
|
+
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
## Overview
|
|
8
|
+
|
|
9
|
+
The skills library is a curated collection of implementation patterns, methodologies, and domain knowledge. Agents reference skills during planning and execution to ensure consistent, high-quality implementations.
|
|
10
|
+
|
|
11
|
+
---
|
|
12
|
+
|
|
13
|
+
## Searching the Skills Library
|
|
14
|
+
|
|
15
|
+
### Using /fire-search
|
|
16
|
+
|
|
17
|
+
The primary way to find relevant skills:
|
|
18
|
+
|
|
19
|
+
```
|
|
20
|
+
/fire-search authentication JWT tokens
|
|
21
|
+
```
|
|
22
|
+
|
|
23
|
+
**Search Behavior:**
|
|
24
|
+
1. Searches skill titles and descriptions
|
|
25
|
+
2. Searches skill content for keywords
|
|
26
|
+
3. Returns ranked results by relevance
|
|
27
|
+
4. Shows skill category and file path
|
|
28
|
+
|
|
29
|
+
**Example Output:**
|
|
30
|
+
```
|
|
31
|
+
━━━ DOMINION FLOW > SKILL SEARCH ━━━
|
|
32
|
+
|
|
33
|
+
Query: "authentication JWT tokens"
|
|
34
|
+
|
|
35
|
+
Results (3 matches):
|
|
36
|
+
|
|
37
|
+
1. [HIGH] authentication/jwt-implementation.md
|
|
38
|
+
JWT token generation, validation, and refresh patterns
|
|
39
|
+
Category: security
|
|
40
|
+
|
|
41
|
+
2. [MED] authentication/session-management.md
|
|
42
|
+
Session handling with various token strategies
|
|
43
|
+
Category: security
|
|
44
|
+
|
|
45
|
+
3. [LOW] api-design/protected-routes.md
|
|
46
|
+
Securing API routes with middleware
|
|
47
|
+
Category: api-design
|
|
48
|
+
|
|
49
|
+
Use: Read skill with /fire-skill authentication/jwt-implementation
|
|
50
|
+
```
|
|
51
|
+
|
|
52
|
+
### Search Tips
|
|
53
|
+
|
|
54
|
+
| Goal | Search Query |
|
|
55
|
+
|------|-------------|
|
|
56
|
+
| Find implementation patterns | `"pattern" + domain` (e.g., "pattern repository") |
|
|
57
|
+
| Find security guidance | `"security" + topic` (e.g., "security file upload") |
|
|
58
|
+
| Find database patterns | `"database" + operation` (e.g., "database migration") |
|
|
59
|
+
| Find testing approaches | `"test" + type` (e.g., "test integration API") |
|
|
60
|
+
|
|
61
|
+
### Browsing by Category
|
|
62
|
+
|
|
63
|
+
Skills are organized into categories:
|
|
64
|
+
|
|
65
|
+
```
|
|
66
|
+
skills-library/
|
|
67
|
+
├── methodology/ # Development processes
|
|
68
|
+
├── complexity-metrics/ # Effort estimation
|
|
69
|
+
├── ecommerce/ # E-commerce patterns
|
|
70
|
+
├── integrations/ # Third-party integrations
|
|
71
|
+
├── video-media/ # Media processing
|
|
72
|
+
├── deployment-security/ # DevOps and security
|
|
73
|
+
├── database-solutions/ # Database patterns
|
|
74
|
+
├── form-solutions/ # Form handling
|
|
75
|
+
├── advanced-features/ # Complex implementations
|
|
76
|
+
├── automation/ # CI/CD, scripting
|
|
77
|
+
├── document-processing/ # File handling
|
|
78
|
+
└── patterns-standards/ # General patterns
|
|
79
|
+
```
|
|
80
|
+
|
|
81
|
+
---
|
|
82
|
+
|
|
83
|
+
## Referencing Skills in Plans
|
|
84
|
+
|
|
85
|
+
### BLUEPRINT.md Frontmatter
|
|
86
|
+
|
|
87
|
+
When creating execution plans, reference relevant skills:
|
|
88
|
+
|
|
89
|
+
```yaml
|
|
90
|
+
---
|
|
91
|
+
phase: 3
|
|
92
|
+
task: "Implement user authentication"
|
|
93
|
+
skills_referenced:
|
|
94
|
+
- authentication/jwt-implementation.md
|
|
95
|
+
- database-solutions/user-schema.md
|
|
96
|
+
- testing/auth-test-patterns.md
|
|
97
|
+
complexity: medium
|
|
98
|
+
estimated_breaths: 3
|
|
99
|
+
---
|
|
100
|
+
```
|
|
101
|
+
|
|
102
|
+
### Skill Citation in Plan Body
|
|
103
|
+
|
|
104
|
+
Reference skills when describing implementation approach:
|
|
105
|
+
|
|
106
|
+
```markdown
|
|
107
|
+
## Implementation Approach
|
|
108
|
+
|
|
109
|
+
Following the JWT implementation pattern from `authentication/jwt-implementation.md`:
|
|
110
|
+
|
|
111
|
+
1. **Token Generation** (per skill section 2.1)
|
|
112
|
+
- Use RS256 algorithm for production
|
|
113
|
+
- Include standard claims (iss, sub, exp, iat)
|
|
114
|
+
- Add custom claims for user roles
|
|
115
|
+
|
|
116
|
+
2. **Token Validation** (per skill section 2.2)
|
|
117
|
+
- Validate signature before claims
|
|
118
|
+
- Check expiration with clock skew tolerance
|
|
119
|
+
- Verify issuer matches expected value
|
|
120
|
+
```
|
|
121
|
+
|
|
122
|
+
### Inline References
|
|
123
|
+
|
|
124
|
+
During implementation details:
|
|
125
|
+
|
|
126
|
+
```markdown
|
|
127
|
+
### Database Schema
|
|
128
|
+
|
|
129
|
+
Per `database-solutions/user-schema.md`:
|
|
130
|
+
- Use UUID for primary keys
|
|
131
|
+
- Include audit timestamps
|
|
132
|
+
- Soft delete with deletedAt column
|
|
133
|
+
|
|
134
|
+
### Password Handling
|
|
135
|
+
|
|
136
|
+
Per `security/password-hashing.md`:
|
|
137
|
+
- Use bcrypt with cost factor 12
|
|
138
|
+
- Never store plaintext
|
|
139
|
+
- Implement password strength validation
|
|
140
|
+
```
|
|
141
|
+
|
|
142
|
+
---
|
|
143
|
+
|
|
144
|
+
## Citing Skills in RECORD.md
|
|
145
|
+
|
|
146
|
+
### Skills Used Section
|
|
147
|
+
|
|
148
|
+
Always document which skills informed the implementation:
|
|
149
|
+
|
|
150
|
+
```markdown
|
|
151
|
+
## Skills Referenced
|
|
152
|
+
|
|
153
|
+
| Skill | Usage | Sections Used |
|
|
154
|
+
|-------|-------|---------------|
|
|
155
|
+
| `authentication/jwt-implementation.md` | Token generation and validation | 2.1, 2.2, 3.1 |
|
|
156
|
+
| `database-solutions/user-schema.md` | User table design | Full |
|
|
157
|
+
| `testing/auth-test-patterns.md` | Test structure | 1.1, 2.3 |
|
|
158
|
+
|
|
159
|
+
### Deviations from Skills
|
|
160
|
+
|
|
161
|
+
1. **JWT Algorithm**: Used HS256 instead of RS256
|
|
162
|
+
- Reason: Simpler key management for MVP
|
|
163
|
+
- Trade-off: Less suitable for distributed systems
|
|
164
|
+
- Future: Migrate to RS256 when scaling
|
|
165
|
+
```
|
|
166
|
+
|
|
167
|
+
### Knowledge Gaps Identified
|
|
168
|
+
|
|
169
|
+
Document when skills were missing:
|
|
170
|
+
|
|
171
|
+
```markdown
|
|
172
|
+
## Skills Gaps Identified
|
|
173
|
+
|
|
174
|
+
The following patterns would benefit from skills documentation:
|
|
175
|
+
|
|
176
|
+
1. **Rate Limiting with Redis**
|
|
177
|
+
- Needed: Distributed rate limiting patterns
|
|
178
|
+
- Currently: Implemented from scratch
|
|
179
|
+
- Recommendation: Create `security/rate-limiting-redis.md`
|
|
180
|
+
|
|
181
|
+
2. **GraphQL Subscriptions**
|
|
182
|
+
- Needed: WebSocket management patterns
|
|
183
|
+
- Currently: Following Apollo docs only
|
|
184
|
+
- Recommendation: Create `api-design/graphql-subscriptions.md`
|
|
185
|
+
```
|
|
186
|
+
|
|
187
|
+
---
|
|
188
|
+
|
|
189
|
+
## Contributing New Skills
|
|
190
|
+
|
|
191
|
+
### Using /fire-contribute
|
|
192
|
+
|
|
193
|
+
When you've implemented a pattern worth documenting:
|
|
194
|
+
|
|
195
|
+
```
|
|
196
|
+
/fire-contribute "Redis Rate Limiting Pattern"
|
|
197
|
+
```
|
|
198
|
+
|
|
199
|
+
**Contribution Flow:**
|
|
200
|
+
1. Describe the pattern you implemented
|
|
201
|
+
2. Answer classification questions
|
|
202
|
+
3. Review generated skill draft
|
|
203
|
+
4. Submit for library addition
|
|
204
|
+
|
|
205
|
+
### Skill Document Structure
|
|
206
|
+
|
|
207
|
+
```markdown
|
|
208
|
+
# [Skill Title]
|
|
209
|
+
|
|
210
|
+
> One-line description of what this skill covers
|
|
211
|
+
|
|
212
|
+
---
|
|
213
|
+
|
|
214
|
+
## Overview
|
|
215
|
+
|
|
216
|
+
Brief explanation of when to use this pattern and why.
|
|
217
|
+
|
|
218
|
+
## Prerequisites
|
|
219
|
+
|
|
220
|
+
- Required knowledge
|
|
221
|
+
- Dependencies
|
|
222
|
+
- Environment setup
|
|
223
|
+
|
|
224
|
+
## Implementation
|
|
225
|
+
|
|
226
|
+
### Step 1: [First Step]
|
|
227
|
+
|
|
228
|
+
Detailed instructions with code examples.
|
|
229
|
+
|
|
230
|
+
```typescript
|
|
231
|
+
// Example code
|
|
232
|
+
```
|
|
233
|
+
|
|
234
|
+
### Step 2: [Second Step]
|
|
235
|
+
|
|
236
|
+
Continue with clear, actionable steps.
|
|
237
|
+
|
|
238
|
+
## Variations
|
|
239
|
+
|
|
240
|
+
### Variation A: [Name]
|
|
241
|
+
|
|
242
|
+
When to use this variation and how it differs.
|
|
243
|
+
|
|
244
|
+
### Variation B: [Name]
|
|
245
|
+
|
|
246
|
+
Alternative approach for different requirements.
|
|
247
|
+
|
|
248
|
+
## Testing
|
|
249
|
+
|
|
250
|
+
How to verify the implementation works.
|
|
251
|
+
|
|
252
|
+
## Common Pitfalls
|
|
253
|
+
|
|
254
|
+
1. **Pitfall Name**: What goes wrong and how to avoid it
|
|
255
|
+
2. **Another Pitfall**: Description and prevention
|
|
256
|
+
|
|
257
|
+
## References
|
|
258
|
+
|
|
259
|
+
- External documentation links
|
|
260
|
+
- Related skills in the library
|
|
261
|
+
|
|
262
|
+
---
|
|
263
|
+
|
|
264
|
+
*Last updated: [Date] | Contributor: [Name/Agent]*
|
|
265
|
+
```
|
|
266
|
+
|
|
267
|
+
### Contribution Guidelines
|
|
268
|
+
|
|
269
|
+
1. **Be Specific**: Generic advice isn't helpful
|
|
270
|
+
2. **Include Code**: Show, don't just tell
|
|
271
|
+
3. **Test Your Pattern**: Only document what works
|
|
272
|
+
4. **Note Limitations**: Be honest about trade-offs
|
|
273
|
+
5. **Update Existing**: Improve skills rather than duplicating
|
|
274
|
+
|
|
275
|
+
---
|
|
276
|
+
|
|
277
|
+
## Syncing with Global Library
|
|
278
|
+
|
|
279
|
+
### Check for Updates
|
|
280
|
+
|
|
281
|
+
```
|
|
282
|
+
/fire-sync check
|
|
283
|
+
```
|
|
284
|
+
|
|
285
|
+
**Output:**
|
|
286
|
+
```
|
|
287
|
+
━━━ DOMINION FLOW > SKILL SYNC CHECK ━━━
|
|
288
|
+
|
|
289
|
+
Local Library: v2.3.1
|
|
290
|
+
Global Library: v2.4.0
|
|
291
|
+
|
|
292
|
+
Updates Available:
|
|
293
|
+
+ NEW: api-design/graphql-subscriptions.md
|
|
294
|
+
~ UPD: authentication/jwt-implementation.md (security patch)
|
|
295
|
+
~ UPD: database-solutions/migration-patterns.md (new examples)
|
|
296
|
+
|
|
297
|
+
Run /fire-sync pull to update
|
|
298
|
+
```
|
|
299
|
+
|
|
300
|
+
### Pull Updates
|
|
301
|
+
|
|
302
|
+
```
|
|
303
|
+
/fire-sync pull
|
|
304
|
+
```
|
|
305
|
+
|
|
306
|
+
**Behavior:**
|
|
307
|
+
1. Downloads new/updated skills
|
|
308
|
+
2. Preserves local modifications
|
|
309
|
+
3. Shows changelog for updates
|
|
310
|
+
4. Reports conflicts if any
|
|
311
|
+
|
|
312
|
+
### Push Contributions
|
|
313
|
+
|
|
314
|
+
```
|
|
315
|
+
/fire-sync push authentication/custom-sso.md
|
|
316
|
+
```
|
|
317
|
+
|
|
318
|
+
**Behavior:**
|
|
319
|
+
1. Validates skill format
|
|
320
|
+
2. Runs quality checks
|
|
321
|
+
3. Submits to global library
|
|
322
|
+
4. Tracks contribution credit
|
|
323
|
+
|
|
324
|
+
---
|
|
325
|
+
|
|
326
|
+
## Skill Discovery During Execution
|
|
327
|
+
|
|
328
|
+
### Automatic Suggestions
|
|
329
|
+
|
|
330
|
+
During planning, Dominion Flow may suggest relevant skills:
|
|
331
|
+
|
|
332
|
+
```
|
|
333
|
+
━━━ DOMINION FLOW > PLANNING ━━━
|
|
334
|
+
|
|
335
|
+
Analyzing task: "Add payment processing"
|
|
336
|
+
|
|
337
|
+
Suggested Skills:
|
|
338
|
+
├─ ecommerce/payment-integration.md
|
|
339
|
+
│ └─ Stripe, PayPal integration patterns
|
|
340
|
+
├─ security/pci-compliance.md
|
|
341
|
+
│ └─ Payment data handling requirements
|
|
342
|
+
└─ testing/payment-mocking.md
|
|
343
|
+
└─ How to test without real transactions
|
|
344
|
+
|
|
345
|
+
Review skills before proceeding? (y/n)
|
|
346
|
+
```
|
|
347
|
+
|
|
348
|
+
### Skill Quick Reference
|
|
349
|
+
|
|
350
|
+
During execution, quickly check a skill:
|
|
351
|
+
|
|
352
|
+
```
|
|
353
|
+
/fire-skill-quick authentication/jwt-implementation 2.1
|
|
354
|
+
```
|
|
355
|
+
|
|
356
|
+
**Output:**
|
|
357
|
+
```
|
|
358
|
+
━━━ JWT Implementation - Section 2.1 ━━━
|
|
359
|
+
|
|
360
|
+
Token Generation:
|
|
361
|
+
|
|
362
|
+
```typescript
|
|
363
|
+
import jwt from 'jsonwebtoken';
|
|
364
|
+
|
|
365
|
+
const generateToken = (user: User): string => {
|
|
366
|
+
return jwt.sign(
|
|
367
|
+
{
|
|
368
|
+
sub: user.id,
|
|
369
|
+
email: user.email,
|
|
370
|
+
roles: user.roles,
|
|
371
|
+
},
|
|
372
|
+
process.env.JWT_SECRET,
|
|
373
|
+
{
|
|
374
|
+
algorithm: 'RS256',
|
|
375
|
+
expiresIn: '15m',
|
|
376
|
+
issuer: 'your-app-name',
|
|
377
|
+
}
|
|
378
|
+
);
|
|
379
|
+
};
|
|
380
|
+
```
|
|
381
|
+
|
|
382
|
+
Key Points:
|
|
383
|
+
- Use short expiration (15m) with refresh tokens
|
|
384
|
+
- Include minimal claims (reduce token size)
|
|
385
|
+
- Always set issuer for validation
|
|
386
|
+
```
|
|
387
|
+
|
|
388
|
+
---
|
|
389
|
+
|
|
390
|
+
## Best Practices
|
|
391
|
+
|
|
392
|
+
### When to Search for Skills
|
|
393
|
+
|
|
394
|
+
1. **Before starting any new feature** - Check if patterns exist
|
|
395
|
+
2. **When encountering unfamiliar territory** - Learn from documented experience
|
|
396
|
+
3. **Before implementing security features** - Always use vetted patterns
|
|
397
|
+
4. **When estimating complexity** - Skills help gauge effort
|
|
398
|
+
|
|
399
|
+
### When to Create New Skills
|
|
400
|
+
|
|
401
|
+
1. **After implementing something novel** - Share the knowledge
|
|
402
|
+
2. **When you spent significant time researching** - Save others the effort
|
|
403
|
+
3. **When you found a better way** - Improve on existing patterns
|
|
404
|
+
4. **When you hit undocumented pitfalls** - Prevent others from same mistakes
|
|
405
|
+
|
|
406
|
+
### Skill Maintenance
|
|
407
|
+
|
|
408
|
+
1. **Update when patterns evolve** - Keep skills current
|
|
409
|
+
2. **Add examples from real implementations** - Concrete beats abstract
|
|
410
|
+
3. **Note deprecated approaches** - Warn about outdated patterns
|
|
411
|
+
4. **Cross-reference related skills** - Build a connected knowledge base
|
|
412
|
+
|
|
413
|
+
---
|
|
414
|
+
|
|
415
|
+
## Quick Command Reference
|
|
416
|
+
|
|
417
|
+
| Command | Purpose |
|
|
418
|
+
|---------|---------|
|
|
419
|
+
| `/fire-search <query>` | Find relevant skills |
|
|
420
|
+
| `/fire-skill <path>` | Read full skill document |
|
|
421
|
+
| `/fire-skill-quick <path> <section>` | Read specific section |
|
|
422
|
+
| `/fire-contribute <title>` | Start new skill contribution |
|
|
423
|
+
| `/fire-sync check` | Check for library updates |
|
|
424
|
+
| `/fire-sync pull` | Download library updates |
|
|
425
|
+
| `/fire-sync push <path>` | Submit skill contribution |
|
|
426
|
+
|
|
427
|
+
---
|
|
428
|
+
|
|
429
|
+
*The skills library grows with every implementation. Contribute what you learn.*
|
|
@@ -0,0 +1,131 @@
|
|
|
1
|
+
# Dominion Flow TDD Reference
|
|
2
|
+
|
|
3
|
+
> **Origin:** Ported from Dominion Flow `tdd.md` with Dominion Flow Git Flow integration.
|
|
4
|
+
|
|
5
|
+
## Overview
|
|
6
|
+
|
|
7
|
+
Test-Driven Development in Dominion Flow. Plans with `type: tdd` in frontmatter follow the Red-Green-Refactor cycle with atomic commits at each stage.
|
|
8
|
+
|
|
9
|
+
---
|
|
10
|
+
|
|
11
|
+
## When to Use TDD Plans
|
|
12
|
+
|
|
13
|
+
- Pure functions, utilities, validators, formatters
|
|
14
|
+
- API endpoints with clear input/output contracts
|
|
15
|
+
- Business logic with defined rules
|
|
16
|
+
- Data transformations
|
|
17
|
+
|
|
18
|
+
## When NOT to Use TDD Plans
|
|
19
|
+
|
|
20
|
+
- UI components (test after build with checkpoint:human-verify)
|
|
21
|
+
- Configuration/setup tasks
|
|
22
|
+
- Database migrations (test with integration tests)
|
|
23
|
+
- One-off scripts
|
|
24
|
+
|
|
25
|
+
---
|
|
26
|
+
|
|
27
|
+
## TDD Plan Frontmatter
|
|
28
|
+
|
|
29
|
+
```yaml
|
|
30
|
+
---
|
|
31
|
+
phase: XX-name
|
|
32
|
+
plan: NN
|
|
33
|
+
type: tdd
|
|
34
|
+
feature: "[Single feature being tested]"
|
|
35
|
+
test_framework: jest|vitest|pytest|go-test
|
|
36
|
+
---
|
|
37
|
+
```
|
|
38
|
+
|
|
39
|
+
**One feature per TDD plan.** If features are trivial enough to batch, skip TDD.
|
|
40
|
+
|
|
41
|
+
---
|
|
42
|
+
|
|
43
|
+
## Red-Green-Refactor Cycle
|
|
44
|
+
|
|
45
|
+
### RED - Write failing test
|
|
46
|
+
|
|
47
|
+
1. Ensure correct feature branch: `feature/phase-XX-description`
|
|
48
|
+
2. Create test file following project conventions
|
|
49
|
+
3. Write test describing expected behavior
|
|
50
|
+
4. Run test - MUST fail
|
|
51
|
+
5. If test passes: feature already exists or test is wrong
|
|
52
|
+
6. Commit: `test({phase}-{plan}): add failing test for [feature]`
|
|
53
|
+
|
|
54
|
+
### GREEN - Implement to pass
|
|
55
|
+
|
|
56
|
+
1. Write minimal code to make test pass
|
|
57
|
+
2. No cleverness, no optimization - just make it work
|
|
58
|
+
3. Run test - MUST pass
|
|
59
|
+
4. If not passing, iterate. Do not move to REFACTOR.
|
|
60
|
+
5. Commit: `feat({phase}-{plan}): implement [feature]`
|
|
61
|
+
|
|
62
|
+
### REFACTOR (if needed)
|
|
63
|
+
|
|
64
|
+
1. Clean up if obvious improvements exist
|
|
65
|
+
2. Run tests - MUST still pass
|
|
66
|
+
3. Only commit if changes made: `refactor({phase}-{plan}): clean up [feature]`
|
|
67
|
+
|
|
68
|
+
**Result:** 2-3 atomic commits on the feature branch per TDD plan.
|
|
69
|
+
|
|
70
|
+
---
|
|
71
|
+
|
|
72
|
+
## Test Quality Standards
|
|
73
|
+
|
|
74
|
+
- **Test behavior, not implementation** - tests should survive refactors
|
|
75
|
+
- **One concept per test** - separate tests for valid, empty, malformed input
|
|
76
|
+
- **Descriptive names** - "should reject empty email" not "test1"
|
|
77
|
+
- **No implementation details** - test public API, observable behavior
|
|
78
|
+
- **Meaningful assertions** - NEVER `expect(true).toBe(true)`
|
|
79
|
+
- **Cover happy + error paths** - at minimum one of each
|
|
80
|
+
|
|
81
|
+
---
|
|
82
|
+
|
|
83
|
+
## Test Types
|
|
84
|
+
|
|
85
|
+
| Type | When | Speed | Example |
|
|
86
|
+
|------|------|-------|---------|
|
|
87
|
+
| Unit | Pure functions, utilities | Milliseconds | `validateEmail('bad') === false` |
|
|
88
|
+
| Integration | API routes, DB operations | Seconds | `POST /api/register -> 201` |
|
|
89
|
+
| E2E | Complete user journeys | Seconds-minutes | Playwright checkout flow |
|
|
90
|
+
|
|
91
|
+
---
|
|
92
|
+
|
|
93
|
+
## Branch Integration
|
|
94
|
+
|
|
95
|
+
TDD work follows Git Flow:
|
|
96
|
+
|
|
97
|
+
1. Ensure feature branch: `git checkout -b feature/phase-XX-desc develop`
|
|
98
|
+
2. RED commit on feature branch
|
|
99
|
+
3. GREEN commit on feature branch
|
|
100
|
+
4. REFACTOR commit on feature branch
|
|
101
|
+
5. Create PR from feature to develop
|
|
102
|
+
6. Merge after review. Never directly to main.
|
|
103
|
+
|
|
104
|
+
---
|
|
105
|
+
|
|
106
|
+
## Error Handling
|
|
107
|
+
|
|
108
|
+
| Situation | Action |
|
|
109
|
+
|-----------|--------|
|
|
110
|
+
| Test doesn't fail in RED | Feature may exist. Investigate before proceeding. |
|
|
111
|
+
| Test doesn't pass in GREEN | Keep iterating. Don't skip to REFACTOR. |
|
|
112
|
+
| Tests fail in REFACTOR | Undo refactor immediately. Smaller steps. |
|
|
113
|
+
| Unrelated tests break | Stop. Investigate coupling. Create P1 blocker if needed. |
|
|
114
|
+
| Stuck after 3 GREEN attempts | Create P1 blocker in BLOCKERS.md. |
|
|
115
|
+
|
|
116
|
+
---
|
|
117
|
+
|
|
118
|
+
## Commit Pattern
|
|
119
|
+
|
|
120
|
+
```
|
|
121
|
+
test(08-02): add failing test for email validation
|
|
122
|
+
- Tests valid email formats accepted
|
|
123
|
+
- Tests invalid formats rejected
|
|
124
|
+
|
|
125
|
+
feat(08-02): implement email validation
|
|
126
|
+
- Regex pattern matches RFC 5322
|
|
127
|
+
- Returns boolean for validity
|
|
128
|
+
|
|
129
|
+
refactor(08-02): extract regex to constant (optional)
|
|
130
|
+
- No behavior changes, tests still pass
|
|
131
|
+
```
|