phasegate 0.283.0 → 0.315.0

package/scripts/harness/agent-integration/infrastructure/adapters/file-system-full-mode-session-query-adapter.ts

@@ -1,17 +1,59 @@
 // @unit agent-integration
 // @layer infrastructure
 // @work-item-id WI-206
+// @work-item-id WI-348
+// @work-item-id WI-350
 
-import * as fs from 'node:fs/promises';
-import * as path from 'node:path';
-
+import * as fs from "node:fs/promises";
+import * as path from "node:path";
+import type { ConfigQueryPort } from "../../domain/ports/config-query-port.js";
 import type {
   FullModeSessionQueryInput,
   FullModeSessionQueryPort,
   FullModeSessionQueryResult,
-} from '../../domain/ports/full-mode-session-query-port.js';
-import type { ConfigQueryPort } from '../../domain/ports/config-query-port.js';
-import { WriteTargetScope } from '../../domain/value-objects/write-target-scope.js';
+} from "../../domain/ports/full-mode-session-query-port.js";
+import { WriteTargetScope } from "../../domain/value-objects/write-target-scope.js";
+
+/**
+ * quick-mode の `ChangeCategoryValue` と同一の語彙。
+ * 照合相手は quick-mode 分類結果（`dominantCategory`）なので、この語彙以外は
+ * session.json 側の書式誤りとして扱う。
+ * `session begin` 側との同期は
+ * `__tests__/integration/harness-api/session-begin-allowed-categories.integration.test.ts`
+ * が検証する（WI-348）。
+ */
+const KNOWN_CHANGE_CATEGORIES: ReadonlySet<string> = new Set([
+  "bugfix",
+  "docs",
+  "test",
+  "config",
+  "feature",
+  "domain",
+  "api",
+]);
+
+/**
+ * WI-348: v0.301.0 以前の `phasegate session begin` は allowedCategories に
+ * レイヤー名（domain / application / infrastructure / presentation / config）を
+ * 書き出していた。この語彙は ChangeCategory と domain / config でしか交差せず、
+ * 旧形式 session.json をそのまま照合すると feature / api / bugfix などが
+ * 恒常的に拒否され session が事実上無力化する。
+ *
+ * `session begin` はカテゴリ指定フラグを持たず常に定数を書き出すため、
+ * 「ChangeCategory 語彙に無い値が混ざっている」= 旧形式生成物と判断できる。
+ * その場合のみ全カテゴリ許可へ正規化する。
+ * 全要素が既知カテゴリなら手編集による意図的な絞り込みとみなし原文を尊重する。
+ * unit / 期限 / target path のスコープ判定は正規化後も従来どおり効く。
+ */
+function normalizeAllowedCategories(rawCategories: readonly string[]): readonly string[] {
+  if (rawCategories.length === 0) {
+    return rawCategories;
+  }
+  if (rawCategories.every((value) => KNOWN_CHANGE_CATEGORIES.has(value))) {
+    return rawCategories;
+  }
+  return [...KNOWN_CHANGE_CATEGORIES];
+}
 
 interface FullModeSessionDocument {
   readonly mode?: unknown;
@@ -35,52 +77,62 @@ export class FileSystemFullModeSessionQueryAdapter implements FullModeSessionQue
   async check(input: FullModeSessionQueryInput): Promise<FullModeSessionQueryResult> {
     let document: FullModeSessionDocument;
     try {
-      const raw = await fs.readFile(path.join(this.options.rootDir, '.phasegate', 'session.json'), 'utf8');
+      const raw = await fs.readFile(path.join(this.options.rootDir, ".phasegate", "session.json"), "utf8");
       document = JSON.parse(raw) as FullModeSessionDocument;
     } catch {
-      return { active: false, allowed: false, reason: 'session marker not found or unreadable' };
+      return { active: false, allowed: false, reason: "session marker not found or unreadable" };
     }
 
-    if (document.mode !== 'full') {
-      return { active: false, allowed: false, reason: 'session mode is not full' };
+    if (document.mode !== "full") {
+      return { active: false, allowed: false, reason: "session mode is not full" };
     }
-    if (typeof document.unit !== 'string' || document.unit === '') {
-      return { active: true, allowed: false, reason: 'session unit is missing' };
+    if (typeof document.unit !== "string" || document.unit === "") {
+      return { active: true, allowed: false, reason: "session unit is missing" };
     }
-    if (typeof document.workItemId !== 'string' || !/^WI-\d+$/.test(document.workItemId)) {
-      return { active: true, allowed: false, reason: 'session work item is invalid' };
+    if (typeof document.workItemId !== "string" || !/^WI-\d+$/.test(document.workItemId)) {
+      return { active: true, allowed: false, reason: "session work item is invalid" };
     }
-    if (typeof document.expiresAt !== 'string' || Number.isNaN(Date.parse(document.expiresAt))) {
-      return { active: true, allowed: false, reason: 'session expiry is invalid' };
+    if (typeof document.expiresAt !== "string" || Number.isNaN(Date.parse(document.expiresAt))) {
+      return { active: true, allowed: false, reason: "session expiry is invalid" };
     }
     if ((this.options.now?.() ?? new Date()).getTime() >= Date.parse(document.expiresAt)) {
       return {
         active: true,
         allowed: false,
-        reason: 'session expired',
+        reason: "session expired",
         workItemId: document.workItemId,
         unit: document.unit,
         expiresAt: document.expiresAt,
       };
     }
-    if (!Array.isArray(document.allowedCategories) || document.allowedCategories.some((value) => typeof value !== 'string')) {
-      return { active: true, allowed: false, reason: 'session allowedCategories is invalid' };
+    if (
+      !Array.isArray(document.allowedCategories) ||
+      document.allowedCategories.some((value) => typeof value !== "string")
+    ) {
+      return { active: true, allowed: false, reason: "session allowedCategories is invalid" };
     }
-    if (input.dominantCategory === undefined || !document.allowedCategories.includes(input.dominantCategory)) {
+    const allowedCategories = normalizeAllowedCategories(document.allowedCategories as readonly string[]);
+    if (input.dominantCategory === undefined || !allowedCategories.includes(input.dominantCategory)) {
       return {
         active: true,
         allowed: false,
-        reason: `category ${input.dominantCategory ?? '<unknown>'} is not allowed by session`,
+        reason: `category ${input.dominantCategory ?? "<unknown>"} is not allowed by session`,
         workItemId: document.workItemId,
         unit: document.unit,
         expiresAt: document.expiresAt,
       };
     }
-    if (input.unitId === undefined || input.unitId !== document.unit) {
+    // WI-350: unit を持たないパス（プロジェクト直下のファイル・__tests__ 配下など）は
+    // 集約 unitId が undefined になる。旧実装はこれを即拒否していたが、
+    // 直下の allTargetPathsBelongToUnit は unitless パスを許容しており内部矛盾していた。
+    // 集約チェックは「unitId が定義済みかつ session unit と不一致」の場合のみ拒否し、
+    // unitless は per-path チェックへ委ねる。unit 付きパスが 1 つでも混ざれば
+    // per-path 側が従来どおり拒否するため、unit 境界は緩まない。
+    if (input.unitId !== undefined && input.unitId !== document.unit) {
       return {
         active: true,
         allowed: false,
-        reason: `target unit ${input.unitId ?? '<unknown>'} does not match session unit ${document.unit}`,
+        reason: `target unit ${input.unitId} does not match session unit ${document.unit}`,
         workItemId: document.workItemId,
         unit: document.unit,
         expiresAt: document.expiresAt,

package/scripts/harness/agent-integration/infrastructure/adapters/harness-config-config-query-adapter.ts

@@ -83,9 +83,24 @@ export class HarnessConfigConfigQueryAdapter implements ConfigQueryPort {
     // GitHub #40: config が JSON として壊れている場合に hook プロセス全体を throw で
     // 落とすと、エージェントの全ツール呼び出しが遮断され config の修復自体が不能になる。
     // main.ts の ConfigPersistenceError と同じ意味論（警告 + 既定値で続行）に揃える。
-    // ファイル不在等の fs エラーは既存契約どおり throw する（上流の catch が defaults を
-    // 適用する）。gated スコープへの書き込みは phase-gate 側が fail-closed でブロックする。
-    const raw = fs.readFileSync(this.configPath, "utf8");
+    // ADR-038 §4 G1 / WI-333: config **不在**（ENOENT）も同じデッドロックを起こすため、
+    // invalid-json と同じ「警告 + 既定値で続行」の fail-open とする。config を作成する
+    // Write 自体が遮断されない（自己修復経路が残る）ことがこの adapter の契約。
+    // それ以外の fs エラー（EACCES 等の真の異常）は従来どおり throw する。
+    // gated スコープへの書き込みは phase-gate 側が既定設定で fail-closed を維持する。
+    let raw: string;
+    try {
+      raw = fs.readFileSync(this.configPath, "utf8");
+    } catch (error) {
+      if ((error as NodeJS.ErrnoException).code === "ENOENT") {
+        process.stderr.write(
+          `Warning: phasegate.config.json not found at ${this.configPath}; continuing with default hook settings so self-repair stays possible (run \`phasegate init\` to generate one).\n`,
+        );
+        this.cachedConfig = {};
+        return this.cachedConfig;
+      }
+      throw error;
+    }
     let doc: HarnessConfigDocument;
     try {
       doc = JSON.parse(raw) as HarnessConfigDocument;

package/scripts/harness/agent-integration/presentation/pre-tool-use-hook.ts

@@ -4,6 +4,8 @@
  * @work-item-id WI-202 / WI-204
  * @work-item-id WI-206
  * @work-item-id WI-208
+ * @work-item-id WI-345
+ * @work-item-id WI-347
  *
  * PreToolUse Hook Adapter
  * Claude Code の PreToolUse Hook エントリポイント
@@ -80,8 +82,10 @@ function projectRootForConfig(configPath: string): string {
   return path.basename(configDir) === '.phasegate-local' ? path.dirname(configDir) : configDir;
 }
 
-function isProjectExternalAbsolutePath(filePath: string): boolean {
-  return path.isAbsolute(filePath);
+function isProjectExternalPath(filePath: string, cwd: string, projectRoot: string): boolean {
+  const resolvedPath = path.resolve(cwd, filePath);
+  const relativePath = path.relative(projectRoot, resolvedPath);
+  return relativePath === '..' || relativePath.startsWith(`..${path.sep}`) || path.isAbsolute(relativePath);
 }
 
 async function main(): Promise<void> {
@@ -141,15 +145,29 @@ async function main(): Promise<void> {
     const bashTargets = extractor.extract(input.tool_input.command);
     if (bashTargets.length > 0) {
       targetFilePaths.push(...bashTargets.map(toRelative));
+      const recordedTargetPaths = new Set(targetChanges.map((change) => change.filePath));
+      const bashTargetChanges = await Promise.all(
+        bashTargets.map((bashTarget) => buildBashTargetChange(cwd, bashTarget, toRelative)),
+      );
+      for (const change of bashTargetChanges) {
+        if (!recordedTargetPaths.has(change.filePath)) {
+          targetChanges.push(change);
+          recordedTargetPaths.add(change.filePath);
+        }
+      }
       effectiveToolName = 'Write';
     }
   }
 
   try {
     const configPath = await findConfigPath(cwd);
-    const projectTargetFilePaths = targetFilePaths.filter((filePath) => !isProjectExternalAbsolutePath(filePath));
-    const projectTargetChanges = targetChanges.filter((change) => !isProjectExternalAbsolutePath(change.filePath));
     const projectRoot = projectRootForConfig(configPath);
+    const projectTargetFilePaths = targetFilePaths.filter(
+      (filePath) => !isProjectExternalPath(filePath, cwd, projectRoot),
+    );
+    const projectTargetChanges = targetChanges.filter(
+      (change) => !isProjectExternalPath(change.filePath, cwd, projectRoot),
+    );
     const configQueryPort = new HarnessConfigConfigQueryAdapter(configPath);
     const phaseGateQueryPort = new PhaseGateQueryAdapter();
     const storyReflectionQueryPort = new FileSystemStoryReflectionQueryAdapter({
@@ -157,7 +175,8 @@ async function main(): Promise<void> {
       configPath,
     });
     const fullModeRequirementQueryPort = new QuickModeFullModeRequirementAdapter({
-      classifyUseCaseFactory: () => createQuickModeCompositionRoot().classifyUseCase,
+      classifyUseCaseFactory: () =>
+        createQuickModeCompositionRoot({ configPath, rootDir: projectRoot }).classifyUseCase,
     });
     const baselineGrandfatherQueryPort = new CiGovernanceBaselineGrandfatherAdapter({
       baseDir: projectRoot,
@@ -222,7 +241,7 @@ async function buildTargetChanges(
   toRelative: (p: string) => string,
 ): Promise<TargetChange[]> {
   const toolInput = input.tool_input;
-  if (toolInput === undefined) {
+  if (toolInput == null || typeof toolInput !== 'object') {
     return [];
   }
 
@@ -258,6 +277,32 @@ async function readExistingContent(cwd: string, filePath: string): Promise<strin
   }
 }
 
+async function buildBashTargetChange(
+  cwd: string,
+  rawPath: string,
+  toRelative: (p: string) => string,
+): Promise<TargetChange> {
+  const filePath = toRelative(rawPath);
+  if (rawPath.includes('$') || rawPath.includes('`') || rawPath.startsWith('~')) {
+    // hook ではシェル展開後の実パスを解決できないため、存在チェックを避けて MODIFY 既定にする。
+    return { filePath };
+  }
+  const absolutePath = path.isAbsolute(rawPath) ? rawPath : path.join(cwd, rawPath);
+  try {
+    await fs.stat(absolutePath);
+    return { filePath };
+  } catch (error) {
+    const code = (error as NodeJS.ErrnoException).code;
+    if (code === 'ENOENT' || code === 'ENOTDIR') {
+      // Bash では afterContent を取得できないため、空文字を CREATE 判定用の
+      // 「変更後内容あり」sentinel として渡す。実ファイルへの書き込みは行わない。
+      return { filePath, beforeContent: null, afterContent: '' };
+    }
+    // 権限エラー等で存在を確認できない場合は従来どおり MODIFY 既定（安全側）。
+    return { filePath };
+  }
+}
+
 main().catch((error) => {
   process.stderr.write(`予期しないエラー: ${String(error)}\n`);
   process.exit(2);

package/scripts/harness/ci-governance/composition-root.ts

@@ -126,11 +126,11 @@ export function buildCiGovernance(baseDir: string, harnessRoot: string = baseDir
 
   // Handlers
   const generateCiTemplateHandler = new GenerateCiTemplateHandler(generateCiTemplateUseCase, renderCiTemplateUseCase);
-  const migrateAgentsMdHandler = new MigrateAgentsMdHandler(migrateAgentsMdUseCase);
+  const migrateAgentsMdHandler = new MigrateAgentsMdHandler(migrateAgentsMdUseCase, validatePointersUseCase);
   const refreshAgentContextHandler = new RefreshAgentContextHandler(refreshAgentContextUseCase);
   const refreshClaudeMdHandler = new RefreshClaudeMdHandler(refreshClaudeMdUseCase);
   const checkAgentContextHandler = new CheckAgentContextHandler(checkAgentContextUseCase);
-  const checkRepetitionHandler = new CheckRepetitionHandler(checkEscalationUseCase);
+  const checkRepetitionHandler = new CheckRepetitionHandler(checkEscalationUseCase, resetRepetitionUseCase);
   const createBaselineHandler = new CreateBaselineHandler(createBaselineUseCase);
   const scaffoldDesignHandler = new ScaffoldDesignHandler(scaffoldDesignUseCase);
 

package/scripts/harness/ci-governance/domain/services/claude-md-composer.ts

@@ -1,6 +1,7 @@
 /**
  * @layer domain
  * @unit ci-governance
+ * @work-item-id WI-331
  */
 
 export interface ClaudeMdTemplateValues {
@@ -9,20 +10,28 @@ export interface ClaudeMdTemplateValues {
   readonly presets: readonly string[];
 }
 
-const USER_SECTION_START = '<!-- phasegate:user-section:start -->';
-const USER_SECTION_END = '<!-- phasegate:user-section:end -->';
-const DEFAULT_USER_SECTION = 'Project-specific agent instructions go here.';
+const USER_SECTION_START = "<!-- phasegate:user-section:start -->";
+const USER_SECTION_END = "<!-- phasegate:user-section:end -->";
+const DEFAULT_USER_SECTION = "Project-specific agent instructions go here.";
 
 export class ClaudeMdComposer {
+  // Rebuilds CLAUDE.md from the current template, carrying over the existing
+  // user-section body. Because the whole file is re-rendered, a legacy file
+  // whose user-section still sits inside the managed block (pre-WI-331
+  // template shape) is migrated to the current outside-of-block structure.
   compose(template: string, existing: string | null, values: ClaudeMdTemplateValues): string {
     const userSection = this.extractUserSection(existing) ?? DEFAULT_USER_SECTION;
-    return template
-      .replace('{{PHASEGATE_COMMANDS}}', this.toList(values.commands))
-      .replace('{{PHASEGATE_SKILLS}}', this.toList(values.skills))
-      .replace('{{PHASEGATE_PRESETS}}', this.toList(values.presets))
-      .replace('{{PHASEGATE_USER_SECTION}}', userSection)
-      .replace(/\n{3,}/g, '\n\n')
-      .trimEnd() + '\n';
+    const rendered = template
+      .replace("{{PHASEGATE_COMMANDS}}", this.toList(values.commands))
+      .replace("{{PHASEGATE_SKILLS}}", this.toList(values.skills))
+      .replace("{{PHASEGATE_PRESETS}}", this.toList(values.presets))
+      // Collapse blank-line runs before injecting the user body so
+      // user-authored spacing is preserved byte-for-byte.
+      .replace(/\n{3,}/g, "\n\n")
+      // Replacer function keeps `$`-sequences in the user body from being
+      // interpreted as String.replace substitution patterns.
+      .replace("{{PHASEGATE_USER_SECTION}}", () => userSection);
+    return `${rendered.trimEnd()}\n`;
   }
 
   private extractUserSection(existing: string | null): string | null {
@@ -34,6 +43,6 @@ export class ClaudeMdComposer {
   }
 
   private toList(values: readonly string[]): string {
-    return values.map((value) => `- \`${value}\``).join('\n');
+    return values.map((value) => `- \`${value}\``).join("\n");
   }
 }

package/scripts/harness/ci-governance/presentation/handlers/check-repetition-handler.ts

@@ -29,6 +29,13 @@ export class CheckRepetitionHandler {
   async handle(args: CheckRepetitionHandlerArgs): Promise<CheckRepetitionHandlerResult> {
     const { errorCode, reset = false, format = 'human' } = args;
 
+    if (errorCode.trim().length === 0) {
+      return {
+        exitCode: 1,
+        output: 'Error: --code <errorCode> is required. Usage: phasegate ci:check-repetition --code <errorCode> [--reset] [--json]',
+      };
+    }
+
     if (reset && this.resetUseCase) {
       const result = await this.resetUseCase.execute({ errorCode, confirmedResolution: true });
 
@@ -49,9 +56,10 @@ export class CheckRepetitionHandler {
     }
 
     const result = await this.checkUseCase.execute({ errorCode });
+    const exitCode = result.escalated === true ? 1 : 0;
 
     if (format === 'json') {
-      return { exitCode: result.exists ? 0 : 1, output: JSON.stringify(result, null, 2) };
+      return { exitCode, output: JSON.stringify(result, null, 2) };
     }
 
     const lines: string[] = [];
@@ -63,6 +71,6 @@ export class CheckRepetitionHandler {
       lines.push(`  Escalated: ${result.escalated}`);
     }
 
-    return { exitCode: result.exists ? 0 : 1, output: lines.join('\n') };
+    return { exitCode, output: lines.join('\n') };
   }
 }

package/scripts/harness/config-foundation/infrastructure/presets/minimal.json

@@ -21,7 +21,7 @@
     }
   },
   "quickMode": {
-    "allowedCategories": ["bugfix"],
+    "allowedCategories": ["bugfix", "docs", "test", "config"],
     "maintainedLayers": ["L1", "L2"],
     "relaxedGates": []
   },

package/scripts/harness/config-foundation/infrastructure/presets/standard.json

@@ -21,7 +21,7 @@
     }
   },
   "quickMode": {
-    "allowedCategories": ["bugfix"],
+    "allowedCategories": ["bugfix", "docs", "test", "config"],
     "maintainedLayers": ["L1", "L2"],
     "relaxedGates": []
   },

package/scripts/harness/config-foundation/infrastructure/presets/strict.json

@@ -21,7 +21,7 @@
     }
   },
   "quickMode": {
-    "allowedCategories": ["bugfix"],
+    "allowedCategories": ["bugfix", "docs", "test", "config"],
     "maintainedLayers": ["L1", "L2"],
     "relaxedGates": []
   },

package/scripts/harness/harness-api/domain/services/command-dispatch-service.ts

@@ -1,6 +1,6 @@
 // @layer domain
 // @unit harness-api
-// @work-item-id WI-108 / WI-114, WI-186, WI-318, WI-321, WI-328
+// @work-item-id WI-108 / WI-114, WI-186, WI-318, WI-321, WI-328, WI-339
 // command-dispatch-service.ts — CommandDispatchService Domain Service
 
 import type { ArtifactScannerPort } from "../ports/artifact-scanner-port.js";
@@ -10,7 +10,7 @@ import type { ImpactAnalysisPort } from "../ports/impact-analysis-port.js";
 import type { PhaseGateQueryPort } from "../ports/phase-gate-query-port.js";
 import type { ValidatorExecutionPort } from "../ports/validator-execution-port.js";
 import { CheckReadyResult } from "../value-objects/check-ready-result.js";
-import { CiCheckResult } from "../value-objects/ci-check-result.js";
+import { CiCheckResult, type ValidatorCheckItem } from "../value-objects/ci-check-result.js";
 import { DriftReportSummary } from "../value-objects/drift-report-summary.js";
 import { type ExitCode, HarnessApiResponse, type HarnessError } from "../value-objects/harness-api-response.js";
 import type {
@@ -76,6 +76,15 @@ function summarizeLayerResults(
   return result;
 }
 
+/**
+ * WI-339: status の layer 集計へ渡す前に、ci-check / complete-check と同じ
+ * CiCheckResult 投影で raw passed を実効 passed に正規化する。
+ * failOnWarning は既存 harness-api 経路と同じ既定値 false を使用する。
+ */
+function toEffectiveValidatorResults(items: readonly ValidatorCheckItem[]): readonly ValidatorCheckItem[] {
+  return items.length === 0 ? items : CiCheckResult.fromResults(items).validatorResults;
+}
+
 function hasEnabledLiveFailure(
   enabledLayers: readonly LayerId[],
   liveValidationByLayer: Partial<Record<LayerId, LiveValidationState>>,
@@ -248,7 +257,7 @@ export class CommandDispatchService {
             this.ports.validatorExecutionPort.runAllValidators(),
           ]);
           liveValidationByLayer.L1 = lintResult.passed ? "pass" : "fail";
-          Object.assign(liveValidationByLayer, summarizeLayerResults(validatorResults));
+          Object.assign(liveValidationByLayer, summarizeLayerResults(toEffectiveValidatorResults(validatorResults)));
         } catch {
           liveValidationByLayer.L1 = liveValidationByLayer.L1 ?? "error";
         }

package/scripts/harness/harness-api/domain/value-objects/ci-check-result.ts

@@ -1,8 +1,10 @@
 // @layer domain
 // @unit harness-api
 // @work-item-id WI-307
+// @work-item-id WI-332
 // ci-check-result.ts — CiCheckResult Value Object
 
+import { isEffectivelyPassed } from "../../../validator-system/domain/services/effective-severity-policy.js";
 import type { HarnessError } from "./harness-api-response.js";
 
 export interface ValidatorCheckItem {
@@ -19,28 +21,6 @@ export interface CiCheckResultProps {
   failOnWarning?: boolean;
 }
 
-/**
- * WI-260 / ADR-017: validator 1件が「実質 pass」か判定する severity-aware ルール。
- * `validate` 経路の AggregateValidationResultsUseCase と同一の判定式を harness-api の
- * domain VO 側に持たせ、ci-check と validate の集約挙動を一致させる（レイヤー越境回避のため
- * usecase を直接 import せず判定ロジックを共有）。
- *
- * - skipped: 実質 pass
- * - passed=true: 実質 pass
- * - passed=false かつ error severity（!= warning）を含む: fail
- * - passed=false かつ errors=[]（severity 判定不能）: 安全側に倒して fail
- * - passed=false かつ warning のみ: failOnWarning=false（既定）で実質 pass、true で fail
- */
-function isEffectivelyPassed(item: ValidatorCheckItem, failOnWarning: boolean): boolean {
-  if (item.skipped || item.passed) return true;
-  const errors = item.errors ?? [];
-  const hasNonWarningError = errors.some((e) => e.severity !== "warning");
-  const hasWarnings = errors.some((e) => e.severity === "warning");
-  const isEmptyFail = errors.length === 0;
-  const hasFail = isEmptyFail || hasNonWarningError || (failOnWarning && hasWarnings);
-  return !hasFail;
-}
-
 /**
  * WI-307: public ci-check projection の `passed` を aggregate policy と一致させる。
  * skipped は独立した green state のため raw passed を維持し、warning-only failure だけを

package/scripts/harness/harness-error/application/dto/create-harness-error-input.ts

@@ -8,11 +8,13 @@ export interface CreateHarnessErrorInput {
   readonly code: string;
   readonly message: string;
   readonly suggestion: string;
-  readonly severity?: 'error' | 'warning';
+  readonly severity?: "error" | "warning";
   readonly adrRef?: string;
   readonly fixExample?: string;
   readonly validatorId: string;
   readonly suggestedSkill?: string;
   readonly scaffoldCommand?: string;
   readonly templatePath?: string;
+  /** WI-335: 修復方式分類。省略時は ErrorDefinition の既定（未定義なら manual 扱い）。 */
+  readonly remediationType?: "mechanical" | "ai-assisted" | "manual";
 }

package/scripts/harness/harness-error/application/dto/harness-error-contract.ts

@@ -3,7 +3,7 @@
 
 export interface HarnessErrorContract {
   readonly code: string;
-  readonly severity: 'error' | 'warning';
+  readonly severity: "error" | "warning";
   readonly message: string;
   readonly suggestion: string;
   readonly adr_ref?: string;
@@ -11,4 +11,6 @@ export interface HarnessErrorContract {
   readonly suggested_skill?: string;
   readonly scaffold_command?: string;
   readonly template_path?: string;
+  /** WI-335: 修復方式分類（mechanical / ai-assisted / manual）。未設定は manual 扱い。 */
+  readonly remediation_type?: "mechanical" | "ai-assisted" | "manual";
 }

package/scripts/harness/harness-error/application/mappers/harness-error-contract-mapper.ts

@@ -4,8 +4,9 @@
  *
  * HarnessErrorContract への投影Mapper
  */
-import type { HarnessErrorContract } from '../dto/harness-error-contract.js';
-import type { HarnessError } from '../../domain/value-objects/harness-error.js';
+
+import type { HarnessError } from "../../domain/value-objects/harness-error.js";
+import type { HarnessErrorContract } from "../dto/harness-error-contract.js";
 
 export class HarnessErrorContractMapper {
   toReadonlyContract(harnessError: HarnessError): Readonly<HarnessErrorContract> {
@@ -14,21 +15,12 @@ export class HarnessErrorContractMapper {
       severity: harnessError.severity.toString(),
       message: harnessError.message,
       suggestion: harnessError.suggestion,
-      ...(harnessError.adrRef !== null
-        ? { adr_ref: harnessError.adrRef.toString() }
-        : {}),
-      ...(harnessError.fixExample !== null
-        ? { fix_example: harnessError.fixExample.toString() }
-        : {}),
-      ...(harnessError.suggestedSkill !== null
-        ? { suggested_skill: harnessError.suggestedSkill }
-        : {}),
-      ...(harnessError.scaffoldCommand !== null
-        ? { scaffold_command: harnessError.scaffoldCommand }
-        : {}),
-      ...(harnessError.templatePath !== null
-        ? { template_path: harnessError.templatePath }
-        : {}),
+      ...(harnessError.adrRef !== null ? { adr_ref: harnessError.adrRef.toString() } : {}),
+      ...(harnessError.fixExample !== null ? { fix_example: harnessError.fixExample.toString() } : {}),
+      ...(harnessError.suggestedSkill !== null ? { suggested_skill: harnessError.suggestedSkill } : {}),
+      ...(harnessError.scaffoldCommand !== null ? { scaffold_command: harnessError.scaffoldCommand } : {}),
+      ...(harnessError.templatePath !== null ? { template_path: harnessError.templatePath } : {}),
+      ...(harnessError.remediationType !== null ? { remediation_type: harnessError.remediationType } : {}),
     };
 
     return Object.freeze(contract);

package/scripts/harness/harness-error/application/usecases/create-harness-error-use-case.ts

@@ -4,10 +4,11 @@
  *
  * 単一draftを HarnessErrorContract へ変換するUseCase
  */
-import type { CreateHarnessErrorInput } from '../dto/create-harness-error-input.js';
-import type { HarnessErrorContract } from '../dto/harness-error-contract.js';
-import type { HarnessErrorContractMapper } from '../mappers/harness-error-contract-mapper.js';
-import type { HarnessErrorFactory } from '../../domain/services/harness-error-factory.js';
+
+import type { HarnessErrorFactory } from "../../domain/services/harness-error-factory.js";
+import type { CreateHarnessErrorInput } from "../dto/create-harness-error-input.js";
+import type { HarnessErrorContract } from "../dto/harness-error-contract.js";
+import type { HarnessErrorContractMapper } from "../mappers/harness-error-contract-mapper.js";
 
 export interface CreateHarnessErrorUseCaseDeps {
   readonly harnessErrorFactory: HarnessErrorFactory;
@@ -23,9 +24,7 @@ export class CreateHarnessErrorUseCase {
     this.contractMapper = deps.contractMapper;
   }
 
-  async execute(
-    input: CreateHarnessErrorInput
-  ): Promise<Readonly<HarnessErrorContract>> {
+  async execute(input: CreateHarnessErrorInput): Promise<Readonly<HarnessErrorContract>> {
     const harnessError = await this.harnessErrorFactory.create({
       code: input.code,
       message: input.message,
@@ -37,6 +36,7 @@ export class CreateHarnessErrorUseCase {
       suggestedSkill: input.suggestedSkill,
       scaffoldCommand: input.scaffoldCommand,
       templatePath: input.templatePath,
+      remediationType: input.remediationType,
     });
 
     return this.contractMapper.toReadonlyContract(harnessError);