phasegate 0.183.0 → 0.212.0

package/scripts/harness/phase-dependency-model/domain/ports/story-reflection-file-system-port.ts

@@ -4,6 +4,8 @@
 export interface StoryReflectionFileSystemPort {
   listStoryDirectories(unitId: string): Promise<readonly string[]>;
   storyAffectsUnit(storyId: string, unitId: string): Promise<boolean>;
+  /** WI のコミット（Work-Item trailer）が scripts/harness/{unitId}/{layer}/ 配下のソースを変更したか */
+  storyTouchesUnitLayer(storyId: string, unitId: string, layer: string): Promise<boolean>;
   fileExists(path: string): Promise<boolean>;
   /** product 文書内に @story-id {storyId} アノテーションが含まれているか */
   fileContainsStoryAnnotation(productPath: string, storyId: string): Promise<boolean>;

package/scripts/harness/phase-dependency-model/domain/services/story-reflection-checker.ts

@@ -49,6 +49,14 @@ export class StoryReflectionChecker {
           continue;
         }
 
+        if (
+          resolvedInception.isCrossWorkItem &&
+          productPath.endsWith("domain_model.md") &&
+          !(await this.fsPort.storyTouchesUnitLayer(storyId, unitId, "domain"))
+        ) {
+          continue;
+        }
+
         const hasAnnotation = await this.fsPort.fileContainsStoryAnnotation(productPath, storyId);
 
         if (!hasAnnotation) {

package/scripts/harness/phase-dependency-model/infrastructure/filesystem/file-system-story-reflection-adapter.ts

@@ -4,15 +4,24 @@
  * @work-item-id WI-115
  */
 
+import { execFile } from "node:child_process";
 import type { Dirent } from "node:fs";
 import { access, readdir, readFile } from "node:fs/promises";
 import path from "node:path";
+import { promisify } from "node:util";
 import type { StoryReflectionFileSystemPort } from "../../domain/ports/story-reflection-file-system-port.js";
 
 const CROSS_WORK_ITEM_PATTERN = /^WI-\d+$/;
 const FRONTMATTER_PATTERN = /^---\r?\n([\s\S]*?)\r?\n---/;
 const AFFECTS_INLINE_PATTERN = /^affects:\s*\[([^\]]*)\]\s*$/m;
 const LEGACY_ID_PATTERN = /^legacy_id:\s*([A-Z][\w]+-\d+)\s*$/m;
+/** コミット本文の `Work-Item: WI-NNN` trailer 行から WI を抽出する */
+const WORK_ITEM_TRAILER_PATTERN = /^Work-Item:\s*(WI-\d+)\s*$/gm;
+/** ソースファイル先頭の `@work-item-id` / `@story` タグ配下の WI を抽出する */
+const SOURCE_TAG_PATTERN = /@(?:work-item-id|story)\b[^\n\r]*/g;
+/** harness ソースパスから unit / layer を切り出す（帰属フィルタ適用対象の判定） */
+const HARNESS_SOURCE_PATH_PATTERN = /^scripts\/harness\/[^/]+\/[^/]+\//;
+const execFileAsync = promisify(execFile);
 
 export interface FileSystemStoryReflectionAdapterDeps {
   readonly rootDir: string;
@@ -34,6 +43,7 @@ export interface FileSystemStoryReflectionAdapterDeps {
 export class FileSystemStoryReflectionAdapter implements StoryReflectionFileSystemPort {
   private readonly rootDir: string;
   private readonly inceptionRoot: string;
+  private readonly changedPathsByStoryId = new Map<string, Promise<ReadonlySet<string>>>();
 
   constructor(deps: FileSystemStoryReflectionAdapterDeps) {
     this.rootDir = deps.rootDir;
@@ -110,7 +120,7 @@ export class FileSystemStoryReflectionAdapter implements StoryReflectionFileSyst
 
     const frontmatter = FRONTMATTER_PATTERN.exec(content)?.[1];
     if (frontmatter === undefined) {
-      return true;
+      return false;
     }
 
     const affects = AFFECTS_INLINE_PATTERN.exec(frontmatter)?.[1]
@@ -119,12 +129,137 @@ export class FileSystemStoryReflectionAdapter implements StoryReflectionFileSyst
       .filter((value) => value.length > 0);
 
     if (affects === undefined) {
-      return true;
+      return false;
     }
 
     return affects.includes(unitId);
   }
 
+  async storyTouchesUnitLayer(storyId: string, unitId: string, layer: string): Promise<boolean> {
+    const paths = await this.readChangedPathsForStory(storyId);
+    const prefix = `scripts/harness/${unitId}/${layer}/`;
+
+    for (const changedPath of paths) {
+      if (changedPath.startsWith(prefix)) {
+        return true;
+      }
+    }
+
+    return false;
+  }
+
+  private readChangedPathsForStory(storyId: string): Promise<ReadonlySet<string>> {
+    const cached = this.changedPathsByStoryId.get(storyId);
+    if (cached !== undefined) {
+      return cached;
+    }
+
+    const promise = this.fetchChangedPathsForStory(storyId);
+    this.changedPathsByStoryId.set(storyId, promise);
+    return promise;
+  }
+
+  private async fetchChangedPathsForStory(storyId: string): Promise<ReadonlySet<string>> {
+    try {
+      const { stdout } = await execFileAsync(
+        "git",
+        [
+          "log",
+          "--format=@@COMMIT@@%x00%B%x00",
+          "--name-only",
+          "--grep",
+          `Work-Item:.*\\b${storyId}\\b`,
+        ],
+        { cwd: this.rootDir, maxBuffer: 32 * 1024 * 1024 },
+      );
+
+      return await this.extractChangedPathsForStory(stdout, storyId);
+    } catch {
+      return new Set<string>();
+    }
+  }
+
+  private async extractChangedPathsForStory(output: string, storyId: string): Promise<ReadonlySet<string>> {
+    const changedPaths = new Set<string>();
+    const trailerPattern = new RegExp(`Work-Item:[^\\n]*\\b${storyId}\\b`);
+
+    for (const commitBlock of output.split("@@COMMIT@@\u0000")) {
+      if (commitBlock.length === 0) continue;
+
+      const bodyEnd = commitBlock.indexOf("\u0000");
+      if (bodyEnd === -1) continue;
+
+      const body = commitBlock.slice(0, bodyEnd);
+      if (!trailerPattern.test(body)) continue;
+
+      // 複数 Work-Item trailer を同梱するバッチコミットでは、changed paths が
+      // 全 trailer WI に一律帰属してしまう（over-attribution）。ソースパスに限り
+      // ファイルの帰属タグで storyId への帰属を絞り込む（WI-251）。
+      const isMultiWorkItemCommit = this.countWorkItemTrailers(body) >= 2;
+      const nameOnly = commitBlock.slice(bodyEnd + 1);
+      for (const changedPath of nameOnly.split(/\r?\n/)) {
+        const trimmedPath = changedPath.trim();
+        if (trimmedPath.length === 0 || trimmedPath === "@@COMMIT@@") continue;
+
+        if (await this.pathAttributesToStory(trimmedPath, storyId, isMultiWorkItemCommit)) {
+          changedPaths.add(trimmedPath);
+        }
+      }
+    }
+
+    return changedPaths;
+  }
+
+  private countWorkItemTrailers(body: string): number {
+    return body.match(WORK_ITEM_TRAILER_PATTERN)?.length ?? 0;
+  }
+
+  /**
+   * バッチコミットの changed path を storyId に帰属させてよいか判定する。
+   * - 単一 WI trailer コミット → 常に帰属（現行挙動を維持）
+   * - 複数 WI trailer × 非ソースパス → 帰属（絞り込み対象外）
+   * - 複数 WI trailer × ソースパス → ファイル内容(HEAD)のタグに storyId を含むときのみ帰属。
+   *   タグ無し / ファイルが読めない → fail-closed で帰属維持（反映要求を緩めない）。
+   */
+  private async pathAttributesToStory(
+    changedPath: string,
+    storyId: string,
+    isMultiWorkItemCommit: boolean,
+  ): Promise<boolean> {
+    if (!isMultiWorkItemCommit) {
+      return true;
+    }
+
+    if (!HARNESS_SOURCE_PATH_PATTERN.test(changedPath)) {
+      return true;
+    }
+
+    const tags = await this.readSourceWorkItemTags(changedPath);
+    if (tags === null || tags.size === 0) {
+      return true;
+    }
+
+    return tags.has(storyId);
+  }
+
+  private async readSourceWorkItemTags(relativePath: string): Promise<Set<string> | null> {
+    let content: string;
+    try {
+      content = await readFile(path.join(this.rootDir, relativePath), "utf8");
+    } catch {
+      return null;
+    }
+
+    const tags = new Set<string>();
+    for (const line of content.match(SOURCE_TAG_PATTERN) ?? []) {
+      for (const id of line.match(/WI-\d+/g) ?? []) {
+        tags.add(id);
+      }
+    }
+
+    return tags;
+  }
+
   async fileContainsStoryAnnotation(productPath: string, storyId: string): Promise<boolean> {
     const absolutePath = path.join(this.rootDir, productPath);
 

package/scripts/harness/quick-mode/domain/services/quick-mode-judgment-engine.ts

@@ -6,12 +6,12 @@
  * ChangedFile[]をChangeClassificationに変換し、3拒否ルールを評価してQuickModeEligibilityを返すドメインサービス
  */
 
-import { ChangeCategory } from '../value-objects/change-category.js';
-import { ChangeClassification } from '../value-objects/change-classification.js';
-import { QuickModeEligibility } from '../value-objects/quick-mode-eligibility.js';
-import { isCommentOnlyDiff } from './comment-only-diff-detector.js';
-import type { ChangedFile } from '../value-objects/changed-file.js';
-import type { QuickModeConfig } from '../value-objects/quick-mode-config.js';
+import { ChangeCategory } from "../value-objects/change-category.js";
+import { ChangeClassification } from "../value-objects/change-classification.js";
+import type { ChangedFile } from "../value-objects/changed-file.js";
+import type { QuickModeConfig } from "../value-objects/quick-mode-config.js";
+import { QuickModeEligibility } from "../value-objects/quick-mode-eligibility.js";
+import { isCommentOnlyDiff } from "./comment-only-diff-detector.js";
 
 // リスク順優先度（api > domain > feature > bugfix > test > config > docs）
 const RISK_PRIORITY: Record<string, number> = {
@@ -30,48 +30,54 @@ function categorizeFile(file: ChangedFile): ChangeCategory {
   // Config files must stay config even when Edit payload snippets look like
   // comment/whitespace-only diffs. Config recovery guidance depends on this.
   if (
-    filePath.endsWith('.config.json') ||
-    filePath.endsWith('.config.ts') ||
-    filePath.endsWith('phasegate.config.json')
+    filePath.endsWith(".config.json") ||
+    filePath.endsWith(".config.ts") ||
+    filePath.endsWith("phasegate.config.json")
   ) {
-    return ChangeCategory.fromString('config');
+    return ChangeCategory.fromString("config");
   }
 
   if (isCommentOnlyDiff(file)) {
-    return ChangeCategory.fromString('docs');
+    return ChangeCategory.fromString("docs");
   }
 
   // api: *port.ts or *adapter.ts（最高優先度）
-  if (filePath.endsWith('port.ts') || filePath.endsWith('adapter.ts')) {
-    return ChangeCategory.fromString('api');
+  if (filePath.endsWith("port.ts") || filePath.endsWith("adapter.ts")) {
+    return ChangeCategory.fromString("api");
   }
 
   // test: __tests__/ 配下 or *.test.ts or *.spec.ts（domainより優先）
-  if (
-    filePath.includes('__tests__/') ||
-    filePath.endsWith('.test.ts') ||
-    filePath.endsWith('.spec.ts')
-  ) {
-    return ChangeCategory.fromString('test');
+  if (filePath.includes("__tests__/") || filePath.endsWith(".test.ts") || filePath.endsWith(".spec.ts")) {
+    return ChangeCategory.fromString("test");
   }
 
   // domain: domain/ 配下
-  if (filePath.includes('/domain/') || filePath.startsWith('domain/')) {
-    return ChangeCategory.fromString('domain');
+  if (filePath.includes("/domain/") || filePath.startsWith("domain/")) {
+    return ChangeCategory.fromString("domain");
   }
 
   // docs: docs/ 配下
-  if (filePath.startsWith('docs/') || filePath.includes('/docs/')) {
-    return ChangeCategory.fromString('docs');
+  if (filePath.startsWith("docs/") || filePath.includes("/docs/")) {
+    return ChangeCategory.fromString("docs");
+  }
+
+  // docs: skills/ 配下の .md（指示文書。SKILL.md / references/*.md 等）は
+  // ソースコードのフェーズゲート対象外とし docs に分類する。品質・完全性は
+  // 専用防御（skill-quality corpus 適合テスト・advisory allowlist pin・
+  // integrity pin(WI-254)・L3-006 injection scanner(WI-259)）が担う。
+  // skills/ 配下の非 .md は下の feature/bugfix フォールバックに落ちて fail-closed を維持する。
+  // @work-item-id WI-261
+  if (filePath.startsWith("skills/") && filePath.endsWith(".md")) {
+    return ChangeCategory.fromString("docs");
   }
 
   // feature: domain/ 以外の CREATE
-  if (changeKind === 'CREATE') {
-    return ChangeCategory.fromString('feature');
+  if (changeKind === "CREATE") {
+    return ChangeCategory.fromString("feature");
   }
 
   // bugfix: domain/ 以外の MODIFY/DELETE
-  return ChangeCategory.fromString('bugfix');
+  return ChangeCategory.fromString("bugfix");
 }
 
 export class QuickModeJudgmentEngine {
@@ -107,7 +113,7 @@ export class QuickModeJudgmentEngine {
     const classification = this.classify(changedFiles, config);
 
     // 1. MIXED_CHANGES評価: allowedCategories 外のカテゴリが含まれる場合
-    if (config.isFullModeRequiredFor('mixedCategories')) {
+    if (config.isFullModeRequiredFor("mixedCategories")) {
       const notAllowedFiles: ChangedFile[] = [];
       classification.categorizedFiles.forEach((files, categoryKey) => {
         if (!config.isAllowed(categoryKey)) {
@@ -117,45 +123,43 @@ export class QuickModeJudgmentEngine {
 
       if (notAllowedFiles.length > 0) {
         return QuickModeEligibility.rejected(
-          'MIXED_CHANGES',
+          "MIXED_CHANGES",
           notAllowedFiles,
-          `allowedCategories外のファイルが含まれています: ${notAllowedFiles.map((f) => f.filePath).join(', ')}`
+          `allowedCategories外のファイルが含まれています: ${notAllowedFiles.map((f) => f.filePath).join(", ")}`,
         );
       }
     }
 
     // 2. NEW_DOMAIN評価: domain/ 配下かつ changeKind=CREATE
-    if (config.isFullModeRequiredFor('newDomainFile')) {
+    if (config.isFullModeRequiredFor("newDomainFile")) {
       const newDomainFiles = changedFiles.filter(
-        (f) =>
-          (f.filePath.includes('/domain/') || f.filePath.startsWith('domain/')) &&
-          f.changeKind === 'CREATE'
+        (f) => (f.filePath.includes("/domain/") || f.filePath.startsWith("domain/")) && f.changeKind === "CREATE",
       );
 
       if (newDomainFiles.length > 0) {
         return QuickModeEligibility.rejected(
-          'NEW_DOMAIN',
+          "NEW_DOMAIN",
           newDomainFiles,
-          `domain/配下に新規ファイルが追加されています: ${newDomainFiles.map((f) => f.filePath).join(', ')}`
+          `domain/配下に新規ファイルが追加されています: ${newDomainFiles.map((f) => f.filePath).join(", ")}`,
         );
       }
     }
 
     // 3. API_CONTRACT評価: *port.ts / *adapter.ts の変更
-    if (config.isFullModeRequiredFor('apiContractChange')) {
+    if (config.isFullModeRequiredFor("apiContractChange")) {
       const apiContractFiles = changedFiles.filter(
-        (f) => (f.filePath.endsWith('port.ts') || f.filePath.endsWith('adapter.ts')) && !isCommentOnlyDiff(f)
+        (f) => (f.filePath.endsWith("port.ts") || f.filePath.endsWith("adapter.ts")) && !isCommentOnlyDiff(f),
       );
 
       if (apiContractFiles.length > 0) {
         return QuickModeEligibility.rejected(
-          'API_CONTRACT',
+          "API_CONTRACT",
           apiContractFiles,
-          `Port/Adapterインターフェースファイルの変更が含まれています: ${apiContractFiles.map((f) => f.filePath).join(', ')}`
+          `Port/Adapterインターフェースファイルの変更が含まれています: ${apiContractFiles.map((f) => f.filePath).join(", ")}`,
         );
       }
     }
 
-    return QuickModeEligibility.eligible('すべてのファイルが許可カテゴリ内です');
+    return QuickModeEligibility.eligible("すべてのファイルが許可カテゴリ内です");
   }
 }

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

@@ -33,16 +33,14 @@ export const SKILL_CATEGORIES: Record<SkillCategory, readonly string[]> = {
     "cascade-updater",
     "codebase-mapper",
     "consistency-checker",
-    "doc-freshness-checker",
+    "doc-health-checker",
     "engineering-perspective",
     "implementation-readiness-checker",
-    "pointer-validator",
     "test-coverage-checker",
   ],
   aidlc: [
     "domain-designer",
     "environment-designer",
-    "implementation-planner",
     "it-test-designer",
     "it-test-logic-designer",
     "logical-designer",
@@ -60,7 +58,7 @@ export const SKILL_CATEGORIES: Record<SkillCategory, readonly string[]> = {
     "unit-test-logic-designer",
   ],
   utility: ["codex-delegator", "skill-creator"],
-  guidance: ["phasegate-toolkit-guide", "phasegate-config-doctor"],
+  guidance: ["phasegate-toolkit-guide", "phasegate-config-doctor", "release-publisher"],
 } as const;
 
 export function getSkillsForSet(skillSet: SkillSet): string[] {
@@ -173,9 +171,12 @@ export function renderSkillForModelDelegation(content: string, policy: ModelDele
   return content
     .replace(/^model:\s+\S+\n/gm, "")
     .replace(/^review:\s+\S+\n/gm, "")
-    .replaceAll("Sonnet 4.6 に委任して成果物を生成する（`npx phasegate delegate-sonnet` 経由）", "メインセッションが成果物を生成する")
+    .replaceAll(
+      "委任先モデルに委任して成果物を生成する（`npx phasegate delegate-sonnet` 経由）",
+      "メインセッションが成果物を生成する",
+    )
     .replaceAll("Opus が", "メインセッションが")
-    .replaceAll("Sonnet 4.6", "メインセッション")
+    .replaceAll("委任先モデル", "メインセッション")
     .replaceAll("`npx phasegate delegate-sonnet`", "メインセッション");
 }
 
@@ -726,10 +727,7 @@ export async function deployHuskyCommitMsgHook(
   return { created: true, path: targetPath };
 }
 
-export async function deployHuskyPrePushHook(
-  harnessRoot: string,
-  projectRoot: string,
-): Promise<DeployHuskyHookResult> {
+export async function deployHuskyPrePushHook(harnessRoot: string, projectRoot: string): Promise<DeployHuskyHookResult> {
   const targetPath = join(projectRoot, ".husky", "pre-push");
 
   try {

package/scripts/harness/skill-quality/domain/services/skill-structure-validator.ts

@@ -1,12 +1,13 @@
 /**
  * @layer domain
  * @unit skill-quality
- * @work-item-id WI-212
+ * @work-item-id WI-212, WI-241
  */
 import { SkillStructure } from '../value-objects/skill-structure.js';
 import { SkillValidationResult } from '../value-objects/skill-validation-result.js';
 import type { SkillFileReaderPort } from '../ports/skill-file-reader-port.js';
 import type { SectionName } from '../types/section-name.js';
+import type { SkillKind } from '../types/skill-kind.js';
 
 export class SkillStructureValidator {
   constructor(private readonly skillFileReaderPort: SkillFileReaderPort) {}
@@ -14,7 +15,8 @@ export class SkillStructureValidator {
   async validate(skillFilePath: string): Promise<SkillValidationResult> {
     const rawContent = await this.skillFileReaderPort.read(skillFilePath);
     const actualSections = this.extractSections(rawContent);
-    const structure = SkillStructure.default();
+    const kind = this.extractKind(rawContent);
+    const structure = SkillStructure.forKind(kind);
     const missingSections = structure.getMissingSections(actualSections);
 
     if (missingSections.length === 0) {
@@ -74,6 +76,15 @@ export class SkillStructureValidator {
     return sections;
   }
 
+  private extractKind(content: string): SkillKind {
+    const lines = content.split('\n');
+    const closingDelimiterIndex = lines.findIndex((line, index) => index > 0 && line.trim() === '---');
+    if (lines[0]?.trim() !== '---' || closingDelimiterIndex < 0) return 'lifecycle';
+
+    const frontmatterLines = lines.slice(1, closingDelimiterIndex);
+    return frontmatterLines.some((line) => /^\s*kind\s*:\s*advisory\s*$/.test(line)) ? 'advisory' : 'lifecycle';
+  }
+
   private hasLanguageMetadata(lines: readonly string[]): boolean {
     const closingDelimiterIndex = lines.findIndex((line, index) => index > 0 && line.trim() === '---');
     if (closingDelimiterIndex < 0) return false;

package/scripts/harness/skill-quality/domain/types/skill-kind.ts

@@ -0,0 +1,6 @@
+/**
+ * @layer domain
+ * @unit skill-quality
+ * @work-item-id WI-241
+ */
+export type SkillKind = 'lifecycle' | 'advisory';

package/scripts/harness/skill-quality/domain/value-objects/skill-structure.ts

@@ -1,11 +1,12 @@
 /**
  * @layer domain
  * @unit skill-quality
- * @work-item-id WI-212
+ * @work-item-id WI-212, WI-241
  */
 import type { SectionName } from '../types/section-name.js';
+import type { SkillKind } from '../types/skill-kind.js';
 
-const REQUIRED_SECTIONS: readonly SectionName[] = [
+const REQUIRED_SECTIONS: readonly SectionName[] = Object.freeze([
   'frontmatter',
   'languageMetadata',
   'purpose',
@@ -13,7 +14,13 @@ const REQUIRED_SECTIONS: readonly SectionName[] = [
   'outputs',
   'prerequisites',
   'executionFlow',
-];
+]);
+
+const ADVISORY_REQUIRED_SECTIONS: readonly SectionName[] = Object.freeze([
+  'frontmatter',
+  'languageMetadata',
+  'purpose',
+]);
 
 export class SkillStructure {
   readonly requiredSections: readonly SectionName[];
@@ -23,13 +30,22 @@ export class SkillStructure {
     Object.freeze(this);
   }
 
-  private static _instance: SkillStructure | null = null;
+  private static _instances: Partial<Record<SkillKind, SkillStructure>> = {};
 
-  static default(): SkillStructure {
-    if (!SkillStructure._instance) {
-      SkillStructure._instance = new SkillStructure(REQUIRED_SECTIONS);
+  static forKind(kind: SkillKind): SkillStructure {
+    const cached = SkillStructure._instances[kind];
+    if (cached) {
+      return cached;
     }
-    return SkillStructure._instance;
+
+    const requiredSections = kind === 'advisory' ? ADVISORY_REQUIRED_SECTIONS : REQUIRED_SECTIONS;
+    const instance = new SkillStructure(requiredSections);
+    SkillStructure._instances[kind] = instance;
+    return instance;
+  }
+
+  static default(): SkillStructure {
+    return SkillStructure.forKind('lifecycle');
   }
 
   getMissingSections(actualSections: readonly SectionName[]): readonly SectionName[] {