@anhth2/spec-driven-dev-plugin 0.5.0

package/skills/test/SKILL.tmpl

@@ -0,0 +1,296 @@
+---
+description: Generates unit and integration tests from BDD specs, runs the test suite and reports results, or smoke-tests live API endpoints on a running service. Trigger when: "/generate-tests", "/run-tests", "/smoke-test", "tạo test", "viết test", "chạy test", "generate tests", "run tests", "test kết quả", "smoke test", "test API", "kiểm tra endpoint đang chạy".
+---
+
+# Test Skills — Generate, Run & Smoke Test
+
+This skill handles three commands: `/generate-tests`, `/run-tests`, and `/smoke-test`.
+
+---
+
+## /generate-tests — Generate Unit & Integration Tests
+
+### Gate
+
+<!-- Directory: specs/bdd/**/*.feature — or pass UC-ID/class path as argument -->
+{{include:steps/gate.md}}
+
+Also read:
+- `.feature` file to understand expected behaviors
+- Find files to test: Controllers with `@trace.implements={UC-ID}`, Service implementations, Facade implementations (if applicable)
+
+### CHECKPOINT — Test Plan
+
+Before generating, present plan:
+
+```
+Test Plan — {UC-ID}:
+
+Unit Tests ({unit test framework}):
+  - {ServiceName}ServiceImplTest
+    • {method}: {scenario} → {expected result}
+  - {FacadeName}FacadeImplTest (if applicable)
+    • {method}: {scenario} → {expected result}
+
+Integration Tests:
+  - {ControllerName}ControllerTest
+    • {endpoint}: {scenario} → HTTP {status}
+
+Total: {N} test classes, ~{M} test cases
+Proceed? (Y/N)
+```
+
+Wait for confirmation.
+
+### Generate
+
+#### Unit Test Template
+
+```
+// @trace.verifies={UC-ID}
+// @trace.test_type=unit
+[Test class using your project's test framework]
+
+class {Resource}ServiceImplTest {
+
+  // Mock dependencies
+
+  @Test
+  // "Should {expected outcome} when {condition}"
+  void methodName_whenValid_shouldReturnExpected() {
+    // Given — set up mocks and inputs
+    // When  — call the method under test
+    // Then  — assert results and verify interactions
+  }
+
+  @Test
+  // "Should throw {Exception} when {resource} not found"
+  void methodName_whenNotFound_shouldThrowException() {
+    // Given — mock returns empty/null
+    // When & Then — assert exception is thrown
+  }
+}
+```
+
+#### Integration Test Template (Controller/HTTP layer)
+
+```
+// @trace.verifies={UC-ID}
+// @trace.test_type=integration
+[Integration test class for HTTP layer]
+
+class {Resource}ControllerTest {
+
+  // Inject MockMvc or HTTP test client
+  // MockBean/stub the Facade/Service
+
+  @Test
+  // "GET /v1/{resource} - should return 200 with data"
+  void filter_shouldReturn200() {
+    // Given — stub facade/service
+    // When  — perform HTTP GET
+    // Then  — assert status 200 and response body
+  }
+
+  @Test
+  // "POST /v1/{resource} - should return 201 when valid"
+  void create_shouldReturn201() {
+    // Given — stub facade/service
+    // When  — perform HTTP POST with valid body
+    // Then  — assert status 201
+  }
+
+  @Test
+  // "POST /v1/{resource} - should return 400 when invalid"
+  void create_shouldReturn400WhenInvalid() {
+    // Given — no stub needed
+    // When  — perform HTTP POST with missing required fields
+    // Then  — assert status 400
+  }
+}
+```
+
+### File Placement
+
+Follow project test structure from CLAUDE.md. Typical:
+```
+src/test/{language}/{package}/
+├── service/impl/{Resource}ServiceImplTest.{ext}
+├── facade/impl/{Resource}FacadeImplTest.{ext}  (if applicable)
+└── controller/{Resource}ControllerTest.{ext}
+```
+
+### Self-Review Checklist
+
+- [ ] `@trace.verifies` tag on every test class
+- [ ] Every scenario in .feature has at least one test
+- [ ] Mock only the correct layer (no repo mocks in controller tests)
+- [ ] Test names follow `methodName_whenCondition_shouldOutcome` pattern
+- [ ] No debug print statements in tests
+
+### Output
+
+```
+/generate-tests Complete — {UC-ID}:
+  ✅ service/{Service}Test.{ext} ({M} tests)
+  ✅ facade/{Facade}Test.{ext} ({M} tests)
+  ✅ controller/{Controller}Test.{ext} ({M} tests)
+```
+
+{{include:steps/report-footer.md}}
+
+---
+
+## /run-tests — Run Tests & Report Results
+
+### Gate
+
+{{include:steps/context-loader.md}}
+
+Identify service/module from argument (UC-ID or service name).
+Read `conventions.test_command` from the loaded project context.
+
+### Run
+
+```bash
+# Run all tests in module
+{TEST_COMMAND}
+
+# Run specific test class (faster)
+{TEST_COMMAND_FOR_CLASS}
+
+# Run by pattern
+{TEST_COMMAND_BY_PATTERN}
+```
+
+### Analyze Results
+
+#### When tests PASS
+
+```
+✅ Tests passed
+- Total: {N}
+- Passed: {N}
+- Duration: {X}s
+```
+
+#### When tests FAIL — Debug Mode
+
+Read stack trace and analyze:
+
+| Error Pattern | Common Cause | Suggested Fix |
+|---|---|---|
+| NullPointerException in test | Missing mock setup | Check `given(...)` setup for null dependency |
+| Bean/dependency not found | Missing mock/stub declaration | Add mock for missing dependency |
+| Expected 200 but got 403 | Missing auth setup in test | Add auth user/role to test context |
+| Expected 200 but got 400 | Request body fails validation | Check required fields in request DTO |
+| LazyInitializationException | Lazy collection accessed outside transaction | Add `@Transactional` on test or use eager fetch |
+| Mapper/mapper implementation not found | Code generator (e.g. MapStruct) not run | Run compile step before tests |
+
+### Output
+
+```
+/run-tests Report — {service}
+Run at: {datetime}
+
+## Summary
+✅ Passed: {N} | ❌ Failed: {M} | ⏭️ Skipped: {K}
+Duration: {X}s
+
+## Failed Tests
+| Test | Error | Root Cause |
+|------|-------|-----------|
+| {TestClass}.{method} | {exception} | {analysis} |
+
+## Recommendations
+{specific fix for each failure}
+```
+
+{{include:steps/report-footer.md}}
+
+---
+
+## /smoke-test — Test Live API Endpoints
+
+Use when the service is **already running** to verify endpoints work correctly.
+Different from `/run-tests` (which runs unit/integration tests without a live server).
+
+### Phase 1 — Find Service URL
+
+{{include:steps/context-loader.md}}
+
+Read `conventions.service_run` from the loaded project context to find port.
+Default: `http://localhost:8080`
+
+Check if service is running:
+```bash
+curl -s http://localhost:{port}/health
+# or /actuator/health for Spring Boot
+```
+
+If not running: "Service is not running. Start it with: `{RUN_COMMAND}` from your project root."
+
+### Phase 2 — Identify Endpoints
+
+From UC-ID → find Controller with `@trace.implements={UC-ID}`:
+- List: method, path, required auth/role
+
+### Phase 3 — Get Auth Token (if needed)
+
+If endpoints require auth, ask:
+```
+This endpoint requires authentication. Options:
+1. Paste your Bearer token (from Postman / browser DevTools / test login)
+2. Use a test/dev token from your local config
+3. Skip auth — only test public endpoints
+```
+
+### Phase 4 — Run Smoke Test
+
+```bash
+# GET
+curl -s -X GET "http://localhost:{port}/v1/{resource}?page=0&size=5" \
+  -H "Authorization: Bearer {token}" | {JSON_FORMATTER}
+
+# POST
+curl -s -X POST "http://localhost:{port}/v1/{resource}" \
+  -H "Authorization: Bearer {token}" \
+  -H "Content-Type: application/json" \
+  -d '{"field1": "test_value"}'
+```
+
+### Phase 5 — Interpret Results
+
+| Result | Meaning |
+|--------|---------|
+| 200/201 + correct data | ✅ OK |
+| 200 + wrong data | ⚠️ Logic bug → use `/debug` |
+| 400 | Request body wrong → check required fields |
+| 401 | Token expired or wrong realm |
+| 403 | Wrong role → check auth config |
+| 500 + stacktrace | Server error → paste into `/debug` |
+| Connection refused | Service not running or wrong port |
+
+If errors in logs:
+```bash
+tail -n 100 {LOG_FILE_PATH}
+```
+
+### Output
+
+```
+/smoke-test Report — {UC-ID}
+Tested at: {datetime}
+Base URL: http://localhost:{port}
+
+## Endpoints Tested
+| Method | Path | Status | Result |
+|--------|------|--------|--------|
+| GET    | /v1/{resource} | 200 ✅ | 5 records returned |
+| POST   | /v1/{resource} | 201 ✅ | Created id=42 |
+
+## Issues Found
+{describe any failures}
+```
+
+{{include:steps/report-footer.md}}

package/steps/context-loader.md

@@ -0,0 +1,163 @@
+# Context Loader — Load All Project Context
+
+Execute these steps in order. Store everything in memory for the duration of the command session.
+
+**Priority guide (anti-lost-in-middle):**
+- Steps 1–2 are PROJECT-CONFIG — loaded first, resolve all paths and metadata.
+- Step 3 is CRITICAL — architecture + coding standards, the highest-priority facts for generation.
+- Step 4 is SAFETY — data protection rules, enforced silently for the entire session.
+- Steps 5–6 are DOMAIN KNOWLEDGE — terminology and entity definitions.
+- Step 7 is the WORKING MEMORY RECAP — locks critical facts into the top of working memory.
+
+---
+
+## Step 1 — [PROJECT-CONFIG] Load project-context.yaml
+
+Read `.agent/project-context.yaml`. Extract and store:
+
+**Tech Stack:**
+- `tech_stack.language` → active language (e.g., Java 17, TypeScript, C#, Go)
+- `tech_stack.framework` → active framework (e.g., Spring Boot 3.2, Angular 17, .NET 8)
+- `tech_stack.build_tool` → build tool (e.g., Maven, npm, dotnet, go)
+- `tech_stack.test_framework` → test framework (e.g., JUnit 5 + Mockito, Jest, xUnit)
+- `tech_stack.database` → database (e.g., PostgreSQL, MySQL, MongoDB)
+- `tech_stack.module` → active module profile (e.g., java-spring, angular, dotnet, golang, context-engineering)
+
+**Conventions:**
+- `conventions.build_command` → how to compile/build
+- `conventions.test_command` → how to run tests
+- `conventions.service_run` → how to start the service
+- `conventions.ticket_prefix` → ticket ID prefix (e.g., PROJ, FEAT, UC)
+
+**Domains:**
+- `domains` → list of active business domains
+
+**Paths (if present):**
+- `paths.specs_dir` → BDD specs root
+- `paths.prd_dir` → PRD documents root
+- `paths.refinement_dir` → findings/review output dir
+- `paths.product_definitions_dir` → product definitions root
+- `paths.domain_knowledge_dir` → domain knowledge root
+- `paths.business_dictionary` → path to business-dictionary.md
+- `paths.core_entities` → path to core-entities.md
+- `paths.tech_docs_dir` → technical documentation root
+- `paths.trace_dir` → trace state directory
+
+If `paths` section is absent, use these defaults:
+- `specs_dir` = `specs/bdd`
+- `prd_dir` = `specs/prd`
+- `refinement_dir` = `.agent/review`
+- `product_definitions_dir` = `specs/product-definition`
+- `domain_knowledge_dir` = `specs/domain-knowledge`
+- `business_dictionary` = `specs/domain-knowledge/business-dictionary.md`
+- `core_entities` = `specs/domain-knowledge/core-entities.md`
+- `tech_docs_dir` = `tech-docs`
+- `trace_dir` = `.trace`
+
+If `tech_stack.module` is set, also load `.agent/modules/{module}/stack-profile.yaml` if it exists.
+
+---
+
+## Step 2 — [PROJECT-CONFIG] Load module stack profile (conditional)
+
+If `tech_stack.module` is set, read `.agent/modules/{module}/stack-profile.yaml`.
+Merge framework-specific conventions (layer patterns, test patterns, naming rules) into the loaded context.
+If the file does not exist → skip silently.
+
+---
+
+## Step 3 — [CRITICAL] Load CLAUDE.md
+
+*This is the highest-priority context — it defines HOW to write code and documents for this project.*
+
+Read `CLAUDE.md`. Extract and store:
+
+- **§1 Project Overview** → project name, language, framework, build/test commands, domains
+- **§2 Architecture** → layer order (e.g., Controller → Facade → Service → Repository), architectural rules
+- **§3 Coding Standards** → naming conventions (classes, methods), response wrapper type, forbidden patterns
+- **§5 Error Handling** → exception types, HTTP status code mapping, not-found exception class name
+- **§7 Git Conventions** → branch naming pattern, commit message format
+
+If `CLAUDE.md` does not exist → note it as missing and continue with project-context.yaml data only.
+
+---
+
+## Step 4 — [SAFETY] Load Data Protection Rules
+
+Read `.agent/rules/data-protection.md` (or `rules/data-protection.md` from the framework installation).
+
+Store the sensitive file patterns — you must **never** read, write, display, or reference content from files matching those patterns for the entire session.
+
+If neither file exists → apply built-in defaults: never access `.env*`, `*.key`, `*.pem`, `*secret*`, `*password*`, `*credential*`.
+
+---
+
+## Step 5 — [DOMAIN] Load Business Dictionary (conditional)
+
+Check if the business dictionary file exists (use `paths.business_dictionary` resolved in Step 1).
+
+If it exists, read it and extract:
+- **Canonical Terms** → complete list of approved terms and their definitions
+- **Banned Terms** → complete list of banned terms and their canonical replacements
+- **Status / Enum Registry** → allowed enum values per entity
+
+Store the banned terms list for **active enforcement** throughout the command session:
+- When generating any text (PRD, BDD, code comments, tech docs), verify no banned terms appear
+- Replace banned terms with their canonical equivalents automatically
+
+If the file does not exist → skip silently. Do not warn or block.
+
+---
+
+## Step 6 — [DOMAIN] Load Core Entities (conditional)
+
+Check if the core entities file exists at `paths.core_entities` (resolved in Step 1).
+Default path: `specs/domain-knowledge/core-entities.md`.
+
+If it exists, read it and store:
+- **Entity catalog** → for each entity: its name, purpose, owner service, key fields (name + type), business invariants, and relationships
+- **Field name registry** → canonical field names to use in generated code and documents
+- **Relationship map** → how entities relate to each other (1:N, N:N, embedded, etc.)
+
+**How to use this catalog:**
+- When generating code: use the field names, types, and relationships defined here — do NOT infer from existing code
+- When generating PRD/BDD: reference entity names from this catalog for consistency
+- When generating tech-docs: use this catalog as the source-of-truth for entity definitions
+
+If the file does not exist → skip silently.
+
+---
+
+## Step 7 — [RECAP] Working Memory Recap (anti-lost-in-middle)
+
+After loading all context, synthesize and output a compact summary block.
+This recap ensures the most critical facts are stated at the END of context loading
+(recency effect — freshest in working memory when the task begins).
+
+Output exactly this block:
+```
+[CTX LOADED]
+Stack     : {language} / {framework} / {database}
+Layers    : {layer order from CLAUDE.md §2, e.g., Controller → Facade → Service → Repository}
+Ticket    : {ticket_prefix}-
+Dict      : {loaded — N canonical terms, M banned terms | missing}
+Entities  : {loaded — EntityA, EntityB, EntityC | missing}
+Status    : {FULL | PARTIAL — missing: CLAUDE.md / business-dict / core-entities | MINIMAL}
+```
+
+If any CRITICAL file is missing (CLAUDE.md), flag it clearly so the user can decide whether to proceed.
+
+---
+
+## Context Load Complete
+
+After completing all steps, you have loaded:
+- Project identity, tech stack, module conventions
+- Architecture rules and layer order  ← **[CRITICAL — hold in working memory]**
+- Coding standards and naming conventions  ← **[CRITICAL — hold in working memory]**
+- Data protection rules (sensitive file patterns to never access)
+- Terminology rules with banned-term list  ← **[DOMAIN — apply to every generated word]**
+- Entity catalog (field names, types, invariants)  ← **[DOMAIN — use for code generation]**
+- All configured paths
+
+Proceed to the next step of the calling command.

package/steps/gate.md

@@ -0,0 +1,81 @@
+# Gate — Universal Entry Procedure
+
+Every command must execute this gate before proceeding with its specific logic.
+
+## Step 0 — Sub-Agent Mode Check
+
+Before anything else, check if `$ARGUMENTS` is a JSON payload from an orchestrator:
+
+1. Attempt to parse `$ARGUMENTS` as JSON.
+2. If it parses successfully **and** contains `"_agent_mode": true`:
+   - **Skip Steps 1, 2, and 3 of this Gate entirely.**
+   - Set target file = `payload.target_file`
+   - Set loaded context = `payload.context` (do NOT run context-loader.md)
+   - Set UC scope = `payload.uc_id` (process only this UC)
+   - Set line range = `payload.uc_section` (read only that PRD section)
+   - Proceed directly to the command-specific logic.
+3. If `$ARGUMENTS` is not JSON or `_agent_mode` is absent → continue to Step 1 (normal mode).
+
+## Step 0-B — Model Check
+
+*Skip this step if `_agent_mode: true` (sub-agent — orchestrator already validated).*
+
+Complex generation and review commands require strong reasoning.
+Using a smaller model risks missed edge cases, incomplete spec analysis, and architecture violations.
+
+Display and wait for response:
+
+```
+⚙️  MODEL CHECK
+──────────────────────────────────────────────────────────────────
+  Recommended  : claude-opus-4-5 (or claude-opus-4)
+  Why needed   : Spec analysis, architecture review, code generation
+                 require deep reasoning. Smaller models miss edge cases.
+
+  To switch in Claude Code:
+    • Settings → Model → select "claude-opus"
+    • or: /model → choose claude-opus
+
+  Running on claude-opus?
+    Y — yes, on claude-opus → proceed
+    S — skip check (I accept lower quality risk with current model)
+──────────────────────────────────────────────────────────────────
+```
+
+- "Y" → proceed to Step 1.
+- "S" → proceed to Step 1 (user accepts risk, add ⚠️ to final report).
+- "N" or anything else → **STOP.** Output: "Please switch to claude-opus, then re-run this command."
+
+## Step 1 — Resolve Target File
+
+1. If `$ARGUMENTS` is provided and points to an existing file → use it directly as the target.
+2. If `$ARGUMENTS` is a UC-ID, ticket ID, or partial name → search for matching files in the relevant directory.
+3. If `$ARGUMENTS` is empty or no match found:
+   - List files in the relevant directory for this command (e.g., `specs/prd/**/*.md` for PRD commands, `specs/bdd/**/*.feature` for BDD commands).
+   - Present the list to the user and ask: "Which file do you want to work with? (Enter number or filename)"
+   - Wait for user selection before continuing.
+
+## Step 2 — Execute Context Loader
+
+Load all project context by following the procedure in `steps/context-loader.md`.
+Store all loaded context in memory for use throughout this command session.
+
+## Step 3 — CHECKPOINT
+
+After completing Steps 1 and 2, display a summary and wait for confirmation:
+
+```
+CHECKPOINT
+-----------
+Target     : {resolved file path}
+Project    : {project.name from project-context.yaml}
+Tech stack : {language} / {framework}
+Module     : {module if set, else "not configured"}
+Domains    : {comma-separated domain list}
+
+Proceed? (Y/N)
+```
+
+Wait for explicit "Y" or "N" from the user before continuing.
+- "Y" → proceed to the command-specific steps below.
+- "N" → stop and ask what the user wants to change.

package/steps/report-footer.md

@@ -0,0 +1,53 @@
+# Report Footer — Standard Command Output Format
+
+Every command report must end with this standard footer section.
+
+## Status Badge
+
+Choose one based on outcome:
+- `✅ Complete` — all steps succeeded, no issues found
+- `❌ Failed` — command could not complete due to a blocking error
+- `⚠️ Warnings` — completed with non-blocking issues that should be reviewed
+
+## Output Artifacts
+
+List every file created or modified by this command:
+```
+Output Artifacts:
+  {created|updated} {file-path} ({brief description})
+  {created|updated} {file-path} ({brief description})
+```
+
+If no files were written (e.g., review or analysis commands) → write `Output Artifacts: none (read-only)`.
+
+## Next Command Suggestion
+
+Suggest the logical next command based on workflow phase:
+
+| Current command         | Suggest next                                  |
+|-------------------------|-----------------------------------------------|
+| /define-product         | `/generate-prd {product-definition-file}`     |
+| /generate-prd           | `/refine-prd {prd-file}` then `/review-context {prd-file}` |
+| /refine-prd             | Open Review Board → update PRD → `/review-context {prd-file}` |
+| /review-context (PRD)   | `/generate-bdd {prd-file}` if APPROVED; fix PRD if NEEDS_FIX |
+| /generate-bdd           | `/review-context {feature-file}` to verify coverage |
+| /review-context (BDD)   | `/generate-tech-docs {UC-ID}` if APPROVED; regenerate if NEEDS_FIX |
+| /generate-tech-docs     | `/review-tech-docs {tech-design-file}`        |
+| /review-tech-docs       | `/generate-code {feature-file}` if APPROVED; fix doc if NEEDS_FIX |
+| /generate-code          | `/generate-tests {UC-ID}`                     |
+| /generate-tests         | `/run-tests {UC-ID}`                          |
+| /run-tests (passing)    | `/review-code {UC-ID}`                        |
+| /run-tests (failing)    | `/fix-bug {ticket-id}` or `/debug {error}`    |
+| /review-code            | `/smoke-test {UC-ID}` or create PR            |
+| /smoke-test             | Create PR and link to ticket                  |
+| /validate-traces        | `/generate-code {UC-ID}` for gaps             |
+| /fix-bug                | Create PR and link to ticket                  |
+| /debug                  | `/fix-bug {ticket-id}` if fix needed          |
+
+Format the footer as:
+```
+---
+Status : {badge}
+{Output Artifacts block}
+Next   : {suggested command with example arguments}
+```