specweave 1.0.362 → 1.0.367

package/plugins/specweave/skills/team-lead/agents/security.md

@@ -26,21 +26,21 @@ WORKFLOW:
   3. Create YOUR increment in YOUR repo: .specweave/increments/[ID]/
   4. Activate the increment: Edit metadata.json to set "status": "active" and update "lastActivity" timestamp
   5. Read the MASTER SPEC at [MASTER_INCREMENT_PATH]/spec.md for scope and ACs
-  5. Audit code produced by other agents for security issues
-  6. Create plan files (plan.md, tasks.md) for your increment
-  7. Send plan to team-lead and WAIT for approval:
+  6. Audit code produced by other agents for security issues
+  7. Create plan files (plan.md, tasks.md) for your increment
+  8. Send plan to team-lead and WAIT for approval:
      SendMessage({ type: "message", recipient: "team-lead",
        content: "PLAN_READY: [increment path]. [summary of security findings, hardening plan].",
        summary: "Security plan ready for review" })
-  8. WAIT for "PLAN_APPROVED" message. If "PLAN_REJECTED", revise and re-submit.
-  9. Implement auth/authz middleware if needed
-  10. Add input validation and sanitization
-  11. Execute tasks autonomously: /sw:auto --simple (minimal context mode to prevent context overflow)
-  12. Run all tests for owned code (security tests): npm test
-  13. Run security audit tools (npm audit, dependency check)
-  14. Do NOT signal completion until all tests pass
-  15. Signal COMPLETION via SendMessage to team-lead with summary of tasks done, test results, and security findings
-  16. Do NOT run /sw:done or /sw:grill yourself — team-lead handles closure centrally
+  9. WAIT for "PLAN_APPROVED" message. If "PLAN_REJECTED", revise and re-submit.
+  10. Implement auth/authz middleware if needed
+  11. Add input validation and sanitization
+  12. Execute tasks autonomously: /sw:auto --simple (minimal context mode to prevent context overflow)
+  13. Run all tests for owned code (security tests): npm test
+  14. Run security audit tools (npm audit, dependency check)
+  15. Do NOT signal completion until all tests pass
+  16. Signal COMPLETION via SendMessage to team-lead with summary of tasks done, test results, and security findings
+  17. Do NOT run /sw:done or /sw:grill yourself — team-lead handles closure centrally
 
 RULES:
   - WRITE only to files you own (listed above)

package/plugins/specweave/skills/team-lead/agents/testing.md

@@ -31,21 +31,21 @@ WORKFLOW:
   3. Create YOUR increment in YOUR repo: .specweave/increments/[ID]/
   4. Activate the increment: Edit metadata.json to set "status": "active" and update "lastActivity" timestamp
   5. Read the MASTER SPEC at [MASTER_INCREMENT_PATH]/spec.md for scope and ACs
-  5. Wait for ALL other agents to produce initial code
-  6. Create plan files (plan.md, tasks.md) for your increment
-  7. Send plan to team-lead and WAIT for approval:
+  6. Wait for ALL other agents to produce initial code
+  7. Create plan files (plan.md, tasks.md) for your increment
+  8. Send plan to team-lead and WAIT for approval:
      SendMessage({ type: "message", recipient: "team-lead",
        content: "PLAN_READY: [increment path]. [summary of test strategy, coverage plan].",
        summary: "Testing plan ready for review" })
-  8. WAIT for "PLAN_APPROVED" message. If "PLAN_REJECTED", revise and re-submit.
-  9. Write unit tests for new services/components
-  10. Write integration tests for API endpoints
-  11. Write E2E tests for user journeys
-  12. Execute tasks autonomously: /sw:auto --simple (minimal context mode to prevent context overflow)
-  13. Run all tests (unit + integration + E2E): npm test && npx playwright test
-  14. Do NOT signal completion until all tests pass -- if tests fail, fix and repeat
-  15. Signal COMPLETION via SendMessage to team-lead with summary of tasks done and test results
-  16. Do NOT run /sw:done or /sw:grill yourself — team-lead handles closure centrally
+  9. WAIT for "PLAN_APPROVED" message. If "PLAN_REJECTED", revise and re-submit.
+  10. Write unit tests for new services/components
+  11. Write integration tests for API endpoints
+  12. Write E2E tests for user journeys
+  13. Execute tasks autonomously: /sw:auto --simple (minimal context mode to prevent context overflow)
+  14. Run all tests (unit + integration + E2E): npm test && npx playwright test
+  15. Do NOT signal completion until all tests pass -- if tests fail, fix and repeat
+  16. Signal COMPLETION via SendMessage to team-lead with summary of tasks done and test results
+  17. Do NOT run /sw:done or /sw:grill yourself — team-lead handles closure centrally
 
 RULES:
   - WRITE only to test files (listed above)

package/plugins/specweave/skills/team-merge/SKILL.md

@@ -71,15 +71,26 @@ Closure order respects contract chain:
 For each teammate's increment, in dependency order:
 
 ```bash
+# PRE-CLOSURE: Ensure increment is in "active" status (agents may not have activated it)
+STATUS=$(jq -r '.status' .specweave/increments/<id>/metadata.json)
+if [ "$STATUS" = "planned" ] || [ "$STATUS" = "backlog" ]; then
+  # Edit metadata.json to set "status": "active"
+fi
+
 # Run /sw:done --auto per increment -- triggers quality gates, skips user confirmation
 /sw:done <increment-id> --auto
+
+# If /sw:done fails, fix root cause and retry (max 2 retries)
+# Common fixes: sync ACs, update task counts, write missing reports
 ```
 
 This ensures:
+- Increment is in correct lifecycle status before closure attempt
 - `/sw:grill` runs for each increment
 - `tasks.md` and `spec.md` ACs are validated
 - `metadata.json` is updated to `completed`
 - Living docs are generated
+- Failures are retried rather than silently skipped
 
 ### Step 5: Trigger Sync
 

package/plugins/specweave/skills/test-aware-planner/SKILL.md

@@ -1,7 +1,7 @@
 ---
 description: Generate tasks.md with embedded test plans in BDD format, one user story at a time to prevent crashes. Use for test-first task planning where each task includes Given/When/Then scenarios. Produces implementation tasks with inline test specifications.
 context: fork
-model: opus
+model: sonnet
 ---
 
 # Test-Aware Planner Skill

package/plugins/specweave-ado/lib/ado-multi-project-sync.js

@@ -373,7 +373,6 @@ ${userStory.technicalContext}
         return mapping.task || "Task";
       case "Subtask":
         return mapping.task || "Task";
-      // ADO doesn't have subtasks, use Task
       default:
         return "User Story";
     }

package/plugins/specweave-ado/lib/enhanced-ado-sync.js

@@ -0,0 +1,180 @@
+import { AdoClientV2 } from "./ado-client-v2.js";
+import { EnhancedContentBuilder } from "../../../src/core/sync/enhanced-content-builder.js";
+import { SpecIncrementMapper } from "../../../src/core/sync/spec-increment-mapper.js";
+import { parseSpecContent } from "../../../src/core/spec-content-sync.js";
+import path from "path";
+import fs from "fs/promises";
+async function syncSpecToAdoWithEnhancedContent(options) {
+  const { specPath, organization, project, dryRun = false, verbose = false } = options;
+  try {
+    const baseSpec = await parseSpecContent(specPath);
+    if (!baseSpec) {
+      return {
+        success: false,
+        action: "error",
+        error: "Failed to parse spec content"
+      };
+    }
+    if (verbose) {
+      console.log(`\u{1F4C4} Parsed spec: ${baseSpec.identifier.compact}`);
+    }
+    const specId = baseSpec.identifier.full || baseSpec.identifier.compact;
+    const rootDir = await findSpecWeaveRoot(specPath);
+    const mapper = new SpecIncrementMapper(rootDir);
+    const mapping = await mapper.mapSpecToIncrements(specId);
+    if (verbose) {
+      console.log(`\u{1F517} Found ${mapping.increments.length} related increments`);
+    }
+    const taskMapping = buildTaskMapping(mapping.increments, organization, project);
+    const architectureDocs = await findArchitectureDocs(rootDir, specId);
+    const enhancedSpec = {
+      ...baseSpec,
+      summary: baseSpec.description,
+      taskMapping,
+      architectureDocs
+    };
+    const builder = new EnhancedContentBuilder();
+    const description = builder.buildExternalDescription(enhancedSpec);
+    if (verbose) {
+      console.log(`\u{1F4DD} Generated description: ${description.length} characters`);
+    }
+    if (dryRun) {
+      console.log("\u{1F50D} DRY RUN - Would create/update feature with:");
+      console.log(`   Title: ${baseSpec.title}`);
+      console.log(`   Description length: ${description.length}`);
+      return {
+        success: true,
+        action: "no-change",
+        tasksLinked: taskMapping?.tasks.length || 0
+      };
+    }
+    if (!organization || !project) {
+      return {
+        success: false,
+        action: "error",
+        error: "Azure DevOps organization/project not specified"
+      };
+    }
+    const profile = {
+      provider: "ado",
+      displayName: `${organization}/${project}`,
+      config: {
+        organization,
+        project
+      },
+      timeRange: { default: "1M", max: "6M" }
+    };
+    const pat = process.env.AZURE_DEVOPS_PAT || "";
+    const client = new AdoClientV2(profile, pat);
+    const existingFeature = await findExistingFeature(client, baseSpec.identifier.compact);
+    let result;
+    if (existingFeature) {
+      await client.updateWorkItem(existingFeature.id, {
+        title: `[${baseSpec.identifier.compact}] ${baseSpec.title}`,
+        description
+      });
+      result = {
+        success: true,
+        action: "updated",
+        featureId: existingFeature.id,
+        featureUrl: `https://dev.azure.com/${organization}/${project}/_workitems/edit/${existingFeature.id}`,
+        tasksLinked: taskMapping?.tasks.length || 0
+      };
+    } else {
+      const feature = await client.createEpic({
+        title: `[${baseSpec.identifier.compact}] ${baseSpec.title}`,
+        description,
+        tags: ["spec", "external-tool-sync"]
+      });
+      result = {
+        success: true,
+        action: "created",
+        featureId: feature.id,
+        featureUrl: `https://dev.azure.com/${organization}/${project}/_workitems/edit/${feature.id}`,
+        tasksLinked: taskMapping?.tasks.length || 0
+      };
+    }
+    if (verbose) {
+      console.log(`\u2705 ${result.action === "created" ? "Created" : "Updated"} feature #${result.featureId}`);
+    }
+    return result;
+  } catch (error) {
+    return {
+      success: false,
+      action: "error",
+      error: error.message
+    };
+  }
+}
+async function findSpecWeaveRoot(specPath) {
+  let currentDir = path.dirname(specPath);
+  while (true) {
+    const specweaveDir = path.join(currentDir, ".specweave");
+    try {
+      await fs.access(specweaveDir);
+      return currentDir;
+    } catch {
+      const parentDir = path.dirname(currentDir);
+      if (parentDir === currentDir) {
+        throw new Error(".specweave directory not found");
+      }
+      currentDir = parentDir;
+    }
+  }
+}
+function buildTaskMapping(increments, organization, project) {
+  if (increments.length === 0) return void 0;
+  const firstIncrement = increments[0];
+  const tasks = firstIncrement.tasks.map((task) => ({
+    id: task.id,
+    title: task.title,
+    userStories: task.userStories
+  }));
+  // Derive repository name from git remote or fall back to project name
+  let repoName = project;
+  try {
+    const { execSync } = require("child_process");
+    const remoteUrl = execSync("git remote get-url origin", { encoding: "utf-8" }).trim();
+    const match = remoteUrl.match(/\/([^/]+?)(?:\.git)?$/);
+    if (match) repoName = match[1];
+  } catch {
+    // Fallback to project name if git is unavailable
+  }
+  return {
+    incrementId: firstIncrement.id,
+    tasks,
+    tasksUrl: `https://dev.azure.com/${organization}/${project}/_git/${repoName}?path=/.specweave/increments/${firstIncrement.id}/tasks.md`
+  };
+}
+async function findArchitectureDocs(rootDir, specId) {
+  const docs = [];
+  const archDir = path.join(rootDir, ".specweave/docs/internal/architecture");
+  try {
+    const adrDir = path.join(archDir, "adr");
+    try {
+      const adrs = await fs.readdir(adrDir);
+      const relatedAdrs = adrs.filter((file) => file.includes(specId.replace("spec-", "")));
+      for (const adr of relatedAdrs) {
+        docs.push({
+          type: "adr",
+          path: path.join(adrDir, adr),
+          title: adr.replace(".md", "").replace(/-/g, " ")
+        });
+      }
+    } catch {
+    }
+  } catch {
+  }
+  return docs;
+}
+async function findExistingFeature(client, specId) {
+  try {
+    const features = await client.queryWorkItems(`[System.Title] Contains '[${specId}]' AND [System.WorkItemType] = 'Feature'`);
+    return features[0] || null;
+  } catch {
+    return null;
+  }
+}
+export {
+  syncSpecToAdoWithEnhancedContent
+};