forge-workflow 0.0.9 → 0.0.10

package/lib/workflow/enforce-stage.js

@@ -10,6 +10,8 @@ const {
 } = require('./state');
 const { loadState, WORKFLOW_STATE_FILENAME } = require('./state-manager');
 
+const STATELESS_ENTRY_STAGES = new Set(['plan', 'dev', 'validate', 'verify']);
+
 function getOverrideInput(flags = {}) {
   if (Object.hasOwn(flags, 'overrideStage')) {
     return flags.overrideStage;
@@ -131,7 +133,7 @@ async function enforceStageEntry({ commandName, args = [], flags = {}, projectRo
   const stateInput = resolveWorkflowStateInput(workflowState, flags, args, projectRoot);
   const currentState = readWorkflowStateInput(stateInput);
   if (!currentState) {
-    if (stageId === 'plan') {
+    if (STATELESS_ENTRY_STAGES.has(stageId)) {
       return { allowed: true, stage: stageId, workflowState: null };
     }
 

package/lib/workflow/state-manager.js

@@ -9,14 +9,122 @@ const { getWorkflowPath, WORKFLOW_CLASSIFICATIONS, normalizeStageId } = require(
 
 const WORKFLOW_STATE_FILENAME = '.forge-state.json';
 
+function parseRelaxedWorkflowStatePayload(payload = '') {
+  const currentStage = String(payload).match(/currentStage:([a-z]+)/)?.[1];
+  if (!currentStage) {
+    return null;
+  }
+
+  const completedStages = (String(payload).match(/completedStages:\[([^\]]*)\]/)?.[1] || '')
+    .split(',')
+    .map(stage => stage.trim())
+    .filter(Boolean);
+  const skippedStages = (String(payload).match(/skippedStages:\[([^\]]*)\]/)?.[1] || '')
+    .split(',')
+    .map(stage => stage.trim())
+    .filter(Boolean);
+  const classification = String(payload).match(/classification:([a-z]+)/)?.[1] || 'standard';
+  const reason = String(payload).match(/reason:([^,}\]]+)/)?.[1]?.trim() || 'legacy-comment';
+
+  return readWorkflowState(JSON.stringify({
+    currentStage,
+    completedStages,
+    skippedStages,
+    workflowDecisions: {
+      classification,
+      reason,
+      userOverride: false,
+      overrides: [],
+    },
+    parallelTracks: [],
+  }));
+}
+
 function extractWorkflowStateFromComments(comments = '') {
-  const matches = String(comments).match(/^WorkflowState:\s*(\{.*\})$/gm);
+  const matches = String(comments)
+    .split(/\r?\n/)
+    .filter(line => line.startsWith('WorkflowState:'));
   if (!matches || matches.length === 0) {
     return null;
   }
 
-  const latest = matches.at(-1).replace(/^WorkflowState:\s*/, '');
-  return readWorkflowState(latest);
+  for (const match of [...matches].reverse()) {
+    const payload = match.replace(/^WorkflowState:\s*/, '');
+    try {
+      return readWorkflowState(payload);
+    } catch (_strictError) {
+      try {
+        const relaxedState = parseRelaxedWorkflowStatePayload(payload);
+        if (relaxedState) {
+          return relaxedState;
+        }
+      } catch (_relaxedError) {
+        // Continue to any earlier payloads.
+      }
+    }
+  }
+
+  return null;
+}
+
+function extractWorkflowStateFromIssue(issue = {}) {
+  if (!issue || typeof issue !== 'object') {
+    return null;
+  }
+
+  if (typeof issue.comments === 'string') {
+    return extractWorkflowStateFromComments(issue.comments);
+  }
+
+  if (!Array.isArray(issue.comments) || issue.comments.length === 0) {
+    return null;
+  }
+
+  const mergedComments = issue.comments
+    .map(comment => {
+      if (typeof comment === 'string') {
+        return comment;
+      }
+
+      return comment && typeof comment === 'object' ? comment.text || '' : '';
+    })
+    .filter(Boolean)
+    .join('\n');
+
+  if (!mergedComments) {
+    return null;
+  }
+
+  return extractWorkflowStateFromComments(mergedComments);
+}
+
+function readBeadsIssue(issueId, options = {}) {
+  if (!issueId) {
+    return null;
+  }
+
+  try {
+    const { projectRoot, ...execOptions } = options;
+    const raw = secureExecFileSync('bd', ['show', issueId, '--json'], {
+      encoding: 'utf8',
+      cwd: projectRoot,
+      stdio: ['pipe', 'pipe', 'pipe'],
+      ...execOptions,
+    }).trim();
+
+    if (!raw) {
+      return null;
+    }
+
+    const parsed = JSON.parse(raw);
+    if (Array.isArray(parsed)) {
+      return parsed[0] || null;
+    }
+
+    return parsed;
+  } catch (_error) {
+    return null;
+  }
 }
 
 function readWorkflowStateFromBeads(issueId, options = {}) {
@@ -24,8 +132,29 @@ function readWorkflowStateFromBeads(issueId, options = {}) {
     return null;
   }
 
-  const comments = options.comments || secureExecFileSync('bd', ['comments', 'list', issueId], {
+  if (options.issue) {
+    const state = extractWorkflowStateFromIssue(options.issue);
+    if (state) {
+      return state;
+    }
+  }
+
+  let comments = options.comments;
+  if (!comments) {
+    try {
+      const issue = readBeadsIssue(issueId, { projectRoot: options.projectRoot });
+      const state = extractWorkflowStateFromIssue(issue);
+      if (state) {
+        return state;
+      }
+    } catch (_error) {
+      // Fall through to comments-list fallback.
+    }
+  }
+
+  comments = comments || secureExecFileSync('bd', ['comments', 'list', issueId], {
     encoding: 'utf8',
+    cwd: options.projectRoot,
     stdio: ['pipe', 'pipe', 'pipe'],
   }).trim();
 
@@ -36,7 +165,7 @@ function readWorkflowStateFromBeads(issueId, options = {}) {
   return extractWorkflowStateFromComments(comments);
 }
 
-function loadStateFromBeads(options) {
+function loadStateFromBeads(options, projectRoot) {
   if (options.comments) {
     const state = extractWorkflowStateFromComments(options.comments);
     if (state) {
@@ -44,8 +173,19 @@ function loadStateFromBeads(options) {
     }
   }
 
+  if (options.issue) {
+    const state = extractWorkflowStateFromIssue(options.issue);
+    if (state) {
+      return { state, source: 'beads' };
+    }
+  }
+
   if (options.issueId) {
-    const state = readWorkflowStateFromBeads(options.issueId, { comments: options.comments });
+    const state = readWorkflowStateFromBeads(options.issueId, {
+      comments: options.comments,
+      issue: options.issue,
+      projectRoot,
+    });
     if (state) {
       return { state, source: 'beads' };
     }
@@ -56,10 +196,24 @@ function loadStateFromBeads(options) {
 
 function loadState(projectRoot, options = {}) {
   if (!projectRoot) {
-    const beadsResult = loadStateFromBeads(options);
+    const beadsResult = loadStateFromBeads(options, null);
     return beadsResult || { state: null, source: null };
   }
 
+  if (options.preferBeads && (options.comments || options.issue || options.issueId)) {
+    try {
+      const beadsResult = loadStateFromBeads(options, projectRoot);
+      if (beadsResult) {
+        return beadsResult;
+      }
+    } catch (error) {
+      if (typeof options.onBeadsError === 'function') {
+        options.onBeadsError(error);
+      }
+      // Fall through to the on-disk state file when Beads cannot be read.
+    }
+  }
+
   const statePath = path.join(projectRoot, WORKFLOW_STATE_FILENAME);
   if (fs.existsSync(statePath)) {
     try {
@@ -70,7 +224,7 @@ function loadState(projectRoot, options = {}) {
     }
   }
 
-  const beadsResult = loadStateFromBeads(options);
+  const beadsResult = loadStateFromBeads(options, projectRoot);
   if (beadsResult) {
     return beadsResult;
   }
@@ -185,8 +339,10 @@ function transitionStage(projectRoot, toStage, options = {}) {
 module.exports = {
   WORKFLOW_STATE_FILENAME,
   extractWorkflowStateFromComments,
+  extractWorkflowStateFromIssue,
   initializeState,
   loadState,
+  readBeadsIssue,
   readWorkflowStateFromBeads,
   saveState,
   transitionStage,

package/package.json

@@ -1,9 +1,10 @@
 {
   "name": "forge-workflow",
-  "version": "0.0.9",
+  "version": "0.0.10",
   "description": "7-stage TDD workflow for ALL AI coding agents (Claude, Cursor, Cline, OpenCode, Copilot, Kilo Code, Roo Code, Codex)",
   "bin": {
     "forge": "bin/forge.js",
+    "forge-workflow": "bin/forge.js",
     "forge-preflight": "bin/forge-preflight.js"
   },
   "workspaces": [
@@ -14,16 +15,23 @@
     "test:setup": "bash test-env/automation/setup-fixtures.sh",
     "test:all": "bun run test:setup && bun test",
     "test:coverage": "c8 --check-coverage bun test",
+    "test:ci:shard": "node scripts/test-ci-shard.js",
     "typecheck": "echo 'No TypeScript in project - skipping type check'",
     "lint": "eslint . --max-warnings 0",
-    "check": "bash scripts/validate.sh",
+    "check": "node scripts/validate.js",
     "prepare": "lefthook install || echo 'Note: lefthook not installed. Git hooks disabled. Run: bun add -d lefthook'",
     "setup": "node ./bin/forge.js setup",
     "help": "node ./bin/forge.js --help",
     "validate:yaml": "find . \\( -name '*.yml' -o -name '*.yaml' \\) | grep -v node_modules | while IFS= read -r file; do echo \"Checking $file...\" && bun x js-yaml \"$file\" > /dev/null || exit 1; done",
     "test:mutation": "npx stryker run",
-    "test:benchmark": "node scripts/benchmark.js",
-    "test:dashboard": "node scripts/test-dashboard.js"
+    "test:benchmark": "node scripts/benchmark.js --output test-results/benchmark-results.json --profile-dir test-results/benchmark-profiles",
+    "test:benchmark:full": "node scripts/benchmark.js --output test-results/benchmark-full-results.json --profile-dir test-results/benchmark-full-profiles --group full-suite --group validate",
+    "test:full:parallel": "node scripts/test-full-suite.js",
+    "test:dashboard": "node scripts/test-dashboard.js",
+    "test:profile": "node scripts/test-profile.js --input-dir test-results --output test-results/test-profile.json --label local",
+    "test:randomized": "bun test --randomize",
+    "test:rerun": "bun test --rerun-each 2",
+    "agentic:sync": "node scripts/sync-agentic-workflow.js"
   },
   "peerDependencies": {
     "lefthook": "^1.0.0"

package/scripts/beads-context.sh

@@ -41,11 +41,124 @@ die() {
   exit 1
 }
 
+# Resolve a Windows bd.exe installation for bash-based helper flows.
+# This covers WSL/Git Bash cases where PowerShell can run bd.exe but bash PATH
+# does not include the Windows install directory.
+convert_windows_path() {
+  local raw="${1%$'\r'}"
+
+  if [[ -z "$raw" ]]; then
+    return 1
+  fi
+
+  if [[ ! "$raw" =~ ^[A-Za-z]:\\ ]]; then
+    printf '%s' "$raw"
+    return 0
+  fi
+
+  if command -v wslpath >/dev/null 2>&1; then
+    wslpath -u "$raw" 2>/dev/null && return 0
+  fi
+
+  if command -v cygpath >/dev/null 2>&1; then
+    cygpath -u "$raw" 2>/dev/null && return 0
+  fi
+
+  local drive rest
+  drive="$(printf '%s' "$raw" | cut -c1 | tr '[:upper:]' '[:lower:]')"
+  rest="${raw:2}"
+  rest="${rest//\\//}"
+  printf '/mnt/%s%s' "$drive" "$rest"
+}
+
+is_runnable_bd_candidate() {
+  local candidate="${1:-}"
+
+  if [[ -z "$candidate" || ! -f "$candidate" ]]; then
+    return 1
+  fi
+
+  [[ -x "$candidate" || "$candidate" == *.exe ]]
+}
+
+resolve_bd_cmd() {
+  local candidate=""
+  local converted=""
+
+  if [[ -n "${BD_CMD:-}" ]]; then
+    if [[ "${BD_CMD}" == *"/"* || "${BD_CMD}" == *"\\"* ]]; then
+      converted="$(convert_windows_path "$BD_CMD")"
+      if is_runnable_bd_candidate "$converted"; then
+        printf '%s' "$converted"
+        return 0
+      fi
+
+      is_runnable_bd_candidate "$BD_CMD" || return 1
+    fi
+    printf '%s' "$BD_CMD"
+    return 0
+  fi
+
+  if command -v bd >/dev/null 2>&1; then
+    printf '%s' "bd"
+    return 0
+  fi
+
+  if command -v bd.exe >/dev/null 2>&1; then
+    printf '%s' "bd.exe"
+    return 0
+  fi
+
+  for candidate in \
+    "$HOME/.local/bin/bd" \
+    "$HOME/.local/bin/bd.exe" \
+    "$HOME/.bun/bin/bd" \
+    "$HOME/.bun/bin/bd.exe"
+  do
+    if is_runnable_bd_candidate "$candidate"; then
+      printf '%s' "$candidate"
+      return 0
+    fi
+  done
+
+  if command -v where.exe >/dev/null 2>&1; then
+    while IFS= read -r candidate; do
+      candidate="${candidate%$'\r'}"
+      [[ -z "$candidate" ]] && continue
+
+      converted="$(convert_windows_path "$candidate")"
+      if is_runnable_bd_candidate "$converted"; then
+        printf '%s' "$converted"
+        return 0
+      fi
+
+      if is_runnable_bd_candidate "$candidate"; then
+        printf '%s' "$candidate"
+        return 0
+      fi
+    done < <(where.exe bd 2>/dev/null || true)
+  fi
+
+  return 1
+}
+
+BD=""
+
+get_bd_cmd() {
+  if [[ -z "$BD" ]]; then
+    BD="$(resolve_bd_cmd)" || die "bd is required but not found"
+  fi
+
+  printf '%s' "$BD"
+}
+
 # Run bd update and check for errors in both exit code and output.
 # bd update exits 0 even for non-existent issues, so we check stdout too.
 bd_update() {
+  local bd_cmd
+  bd_cmd="$(get_bd_cmd)"
   local output
-  output="$(bd update "$@" 2>&1)"
+  output="$("$bd_cmd" update "$@" 2>&1)"
   local rc=$?
 
   if [[ $rc -ne 0 ]]; then
@@ -66,8 +179,10 @@ bd_update() {
 
 # Run bd comments add and check for errors similarly.
 bd_comment() {
+  local bd_cmd
+  bd_cmd="$(get_bd_cmd)"
   local output
-  output="$(bd comments add "$@" 2>&1)"
+  output="$("$bd_cmd" comments add "$@" 2>&1)"
   local rc=$?
 
   if [[ $rc -ne 0 ]]; then
@@ -172,10 +287,12 @@ cmd_parse_progress() {
   fi
 
   local issue_id="$1"
+  local bd_cmd
+  bd_cmd="$(get_bd_cmd)"
 
   # Get the issue JSON — detect non-existent issues
   local json
-  json="$(bd show "$issue_id" --json 2>&1)" || die "Failed to show issue ${issue_id}"
+  json="$("$bd_cmd" show "$issue_id" --json 2>&1)" || die "Failed to show issue ${issue_id}"
 
   # bd show may exit 0 but print an error for non-existent issues
   # Match specific bd error patterns, not the word "error" in data fields
@@ -320,10 +437,12 @@ cmd_validate() {
 
   local issue_id="$1"
   local warnings=0
+  local bd_cmd
+  bd_cmd="$(get_bd_cmd)"
 
   # Get issue JSON
   local json
-  json="$(bd show "$issue_id" --json 2>&1)" || {
+  json="$("$bd_cmd" show "$issue_id" --json 2>&1)" || {
     echo "Error: Failed to retrieve issue ${issue_id}" >&2
     exit 1
   }
@@ -357,7 +476,7 @@ cmd_validate() {
 
   # Get comments to check for stage transitions
   local comments
-  comments="$(bd comments list "$issue_id" 2>/dev/null || true)"
+  comments="$("$bd_cmd" comments "$issue_id" 2>/dev/null || true)"
 
   # Check 2: At least one stage transition exists
   local has_transition=false

package/scripts/beads-context.test.js

@@ -22,16 +22,24 @@ function isBdAvailable() {
 	}
 }
 
+function shouldRunBeadsIntegration() {
+	return process.env.RUN_BEADS_INTEGRATION === '1' && isBdAvailable();
+}
+
 /**
  * Helper: run the beads-context.sh script with given args.
  * Returns { exitCode, stdout, stderr }.
  */
 async function run(...args) {
+	return runWithEnv(args);
+}
+
+async function runWithEnv(args, envOverrides = {}) {
 	const proc = Bun.spawn([resolveBashCommand(), SCRIPT_PATH, ...args], {
 		cwd: WORKTREE_ROOT,
 		stdout: 'pipe',
 		stderr: 'pipe',
-		env: { ...process.env },
+		env: { ...process.env, ...envOverrides },
 	});
 	const stdout = await new Response(proc.stdout).text();
 	const stderr = await new Response(proc.stderr).text();
@@ -55,7 +63,16 @@ async function bd(...args) {
 	return { exitCode, stdout, stderr };
 }
 
-describe.skipIf(!isBdAvailable())('scripts/beads-context.sh', () => {
+describe('scripts/beads-context.sh command resolution', () => {
+	test('script includes Windows-aware bd resolution fallback', () => {
+		const content = fs.readFileSync(SCRIPT_PATH, 'utf8');
+		expect(content).toContain('resolve_bd_cmd');
+		expect(content).toContain('where.exe');
+		expect(content).toContain('$HOME/.local/bin/bd.exe');
+	});
+});
+
+describe.skipIf(!shouldRunBeadsIntegration())('scripts/beads-context.sh', () => {
 	let testIssueId;
 
 	// Create a temporary test issue for isolation
@@ -470,7 +487,7 @@ describe('/dev command integration with beads-context.sh', () => {
 	});
 });
 
-describe('/status command integration with smart-status.sh', () => {
+describe('/status command integration with smart-status', () => {
 	const STATUS_MD_PATH = path.join(
 		__dirname,
 		'..',
@@ -485,7 +502,7 @@ describe('/status command integration with smart-status.sh', () => {
 	});
 
 	test('status.md should reference smart-status.sh as the primary status command', () => {
-		expect(statusContent).toContain('smart-status.sh');
+		expect(statusContent).toContain('bash scripts/smart-status.sh');
 	});
 
 	test('status.md should include hint to use bd show for full context', () => {

package/scripts/beads-migrate-to-dolt.sh

@@ -0,0 +1,7 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+NODE_CMD="${NODE_CMD:-node}"
+
+exec "$NODE_CMD" "$SCRIPT_DIR/lib/beads-migrate-to-dolt.mjs" "$@"