owflow 0.1.5 → 0.1.6
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/README.md +34 -21
- package/dist/__tests__/verify_template.test.js +102 -0
- package/dist/agents/task-classifier.md +7 -19
- package/dist/commands/quick-dev.md +48 -6
- package/dist/commands/quick-plan.md +39 -4
- package/dist/commands/work.md +13 -17
- package/dist/index.js +4 -0
- package/dist/skills/development/SKILL.md +83 -115
- package/dist/skills/migration/SKILL.md +2 -33
- package/dist/skills/orchestrator-framework/references/orchestrator-creation-checklist.md +1 -0
- package/dist/skills/orchestrator-framework/references/orchestrator-patterns.md +4 -59
- package/dist/skills/performance/SKILL.md +2 -30
- package/dist/skills/quick-bugfix/SKILL.md +96 -26
- package/dist/skills/research/SKILL.md +3 -42
- package/dist/templates/orchestrator-state-base.yml +17 -0
- package/dist/templates/orchestrator-state-development.yml +94 -0
- package/dist/templates/orchestrator-state-migration.yml +55 -0
- package/dist/templates/orchestrator-state-performance.yml +53 -0
- package/dist/templates/orchestrator-state-research.yml +61 -0
- package/dist/templates/quick-bugfix-task.yml +10 -0
- package/dist/templates/quick-dev-task.yml +10 -0
- package/dist/templates/quick-plan-task.yml +11 -0
- package/dist/tools/verify_template.js +61 -0
- package/package.json +4 -3
- package/dist/agents/ui-mockup-generator.md +0 -369
- package/dist/commands/product-design.md +0 -37
- package/dist/skills/product-design/SKILL.md +0 -843
- package/dist/skills/product-design/references/characteristic-detection.md +0 -92
- package/dist/skills/product-design/references/interaction-patterns.md +0 -206
- package/dist/skills/product-design/references/visual-companion.md +0 -190
- package/dist/skills/product-design/server/index.js +0 -302
- package/dist/skills/product-design/server/index.ts +0 -359
- package/dist/skills/product-design/server/template.html +0 -256
|
@@ -18,7 +18,7 @@ Unified workflow for all development tasks — bug fixes, enhancements, and new
|
|
|
18
18
|
|
|
19
19
|
1. `../orchestrator-framework/references/orchestrator-patterns.md` - Delegation rules, interactive mode, state schema, initialization, context passing, issue resolution
|
|
20
20
|
|
|
21
|
-
### Step 2: Detect
|
|
21
|
+
### Step 2: Detect Prior Work Context
|
|
22
22
|
|
|
23
23
|
**If argument is a research folder path** (matches `.owflow/tasks/research/*`):
|
|
24
24
|
|
|
@@ -32,11 +32,39 @@ Unified workflow for all development tasks — bug fixes, enhancements, and new
|
|
|
32
32
|
- Copy to `analysis/research-context/`
|
|
33
33
|
- Set `research_reference` in state
|
|
34
34
|
|
|
35
|
+
**If argument is a quick-\* task folder path** (matches `.owflow/tasks/quick-*/`):
|
|
36
|
+
|
|
37
|
+
1. Read `task.yml` — extract `command`, `description`, `standards_applied`, `escalation_reason`
|
|
38
|
+
2. Read `analysis/findings.md` — extract root cause, affected files, complexity assessment, test strategy
|
|
39
|
+
3. Use `description` as the task description (same as research extracts `research_question`)
|
|
40
|
+
4. Set `quick_reference` in orchestrator state:
|
|
41
|
+
```yaml
|
|
42
|
+
task_context:
|
|
43
|
+
quick_reference:
|
|
44
|
+
path: .owflow/tasks/quick-bugfix/2026-05-28-fix-login-timeout
|
|
45
|
+
command: quick-bugfix
|
|
46
|
+
escalation_reason: "5+ files, unclear root cause"
|
|
47
|
+
phase_summaries:
|
|
48
|
+
quick_analysis:
|
|
49
|
+
{ summary: "...", affected_files: [...], root_cause: "..." }
|
|
50
|
+
```
|
|
51
|
+
5. Update the quick-\* `task.yml`: set `escalated_to` to the new development task path
|
|
52
|
+
|
|
53
|
+
**How quick-\* context informs development phases**:
|
|
54
|
+
|
|
55
|
+
| Phase | How Quick-\* Context is Used |
|
|
56
|
+
| ------- | --------------------------------------------------------------------------- |
|
|
57
|
+
| Phase 1 | Codebase analyzer receives affected files and root cause as search guidance |
|
|
58
|
+
| Phase 2 | Gap analyzer uses complexity assessment for risk level |
|
|
59
|
+
| Phase 3 | TDD gate uses test strategy from findings.md as starting point |
|
|
60
|
+
| Phase 4 | Specification creator receives prior analysis as input context |
|
|
61
|
+
|
|
35
62
|
### Step 3: Initialize Workflow
|
|
36
63
|
|
|
37
64
|
1. **Create Task Items**: Use `TaskCreate` for all phases (see Phase Configuration), then set dependencies with `TaskUpdate addBlockedBy`
|
|
38
65
|
2. **Create Task Directory**: `.owflow/tasks/development/YYYY-MM-DD-task-name/`
|
|
39
66
|
3. **Initialize State**: Create `orchestrator-state.yml` with task info and research reference
|
|
67
|
+
- **CRITICAL**: Use the `verify_template` tool immediately after creation to check YAML validity against `orchestrator-state-development.yml`.
|
|
40
68
|
4. **Discover project documentation**: Read `.owflow/docs/INDEX.md` (if exists), extract ALL file paths from the "Project Documentation" section. This includes predefined docs (vision, roadmap, tech-stack, architecture) AND any user-added project docs (e.g., deployment.md, api-strategy.md). Store complete list as `project_context.project_doc_paths` in state.
|
|
41
69
|
|
|
42
70
|
**Output**:
|
|
@@ -67,17 +95,16 @@ Use for **all development tasks**: bug fixes, enhancements, new features, and an
|
|
|
67
95
|
| 1 | "Analyze codebase & clarify requirements" | "Analyzing codebase & clarifying" | Always |
|
|
68
96
|
| 2 | "Analyze gaps & clarify scope" | "Analyzing gaps & clarifying scope" | Always |
|
|
69
97
|
| 3 | "Write failing test (TDD Red)" | "Writing failing test" | When `has_reproducible_defect` |
|
|
70
|
-
| 4 | "
|
|
71
|
-
| 5 | "
|
|
72
|
-
| 6 | "
|
|
73
|
-
| 7 | "
|
|
74
|
-
| 8 | "
|
|
75
|
-
| 9 | "
|
|
76
|
-
| 10 | "
|
|
77
|
-
| 11 | "
|
|
78
|
-
| 12 | "
|
|
79
|
-
| 13 | "
|
|
80
|
-
| 14 | "Finalize workflow" | "Finalizing workflow" | Always |
|
|
98
|
+
| 4 | "Gather requirements & create specification" | "Gathering requirements & creating specification" | Always |
|
|
99
|
+
| 5 | "Audit specification" | "Auditing specification" | Always (conditional) |
|
|
100
|
+
| 6 | "Plan implementation" | "Planning implementation" | Always |
|
|
101
|
+
| 7 | "Execute implementation" | "Executing implementation" | Always |
|
|
102
|
+
| 8 | "Verify test passes (TDD Green)" | "Verifying test passes" | When Phase 3 was executed |
|
|
103
|
+
| 9 | "Prompt verification options" | "Prompting verification options" | Always |
|
|
104
|
+
| 10 | "Verify implementation & resolve issues" | "Verifying implementation" | Always |
|
|
105
|
+
| 11 | "Run E2E tests" | "Running E2E tests" | When `e2e_enabled` |
|
|
106
|
+
| 12 | "Generate user documentation" | "Generating user documentation" | When `user_docs_enabled` |
|
|
107
|
+
| 13 | "Finalize workflow" | "Finalizing workflow" | Always |
|
|
81
108
|
|
|
82
109
|
---
|
|
83
110
|
|
|
@@ -136,14 +163,12 @@ Use for **all development tasks**: bug fixes, enhancements, new features, and an
|
|
|
136
163
|
|
|
137
164
|
**ANTI-PATTERN — DO NOT DO THIS:**
|
|
138
165
|
|
|
139
|
-
- ❌ "The UI change is small/simple, skipping Phase 4..." — STOP. If `ui_heavy` is true, Phase 4 runs. The gap-analyzer made this assessment, not you.
|
|
140
166
|
- ❌ "No new screens needed, just a component..." — STOP. `ui_heavy` is a signal from the gap-analyzer. Do NOT override it with your own complexity judgment.
|
|
141
167
|
|
|
142
|
-
question - Display executive summary before asking. Read `analysis/gap-analysis.md` and extract: task type detected, risk level, key characteristics enabled (TDD gates,
|
|
168
|
+
question - Display executive summary before asking. Read `analysis/gap-analysis.md` and extract: task type detected, risk level, key characteristics enabled (TDD gates, E2E, user docs), scope decisions made (if any). Then read `task_context.task_characteristics` from `orchestrator-state.yml` and determine the next phase:
|
|
143
169
|
|
|
144
170
|
- If `has_reproducible_defect` is true → ask "Continue to Phase 3: TDD Red Gate?"
|
|
145
|
-
-
|
|
146
|
-
- Otherwise → ask "Continue to Phase 5: Technical Approach, Requirements & Specification?"
|
|
171
|
+
- Otherwise → ask "Continue to Phase 4: Technical Approach, Requirements & Specification?"
|
|
147
172
|
|
|
148
173
|
---
|
|
149
174
|
|
|
@@ -166,30 +191,11 @@ question - "TDD red gate complete. Continue to Phase 4?"
|
|
|
166
191
|
|
|
167
192
|
---
|
|
168
193
|
|
|
169
|
-
### Phase 4:
|
|
194
|
+
### Phase 4: Technical Approach, Requirements & Specification
|
|
170
195
|
|
|
171
196
|
> **Phase gate**: Requires `question` confirmation from the preceding phase before executing.
|
|
172
197
|
|
|
173
|
-
|
|
174
|
-
**Execute**: Task tool - `ui-mockup-generator` subagent
|
|
175
|
-
**Output**: `analysis/ui-mockups.md`
|
|
176
|
-
**State**: Update `phase_summaries.ui_mockups`
|
|
177
|
-
|
|
178
|
-
**Skip if**: `task_characteristics.ui_heavy` is false
|
|
179
|
-
|
|
180
|
-
**Context to pass**: Gap analysis, scope decisions, component choices
|
|
181
|
-
|
|
182
|
-
→ Pause
|
|
183
|
-
|
|
184
|
-
question - "UI mockups complete. Continue to Phase 5?"
|
|
185
|
-
|
|
186
|
-
---
|
|
187
|
-
|
|
188
|
-
### Phase 5: Technical Approach, Requirements & Specification
|
|
189
|
-
|
|
190
|
-
> **Phase gate**: Requires `question` confirmation from the preceding phase before executing.
|
|
191
|
-
|
|
192
|
-
**⛔ ROUTING GUARD**: Read `task_context.task_characteristics` from `orchestrator-state.yml`. If `has_reproducible_defect` is true and Phase 3 is NOT in `completed_phases` → STOP, execute Phase 3 first. If `ui_heavy` is true and Phase 4 is NOT in `completed_phases` → STOP, execute Phase 4 first.
|
|
198
|
+
**⛔ ROUTING GUARD**: Read `task_context.task_characteristics` from `orchestrator-state.yml`. If `has_reproducible_defect` is true and Phase 3 is NOT in `completed_phases` → STOP, execute Phase 3 first.
|
|
193
199
|
|
|
194
200
|
**Purpose**: Resolve technical decisions, gather specification requirements, then create comprehensive specification
|
|
195
201
|
**Execute**:
|
|
@@ -253,9 +259,9 @@ question - Display executive summary before asking. Read `implementation/spec.md
|
|
|
253
259
|
|
|
254
260
|
---
|
|
255
261
|
|
|
256
|
-
### Phase
|
|
262
|
+
### Phase 5: Specification Audit (Recommended)
|
|
257
263
|
|
|
258
|
-
> **Phase gate**: Requires `question` confirmation from Phase
|
|
264
|
+
> **Phase gate**: Requires `question` confirmation from Phase 4 before executing.
|
|
259
265
|
|
|
260
266
|
**Purpose**: Independent review of specification before implementation
|
|
261
267
|
**Execute**: Task tool - `spec-auditor` subagent
|
|
@@ -272,9 +278,9 @@ question - Display executive summary before asking. Read `verification/spec-audi
|
|
|
272
278
|
|
|
273
279
|
---
|
|
274
280
|
|
|
275
|
-
### Phase
|
|
281
|
+
### Phase 6: Implementation Planning
|
|
276
282
|
|
|
277
|
-
> **Phase gate**: Requires `question` confirmation from Phase
|
|
283
|
+
> **Phase gate**: Requires `question` confirmation from Phase 5 before executing.
|
|
278
284
|
|
|
279
285
|
**Purpose**: Break specification into implementation steps
|
|
280
286
|
|
|
@@ -306,9 +312,9 @@ question - Display executive summary before asking. Read `implementation/impleme
|
|
|
306
312
|
|
|
307
313
|
---
|
|
308
314
|
|
|
309
|
-
### Phase
|
|
315
|
+
### Phase 7: Implementation
|
|
310
316
|
|
|
311
|
-
> **Phase gate**: Requires `question` confirmation from Phase
|
|
317
|
+
> **Phase gate**: Requires `question` confirmation from Phase 6 before executing.
|
|
312
318
|
|
|
313
319
|
**Purpose**: Execute the implementation plan
|
|
314
320
|
|
|
@@ -328,8 +334,8 @@ question - Display executive summary before asking. Read `implementation/impleme
|
|
|
328
334
|
**⚠️ POST-IMPLEMENTATION CONTINUATION** — After the skill completes and returns control:
|
|
329
335
|
|
|
330
336
|
1. Read `orchestrator-state.yml` to confirm you are the orchestrator
|
|
331
|
-
2. Update state: add Phase
|
|
332
|
-
3. Evaluate conditional: if `task_characteristics.has_reproducible_defect` AND Phase 3 in `completed_phases` → Phase
|
|
337
|
+
2. Update state: add Phase 7 to `completed_phases`
|
|
338
|
+
3. Evaluate conditional: if `task_characteristics.has_reproducible_defect` AND Phase 3 in `completed_phases` → Phase 8, else → Phase 9
|
|
333
339
|
|
|
334
340
|
→ Pause
|
|
335
341
|
|
|
@@ -337,9 +343,9 @@ question - Display executive summary before asking. Extract from `phase_summarie
|
|
|
337
343
|
|
|
338
344
|
---
|
|
339
345
|
|
|
340
|
-
### Phase
|
|
346
|
+
### Phase 8: TDD Green Gate (Conditional)
|
|
341
347
|
|
|
342
|
-
> **Phase gate**: Requires `question` confirmation from Phase
|
|
348
|
+
> **Phase gate**: Requires `question` confirmation from Phase 7 before executing.
|
|
343
349
|
|
|
344
350
|
**Purpose**: Verify the failing test now passes
|
|
345
351
|
**Execute**: Direct - run the test written in Phase 3
|
|
@@ -352,11 +358,11 @@ question - Display executive summary before asking. Extract from `phase_summarie
|
|
|
352
358
|
|
|
353
359
|
→ Pause
|
|
354
360
|
|
|
355
|
-
question - "TDD gate passed. Continue to Phase
|
|
361
|
+
question - "TDD gate passed. Continue to Phase 9?"
|
|
356
362
|
|
|
357
363
|
---
|
|
358
364
|
|
|
359
|
-
### Phase
|
|
365
|
+
### Phase 9: Verification Options Prompt
|
|
360
366
|
|
|
361
367
|
> **Phase gate**: Requires `question` confirmation from the preceding phase before executing.
|
|
362
368
|
|
|
@@ -398,9 +404,9 @@ Options: "Code review (Recommended)", "Pragmatic review (Recommended)", "Reality
|
|
|
398
404
|
|
|
399
405
|
---
|
|
400
406
|
|
|
401
|
-
### Phase
|
|
407
|
+
### Phase 10: Verification & Issue Resolution
|
|
402
408
|
|
|
403
|
-
> **Phase gate**: Requires `question` confirmation from Phase
|
|
409
|
+
> **Phase gate**: Requires `question` confirmation from Phase 9 before executing.
|
|
404
410
|
|
|
405
411
|
**Purpose**: Comprehensive implementation verification with fix-then-reverify cycles
|
|
406
412
|
**Output**: `verification/implementation-verification.md`, optional code-review/pragmatic/reality reports, updated `implementation/work-log.md`
|
|
@@ -453,18 +459,18 @@ Verification Results:
|
|
|
453
459
|
**⚠️ POST-VERIFICATION CONTINUATION** — After issue resolution completes:
|
|
454
460
|
|
|
455
461
|
1. Read `orchestrator-state.yml` to confirm you are the orchestrator
|
|
456
|
-
2. Update state: add Phase
|
|
457
|
-
3. Proceed to Phase
|
|
462
|
+
2. Update state: add Phase 10 to `completed_phases`
|
|
463
|
+
3. Proceed to Phase 11
|
|
458
464
|
|
|
459
465
|
→ Pause
|
|
460
466
|
|
|
461
|
-
question - Display executive summary: total issues found, issues fixed, issues remaining by severity. Then "Continue to Phase
|
|
467
|
+
question - Display executive summary: total issues found, issues fixed, issues remaining by severity. Then "Continue to Phase 11?"
|
|
462
468
|
|
|
463
469
|
---
|
|
464
470
|
|
|
465
|
-
### Phase
|
|
471
|
+
### Phase 11: E2E Testing (Optional)
|
|
466
472
|
|
|
467
|
-
> **Phase gate**: Requires `question` confirmation from Phase
|
|
473
|
+
> **Phase gate**: Requires `question` confirmation from Phase 10 before executing.
|
|
468
474
|
|
|
469
475
|
**Purpose**: Runtime browser verification with screenshots (via Playwright MCP tools, not test file generation)
|
|
470
476
|
**Execute**: Task tool - `e2e-test-verifier` subagent
|
|
@@ -476,11 +482,11 @@ question - Display executive summary: total issues found, issues fixed, issues r
|
|
|
476
482
|
|
|
477
483
|
→ Pause
|
|
478
484
|
|
|
479
|
-
question - "E2E complete. Continue to Phase
|
|
485
|
+
question - "E2E complete. Continue to Phase 12?"
|
|
480
486
|
|
|
481
487
|
---
|
|
482
488
|
|
|
483
|
-
### Phase
|
|
489
|
+
### Phase 12: User Documentation (Optional)
|
|
484
490
|
|
|
485
491
|
> **Phase gate**: Requires `question` confirmation from the preceding phase before executing.
|
|
486
492
|
|
|
@@ -494,11 +500,11 @@ question - "E2E complete. Continue to Phase 13?"
|
|
|
494
500
|
|
|
495
501
|
→ Pause
|
|
496
502
|
|
|
497
|
-
question - "Documentation complete. Continue to Phase
|
|
503
|
+
question - "Documentation complete. Continue to Phase 13?"
|
|
498
504
|
|
|
499
505
|
---
|
|
500
506
|
|
|
501
|
-
### Phase
|
|
507
|
+
### Phase 13: Finalization
|
|
502
508
|
|
|
503
509
|
> **Phase gate**: Requires `question` confirmation from the preceding phase before executing.
|
|
504
510
|
|
|
@@ -522,44 +528,7 @@ question - "Documentation complete. Continue to Phase 14?"
|
|
|
522
528
|
|
|
523
529
|
Development-specific fields in `orchestrator-state.yml`:
|
|
524
530
|
|
|
525
|
-
|
|
526
|
-
orchestrator:
|
|
527
|
-
options:
|
|
528
|
-
spec_audit_enabled: true
|
|
529
|
-
skip_test_suite: true
|
|
530
|
-
e2e_enabled: null
|
|
531
|
-
user_docs_enabled: null
|
|
532
|
-
code_review_enabled: true
|
|
533
|
-
pragmatic_review_enabled: true
|
|
534
|
-
reality_check_enabled: true
|
|
535
|
-
production_check_enabled: true
|
|
536
|
-
task_context:
|
|
537
|
-
risk_level: null
|
|
538
|
-
clarifications_resolved: null
|
|
539
|
-
scope_expanded: null
|
|
540
|
-
architecture_decision: null
|
|
541
|
-
task_characteristics:
|
|
542
|
-
has_reproducible_defect: false
|
|
543
|
-
modifies_existing_code: false
|
|
544
|
-
creates_new_entities: false
|
|
545
|
-
involves_data_operations: false
|
|
546
|
-
ui_heavy: false
|
|
547
|
-
research_reference:
|
|
548
|
-
path: null
|
|
549
|
-
research_question: null
|
|
550
|
-
research_type: null
|
|
551
|
-
confidence_level: null
|
|
552
|
-
phase_summaries:
|
|
553
|
-
research: { summary: null, key_findings: [], recommended_approach: null }
|
|
554
|
-
codebase_analysis:
|
|
555
|
-
{ key_files: [], primary_language: null, summary: null }
|
|
556
|
-
clarifications: []
|
|
557
|
-
gap_analysis: { integration_points: [], summary: null }
|
|
558
|
-
scope_clarifications: { scope_expanded: null, summary: null }
|
|
559
|
-
ui_mockups: { components_designed: [], summary: null }
|
|
560
|
-
specification: { summary: null }
|
|
561
|
-
architecture_decision: { decision: null, summary: null }
|
|
562
|
-
```
|
|
531
|
+
Refer to the template [src/templates/orchestrator-state-development.yml](../../templates/orchestrator-state-development.yml).
|
|
563
532
|
|
|
564
533
|
---
|
|
565
534
|
|
|
@@ -574,21 +543,20 @@ orchestrator:
|
|
|
574
543
|
│ ├── clarifications.md # Phase 1
|
|
575
544
|
│ ├── gap-analysis.md # Phase 2
|
|
576
545
|
│ ├── scope-clarifications.md # Phase 2 (conditional)
|
|
577
|
-
│
|
|
578
|
-
│ └── ui-mockups.md # Phase 4 (conditional)
|
|
546
|
+
│ └── technical-clarifications.md # Phase 4 (conditional)
|
|
579
547
|
├── implementation/
|
|
580
|
-
│ ├── spec.md # Phase
|
|
581
|
-
│ ├── requirements.md # Phase
|
|
582
|
-
│ ├── implementation-plan.md # Phase
|
|
583
|
-
│ ├── work-log.md # Phase
|
|
548
|
+
│ ├── spec.md # Phase 4
|
|
549
|
+
│ ├── requirements.md # Phase 4
|
|
550
|
+
│ ├── implementation-plan.md # Phase 6
|
|
551
|
+
│ ├── work-log.md # Phase 7
|
|
584
552
|
│ ├── tdd-red-gate.md # Phase 3 (conditional)
|
|
585
|
-
│ └── tdd-green-gate.md # Phase
|
|
553
|
+
│ └── tdd-green-gate.md # Phase 8 (conditional)
|
|
586
554
|
├── verification/
|
|
587
|
-
│ ├── spec-audit.md # Phase
|
|
588
|
-
│ ├── implementation-verification.md # Phase
|
|
589
|
-
│ └── e2e-verification-report.md # Phase
|
|
555
|
+
│ ├── spec-audit.md # Phase 5 (recommended)
|
|
556
|
+
│ ├── implementation-verification.md # Phase 10
|
|
557
|
+
│ └── e2e-verification-report.md # Phase 11 (optional)
|
|
590
558
|
└── documentation/
|
|
591
|
-
└── user-guide.md # Phase
|
|
559
|
+
└── user-guide.md # Phase 12 (optional)
|
|
592
560
|
```
|
|
593
561
|
|
|
594
562
|
---
|
|
@@ -652,9 +620,9 @@ When research context is detected, read these files from the research folder:
|
|
|
652
620
|
| -------------------- | --------------------------------- | ---------------------------------------------- |
|
|
653
621
|
| State | `orchestrator-state.yml` | research_type, confidence_level |
|
|
654
622
|
| Report | `outputs/research-report.md` | Main findings and conclusions |
|
|
655
|
-
| Solution Exploration | `outputs/solution-exploration.md` | Alternatives and trade-offs (input to Phase
|
|
656
|
-
| High-Level Design | `outputs/high-level-design.md` | C4 architecture (input to Phase
|
|
657
|
-
| Decision Log | `outputs/decision-log.md` | ADR decisions (input to Phase
|
|
623
|
+
| Solution Exploration | `outputs/solution-exploration.md` | Alternatives and trade-offs (input to Phase 4) |
|
|
624
|
+
| High-Level Design | `outputs/high-level-design.md` | C4 architecture (input to Phase 4) |
|
|
625
|
+
| Decision Log | `outputs/decision-log.md` | ADR decisions (input to Phase 4) |
|
|
658
626
|
|
|
659
627
|
### How Research Informs Each Phase
|
|
660
628
|
|
|
@@ -664,8 +632,8 @@ When research context is detected, read these files from the research folder:
|
|
|
664
632
|
| ------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
|
|
665
633
|
| Phase 1 | Codebase analyzer receives research findings as search guidance |
|
|
666
634
|
| Phase 2 | Gap analyzer uses research recommendations for comparison |
|
|
667
|
-
| Phase
|
|
668
|
-
| Phase
|
|
635
|
+
| Phase 4 | Specification creator uses high-level-design.md as INPUT (still creates full spec). Architecture decisions use research report AND decision-log.md (lighter when ADRs comprehensive) |
|
|
636
|
+
| Phase 6 | Implementation planner references research approach for task grouping |
|
|
669
637
|
|
|
670
638
|
---
|
|
671
639
|
|
|
@@ -681,4 +649,4 @@ Invoked via:
|
|
|
681
649
|
## TDD Gate Rules
|
|
682
650
|
|
|
683
651
|
**Phase 3 (Red Gate)**: Test MUST FAIL before implementation (activated when gap-analyzer detects reproducible defect)
|
|
684
|
-
**Phase
|
|
652
|
+
**Phase 8 (Green Gate)**: Test MUST PASS after implementation (activated when Phase 3 was executed)
|
|
@@ -23,6 +23,7 @@ Systematic migration workflow from current state analysis to verified migration
|
|
|
23
23
|
1. **Create Task Items**: Use `TaskCreate` for all phases (see Phase Configuration), then set dependencies with `TaskUpdate addBlockedBy`
|
|
24
24
|
2. **Create Task Directory**: `.owflow/tasks/migrations/YYYY-MM-DD-task-name/`
|
|
25
25
|
3. **Initialize State**: Create `orchestrator-state.yml` with migration context
|
|
26
|
+
- **CRITICAL**: Use the `verify_template` tool immediately after creation to check YAML validity against `orchestrator-state-migration.yml`.
|
|
26
27
|
4. **Discover project documentation**: Read `.owflow/docs/INDEX.md` (if exists), extract ALL file paths from the "Project Documentation" section — includes predefined docs AND any user-added project docs. Store as `project_context.project_doc_paths` in state.
|
|
27
28
|
|
|
28
29
|
**Output**:
|
|
@@ -308,39 +309,7 @@ question - Display executive summary: total issues found, issues fixed, issues r
|
|
|
308
309
|
|
|
309
310
|
Migration-specific fields in `orchestrator-state.yml`:
|
|
310
311
|
|
|
311
|
-
|
|
312
|
-
migration_context:
|
|
313
|
-
migration_type: "code" | "data" | "architecture" | "general"
|
|
314
|
-
current_system:
|
|
315
|
-
description: null
|
|
316
|
-
technologies: []
|
|
317
|
-
target_system:
|
|
318
|
-
description: null
|
|
319
|
-
technologies: []
|
|
320
|
-
migration_strategy:
|
|
321
|
-
approach: "incremental" | "big-bang" | "dual-run" | "phased"
|
|
322
|
-
phases: []
|
|
323
|
-
risk_level: null
|
|
324
|
-
breaking_changes: []
|
|
325
|
-
rollback_plan_created: false
|
|
326
|
-
dual_run_configured: false
|
|
327
|
-
|
|
328
|
-
external_research:
|
|
329
|
-
performed: false
|
|
330
|
-
category: null
|
|
331
|
-
breaking_changes: []
|
|
332
|
-
migration_guide_url: null
|
|
333
|
-
|
|
334
|
-
verification_context:
|
|
335
|
-
last_status: null
|
|
336
|
-
issues_found: null
|
|
337
|
-
fixes_applied: []
|
|
338
|
-
decisions_made: []
|
|
339
|
-
reverify_count: 0
|
|
340
|
-
|
|
341
|
-
options:
|
|
342
|
-
docs_enabled: false
|
|
343
|
-
```
|
|
312
|
+
Refer to the template [src/templates/orchestrator-state-migration.yml](../../templates/orchestrator-state-migration.yml).
|
|
344
313
|
|
|
345
314
|
---
|
|
346
315
|
|
|
@@ -10,6 +10,7 @@ Before considering an orchestrator complete, verify ALL items:
|
|
|
10
10
|
|
|
11
11
|
- [ ] **Step 0: Load Framework** — Initialization reads `orchestrator-patterns.md`
|
|
12
12
|
- [ ] **State file creation** — Explicit step to CREATE `orchestrator-state.yml`
|
|
13
|
+
- [ ] **State verification** — Explicit step to use `verify_template` to check YAML validity after creation
|
|
13
14
|
- [ ] **Phase structure** — Each phase has: Purpose, Execute, Output, State, Transition (`→ Pause` / `→ AUTO-CONTINUE` / `→ Conditional`)
|
|
14
15
|
- [ ] **Delegation enforcement** — Each delegated phase has: ANTI-PATTERN block, INVOKE NOW block, SELF-CHECK
|
|
15
16
|
- [ ] **POST-CONTINUATION blocks** — After Skill tool phases, explicit instructions to read state, update completed_phases, and continue
|
|
@@ -163,42 +163,7 @@ All orchestrators use `orchestrator-state.yml` at `.owflow/tasks/[type]/YYYY-MM-
|
|
|
163
163
|
|
|
164
164
|
### Common Fields
|
|
165
165
|
|
|
166
|
-
|
|
167
|
-
orchestrator:
|
|
168
|
-
# Phase tracking
|
|
169
|
-
started_phase: [phase-name]
|
|
170
|
-
completed_phases: []
|
|
171
|
-
failed_phases: []
|
|
172
|
-
|
|
173
|
-
# Auto-fix tracking (per phase)
|
|
174
|
-
auto_fix_attempts:
|
|
175
|
-
phase-1: 0
|
|
176
|
-
phase-2: 0
|
|
177
|
-
|
|
178
|
-
# Optional phase flags
|
|
179
|
-
options:
|
|
180
|
-
e2e_enabled: true | false | null
|
|
181
|
-
user_docs_enabled: true | false | null
|
|
182
|
-
code_review_enabled: true | false | null
|
|
183
|
-
|
|
184
|
-
# Timestamps
|
|
185
|
-
created: [ISO 8601 timestamp]
|
|
186
|
-
updated: [ISO 8601 timestamp]
|
|
187
|
-
task_path: .owflow/tasks/[type]/YYYY-MM-DD-task-name
|
|
188
|
-
|
|
189
|
-
# Task tracking IDs (maps phase names to TaskCreate IDs)
|
|
190
|
-
task_ids:
|
|
191
|
-
phase-1: null
|
|
192
|
-
phase-2: null
|
|
193
|
-
|
|
194
|
-
# Task metadata
|
|
195
|
-
task:
|
|
196
|
-
title: [human-readable task title]
|
|
197
|
-
description: [full task description]
|
|
198
|
-
status: pending | in_progress | completed | failed | blocked
|
|
199
|
-
tags: []
|
|
200
|
-
priority: null # high | medium | low
|
|
201
|
-
```
|
|
166
|
+
Refer to the template [src/templates/orchestrator-state-base.yml](../../../templates/orchestrator-state-base.yml).
|
|
202
167
|
|
|
203
168
|
### Extension Pattern
|
|
204
169
|
|
|
@@ -217,21 +182,7 @@ See each orchestrator's SKILL.md "Domain Context" section for full schema.
|
|
|
217
182
|
|
|
218
183
|
When development starts from completed research (`--research` flag):
|
|
219
184
|
|
|
220
|
-
|
|
221
|
-
task_context:
|
|
222
|
-
research_reference:
|
|
223
|
-
path: null
|
|
224
|
-
research_question: null
|
|
225
|
-
research_type: null # technical | requirements | literature | mixed
|
|
226
|
-
confidence_level: null # high | medium | low
|
|
227
|
-
|
|
228
|
-
phase_summaries:
|
|
229
|
-
research:
|
|
230
|
-
summary: null
|
|
231
|
-
key_findings: []
|
|
232
|
-
recommended_approach: null
|
|
233
|
-
decisions_made: []
|
|
234
|
-
```
|
|
185
|
+
Refer to `research_reference` in the template [src/templates/orchestrator-state-development.yml](../../../templates/orchestrator-state-development.yml).
|
|
235
186
|
|
|
236
187
|
Research context flows to ALL phases via context passing. Artifacts are also copied to `analysis/research-context/`.
|
|
237
188
|
|
|
@@ -239,14 +190,7 @@ Research context flows to ALL phases via context passing. Artifacts are also cop
|
|
|
239
190
|
|
|
240
191
|
All orchestrators with verification phases use:
|
|
241
192
|
|
|
242
|
-
|
|
243
|
-
verification_context:
|
|
244
|
-
last_status: passed | passed_with_issues | failed | null
|
|
245
|
-
issues_found: []
|
|
246
|
-
fixes_applied: []
|
|
247
|
-
decisions_made: []
|
|
248
|
-
reverify_count: 0 # max 3
|
|
249
|
-
```
|
|
193
|
+
Refer to `verification_context` in the template [src/templates/orchestrator-state-base.yml](../../../templates/orchestrator-state-base.yml) or domain templates.
|
|
250
194
|
|
|
251
195
|
---
|
|
252
196
|
|
|
@@ -258,6 +202,7 @@ verification_context:
|
|
|
258
202
|
2. **Determine starting phase**: New task starts Phase 1; resume reads state for first incomplete phase
|
|
259
203
|
3. **Create task directory**: Standard structure with analysis/, implementation/, verification/, documentation/ _(skip on resume)_
|
|
260
204
|
4. **Create state file**: `orchestrator-state.yml` _(skip on resume)_
|
|
205
|
+
- **CRITICAL**: Use the `verify_template` tool immediately after creation to check YAML validity against the corresponding template.
|
|
261
206
|
5. **Create task items**: `TaskCreate` for all phases, then `TaskUpdate addBlockedBy` for dependencies. On resume, also restore completed phase statuses.
|
|
262
207
|
6. **Output summary**: Show task info, phases, starting message
|
|
263
208
|
|
|
@@ -24,6 +24,7 @@ Static-analysis-first performance optimization workflow. Identifies bottlenecks
|
|
|
24
24
|
2. **Create Task Directory**: `.owflow/tasks/performance/YYYY-MM-DD-task-name/`
|
|
25
25
|
3. **Create Subdirectories**: `analysis/`, `analysis/user-profiling-data/`, `implementation/`, `verification/`
|
|
26
26
|
4. **Initialize State**: Create `orchestrator-state.yml` with performance context
|
|
27
|
+
- **CRITICAL**: Use the `verify_template` tool immediately after creation to check YAML validity against `orchestrator-state-performance.yml`.
|
|
27
28
|
5. **Discover project documentation**: Read `.owflow/docs/INDEX.md` (if exists), extract ALL file paths from the "Project Documentation" section — includes predefined docs AND any user-added project docs. Store as `project_context.project_doc_paths` in state.
|
|
28
29
|
|
|
29
30
|
**Output**:
|
|
@@ -353,36 +354,7 @@ question - Display executive summary: total issues found, issues fixed, issues r
|
|
|
353
354
|
|
|
354
355
|
Performance-specific fields in `orchestrator-state.yml`:
|
|
355
356
|
|
|
356
|
-
|
|
357
|
-
performance_context:
|
|
358
|
-
bottlenecks_identified: null # count from bottleneck-analyzer
|
|
359
|
-
user_data_available: false # whether user provided profiling data
|
|
360
|
-
bottleneck_priorities:
|
|
361
|
-
p0: 0
|
|
362
|
-
p1: 0
|
|
363
|
-
p2: 0
|
|
364
|
-
p3: 0
|
|
365
|
-
phase_summaries:
|
|
366
|
-
codebase_analysis: { key_files: [], summary: null }
|
|
367
|
-
bottleneck_analysis:
|
|
368
|
-
{ bottlenecks: [], summary: null, user_data_incorporated: false }
|
|
369
|
-
specification: { summary: null }
|
|
370
|
-
|
|
371
|
-
verification_context:
|
|
372
|
-
last_status: null
|
|
373
|
-
issues_found: null
|
|
374
|
-
fixes_applied: []
|
|
375
|
-
decisions_made: []
|
|
376
|
-
reverify_count: 0
|
|
377
|
-
|
|
378
|
-
options:
|
|
379
|
-
spec_audit_enabled: null
|
|
380
|
-
skip_test_suite: true
|
|
381
|
-
code_review_enabled: true
|
|
382
|
-
pragmatic_review_enabled: true
|
|
383
|
-
reality_check_enabled: true
|
|
384
|
-
production_check_enabled: null
|
|
385
|
-
```
|
|
357
|
+
Refer to the template [src/templates/orchestrator-state-performance.yml](../../templates/orchestrator-state-performance.yml).
|
|
386
358
|
|
|
387
359
|
---
|
|
388
360
|
|