aiox-core 5.0.7 → 5.0.8
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/.aiox-core/cli/commands/pro/buyer.js +379 -0
- package/.aiox-core/cli/commands/pro/index.js +191 -52
- package/.aiox-core/cli/commands/validate/index.js +2 -0
- package/.aiox-core/core/code-intel/helpers/dev-helper.js +1 -1
- package/.aiox-core/core/code-intel/helpers/devops-helper.js +0 -1
- package/.aiox-core/core/code-intel/helpers/planning-helper.js +1 -1
- package/.aiox-core/core/code-intel/helpers/qa-helper.js +2 -2
- package/.aiox-core/core/config/schemas/framework-config.schema.json +1 -0
- package/.aiox-core/core/config/template-overrides.js +1 -1
- package/.aiox-core/core/doctor/checks/ide-sync.js +81 -25
- package/.aiox-core/core/doctor/checks/rules-files.js +0 -1
- package/.aiox-core/core/doctor/checks/skills-count.js +83 -15
- package/.aiox-core/core/graph-dashboard/cli.js +1 -2
- package/.aiox-core/core/graph-dashboard/data-sources/code-intel-source.js +1 -1
- package/.aiox-core/core/ids/layer-classifier.js +1 -1
- package/.aiox-core/core/pro/pro-updater.js +578 -0
- package/.aiox-core/core/synapse/context/context-tracker.js +107 -9
- package/.aiox-core/core/synapse/layers/layer-processor.js +1 -1
- package/.aiox-core/core-config.yaml +15 -1
- package/.aiox-core/data/capability-detection.js +15 -15
- package/.aiox-core/data/entity-registry.yaml +18 -2
- package/.aiox-core/data/registry-update-log.jsonl +5 -0
- package/.aiox-core/data/tok3-token-comparison.js +0 -4
- package/.aiox-core/data/tool-search-validation.js +1 -1
- package/.aiox-core/development/agents/aiox-master.md +44 -6
- package/.aiox-core/development/agents/data-engineer.md +4 -4
- package/.aiox-core/development/agents/devops.md +52 -2
- package/.aiox-core/development/agents/po.md +1 -1
- package/.aiox-core/development/agents/qa.md +5 -11
- package/.aiox-core/development/agents/sm.md +3 -3
- package/.aiox-core/development/agents/ux-design-expert.md +1 -1
- package/.aiox-core/development/scripts/unified-activation-pipeline.js +29 -3
- package/.aiox-core/development/tasks/dev-develop-story.md +46 -7
- package/.aiox-core/development/tasks/devops-pro-access-grant.md +93 -0
- package/.aiox-core/development/tasks/devops-pro-activate.md +42 -0
- package/.aiox-core/development/tasks/devops-pro-check-access.md +34 -0
- package/.aiox-core/development/tasks/devops-pro-request-reset.md +34 -0
- package/.aiox-core/development/tasks/devops-pro-resend-verification.md +32 -0
- package/.aiox-core/development/tasks/devops-pro-reset-password.md +36 -0
- package/.aiox-core/development/tasks/devops-pro-validate-login.md +36 -0
- package/.aiox-core/development/tasks/devops-pro-verify-status.md +33 -0
- package/.aiox-core/development/tasks/qa-gate.md +54 -4
- package/.aiox-core/development/tasks/validate-next-story.md +39 -2
- package/.aiox-core/framework-config.yaml +1 -0
- package/.aiox-core/infrastructure/scripts/codex-skills-sync/README.md +69 -0
- package/.aiox-core/infrastructure/scripts/codex-skills-sync/bootstrap.js +727 -0
- package/.aiox-core/infrastructure/scripts/codex-skills-sync/index.js +10 -0
- package/.aiox-core/infrastructure/scripts/codex-skills-sync/validate.js +65 -4
- package/.aiox-core/infrastructure/scripts/generate-settings-json.js +29 -4
- package/.aiox-core/infrastructure/scripts/ide-sync/agent-parser.js +4 -0
- package/.aiox-core/infrastructure/scripts/ide-sync/index.js +67 -7
- package/.aiox-core/infrastructure/scripts/ide-sync/transformers/claude-code.js +145 -3
- package/.aiox-core/infrastructure/scripts/repair-agent-references.js +263 -0
- package/.aiox-core/infrastructure/scripts/validate-claude-integration.js +60 -8
- package/.aiox-core/infrastructure/scripts/validate-paths.js +13 -0
- package/.aiox-core/install-manifest.yaml +134 -82
- package/.aiox-core/utils/filters/index.js +2 -1
- package/.claude/commands/AIOX/agents/aiox-master.md +21 -0
- package/.claude/commands/AIOX/agents/analyst.md +21 -0
- package/.claude/commands/AIOX/agents/architect.md +21 -0
- package/.claude/commands/AIOX/agents/data-engineer.md +21 -0
- package/.claude/commands/AIOX/agents/dev.md +21 -0
- package/.claude/commands/AIOX/agents/devops.md +21 -0
- package/.claude/commands/AIOX/agents/pm.md +21 -0
- package/.claude/commands/AIOX/agents/po.md +21 -0
- package/.claude/commands/AIOX/agents/qa.md +21 -0
- package/.claude/commands/AIOX/agents/sm.md +21 -0
- package/.claude/commands/AIOX/agents/squad-creator.md +21 -0
- package/.claude/commands/AIOX/agents/ux-design-expert.md +21 -0
- package/.claude/commands/AIOX/scripts/agent-config-loader.js +624 -0
- package/.claude/commands/AIOX/scripts/generate-greeting.js +160 -0
- package/.claude/commands/AIOX/scripts/greeting-builder.js +866 -0
- package/.claude/commands/AIOX/scripts/session-context-loader.js +286 -0
- package/.claude/commands/AIOX/stories/story-6.1.4.md +1404 -0
- package/.claude/commands/cohort-squad/agents/cohort-manager.md +156 -0
- package/.claude/commands/design-system/agents/brad-frost.md +1097 -0
- package/.claude/commands/design-system/agents/dan-mall.md +857 -0
- package/.claude/commands/design-system/agents/dave-malouf.md +2272 -0
- package/.claude/commands/design-system/agents/design-chief.md +102 -0
- package/.claude/commands/design-system/agents/nano-banana-generator.md +162 -0
- package/.claude/commands/greet.md +101 -0
- package/.claude/commands/synapse/manager.md +75 -0
- package/.claude/commands/synapse/tasks/add-rule.md +94 -0
- package/.claude/commands/synapse/tasks/create-command.md +109 -0
- package/.claude/commands/synapse/tasks/create-domain.md +127 -0
- package/.claude/commands/synapse/tasks/diagnose-synapse.md +245 -0
- package/.claude/commands/synapse/tasks/edit-rule.md +109 -0
- package/.claude/commands/synapse/tasks/suggest-domain.md +116 -0
- package/.claude/commands/synapse/tasks/toggle-domain.md +83 -0
- package/.claude/commands/synapse/templates/domain-template +8 -0
- package/.claude/commands/synapse/templates/manifest-entry-template +4 -0
- package/.claude/commands/synapse/utils/manifest-parser-reference.md +134 -0
- package/.claude/hooks/precompact-session-digest.cjs +2 -2
- package/.claude/skills/AIOX/agents/aiox-master/SKILL.md +511 -0
- package/.claude/skills/AIOX/agents/analyst/SKILL.md +281 -0
- package/.claude/skills/AIOX/agents/architect/SKILL.md +482 -0
- package/.claude/skills/AIOX/agents/data-engineer/SKILL.md +503 -0
- package/.claude/skills/AIOX/agents/dev/SKILL.md +568 -0
- package/.claude/skills/AIOX/agents/devops/SKILL.md +597 -0
- package/.claude/skills/AIOX/agents/pm/SKILL.md +385 -0
- package/.claude/skills/AIOX/agents/po/SKILL.md +343 -0
- package/.claude/skills/AIOX/agents/qa/SKILL.md +451 -0
- package/.claude/skills/AIOX/agents/sm/SKILL.md +295 -0
- package/.claude/skills/AIOX/agents/squad-creator/SKILL.md +352 -0
- package/.claude/skills/AIOX/agents/ux-design-expert/SKILL.md +503 -0
- package/.claude/skills/architect-first/SKILL.md +275 -0
- package/.claude/skills/architect-first/assets/architecture-template.md +505 -0
- package/.claude/skills/architect-first/assets/config-template.yaml +351 -0
- package/.claude/skills/architect-first/references/architecture-checklist.md +216 -0
- package/.claude/skills/architect-first/references/pre-implementation-checklist.md +119 -0
- package/.claude/skills/architect-first/references/stop-rules-guide.md +291 -0
- package/.claude/skills/architect-first/references/testing-strategy-guide.md +477 -0
- package/.claude/skills/architect-first/scripts/architecture_validator.py +490 -0
- package/.claude/skills/architect-first/scripts/check_coupling.py +306 -0
- package/.claude/skills/architect-first/scripts/validate_risk_mitigation.py +382 -0
- package/.claude/skills/checklist-runner/SKILL.md +113 -0
- package/.claude/skills/clone-mind.md +329 -0
- package/.claude/skills/coderabbit-review/SKILL.md +106 -0
- package/.claude/skills/course-generation-workflow.md +76 -0
- package/.claude/skills/enhance-workflow.md +466 -0
- package/.claude/skills/mcp-builder/LICENSE.txt +202 -0
- package/.claude/skills/mcp-builder/SKILL.md +328 -0
- package/.claude/skills/mcp-builder/reference/evaluation.md +602 -0
- package/.claude/skills/mcp-builder/reference/mcp_best_practices.md +915 -0
- package/.claude/skills/mcp-builder/reference/node_mcp_server.md +916 -0
- package/.claude/skills/mcp-builder/reference/python_mcp_server.md +752 -0
- package/.claude/skills/mcp-builder/scripts/connections.py +151 -0
- package/.claude/skills/mcp-builder/scripts/evaluation.py +373 -0
- package/.claude/skills/mcp-builder/scripts/example_evaluation.xml +22 -0
- package/.claude/skills/mcp-builder/scripts/requirements.txt +2 -0
- package/.claude/skills/ralph.md +181 -0
- package/.claude/skills/skill-creator/LICENSE.txt +202 -0
- package/.claude/skills/skill-creator/SKILL.md +209 -0
- package/.claude/skills/skill-creator/scripts/init_skill.py +303 -0
- package/.claude/skills/skill-creator/scripts/package_skill.py +110 -0
- package/.claude/skills/skill-creator/scripts/quick_validate.py +65 -0
- package/.claude/skills/squad.md +301 -0
- package/.claude/skills/synapse/SKILL.md +132 -0
- package/.claude/skills/synapse/assets/README.md +50 -0
- package/.claude/skills/synapse/references/brackets.md +100 -0
- package/.claude/skills/synapse/references/commands.md +118 -0
- package/.claude/skills/synapse/references/domains.md +126 -0
- package/.claude/skills/synapse/references/layers.md +186 -0
- package/.claude/skills/synapse/references/manifest.md +142 -0
- package/.claude/skills/tech-search/SKILL.md +431 -0
- package/.claude/skills/tech-search/prompts/page-extract.md +133 -0
- package/README.en.md +2 -2
- package/README.md +8 -2
- package/bin/aiox.js +55 -4
- package/bin/utils/framework-guard.js +4 -2
- package/bin/utils/pro-detector.js +119 -28
- package/bin/utils/validate-publish.js +6 -6
- package/docs/aiox-agent-flows/devops-system.md +18 -0
- package/docs/aiox-workflows/README.md +1 -0
- package/docs/aiox-workflows/pro-access-grant-workflow.md +218 -0
- package/docs/guides/pro/access-grant-ops-playbook.md +370 -0
- package/docs/guides/pro/install-gate-setup.md +12 -6
- package/docs/guides/pro/squad-creator-handoff-pro-access-ops.md +134 -0
- package/docs/guides/supabase-ops-handoff.md +768 -0
- package/package.json +12 -1
- package/packages/aiox-pro-cli/bin/aiox-pro.js +33 -12
- package/packages/installer/src/config/configure-environment.js +118 -50
- package/packages/installer/src/installer/aiox-core-installer.js +124 -27
- package/packages/installer/src/installer/brownfield-upgrader.js +66 -9
- package/packages/installer/src/installer/dependency-installer.js +4 -0
- package/packages/installer/src/pro/pro-scaffolder.js +5 -5
- package/packages/installer/src/updater/index.js +151 -10
- package/packages/installer/src/wizard/ide-config-generator.js +73 -7
- package/packages/installer/src/wizard/index.js +119 -31
- package/packages/installer/src/wizard/pro-setup.js +118 -47
- package/packages/installer/src/wizard/validation/validators/dependency-validator.js +32 -25
- package/packages/installer/src/wizard/validation/validators/file-structure-validator.js +26 -0
- package/packages/installer/tests/unit/artifact-copy-pipeline/artifact-copy-pipeline.test.js +84 -1
- package/packages/installer/tests/unit/claude-md-template-v5/claude-md-template-v5.test.js +1 -1
- package/packages/installer/tests/unit/doctor/doctor-checks.test.js +85 -19
- package/packages/installer/tests/unit/entity-registry-bootstrap.test.js +4 -4
- package/packages/installer/tests/unit/generate-settings-json/generate-settings-json.test.js +5 -5
- package/packages/installer/tests/unit/ide-sync-integration/ide-sync-integration.test.js +4 -4
- package/packages/installer/tests/unit/merger/yaml-merger.test.js +11 -11
- package/pro/README.md +12 -1
- package/pro/license/index.js +3 -11
- package/pro/license/license-api.js +25 -0
- package/pro/license/license-cache.js +135 -31
- package/pro/license/license-crypto.js +59 -3
- package/pro/package.json +5 -4
- package/pro/squads/README.md +16 -16
- package/pro/squads/index.js +1 -1
- package/scripts/e2e/installed-skills-smoke.js +264 -0
- package/scripts/package-synapse.js +3 -3
- package/scripts/validate-package-completeness.js +8 -11
- package/.aiox-core/lib/build.json +0 -1
|
@@ -0,0 +1,275 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: architect-first
|
|
3
|
+
description: Guide for implementing the Architect-First development philosophy - perfect architecture, pragmatic execution, quality guaranteed by tests. Use this skill when starting new features, refactoring systems, or when architectural decisions are needed. Enforces non-negotiables like complete design/documentation before code, zero coupling, and validation by multiple perspectives before structural decisions.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Architect First
|
|
7
|
+
|
|
8
|
+
## Overview
|
|
9
|
+
|
|
10
|
+
This skill embodies the "Architect-First" development philosophy: **Perfect architecture, pragmatic execution, quality guaranteed by tests**. Apply this skill when making architectural decisions, starting new features, refactoring existing systems, or when quality gates need to be enforced.
|
|
11
|
+
|
|
12
|
+
The core principle: **Architecture and documentation are non-negotiable and must precede implementation. Code quality is negotiable IF backed by tests as a safety net (escape hatch).**
|
|
13
|
+
|
|
14
|
+
## Core Philosophy
|
|
15
|
+
|
|
16
|
+
### Mantra
|
|
17
|
+
"Arquitetura perfeita, execução pragmática, qualidade garantida por testes"
|
|
18
|
+
|
|
19
|
+
### Quality Gates
|
|
20
|
+
|
|
21
|
+
**Non-Negotiable (STOP if violated):**
|
|
22
|
+
- **Architecture**: Complete design and documentation BEFORE any code
|
|
23
|
+
- **Documentation**: Must precede and accompany implementation
|
|
24
|
+
- **Capability Preservation**: Never lose capability/granularity vs previous versions
|
|
25
|
+
- **Zero Coupling**: Expansion packs must be independent
|
|
26
|
+
- **Multi-Agent Validation**: Structural decisions validated by PO/Architect/User
|
|
27
|
+
|
|
28
|
+
**Negotiable (with escape hatch):**
|
|
29
|
+
- **Code Style**: Acceptable if backed by tests as safety net
|
|
30
|
+
- **Feature Completeness**: 80% acceptable IF core use case works
|
|
31
|
+
- **Quick & Dirty Code**: Allowed ONLY with test plan and minimal logging
|
|
32
|
+
|
|
33
|
+
### Decision Modes
|
|
34
|
+
|
|
35
|
+
**Architect-First Mode (default):**
|
|
36
|
+
- Design and document completely before coding
|
|
37
|
+
- Map structure and pointers before proposing implementation
|
|
38
|
+
- Validate architecture with multiple agents/perspectives
|
|
39
|
+
- Externalize all mutable configurations to YAML
|
|
40
|
+
|
|
41
|
+
**Fast Mode (post-validation only):**
|
|
42
|
+
- Binary decisions and rapid delegation
|
|
43
|
+
- Activated ONLY after architectural validation is complete
|
|
44
|
+
- Speed through automation, not shortcuts
|
|
45
|
+
|
|
46
|
+
## Workflow Decision Tree
|
|
47
|
+
|
|
48
|
+
```
|
|
49
|
+
New Task/Feature Request
|
|
50
|
+
↓
|
|
51
|
+
┌──────────────────────────────────────┐
|
|
52
|
+
│ Is this a structural/architectural │
|
|
53
|
+
│ decision? │
|
|
54
|
+
└──────────────────────────────────────┘
|
|
55
|
+
↓ YES ↓ NO
|
|
56
|
+
↓ ↓
|
|
57
|
+
[Architecture Flow] [Execution Flow]
|
|
58
|
+
```
|
|
59
|
+
|
|
60
|
+
### Architecture Flow (Structural Decisions)
|
|
61
|
+
|
|
62
|
+
**STOP and follow this sequence:**
|
|
63
|
+
|
|
64
|
+
1. **Map Before Modify**
|
|
65
|
+
- Document current state completely
|
|
66
|
+
- Identify all dependencies and touch points
|
|
67
|
+
- Create architectural diagrams/flows
|
|
68
|
+
- Load `references/architecture-checklist.md` for validation
|
|
69
|
+
|
|
70
|
+
2. **Multi-Agent Validation**
|
|
71
|
+
- Present A/B/C options with explicit trade-offs
|
|
72
|
+
- Get validation from:
|
|
73
|
+
- Product Owner (business alignment)
|
|
74
|
+
- Architect (technical soundness)
|
|
75
|
+
- User (final decision)
|
|
76
|
+
- Document decision rationale
|
|
77
|
+
|
|
78
|
+
3. **Design Documentation**
|
|
79
|
+
- Complete design document BEFORE code
|
|
80
|
+
- Include:
|
|
81
|
+
- System architecture diagrams
|
|
82
|
+
- Component interactions
|
|
83
|
+
- Data flows
|
|
84
|
+
- Configuration schema (YAML)
|
|
85
|
+
- Integration points
|
|
86
|
+
- Use templates from `assets/architecture-template.md`
|
|
87
|
+
|
|
88
|
+
4. **Gold Standard Baseline**
|
|
89
|
+
- Ensure new design meets/exceeds capability baseline
|
|
90
|
+
- Validate: Does this maintain ALL previous capabilities?
|
|
91
|
+
- STOP if capability loss detected → restore or redesign
|
|
92
|
+
|
|
93
|
+
5. **Zero Coupling Validation**
|
|
94
|
+
- Run validation script: `scripts/check_coupling.py`
|
|
95
|
+
- Ensure expansion pack independence
|
|
96
|
+
- No hardcoded cross-module dependencies
|
|
97
|
+
|
|
98
|
+
6. **Proceed to Implementation**
|
|
99
|
+
- Now and only now: write code
|
|
100
|
+
- Follow execution flow for implementation
|
|
101
|
+
|
|
102
|
+
### Execution Flow (Implementation)
|
|
103
|
+
|
|
104
|
+
1. **Pre-Implementation Checklist**
|
|
105
|
+
- [ ] Architecture documented and validated?
|
|
106
|
+
- [ ] Core use case clearly defined?
|
|
107
|
+
- [ ] Configuration externalized to YAML?
|
|
108
|
+
- [ ] Test strategy defined?
|
|
109
|
+
- Use `references/pre-implementation-checklist.md`
|
|
110
|
+
|
|
111
|
+
2. **Test-Driven Safety Net**
|
|
112
|
+
- Define test plan FIRST
|
|
113
|
+
- Identify logging/observation points
|
|
114
|
+
- Tests permit temporary imperfection (escape hatch)
|
|
115
|
+
- Quality validation via: tests + logs + manual inspection
|
|
116
|
+
|
|
117
|
+
3. **Implementation Style**
|
|
118
|
+
```
|
|
119
|
+
ACCEPTABLE:
|
|
120
|
+
✓ "Ugly" code WITH comprehensive tests
|
|
121
|
+
✓ 80% feature completeness IF core case works
|
|
122
|
+
✓ Quick implementation WITH test plan + logging
|
|
123
|
+
|
|
124
|
+
REJECTED:
|
|
125
|
+
✗ "Ugly" code WITHOUT tests
|
|
126
|
+
✗ Capability loss without explicit justification
|
|
127
|
+
✗ Hardcoded mutable values (must be YAML)
|
|
128
|
+
✗ Deployment without core case working
|
|
129
|
+
```
|
|
130
|
+
|
|
131
|
+
4. **Debugging Philosophy**
|
|
132
|
+
- Observational via logs (console/logging) > static analysis
|
|
133
|
+
- Add strategic log points before debugging
|
|
134
|
+
- Inspect actual runtime behavior
|
|
135
|
+
- Validate through execution, not just reading code
|
|
136
|
+
|
|
137
|
+
5. **Documentation**
|
|
138
|
+
- Update docs as code evolves
|
|
139
|
+
- Keep short and actionable: "How to customize"
|
|
140
|
+
- Include code examples
|
|
141
|
+
- Document configuration options
|
|
142
|
+
|
|
143
|
+
## Heuristics (Decision Rules)
|
|
144
|
+
|
|
145
|
+
Apply these heuristics when making decisions:
|
|
146
|
+
|
|
147
|
+
1. **Gold Standard Baseline**: 22 artifacts minimum (adjust to your context)
|
|
148
|
+
2. **Never Lose Capability**: Accumulate, never reduce
|
|
149
|
+
3. **Architect Before Build**: Design/docs before code, always
|
|
150
|
+
4. **Zero Coupling, Max Modularity**: Independent expansion packs
|
|
151
|
+
5. **Config > Hardcoding**: Externalize to YAML for all mutable values
|
|
152
|
+
6. **Map Before Modify**: Document structure before changing it
|
|
153
|
+
7. **Binary Decision Post-Validation**: Fast execution after architectural validation
|
|
154
|
+
8. **Speed via Automation**: Not via shortcuts or cutting corners
|
|
155
|
+
9. **Quality Escape Hatch**: Tests permit temporary imperfection
|
|
156
|
+
|
|
157
|
+
## Stop Rules (Hard Boundaries)
|
|
158
|
+
|
|
159
|
+
**STOP immediately if detecting:**
|
|
160
|
+
|
|
161
|
+
- ⛔ **Capability loss** vs baseline
|
|
162
|
+
- ⛔ **Structural decision** without multi-agent validation
|
|
163
|
+
- ⛔ **Coupling** between modules
|
|
164
|
+
- ⛔ **Missing architectural documentation**
|
|
165
|
+
- ⛔ **Quick & dirty code** WITHOUT test plan and logs
|
|
166
|
+
- ⛔ **Hardcoded** mutable configuration values
|
|
167
|
+
|
|
168
|
+
When stopped, consult `references/stop-rules-guide.md` for remediation.
|
|
169
|
+
|
|
170
|
+
## Risk Mitigation
|
|
171
|
+
|
|
172
|
+
Common risks and their mitigations:
|
|
173
|
+
|
|
174
|
+
| Risk | Mitigation Strategy |
|
|
175
|
+
|------|-------------------|
|
|
176
|
+
| Excessive planning | Time-box + mandatory POC before full formalization |
|
|
177
|
+
| Perfectionism cascade | Rule of 3: simple pilot → 2 iterations → formalize |
|
|
178
|
+
| Premature configuration | Generalize only after ≥2 real scenarios |
|
|
179
|
+
| Context switching | Checklist of pending items at each pivot |
|
|
180
|
+
|
|
181
|
+
Use `scripts/validate_risk_mitigation.py` to check risk mitigation coverage.
|
|
182
|
+
|
|
183
|
+
## Collaboration Contract
|
|
184
|
+
|
|
185
|
+
When working with this skill, follow these collaboration patterns:
|
|
186
|
+
|
|
187
|
+
✓ **Do:**
|
|
188
|
+
- Present A/B/C options with explicit trade-offs
|
|
189
|
+
- Map structure and pointers before proposing code
|
|
190
|
+
- Externalize configs in YAML; nothing mutable hardcoded
|
|
191
|
+
- Deliver short, actionable docs ("how to customize")
|
|
192
|
+
- For quick code: include test plan and log points
|
|
193
|
+
- Document pending items when pivoting
|
|
194
|
+
|
|
195
|
+
✗ **Don't:**
|
|
196
|
+
- Propose code without architectural context
|
|
197
|
+
- Hardcode values that might change
|
|
198
|
+
- Skip multi-agent validation on structural changes
|
|
199
|
+
- Write "quick & dirty" without tests
|
|
200
|
+
- Make architectural decisions unilaterally
|
|
201
|
+
|
|
202
|
+
## Acceptance Criteria
|
|
203
|
+
|
|
204
|
+
### Will Accept
|
|
205
|
+
- ✓ "Ugly" code WITH comprehensive tests
|
|
206
|
+
- ✓ 80% features IF core case covered
|
|
207
|
+
- ✓ Large refactors that increase flexibility
|
|
208
|
+
- ✓ Extensive documentation if it teaches customization
|
|
209
|
+
|
|
210
|
+
### Will Reject
|
|
211
|
+
- ✗ "Ugly" code WITHOUT tests
|
|
212
|
+
- ✗ Capability loss without explicit justification
|
|
213
|
+
- ✗ Hardcoded mutable values
|
|
214
|
+
- ✗ Deployment without core case working
|
|
215
|
+
|
|
216
|
+
## Unknown Areas (For Future Refinement)
|
|
217
|
+
|
|
218
|
+
Areas not yet fully defined in the philosophy:
|
|
219
|
+
|
|
220
|
+
- Hotfix philosophy (production emergencies)
|
|
221
|
+
- Performance thresholds (latency/throughput minimums)
|
|
222
|
+
- Code duplication tolerance (when to refactor)
|
|
223
|
+
- Observability targets (log levels, correlation, tracing)
|
|
224
|
+
|
|
225
|
+
When encountering these areas, apply the core heuristics and document decisions for future refinement.
|
|
226
|
+
|
|
227
|
+
## Resources
|
|
228
|
+
|
|
229
|
+
### scripts/
|
|
230
|
+
Validation and automation scripts:
|
|
231
|
+
- `check_coupling.py` - Validates zero-coupling principle
|
|
232
|
+
- `validate_risk_mitigation.py` - Checks risk coverage
|
|
233
|
+
- `architecture_validator.py` - Validates architectural completeness
|
|
234
|
+
|
|
235
|
+
### references/
|
|
236
|
+
Detailed checklists and guides:
|
|
237
|
+
- `architecture-checklist.md` - Complete architecture validation checklist
|
|
238
|
+
- `pre-implementation-checklist.md` - Pre-coding validation
|
|
239
|
+
- `stop-rules-guide.md` - Remediation guide when stop rules trigger
|
|
240
|
+
- `testing-strategy-guide.md` - Test-driven development patterns
|
|
241
|
+
|
|
242
|
+
### assets/
|
|
243
|
+
Templates for consistent outputs:
|
|
244
|
+
- `architecture-template.md` - Standard architecture document template
|
|
245
|
+
- `config-template.yaml` - Configuration externalization template
|
|
246
|
+
- `adr-template.md` - Architecture Decision Record template
|
|
247
|
+
|
|
248
|
+
## Quick Reference
|
|
249
|
+
|
|
250
|
+
**Starting a new feature:**
|
|
251
|
+
1. Map current architecture (`references/architecture-checklist.md`)
|
|
252
|
+
2. Design with A/B/C options
|
|
253
|
+
3. Multi-agent validation
|
|
254
|
+
4. Document architecture (`assets/architecture-template.md`)
|
|
255
|
+
5. Define tests
|
|
256
|
+
6. Implement with logging
|
|
257
|
+
7. Validate and iterate
|
|
258
|
+
|
|
259
|
+
**Making architectural changes:**
|
|
260
|
+
1. STOP - Do not code yet
|
|
261
|
+
2. Document current state completely
|
|
262
|
+
3. Present options with trade-offs
|
|
263
|
+
4. Get PO/Architect/User validation
|
|
264
|
+
5. Check coupling (`scripts/check_coupling.py`)
|
|
265
|
+
6. Document decision (`assets/adr-template.md`)
|
|
266
|
+
7. Now implement
|
|
267
|
+
|
|
268
|
+
**Quick implementation (with safety net):**
|
|
269
|
+
1. Pre-implementation checklist
|
|
270
|
+
2. Define test plan
|
|
271
|
+
3. Add log points
|
|
272
|
+
4. Implement (can be "ugly")
|
|
273
|
+
5. Verify tests pass
|
|
274
|
+
6. Inspect logs
|
|
275
|
+
7. Refactor if needed
|