specweave 1.0.350 → 1.0.352

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

@@ -163,7 +163,8 @@ class AdoSpecSync {
         value: this.mapPriorityToAdo(spec.metadata.priority)
       }
     ];
-    const response = await this.client.post("/wit/workitems/$Feature?api-version=7.0", payload);
+    const encodedType = encodeURIComponent(workItemType);
+    const response = await this.client.post(`/wit/workitems/$${encodedType}?api-version=7.0`, payload);
     const featureData = response.data;
     console.log(`   \u2705 Created ADO Feature #${featureData.id}: ${featureData._links.html.href}`);
     return {
@@ -173,29 +174,41 @@ class AdoSpecSync {
     };
   }
   /**
-   * Update existing ADO Feature
+   * Update existing ADO Feature (conditional — only writes changed fields)
    */
   async updateAdoFeature(featureId, spec) {
     const featureTitle = `[${spec.metadata.id.toUpperCase()}] ${spec.metadata.title}`;
     const featureDescription = this.generateFeatureDescription(spec);
-    const payload = [
-      {
+    const current = await this.fetchAdoFeature(featureId);
+    const payload = [];
+    if (current.fields["System.Title"] !== featureTitle) {
+      payload.push({
         op: "replace",
         path: "/fields/System.Title",
         value: featureTitle
-      },
-      {
+      });
+    } else {
+      console.log(`   \u2139\uFE0F  Title unchanged, skipping`);
+    }
+    if (current.fields["System.Description"] !== featureDescription) {
+      payload.push({
         op: "replace",
         path: "/fields/System.Description",
         value: featureDescription
-      }
-    ];
+      });
+    } else {
+      console.log(`   \u2139\uFE0F  Description unchanged, skipping`);
+    }
+    if (payload.length === 0) {
+      console.log(`   \u2139\uFE0F  No changes detected for ADO Feature #${featureId}`);
+      return current;
+    }
     const response = await this.client.patch(
       `/wit/workitems/${featureId}?api-version=7.0`,
       payload
     );
     const featureData = response.data;
-    console.log(`   \u2705 Updated ADO Feature #${featureId}`);
+    console.log(`   \u2705 Updated ADO Feature #${featureId} (${payload.length} field(s) changed)`);
     return {
       id: featureData.id,
       url: featureData._links.html.href,

package/plugins/specweave-ado/lib/ado-spec-sync.ts

@@ -255,7 +255,8 @@ export class AdoSpecSync {
       }
     ];
 
-    const response = await this.client.post('/wit/workitems/$Feature?api-version=7.0', payload);
+    const encodedType = encodeURIComponent(workItemType);
+    const response = await this.client.post(`/wit/workitems/$${encodedType}?api-version=7.0`, payload);
     const featureData = response.data;
 
     console.log(`   ✅ Created ADO Feature #${featureData.id}: ${featureData._links.html.href}`);
@@ -268,24 +269,41 @@ export class AdoSpecSync {
   }
 
   /**
-   * Update existing ADO Feature
+   * Update existing ADO Feature (conditional — only writes changed fields)
    */
   private async updateAdoFeature(featureId: number, spec: SpecContent): Promise<AdoFeature> {
     const featureTitle = `[${spec.metadata.id.toUpperCase()}] ${spec.metadata.title}`;
     const featureDescription = this.generateFeatureDescription(spec);
 
-    const payload = [
-      {
+    // Fetch current values to avoid overwriting ADO-side edits
+    const current = await this.fetchAdoFeature(featureId);
+
+    const payload: any[] = [];
+
+    if (current.fields['System.Title'] !== featureTitle) {
+      payload.push({
         op: 'replace',
         path: '/fields/System.Title',
         value: featureTitle
-      },
-      {
+      });
+    } else {
+      console.log(`   ℹ️  Title unchanged, skipping`);
+    }
+
+    if (current.fields['System.Description'] !== featureDescription) {
+      payload.push({
         op: 'replace',
         path: '/fields/System.Description',
         value: featureDescription
-      }
-    ];
+      });
+    } else {
+      console.log(`   ℹ️  Description unchanged, skipping`);
+    }
+
+    if (payload.length === 0) {
+      console.log(`   ℹ️  No changes detected for ADO Feature #${featureId}`);
+      return current;
+    }
 
     const response = await this.client.patch(
       `/wit/workitems/${featureId}?api-version=7.0`,
@@ -293,7 +311,7 @@ export class AdoSpecSync {
     );
     const featureData = response.data;
 
-    console.log(`   ✅ Updated ADO Feature #${featureId}`);
+    console.log(`   ✅ Updated ADO Feature #${featureId} (${payload.length} field(s) changed)`);
 
     return {
       id: featureData.id,

package/plugins/specweave-ado/lib/conflict-resolver.js

@@ -3,6 +3,7 @@ import * as path from "path";
 import * as yaml from "yaml";
 const STATUS_MAPPING = {
   ado: {
+    // Agile process template
     "New": "draft",
     "Active": "in-progress",
     "Resolved": "implemented",
@@ -10,7 +11,20 @@ const STATUS_MAPPING = {
     "In Review": "in-qa",
     "In QA": "in-qa",
     "Blocked": "blocked",
-    "Removed": "cancelled"
+    "Removed": "cancelled",
+    // Scrum process template
+    "Approved": "draft",
+    "Committed": "in-progress",
+    "Done": "complete",
+    // CMMI process template
+    "Proposed": "draft",
+    // 'Active' already mapped above (shared with CMMI)
+    // 'Resolved' already mapped above (shared with CMMI)
+    // 'Closed' already mapped above (shared with CMMI)
+    // Basic process template
+    "To Do": "draft",
+    "Doing": "in-progress"
+    // 'Done' already mapped above (shared with Basic)
   },
   jira: {
     "To Do": "draft",
@@ -30,6 +44,8 @@ const STATUS_MAPPING = {
   }
 };
 const REVERSE_STATUS_MAPPING = {
+  // Default reverse mapping uses Agile states (most common)
+  // Callers should use process-template-aware mapping when template is known
   ado: {
     "draft": "New",
     "in-progress": "Active",

package/plugins/specweave-ado/lib/conflict-resolver.ts

@@ -87,6 +87,7 @@ export interface ExternalStatus {
 
 const STATUS_MAPPING = {
   ado: {
+    // Agile process template
     'New': 'draft' as SpecStatus,
     'Active': 'in-progress' as SpecStatus,
     'Resolved': 'implemented' as SpecStatus,
@@ -94,7 +95,20 @@ const STATUS_MAPPING = {
     'In Review': 'in-qa' as SpecStatus,
     'In QA': 'in-qa' as SpecStatus,
     'Blocked': 'blocked' as SpecStatus,
-    'Removed': 'cancelled' as SpecStatus
+    'Removed': 'cancelled' as SpecStatus,
+    // Scrum process template
+    'Approved': 'draft' as SpecStatus,
+    'Committed': 'in-progress' as SpecStatus,
+    'Done': 'complete' as SpecStatus,
+    // CMMI process template
+    'Proposed': 'draft' as SpecStatus,
+    // 'Active' already mapped above (shared with CMMI)
+    // 'Resolved' already mapped above (shared with CMMI)
+    // 'Closed' already mapped above (shared with CMMI)
+    // Basic process template
+    'To Do': 'draft' as SpecStatus,
+    'Doing': 'in-progress' as SpecStatus,
+    // 'Done' already mapped above (shared with Basic)
   },
   jira: {
     'To Do': 'draft' as SpecStatus,
@@ -115,6 +129,8 @@ const STATUS_MAPPING = {
 };
 
 const REVERSE_STATUS_MAPPING = {
+  // Default reverse mapping uses Agile states (most common)
+  // Callers should use process-template-aware mapping when template is known
   ado: {
     'draft': 'New',
     'in-progress': 'Active',

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

@@ -130,10 +130,20 @@ function buildTaskMapping(increments, organization, project) {
     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/repo?path=/.specweave/increments/${firstIncrement.id}/tasks.md`
+    tasksUrl: `https://dev.azure.com/${organization}/${project}/_git/${repoName}?path=/.specweave/increments/${firstIncrement.id}/tasks.md`
   };
 }
 async function findArchitectureDocs(rootDir, specId) {

package/plugins/specweave-ado/lib/per-us-sync.js

@@ -1,9 +1,16 @@
 import { consoleLogger } from "../../../src/utils/logger.js";
+const WORK_ITEM_TYPE_BY_TEMPLATE = {
+  agile: "User Story",
+  scrum: "Product Backlog Item",
+  cmmi: "Requirement",
+  basic: "Issue"
+};
 class PerUSAdoSync {
   constructor(adoClient, projectMappings, options = {}) {
     this.adoClient = adoClient;
     this.projectMappings = projectMappings;
     this.logger = options.logger ?? consoleLogger;
+    this.workItemType = options.workItemType ?? WORK_ITEM_TYPE_BY_TEMPLATE[options.processTemplate?.toLowerCase() ?? ""] ?? "User Story";
   }
   /**
    * Sync all user stories to their respective ADO projects
@@ -133,7 +140,7 @@ class PerUSAdoSync {
     } else {
       const newItem = await this.adoClient.createWorkItem(
         mapping.project,
-        "User Story",
+        this.workItemType,
         title,
         description,
         areaPath

package/plugins/specweave-ado/lib/per-us-sync.ts

@@ -81,6 +81,16 @@ export interface AdoClient {
   getWorkItemUrl(project: string, workItemId: number): string;
 }
 
+/**
+ * Default work item type by process template
+ */
+const WORK_ITEM_TYPE_BY_TEMPLATE: Record<string, string> = {
+  agile: 'User Story',
+  scrum: 'Product Backlog Item',
+  cmmi: 'Requirement',
+  basic: 'Issue',
+};
+
 /**
  * Per-US ADO Sync
  *
@@ -91,15 +101,20 @@ export class PerUSAdoSync {
   private projectMappings: ProjectMappings;
   private adoClient: AdoClient;
   private logger: Logger;
+  private workItemType: string;
 
   constructor(
     adoClient: AdoClient,
     projectMappings: ProjectMappings,
-    options: { logger?: Logger } = {}
+    options: { logger?: Logger; workItemType?: string; processTemplate?: string } = {}
   ) {
     this.adoClient = adoClient;
     this.projectMappings = projectMappings;
     this.logger = options.logger ?? consoleLogger;
+    // Explicit workItemType takes priority, then process template lookup, then default
+    this.workItemType = options.workItemType
+      ?? WORK_ITEM_TYPE_BY_TEMPLATE[options.processTemplate?.toLowerCase() ?? '']
+      ?? 'User Story';
   }
 
   /**
@@ -268,7 +283,7 @@ export class PerUSAdoSync {
       // Create new work item
       const newItem = await this.adoClient.createWorkItem(
         mapping.project,
-        'User Story',
+        this.workItemType,
         title,
         description,
         areaPath

package/plugins/specweave-github/hooks/github-auto-create-handler.sh

@@ -74,14 +74,38 @@ else
   [[ "$GH_ENABLED" != "true" ]] && { log "GitHub sync not enabled"; exit 0; }
 fi
 
-# Check auto-create enabled (either autoSync OR auto_create_github_issue)
+# Check auto-create enabled
+# Two flags control auto-creation behavior:
+#   - sync.autoSync (boolean): Global auto-sync toggle. When true, enables ALL auto-sync
+#     operations including issue creation, progress sync, and status updates.
+#   - hooks.post_increment_planning.auto_create_github_issue (boolean): Fine-grained toggle
+#     for ONLY the auto-create-issue behavior in post-increment-planning hooks.
+#
+# Precedence: auto_create_github_issue (specific) > autoSync (global)
+# If BOTH are set, auto_create_github_issue takes precedence for issue creation.
+# If ONLY autoSync is true, issue creation is enabled (as part of the global auto-sync).
 AUTO_SYNC=$(jq -r '.sync.autoSync // false' "$CONFIG_PATH" 2>/dev/null)
 AUTO_CREATE=$(jq -r '.hooks.post_increment_planning.auto_create_github_issue // false' "$CONFIG_PATH" 2>/dev/null)
+
+# Warn if only one flag is set (likely misconfiguration)
+if [[ "$AUTO_SYNC" == "true" ]] && [[ "$AUTO_CREATE" == "false" ]]; then
+  log "Warning: sync.autoSync=true but auto_create_github_issue=false. Issue creation enabled via autoSync. Set auto_create_github_issue=true to be explicit."
+elif [[ "$AUTO_SYNC" == "false" ]] && [[ "$AUTO_CREATE" == "true" ]]; then
+  log "Warning: auto_create_github_issue=true but sync.autoSync=false. Only issue creation is enabled; other sync operations are disabled."
+fi
+
 if [[ "$AUTO_SYNC" != "true" ]] && [[ "$AUTO_CREATE" != "true" ]]; then
   log "Auto-sync disabled (autoSync=$AUTO_SYNC, auto_create=$AUTO_CREATE)"
   exit 0
 fi
 
+# Log which flag triggered activation
+if [[ "$AUTO_CREATE" == "true" ]]; then
+  log "Auto-create enabled via hooks.post_increment_planning.auto_create_github_issue"
+else
+  log "Auto-create enabled via sync.autoSync (global toggle)"
+fi
+
 # Get owner/repo
 OWNER=$(jq -r '.sync.github.owner // ""' "$CONFIG_PATH" 2>/dev/null)
 REPO=$(jq -r '.sync.github.repo // ""' "$CONFIG_PATH" 2>/dev/null)
@@ -115,7 +139,9 @@ fi
 if [[ "${SPECWEAVE_SKIP_DEBOUNCE:-0}" != "1" ]]; then
   DEBOUNCE_FILE="$STATE_DIR/.github-auto-create-pending-$INC_ID"
   if [[ -f "$DEBOUNCE_FILE" ]]; then
-    SIGNAL_AGE=$(($(date +%s) - $(stat -f "%m" "$DEBOUNCE_FILE" 2>/dev/null || echo 0)))
+    # POSIX-portable: use perl for mtime (works on macOS and Linux)
+    FILE_MTIME=$(perl -e 'print((stat($ARGV[0]))[9])' "$DEBOUNCE_FILE" 2>/dev/null || echo 0)
+    SIGNAL_AGE=$(($(date +%s) - FILE_MTIME))
     if (( SIGNAL_AGE < 30 )); then
       log "Debounce: signal age ${SIGNAL_AGE}s < 30s. Deferring."
       exit 0

package/plugins/specweave-github/hooks/post-task-completion.sh

@@ -80,10 +80,13 @@ for i in {1..15}; do
     break
   fi
 
-  # Check for stale lock
+  # Check for stale lock (POSIX-portable: works on macOS and Linux)
   if [[ -d "$LOCK_FILE" ]]; then
-    LOCK_AGE=$(($(date +%s) - $(stat -f "%m" "$LOCK_FILE" 2>/dev/null || echo 0)))
-    if (( LOCK_AGE > LOCK_TIMEOUT )); then
+    # Use find -mmin which works on both macOS and Linux
+    LOCK_TIMEOUT_MIN=$(( (LOCK_TIMEOUT + 59) / 60 ))  # Convert seconds to minutes (ceil)
+    [[ $LOCK_TIMEOUT_MIN -lt 1 ]] && LOCK_TIMEOUT_MIN=1
+    STALE=$(find "$LOCK_FILE" -maxdepth 0 -mmin +${LOCK_TIMEOUT_MIN} 2>/dev/null)
+    if [[ -n "$STALE" ]]; then
       rmdir "$LOCK_FILE" 2>/dev/null || true
       continue
     fi

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

@@ -27,9 +27,10 @@ async function syncSpecWithEnhancedContent(options) {
       console.log(`\u{1F517} Found ${mapping.increments.length} related increments`);
       console.log(`\u{1F4CB} Mapped ${Object.keys(mapping.userStoryMappings).length} user stories to tasks`);
     }
-    const taskMapping = buildTaskMapping(mapping.increments, owner, repo);
+    const defaultBranch = (owner && repo) ? await getDefaultBranch(owner, repo) : null;
+    const taskMapping = buildTaskMapping(mapping.increments, owner, repo, defaultBranch);
     const architectureDocs = await findArchitectureDocs(rootDir, specId);
-    const sourceLinks = buildSourceLinks(mapping.increments[0]?.id, owner, repo);
+    const sourceLinks = buildSourceLinks(mapping.increments[0]?.id, owner, repo, defaultBranch);
     const enhancedSpec = {
       ...baseSpec,
       summary: baseSpec.description,
@@ -153,7 +154,33 @@ async function findSpecWeaveRoot(specPath) {
     }
   }
 }
-function buildTaskMapping(increments, owner, repo) {
+// Cache for default branch per repo per session
+const defaultBranchCache = new Map();
+
+async function getDefaultBranch(owner, repo) {
+  const cacheKey = `${owner}/${repo}`;
+  if (defaultBranchCache.has(cacheKey)) {
+    return defaultBranchCache.get(cacheKey);
+  }
+  try {
+    const { execFileSync } = await import("child_process");
+    const result = execFileSync("gh", [
+      "repo", "view", `${owner}/${repo}`,
+      "--json", "defaultBranchRef",
+      "--jq", ".defaultBranchRef.name"
+    ], { encoding: "utf-8", timeout: 10000 }).trim();
+    if (result) {
+      defaultBranchCache.set(cacheKey, result);
+      return result;
+    }
+  } catch {
+    // On failure, return null — callers omit branch segment
+  }
+  defaultBranchCache.set(cacheKey, null);
+  return null;
+}
+
+function buildTaskMapping(increments, owner, repo, defaultBranch) {
   if (increments.length === 0) return void 0;
   const firstIncrement = increments[0];
   const tasks = firstIncrement.tasks.map((task) => ({
@@ -162,10 +189,11 @@ function buildTaskMapping(increments, owner, repo) {
     userStories: task.userStories,
     githubIssue: task.githubIssue
   }));
+  const branchSegment = defaultBranch ? `/blob/${defaultBranch}` : "";
   return {
     incrementId: firstIncrement.id,
     tasks,
-    tasksUrl: `https://github.com/${owner}/${repo}/blob/develop/.specweave/increments/${firstIncrement.id}/tasks.md`
+    tasksUrl: `https://github.com/${owner}/${repo}${branchSegment}/.specweave/increments/${firstIncrement.id}/tasks.md`
   };
 }
 async function findArchitectureDocs(rootDir, specId) {
@@ -198,9 +226,10 @@ async function findArchitectureDocs(rootDir, specId) {
   }
   return docs;
 }
-function buildSourceLinks(incrementId, owner, repo) {
+function buildSourceLinks(incrementId, owner, repo, defaultBranch) {
   if (!incrementId) return void 0;
-  const baseUrl = `https://github.com/${owner}/${repo}/blob/develop/.specweave`;
+  const branchSegment = defaultBranch ? `/blob/${defaultBranch}` : "";
+  const baseUrl = `https://github.com/${owner}/${repo}${branchSegment}/.specweave`;
   return {
     spec: `${baseUrl}/docs/internal/specs/default/spec-${incrementId.replace(/^\d+-/, "")}.md`,
     plan: `${baseUrl}/increments/${incrementId}/plan.md`,

package/plugins/specweave-github/lib/github-board-resolver.js

@@ -15,9 +15,9 @@ async function fetchBoardsForRepo(owner, repo) {
       "-H",
       "Accept: application/vnd.github+json"
     ], { env: getGhEnv() });
-    if (result.status !== 0) {
+    if (result.exitCode !== 0) {
       console.error(`\u274C Failed to fetch boards for ${owner}/${repo}:`, result.stderr);
-      throw new Error(`GitHub API error: ${result.status} ${result.stderr}`);
+      throw new Error(`GitHub API error (exit code ${result.exitCode}): ${result.stderr}`);
     }
     const boards = result.stdout.trim().split("\n").filter((line) => line.trim()).map((line) => JSON.parse(line));
     console.log(`\u2705 Found ${boards.length} board(s) for repo ${owner}/${repo}`);
@@ -38,9 +38,9 @@ async function fetchBoardsForOrg(org) {
       "-H",
       "Accept: application/vnd.github+json"
     ], { env: getGhEnv() });
-    if (result.status !== 0) {
+    if (result.exitCode !== 0) {
       console.error(`\u274C Failed to fetch boards for org ${org}:`, result.stderr);
-      throw new Error(`GitHub API error: ${result.status} ${result.stderr}`);
+      throw new Error(`GitHub API error (exit code ${result.exitCode}): ${result.stderr}`);
     }
     const boards = result.stdout.trim().split("\n").filter((line) => line.trim()).map((line) => JSON.parse(line));
     console.log(`\u2705 Found ${boards.length} board(s) for org ${org}`);

package/plugins/specweave-github/lib/github-board-resolver.ts

@@ -54,9 +54,9 @@ export async function fetchBoardsForRepo(
       'Accept: application/vnd.github+json',
     ], { env: getGhEnv() });
 
-    if (result.status !== 0) {
+    if (result.exitCode !== 0) {
       console.error(`❌ Failed to fetch boards for ${owner}/${repo}:`, result.stderr);
-      throw new Error(`GitHub API error: ${result.status} ${result.stderr}`);
+      throw new Error(`GitHub API error (exit code ${result.exitCode}): ${result.stderr}`);
     }
 
     // Parse JSONL (one JSON object per line)
@@ -98,9 +98,9 @@ export async function fetchBoardsForOrg(
       'Accept: application/vnd.github+json',
     ], { env: getGhEnv() });
 
-    if (result.status !== 0) {
+    if (result.exitCode !== 0) {
       console.error(`❌ Failed to fetch boards for org ${org}:`, result.stderr);
-      throw new Error(`GitHub API error: ${result.status} ${result.stderr}`);
+      throw new Error(`GitHub API error (exit code ${result.exitCode}): ${result.stderr}`);
     }
 
     // Parse JSONL

package/plugins/specweave-github/lib/github-client-v2.js

@@ -103,13 +103,14 @@ class GitHubClientV2 {
   /**
    * Create or get existing milestone
    */
-  async createOrGetMilestone(title, description, daysFromNow = 2) {
+  async createOrGetMilestone(title, description, daysFromNow) {
+    const dueDays = daysFromNow ?? 2;
     const existing = await this.getMilestoneByTitle(title);
     if (existing) {
       return existing;
     }
     const dueDate = /* @__PURE__ */ new Date();
-    dueDate.setDate(dueDate.getDate() + daysFromNow);
+    dueDate.setDate(dueDate.getDate() + dueDays);
     const dueDateISO = dueDate.toISOString();
     const args = [
       "api",
@@ -136,7 +137,7 @@ class GitHubClientV2 {
   async getMilestoneByTitle(title) {
     const result = await execFileNoThrow("gh", [
       "api",
-      `repos/${this.fullRepo}/milestones`,
+      `repos/${this.fullRepo}/milestones?per_page=100&state=all`,
       "--jq",
       `.[] | select(.title=="${title}") | {number: .number, title: .title, description: .description, state: .state}`
     ], { env: this.getGhEnv() });
@@ -477,10 +478,9 @@ ${body}`;
   async getLastComment(issueNumber) {
     const result = await execFileNoThrow("gh", [
       "api",
-      `repos/${this.fullRepo}/issues/${issueNumber}/comments`,
+      `repos/${this.fullRepo}/issues/${issueNumber}/comments?sort=created&direction=desc&per_page=1`,
       "--jq",
-      ".[-1] | {body: .body, author: .user.login}"
-      // Get last comment only
+      ".[0] | {body: .body, author: .user.login}"
     ], { env: this.getGhEnv() });
     if (result.exitCode !== 0) {
       return null;

package/plugins/specweave-github/lib/github-client-v2.ts

@@ -148,17 +148,20 @@ export class GitHubClientV2 {
   async createOrGetMilestone(
     title: string,
     description?: string,
-    daysFromNow: number = 2
+    daysFromNow?: number
   ): Promise<GitHubMilestone> {
+    // Use configured value or default to 2 days
+    const dueDays = daysFromNow ?? 2;
+
     // Check if milestone already exists
     const existing = await this.getMilestoneByTitle(title);
     if (existing) {
       return existing;
     }
 
-    // Calculate due date
+    // Calculate due date from creation timestamp
     const dueDate = new Date();
-    dueDate.setDate(dueDate.getDate() + daysFromNow);
+    dueDate.setDate(dueDate.getDate() + dueDays);
     const dueDateISO = dueDate.toISOString();
 
     // Build API request
@@ -194,7 +197,7 @@ export class GitHubClientV2 {
   ): Promise<GitHubMilestone | null> {
     const result = await execFileNoThrow('gh', [
       'api',
-      `repos/${this.fullRepo}/milestones`,
+      `repos/${this.fullRepo}/milestones?per_page=100&state=all`,
       '--jq',
       `.[] | select(.title=="${title}") | {number: .number, title: .title, description: .description, state: .state}`,
     ], { env: this.getGhEnv() });
@@ -607,12 +610,13 @@ export class GitHubClientV2 {
    * Returns the most recent comment body, or null if no comments exist
    */
   async getLastComment(issueNumber: number): Promise<{body: string; author: string} | null> {
-    // Get all comments (sorted by creation date, newest last)
+    // Query the last comment directly using per_page=1 + page from last page
+    // sort=created&direction=desc gives newest first, per_page=1 returns just one
     const result = await execFileNoThrow('gh', [
       'api',
-      `repos/${this.fullRepo}/issues/${issueNumber}/comments`,
+      `repos/${this.fullRepo}/issues/${issueNumber}/comments?sort=created&direction=desc&per_page=1`,
       '--jq',
-      '.[-1] | {body: .body, author: .user.login}',  // Get last comment only
+      '.[0] | {body: .body, author: .user.login}',
     ], { env: this.getGhEnv() });
 
     if (result.exitCode !== 0) {

package/plugins/specweave-github/lib/github-client.js

@@ -40,15 +40,16 @@ class GitHubClient {
    *
    * @param title Milestone title
    * @param description Milestone description
-   * @param daysFromNow Days until milestone due (default: 2 days - SpecWeave AI velocity)
+   * @param daysFromNow Days until milestone due (configurable via github.milestoneDueDays, default: 2)
    */
-  async createOrGetMilestone(title, description, daysFromNow = 2) {
+  async createOrGetMilestone(title, description, daysFromNow) {
+    const dueDays = daysFromNow ?? 2;
     const existing = await this.getMilestoneByTitle(title);
     if (existing) {
       return existing;
     }
     const dueDate = /* @__PURE__ */ new Date();
-    dueDate.setDate(dueDate.getDate() + daysFromNow);
+    dueDate.setDate(dueDate.getDate() + dueDays);
     const dueDateISO = dueDate.toISOString();
     const cmd = `gh api repos/${this.repo}/milestones -f title="${title}" ${description ? `-f description="${description}"` : ""} -f due_on="${dueDateISO}" --jq '{number: .number, title: .title, description: .description, state: .state, due_on: .due_on}'`;
     try {
@@ -63,7 +64,7 @@ class GitHubClient {
    */
   async getMilestoneByTitle(title) {
     try {
-      const cmd = `gh api repos/${this.repo}/milestones --jq '.[] | select(.title=="${title}") | {number: .number, title: .title, description: .description, state: .state}'`;
+      const cmd = `gh api "repos/${this.repo}/milestones?per_page=100&state=all" --jq '.[] | select(.title=="${title}") | {number: .number, title: .title, description: .description, state: .state}'`;
       const output = execSync(cmd, { encoding: "utf-8" }).trim();
       return output ? JSON.parse(output) : null;
     } catch {

package/plugins/specweave-github/lib/github-client.ts

@@ -54,22 +54,24 @@ export class GitHubClient {
    *
    * @param title Milestone title
    * @param description Milestone description
-   * @param daysFromNow Days until milestone due (default: 2 days - SpecWeave AI velocity)
+   * @param daysFromNow Days until milestone due (configurable via github.milestoneDueDays, default: 2)
    */
   async createOrGetMilestone(
     title: string,
     description?: string,
-    daysFromNow: number = 2
+    daysFromNow?: number
   ): Promise<GitHubMilestone> {
+    // Use configured value or default to 2 days
+    const dueDays = daysFromNow ?? 2;
     // Check if milestone already exists
     const existing = await this.getMilestoneByTitle(title);
     if (existing) {
       return existing;
     }
 
-    // Calculate due date (SpecWeave default: 1-2 days with AI assistance)
+    // Calculate due date from creation timestamp
     const dueDate = new Date();
-    dueDate.setDate(dueDate.getDate() + daysFromNow);
+    dueDate.setDate(dueDate.getDate() + dueDays);
     const dueDateISO = dueDate.toISOString();
 
     // Create new milestone with due date
@@ -88,7 +90,7 @@ export class GitHubClient {
    */
   private async getMilestoneByTitle(title: string): Promise<GitHubMilestone | null> {
     try {
-      const cmd = `gh api repos/${this.repo}/milestones --jq '.[] | select(.title=="${title}") | {number: .number, title: .title, description: .description, state: .state}'`;
+      const cmd = `gh api "repos/${this.repo}/milestones?per_page=100&state=all" --jq '.[] | select(.title=="${title}") | {number: .number, title: .title, description: .description, state: .state}'`;
       const output = execSync(cmd, { encoding: 'utf-8' }).trim();
       return output ? JSON.parse(output) : null;
     } catch {