ma-agents 3.6.2 → 3.9.0

package/_bmad-output/implementation-artifacts/epic-17-edge-case-review.md

@@ -1,140 +0,0 @@
-# Edge Case Hunter Review: Epic 17 — Sprint Entity Model
-
-**Scope:** Story 17.1
-**Date:** 2026-03-28
-**Method:** Exhaustive path enumeration across all branching paths and boundary conditions
-
-## Findings
-
-```json
-[
-  {
-    "location": "17-1:AC#1/Task2.3",
-    "trigger_condition": "User enters start date after end date (e.g., start=2026-04-14, end=2026-04-01).",
-    "guard_snippet": "Add check: if start > end, reject with error and re-prompt step 3. Also validate ISO format via regex ^\\d{4}-\\d{2}-\\d{2}$.",
-    "potential_consequence": "Sprint created with inverted date range, confusing all downstream story scheduling."
-  },
-  {
-    "location": "17-1:AC#1/Task2.3",
-    "trigger_condition": "User enters invalid ISO date like 2026-02-30 or non-date string.",
-    "guard_snippet": "Add date validation: parse YYYY-MM-DD and verify it resolves to a real calendar date. Re-prompt on failure.",
-    "potential_consequence": "Sprint YAML contains impossible dates, breaking any future date-range logic."
-  },
-  {
-    "location": "17-1:AC#1/Task2.3",
-    "trigger_condition": "User skips both start and end dates (presses Enter on both).",
-    "guard_snippet": "Spec must define behavior for empty dates: write empty string, omit field, or require at least one. AC#1 schema shows quoted dates with no optional marker.",
-    "potential_consequence": "Schema inconsistency: AC#1 shows start/end as required but workflow allows skip."
-  },
-  {
-    "location": "17-1:AC#1/Task2.1",
-    "trigger_condition": "User enters non-numeric sprint ID like 'sprint-abc' or 'three'.",
-    "guard_snippet": "Add validation: sprint number input must match ^[1-9]\\d*$. Reject non-positive-integer and non-numeric inputs with re-prompt.",
-    "potential_consequence": "Malformed sprint ID 'sprint-abc' breaks downstream glob and cross-reference assumptions."
-  },
-  {
-    "location": "17-1:AC#1/Task2.1",
-    "trigger_condition": "User enters zero or negative number for sprint ID.",
-    "guard_snippet": "Add check: sprint number must be a positive integer > 0. Reject zero and negative values.",
-    "potential_consequence": "Sprint file named sprint-0.yaml or sprint--1.yaml, violating naming convention."
-  },
-  {
-    "location": "17-1:AC#1/Task2.1",
-    "trigger_condition": "User enters sprint number with leading zeros like '03'.",
-    "guard_snippet": "Normalize input: strip leading zeros before constructing ID. Validate 03 -> 3 to prevent duplicate sprint-03.yaml vs sprint-3.yaml.",
-    "potential_consequence": "Duplicate sprints created: sprint-03.yaml and sprint-3.yaml both exist for same sprint."
-  },
-  {
-    "location": "17-1:AC#2",
-    "trigger_condition": "Items are added to sprint but no validation enforces capacity at write time.",
-    "guard_snippet": "AC#2 says count never exceeds capacity, but add-sprint only creates empty items. Story must specify which operation enforces capacity check or cross-reference Story 17.4 explicitly as the enforcement point.",
-    "potential_consequence": "Capacity constraint exists in spec but no story owns its runtime enforcement."
-  },
-  {
-    "location": "17-1:AC#3/Task3.1",
-    "trigger_condition": "New sprint is created with status planning; activation path is undefined.",
-    "guard_snippet": "Task 3.4 says activation is 'a separate action (via modify-sprint or a future activate-sprint step)' but modify-sprint is deferred to 17.4/17.5. Add note that single-active validation in Task 3 is dead code until a later story enables activation.",
-    "potential_consequence": "Single-active-sprint validation is specified but unreachable in this story's scope."
-  },
-  {
-    "location": "17-1:AC#3/Task3.3",
-    "trigger_condition": "YAML parse error in one sprint file during glob scan for active sprints.",
-    "guard_snippet": "Add error handling: if a sprint YAML file is malformed or unparseable, log a warning and skip it rather than aborting the entire validation scan.",
-    "potential_consequence": "One corrupt sprint file blocks creation of all new sprints."
-  },
-  {
-    "location": "17-1:AC#3/Task3.2",
-    "trigger_condition": "Sprints directory contains non-sprint YAML files matching sprint-*.yaml glob.",
-    "guard_snippet": "Add schema validation: after glob match, verify file contains required 'id' and 'status' fields before treating it as a sprint entity.",
-    "potential_consequence": "Non-sprint file with status field triggers false active-sprint conflict."
-  },
-  {
-    "location": "17-1:AC#3",
-    "trigger_condition": "Two sprints set to active simultaneously via concurrent workflow executions.",
-    "guard_snippet": "Add advisory note: glob-then-write is not atomic. Document that concurrent sprint activation is unsupported, or add a file-lock/check-after-write pattern.",
-    "potential_consequence": "Race condition: two sprints both pass the active check and both get activated."
-  },
-  {
-    "location": "17-1:AC#3",
-    "trigger_condition": "Sprint transitions directly from closed to active, bypassing planning state.",
-    "guard_snippet": "Define valid transitions explicitly: planning->active, active->closed. Reject closed->active and closed->planning transitions. Task 5.3 mentions documenting transitions but no enforcement spec exists.",
-    "potential_consequence": "Closed sprint reactivated, violating the linear lifecycle assumption downstream."
-  },
-  {
-    "location": "17-1:AC#1/Task2.5",
-    "trigger_condition": "Sprints directory does not exist when workflow runs (Task 1 not yet executed).",
-    "guard_snippet": "Add mkdir -p equivalent in step 5: ensure _bmad-output/implementation-artifacts/sprints/ exists before writing. Do not assume Task 1 has run.",
-    "potential_consequence": "Workflow fails with file-not-found error if directory was deleted or never created."
-  },
-  {
-    "location": "17-1:AC#1/Task2.5",
-    "trigger_condition": "YAML special characters in sprint name (e.g., colons, quotes, hash symbols).",
-    "guard_snippet": "Ensure sprint name is always quoted in YAML output. Add sanitization or escaping for characters that would break YAML parsing: :, #, ', \", {, }, [, ].",
-    "potential_consequence": "Generated YAML is unparseable, breaking all downstream sprint readers."
-  },
-  {
-    "location": "17-1:Task2.4/Step4",
-    "trigger_condition": "User edits sprint number at step 4, changing it to a number that already exists.",
-    "guard_snippet": "After editing sprint number in step 4, re-check overwrite protection against new path sprints/sprint-{new_number}.yaml before proceeding to step 5.",
-    "potential_consequence": "Existing sprint silently overwritten because overwrite check only runs with original number."
-  },
-  {
-    "location": "17-1:AC#1/Schema",
-    "trigger_condition": "Schema in AC#1 omits created_date and last_modified but Task 2.5 includes them.",
-    "guard_snippet": "Reconcile: AC#1 schema shows 7 fields (id, name, status, capacity, start, end, items). Task 2.5 schema adds created_date and last_modified. Clarify which is authoritative to prevent verification task 6.2 from failing.",
-    "potential_consequence": "Verification step 6.2 rejects implementation for schema mismatch with AC#1."
-  },
-  {
-    "location": "17-1:Task5/Guardrails",
-    "trigger_condition": "Guardrails doc title says 'Epic 11 & Epic 12' but now covers Epic 17 rules.",
-    "guard_snippet": "Task 5 should rename or add a section header clarifying the document now covers Epic 17 sprint conventions. Otherwise future readers may ignore it as obsolete.",
-    "potential_consequence": "Developers skip reading the guardrails doc assuming it only applies to Epics 11-12."
-  },
-  {
-    "location": "17-1:Task2.6/Task2.5",
-    "trigger_condition": "Overwrite protection in Task 2.6 only checks new path; old sprint-plan-{n}.yaml may still exist at legacy location.",
-    "guard_snippet": "Add advisory check: if _bmad-output/implementation-artifacts/sprint-plan-{number}.yaml exists at the old path, warn user about potential duplication between old and new format.",
-    "potential_consequence": "Same sprint exists in both old and new paths, causing confusion for partially-migrated workflows."
-  },
-  {
-    "location": "17-1:AC#1/Task2.1",
-    "trigger_condition": "User enters very large sprint number like 999999999.",
-    "guard_snippet": "Add reasonable upper-bound validation (e.g., sprint number <= 9999) to prevent unreasonably long filenames.",
-    "potential_consequence": "Absurd sprint IDs pollute the namespace and could hit filesystem path length limits."
-  },
-  {
-    "location": "17-1:Task3.2",
-    "trigger_condition": "Sprints directory is empty or does not exist during glob for active sprint check.",
-    "guard_snippet": "Handle empty glob result gracefully: if no sprint files found, skip active-sprint validation and proceed.",
-    "potential_consequence": "Glob returns error or unexpected value on empty directory, blocking sprint creation."
-  },
-  {
-    "location": "17-1:AC#1/Step4",
-    "trigger_condition": "User provides unrecognized input at confirmation prompt (not y, n, or e).",
-    "guard_snippet": "Add fallback: if input is not 'y', 'n', or 'e', display error and re-prompt step 4 confirmation.",
-    "potential_consequence": "Ambiguous input like 'yes' or 'Y' falls through without matching any check branch."
-  }
-]
-```
-
-**Total findings: 21**

package/_bmad-output/implementation-artifacts/sprint-status.yaml

@@ -1,139 +0,0 @@
-# generated: 2026-03-15
-# updated: 2026-04-13 — Epic 21 (On-Prem / Local-LLM Tuning) added to backlog (9 stories, 21.1-21.9).
-# updated: 2026-04-14 — Epic 21 corrective plan: added Story 21.10 (Profile Reconfigure). Fixes adversarial review Findings #5 and #7.
-# updated: 2026-04-14 — Epic 21 corrective plan: added Story 21.11 (Profile Uninstall). Fixes adversarial review Finding #17.
-# updated: 2026-04-14 — Epic 21 corrective plan: added Epic 18 tracking block to sprint-status (Story 18.1 already delivered at code level; remaining stories backlog). Fixes adversarial review Finding #16.
-# updated: 2026-04-07 — All Epic 17 stories (17.9-17.24) implemented and marked done. Epic 19 (BMAD Knowledge Graph) added to backlog.
-# project: agents
-# project_key: NOKEY
-# tracking_system: file-system
-# story_location: _bmad-output/implementation-artifacts
-
-# STATUS DEFINITIONS:
-# ==================
-# Epic Status:   backlog → in-progress → done
-# Story Status:  backlog → ready-for-dev → in-progress → review → done
-# Retrospective: optional → done
-#
-# Done epics archived (2026-04-03): 1, 2, 3, 6, 8, 9, 10, 11, 12, 13, 15, 16
-# Done stories for Epic 5 (5.1-5.4) also archived; 5.5-5.6 remain (in review).
-# Old Epic 17 stories (17.1-17.8) archived; replaced by 17.9-17.24.
-
-generated: 2026-03-15
-last_updated: 2026-04-13 — Epic 21 (On-Prem / Local-LLM Tuning) added to backlog
-project: agents
-project_key: NOKEY
-tracking_system: file-system
-story_location: _bmad-output/implementation-artifacts
-
-development_status:
-  # ─── ACTIVE BUGS ──────────────────────────────────────────────────────────────
-  # Standalone bug stories — discoverable via glob: _bmad-output/implementation-artifacts/bug-*.md
-  bug-experimentalwarning-about-commonjs-loading-es-module-during-install: ready-for-dev
-  bug-bmad-recompile-fails-on-airgapped-network: review
-
-  # ─── IN PROGRESS ──────────────────────────────────────────────────────────────
-
-  # Epic 5: Bundled BMAD Installation — stories 5.1-5.4 done (archived), 5.5-5.6 in review
-  epic-5: in-progress
-  5-5-explicit-parameter-passing: review
-  5-6-fix-space-in-path-bug: review
-  epic-5-retrospective: optional
-
-  # Epic 17: Sprint Management Model Rework — Unified sprint-status.yaml
-  # Redesigned 2026-04-03: 16 new stories (17.9-17.24) replacing old 17.1-17.8.
-  # Execution order: 17.9 → 17.15 → 17.10 → 17.16 → 17.11 → 17.12 → 17.13 → 17.14 → 17.17 → 17.18 → 17.19 → 17.20 → 17.24 → 17.21 → 17.22 → 17.23
-  epic-17: done
-  17-9-unified-sprint-status-schema: done
-  17-10-rework-generate-backlog: done
-  17-11-rework-add-to-sprint: done
-  17-12-rework-remove-from-sprint: done
-  17-13-rework-sprint-status-view: done
-  17-14-rework-cleanup-done: done
-  17-15-rework-bmad-sprint-planning: done
-  17-16-rework-add-sprint: done
-  17-17-rework-modify-sprint: done
-  17-18-rework-bmad-dev-story: done
-  17-19-rework-story-status-lookup: done
-  17-20-rework-bmad-sprint-status: done
-  17-21-new-close-sprint: done
-  17-22-jira-adapter-pattern: done
-  17-23-migration-deprecation-old-files: done
-  17-24-rework-prioritize-backlog: done
-  epic-17-retrospective: optional
-
-  # ─── ON HOLD ──────────────────────────────────────────────────────────────────
-
-  # Epic 4: Visual Studio Agent Integration — ON HOLD (2026-03-20)
-  epic-4: on-hold
-  4-1-vs-agent-registry-entry: on-hold
-  4-2-vs-skill-template-format: on-hold
-  epic-4-retrospective: optional
-
-  # Epic 7: Test Suite Foundation — ON HOLD (2026-03-19)
-  epic-7: on-hold
-  7-1-test-infrastructure-setup: on-hold
-  7-2-installer-pipeline-tests: on-hold
-  7-3-bmad-pipeline-tests: on-hold
-  7-4-cli-command-routing-tests: on-hold
-  epic-7-retrospective: optional
-
-  # ─── BACKLOG ──────────────────────────────────────────────────────────────────
-
-  # Epic 19: BMAD Knowledge Graph (Phase 3)
-  # Dependencies: Epic 15 (extension module structure), Epic 17 (sprint skill structure settled)
-  # Execution order: 19.1 → 19.2+19.3+19.4 (parallel) → 19.5 → 19.6 → 19.7
-  epic-19: in-progress
-  19-1-knowledge-graph-core-library: ready-for-dev
-  19-2-graph-emission-create-prd: ready-for-dev
-  19-3-graph-emission-create-architecture-epics: ready-for-dev
-  19-4-graph-emission-create-story-remaining: ready-for-dev
-  19-5-open-graph-skill: ready-for-dev
-  19-6-interactive-visualization-renderer: ready-for-dev
-  19-7-llm-writability-validation-tests: ready-for-dev
-  epic-19-retrospective: optional
-
-  # Epic 18: Roo Code IDE Support (Phase 2)
-  # Dependencies: Epic 9 (OpenCode JSON-merge pattern reused), Epic 13 (project-context template stamping pattern)
-  # Note: Story 18.1 (Roo Code agent registration in lib/agents.js) was delivered ahead of formal Epic 18 tracking. Remaining stories (18.2-18.8) have not been started.
-  # Execution order: 18.1 → 18.2 → 18.3 → 18.4 → 18.5 → 18.6 → 18.7 → 18.8
-  epic-18: backlog
-  18-1-add-roo-code-agent-entry: done
-  18-2-create-bmad-ide-config: backlog
-  18-3-update-installer-roo-code-rules-injection: backlog
-  18-4-generate-roomodes-bmad-personas: backlog
-  18-5-deploy-bmad-workflows-slash-commands: backlog
-  18-6-mode-specific-rules-bmad-agents: backlog
-  18-7-cline-to-roo-code-migration: backlog
-  18-8-end-to-end-validation-test-coverage: backlog
-  epic-18-retrospective: optional
-
-  # Epic 21: On-Prem / Local-LLM Tuning (Phase 3)
-  # Dependencies: Epics 9 (OpenCode JSON-merge), 13 (project-context template stamping pattern), 15 (customize-loader), 18 (Roo Code agent registration)
-  # Architecture: Decision P3-3 in _bmad-output/planning-artifacts/architecture.md
-  # Execution order: 21.1 → 21.2 → (21.3 + 21.4 + 21.5 in parallel) → 21.6 → 21.10 → 21.7 → 21.8 → 21.9 → 21.11
-  epic-21: backlog
-  # 2026-04-14: 21-1 status corrected review → done (PR #34 landed; sprint-status was stale).
-  21-1-install-time-profile-prompt: done
-  21-2-universal-instruction-block-expansion: backlog
-  21-3-roomodes-template-bmad-modes: backlog
-  21-4-agents-md-template-opencode: backlog
-  21-5-clinerules-template-extension: backlog
-  21-6-onprem-layered-guardrails: backlog
-  21-7-bmad-persona-phase-prefix: backlog
-  21-8-vllm-reference-doc-readme: backlog
-  21-9-tests-validation: backlog
-  21-10-profile-reconfigure: backlog
-  21-11-profile-uninstall: backlog
-  epic-21-retrospective: optional
-
-  # Epic 14: External Tooling Integration
-  # Dependencies: Epic 12, Epic 10, Epic 13 | Analyst + Architect required
-  # Stories 14.4, 14.5 blocked on Story 14.3 architecture approval
-        epic-14: backlog
-  14-1-research-sprint-management-integration: backlog
-  14-2-research-knowledge-management-integration: backlog
-  14-3-architecture-external-tooling-abstraction: backlog
-  14-4-implement-jira-backend: backlog
-  14-5-implement-confluence-backend: backlog
-  epic-14-retrospective: optional