phasegate 0.118.0 → 0.120.0

package/CHANGELOG.md

@@ -7,6 +7,30 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.120.0] - 2026-05-07
+
+### Fixed
+
+- **WI-087 Phase B + WI-086 finding #1 統合対応: `phasegate init` のデフォルト値を実プロジェクト構成に追従させる** — 外部レポーター nakataj-mti / junpei-9898 が GitHub Issue [#3](https://github.com/junpei-9898/phasegate/issues/3) / [#2](https://github.com/junpei-9898/phasegate/issues/2) で報告した「デプロイされる `hook-config.json` のデフォルト値が単一パッケージ構成（`targetDirs:["src"]` / `formatter:"biome"`）固定で、モノレポ構成や biome 不在環境ではユーザーが明示的に書き換えない限り hook が silent no-op になる」問題を修正。
+  - **モノレポ workspace 自動検出** (`scripts/harness/setup/skill-deployer.ts:detectWorkspaceTargetDirs`): `pnpm-workspace.yaml` の `packages:` 配列 → `package.json.workspaces` (配列形式 / `{packages:[...]}` オブジェクト形式の両対応) → `lerna.json.packages` の優先順位で検出。検出した workspace glob (`pkg/*` 等) を実 FS 上で展開し、`<workspace>/src` ディレクトリが存在するもののみを `targetDirs` に採用。pnpm-workspace.yaml は依存追加を避けるため line-based の最小 parser で対応 (`packages:` ブロック検出 + `- 'pattern'` 行抽出のみ)。
+  - **formatter 自動検出** (`detectFormatter`): `package.json.devDependencies` (および `dependencies`) を読み、`@biomejs/biome` 存在 → `biome` + `["check","--write"]`、不在 + `prettier` 存在 → `eslint-prettier`、どちらも不在 → `null` (formatter フィールド省略で `format-typescript-hook.sh` の case 文 default フォールスルー、registry pull を回避)。
+  - **`deployHookScripts` 拡張**: `copyDirectory` 直前に既存 `.claude/scripts/hook-config.json` を捕捉し、(a) 既存あり → 元の内容を書き戻して **ユーザーカスタマイズ尊重**、(b) 既存なし → 検出結果を反映した新規 hook-config.json を生成。`DeployHooksResult` に `hookConfigGenerated` / `detectedTargetDirs` / `detectedFormatter` を追加し、`scripts/harness/main.ts` の init 出力に `✓ hook-config.json generated (targetDirs: ...; formatter: ...)` 行を追加。
+  - **`initHarnessConfig` テンプレートに `architecture: { preset: "clean" }` を追加**: 新規プロジェクトで生成される `phasegate.config.json` が schemaVersion = 'v3' と判定されるようにし、`init` 直後の v2 schema warning を解消（GitHub Issue #2 の "v2 schema warning が phasegate init 直後でも出る" を解消）。既存 v2 config を持つプロジェクトは `migrate-schema` CLI / 手動編集ルートを継続。
+  - **テスト追加**: `scripts/harness/__tests__/integration/setup/init-hook-config-detection.integration.test.ts` (14 ケース) — workspace 検出 (pnpm/npm/yarn 配列/yarn オブジェクト/lerna/未定義/src 不在/全 src 不在)、formatter 検出 (biome/prettier/null)、hook-config.json 生成 (既存尊重 / 新規生成)、schema v3 化 (architecture フィールド存在 / `phasegate:status` の stderr に v2 warning が出ないこと)。
+  - **互換性**: 既存 v2 config を読む load-resolved-config-use-case 側は変更なし。既存 `.claude/scripts/hook-config.json` をカスタマイズ済みのプロジェクトで `phasegate init` を再実行しても上書きされない (write-back ガード)。
+  - **スコープ外** (Phase C で対応予定): Quick Mode 通過時の stderr notice (finding #3) / Stop hook `--enforce` flag (finding #4) / `pre-tool-use` の責務範囲ドキュメント明文化 (WI-086)。
+
+## [0.119.0] - 2026-05-07
+
+### Fixed
+
+- **WI-087 Phase A: `phasegate init` がデプロイする hook スクリプトが macOS 標準 bash 3.2 で silent no-op になる問題を修正** — 外部レポーター nakataj-mti が GitHub Issue [#3](https://github.com/junpei-9898/phasegate/issues/3) で報告。`templates/.claude/scripts/format-typescript-hook.sh` および `analyze-errors-hook.sh` の `mapfile -t TARGET_DIRS < <(jq ...)` が bash 4+ builtin であり、macOS 標準 `/bin/bash` (3.2.57) では `mapfile: command not found` エラーで `TARGET_DIRS` 配列が空になり、後続の `${#TARGET_DIRS[@]} -eq 0` 判定で hook が exit 0 で抜けていた。結果としてユーザーには何も表示されず、format / lint hook が deliberately quiet と誤認される状態だった。
+  - `mapfile -t ARRAY < <(...)` を bash 3.2 互換の `while IFS= read -r line; do ARRAY+=("$line"); done < <(...)` idiom に置換（`format-typescript-hook.sh` 内 2 箇所、`analyze-errors-hook.sh` 内 1 箇所）。
+  - shebang は `#!/bin/bash` を維持（追加インストール不要、既存ユーザーへの影響なし）。
+  - 検証: macOS bash 3.2.57 で `targetDirs` を含む hook-config.json をロード後、対象ディレクトリ下のファイルに対して `format-typescript-hook.sh` が `Formatted: ...` を出力、`analyze-errors-hook.sh` が `{"decision": "approve", ...}` を返すことを確認。
+  - 関連: WI-086（GitHub Issue [#2](https://github.com/junpei-9898/phasegate/issues/2), reporter: junpei-9898）の "post-tool-use がセッション中に呼ばれても通知されない" 症状の主因も同じ `mapfile` であり、本修正で副次的に解消する。
+  - スコープ外（後続フェーズで対応予定）: `phasegate init` の monorepo 自動検出（WI-087 Phase B）、Quick Mode 通過時の stderr notice / Stop hook `--enforce` flag（WI-087 Phase C）、`phasegate init` の formatter 自動検出（WI-087 Phase B）。
+
 ## [0.118.0] - 2026-05-07
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "phasegate",
-  "version": "0.118.0",
+  "version": "0.120.0",
   "packageManager": "pnpm@10.30.1",
   "description": "Phasegate — AI-agnostic quality defense toolkit. Enforces structural integrity between design intent and code.",
   "license": "MIT",

package/scripts/harness/main.ts

@@ -469,7 +469,13 @@ async function main(): Promise<void> {
         const configResult = await initHarnessConfig(rootDir, projectName, phasePreset);
         const hooksResult = deployClaude
           ? await deployHookScripts(harnessRoot, rootDir)
-          : { scriptsDeployed: 0, settingsCreated: false };
+          : {
+              scriptsDeployed: 0,
+              settingsCreated: false,
+              hookConfigGenerated: false,
+              detectedTargetDirs: [] as string[],
+              detectedFormatter: null,
+            };
         const codexResult = deployCodex ? await deployCodexHooks(harnessRoot, rootDir) : null;
         const designDocsResult = await deployDesignDocs(harnessRoot, rootDir);
         const withHusky = hasFlag(args, "--with-husky");
@@ -486,6 +492,13 @@ async function main(): Promise<void> {
         if (hooksResult.scriptsDeployed > 0) {
           console.log(`✓ Hook scripts deployed to .claude/scripts/ (${hooksResult.scriptsDeployed} files)`);
         }
+        if (hooksResult.hookConfigGenerated) {
+          const dirsLabel = hooksResult.detectedTargetDirs.join(", ");
+          const formatterLabel = hooksResult.detectedFormatter ?? "none";
+          console.log(
+            `✓ hook-config.json generated (targetDirs: ${dirsLabel}; formatter: ${formatterLabel})`,
+          );
+        }
         if (hooksResult.settingsCreated) {
           console.log(`✓ .claude/settings.json created`);
         } else if (hooksResult.scriptsDeployed > 0) {

package/scripts/harness/setup/skill-deployer.ts

@@ -197,15 +197,159 @@ export async function deployAgentSkillLinks(
 export interface DeployHooksResult {
   scriptsDeployed: number;
   settingsCreated: boolean;
+  hookConfigGenerated: boolean;
+  detectedTargetDirs: string[];
+  detectedFormatter: string | null;
+}
+
+// pnpm-workspace.yaml の packages: 配列を line-based でパースする最小実装。
+// 依存追加を避けつつ pnpm の workspace 定義を扱う。
+function parsePnpmWorkspacePackages(yaml: string): string[] {
+  const lines = yaml.split(/\r?\n/);
+  const packages: string[] = [];
+  let inPackages = false;
+  let packagesIndent = -1;
+  for (const rawLine of lines) {
+    const line = rawLine.replace(/#.*$/, "").replace(/\s+$/, "");
+    if (line.length === 0) continue;
+    const indent = line.length - line.trimStart().length;
+    if (!inPackages) {
+      if (/^packages\s*:\s*$/.test(line)) {
+        inPackages = true;
+        packagesIndent = indent;
+      }
+      continue;
+    }
+    if (indent <= packagesIndent) {
+      // packages: ブロックを抜けた
+      inPackages = false;
+      continue;
+    }
+    const itemMatch = line.match(/^\s*-\s*['"]?([^'"#]+?)['"]?\s*$/);
+    if (itemMatch) {
+      packages.push(itemMatch[1].trim());
+    }
+  }
+  return packages;
+}
+
+// 単純な末尾 `*` glob (`pkg/*`) のみを展開する。`**` 等のネストパターンは対象外。
+async function expandWorkspaceGlobs(projectRoot: string, patterns: string[]): Promise<string[]> {
+  const expanded = new Set<string>();
+  for (const pattern of patterns) {
+    if (pattern.includes("**")) {
+      // 安全のため、再帰 glob は無視（無限ループ・性能を避ける）
+      continue;
+    }
+    if (pattern.endsWith("/*")) {
+      const baseDir = pattern.slice(0, -2);
+      try {
+        const entries = await fs.readdir(join(projectRoot, baseDir), { withFileTypes: true });
+        for (const entry of entries) {
+          if (entry.isDirectory() && !entry.name.startsWith(".")) {
+            expanded.add(`${baseDir}/${entry.name}`);
+          }
+        }
+      } catch {}
+    } else {
+      // glob なし: そのまま採用（ディレクトリ存在チェックは後段で実施）
+      expanded.add(pattern);
+    }
+  }
+  return [...expanded];
+}
+
+export async function detectWorkspaceTargetDirs(projectRoot: string): Promise<string[]> {
+  let patterns: string[] = [];
+
+  // 1. pnpm-workspace.yaml
+  try {
+    const yamlContent = await fs.readFile(join(projectRoot, "pnpm-workspace.yaml"), "utf-8");
+    patterns = parsePnpmWorkspacePackages(yamlContent);
+  } catch {}
+
+  // 2. package.json の workspaces (npm/yarn)
+  if (patterns.length === 0) {
+    try {
+      const pkgRaw = await fs.readFile(join(projectRoot, "package.json"), "utf-8");
+      const pkg = JSON.parse(pkgRaw) as { workspaces?: string[] | { packages?: string[] } };
+      const workspaces = pkg.workspaces;
+      if (Array.isArray(workspaces)) {
+        patterns = workspaces;
+      } else if (workspaces && Array.isArray(workspaces.packages)) {
+        patterns = workspaces.packages;
+      }
+    } catch {}
+  }
+
+  // 3. lerna.json
+  if (patterns.length === 0) {
+    try {
+      const lernaRaw = await fs.readFile(join(projectRoot, "lerna.json"), "utf-8");
+      const lerna = JSON.parse(lernaRaw) as { packages?: string[] };
+      if (Array.isArray(lerna.packages)) {
+        patterns = lerna.packages;
+      }
+    } catch {}
+  }
+
+  if (patterns.length === 0) {
+    return ["src"];
+  }
+
+  const workspaceDirs = await expandWorkspaceGlobs(projectRoot, patterns);
+  const targetDirs: string[] = [];
+  for (const ws of workspaceDirs) {
+    try {
+      const stats = await fs.stat(join(projectRoot, ws, "src"));
+      if (stats.isDirectory()) {
+        targetDirs.push(`${ws}/src`);
+      }
+    } catch {}
+  }
+
+  // workspace 定義は見つかったが src/ を持つものが一切ない場合は ["src"] にフォールバック
+  return targetDirs.length > 0 ? targetDirs.sort() : ["src"];
+}
+
+export interface DetectedFormatter {
+  formatter: string | null;
+  formatterArgs: string[];
+}
+
+export async function detectFormatter(projectRoot: string): Promise<DetectedFormatter> {
+  try {
+    const pkgRaw = await fs.readFile(join(projectRoot, "package.json"), "utf-8");
+    const pkg = JSON.parse(pkgRaw) as {
+      devDependencies?: Record<string, string>;
+      dependencies?: Record<string, string>;
+    };
+    const allDeps = { ...(pkg.dependencies ?? {}), ...(pkg.devDependencies ?? {}) };
+    if (allDeps["@biomejs/biome"]) {
+      return { formatter: "biome", formatterArgs: ["check", "--write"] };
+    }
+    if (allDeps["prettier"]) {
+      return { formatter: "eslint-prettier", formatterArgs: [] };
+    }
+  } catch {}
+  return { formatter: null, formatterArgs: [] };
 }
 
 export async function deployHookScripts(harnessRoot: string, projectRoot: string): Promise<DeployHooksResult> {
   const templateDir = join(harnessRoot, HOOKS_TEMPLATE_DIR);
   const targetDir = join(projectRoot, HOOKS_TARGET_DIR);
 
-  // scripts/ ディレクトリをコピー
   const scriptsSource = join(templateDir, "scripts");
   const scriptsTarget = join(targetDir, "scripts");
+  const hookConfigPath = join(scriptsTarget, "hook-config.json");
+
+  // copyDirectory で全 scripts/ を上書きするため、ユーザーカスタマイズを尊重するには
+  // 事前に hook-config.json の既存内容を捕捉して後で復元する。
+  let preexistingHookConfig: string | null = null;
+  try {
+    preexistingHookConfig = await fs.readFile(hookConfigPath, "utf-8");
+  } catch {}
+
   let scriptsDeployed = 0;
 
   try {
@@ -221,6 +365,29 @@ export async function deployHookScripts(harnessRoot: string, projectRoot: string
     }
   } catch {}
 
+  // hook-config.json の決定:
+  //   既存あり → ユーザーカスタマイズ尊重で元の内容を書き戻し
+  //   既存なし → 検出結果を反映して新規生成
+  const detectedTargetDirs = await detectWorkspaceTargetDirs(projectRoot);
+  const { formatter, formatterArgs } = await detectFormatter(projectRoot);
+  let hookConfigGenerated = false;
+
+  if (scriptsDeployed > 0) {
+    if (preexistingHookConfig !== null) {
+      await fs.writeFile(hookConfigPath, preexistingHookConfig, "utf-8");
+    } else {
+      const hookConfig: Record<string, unknown> = {
+        targetDirs: detectedTargetDirs,
+      };
+      if (formatter !== null) {
+        hookConfig.formatter = formatter;
+        hookConfig.formatterArgs = formatterArgs;
+      }
+      await fs.writeFile(hookConfigPath, JSON.stringify(hookConfig, null, 2) + "\n", "utf-8");
+      hookConfigGenerated = true;
+    }
+  }
+
   // settings.json を作成（既存があればスキップ）
   const settingsSource = join(templateDir, "settings.json");
   const settingsTarget = join(targetDir, "settings.json");
@@ -238,7 +405,13 @@ export async function deployHookScripts(harnessRoot: string, projectRoot: string
     } catch {}
   }
 
-  return { scriptsDeployed, settingsCreated };
+  return {
+    scriptsDeployed,
+    settingsCreated,
+    hookConfigGenerated,
+    detectedTargetDirs,
+    detectedFormatter: formatter,
+  };
 }
 
 export async function initHarnessConfig(
@@ -259,6 +432,9 @@ export async function initHarnessConfig(
       name: projectName,
       preset: "standard",
     },
+    architecture: {
+      preset: "clean",
+    },
     layers: {},
     quickMode: {},
     phaseDependencies: {

package/templates/.claude/scripts/analyze-errors-hook.sh

@@ -15,7 +15,10 @@ CONFIG_FILE="$SCRIPT_DIR/hook-config.json"
 TARGET_DIRS=()
 
 if [[ -f "$CONFIG_FILE" ]] && command -v jq >/dev/null 2>&1; then
-    mapfile -t TARGET_DIRS < <(jq -r '.targetDirs[]' "$CONFIG_FILE" 2>/dev/null)
+    # bash 3.2 (macOS default) has no mapfile; use portable while-read.
+    while IFS= read -r line; do
+        [[ -n "$line" ]] && TARGET_DIRS+=("$line")
+    done < <(jq -r '.targetDirs[]' "$CONFIG_FILE" 2>/dev/null)
 fi
 
 if [[ ${#TARGET_DIRS[@]} -eq 0 ]]; then

package/templates/.claude/scripts/format-typescript-hook.sh

@@ -17,9 +17,14 @@ FORMATTER="biome"
 FORMATTER_ARGS=()
 
 if [[ -f "$CONFIG_FILE" ]] && command -v jq &> /dev/null; then
-    mapfile -t TARGET_DIRS < <(jq -r '.targetDirs[]' "$CONFIG_FILE" 2>/dev/null)
+    # bash 3.2 (macOS default) has no mapfile; use portable while-read.
+    while IFS= read -r line; do
+        [[ -n "$line" ]] && TARGET_DIRS+=("$line")
+    done < <(jq -r '.targetDirs[]' "$CONFIG_FILE" 2>/dev/null)
     FORMATTER=$(jq -r '.formatter // "biome"' "$CONFIG_FILE" 2>/dev/null)
-    mapfile -t FORMATTER_ARGS < <(jq -r '.formatterArgs[]' "$CONFIG_FILE" 2>/dev/null)
+    while IFS= read -r line; do
+        [[ -n "$line" ]] && FORMATTER_ARGS+=("$line")
+    done < <(jq -r '.formatterArgs[]' "$CONFIG_FILE" 2>/dev/null)
 fi
 
 if [[ ${#TARGET_DIRS[@]} -eq 0 ]]; then