phasegate 0.283.0 → 0.315.0

package/scripts/harness/installation/infrastructure/adapters/config-status-probe-adapter.ts

@@ -0,0 +1,79 @@
+// @unit installation
+// @layer infrastructure
+// @work-item-id WI-330
+
+import { access } from "node:fs/promises";
+import { join } from "node:path";
+import { createConfigFoundationModule } from "../../../config-foundation/composition-root.js";
+import { ConfigValidationError } from "../../../config-foundation/domain/errors/config-validation-error.js";
+import {
+  ConfigNotFoundError,
+  ConfigParseError,
+} from "../../../config-foundation/infrastructure/repositories/file-system-config-repository.js";
+import type { ConfigStatusProbePort } from "../../application/ports/config-status-probe-port.js";
+import type { ConfigStatusProbeResult } from "../../domain/config-status.js";
+
+const PROJECT_CONFIG_PATH = "phasegate.config.json";
+const PERSONAL_CONFIG_PATH = join(".phasegate-local", "phasegate.config.json");
+
+async function exists(targetPath: string): Promise<boolean> {
+  try {
+    await access(targetPath);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+/**
+ * WI-330: doctor 用の config 状態 probe。
+ *
+ * FileSystemConfigRepository と同じ候補順（project 直下 → .phasegate-local/ の personal install）で
+ * projectRoot 直下のみを解決する。cwd からの上方探索は行わない — doctor は「このプロジェクトの
+ * config」を診断するため、親ディレクトリの config を拾うと診断が偽装される。
+ * 妥当性判定は config-foundation の実 load 経路（JSON parse + AJV schema + preset 解決）を
+ * そのまま使い、CLI 本体と同じ基準で invalid を検出する。
+ */
+export class ConfigStatusProbeAdapter implements ConfigStatusProbePort {
+  private readonly cache = new Map<string, Promise<ConfigStatusProbeResult>>();
+
+  probe(projectRoot: string): Promise<ConfigStatusProbeResult> {
+    const cached = this.cache.get(projectRoot);
+    if (cached !== undefined) {
+      return cached;
+    }
+    const inspected = this.inspect(projectRoot);
+    this.cache.set(projectRoot, inspected);
+    return inspected;
+  }
+
+  private async inspect(projectRoot: string): Promise<ConfigStatusProbeResult> {
+    for (const relativePath of [PROJECT_CONFIG_PATH, PERSONAL_CONFIG_PATH]) {
+      const absolutePath = join(projectRoot, relativePath);
+      if (await exists(absolutePath)) {
+        return await this.classify(absolutePath, relativePath);
+      }
+    }
+    return { status: "missing", configPath: PROJECT_CONFIG_PATH, detail: null };
+  }
+
+  private async classify(absolutePath: string, relativePath: string): Promise<ConfigStatusProbeResult> {
+    try {
+      await createConfigFoundationModule().usecases.loadResolvedConfigUseCase.execute(absolutePath);
+      return { status: "valid", configPath: relativePath, detail: null };
+    } catch (error) {
+      if (error instanceof ConfigNotFoundError) {
+        return { status: "missing", configPath: relativePath, detail: null };
+      }
+      if (error instanceof ConfigParseError) {
+        const detail = error.cause instanceof Error ? error.cause.message : error.message;
+        return { status: "invalid-json", configPath: relativePath, detail };
+      }
+      if (error instanceof ConfigValidationError) {
+        return { status: "invalid-schema", configPath: relativePath, detail: error.message };
+      }
+      const detail = error instanceof Error ? error.message : String(error);
+      return { status: "invalid-schema", configPath: relativePath, detail };
+    }
+  }
+}

package/scripts/harness/installation/presentation/cli/doctor-handler.ts

@@ -3,10 +3,14 @@
 // @work-item-id WI-145
 // @work-item-id WI-178
 // @work-item-id WI-208
+// @work-item-id WI-330
 
 import { mkdir, writeFile } from "node:fs/promises";
 import { dirname, isAbsolute, join } from "node:path";
-import type { DoctorAgentScope, RunDoctorDiagnosticsUseCase } from "../../application/usecases/run-doctor-diagnostics.js";
+import type {
+  DoctorAgentScope,
+  RunDoctorDiagnosticsUseCase,
+} from "../../application/usecases/run-doctor-diagnostics.js";
 import { DiagnosticReportFormatter } from "../formatters/diagnostic-report-formatter.js";
 
 export interface DoctorHandlerInput {
@@ -39,6 +43,7 @@ export class DoctorHandler {
       report: result.report,
       agent: result.agent,
       installationMode: result.installationMode,
+      configStatus: result.configStatus,
       scopedOutFindings: result.scopedOutFindings,
       phasegateVersion: input.phasegateVersion,
       projectRoot: input.projectRoot,

package/scripts/harness/installation/presentation/formatters/diagnostic-report-formatter.ts

@@ -5,14 +5,20 @@
 // @work-item-id WI-179
 // @work-item-id WI-180
 // @work-item-id WI-208
+// @work-item-id WI-330
 
-import type { DoctorAgentScope, ScopedOutDiagnosticFinding } from "../../application/usecases/run-doctor-diagnostics.js";
+import type {
+  DoctorAgentScope,
+  ScopedOutDiagnosticFinding,
+} from "../../application/usecases/run-doctor-diagnostics.js";
+import type { ConfigStatus } from "../../domain/config-status.js";
 import type { DiagnosticReport } from "../../domain/diagnostic-report.js";
 
 export interface DiagnosticReportFormatterInput {
   readonly report: DiagnosticReport;
   readonly agent: DoctorAgentScope;
   readonly installationMode: "project" | "personal";
+  readonly configStatus: ConfigStatus;
   readonly scopedOutFindings: readonly ScopedOutDiagnosticFinding[];
   readonly phasegateVersion: string;
   readonly projectRoot: string;
@@ -32,6 +38,7 @@ export class DiagnosticReportFormatter {
           description: scopeDescription(input.agent, input.installationMode),
         },
         overallStatus: input.report.overallStatus,
+        configStatus: input.configStatus,
         findings: input.report.findings.map((finding) => ({
           ...finding.toJSON(),
           applicability: "applicable",
@@ -64,6 +71,7 @@ export class DiagnosticReportFormatter {
       `phasegate doctor v${input.phasegateVersion}`,
       `Project: ${input.projectRoot}`,
       `Scope: ${input.agent} / ${input.installationMode} (${scopeDescription(input.agent, input.installationMode)})`,
+      `Config: ${input.configStatus}`,
       "",
     ];
     for (const finding of input.report.findings) {
@@ -79,10 +87,14 @@ export class DiagnosticReportFormatter {
     }
     const redCount = input.report.findings.filter((finding) => finding.severity === "red").length;
     const warnCount = input.report.findings.filter((finding) => finding.severity === "warn").length;
-    lines.push(`Status: ${input.report.overallStatus.toUpperCase()} (${input.report.findings.length} findings: ${redCount} red, ${warnCount} warn)`);
+    lines.push(
+      `Status: ${input.report.overallStatus.toUpperCase()} (${input.report.findings.length} findings: ${redCount} red, ${warnCount} warn)`,
+    );
     if (input.scopedOutFindings.length > 0) {
       const checkIds = input.scopedOutFindings.map(({ finding }) => finding.checkId).join(", ");
-      lines.push(`Scoped out: ${input.scopedOutFindings.length} informational findings not applicable to --agent ${input.agent}; not repair targets for this scope: ${checkIds}.`);
+      lines.push(
+        `Scoped out: ${input.scopedOutFindings.length} informational findings not applicable to --agent ${input.agent}; not repair targets for this scope: ${checkIds}.`,
+      );
     }
     lines.push(`Exit: ${input.exitCode}`);
     return lines.join("\n");
@@ -91,8 +103,10 @@ export class DiagnosticReportFormatter {
 
 function scopeDescription(agent: DoctorAgentScope, installationMode: "project" | "personal"): string {
   if (installationMode === "personal") {
-    if (agent === "claude") return "Personal Claude Code sandbox; team/project Husky, CI, package, and Codex-only findings are not repair targets.";
-    if (agent === "codex") return "Personal Codex sandbox; team/project Husky, CI, package, and Claude-only findings are not repair targets.";
+    if (agent === "claude")
+      return "Personal Claude Code sandbox; team/project Husky, CI, package, and Codex-only findings are not repair targets.";
+    if (agent === "codex")
+      return "Personal Codex sandbox; team/project Husky, CI, package, and Claude-only findings are not repair targets.";
     return "Personal sandbox diagnostics; team/project Husky, CI, and package findings are not repair targets.";
   }
   if (agent === "claude") return "Claude Code and shared setup targets; Codex-only findings are not applicable.";

package/scripts/harness/integrations/pre-commit.ts

@@ -5,6 +5,7 @@
  * @work-item-id WI-109
  * @work-item-id WI-189
  * @work-item-id WI-305
+ * @work-item-id WI-332
  *
  * Pre-commit CLI entry.
  * Runs L2 validators against staged TypeScript files AND design-document
@@ -27,6 +28,7 @@ import type { ValidateMetadataCommandOutput } from "../traceability-model/presen
 import type { AggregatedValidationReport } from "../validator-system/application/dto/aggregated-validation-report.js";
 import type { ValidationResultContract } from "../validator-system/application/dto/validation-result-contract.js";
 import { createValidatorSystemModule } from "../validator-system/composition-root.js";
+import { isEffectivelyPassed } from "../validator-system/domain/services/effective-severity-policy.js";
 import { HumanValidationResultFormatter } from "../validator-system/presentation/formatters/human-validation-result-formatter.js";
 import { createWorldModelModule } from "../world-model/index.js";
 
@@ -293,9 +295,19 @@ function getStagedDesignChangeFiles(): string[] {
   }
 }
 
+/**
+ * WI-332 / ADR-017: pre-commit 経路には config の failOnWarning 配線が存在しないため、
+ * ADR-017 の既定値 (false) に固定する。validate.failOnWarning を pre-commit にも効かせる
+ * 判断をする場合は、この定数を options 経由の配線に置き換えること。
+ */
+const PRE_COMMIT_FAIL_ON_WARNING = false;
+
 function buildReport(results: readonly ValidationResultContract[]): AggregatedValidationReport {
-  const passed = results.filter((r) => r.passed && !r.skipped).length;
-  const failed = results.filter((r) => !r.passed && !r.skipped).length;
+  // WI-332 / ADR-017: 手動集約 (`failed === 0` の raw passed 判定) をやめ、
+  // validate / ci-check / complete-check と同じ共有実効判定 isEffectivelyPassed を通す。
+  // warning-only の validator failure は既定で effectively passed = exit 0。
+  const passed = results.filter((r) => !r.skipped && isEffectivelyPassed(r, PRE_COMMIT_FAIL_ON_WARNING)).length;
+  const failed = results.filter((r) => !r.skipped && !isEffectivelyPassed(r, PRE_COMMIT_FAIL_ON_WARNING)).length;
   const skipped = results.filter((r) => r.skipped).length;
   const allErrors = results.flatMap((r) => r.errors);
   const errorCount = allErrors.filter((e) => e.severity === "error").length;
@@ -452,7 +464,9 @@ export async function validateBypassTrailers(
 }
 
 function classifyValidatorFailure(result: ValidationResultContract): BypassBlockerClass | undefined {
-  if (result.passed || result.skipped) return undefined;
+  // WI-332: blocker 分類も buildReport と同じ実効判定を通す。warning-only failure は
+  // exit 0 (effectively passed) なので、bypass audit の blocker としても数えない。
+  if (isEffectivelyPassed(result, PRE_COMMIT_FAIL_ON_WARNING)) return undefined;
   const nonBypassable = NON_BYPASSABLE_VALIDATOR_IDS.includes(result.validatorId);
   return {
     code: result.validatorId,

package/scripts/harness/main.ts

@@ -41,7 +41,7 @@ import {
   rm as fsRm,
   writeFile as fsWriteFile,
 } from "node:fs/promises";
-import { dirname, join, resolve } from "node:path";
+import { basename, dirname, join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 import { createAdrFoundationModule } from "./adr-foundation/composition-root.js";
 import { createBiomeAstEngineModule } from "./biome-ast-engine/composition-root.js";
@@ -75,7 +75,7 @@ import {
 import { FileSystemStoryReflectionAdapter } from "./phase-dependency-model/infrastructure/filesystem/file-system-story-reflection-adapter.js";
 import { StoryReflectionStatusPresenter } from "./phase-dependency-model/presentation/cli/story-reflection-status-presenter.js";
 import { buildPhase2Extensions } from "./phase2-extensions/composition-root.js";
-import { createQuickModeCompositionRoot } from "./quick-mode/composition-root.js";
+import { createQuickModeCompositionRoot, type QuickModeCompositionRootOptions } from "./quick-mode/composition-root.js";
 import { buildRegressionSuite } from "./regression-suite/composition-root.js";
 import type { SkillSet } from "./setup/skill-deployer.js";
 import {
@@ -228,7 +228,7 @@ Commands:
 
   lint                         Run lint checks (--json, --target <path>)
 
-  validate                     Run validators (--layer L2|L3|L4|all; L0 prints runtime hook info, --unit, --format human|agent|ci|json, --json)
+  validate                     Run validators (--layer L0|L2|L3|L4|all; L0 prints runtime hook info, --unit, --format human|agent|ci|json, --json)
   ci-check                     CI check (--quick for quick mode, --fail-on-reject, --dry-run, --files)
   check-change-category        Classify changed paths for quick mode (--paths <csv>, --format human|json)
 
@@ -501,6 +501,15 @@ function parseValidateFormat(args: readonly string[]): "human" | "agent" | "ci"
   throw new Error(`Invalid --format value for validate: '${raw}'. Supported values: human, agent, ci, json.`);
 }
 
+function parseValidateLayer(args: readonly string[]): "L0" | "L2" | "L3" | "L4" | "all" | undefined {
+  const raw = parseFlag(args, "--layer");
+  if (raw === undefined) return undefined;
+  if (raw === "L0" || raw === "L2" || raw === "L3" || raw === "L4" || raw === "all") return raw;
+
+  const lintGuidance = raw === "L1" ? "\nL1 は `npx phasegate lint` で実行してください。" : "";
+  throw new Error(`不正な --layer 値: ${raw}\n有効値: L0, L2, L3, L4, all${lintGuidance}`);
+}
+
 function levenshtein(a: string, b: string): number {
   const m = a.length;
   const n = b.length;
@@ -553,12 +562,25 @@ function validateKnownFlags(args: readonly string[], known: readonly string[]):
   return null;
 }
 
+/**
+ * Full Mode session が許可する変更カテゴリ。
+ *
+ * WI-348: 照合相手は quick-mode の `ChangeCategoryValue`
+ * (`bugfix | docs | test | config | feature | domain | api`) であり、
+ * レイヤー名（application / infrastructure / presentation）ではない。
+ * 旧実装はレイヤー名語彙を書き出していたため、交差が domain / config のみとなり
+ * session を張っても feature / api の書き込みがブロックされ続けていた。
+ * ChangeCategory の全語彙を許可し、session のスコープ制御は
+ * unit / 期限 / target path 側の判定に委ねる。
+ */
 const FULL_MODE_SESSION_ALLOWED_CATEGORIES = Object.freeze([
-  "domain",
-  "application",
-  "infrastructure",
-  "presentation",
+  "bugfix",
+  "docs",
+  "test",
   "config",
+  "feature",
+  "domain",
+  "api",
 ]);
 
 interface FullModeSessionFile {
@@ -1901,6 +1923,32 @@ async function loadResolvedConfig(command?: string): Promise<HarnessConfigV2 | u
   }
 }
 
+/**
+ * WI-351: quick-mode composition root へ解決済み configPath / rootDir を注入する。
+ *
+ * 無指定だと `HarnessConfigQuickModeConfigAdapter` は `process.cwd()/phasegate.config.json`、
+ * `FsFileExistenceAdapter` は `process.cwd()` を基準にする。サブディレクトリから CLI を
+ * 実行すると config を見失い、相対パスの存在判定（= CREATE/MODIFY 推定）も
+ * プロジェクトルート基準の hook とずれて分類結果が食い違う。
+ * config-foundation が上方探索で解決した sourcePath を基準に揃える
+ * （hook 側で WI-346 が行ったのと同じ整合）。
+ * config 未検出・不正時は従来どおり cwd 基準へフォールバックする（fail-open）。
+ */
+async function resolveQuickModeCompositionOptions(): Promise<QuickModeCompositionRootOptions> {
+  try {
+    const configModule = createConfigFoundationModule();
+    const { sourcePath } = await configModule.usecases.loadResolvedConfigUseCase.execute();
+    if (typeof sourcePath === "string" && sourcePath !== "") {
+      const configDir = dirname(sourcePath);
+      const rootDir = basename(configDir) === ".phasegate-local" ? dirname(configDir) : configDir;
+      return { configPath: sourcePath, rootDir };
+    }
+  } catch {
+    // 解決できない場合はフォールバック（下の return）
+  }
+  return { rootDir: getProjectRoot() };
+}
+
 async function loadWorldResolvedConfig() {
   try {
     const configModule = createConfigFoundationModule();
@@ -2888,7 +2936,7 @@ async function main(): Promise<void> {
       // ── validator-system ──
       case "validate": {
         const mod = createValidatorSystemModule(toValidatorSystemConfig(resolvedConfig));
-        const layer = parseFlag(args, "--layer") as "L0" | "L2" | "L3" | "L4" | "all" | undefined;
+        const layer = parseValidateLayer(args);
         const unit = parseFlag(args, "--unit");
         const phase = parseFlag(args, "--phase");
         const format = parseValidateFormat(args) ?? (json ? "ci" : undefined);
@@ -2956,7 +3004,7 @@ async function main(): Promise<void> {
           );
           return;
         }
-        const mod = createQuickModeCompositionRoot();
+        const mod = createQuickModeCompositionRoot(await resolveQuickModeCompositionOptions());
         const paths = parseFlag(args, "--paths");
         const format = parseFlag(args, "--format") as "human" | "json" | undefined;
         const failOnFullRequired = hasFlag(args, "--fail-on-full-required");

package/scripts/harness/phase-dependency-model/infrastructure/filesystem/markdown-plan-document-reader.ts

@@ -3,20 +3,32 @@
  * @unit phase-dependency-model
  */
 
-import { readFile } from 'node:fs/promises';
-import * as path from 'node:path';
-import type { PlanDocumentReaderPort } from '../../domain/ports/plan-document-reader-port.js';
-import type { PhaseNode } from '../../domain/values/phase-node.js';
-import type { PlanningMode } from '../../domain/values/planning-mode.js';
-import { PlanEvidence } from '../../domain/values/plan-evidence.js';
+import { readFile } from "node:fs/promises";
+import * as path from "node:path";
+import type { PlanDocumentReaderPort } from "../../domain/ports/plan-document-reader-port.js";
+import type { PhaseNode } from "../../domain/values/phase-node.js";
+import { PlanEvidence } from "../../domain/values/plan-evidence.js";
+import type { PlanningMode } from "../../domain/values/planning-mode.js";
 
 export interface MarkdownPlanDocumentReaderDeps {
   readonly rootDir: string;
 }
 
-const QA_SECTION_PATTERN = /^## QA\b/m;
-const QUESTION_PATTERN = /^Q:/gm;
-const ANSWER_PATTERN = /^A:/gm;
+/**
+ * WI-358 (issue #29): 見出しの表記ゆれで Planning Mode の evidence 判定が落ちる罠を潰す。
+ *
+ * 旧パターンは "QA" 綴りしか受け付けず、エージェントが自然に書く `## Q&A` /
+ * 全角の `## Q＆A` を弾いていた。中身は同一なのに phase gate だけが落ち、
+ * 原因が見出しの綴りだと気付けないまま止まる。
+ *
+ * 緩和方向のみの変更であり、既存の `## QA` / `## 4. QA（不明点・確認事項）` は
+ * すべて従来どおりマッチする。
+ */
+const QA_SECTION_PATTERN = /^##+\s*(?:\d+[.．]\s*)?Q[&＆]?A\b/m;
+const QUESTION_PATTERN = /^(?:Q:|###\s*\[Question\])/gm;
+const LEGACY_ANSWER_PATTERN = /^A:/gm;
+const STRUCTURED_ANSWER_PATTERN = /^\[Answer\][ \t]*$/gm;
+const ANSWER_BODY_BOUNDARY_PATTERN = /^(?:###\s*\[Question\]|#{1,6}\s+)/m;
 
 export class MarkdownPlanDocumentReader implements PlanDocumentReaderPort {
   private readonly rootDir: string;
@@ -55,7 +67,7 @@ export class MarkdownPlanDocumentReader implements PlanDocumentReaderPort {
 
     let content: string;
     try {
-      content = await readFile(absolutePath, 'utf8');
+      content = await readFile(absolutePath, "utf8");
     } catch {
       return PlanEvidence.create({
         exists: false,
@@ -66,11 +78,7 @@ export class MarkdownPlanDocumentReader implements PlanDocumentReaderPort {
 
     const hasQaSection = QA_SECTION_PATTERN.test(content);
     const qaComplete = this.isQaComplete(content, hasQaSection, expectedMode);
-    const planningModeMatch = this.detectPlanningModeMatch(
-      content,
-      hasQaSection,
-      expectedMode,
-    );
+    const planningModeMatch = this.detectPlanningModeMatch(content, hasQaSection, expectedMode);
 
     return PlanEvidence.create({
       exists: true,
@@ -79,16 +87,12 @@ export class MarkdownPlanDocumentReader implements PlanDocumentReaderPort {
     });
   }
 
-  private isQaComplete(
-    content: string,
-    hasQaSection: boolean,
-    expectedMode: PlanningMode,
-  ): boolean {
+  private isQaComplete(content: string, hasQaSection: boolean, expectedMode: PlanningMode): boolean {
     if (expectedMode.requiresAnsweredQa()) {
       if (!hasQaSection) return false;
       const qaSection = this.extractQaSection(content);
       const questionCount = (qaSection.match(QUESTION_PATTERN) ?? []).length;
-      const answerCount = (qaSection.match(ANSWER_PATTERN) ?? []).length;
+      const answerCount = this.countAnswers(qaSection);
       return questionCount > 0 && questionCount === answerCount;
     }
 
@@ -99,16 +103,12 @@ export class MarkdownPlanDocumentReader implements PlanDocumentReaderPort {
     return true;
   }
 
-  private detectPlanningModeMatch(
-    content: string,
-    hasQaSection: boolean,
-    expectedMode: PlanningMode,
-  ): boolean {
+  private detectPlanningModeMatch(content: string, hasQaSection: boolean, expectedMode: PlanningMode): boolean {
     if (expectedMode.requiresAnsweredQa()) {
       if (!hasQaSection) return false;
       const qaSection = this.extractQaSection(content);
       const questionCount = (qaSection.match(QUESTION_PATTERN) ?? []).length;
-      const answerCount = (qaSection.match(ANSWER_PATTERN) ?? []).length;
+      const answerCount = this.countAnswers(qaSection);
       return questionCount > 0 && answerCount === questionCount;
     }
 
@@ -121,14 +121,42 @@ export class MarkdownPlanDocumentReader implements PlanDocumentReaderPort {
 
   private extractQaSection(content: string): string {
     const match = content.match(QA_SECTION_PATTERN);
-    if (!match || match.index === undefined) return '';
+    if (!match || match.index === undefined) return "";
 
     const start = match.index;
-    const nextSectionMatch = content.slice(start + match[0].length).match(/^## /m);
-    const end = nextSectionMatch?.index
-      ? start + match[0].length + nextSectionMatch.index
-      : content.length;
+    const sectionContentStart = start + match[0].length;
+    const sectionContent = content.slice(sectionContentStart);
+    const headingMarker = match[0].match(/^##+/)?.[0];
+    if (!headingMarker) return "";
+
+    const sameLevelHeadingPattern = new RegExp(`^${headingMarker}(?!#)[ \\t]+(.+)$`, "gm");
+    let end = content.length;
+    for (const headingMatch of sectionContent.matchAll(sameLevelHeadingPattern)) {
+      if (/^\[Question\](?:\s|$)/.test(headingMatch[1].trimStart())) continue;
+      if (headingMatch.index !== undefined) {
+        end = sectionContentStart + headingMatch.index;
+        break;
+      }
+    }
 
     return content.slice(start, end);
   }
+
+  private countAnswers(qaSection: string): number {
+    const legacyAnswerCount = (qaSection.match(LEGACY_ANSWER_PATTERN) ?? []).length;
+    let structuredAnswerCount = 0;
+
+    for (const answerMatch of qaSection.matchAll(STRUCTURED_ANSWER_PATTERN)) {
+      if (answerMatch.index === undefined) continue;
+      const bodyStart = answerMatch.index + answerMatch[0].length;
+      const followingContent = qaSection.slice(bodyStart);
+      const boundaryMatch = followingContent.match(ANSWER_BODY_BOUNDARY_PATTERN);
+      const answerBody =
+        boundaryMatch?.index === undefined ? followingContent : followingContent.slice(0, boundaryMatch.index);
+      const hasBody = answerBody.split(/\r?\n/).some((line) => line.trim().length > 0);
+      if (hasBody) structuredAnswerCount += 1;
+    }
+
+    return legacyAnswerCount + structuredAnswerCount;
+  }
 }

package/scripts/harness/phase2-extensions/presentation/handlers/check-freshness-handler.ts

@@ -2,8 +2,8 @@
  * @layer presentation
  * @unit phase2-extensions
  */
-import type { CheckDocFreshnessUseCase } from '../../application/usecases/check-doc-freshness-usecase.js';
-import { FreshnessResultFormatter } from '../formatters/freshness-result-formatter.js';
+import type { CheckDocFreshnessUseCase } from "../../application/usecases/check-doc-freshness-usecase.js";
+import { FreshnessResultFormatter } from "../formatters/freshness-result-formatter.js";
 
 export class CheckFreshnessHandler {
   private readonly formatter = new FreshnessResultFormatter();
@@ -12,26 +12,34 @@ export class CheckFreshnessHandler {
 
   async handle(args: string[]): Promise<{ exitCode: number; stdout: string }> {
     let targetPattern: string | undefined;
-    let format: 'text' | 'json' = 'text';
+    let format: "text" | "json" = "text";
     let dryRun = false;
 
     for (let index = 0; index < args.length; index += 1) {
       const arg = args[index];
-      if (arg === '--pattern') {
+      if (arg === "--pattern") {
         targetPattern = args[index + 1];
         index += 1;
-      } else if (arg === '--format') {
-        format = (args[index + 1] as 'text' | 'json') ?? 'text';
+      } else if (arg === "--format") {
+        format = (args[index + 1] as "text" | "json") ?? "text";
         index += 1;
-      } else if (arg === '--dry-run') {
+      } else if (arg === "--dry-run") {
         dryRun = true;
       }
     }
 
     const result = await this.useCase.execute({ targetPattern, format, dryRun });
+    // WI-359: `--dry-run` は「副作用なしで診断のみ」（skills/doc-health-checker/SKILL.md）であり、
+    // シナリオ仕様 SC-P2-002 の期待結果も exit 0 である。従来は dryRun をパースだけして
+    // 捨てており、error 閾値超過の文書が 1 件でもあると診断のみの実行でも exit 1 になっていた。
+    // その結果 SC-P2-002 の e2e は「コードを一切変えなくても暦の経過だけで落ちる」状態になっていた。
+    // 実運用の鮮度ゲートは L4-004（`validate --layer L4`）が担い、そちらは本 handler を
+    // 経由しないため、dry-run を report-only にしてもゲートは緩まない。
+    // フラグ未指定時の exit 1 は従来どおり維持する。
+    const exitCode = dryRun ? 0 : result.summary.error > 0 ? 1 : 0;
     return {
-      exitCode: result.summary.error > 0 ? 1 : 0,
-      stdout: format === 'json' ? this.formatter.formatJson(result) : this.formatter.formatText(result),
+      exitCode,
+      stdout: format === "json" ? this.formatter.formatJson(result) : this.formatter.formatText(result),
     };
   }
 }

package/scripts/harness/quick-mode/application/ports/file-existence-port.ts

@@ -0,0 +1,15 @@
+/**
+ * @layer application
+ * @unit quick-mode
+ * @work-item-id WI-334
+ *
+ * ファイルの存在確認を行う Application 層 Port。
+ * ClassifyChangeCategoryUseCase が targetChanges を受け取らない経路
+ * （CLI: check-change-category --paths）で changeKind を CREATE/MODIFY と
+ * 推定するために使用する。hook 経路（beforeContent=null → CREATE）との
+ * 判定一致を保証する目的で導入された（WI-334）。
+ */
+
+export interface FileExistencePort {
+  exists(filePath: string): Promise<boolean>;
+}