specweave 1.0.338 → 1.0.339

package/plugins/specweave-github/lib/github-batch-sync.ts

@@ -104,6 +104,10 @@ const US_HEADER_RE = /^###?\s+(US-\d{3,}E?):\s*(.+)$/;
 const AC_CHECKBOX_RE = /^-\s+\[([ xX])\]\s+\*\*(?<acId>AC-US\d+E?-\d{2})\*\*:\s*(?<desc>.+)$/;
 // Priority: **Priority**: P1
 const PRIORITY_RE = /\*\*Priority\*\*:\s*(P[0-3])/;
+// Section headers that end description collection
+const AC_SECTION_RE = /^\*\*(?:Acceptance Criteria|ACs?)\*\*:/i;
+// Metadata lines to exclude from description
+const METADATA_RE = /^\*\*(?:Project|Board|External)\*\*:/i;
 
 /**
  * Parse user stories from spec.md content.
@@ -112,11 +116,15 @@ function parseUserStoriesFromSpec(content: string): UserStoryForSync[] {
   const lines = content.split('\n');
   const userStories: UserStoryForSync[] = [];
   let current: UserStoryForSync | null = null;
+  let collectingDescription = false;
 
   for (const line of lines) {
     const usMatch = line.match(US_HEADER_RE);
     if (usMatch) {
-      if (current) userStories.push(current);
+      if (current) {
+        current.description = current.description.trim();
+        userStories.push(current);
+      }
       current = {
         id: usMatch[1],
         title: usMatch[2],
@@ -125,6 +133,7 @@ function parseUserStoriesFromSpec(content: string): UserStoryForSync[] {
         status: 'planned',
         acceptanceCriteria: [],
       };
+      collectingDescription = true;
       continue;
     }
 
@@ -132,6 +141,7 @@ function parseUserStoriesFromSpec(content: string): UserStoryForSync[] {
 
     const acMatch = line.match(AC_CHECKBOX_RE);
     if (acMatch?.groups) {
+      collectingDescription = false;
       current.acceptanceCriteria.push({
         id: acMatch.groups.acId,
         description: acMatch.groups.desc.trim(),
@@ -142,11 +152,29 @@ function parseUserStoriesFromSpec(content: string): UserStoryForSync[] {
 
     const prioMatch = line.match(PRIORITY_RE);
     if (prioMatch) {
+      collectingDescription = false;
       current.priority = prioMatch[1];
+      continue;
+    }
+
+    if (AC_SECTION_RE.test(line)) {
+      collectingDescription = false;
+      continue;
+    }
+
+    // Collect description lines between US header and first AC/Priority
+    if (collectingDescription) {
+      if (METADATA_RE.test(line)) continue;
+      if (line.trim() !== '' || current.description.length > 0) {
+        current.description += line + '\n';
+      }
     }
   }
 
-  if (current) userStories.push(current);
+  if (current) {
+    current.description = current.description.trim();
+    userStories.push(current);
+  }
 
   return userStories;
 }

package/src/templates/AGENTS.md.template

@@ -51,11 +51,10 @@ See **Task Format** and **User Story Format** sections for templates.
 
 Never mark a task complete without proving it works:
 - Code compiles/builds successfully
-- Run tests: `npx vitest run` (unit) + `npx playwright test` (E2E) after every task
-- For critical paths: `/sw:grill` for quality, `/sw:judge-llm` for independent validation
-- Ask user to manually verify: new UI flows, auth, payments, data migrations
+- Run tests after every task: `npx vitest run` + `npx playwright test`
+- `/sw:grill` writes `grill-report.json` — CLI blocks closure without it
+- `/sw:judge-llm` writes `judge-llm-report.json` — WAIVED if consent denied
 - Acceptance criteria actually satisfied
-- Ask: "Would a staff engineer approve this?"
 
 ### 3. Dependencies First
 
@@ -370,13 +369,11 @@ specweave context projects
 8. If 3 consecutive test failures: STOP, re-plan, ask user
 
 ### Closing Increment
-1. Full test suite: `npx vitest run` (all unit + integration)
-2. Full E2E suite: `npx playwright test` (all scenarios)
-3. Coverage check: `npx vitest run --coverage` (must meet targets in config.json)
-4. Ask user for manual acceptance: new UI, auth, payments, data migrations
-5. `/sw:done 0001` — PM validates 3 gates (tasks, tests, docs)
-6. Living docs synced automatically
-7. GitHub/Jira issue closed if enabled
+1. Full test suite: `npx vitest run`
+2. Full E2E: `npx playwright test`
+3. `/sw:grill <id>` — writes `grill-report.json` (CLI requires it)
+4. User acceptance for critical flows (UI, auth, payments)
+5. `/sw:done <id>` — validates report files + PM 3 gates (tasks, tests, docs)
 <!-- /SECTION -->
 
 ---

package/src/templates/CLAUDE.md.template

@@ -77,9 +77,9 @@ SpecWeave auto-detects product descriptions and routes to `/sw:increment`:
 
 ### 3. Verification Before Done
 - Never mark a task complete without proving it works
-- Run tests: `npx vitest run` (unit) + `npx playwright test` (E2E) after every task
-- For critical paths: `/sw:grill` for quality, `/sw:judge-llm` for independent validation
-- Ask user to manually verify: new UI flows, auth, payments, data migrations
+- Run tests after every task: `npx vitest run` + `npx playwright test`
+- `/sw:grill` writes `grill-report.json` — CLI blocks closure without it
+- `/sw:judge-llm` writes `judge-llm-report.json` — WAIVED if consent denied
 - Ask yourself: **"Would a staff engineer approve this?"**
 
 ### 4. Think-Before-Act (Dependencies)
@@ -201,9 +201,10 @@ Primary: `/sw:progress-sync`. Individual: `/sw-github:push`, `/sw-github:close`.
 - Never mark a task `[x]` until its tests pass
 
 ### Before Closing (`/sw:done`)
-- `/sw:grill` + `/sw:validate` — code quality + 130+ rule checks
-- E2E with Playwright CLI: `npx playwright test` (blocking gate)
-- Ask user for manual acceptance testing when: new UI flows, auth changes, payment flows, data migrations
+- `/sw:grill` writes `grill-report.json` — CLI blocks closure without it
+- `/sw:judge-llm` writes `judge-llm-report.json` — WAIVED if consent denied
+- `/sw:validate` — 130+ rule checks
+- E2E: `npx playwright test` (blocking gate)
 
 ### Test Stack
 - Unit/Integration: Vitest (`.test.ts`), ESM mocking with `vi.hoisted()` + `vi.mock()`