@curdx/flow 7.0.2 → 7.1.1

package/CHANGELOG.md

@@ -2,6 +2,33 @@
 
 All notable changes to `@curdx/flow` are documented here. Format follows [Keep a Changelog](https://keepachangelog.com/) and the project follows [Semantic Versioning](https://semver.org/).
 
+## 7.1.1 — 2026-05-04
+
+### Fixed
+
+- **Windows Chrome detection in the installer's `chrome-devtools-mcp` pre-req check.** `src/registry/plugins/chrome-devtools-mcp.ts`'s `checkChrome()` previously ran `test -x /Applications/...` plus `which google-chrome` / `which chromium` on every platform — none of which work on Windows (`test` is a Unix builtin, `which` isn't on `cmd`/PowerShell PATH by default, and the macOS app-bundle path doesn't exist). Result: the installer rejected Windows machines with Chrome installed, reporting "需要本机已安装 Chrome / Requires Chrome installed locally". Detection is now per-platform, mirroring `GoogleChrome/chrome-launcher`'s `chrome-finder` strategy: on `win32`, scan `LOCALAPPDATA` / `PROGRAMFILES` / `PROGRAMFILES(X86)` for `Google\Chrome\Application\chrome.exe` and `Google\Chrome SxS\Application\chrome.exe` (Canary) via `fs.existsSync`; on `darwin`, check the canonical app-bundle path; on Linux, the existing `which` lookup. A `CHROME_PATH` env var now overrides on all platforms (matches chrome-launcher / Lighthouse convention).
+
+
+
+### Added
+
+- **`completed: boolean` and `completedAt: string` fields in `.curdx-state.json` schema.** New optional fields on the spec state file (`schemas/spec.schema.json`). `completed === true` flips the spec into "done, retained for audit" mode; `completedAt` carries an ISO 8601 timestamp (`new Date().toISOString()`, includes milliseconds, e.g. `2026-05-04T20:17:00.123Z`). Legacy v7.0.x state files with `completed === undefined` continue to be treated as in-progress (backwards-compat — see Migration).
+- **`merge-state` `$unset` operator.** `plugins/curdx-flow/hooks/scripts/lib/merge-state.mjs` now accepts a JSON patch with `"$unset": ["key1", "key2"]` to remove specific fields atomically. Used by `/curdx-flow:refactor` to clear `completedAt` when a spec is intentionally re-opened. Patches without `$unset` behave exactly as before (zero-behavior-change for existing callers).
+- **`ensure-gitignore` wire-in to `/curdx-flow:start`.** The `ensure-gitignore` lib utility (already shipped in v7.0.0 but not invoked) is now called from `commands/start.md` so first-spec creation guarantees `**/.progress.md` is gitignored. Closes the "working tree permanently dirty" footgun that motivated this spec.
+- **Shared `CurdxState` interface in `src/hooks/_shared/types.ts`.** Single source of truth for the `.curdx-state.json` shape across all 4 reader hooks (`load-spec-context`, `quick-mode-guard`, `stop-watcher`, `update-spec-index`). Type-only export — esbuild erases at bundle time, zero runtime cost.
+
+### Changed
+
+- **`.curdx-state.json` is no longer deleted on `ALL_TASKS_COMPLETE`.** Coordinator and `commands/implement.md` Step 5 now write `{"completed":true,"completedAt":"<ISO>","awaitingApproval":false}` via `merge-state` instead of `rm -f .curdx-state.json`. The state file is **retained** as the structured source of truth for completed specs (audit trail: `discoveredSkills`, `granularity`, `commitSpec`, `quickMode` interview decisions all survive). Eliminates the test008-class regression where state-deletion left a permanently-dirty working tree, and lets `update-spec-index` short-circuit to `phase=completed` without falling back to fragile markdown reverse-parsing.
+- **5 reader hooks now use strict `state.completed === true` equality.** `stop-watcher.ts:601`, `load-spec-context.ts:147`, `update-spec-index.ts:278` (plus `quick-mode-guard.ts` type-import sync) check `state.completed === true` (never `if (state.completed)`). This is the backwards-compat contract: legacy v7.0.x state files with `completed === undefined` falsy-evaluate and behave identically to pre-7.1.0. Lint-enforced via grep gate (≥3 strict-equality occurrences, 0 truthy-checks).
+- **`update-spec-index` short-circuits to `phase=completed`** when `state.completed === true`, without invoking `inferPhaseFromFiles()`. The fallback path (markdown reverse-parse) remains as a second-tier safety net for human-deleted state files / third-party forks / pre-v7 residue.
+- **`/curdx-flow:refactor` resets via `merge-state $unset` instead of state file deletion.** When a completed spec is re-opened for refactor, `commands/refactor.md` now runs `merge-state .curdx-state.json '{"$unset":["completedAt"],"completed":false}'` rather than `rm -f`. Preserves audit history; matches the v7.1.0 retention model end-to-end.
+
+### Migration
+
+- Backwards-compatible by design — `completed === undefined` (legacy v7.0.x state files) is treated as in-progress, no manual migration required.
+- See [docs/MIGRATION-V7.md → v7.1.0](./docs/MIGRATION-V7.md#v710-state-retention--completion-marker) for the full upgrade note + a `jq`-style snippet to backfill `completed:true` on specs whose `.curdx-state.json` was deleted under v7.0.x (AC-8.3).
+
 ## 7.0.2 — 2026-05-04
 
 ### Fixed

package/dist/index.mjs

@@ -522,17 +522,38 @@ var claudeMem = {
 var claude_mem_default = claudeMem;
 
 // src/registry/plugins/chrome-devtools-mcp.ts
+import { existsSync } from "fs";
+import path2 from "path";
 var PLUGIN_ID3 = "chrome-devtools-mcp@chrome-devtools-plugins";
 var MARKETPLACE_NAME3 = "chrome-devtools-plugins";
 var MARKETPLACE_SOURCE3 = "ChromeDevTools/chrome-devtools-mcp";
 async function checkChrome() {
-  const macPath = "/Applications/Google Chrome.app/Contents/MacOS/Google Chrome";
-  const [stat, viaPath, viaPathChromium] = await Promise.all([
-    run("test", ["-x", macPath]),
+  if (process.env.CHROME_PATH && existsSync(process.env.CHROME_PATH)) return true;
+  if (process.platform === "win32") {
+    const suffixes = [
+      path2.join("Google", "Chrome SxS", "Application", "chrome.exe"),
+      path2.join("Google", "Chrome", "Application", "chrome.exe")
+    ];
+    const prefixes = [
+      process.env.LOCALAPPDATA,
+      process.env.PROGRAMFILES,
+      process.env["PROGRAMFILES(X86)"]
+    ].filter((p10) => Boolean(p10));
+    for (const prefix of prefixes) {
+      for (const suffix of suffixes) {
+        if (existsSync(path2.join(prefix, suffix))) return true;
+      }
+    }
+    return false;
+  }
+  if (process.platform === "darwin") {
+    return existsSync("/Applications/Google Chrome.app/Contents/MacOS/Google Chrome");
+  }
+  const [viaPath, viaPathChromium] = await Promise.all([
     run("which", ["google-chrome"]),
     run("which", ["chromium"])
   ]);
-  return stat.exitCode === 0 || viaPath.exitCode === 0 || viaPathChromium.exitCode === 0;
+  return viaPath.exitCode === 0 || viaPathChromium.exitCode === 0;
 }
 var chromeDevtoolsMcp = {
   id: "chrome-devtools-mcp",
@@ -713,14 +734,14 @@ function findPkg(id) {
 
 // src/runner/claudeMd.ts
 import { promises as fs2 } from "fs";
-import path2 from "path";
+import path3 from "path";
 import os2 from "os";
 import * as p3 from "@clack/prompts";
 var BEGIN_MARKER = "<!-- BEGIN @curdx/flow v1 -->";
 var END_MARKER = "<!-- END @curdx/flow v1 -->";
 var BLOCK_RE = /<!-- BEGIN @curdx\/flow v\d+[^>]*-->[\s\S]*?<!-- END @curdx\/flow v\d+ -->/;
 function claudeMdPath() {
-  return path2.join(os2.homedir(), ".claude", "CLAUDE.md");
+  return path3.join(os2.homedir(), ".claude", "CLAUDE.md");
 }
 function buildCombinationPatterns(ids) {
   const has = (k) => ids.has(k);
@@ -896,7 +917,7 @@ async function syncClaudeMd(opts) {
     if (next === existing) {
       return { status: "unchanged", path: file };
     }
-    await fs2.mkdir(path2.dirname(file), { recursive: true });
+    await fs2.mkdir(path3.dirname(file), { recursive: true });
     const tmp = `${file}.tmp.${process.pid}`;
     await fs2.writeFile(tmp, next, "utf8");
     await fs2.rename(tmp, file);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@curdx/flow",
-  "version": "7.0.2",
+  "version": "7.1.1",
   "description": "Interactive installer for Claude Code plugins and MCP servers",
   "type": "module",
   "bin": "./dist/index.mjs",