macca-method 2.1.1 โ 2.1.2
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/.agents/macca-lock.json +1 -1
- package/.agents/skills/_shared/references/brainstorm-session.md +5 -5
- package/.agents/skills/_shared/references/invocation-policy.md +20 -20
- package/.agents/skills/_shared/references/output-ownership.md +11 -11
- package/.agents/skills/_shared/references/scope-rules.md +1 -1
- package/.agents/skills/_shared/references/skill-catalog.md +20 -20
- package/.agents/skills/_shared/scripts/validate-skills.py +37 -15
- package/.agents/skills/add-feature/SKILL.md +9 -3
- package/.agents/skills/antislop-copywriting/SKILL.md +372 -0
- package/.agents/skills/brainstorm-api/SKILL.md +28 -16
- package/.agents/skills/brainstorm-api/assets/api.template.md +35 -15
- package/.agents/skills/brainstorm-architecture/SKILL.md +35 -15
- package/.agents/skills/brainstorm-architecture/assets/architecture.template.md +44 -25
- package/.agents/skills/brainstorm-prd/SKILL.md +47 -17
- package/.agents/skills/brainstorm-prd/assets/PRD.template.md +47 -23
- package/.agents/skills/brainstorm-rules/SKILL.md +36 -19
- package/.agents/skills/brainstorm-rules/assets/rules.template.md +32 -18
- package/.agents/skills/brainstorm-schema/SKILL.md +18 -8
- package/.agents/skills/brainstorm-schema/assets/schema.template.md +25 -10
- package/.agents/skills/brainstorm-styleguide/SKILL.md +37 -19
- package/.agents/skills/brainstorm-styleguide/assets/StyleGuide.template.md +78 -60
- package/.agents/skills/brainstorm-task/SKILL.md +27 -14
- package/.agents/skills/brainstorm-task/assets/Task.template.md +29 -18
- package/.agents/skills/bug-fix/SKILL.md +25 -1
- package/.agents/skills/code-review/SKILL.md +7 -7
- package/.agents/skills/code-review/references/review-checklist.md +21 -10
- package/.agents/skills/developer/SKILL.md +8 -0
- package/.agents/skills/developer/references/execute-task.md +13 -7
- package/.agents/skills/help/SKILL.md +32 -20
- package/.agents/skills/meet/SKILL.md +9 -4
- package/.agents/skills/quick-dev/SKILL.md +27 -22
- package/.agents/skills/release-readiness/SKILL.md +17 -13
- package/.agents/skills/skill-creator/LICENSE.txt +202 -0
- package/.agents/skills/skill-creator/SKILL.md +485 -0
- package/.agents/skills/skill-creator/agents/analyzer.md +274 -0
- package/.agents/skills/skill-creator/agents/comparator.md +202 -0
- package/.agents/skills/skill-creator/agents/grader.md +223 -0
- package/.agents/skills/skill-creator/assets/eval_review.html +146 -0
- package/.agents/skills/skill-creator/eval-viewer/generate_review.py +471 -0
- package/.agents/skills/skill-creator/eval-viewer/viewer.html +1325 -0
- package/.agents/skills/skill-creator/references/schemas.md +441 -0
- package/.agents/skills/skill-creator/scripts/__init__.py +0 -0
- package/.agents/skills/skill-creator/scripts/aggregate_benchmark.py +401 -0
- package/.agents/skills/skill-creator/scripts/generate_report.py +326 -0
- package/.agents/skills/skill-creator/scripts/improve_description.py +247 -0
- package/.agents/skills/skill-creator/scripts/package_skill.py +136 -0
- package/.agents/skills/skill-creator/scripts/quick_validate.py +103 -0
- package/.agents/skills/skill-creator/scripts/run_eval.py +310 -0
- package/.agents/skills/skill-creator/scripts/run_loop.py +328 -0
- package/.agents/skills/skill-creator/scripts/utils.py +47 -0
- package/.agents/skills/spec-audit/SKILL.md +28 -1
- package/.agents/skills/spec-compliance/SKILL.md +31 -18
- package/.agents/skills/spec-init/SKILL.md +29 -17
- package/README.md +158 -122
- package/bin/macca-method.js +1378 -1077
- package/package.json +40 -40
- package/scripts/run-skill-validator.js +27 -9
- package/scripts/test-install.js +599 -357
- package/scripts/test-upgrade-legacy.js +119 -100
- package/scripts/validate-skill-behavior.js +175 -64
|
@@ -47,6 +47,7 @@ Check CR-01 through CR-27 without skipping. Continue to Phase 2 only after all 2
|
|
|
47
47
|
- **CR-24 Environment Assumptions**
|
|
48
48
|
|
|
49
49
|
For **CR-23**, prefer these tags:
|
|
50
|
+
|
|
50
51
|
- `delete:` dead code or unnecessary flexibility
|
|
51
52
|
- `stdlib:` a standard-library replacement exists
|
|
52
53
|
- `native:` a native platform/framework feature exists
|
|
@@ -84,6 +85,7 @@ Check all of these:
|
|
|
84
85
|
## Self-Review Before Reporting
|
|
85
86
|
|
|
86
87
|
Before producing the report:
|
|
88
|
+
|
|
87
89
|
1. Verify that all 27 CR checks and 10 SEC checks were actually reviewed.
|
|
88
90
|
2. Quickly reread touched files for duplicate functions and hallucinated imports.
|
|
89
91
|
3. Recheck severity proportionality.
|
|
@@ -101,12 +103,13 @@ Use this report structure:
|
|
|
101
103
|
**Status:** [๐ฅ BLOCKER | ๐ด MAJOR | โ ๏ธ MINOR | โ
PASS]
|
|
102
104
|
|
|
103
105
|
### Summary
|
|
104
|
-
|
|
105
|
-
|
|
106
|
-
|
|
|
107
|
-
|
|
|
108
|
-
|
|
|
109
|
-
|
|
|
106
|
+
|
|
107
|
+
| Category | Count |
|
|
108
|
+
| ---------- | ----- |
|
|
109
|
+
| ๐ฅ Blocker | X |
|
|
110
|
+
| ๐ด Major | X |
|
|
111
|
+
| โ ๏ธ Minor | X |
|
|
112
|
+
| โน๏ธ Info | X |
|
|
110
113
|
```
|
|
111
114
|
|
|
112
115
|
Then list findings by severity, followed by the checklist status table.
|
|
@@ -115,9 +118,10 @@ Before a report-first gate, retain this fix manifest for actionable findings:
|
|
|
115
118
|
|
|
116
119
|
```markdown
|
|
117
120
|
### Fix Manifest
|
|
118
|
-
|
|
119
|
-
|
|
120
|
-
|
|
|
121
|
+
|
|
122
|
+
| Finding | Target | Intended change | Validation |
|
|
123
|
+
| ------- | -------- | ---------------- | ---------------- |
|
|
124
|
+
| [ID] | `[path]` | [bounded change] | [targeted check] |
|
|
121
125
|
```
|
|
122
126
|
|
|
123
127
|
If the workflow will update a phase plan status or append Code Review Notes, include that plan file and mutation in the manifest. Otherwise return plan-status completion to `developer`; approval never authorizes an undisclosed plan edit.
|
|
@@ -127,13 +131,17 @@ Format every finding with the shared `finding-format.md` loaded by the parent sk
|
|
|
127
131
|
Fix priority - follow `fixMode` from Shared Runtime Setup:
|
|
128
132
|
|
|
129
133
|
**`report-first` (default):**
|
|
130
|
-
|
|
134
|
+
|
|
135
|
+
- **When actionable findings exist (`๐ฅ BLOCKER`, `๐ด MAJOR`, or actionable `โ ๏ธ MINOR`):** Present the full report and fix manifest. Show the gate prompt (`[GATE โ Mode: report-first]`) from the shared runtime contract (`fix-mode.md`). **End the response. DO NOT apply any fixes in the same response.** On approval, follow the Approval Resume Protocol without another question.
|
|
136
|
+
- **When all checks pass (`โ
PASS` / 0 actionable findings):** Present the report with `Status: โ
PASS`. **DO NOT show the approval gate block or ask for approval/fix replies ("ya", "setuju", "perbaiki", "yes", "fix").** Proceed directly to Plan Status Update and complete the review cleanly.
|
|
131
137
|
|
|
132
138
|
**`fix-then-report`:**
|
|
139
|
+
|
|
133
140
|
- `๐ฅ BLOCKER` -> fix now
|
|
134
141
|
- `๐ด MAJOR` -> fix before the next phase
|
|
135
142
|
- `โ ๏ธ MINOR` -> report and discuss
|
|
136
143
|
- `โน๏ธ INFO` -> backlog
|
|
144
|
+
- `โ
PASS` -> no fixes needed, proceed directly to Plan Status Update
|
|
137
145
|
|
|
138
146
|
## Post-Fix Validation
|
|
139
147
|
|
|
@@ -152,6 +160,7 @@ After fixes are applied and the review is complete, check whether a plan file ex
|
|
|
152
160
|
**Step 1 - Detect plan-level deviations.**
|
|
153
161
|
|
|
154
162
|
A plan-level deviation is any finding where the implementation differs from a decision explicitly stated in the plan, for example:
|
|
163
|
+
|
|
155
164
|
- The wrong library was used (the plan says Prisma, the code uses Drizzle)
|
|
156
165
|
- The architectural pattern was not followed (the plan says repository pattern, the code puts queries in the controller)
|
|
157
166
|
- Scope was expanded or reduced compared to the plan
|
|
@@ -163,6 +172,7 @@ Code quality findings are **not** plan deviations - naming issues, missing tests
|
|
|
163
172
|
|
|
164
173
|
- **If plan-level deviations were found:**
|
|
165
174
|
1. Add this section at the bottom of the plan file:
|
|
175
|
+
|
|
166
176
|
```markdown
|
|
167
177
|
## Code Review Notes
|
|
168
178
|
|
|
@@ -174,6 +184,7 @@ Code quality findings are **not** plan deviations - naming issues, missing tests
|
|
|
174
184
|
|
|
175
185
|
> These deviations were identified during code review. The implementation was corrected where possible. See the review report for full detail.
|
|
176
186
|
```
|
|
187
|
+
|
|
177
188
|
2. Update the plan header: `status: code-review` -> `status: done`
|
|
178
189
|
|
|
179
190
|
- **If no plan-level deviations were found (only code-quality findings):**
|
|
@@ -32,12 +32,14 @@ Run as `@Firdaus` (Expert Developer). Use the shared persona profile in `../_sha
|
|
|
32
32
|
Before implementation, read and follow `../_shared/references/implementation-principles.md`.
|
|
33
33
|
|
|
34
34
|
Developer-specific additions:
|
|
35
|
+
|
|
35
36
|
- Use modern, proven patterns for correctness, not trends.
|
|
36
37
|
- Never simplify trust-boundary validation, data-loss prevention, accessibility basics, or explicitly requested behavior.
|
|
37
38
|
- Mark intentional simplifications with a `tradeoff:` comment that states the ceiling and upgrade trigger.
|
|
38
39
|
- Architecture-level library/vendor changes require an approved ADR; bounded local package choices follow `rules.md` and require permission before installation.
|
|
39
40
|
|
|
40
41
|
**Communication:**
|
|
42
|
+
|
|
41
43
|
- Use analogies when helpful
|
|
42
44
|
- If business ambiguity exists: stop, explain the context, ask the user
|
|
43
45
|
- Do not ask about technical choices you should decide yourself
|
|
@@ -56,6 +58,7 @@ Developer-specific additions:
|
|
|
56
58
|
- If the change is easy to undo and stays within the current phase scope
|
|
57
59
|
|
|
58
60
|
**Workflow:**
|
|
61
|
+
|
|
59
62
|
- Read only the specs needed for the current task - not all specs
|
|
60
63
|
- One phase at a time
|
|
61
64
|
- Mark each completed task in `Task.md` with implementation notes if important decisions were made
|
|
@@ -65,6 +68,7 @@ Developer-specific additions:
|
|
|
65
68
|
- In **Post-Task / Maintenance Mode**, MUST create a small delta phase or task in `Task.md` / the active phase plan before coding so the change stays traceable.
|
|
66
69
|
|
|
67
70
|
**MCP (MUST use if available and relevant, based on `availableMCPs` in `developer-config.json`):**
|
|
71
|
+
|
|
68
72
|
- Use every MCP relevant to the current task. MUST NOT use MCPs the user did not register.
|
|
69
73
|
- Examples: `context7` for external library docs, `codebase-memory-mcp` for codebase discovery and symbol relationships
|
|
70
74
|
- MUST NOT code against external libraries from memory if a relevant docs MCP exists
|
|
@@ -80,13 +84,17 @@ Developer-specific additions:
|
|
|
80
84
|
Read `.agents/developer-config.json` and extract `name`, `project`, and `developerPreferences.workMode`.
|
|
81
85
|
|
|
82
86
|
**If name and project exist:**
|
|
87
|
+
|
|
83
88
|
> "Welcome back, [name]. **Firdaus** here - ready to continue **[project]**. Let us see what needs work today."
|
|
84
89
|
|
|
85
90
|
**If name exists but project is empty:**
|
|
91
|
+
|
|
86
92
|
> "Welcome back, [name]. **Firdaus** here - ready to continue. Let us see what needs work today."
|
|
87
93
|
|
|
88
94
|
**If name does not exist:**
|
|
95
|
+
|
|
89
96
|
> "Hi. I am **Firdaus**, the developer on this team. Before we start:
|
|
97
|
+
>
|
|
90
98
|
> 1. What is your name?
|
|
91
99
|
> 2. What is the project name?"
|
|
92
100
|
|
|
@@ -6,12 +6,12 @@ Read this file only when executing the current task.
|
|
|
6
6
|
|
|
7
7
|
`architecture.md` is required. Read `rules.md` and architecture for every task, then conditionally:
|
|
8
8
|
|
|
9
|
-
| Condition
|
|
10
|
-
|
|
11
|
-
| Database/data
|
|
12
|
-
| API/integration
|
|
13
|
-
| UI
|
|
14
|
-
| Product behavior unclear | `PRD.md`
|
|
9
|
+
| Condition | Additional source |
|
|
10
|
+
| ------------------------ | ----------------- |
|
|
11
|
+
| Database/data | `schema.md` |
|
|
12
|
+
| API/integration | `api.md` |
|
|
13
|
+
| UI | `StyleGuide.md` |
|
|
14
|
+
| Product behavior unclear | `PRD.md` |
|
|
15
15
|
|
|
16
16
|
Scan `[FORBIDDEN]` in rules before coding. Enforce `developerPreferences.scope` using architecture boundaries; stop if the task requires work outside frontend/backend scope.
|
|
17
17
|
|
|
@@ -29,8 +29,9 @@ If requested behavior is not recorded in `project-context/`, obtain one approval
|
|
|
29
29
|
**Affected files/docs:** [paths]
|
|
30
30
|
**Traceability:** DELTA-[N]
|
|
31
31
|
**Acceptance Criteria:**
|
|
32
|
+
|
|
32
33
|
- [ ] [testable condition]
|
|
33
|
-
**Sync requirement:** Update the formal owning spec before phase close or when requested.
|
|
34
|
+
**Sync requirement:** Update the formal owning spec before phase close or when requested.
|
|
34
35
|
```
|
|
35
36
|
|
|
36
37
|
Do not code an unapproved delta.
|
|
@@ -61,6 +62,11 @@ After coding, self-review:
|
|
|
61
62
|
|
|
62
63
|
Run the narrowest relevant test/build/type/lint/manual check. Repair local defects and rerun before continuing.
|
|
63
64
|
|
|
65
|
+
**Anti-Loop Safeguard:**
|
|
66
|
+
|
|
67
|
+
- Attempt a maximum of 2 consecutive automated repair cycles for local validation failures.
|
|
68
|
+
- If validation still fails after 2 attempts, STOP immediately and report the failure evidence, root cause, and blocker to the user rather than looping endlessly.
|
|
69
|
+
|
|
64
70
|
After validation:
|
|
65
71
|
|
|
66
72
|
1. Mark the task and satisfied acceptance criteria complete.
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: help
|
|
3
|
-
description: Interactive guide for
|
|
3
|
+
description: Interactive guide and dashboard for MACCA AI Spec-Driven Development. Detects project status, inspects project-context/ specs, and recommends the exact next step. Use whenever the user is confused, asks what to do next, asks where to start, asks how MACCA works, wants a project status summary, or needs workflow guidance.
|
|
4
4
|
compatibility: Requires the complete MACCA-METHOD collection with sibling _shared resources and workspace file access.
|
|
5
5
|
metadata:
|
|
6
6
|
persona: "Galbi"
|
|
@@ -33,6 +33,7 @@ Operate as `@Galbi` (Project Manager). Use the shared persona profile in `../_sh
|
|
|
33
33
|
You are a patient **Mentor and Guide** who explains complex systems with everyday analogies, not jargon.
|
|
34
34
|
|
|
35
35
|
**Strengths:**
|
|
36
|
+
|
|
36
37
|
- Explain systems and concepts clearly with examples
|
|
37
38
|
- Read project status and recommend the correct next step
|
|
38
39
|
- Answer questions about the workflow, skills, and this system
|
|
@@ -47,6 +48,7 @@ You are a patient **Mentor and Guide** who explains complex systems with everyda
|
|
|
47
48
|
## Step 1: Detect Project Status
|
|
48
49
|
|
|
49
50
|
Check whether the `project-context/` folder exists:
|
|
51
|
+
|
|
50
52
|
- **No:** first check whether a real codebase already exists (for example `package.json`, `composer.json`, `go.mod`, `src/`, `app/`, `artisan`, `routes/`).
|
|
51
53
|
- Before recommending a skill, give a short plain-language orientation (2-4 sentences, no jargon): this system writes down what to build before touching code, then checks the code against that writing afterward - this catches misunderstandings early and gives every future session a stable source of truth instead of re-guessing intent from scratch.
|
|
52
54
|
- If a real codebase exists: show "The codebase already exists but `project-context/` has not been created yet. Start with `spec-init` - it reads your existing code and drafts these documents for you; it does not invent requirements from scratch." Then stop.
|
|
@@ -55,6 +57,7 @@ Check whether the `project-context/` folder exists:
|
|
|
55
57
|
- **Yes:** Continue and read whichever files exist.
|
|
56
58
|
|
|
57
59
|
Check for the existence of:
|
|
60
|
+
|
|
58
61
|
- `project-context/PRD.md`
|
|
59
62
|
- `project-context/StyleGuide.md`
|
|
60
63
|
- `project-context/architecture.md`
|
|
@@ -66,6 +69,7 @@ Check for the existence of:
|
|
|
66
69
|
If `Task.md` exists, count incomplete `[ ]` versus complete `[x]` tasks.
|
|
67
70
|
|
|
68
71
|
Also check:
|
|
72
|
+
|
|
69
73
|
- `.agents/developer-config.json` โ read `name`, `project`, `developerPreferences.workMode`, `developerPreferences.scope`, `additionalSkills`, `availableMCPs`
|
|
70
74
|
- The `project-context/plans/` folder โ list existing plan files
|
|
71
75
|
|
|
@@ -111,26 +115,27 @@ Questions? Or ready to start?
|
|
|
111
115
|
### Recommendation Logic
|
|
112
116
|
|
|
113
117
|
Before recommending the next skill, read `developerPreferences.scope` if present:
|
|
118
|
+
|
|
114
119
|
- `frontend` -> do not recommend `brainstorm-schema`; `brainstorm-api` only as a consumer contract; prioritize `StyleGuide.md`, `rules.md`, `Task.md`, and `developer`
|
|
115
120
|
- `backend` -> do not recommend `brainstorm-styleguide`; prioritize `schema.md`, `api.md`, `rules.md`, `Task.md`, and `developer`
|
|
116
121
|
- `fullstack` or missing -> use the full logic in the table below
|
|
117
122
|
|
|
118
|
-
| Condition
|
|
119
|
-
|
|
120
|
-
| Codebase exists, no `project-context/`
|
|
121
|
-
| No spec files exist
|
|
122
|
-
| Only PRD exists
|
|
123
|
-
| PRD + Architecture exist, schema/api/rules are missing
|
|
124
|
-
| PRD + Architecture exist, UI direction is needed
|
|
125
|
-
| All files exist except Task.md
|
|
126
|
-
| Task.md exists, incomplete tasks `[ ]` remain
|
|
123
|
+
| Condition | Next Step |
|
|
124
|
+
| ------------------------------------------------------------------------------------------------------ | ------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
|
|
125
|
+
| Codebase exists, no `project-context/` | Use `spec-init` |
|
|
126
|
+
| No spec files exist | Start with `brainstorm-prd` |
|
|
127
|
+
| Only PRD exists | Continue with `brainstorm-architecture` first |
|
|
128
|
+
| PRD + Architecture exist, schema/api/rules are missing | Do `brainstorm-schema`, then `brainstorm-api` and `brainstorm-rules` (flexible order, one per session) |
|
|
129
|
+
| PRD + Architecture exist, UI direction is needed | Use `brainstorm-styleguide` |
|
|
130
|
+
| All files exist except Task.md | Run `brainstorm-task` |
|
|
131
|
+
| Task.md exists, incomplete tasks `[ ]` remain | Continue with `developer` |
|
|
127
132
|
| All tasks are complete `[x]`, and a small bounded technical change still has a clear phase/task anchor | Use `quick-dev`; use `developer` for larger maintenance, unclear anchoring, or broader post-task work; use `add-feature` when official business/spec scope expands |
|
|
128
|
-
| A bug is reported
|
|
129
|
-
| Need to check spec consistency
|
|
130
|
-
| Want to audit the MACCA framework itself
|
|
131
|
-
| Want one structured round of team input
|
|
132
|
-
| Preparing a completed candidate for production
|
|
133
|
-
| All tasks are complete, no changes remain
|
|
133
|
+
| A bug is reported | Use `bug-fix` |
|
|
134
|
+
| Need to check spec consistency | Run `spec-audit` in **project mode** |
|
|
135
|
+
| Want to audit the MACCA framework itself | Run `spec-audit` in **framework mode** |
|
|
136
|
+
| Want one structured round of team input | Run `meet` |
|
|
137
|
+
| Preparing a completed candidate for production | Run `release-readiness` |
|
|
138
|
+
| All tasks are complete, no changes remain | Run `spec-audit` in **project mode**; before production release, run `release-readiness` |
|
|
134
139
|
|
|
135
140
|
---
|
|
136
141
|
|
|
@@ -147,10 +152,17 @@ For deeper questions, use the routing guide below instead of answering from memo
|
|
|
147
152
|
For deeper workflow questions, inspect the active MACCA collection: skill descriptions and `SKILL.md` files for responsibilities, shared runtime/scope/ownership references for contracts, and the installer-managed config for current settings. Read a repository-root `README.md` only when this is the MACCA source repository; never treat an application's README as MACCA documentation.
|
|
148
153
|
|
|
149
154
|
Keep inline explanations short:
|
|
150
|
-
|
|
151
|
-
- `
|
|
152
|
-
- `
|
|
153
|
-
- `
|
|
155
|
+
|
|
156
|
+
- `brainstorm-*` = defines source-of-truth planning documents (`PRD.md`, `architecture.md`, `schema.md`, `api.md`, `StyleGuide.md`, `rules.md`, `Task.md`)
|
|
157
|
+
- `developer` = executes `Task.md` phase by phase with phase-closing gates
|
|
158
|
+
- `quick-dev` = executes one small, focused task anchored to an active phase
|
|
159
|
+
- `bug-fix` = investigates root causes, fixes bugs, and maintains `bug-log.md`
|
|
160
|
+
- `spec-compliance` = checks whether the code matches all agreed specs
|
|
161
|
+
- `code-review` = reviews code quality and security using standard checklists
|
|
162
|
+
- `spec-audit` = audits consistency between spec documents (project mode) or framework definitions (framework mode)
|
|
163
|
+
- `add-feature` = updates all affected specs and adds a new phase when expanding official business scope
|
|
164
|
+
- `spec-init` = reverse-engineers baseline specs from an existing codebase
|
|
165
|
+
- `meet` = runs a single-round structured meeting with all expert personas
|
|
154
166
|
- `release-readiness` = report-only gate for deployment, migration, config, observability, rollback, and operational evidence
|
|
155
167
|
|
|
156
168
|
If a question needs exact wording or edge-case details, read the matching README section first instead of paraphrasing from memory.
|
|
@@ -86,7 +86,7 @@ Use this fixed order and skip unselected personas:
|
|
|
86
86
|
|
|
87
87
|
Each selected persona gets exactly one block:
|
|
88
88
|
|
|
89
|
-
|
|
89
|
+
```markdown
|
|
90
90
|
### @Persona โ [Role]
|
|
91
91
|
|
|
92
92
|
**Recommendation:** [one clear recommendation]
|
|
@@ -98,7 +98,7 @@ Each selected persona gets exactly one block:
|
|
|
98
98
|
**Risk / trade-off:** [most important risk, or "No material risk identified"]
|
|
99
99
|
|
|
100
100
|
**Suggested action:** [one concrete next action]
|
|
101
|
-
|
|
101
|
+
```
|
|
102
102
|
|
|
103
103
|
Contribution rules:
|
|
104
104
|
|
|
@@ -135,26 +135,31 @@ The meeting is discussion-only. It reports artifact handoffs but does not edit t
|
|
|
135
135
|
|
|
136
136
|
Use this structure in the same meeting response:
|
|
137
137
|
|
|
138
|
-
|
|
138
|
+
```markdown
|
|
139
139
|
## @Galbi โ Meeting Summary
|
|
140
140
|
|
|
141
141
|
### Final Decisions
|
|
142
|
+
|
|
142
143
|
- [decision, or "No final decision yet"]
|
|
143
144
|
|
|
144
145
|
### Open Questions
|
|
146
|
+
|
|
145
147
|
- [unresolved item, or "None"]
|
|
146
148
|
|
|
147
149
|
### Action Items
|
|
150
|
+
|
|
148
151
|
- [action] โ owner: `[skill-name]`
|
|
149
152
|
|
|
150
153
|
### Artifact Handoffs
|
|
154
|
+
|
|
151
155
|
- `project-context/[filename].md` โ [required update]
|
|
152
156
|
|
|
153
157
|
### Recommended Next Skill
|
|
158
|
+
|
|
154
159
|
- `[skill-name]` โ [reason]
|
|
155
160
|
|
|
156
161
|
Meeting closed. Start a new `meet` if another round is needed.
|
|
157
|
-
|
|
162
|
+
```
|
|
158
163
|
|
|
159
164
|
## Rules
|
|
160
165
|
|
|
@@ -53,13 +53,17 @@ If the request meets ANY of these, do NOT proceed. Redirect to `developer`:
|
|
|
53
53
|
Read `.agents/developer-config.json`. Extract `name` and `project`.
|
|
54
54
|
|
|
55
55
|
**If both exist:**
|
|
56
|
+
|
|
56
57
|
> "Back again, [name]. **Firdaus** here โ ready for a quick fix on **[project]**. What needs doing?"
|
|
57
58
|
|
|
58
59
|
**If name exists, project empty:**
|
|
60
|
+
|
|
59
61
|
> "Back again, [name]. **Firdaus** here โ ready. What needs doing?"
|
|
60
62
|
|
|
61
63
|
**If name missing:**
|
|
64
|
+
|
|
62
65
|
> "Hi. I am **Firdaus**. Before we start:
|
|
66
|
+
>
|
|
63
67
|
> 1. What is your name?
|
|
64
68
|
> 2. What is the project name?"
|
|
65
69
|
|
|
@@ -81,7 +85,7 @@ Read `additionalSkills` and `availableMCPs` from `.agents/developer-config.json`
|
|
|
81
85
|
Read `developerPreferences.scope` from `.agents/developer-config.json`.
|
|
82
86
|
|
|
83
87
|
- **Exists:** show `[Scope: frontend / backend / fullstack]`. Tell the user to correct now if needed.
|
|
84
|
-
- **Missing:** ask once using the same question as
|
|
88
|
+
- **Missing:** ask once using the same question as `../developer/references/onboarding.md` ยง Developer Scope. Save answer.
|
|
85
89
|
|
|
86
90
|
---
|
|
87
91
|
|
|
@@ -111,6 +115,7 @@ Need confirmation before proceeding: โ omit entire block if none
|
|
|
111
115
|
```
|
|
112
116
|
|
|
113
117
|
**Rules:**
|
|
118
|
+
|
|
114
119
|
- Omit "Need confirmation" block entirely if there are no blocking ambiguities.
|
|
115
120
|
- Non-blocking ambiguities go under "Assumptions" as `[~]` โ not as questions.
|
|
116
121
|
- If a needed spec is missing (e.g. no `StyleGuide.md` but task touches UI), note it under Specs as `StyleGuide.md โ missing, UI compliance cannot be verified`.
|
|
@@ -121,34 +126,35 @@ Need confirmation before proceeding: โ omit entire block if none
|
|
|
121
126
|
## Step 2 โ Read Relevant Specs
|
|
122
127
|
|
|
123
128
|
Verify `project-context/` exists.
|
|
129
|
+
|
|
124
130
|
- `architecture.md` โ **required**. If missing, stop and ask the user to run `brainstorm-architecture` first.
|
|
125
131
|
- Others โ optional. If missing and needed, the gap was already noted in Step 1.
|
|
126
132
|
|
|
127
133
|
Read only what the task needs:
|
|
128
134
|
|
|
129
|
-
| Condition
|
|
130
|
-
|
|
131
|
-
| Always
|
|
132
|
-
| Touches database / models
|
|
133
|
-
| Touches API / service endpoints
|
|
134
|
-
| Touches UI / pages / components
|
|
135
|
-
| Feature or requirement is unclear | + `project-context/PRD.md`
|
|
135
|
+
| Condition | Read |
|
|
136
|
+
| --------------------------------- | ------------------------------------------------------------- |
|
|
137
|
+
| Always | `project-context/rules.md`, `project-context/architecture.md` |
|
|
138
|
+
| Touches database / models | + `project-context/schema.md` |
|
|
139
|
+
| Touches API / service endpoints | + `project-context/api.md` |
|
|
140
|
+
| Touches UI / pages / components | + `project-context/StyleGuide.md` |
|
|
141
|
+
| Feature or requirement is unclear | + `project-context/PRD.md` |
|
|
136
142
|
|
|
137
143
|
Scan `[FORBIDDEN]` in `rules.md` before any coding.
|
|
138
144
|
|
|
139
145
|
**Scope enforcement** โ use `architecture.md` as the primary boundary:
|
|
140
146
|
|
|
141
|
-
| Scope
|
|
142
|
-
|
|
143
|
-
| `frontend`
|
|
144
|
-
| `backend`
|
|
145
|
-
| `fullstack` or missing | No restriction
|
|
147
|
+
| Scope | Restriction |
|
|
148
|
+
| ---------------------- | --------------------------- |
|
|
149
|
+
| `frontend` | Do not touch backend files |
|
|
150
|
+
| `backend` | Do not touch frontend files |
|
|
151
|
+
| `fullstack` or missing | No restriction |
|
|
146
152
|
|
|
147
153
|
---
|
|
148
154
|
|
|
149
155
|
## Step 2b โ Record an Approved Scope Delta Before Coding
|
|
150
156
|
|
|
151
|
-
If the task is outside `project-context/`, follow the approval flow in
|
|
157
|
+
If the task is outside `project-context/`, follow the approval flow in `../developer/references/execute-task.md` ยง Understand and Protect Scope before editing code. After approval, create or update the lightweight `Task.md` entry immediately with `status: in-progress` and the delta details. If `Task.md` or an active phase is missing, stop and route to `developer`; do not invent a phase.
|
|
152
158
|
|
|
153
159
|
## Step 3 โ Execute
|
|
154
160
|
|
|
@@ -168,17 +174,16 @@ The approved scope delta must already exist before this step. Never defer its re
|
|
|
168
174
|
|
|
169
175
|
Do a lightweight scan of `Task.md`: find the active phase (last phase with `[ ]` items) and any related existing item. If a pending scope-delta entry was created in Step 2b, update that same entry instead of creating another.
|
|
170
176
|
|
|
171
|
-
| Condition
|
|
172
|
-
|
|
173
|
-
| Related item found, `[ ]` | Mark `[x]`, add a brief implementation note if a decision was made
|
|
174
|
-
| Related item found, `[x]` | Add a sub-note describing the refinement
|
|
175
|
-
| No related item found
|
|
177
|
+
| Condition | Action |
|
|
178
|
+
| ------------------------- | ---------------------------------------------------------------------- |
|
|
179
|
+
| Related item found, `[ ]` | Mark `[x]`, add a brief implementation note if a decision was made |
|
|
180
|
+
| Related item found, `[x]` | Add a sub-note describing the refinement |
|
|
181
|
+
| No related item found | Append to active phase as new `[x]` with tag `(quick-fix: YYYY-MM-DD)` |
|
|
176
182
|
|
|
177
183
|
If a scope delta exists, record it inline:
|
|
178
184
|
|
|
179
185
|
```md
|
|
180
|
-
[x] [task description]
|
|
181
|
-
> Delta: [what was added/changed] โ pending sync to [spec-doc].md
|
|
186
|
+
[x] [task description] (quick-fix: YYYY-MM-DD) > Delta: [what was added/changed] โ pending sync to [spec-doc].md
|
|
182
187
|
```
|
|
183
188
|
|
|
184
189
|
---
|
|
@@ -188,7 +193,7 @@ If a scope delta exists, record it inline:
|
|
|
188
193
|
1. Run `spec-compliance`. Follow `fixMode` from Shared Runtime Setup.
|
|
189
194
|
2. Run `code-review`. Follow `fixMode` from Shared Runtime Setup.
|
|
190
195
|
|
|
191
|
-
Both follow the same `spec-compliance` -> `code-review` gate sequence as
|
|
196
|
+
Both follow the same `spec-compliance` -> `code-review` gate sequence as `../developer/references/close-phase.md`. Do not proceed to Step 6 until both pass.
|
|
192
197
|
|
|
193
198
|
---
|
|
194
199
|
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: release-readiness
|
|
3
|
-
description: Produces a report-only production release readiness
|
|
3
|
+
description: Produces a report-only production release readiness audit from existing specs, quality gates, deployment configs, migrations, observability, rollback, and runbooks. Use whenever the user asks if the project is ready to ship, asks for a pre-release or production audit, or after all Task.md phases are finished. Never deploys or alters infrastructure.
|
|
4
4
|
compatibility: Requires the complete MACCA-METHOD collection with sibling _shared resources, git when reviewing diffs, and workspace file access.
|
|
5
5
|
metadata:
|
|
6
6
|
persona: "Fachri"
|
|
@@ -111,7 +111,7 @@ Verdicts:
|
|
|
111
111
|
|
|
112
112
|
Use this report:
|
|
113
113
|
|
|
114
|
-
|
|
114
|
+
```markdown
|
|
115
115
|
# Release Readiness Report
|
|
116
116
|
|
|
117
117
|
**Candidate:** [branch/commit/tag/version]
|
|
@@ -119,26 +119,30 @@ Use this report:
|
|
|
119
119
|
**Verdict:** READY / CONDITIONAL / NOT READY
|
|
120
120
|
|
|
121
121
|
## Evidence Summary
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
|
125
|
-
| RR-
|
|
126
|
-
| RR-
|
|
127
|
-
| RR-
|
|
128
|
-
| RR-
|
|
129
|
-
| RR-
|
|
130
|
-
| RR-
|
|
131
|
-
| RR-
|
|
122
|
+
|
|
123
|
+
| Area | Status | Evidence |
|
|
124
|
+
| ----------------------------------- | -------------------------------- | -------- |
|
|
125
|
+
| RR-01 Scope & quality gates | PASS / FAIL / NOT VERIFIED / N/A | [source] |
|
|
126
|
+
| RR-02 Build, tests & smoke | ... | ... |
|
|
127
|
+
| RR-03 Config & secrets | ... | ... |
|
|
128
|
+
| RR-04 Migration & recovery | ... | ... |
|
|
129
|
+
| RR-05 Deploy, rollback & flags | ... | ... |
|
|
130
|
+
| RR-06 Observability & incidents | ... | ... |
|
|
131
|
+
| RR-07 Compatibility & communication | ... | ... |
|
|
132
|
+
| RR-08 UI operational experience | ... | ... |
|
|
132
133
|
|
|
133
134
|
## Blockers
|
|
135
|
+
|
|
134
136
|
- [blocker, evidence, owner]
|
|
135
137
|
|
|
136
138
|
## Required Before Release
|
|
139
|
+
|
|
137
140
|
- [action] โ owner: [role] โ validation: [proof required]
|
|
138
141
|
|
|
139
142
|
## Accepted Residual Risks
|
|
143
|
+
|
|
140
144
|
- [risk and explicit owner/expiry] / None
|
|
141
|
-
|
|
145
|
+
```
|
|
142
146
|
|
|
143
147
|
## Rules
|
|
144
148
|
|