@vpxa/aikit 0.1.1 → 0.1.3
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 +8 -8
- package/package.json +1 -1
- package/packages/aikit-client/dist/types.d.ts +3 -3
- package/packages/cli/dist/aikit-init.d.ts +1 -1
- package/packages/cli/dist/aikit-init.js +1 -1
- package/packages/cli/dist/commands/context-cmds.js +1 -1
- package/packages/cli/dist/commands/environment.js +1 -1
- package/packages/cli/dist/commands/init/config.d.ts +1 -1
- package/packages/cli/dist/commands/init/config.js +2 -2
- package/packages/cli/dist/commands/init/constants.d.ts +3 -3
- package/packages/cli/dist/commands/init/index.d.ts +4 -4
- package/packages/cli/dist/commands/init/index.js +4 -4
- package/packages/cli/dist/commands/init/templates.js +12 -12
- package/packages/cli/dist/commands/init/user.d.ts +3 -3
- package/packages/cli/dist/commands/init/user.js +2 -2
- package/packages/cli/dist/commands/search.js +1 -1
- package/packages/cli/dist/commands/system.js +4 -4
- package/packages/cli/dist/commands/upgrade.js +1 -1
- package/packages/cli/dist/commands/workspace.js +1 -1
- package/packages/cli/dist/helpers.js +1 -1
- package/packages/cli/dist/index.d.ts +1 -1
- package/packages/core/dist/constants.d.ts +3 -3
- package/packages/core/dist/global-registry.d.ts +7 -1
- package/packages/core/dist/global-registry.js +1 -1
- package/packages/core/dist/index.d.ts +2 -2
- package/packages/core/dist/index.js +1 -1
- package/packages/core/dist/types.d.ts +6 -2
- package/packages/dashboard/dist/assets/{index-BjA4YODs.js → index-CO2S9BKY.js} +2 -2
- package/packages/dashboard/dist/assets/index-CO2S9BKY.js.map +1 -0
- package/packages/dashboard/dist/index.html +2 -2
- package/packages/enterprise-bridge/dist/er-client.d.ts +1 -1
- package/packages/flows/dist/git.js +1 -1
- package/packages/flows/dist/registry.d.ts +3 -3
- package/packages/flows/dist/registry.js +1 -1
- package/packages/flows/dist/symlinks.js +1 -1
- package/packages/indexer/dist/filesystem-crawler.js +1 -1
- package/packages/indexer/dist/hash-cache.js +1 -1
- package/packages/indexer/dist/incremental-indexer.js +1 -1
- package/packages/kb-client/dist/direct-client.d.ts +37 -0
- package/packages/kb-client/dist/direct-client.js +1 -0
- package/packages/kb-client/dist/index.d.ts +5 -0
- package/packages/kb-client/dist/index.js +1 -0
- package/packages/kb-client/dist/mcp-client.d.ts +19 -0
- package/packages/kb-client/dist/mcp-client.js +4 -0
- package/packages/kb-client/dist/parsers.d.ts +35 -0
- package/packages/kb-client/dist/parsers.js +2 -0
- package/packages/kb-client/dist/types.d.ts +62 -0
- package/packages/kb-client/dist/types.js +1 -0
- package/packages/present/dist/index.html +28 -28
- package/packages/server/dist/background-task.d.ts +47 -0
- package/packages/server/dist/background-task.js +1 -0
- package/packages/server/dist/config.js +1 -1
- package/packages/server/dist/idle-timer.d.ts +33 -0
- package/packages/server/dist/idle-timer.js +1 -0
- package/packages/server/dist/index.js +1 -1
- package/packages/server/dist/memory-monitor.d.ts +37 -0
- package/packages/server/dist/memory-monitor.js +1 -0
- package/packages/server/dist/prompts.js +5 -5
- package/packages/server/dist/resource-links.d.ts +1 -1
- package/packages/server/dist/resource-links.js +1 -1
- package/packages/server/dist/resources/curated-resources.d.ts +2 -2
- package/packages/server/dist/resources/curated-resources.js +2 -2
- package/packages/server/dist/resources/resource-notifier.d.ts +1 -1
- package/packages/server/dist/resources/resource-notifier.js +1 -1
- package/packages/server/dist/resources/resources.js +1 -1
- package/packages/server/dist/server.d.ts +4 -2
- package/packages/server/dist/server.js +3 -3
- package/packages/server/dist/tool-metadata.d.ts +1 -1
- package/packages/server/dist/tool-metadata.js +1 -1
- package/packages/server/dist/tool-timeout.d.ts +27 -0
- package/packages/server/dist/tool-timeout.js +1 -0
- package/packages/server/dist/tools/bridge.tools.d.ts +1 -1
- package/packages/server/dist/tools/bridge.tools.js +3 -3
- package/packages/server/dist/tools/config.tool.d.ts +8 -0
- package/packages/server/dist/tools/config.tool.js +12 -0
- package/packages/server/dist/tools/evolution.tools.js +1 -1
- package/packages/server/dist/tools/flow.tools.js +1 -1
- package/packages/server/dist/tools/infra.tools.js +1 -1
- package/packages/server/dist/tools/onboard.tool.js +1 -1
- package/packages/server/dist/tools/present/browser.js +9 -9
- package/packages/server/dist/tools/present/tool.js +4 -4
- package/packages/server/dist/tools/reindex.tool.js +1 -1
- package/packages/server/dist/tools/search.tool.js +4 -4
- package/packages/server/dist/tools/status.tool.d.ts +1 -1
- package/packages/server/dist/tools/status.tool.js +3 -3
- package/packages/store/dist/sqlite-graph-store.d.ts +3 -0
- package/packages/store/dist/sqlite-graph-store.js +3 -3
- package/packages/tools/dist/checkpoint.js +1 -1
- package/packages/tools/dist/config-extractor.js +1 -1
- package/packages/tools/dist/evidence-map.js +1 -1
- package/packages/tools/dist/find.d.ts +1 -1
- package/packages/tools/dist/forge-ground.d.ts +1 -1
- package/packages/tools/dist/guide.js +1 -1
- package/packages/tools/dist/lane.js +1 -1
- package/packages/tools/dist/onboard.d.ts +2 -2
- package/packages/tools/dist/onboard.js +2 -2
- package/packages/tools/dist/queue.js +1 -1
- package/packages/tools/dist/replay.js +1 -1
- package/packages/tools/dist/response-envelope.d.ts +1 -1
- package/packages/tools/dist/restore-points.js +1 -1
- package/packages/tools/dist/schema-validate.js +1 -1
- package/packages/tools/dist/snippet.js +1 -1
- package/packages/tools/dist/stash.js +1 -1
- package/packages/tools/dist/synthesis-engine.js +2 -2
- package/packages/tools/dist/workset.js +1 -1
- package/packages/tui/dist/{App-DU2KEylW.js → App-DpjN3iS-.js} +1 -1
- package/packages/tui/dist/App.d.ts +1 -1
- package/packages/tui/dist/App.js +1 -1
- package/packages/tui/dist/LogPanel-Db-SeZhR.js +3 -0
- package/packages/tui/dist/hooks/useKBClient.d.ts +1 -1
- package/packages/tui/dist/{index-BXafekwr.d.ts → index-MXJeXmCf.d.ts} +3 -3
- package/packages/tui/dist/index.d.ts +1 -1
- package/packages/tui/dist/index.js +1 -1
- package/packages/tui/dist/panels/LogPanel.js +1 -1
- package/scaffold/README.md +192 -192
- package/scaffold/definitions/bodies.mjs +16 -16
- package/scaffold/definitions/plugins.mjs +1 -1
- package/scaffold/definitions/prompts.mjs +6 -6
- package/scaffold/definitions/protocols.mjs +12 -12
- package/scaffold/definitions/tools.mjs +1 -1
- package/scaffold/flows/aikit-advanced/skills/execute/SKILL.md +124 -124
- package/scaffold/flows/aikit-advanced/skills/plan/SKILL.md +100 -100
- package/scaffold/flows/aikit-advanced/skills/spec/SKILL.md +100 -100
- package/scaffold/flows/aikit-advanced/skills/task/SKILL.md +99 -99
- package/scaffold/flows/aikit-advanced/skills/verify/SKILL.md +122 -122
- package/scaffold/flows/aikit-basic/skills/assess/SKILL.md +82 -82
- package/scaffold/flows/aikit-basic/skills/implement/SKILL.md +105 -105
- package/scaffold/flows/aikit-basic/skills/verify/SKILL.md +96 -96
- package/scaffold/general/agents/Debugger.agent.md +2 -2
- package/scaffold/general/agents/Documenter.agent.md +2 -2
- package/scaffold/general/agents/Explorer.agent.md +2 -2
- package/scaffold/general/agents/Frontend.agent.md +1 -1
- package/scaffold/general/agents/Implementer.agent.md +2 -2
- package/scaffold/general/agents/Orchestrator.agent.md +1 -1
- package/scaffold/general/agents/Planner.agent.md +2 -2
- package/scaffold/general/agents/Refactor.agent.md +2 -2
- package/scaffold/general/agents/Security.agent.md +2 -2
- package/scaffold/general/agents/_shared/architect-reviewer-base.md +1 -1
- package/scaffold/general/agents/_shared/code-agent-base.md +6 -6
- package/scaffold/general/agents/_shared/code-reviewer-base.md +1 -1
- package/scaffold/general/agents/_shared/forge-protocol.md +1 -1
- package/scaffold/general/agents/_shared/researcher-base.md +3 -3
- package/scaffold/general/prompts/ask.prompt.md +4 -4
- package/scaffold/general/prompts/debug.prompt.md +1 -1
- package/scaffold/general/prompts/plan.prompt.md +1 -1
- package/scaffold/general/skills/aikit/SKILL.md +5 -5
- package/scaffold/general/skills/multi-agents-development/SKILL.md +435 -435
- package/scaffold/general/skills/present/SKILL.md +424 -424
- package/packages/dashboard/dist/assets/index-BjA4YODs.js.map +0 -1
- package/packages/tui/dist/LogPanel-Bo8a8QXB.js +0 -3
|
@@ -1,122 +1,122 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: verify
|
|
3
|
-
description: Dual code review, architecture review, security review, and comprehensive test validation.
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Verification (Advanced)
|
|
7
|
-
|
|
8
|
-
## Purpose
|
|
9
|
-
|
|
10
|
-
Perform thorough multi-perspective validation of all changes through parallel dual code review, architecture review, and security analysis.
|
|
11
|
-
|
|
12
|
-
## Inputs
|
|
13
|
-
|
|
14
|
-
- `.spec/<slug>/spec.md` — original requirements and acceptance criteria
|
|
15
|
-
- `.spec/<slug>/plan.md` — architecture decisions and phase design
|
|
16
|
-
- `.spec/<slug>/tasks.md` — task breakdown with per-task acceptance criteria
|
|
17
|
-
- `.spec/<slug>/progress.md` — implementation status and changes made
|
|
18
|
-
|
|
19
|
-
## Process
|
|
20
|
-
|
|
21
|
-
1. **Load all artifacts** — Read spec, plan, tasks, and progress
|
|
22
|
-
2. **Dual code review** (parallel):
|
|
23
|
-
- Code-Reviewer-Alpha: focus on correctness, conventions, quality
|
|
24
|
-
- Code-Reviewer-Beta: focus on edge cases, error handling, maintainability
|
|
25
|
-
3. **Architecture review** (parallel with code review):
|
|
26
|
-
- Architect-Reviewer-Alpha: validate changes align with plan and ADRs
|
|
27
|
-
- Architect-Reviewer-Beta: assess long-term maintainability and evolution
|
|
28
|
-
4. **Security review**:
|
|
29
|
-
- Security agent: OWASP Top 10, auth/authz, input validation, secrets
|
|
30
|
-
5. **Quality gates** — `check({})` + `test_run({})` must pass
|
|
31
|
-
6. **Blast radius** — `blast_radius({ changed_files: [...] })` on all modified files
|
|
32
|
-
7. **Acceptance criteria** — Verify each spec acceptance criterion is met
|
|
33
|
-
8. **FORGE gate** — `evidence_map({ action: "gate" })` for final quality assessment
|
|
34
|
-
9. **Synthesize report** — Merge all reviewer findings into unified verdict
|
|
35
|
-
|
|
36
|
-
## Outputs
|
|
37
|
-
|
|
38
|
-
Produce `.spec/<slug>/verify-report.md` with:
|
|
39
|
-
|
|
40
|
-
```markdown
|
|
41
|
-
# Verification Report: <feature title>
|
|
42
|
-
|
|
43
|
-
## Verdict: PASS | FAIL
|
|
44
|
-
|
|
45
|
-
## Acceptance Criteria
|
|
46
|
-
- [x] <criterion> — verified by <method>
|
|
47
|
-
- [ ] <criterion> — FAILED: <reason>
|
|
48
|
-
|
|
49
|
-
## Quality Gates
|
|
50
|
-
- check: PASS/FAIL
|
|
51
|
-
- test_run: PASS/FAIL (<N> passed, <M> failed)
|
|
52
|
-
- blast_radius: <impact summary>
|
|
53
|
-
- FORGE gate: YIELD/HOLD/HARD_BLOCK
|
|
54
|
-
|
|
55
|
-
## Code Review (Alpha)
|
|
56
|
-
<findings with severity: critical/major/minor/suggestion>
|
|
57
|
-
|
|
58
|
-
## Code Review (Beta)
|
|
59
|
-
<findings with severity>
|
|
60
|
-
|
|
61
|
-
## Architecture Review
|
|
62
|
-
<alignment assessment, any concerns>
|
|
63
|
-
|
|
64
|
-
## Security Review
|
|
65
|
-
<vulnerabilities found, OWASP compliance>
|
|
66
|
-
|
|
67
|
-
## Recommendation
|
|
68
|
-
APPROVE | REQUEST CHANGES
|
|
69
|
-
|
|
70
|
-
### Required Changes (if any)
|
|
71
|
-
1. <change needed>
|
|
72
|
-
2. ...
|
|
73
|
-
```
|
|
74
|
-
|
|
75
|
-
## Agents
|
|
76
|
-
|
|
77
|
-
| Agent | Role |
|
|
78
|
-
|-------|------|
|
|
79
|
-
| **Code-Reviewer-Alpha** | Primary code review — correctness, quality, conventions |
|
|
80
|
-
| **Code-Reviewer-Beta** | Secondary code review — edge cases, error handling, maintainability |
|
|
81
|
-
| **Architect-Reviewer-Alpha** | Primary architecture review — alignment with plan and ADRs |
|
|
82
|
-
| **Architect-Reviewer-Beta** | Secondary architecture review — long-term evolution |
|
|
83
|
-
| **Security** | Security analysis — OWASP, auth, input validation |
|
|
84
|
-
|
|
85
|
-
**Parallelism**: All 5 reviewers can run in parallel — they are read-only.
|
|
86
|
-
|
|
87
|
-
## Foundation Integration
|
|
88
|
-
|
|
89
|
-
Load these skills BEFORE executing this step:
|
|
90
|
-
|
|
91
|
-
| Skill | Purpose | When |
|
|
92
|
-
|-------|---------|------|
|
|
93
|
-
| `aikit` | Core MCP tools — search, analyze, remember, validate | Always (auto-loaded) |
|
|
94
|
-
| `present` | Rich rendering for any structured output — assessments, reports, comparisons, reviews, status boards, tables, charts, and all artifact content | Use for ANY output that benefits from rich rendering, not just dashboards |
|
|
95
|
-
| `multi-agents-development` | Dispatch templates, task decomposition, review pipeline patterns | Before dispatching any subagent |
|
|
96
|
-
| `brainstorming` | Structured ideation for design/creative decisions | Before any design choice or new feature exploration |
|
|
97
|
-
|
|
98
|
-
### Presentation Rules
|
|
99
|
-
- Use `present` for **any output** that benefits from rich rendering — not limited to dashboards
|
|
100
|
-
- Assessments, reports, comparisons, reviews, status boards → `present({ format: "html" })`
|
|
101
|
-
- Tables, charts, progress tracking, code review findings → always present
|
|
102
|
-
- Artifact content and summaries → present with structured layout
|
|
103
|
-
- Only use plain text for brief confirmations and simple questions
|
|
104
|
-
|
|
105
|
-
### FORGE Quality Gate
|
|
106
|
-
|
|
107
|
-
After all reviews complete:
|
|
108
|
-
1. `evidence_map({ action: "gate", task_id: "<slug>" })` → returns YIELD/HOLD/HARD_BLOCK
|
|
109
|
-
2. YIELD → approved, proceed to commit
|
|
110
|
-
3. HOLD → minor issues, fix then re-gate (max 3 rounds)
|
|
111
|
-
4. HARD_BLOCK → critical issues, escalate to user
|
|
112
|
-
|
|
113
|
-
## Completion Criteria
|
|
114
|
-
|
|
115
|
-
- [ ] Dual code review complete (2 reviewers)
|
|
116
|
-
- [ ] Architecture review complete (2 reviewers)
|
|
117
|
-
- [ ] Security review complete
|
|
118
|
-
- [ ] `check({})` passes
|
|
119
|
-
- [ ] `test_run({})` passes
|
|
120
|
-
- [ ] All spec acceptance criteria verified
|
|
121
|
-
- [ ] FORGE gate passed (YIELD)
|
|
122
|
-
- [ ] `.spec/<slug>/verify-report.md` written with clear verdict
|
|
1
|
+
---
|
|
2
|
+
name: verify
|
|
3
|
+
description: Dual code review, architecture review, security review, and comprehensive test validation.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Verification (Advanced)
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
|
|
10
|
+
Perform thorough multi-perspective validation of all changes through parallel dual code review, architecture review, and security analysis.
|
|
11
|
+
|
|
12
|
+
## Inputs
|
|
13
|
+
|
|
14
|
+
- `.spec/<slug>/spec.md` — original requirements and acceptance criteria
|
|
15
|
+
- `.spec/<slug>/plan.md` — architecture decisions and phase design
|
|
16
|
+
- `.spec/<slug>/tasks.md` — task breakdown with per-task acceptance criteria
|
|
17
|
+
- `.spec/<slug>/progress.md` — implementation status and changes made
|
|
18
|
+
|
|
19
|
+
## Process
|
|
20
|
+
|
|
21
|
+
1. **Load all artifacts** — Read spec, plan, tasks, and progress
|
|
22
|
+
2. **Dual code review** (parallel):
|
|
23
|
+
- Code-Reviewer-Alpha: focus on correctness, conventions, quality
|
|
24
|
+
- Code-Reviewer-Beta: focus on edge cases, error handling, maintainability
|
|
25
|
+
3. **Architecture review** (parallel with code review):
|
|
26
|
+
- Architect-Reviewer-Alpha: validate changes align with plan and ADRs
|
|
27
|
+
- Architect-Reviewer-Beta: assess long-term maintainability and evolution
|
|
28
|
+
4. **Security review**:
|
|
29
|
+
- Security agent: OWASP Top 10, auth/authz, input validation, secrets
|
|
30
|
+
5. **Quality gates** — `check({})` + `test_run({})` must pass
|
|
31
|
+
6. **Blast radius** — `blast_radius({ changed_files: [...] })` on all modified files
|
|
32
|
+
7. **Acceptance criteria** — Verify each spec acceptance criterion is met
|
|
33
|
+
8. **FORGE gate** — `evidence_map({ action: "gate" })` for final quality assessment
|
|
34
|
+
9. **Synthesize report** — Merge all reviewer findings into unified verdict
|
|
35
|
+
|
|
36
|
+
## Outputs
|
|
37
|
+
|
|
38
|
+
Produce `.spec/<slug>/verify-report.md` with:
|
|
39
|
+
|
|
40
|
+
```markdown
|
|
41
|
+
# Verification Report: <feature title>
|
|
42
|
+
|
|
43
|
+
## Verdict: PASS | FAIL
|
|
44
|
+
|
|
45
|
+
## Acceptance Criteria
|
|
46
|
+
- [x] <criterion> — verified by <method>
|
|
47
|
+
- [ ] <criterion> — FAILED: <reason>
|
|
48
|
+
|
|
49
|
+
## Quality Gates
|
|
50
|
+
- check: PASS/FAIL
|
|
51
|
+
- test_run: PASS/FAIL (<N> passed, <M> failed)
|
|
52
|
+
- blast_radius: <impact summary>
|
|
53
|
+
- FORGE gate: YIELD/HOLD/HARD_BLOCK
|
|
54
|
+
|
|
55
|
+
## Code Review (Alpha)
|
|
56
|
+
<findings with severity: critical/major/minor/suggestion>
|
|
57
|
+
|
|
58
|
+
## Code Review (Beta)
|
|
59
|
+
<findings with severity>
|
|
60
|
+
|
|
61
|
+
## Architecture Review
|
|
62
|
+
<alignment assessment, any concerns>
|
|
63
|
+
|
|
64
|
+
## Security Review
|
|
65
|
+
<vulnerabilities found, OWASP compliance>
|
|
66
|
+
|
|
67
|
+
## Recommendation
|
|
68
|
+
APPROVE | REQUEST CHANGES
|
|
69
|
+
|
|
70
|
+
### Required Changes (if any)
|
|
71
|
+
1. <change needed>
|
|
72
|
+
2. ...
|
|
73
|
+
```
|
|
74
|
+
|
|
75
|
+
## Agents
|
|
76
|
+
|
|
77
|
+
| Agent | Role |
|
|
78
|
+
|-------|------|
|
|
79
|
+
| **Code-Reviewer-Alpha** | Primary code review — correctness, quality, conventions |
|
|
80
|
+
| **Code-Reviewer-Beta** | Secondary code review — edge cases, error handling, maintainability |
|
|
81
|
+
| **Architect-Reviewer-Alpha** | Primary architecture review — alignment with plan and ADRs |
|
|
82
|
+
| **Architect-Reviewer-Beta** | Secondary architecture review — long-term evolution |
|
|
83
|
+
| **Security** | Security analysis — OWASP, auth, input validation |
|
|
84
|
+
|
|
85
|
+
**Parallelism**: All 5 reviewers can run in parallel — they are read-only.
|
|
86
|
+
|
|
87
|
+
## Foundation Integration
|
|
88
|
+
|
|
89
|
+
Load these skills BEFORE executing this step:
|
|
90
|
+
|
|
91
|
+
| Skill | Purpose | When |
|
|
92
|
+
|-------|---------|------|
|
|
93
|
+
| `aikit` | Core MCP tools — search, analyze, remember, validate | Always (auto-loaded) |
|
|
94
|
+
| `present` | Rich rendering for any structured output — assessments, reports, comparisons, reviews, status boards, tables, charts, and all artifact content | Use for ANY output that benefits from rich rendering, not just dashboards |
|
|
95
|
+
| `multi-agents-development` | Dispatch templates, task decomposition, review pipeline patterns | Before dispatching any subagent |
|
|
96
|
+
| `brainstorming` | Structured ideation for design/creative decisions | Before any design choice or new feature exploration |
|
|
97
|
+
|
|
98
|
+
### Presentation Rules
|
|
99
|
+
- Use `present` for **any output** that benefits from rich rendering — not limited to dashboards
|
|
100
|
+
- Assessments, reports, comparisons, reviews, status boards → `present({ format: "html" })`
|
|
101
|
+
- Tables, charts, progress tracking, code review findings → always present
|
|
102
|
+
- Artifact content and summaries → present with structured layout
|
|
103
|
+
- Only use plain text for brief confirmations and simple questions
|
|
104
|
+
|
|
105
|
+
### FORGE Quality Gate
|
|
106
|
+
|
|
107
|
+
After all reviews complete:
|
|
108
|
+
1. `evidence_map({ action: "gate", task_id: "<slug>" })` → returns YIELD/HOLD/HARD_BLOCK
|
|
109
|
+
2. YIELD → approved, proceed to commit
|
|
110
|
+
3. HOLD → minor issues, fix then re-gate (max 3 rounds)
|
|
111
|
+
4. HARD_BLOCK → critical issues, escalate to user
|
|
112
|
+
|
|
113
|
+
## Completion Criteria
|
|
114
|
+
|
|
115
|
+
- [ ] Dual code review complete (2 reviewers)
|
|
116
|
+
- [ ] Architecture review complete (2 reviewers)
|
|
117
|
+
- [ ] Security review complete
|
|
118
|
+
- [ ] `check({})` passes
|
|
119
|
+
- [ ] `test_run({})` passes
|
|
120
|
+
- [ ] All spec acceptance criteria verified
|
|
121
|
+
- [ ] FORGE gate passed (YIELD)
|
|
122
|
+
- [ ] `.spec/<slug>/verify-report.md` written with clear verdict
|
|
@@ -1,82 +1,82 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: assess
|
|
3
|
-
description: Understand scope, analyze the codebase, and identify the implementation approach.
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Assessment
|
|
7
|
-
|
|
8
|
-
## Purpose
|
|
9
|
-
|
|
10
|
-
Analyze the task requirements and codebase to produce a clear, actionable assessment before any code changes begin.
|
|
11
|
-
|
|
12
|
-
## Inputs
|
|
13
|
-
|
|
14
|
-
- User's task description or issue reference
|
|
15
|
-
- Codebase (accessed via aikit MCP tools)
|
|
16
|
-
|
|
17
|
-
## Process
|
|
18
|
-
|
|
19
|
-
1. **Parse the goal** — Extract what needs to change, success criteria, and constraints
|
|
20
|
-
2. **Search for prior work** — `search({ query: "<task keywords>" })` to check for existing decisions or related code
|
|
21
|
-
3. **Map affected scope** — `scope_map({ task: "<description>" })` to identify files and modules involved
|
|
22
|
-
4. **Analyze structure** — `file_summary()` on each affected file; `compact()` for deeper sections
|
|
23
|
-
5. **Identify risks** — Note dependencies, breaking change potential, test coverage gaps
|
|
24
|
-
6. **Draft approach** — Outline the implementation strategy in 3–7 steps
|
|
25
|
-
|
|
26
|
-
## Outputs
|
|
27
|
-
|
|
28
|
-
Produce `.spec/<slug>/assessment.md` with:
|
|
29
|
-
|
|
30
|
-
```markdown
|
|
31
|
-
# Assessment: <task title>
|
|
32
|
-
|
|
33
|
-
## Goal
|
|
34
|
-
<what needs to happen>
|
|
35
|
-
|
|
36
|
-
## Affected Files
|
|
37
|
-
<list of files with brief reason>
|
|
38
|
-
|
|
39
|
-
## Approach
|
|
40
|
-
<numbered implementation steps>
|
|
41
|
-
|
|
42
|
-
## Risks
|
|
43
|
-
<potential issues and mitigations>
|
|
44
|
-
|
|
45
|
-
## Open Questions
|
|
46
|
-
<anything that needs clarification before proceeding>
|
|
47
|
-
```
|
|
48
|
-
|
|
49
|
-
## Agents
|
|
50
|
-
|
|
51
|
-
| Agent | Role |
|
|
52
|
-
|-------|------|
|
|
53
|
-
| **Explorer** | Rapid file discovery, dependency tracing, structural context |
|
|
54
|
-
| **Researcher-Alpha** | Deep analysis of complex logic, prior decisions, architectural implications |
|
|
55
|
-
|
|
56
|
-
Use Explorer first for breadth, then Researcher-Alpha for depth on complex areas.
|
|
57
|
-
|
|
58
|
-
## Foundation Integration
|
|
59
|
-
|
|
60
|
-
Load these skills BEFORE executing this step:
|
|
61
|
-
|
|
62
|
-
| Skill | Purpose | When |
|
|
63
|
-
|-------|---------|------|
|
|
64
|
-
| `aikit` | Core MCP tools — search, analyze, remember, validate | Always (auto-loaded) |
|
|
65
|
-
| `present` | Rich rendering for any structured output — assessments, reports, comparisons, reviews, status boards, tables, charts, and all artifact content | Use for ANY output that benefits from rich rendering, not just dashboards |
|
|
66
|
-
| `multi-agents-development` | Dispatch templates, task decomposition, review pipeline patterns | Before dispatching any subagent |
|
|
67
|
-
| `brainstorming` | Structured ideation for design/creative decisions | Before any design choice or new feature exploration |
|
|
68
|
-
|
|
69
|
-
### Presentation Rules
|
|
70
|
-
- Use `present` for **any output** that benefits from rich rendering — not limited to dashboards
|
|
71
|
-
- Assessments, reports, comparisons, reviews, status boards → `present({ format: "html" })`
|
|
72
|
-
- Tables, charts, progress tracking, code review findings → always present
|
|
73
|
-
- Artifact content and summaries → present with structured layout
|
|
74
|
-
- Only use plain text for brief confirmations and simple questions
|
|
75
|
-
|
|
76
|
-
## Completion Criteria
|
|
77
|
-
|
|
78
|
-
- [ ] All affected files identified
|
|
79
|
-
- [ ] Implementation approach is concrete (not vague)
|
|
80
|
-
- [ ] Risks documented with mitigations
|
|
81
|
-
- [ ] No unresolved open questions that block implementation
|
|
82
|
-
- [ ] `.spec/<slug>/assessment.md` written
|
|
1
|
+
---
|
|
2
|
+
name: assess
|
|
3
|
+
description: Understand scope, analyze the codebase, and identify the implementation approach.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Assessment
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
|
|
10
|
+
Analyze the task requirements and codebase to produce a clear, actionable assessment before any code changes begin.
|
|
11
|
+
|
|
12
|
+
## Inputs
|
|
13
|
+
|
|
14
|
+
- User's task description or issue reference
|
|
15
|
+
- Codebase (accessed via aikit MCP tools)
|
|
16
|
+
|
|
17
|
+
## Process
|
|
18
|
+
|
|
19
|
+
1. **Parse the goal** — Extract what needs to change, success criteria, and constraints
|
|
20
|
+
2. **Search for prior work** — `search({ query: "<task keywords>" })` to check for existing decisions or related code
|
|
21
|
+
3. **Map affected scope** — `scope_map({ task: "<description>" })` to identify files and modules involved
|
|
22
|
+
4. **Analyze structure** — `file_summary()` on each affected file; `compact()` for deeper sections
|
|
23
|
+
5. **Identify risks** — Note dependencies, breaking change potential, test coverage gaps
|
|
24
|
+
6. **Draft approach** — Outline the implementation strategy in 3–7 steps
|
|
25
|
+
|
|
26
|
+
## Outputs
|
|
27
|
+
|
|
28
|
+
Produce `.spec/<slug>/assessment.md` with:
|
|
29
|
+
|
|
30
|
+
```markdown
|
|
31
|
+
# Assessment: <task title>
|
|
32
|
+
|
|
33
|
+
## Goal
|
|
34
|
+
<what needs to happen>
|
|
35
|
+
|
|
36
|
+
## Affected Files
|
|
37
|
+
<list of files with brief reason>
|
|
38
|
+
|
|
39
|
+
## Approach
|
|
40
|
+
<numbered implementation steps>
|
|
41
|
+
|
|
42
|
+
## Risks
|
|
43
|
+
<potential issues and mitigations>
|
|
44
|
+
|
|
45
|
+
## Open Questions
|
|
46
|
+
<anything that needs clarification before proceeding>
|
|
47
|
+
```
|
|
48
|
+
|
|
49
|
+
## Agents
|
|
50
|
+
|
|
51
|
+
| Agent | Role |
|
|
52
|
+
|-------|------|
|
|
53
|
+
| **Explorer** | Rapid file discovery, dependency tracing, structural context |
|
|
54
|
+
| **Researcher-Alpha** | Deep analysis of complex logic, prior decisions, architectural implications |
|
|
55
|
+
|
|
56
|
+
Use Explorer first for breadth, then Researcher-Alpha for depth on complex areas.
|
|
57
|
+
|
|
58
|
+
## Foundation Integration
|
|
59
|
+
|
|
60
|
+
Load these skills BEFORE executing this step:
|
|
61
|
+
|
|
62
|
+
| Skill | Purpose | When |
|
|
63
|
+
|-------|---------|------|
|
|
64
|
+
| `aikit` | Core MCP tools — search, analyze, remember, validate | Always (auto-loaded) |
|
|
65
|
+
| `present` | Rich rendering for any structured output — assessments, reports, comparisons, reviews, status boards, tables, charts, and all artifact content | Use for ANY output that benefits from rich rendering, not just dashboards |
|
|
66
|
+
| `multi-agents-development` | Dispatch templates, task decomposition, review pipeline patterns | Before dispatching any subagent |
|
|
67
|
+
| `brainstorming` | Structured ideation for design/creative decisions | Before any design choice or new feature exploration |
|
|
68
|
+
|
|
69
|
+
### Presentation Rules
|
|
70
|
+
- Use `present` for **any output** that benefits from rich rendering — not limited to dashboards
|
|
71
|
+
- Assessments, reports, comparisons, reviews, status boards → `present({ format: "html" })`
|
|
72
|
+
- Tables, charts, progress tracking, code review findings → always present
|
|
73
|
+
- Artifact content and summaries → present with structured layout
|
|
74
|
+
- Only use plain text for brief confirmations and simple questions
|
|
75
|
+
|
|
76
|
+
## Completion Criteria
|
|
77
|
+
|
|
78
|
+
- [ ] All affected files identified
|
|
79
|
+
- [ ] Implementation approach is concrete (not vague)
|
|
80
|
+
- [ ] Risks documented with mitigations
|
|
81
|
+
- [ ] No unresolved open questions that block implementation
|
|
82
|
+
- [ ] `.spec/<slug>/assessment.md` written
|
|
@@ -1,105 +1,105 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: implement
|
|
3
|
-
description: Write code following the assessment plan, using TDD practices where applicable.
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Implementation
|
|
7
|
-
|
|
8
|
-
## Purpose
|
|
9
|
-
|
|
10
|
-
Execute the implementation plan from the assessment, writing production code and tests.
|
|
11
|
-
|
|
12
|
-
## Inputs
|
|
13
|
-
|
|
14
|
-
- `.spec/<slug>/assessment.md` — the approach, affected files, and risks
|
|
15
|
-
|
|
16
|
-
## Process
|
|
17
|
-
|
|
18
|
-
1. **Read assessment** — Load `.spec/<slug>/assessment.md` and internalize the approach
|
|
19
|
-
2. **Set up tests first** — Where applicable, write failing tests that define success
|
|
20
|
-
3. **Implement changes** — Follow the approach steps sequentially
|
|
21
|
-
- One logical change per commit-worthy chunk
|
|
22
|
-
- Stay within the assessed file scope — do not expand without re-assessment
|
|
23
|
-
4. **Run validation** — `check({})` for type/lint errors, `test_run({})` for test results
|
|
24
|
-
5. **Fix issues** — Iterate until `check` and `test_run` pass (max 3 rounds)
|
|
25
|
-
6. **Document progress** — Write progress artifact with what was done and any deviations
|
|
26
|
-
|
|
27
|
-
## Outputs
|
|
28
|
-
|
|
29
|
-
Produce `.spec/<slug>/progress.md` with:
|
|
30
|
-
|
|
31
|
-
```markdown
|
|
32
|
-
# Implementation Progress: <task title>
|
|
33
|
-
|
|
34
|
-
## Changes Made
|
|
35
|
-
<list of files changed with brief description>
|
|
36
|
-
|
|
37
|
-
## Tests
|
|
38
|
-
<tests added or modified>
|
|
39
|
-
|
|
40
|
-
## Deviations from Assessment
|
|
41
|
-
<any changes to the original plan and why>
|
|
42
|
-
|
|
43
|
-
## Validation
|
|
44
|
-
- check: PASS/FAIL
|
|
45
|
-
- test_run: PASS/FAIL (<N> passed, <M> failed)
|
|
46
|
-
|
|
47
|
-
## Notes
|
|
48
|
-
<anything the reviewer should know>
|
|
49
|
-
```
|
|
50
|
-
|
|
51
|
-
## Agents
|
|
52
|
-
|
|
53
|
-
| Agent | Role |
|
|
54
|
-
|-------|------|
|
|
55
|
-
| **Implementer** | Primary code writer — follows TDD, writes production code and tests |
|
|
56
|
-
| **Frontend** | UI/UX specialist — use when changes involve React components, styling, or responsive design |
|
|
57
|
-
|
|
58
|
-
Dispatch Implementer for backend/logic changes, Frontend for UI changes. Both can run in parallel if working on different files.
|
|
59
|
-
|
|
60
|
-
## Foundation Integration
|
|
61
|
-
|
|
62
|
-
Load these skills BEFORE executing this step:
|
|
63
|
-
|
|
64
|
-
| Skill | Purpose | When |
|
|
65
|
-
|-------|---------|------|
|
|
66
|
-
| `aikit` | Core MCP tools — search, analyze, remember, validate | Always (auto-loaded) |
|
|
67
|
-
| `present` | Rich rendering for any structured output — assessments, reports, comparisons, reviews, status boards, tables, charts, and all artifact content | Use for ANY output that benefits from rich rendering, not just dashboards |
|
|
68
|
-
| `multi-agents-development` | Dispatch templates, task decomposition, review pipeline patterns | Before dispatching any subagent |
|
|
69
|
-
| `brainstorming` | Structured ideation for design/creative decisions | Before any design choice or new feature exploration |
|
|
70
|
-
|
|
71
|
-
### Presentation Rules
|
|
72
|
-
- Use `present` for **any output** that benefits from rich rendering — not limited to dashboards
|
|
73
|
-
- Assessments, reports, comparisons, reviews, status boards → `present({ format: "html" })`
|
|
74
|
-
- Tables, charts, progress tracking, code review findings → always present
|
|
75
|
-
- Artifact content and summaries → present with structured layout
|
|
76
|
-
- Only use plain text for brief confirmations and simple questions
|
|
77
|
-
|
|
78
|
-
### Orchestrator Dispatch Protocol
|
|
79
|
-
|
|
80
|
-
Follow the `multi-agents-development` skill patterns for dispatch:
|
|
81
|
-
|
|
82
|
-
1. **Independence Check** before parallelizing:
|
|
83
|
-
- Same files? → sequential
|
|
84
|
-
- Shared mutable state? → sequential
|
|
85
|
-
- Execution-order dependent? → sequential
|
|
86
|
-
- Need shared new types? → define contract first, then parallel
|
|
87
|
-
- All clear? → **parallel dispatch**
|
|
88
|
-
|
|
89
|
-
2. **Subagent Context Template** (each dispatch includes):
|
|
90
|
-
- **Scope**: exact files + boundary (do NOT touch)
|
|
91
|
-
- **Goal**: acceptance criteria, testable
|
|
92
|
-
- **Arch Context**: actual code snippets via `compact()`/`digest()`
|
|
93
|
-
- **Constraints**: patterns, conventions, anti-patterns
|
|
94
|
-
- **Self-Review**: checklist before declaring DONE
|
|
95
|
-
|
|
96
|
-
3. **Status Protocol**: `DONE` | `DONE_WITH_CONCERNS` | `NEEDS_CONTEXT` | `BLOCKED`
|
|
97
|
-
4. **Max 2 retries** per task — then escalate to user
|
|
98
|
-
|
|
99
|
-
## Completion Criteria
|
|
100
|
-
|
|
101
|
-
- [ ] All assessment steps implemented
|
|
102
|
-
- [ ] `check({})` passes (no type/lint errors)
|
|
103
|
-
- [ ] `test_run({})` passes (no test failures)
|
|
104
|
-
- [ ] No files modified outside assessed scope
|
|
105
|
-
- [ ] `.spec/<slug>/progress.md` written
|
|
1
|
+
---
|
|
2
|
+
name: implement
|
|
3
|
+
description: Write code following the assessment plan, using TDD practices where applicable.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Implementation
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
|
|
10
|
+
Execute the implementation plan from the assessment, writing production code and tests.
|
|
11
|
+
|
|
12
|
+
## Inputs
|
|
13
|
+
|
|
14
|
+
- `.spec/<slug>/assessment.md` — the approach, affected files, and risks
|
|
15
|
+
|
|
16
|
+
## Process
|
|
17
|
+
|
|
18
|
+
1. **Read assessment** — Load `.spec/<slug>/assessment.md` and internalize the approach
|
|
19
|
+
2. **Set up tests first** — Where applicable, write failing tests that define success
|
|
20
|
+
3. **Implement changes** — Follow the approach steps sequentially
|
|
21
|
+
- One logical change per commit-worthy chunk
|
|
22
|
+
- Stay within the assessed file scope — do not expand without re-assessment
|
|
23
|
+
4. **Run validation** — `check({})` for type/lint errors, `test_run({})` for test results
|
|
24
|
+
5. **Fix issues** — Iterate until `check` and `test_run` pass (max 3 rounds)
|
|
25
|
+
6. **Document progress** — Write progress artifact with what was done and any deviations
|
|
26
|
+
|
|
27
|
+
## Outputs
|
|
28
|
+
|
|
29
|
+
Produce `.spec/<slug>/progress.md` with:
|
|
30
|
+
|
|
31
|
+
```markdown
|
|
32
|
+
# Implementation Progress: <task title>
|
|
33
|
+
|
|
34
|
+
## Changes Made
|
|
35
|
+
<list of files changed with brief description>
|
|
36
|
+
|
|
37
|
+
## Tests
|
|
38
|
+
<tests added or modified>
|
|
39
|
+
|
|
40
|
+
## Deviations from Assessment
|
|
41
|
+
<any changes to the original plan and why>
|
|
42
|
+
|
|
43
|
+
## Validation
|
|
44
|
+
- check: PASS/FAIL
|
|
45
|
+
- test_run: PASS/FAIL (<N> passed, <M> failed)
|
|
46
|
+
|
|
47
|
+
## Notes
|
|
48
|
+
<anything the reviewer should know>
|
|
49
|
+
```
|
|
50
|
+
|
|
51
|
+
## Agents
|
|
52
|
+
|
|
53
|
+
| Agent | Role |
|
|
54
|
+
|-------|------|
|
|
55
|
+
| **Implementer** | Primary code writer — follows TDD, writes production code and tests |
|
|
56
|
+
| **Frontend** | UI/UX specialist — use when changes involve React components, styling, or responsive design |
|
|
57
|
+
|
|
58
|
+
Dispatch Implementer for backend/logic changes, Frontend for UI changes. Both can run in parallel if working on different files.
|
|
59
|
+
|
|
60
|
+
## Foundation Integration
|
|
61
|
+
|
|
62
|
+
Load these skills BEFORE executing this step:
|
|
63
|
+
|
|
64
|
+
| Skill | Purpose | When |
|
|
65
|
+
|-------|---------|------|
|
|
66
|
+
| `aikit` | Core MCP tools — search, analyze, remember, validate | Always (auto-loaded) |
|
|
67
|
+
| `present` | Rich rendering for any structured output — assessments, reports, comparisons, reviews, status boards, tables, charts, and all artifact content | Use for ANY output that benefits from rich rendering, not just dashboards |
|
|
68
|
+
| `multi-agents-development` | Dispatch templates, task decomposition, review pipeline patterns | Before dispatching any subagent |
|
|
69
|
+
| `brainstorming` | Structured ideation for design/creative decisions | Before any design choice or new feature exploration |
|
|
70
|
+
|
|
71
|
+
### Presentation Rules
|
|
72
|
+
- Use `present` for **any output** that benefits from rich rendering — not limited to dashboards
|
|
73
|
+
- Assessments, reports, comparisons, reviews, status boards → `present({ format: "html" })`
|
|
74
|
+
- Tables, charts, progress tracking, code review findings → always present
|
|
75
|
+
- Artifact content and summaries → present with structured layout
|
|
76
|
+
- Only use plain text for brief confirmations and simple questions
|
|
77
|
+
|
|
78
|
+
### Orchestrator Dispatch Protocol
|
|
79
|
+
|
|
80
|
+
Follow the `multi-agents-development` skill patterns for dispatch:
|
|
81
|
+
|
|
82
|
+
1. **Independence Check** before parallelizing:
|
|
83
|
+
- Same files? → sequential
|
|
84
|
+
- Shared mutable state? → sequential
|
|
85
|
+
- Execution-order dependent? → sequential
|
|
86
|
+
- Need shared new types? → define contract first, then parallel
|
|
87
|
+
- All clear? → **parallel dispatch**
|
|
88
|
+
|
|
89
|
+
2. **Subagent Context Template** (each dispatch includes):
|
|
90
|
+
- **Scope**: exact files + boundary (do NOT touch)
|
|
91
|
+
- **Goal**: acceptance criteria, testable
|
|
92
|
+
- **Arch Context**: actual code snippets via `compact()`/`digest()`
|
|
93
|
+
- **Constraints**: patterns, conventions, anti-patterns
|
|
94
|
+
- **Self-Review**: checklist before declaring DONE
|
|
95
|
+
|
|
96
|
+
3. **Status Protocol**: `DONE` | `DONE_WITH_CONCERNS` | `NEEDS_CONTEXT` | `BLOCKED`
|
|
97
|
+
4. **Max 2 retries** per task — then escalate to user
|
|
98
|
+
|
|
99
|
+
## Completion Criteria
|
|
100
|
+
|
|
101
|
+
- [ ] All assessment steps implemented
|
|
102
|
+
- [ ] `check({})` passes (no type/lint errors)
|
|
103
|
+
- [ ] `test_run({})` passes (no test failures)
|
|
104
|
+
- [ ] No files modified outside assessed scope
|
|
105
|
+
- [ ] `.spec/<slug>/progress.md` written
|