@anhth2/spec-driven-dev-plugin 0.8.0 → 0.9.0

package/core/commands/fix-bug.md

@@ -178,6 +178,37 @@ If `services` section is present:
 - Override `paths.domain_knowledge_dir` → `{spec_source}/specs/domain-knowledge`
 - Override `paths.business_dictionary` → `{spec_source}/specs/domain-knowledge/business-dictionary.md`
 - Override `paths.core_entities` → `{spec_source}/specs/domain-knowledge/core-entities.md`
+- Override `paths.bug_reports_dir` → `{spec_source}/feedback/bug-reports`
+- Override `paths.bdd_proposals_dir` → `{spec_source}/feedback/bdd-proposals`
+
+> **Why under `spec_source`:** tester feedback (`/report-bug`, `/propose-scenario`) must land in the **shared spec repo** so PO/Dev see it when they `/sync`. In single-service mode (no `spec_source`), these default to `feedback/bug-reports` and `feedback/bdd-proposals` at repo root — still shared, same repo.
+
+---
+
+## Step 1.6 — [SERVICE CONVENTIONS] Load service-specific conventions (umbrella mode)
+
+*Skip this step entirely if `active_service` is `"unresolved"` or context is single-service mode.*
+
+When `active_service` has been resolved to a real path in Step 1.5 (e.g., `user-service/`):
+
+**1. Locate service config** — try in priority order:
+- `{active_service}/.agent/project-context.yaml`
+- `{active_service}/project-context.yaml`
+
+**2. If found, override with service-specific values:**
+
+| Variable | Source |
+|----------|--------|
+| `conventions.test_command` | service's `conventions.test_command` |
+| `conventions.build_command` | service's `conventions.build_command` |
+| `paths.trace_dir` | `{active_service}/{service paths.trace_dir}` — default: `{active_service}/.trace` |
+| `paths.specs_dir` | `{active_service}/{service paths.specs_dir}` (if set in service config, else keep Step 1.5 override) |
+
+**3. Store** `service_root = {active_service}` as the working directory anchor for all downstream commands:
+- Shell commands (`/run-tests`, `/generate-tests`) run **from within** `service_root`
+- File write operations (test files, trace TSVs) use paths **relative to** `service_root`
+
+**4. If service config not found** — keep umbrella defaults, still set `service_root = {active_service}` (path anchor is always needed even without a config override).
 
 ---
 
@@ -271,6 +302,25 @@ These two variables (`active_module`, `platform_type`) are the canonical source
 
 ---
 
+## Step 6.7 — [GUARDRAILS] Load Project Lessons (conditional)
+
+*Accumulated mistakes the AI must not repeat in this project. These are added over time via `/learn`
+or accepted during `/review-code`, `/fix-bug`, `/debug`.*
+
+Resolve the lessons file path:
+- Use `paths.lessons_file` if set (may be service-overridden in umbrella mode, Step 1.6)
+- Else default `specs/domain-knowledge/lessons-learned.md`
+- In umbrella/service mode (when `service_root` is set), if `paths.lessons_file` is unset, default to `{service_root}/.agent/project-lessons.md`
+
+If the file exists, read it and store ALL lessons as **ACTIVE GUARDRAILS** for the session:
+- Treat each lesson's **Rule** as a hard constraint — same priority as CLAUDE.md coding standards (Step 3).
+- Before generating or modifying any artifact (PRD, BDD, tech-doc, code, test), check the output against every lesson whose `category` matches the current command AND whose `scope` matches the target (domain / file).
+- If a generated output would violate a lesson → correct it **before** presenting, and note which lesson (`L-NNN`) was applied.
+
+If the file does not exist → skip silently (no lessons captured yet).
+
+---
+
 ## Step 7 — [RECAP] Working Memory Recap (anti-lost-in-middle)
 
 After loading all context, synthesize and output a compact summary block.
@@ -286,7 +336,9 @@ Layers    : {layer order from CLAUDE.md §2, e.g., Controller → Facade → Ser
 Ticket    : {ticket_prefix}-
 Dict      : {loaded — N canonical terms, M banned terms | missing}
 Entities  : {loaded — EntityA, EntityB, EntityC | missing}
+Lessons   : {loaded — N guardrails | none yet}
 Service   : {active_service} ({active_service_module}) | single-service
+Svc Root  : {service_root} — conventions + trace_dir loaded from service config | —
 Status    : {FULL | PARTIAL — missing: CLAUDE.md / business-dict / core-entities | MINIMAL}
 ```
 
@@ -410,6 +462,101 @@ git commit -m "fix({TICKET_ID}): {description}"
 git push -u origin fix/{TICKET_ID}-{slug}
 ```
 
+## Phase 6 — Offer to Record a Lesson (optional)
+
+If the root cause was a **mistake the AI made during generation and could repeat**
+(e.g. it generated code that skipped a layer, missed a null guard, used a wrong pattern —
+NOT an external cause like a third-party outage or bad input data), ask:
+
+```
+This root cause looks like a repeatable AI mistake.
+Record it as a project lesson so it isn't generated again? (Y/N)
+```
+
+If `Y` → run the capture procedure below with `source=/fix-bug {TICKET_ID}`, an appropriate
+`category` (usually `code-gen`), and `scope` = the affected domain or file glob.
+
+# Capture Lesson — Record a Recurring Mistake as a Guardrail
+
+Reusable procedure to persist a "lesson" so the AI does not repeat a mistake in this project.
+Used by `/learn` (manual) and offered by `/review-code`, `/fix-bug`, `/debug` (auto).
+
+> **Project memory, not model training.** A lesson is plain text injected into context at the
+> start of every command (context-loader Step 6.7). Functionally this stops the repeat — the AI
+> sees the guardrail before it generates. No model weights change.
+
+## L1 — Resolve the lessons file
+
+Resolve `lessons_path` in this order:
+1. `paths.lessons_file` from loaded context (may be service-overridden in umbrella mode, Step 1.6)
+2. Default `specs/domain-knowledge/lessons-learned.md` (single-service)
+3. In umbrella/service mode (when `service_root` is set) default `{service_root}/.agent/project-lessons.md`
+
+## L2 — Build the lesson
+
+Gather these fields — from `$ARGUMENTS` (for `/learn`) or from the calling command's findings
+(for `/review-code`, `/fix-bug`, `/debug`):
+
+| Field | Meaning |
+|-------|---------|
+| `category` | one of: `code-gen` \| `bdd` \| `tech-docs` \| `tests` \| `prd` \| `general` |
+| `title` | short phrase naming the mistake |
+| `mistake` | concretely, what the AI did wrong |
+| `rule` | imperative correction — "Always …" / "Never …" — testable, not vague |
+| `scope` | where it applies: a domain, a file glob (e.g. `*Controller.*`), or `all` |
+| `source` | how captured: `/learn` \| `/review-code {UC-ID}` \| `/fix-bug {TICKET}` \| `/debug` |
+
+If `rule` is vague (e.g. "be careful"), rewrite it into a concrete, checkable instruction before saving.
+
+## L3 — De-duplicate
+
+Read existing lessons in `lessons_path`. If one already covers the same mistake:
+- **Refine** that entry (tighten the Rule, widen/narrow Scope, bump Date, append the new Source) — do NOT add a duplicate.
+
+Otherwise assign the next id `L-{NNN}` = (highest existing number + 1), zero-padded to 3 digits.
+
+## L4 — Write
+
+If `lessons_path` does not exist, create it with this header first:
+
+```markdown
+# Project Lessons — Learned Guardrails
+
+> Mistakes the AI must NOT repeat in this project. Loaded by context-loader at the start of
+> every command and treated as hard constraints (same priority as CLAUDE.md coding standards).
+> Add with /learn, or accept the prompt during /review-code, /fix-bug, /debug.
+> Commit this file so the whole team shares the guardrails.
+
+| Category | Applies to |
+|----------|-----------|
+| code-gen | /generate-code output |
+| bdd | /generate-bdd output |
+| tech-docs | /generate-tech-docs output |
+| tests | /generate-tests output |
+| prd | /generate-prd, /refine-prd output |
+| general | every command |
+
+---
+```
+
+Insert the new lesson directly under the `---` separator (**newest first**), in this exact shape:
+
+```markdown
+### L-{NNN} — [{category}] {title}
+- **Date**: {today YYYY-MM-DD}
+- **Scope**: {scope}
+- **Mistake**: {mistake}
+- **Rule**: {rule}
+- **Source**: {source}
+
+```
+
+## L5 — Confirm
+
+Print: `📝 Lesson {id} recorded → {lessons_path}  ([{category}] {title})`
+Then remind: `Commit {lessons_path} so the team shares this guardrail.`
+
+
 ## Output
 
 # Report Footer — Standard Command Output Format
@@ -459,6 +606,11 @@ Suggest the logical next command based on workflow phase:
 | /validate-traces        | DRIFT/UNTRACKED → `/generate-code {UC-ID}`; GAP → `/generate-tests {UC-ID}`; all OK → create PR |
 | /fix-bug                | Create PR and link to ticket                  |
 | /debug                  | `/fix-bug {ticket-id}` if fix needed          |
+| /report-bug             | Send to dev (`/fix-bug {BUG-ID}`); if coverage gap → `/propose-scenario {UC-ID}` |
+| /propose-scenario       | Notify PO/Dev to review the proposal in `feedback/bdd-proposals/` |
+| /learn                  | Continue working — lesson applies on next command |
+| /sync                   | `/validate-traces` for full coverage; act on any `📥 tester feedback` surfaced |
+| /update-framework       | Review `git diff .agent/`, commit; `/sync` for project content |
 
 Format the footer as:
 ```
@@ -474,6 +626,7 @@ Next   : {suggested command with example arguments}
 Root Cause: {analysis}
 Changes: {list}
 ✅ Regression test added | ✅ Build: SUCCESS
+{📝 Lesson L-NNN recorded (if captured)}
 Branch: fix/{TICKET_ID}-{slug}
 Next: Create PR and link to ticket.
 ```

package/core/commands/generate-bdd.md

@@ -176,6 +176,37 @@ If `services` section is present:
 - Override `paths.domain_knowledge_dir` → `{spec_source}/specs/domain-knowledge`
 - Override `paths.business_dictionary` → `{spec_source}/specs/domain-knowledge/business-dictionary.md`
 - Override `paths.core_entities` → `{spec_source}/specs/domain-knowledge/core-entities.md`
+- Override `paths.bug_reports_dir` → `{spec_source}/feedback/bug-reports`
+- Override `paths.bdd_proposals_dir` → `{spec_source}/feedback/bdd-proposals`
+
+> **Why under `spec_source`:** tester feedback (`/report-bug`, `/propose-scenario`) must land in the **shared spec repo** so PO/Dev see it when they `/sync`. In single-service mode (no `spec_source`), these default to `feedback/bug-reports` and `feedback/bdd-proposals` at repo root — still shared, same repo.
+
+---
+
+## Step 1.6 — [SERVICE CONVENTIONS] Load service-specific conventions (umbrella mode)
+
+*Skip this step entirely if `active_service` is `"unresolved"` or context is single-service mode.*
+
+When `active_service` has been resolved to a real path in Step 1.5 (e.g., `user-service/`):
+
+**1. Locate service config** — try in priority order:
+- `{active_service}/.agent/project-context.yaml`
+- `{active_service}/project-context.yaml`
+
+**2. If found, override with service-specific values:**
+
+| Variable | Source |
+|----------|--------|
+| `conventions.test_command` | service's `conventions.test_command` |
+| `conventions.build_command` | service's `conventions.build_command` |
+| `paths.trace_dir` | `{active_service}/{service paths.trace_dir}` — default: `{active_service}/.trace` |
+| `paths.specs_dir` | `{active_service}/{service paths.specs_dir}` (if set in service config, else keep Step 1.5 override) |
+
+**3. Store** `service_root = {active_service}` as the working directory anchor for all downstream commands:
+- Shell commands (`/run-tests`, `/generate-tests`) run **from within** `service_root`
+- File write operations (test files, trace TSVs) use paths **relative to** `service_root`
+
+**4. If service config not found** — keep umbrella defaults, still set `service_root = {active_service}` (path anchor is always needed even without a config override).
 
 ---
 
@@ -269,6 +300,25 @@ These two variables (`active_module`, `platform_type`) are the canonical source
 
 ---
 
+## Step 6.7 — [GUARDRAILS] Load Project Lessons (conditional)
+
+*Accumulated mistakes the AI must not repeat in this project. These are added over time via `/learn`
+or accepted during `/review-code`, `/fix-bug`, `/debug`.*
+
+Resolve the lessons file path:
+- Use `paths.lessons_file` if set (may be service-overridden in umbrella mode, Step 1.6)
+- Else default `specs/domain-knowledge/lessons-learned.md`
+- In umbrella/service mode (when `service_root` is set), if `paths.lessons_file` is unset, default to `{service_root}/.agent/project-lessons.md`
+
+If the file exists, read it and store ALL lessons as **ACTIVE GUARDRAILS** for the session:
+- Treat each lesson's **Rule** as a hard constraint — same priority as CLAUDE.md coding standards (Step 3).
+- Before generating or modifying any artifact (PRD, BDD, tech-doc, code, test), check the output against every lesson whose `category` matches the current command AND whose `scope` matches the target (domain / file).
+- If a generated output would violate a lesson → correct it **before** presenting, and note which lesson (`L-NNN`) was applied.
+
+If the file does not exist → skip silently (no lessons captured yet).
+
+---
+
 ## Step 7 — [RECAP] Working Memory Recap (anti-lost-in-middle)
 
 After loading all context, synthesize and output a compact summary block.
@@ -284,7 +334,9 @@ Layers    : {layer order from CLAUDE.md §2, e.g., Controller → Facade → Ser
 Ticket    : {ticket_prefix}-
 Dict      : {loaded — N canonical terms, M banned terms | missing}
 Entities  : {loaded — EntityA, EntityB, EntityC | missing}
+Lessons   : {loaded — N guardrails | none yet}
 Service   : {active_service} ({active_service_module}) | single-service
+Svc Root  : {service_root} — conventions + trace_dir loaded from service config | —
 Status    : {FULL | PARTIAL — missing: CLAUDE.md / business-dict / core-entities | MINIMAL}
 ```
 
@@ -306,6 +358,194 @@ After completing all steps, you have loaded:
 Proceed to the next step of the calling command.
 
 
+> **Tester proposals (optional input):** before generating, check `{paths.bdd_proposals_dir}/` (default `{spec_source}/feedback/bdd-proposals/`) for scenarios proposed by testers via `/propose-scenario` that map to this UC's ACs. Incorporate any the PO/Dev has accepted — then the tester's draft is removed/archived from `feedback/`. Skip if the folder is empty.
+
+---
+
+## Repo Mode Detection
+
+After loading context, determine which mode to operate in:
+
+- **Spec repo mode**: `project-context.yaml` has NO `services` section OR `setup.mode: spec`
+- **Umbrella mode**: `project-context.yaml` HAS `services` section AND `setup.mode: umbrella`
+
+→ Spec repo mode → proceed to **Platform Selection** below (skip Service Detection)
+→ Umbrella mode → proceed to **Service Detection** below (skip Platform Selection)
+
+---
+
+## Platform Selection (Spec Repo Mode Only)
+
+*Skip this section if running in umbrella mode.*
+
+Ask the user to select the target platform:
+
+```
+Which platform is this BDD for?
+  1. web    — FE/Web (React, Next.js, Angular, Vue, Nuxt)
+  2. app    — Mobile (Flutter, React Native, iOS, Android)
+  3. system — System/BE BDD (synthesized from existing web + app BDDs)
+```
+
+Wait for user selection. Set `active_platform` = chosen value.
+
+**Output path (spec repo mode):**
+`{paths.specs_dir}/{domain}/{active_platform}/{TICKET-ID}-UC{N}-{slug}.feature`
+
+**Platform vocabulary:**
+
+| Platform | "user action" | "type/input" | "observe" | "navigate" |
+|---|---|---|---|---|
+| web | "clicks" | "types into" / "enters" | "sees" / "the page shows" | "navigates to" / "goes to" |
+| app | "taps" | "enters" / "inputs" | "sees" / "the screen shows" | "navigates to" / "opens" |
+| system | N/A — use business events | — | "the system returns" / "receives response" | — |
+
+---
+
+## System BDD Synthesis (active_platform = system)
+
+*Only applies when platform = system. Skip for web and app.*
+
+### Step S0 — Brownfield Check
+
+Check the source PRD Metadata table for `| **API Source** | existing |`.
+
+**Nếu `API Source: existing`:**
+- API contract đã được PO ghi trong phần "Existing API Contract" của PRD.
+- **Skip Steps S1–S3** — không cần scan FE/App BDD, không cần conflict resolution.
+- Dùng bảng "Existing API Contract" trong PRD làm contract input cho Step S4.
+- Set `# @trace.api_source: existing` trong header của file system BDD được gen.
+
+**Nếu `API Source` không có hoặc không phải `existing`:**
+- Tiếp tục Steps S1–S3 (normal synthesis flow).
+
+---
+
+### Step S1 — Scan available FE/App BDDs
+
+Search for existing BDDs for this TICKET-ID:
+- Web BDDs: `{paths.specs_dir}/{domain}/web/{TICKET-ID}-*.feature`
+- App BDDs: `{paths.specs_dir}/{domain}/app/{TICKET-ID}-*.feature`
+
+Classify the feature:
+
+| Condition | Mode |
+|---|---|
+| Both web + app BDDs found | **Multi-platform** — synthesize from both |
+| Only web BDD found | **Web-only** — synthesize from web |
+| Only app BDD found | **App-only** — synthesize from app |
+| No FE/App BDDs found | **Backend-only** — gen directly from PRD |
+
+---
+
+### Step S2 — Extract expected contracts per platform
+
+For each found BDD file, extract:
+- **Triggers**: what user actions call the backend? (map to logical "request" events)
+- **Expected response data**: what fields/shape does each `Then` clause need from the system?
+- **Error signals**: what error states must the backend signal?
+- **Business rules**: what invariants does each platform assume the system enforces?
+
+---
+
+### Step S3 — Cross-Platform Conflict Check (multi-platform mode only)
+
+*Skip if web-only, app-only, or backend-only.*
+
+Compare the extracted contracts across platforms. Flag a conflict if any of these differ:
+
+| Conflict type | Example |
+|---|---|
+| **Response shape mismatch** | Web expects `{ token, redirect_url }`, App expects `{ token, user_profile }` |
+| **Error semantics mismatch** | Web expects HTTP 423 for lock, App expects custom error code `ACC_LOCKED` |
+| **Business rule contradiction** | Web BDD says "lock after 5 attempts", App BDD says "lock after 3 attempts" |
+| **Data field conflict** | Web expects `expires_in: seconds`, App expects `expires_at: ISO timestamp` |
+
+**If conflicts detected → CHECKPOINT (mandatory, cannot skip):**
+
+```
+⚠️  CROSS-PLATFORM CONTRACT CONFLICT
+──────────────────────────────────────────────────────────────────
+Feature : {TICKET-ID} — {UC name}
+
+Conflict 1: Response shape mismatch
+  Web BDD (Then): user sees dashboard → implies { token, redirect_url }
+  App BDD (Then): app navigates to HomeScreen → implies { token, user_profile }
+
+Resolution options:
+  A — Union response
+      BE returns all fields: { token, redirect_url, user_profile }
+      Clients ignore fields they don't use. Simple, slight over-fetch.
+  B — Platform hint in request
+      Client sends X-Platform: web|app in header, BE tailors response.
+      Cleaner responses, more BE logic.
+  C — Separate endpoints
+      POST /auth/login/web  and  POST /auth/login/app
+      Maximum flexibility, more endpoints to maintain.
+  D — Custom: describe your approach
+──────────────────────────────────────────────────────────────────
+Choose resolution for each conflict (A/B/C/D):
+```
+
+Wait for PO's resolution per conflict. Record each decision as a `# @system.resolution:` annotation in the generated system BDD file.
+
+**If no conflicts → proceed to Step S4 directly.**
+
+---
+
+### Step S4 — Generate System BDD scenarios
+
+Generate scenarios based on mode and resolved conflicts:
+
+- **Multi-platform**: synthesize from both web + app contracts, applying resolved resolutions
+- **Web-only / App-only**: derive from the single platform's contracts
+- **Backend-only**: derive directly from PRD AC/BR using business event language (not HTTP)
+
+System BDD step vocabulary (always use — regardless of FE/App vocabulary):
+- Triggers: "the system receives {event}" / "a {actor} submits {action}"
+- Assertions: "the system returns {data}" / "the system signals {error}" / "the system stores {state}"
+- Do NOT use UI words (click, tap, see, navigate) in system BDD
+
+**If multi-platform with resolution A (union):**
+- System BDD shows the full response contract: all fields from all platforms
+- Add comment: `# @system.resolution: union — clients receive all fields`
+
+**If resolution B (platform hint):**
+- Write separate `Scenario Outline` using Examples table for `web` vs `app` response variants
+- Add comment: `# @system.resolution: platform-hint — X-Platform header determines response shape`
+
+**If resolution C (separate endpoints):**
+- Write separate Scenarios for each endpoint
+- Note the endpoint split explicitly in the SCOPE section
+
+---
+
+## Service Detection (Umbrella Mode Only)
+
+*Skip this section if running in spec repo mode.*
+
+Read the PRD Metadata table for `| **Service** |` and `| **Module** |` rows.
+
+| Condition | Action |
+|---|---|
+| PRD has `Service` row AND value is not "default" | Use it as `active_service` + `active_module`. Load catalog for that module. |
+| PRD has no `Service` row AND `services` defined in project-context.yaml | Ask: "Which service is this BDD for?" (list services, wait for selection) |
+| Single-service project (no `services` array) | `active_service = "default"`, `active_module = tech_stack.module` |
+
+**Output path (umbrella mode):**
+- `active_service = "default"` → `{paths.specs_dir}/{domain}/{TICKET-ID}-UC{N}-{slug}.feature`
+- `active_service ≠ "default"` → `{paths.specs_dir}/{domain}/{active_service}/{TICKET-ID}-UC{N}-{slug}.feature`
+
+**Platform vocabulary** — adapt BDD step wording based on `active_module`:
+
+| Platform type | Modules | "click" | "type" | "see" | "navigate" |
+|---|---|---|---|---|---|
+| Web | react, nextjs, vue, nuxt, angular | "clicks" | "types into" / "enters" | "sees" / "the page shows" | "navigates to" / "goes to" |
+| Mobile | flutter, react-native, ios-swiftui, android-compose | "taps" | "enters" / "inputs" | "sees" / "the screen shows" | "navigates to" / "opens" |
+| Backend / API | java-spring, golang, dotnet, php-laravel | *(no UI steps — use)* "submits a request" / "calls the API" | — | "receives response" / "the system returns" | — |
+
+Apply this vocabulary silently when writing Gherkin steps. Do NOT mix web and mobile terms in the same feature file.
+
 ---
 
 ## Orchestration Check
@@ -342,7 +582,9 @@ After generating all `.feature` and `.tsv` files for the assigned UC, return the
 
 Before generating, check for existing `.feature` files for this PRD:
 
-1. Look for `{paths.specs_dir}/{domain}/{TICKET-ID}-UC*.feature`.
+1. Resolve the search path based on mode:
+   - **Spec repo mode**: `{paths.specs_dir}/{domain}/{active_platform}/{TICKET-ID}-UC*.feature`
+   - **Umbrella mode**: `{paths.specs_dir}/{domain}/{TICKET-ID}-UC*.feature` (or with `{active_service}/` if multi-service)
 2. Read current PRD `| **Version** |` from metadata (e.g., `1.2`).
 
 **If no existing feature files** → fresh generation, proceed normally. Use PRD version as `@trace.prd_version`.
@@ -445,7 +687,12 @@ CHECKPOINT: "Does this outline look correct? Do you want to add or remove any SC
 
 ## Generate
 
-For each UC, write `{paths.specs_dir}/{domain}/{TICKET-ID}-UC{N}-{slug}.feature`:
+**Output path by mode:**
+- **Spec repo mode**: `{paths.specs_dir}/{domain}/{active_platform}/{TICKET-ID}-UC{N}-{slug}.feature`
+- **Umbrella mode (single-service)**: `{paths.specs_dir}/{domain}/{TICKET-ID}-UC{N}-{slug}.feature`
+- **Umbrella mode (multi-service)**: `{paths.specs_dir}/{domain}/{active_service}/{TICKET-ID}-UC{N}-{slug}.feature`
+
+For each UC, write to the resolved path above. Use vocabulary for the active platform (from Platform Selection or Service Detection).
 
 ```gherkin
 # ============================================================
@@ -453,8 +700,9 @@ For each UC, write `{paths.specs_dir}/{domain}/{TICKET-ID}-UC{N}-{slug}.feature`
 # @trace.title: <Feature name>
 # @trace.revision: 1  ← static field; use @trace.bdd_version for version tracking (incremented by /review-context --fix or --resume)
 # @trace.domain: <domain>
-# @trace.service: <ServiceName>
-# @trace.module: {tech_stack.module value — e.g. java-spring | react | flutter; write "unknown" if not set in project-context.yaml}
+# @trace.platform: {active_platform — web | app | system | (omit in umbrella mode)}
+# @trace.service: {active_service — omit in spec repo mode}
+# @trace.module: {active_module in umbrella mode; "unknown" in spec repo mode}
 # @trace.status: draft
 # @trace.author: AI-generated
 # @trace.created_at: {YYYY-MM-DD}
@@ -562,7 +810,7 @@ After generating all `.feature` files, create or update `{paths.trace_dir}/{UC-I
 
 **TSV columns (tab-separated, one header row + one data row per scenario):**
 ```
-sc_id\tsc_title\tspec_ver\tgen_ver\timplemented_by\ttest_count\ttest_classes\tprd_version\tbdd_version\ttech_doc_revision\tprd_status\tuc_status\tstatus\tlast_updated
+sc_id\tsc_title\tspec_ver\tgen_ver\timplemented_by\ttest_count\ttest_classes\tprd_version\tbdd_version\ttech_doc_revision\tprd_status\tuc_status\tfe_phase\tstatus\tlast_updated
 ```
 
 **Rules:**
@@ -589,6 +837,7 @@ sc_id\tsc_title\tspec_ver\tgen_ver\timplemented_by\ttest_count\ttest_classes\tpr
 | `tech_doc_revision` | `—` |
 | `prd_status` | read `\| **Status** \|` from PRD metadata |
 | `uc_status` | `draft` for new UCs; keep existing value for re-gen |
+| `fe_phase` | `—` (set by `/generate-code --phase` when FE implements) |
 | `status` | `UNTRACKED` |
 | `last_updated` | today `YYYY-MM-DD` |
 
@@ -641,6 +890,11 @@ Suggest the logical next command based on workflow phase:
 | /validate-traces        | DRIFT/UNTRACKED → `/generate-code {UC-ID}`; GAP → `/generate-tests {UC-ID}`; all OK → create PR |
 | /fix-bug                | Create PR and link to ticket                  |
 | /debug                  | `/fix-bug {ticket-id}` if fix needed          |
+| /report-bug             | Send to dev (`/fix-bug {BUG-ID}`); if coverage gap → `/propose-scenario {UC-ID}` |
+| /propose-scenario       | Notify PO/Dev to review the proposal in `feedback/bdd-proposals/` |
+| /learn                  | Continue working — lesson applies on next command |
+| /sync                   | `/validate-traces` for full coverage; act on any `📥 tester feedback` surfaced |
+| /update-framework       | Review `git diff .agent/`, commit; `/sync` for project content |
 
 Format the footer as:
 ```
@@ -653,16 +907,26 @@ Next   : {suggested command with example arguments}
 
 ```
 /generate-bdd Complete
+
+[Spec repo mode — platform: {active_platform}]
 Files:
-  {paths.specs_dir}/{domain}/{TICKET-ID}-UC1-{slug}.feature ({N} scenarios)
-  {paths.specs_dir}/{domain}/{TICKET-ID}-UC2-{slug}.feature ({N} scenarios)
+  {paths.specs_dir}/{domain}/{active_platform}/{TICKET-ID}-UC1-{slug}.feature ({N} scenarios)
+  {paths.specs_dir}/{domain}/{active_platform}/{TICKET-ID}-UC2-{slug}.feature ({N} scenarios)
 Trace:
   {paths.trace_dir}/{TICKET-ID}-UC1.tsv ({N} rows)
   {paths.trace_dir}/{TICKET-ID}-UC2.tsv ({N} rows)
-Next: Run /review-context on EACH generated feature file (one per UC):
-        /review-context {paths.specs_dir}/{domain}/{TICKET-ID}-UC1-{slug}.feature
-        /review-context {paths.specs_dir}/{domain}/{TICKET-ID}-UC2-{slug}.feature
-        ...
-      → then per UC: /generate-tech-docs {feature-file}   (if tech design required)
-      → or   per UC: /generate-code {feature-file}         (if skipping tech design)
+Next (spec repo):
+  → Run /generate-bdd again for other platforms (web → app → system)
+  → After all platforms generated: commit + push + notify dev team
+  → Dev team reads BDD from spec submodule — no /generate-bdd on their side
+
+[Umbrella mode — service: {active_service}]
+Files:
+  {paths.specs_dir}/{domain}/{TICKET-ID}-UC1-{slug}.feature ({N} scenarios)
+Trace:
+  {paths.trace_dir}/{TICKET-ID}-UC1.tsv ({N} rows)
+Next (umbrella):
+  → /review-context {feature-file} to verify coverage
+  → /generate-tech-docs {feature-file}
+  → /generate-code {feature-file}
 ```