phasegate 0.264.0 → 0.315.0

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/application/mappers/validator-system-config-mapper.ts

@@ -10,6 +10,7 @@
  * @work-item-id WI-300
  * @work-item-id WI-301
  * @work-item-id WI-302
+ * @work-item-id WI-320
  */
 import type { HarnessConfigV2 } from "../../domain/harness-config.js";
 
@@ -54,7 +55,10 @@ export function toValidatorSystemConfig(resolvedConfig: HarnessConfigV2 | undefi
   return {
     project: {
       preset: resolvedConfig.project.preset,
-      languages: resolvedConfig.project.languages ?? ["typescript"],
+      // WI-320 (github#39): 未宣言時に ["typescript"] を注入せず undefined のまま渡す。
+      // HarnessConfigValidatorConfigAdapter.getProjectLanguages() が未宣言時のみ
+      // ファイルシステム検出（WI-319）にフォールバックできるようにするため。
+      ...(resolvedConfig.project.languages === undefined ? {} : { languages: resolvedConfig.project.languages }),
     },
     paths: {
       designDocs: resolvedConfig.paths.designDocs,

package/scripts/harness/config-foundation/domain/harness-config.ts

@@ -5,6 +5,7 @@
  * @work-item-id WI-212
  * @work-item-id WI-219
  * @work-item-id WI-300
+ * @work-item-id WI-327
  */
 import { ConfigFoundationDomainError } from "./errors/config-foundation-domain-error.js";
 import { ConfigValidationError } from "./errors/config-validation-error.js";
@@ -48,13 +49,15 @@ export interface HarnessConfigSourceDocument {
     preset: PresetId;
     languages?: string[];
   };
-  layers: DeepPartial<HarnessConfigResolvedDocument["layers"]>;
-  quickMode: Partial<HarnessConfigResolvedDocument["quickMode"]>;
-  phaseDependencies: HarnessConfigResolvedDocument["phaseDependencies"];
-  planningMode: HarnessConfigResolvedDocument["planningMode"];
-  harnesses: Partial<HarnessConfigResolvedDocument["harnesses"]>;
-  paths: HarnessConfigResolvedDocument["paths"];
-  reporting: HarnessConfigResolvedDocument["reporting"];
+  // WI-327: project 以外の top-level セクションは手書き config で省略可。
+  // 省略時は PresetResolutionService が防御プリセット（project.preset）のデフォルト値で補完する。
+  layers?: DeepPartial<HarnessConfigResolvedDocument["layers"]>;
+  quickMode?: Partial<HarnessConfigResolvedDocument["quickMode"]>;
+  phaseDependencies?: HarnessConfigResolvedDocument["phaseDependencies"];
+  planningMode?: HarnessConfigResolvedDocument["planningMode"];
+  harnesses?: Partial<HarnessConfigResolvedDocument["harnesses"]>;
+  paths?: HarnessConfigResolvedDocument["paths"];
+  reporting?: HarnessConfigResolvedDocument["reporting"];
   ci?: HarnessConfigResolvedDocument["ci"];
   validate?: HarnessConfigResolvedDocument["validate"];
   preCommit?: Partial<HarnessConfigResolvedDocument["preCommit"]>;

package/scripts/harness/config-foundation/domain/services/preset-resolution-service.ts

@@ -3,6 +3,7 @@
  * @unit config-foundation
  * @work-item-id WI-212
  * @work-item-id WI-300
+ * @work-item-id WI-320
  */
 import { ConfigFoundationDomainError } from "../errors/config-foundation-domain-error.js";
 import type { HarnessConfigResolvedDocument, HarnessConfigSourceDocument } from "../harness-config.js";
@@ -163,7 +164,9 @@ export class PresetResolutionService {
       project: {
         name: sourceDocument.project.name,
         preset: sourceDocument.project.preset,
-        languages: sourceDocument.project.languages ?? ["typescript"],
+        // WI-320 (github#39): 未宣言時に ["typescript"] を注入しない。「未宣言」を resolved config まで
+        // 生存させ、validator-system adapter のファイルシステム言語検出（WI-319）を実 CLI 経路で有効にする。
+        ...(sourceDocument.project.languages === undefined ? {} : { languages: sourceDocument.project.languages }),
       },
       layers,
       quickMode: deepMerge(presetDefinition.quickMode, sourceDocument.quickMode, "quickMode"),

package/scripts/harness/config-foundation/domain/value-objects/project-config.ts

@@ -2,11 +2,12 @@
  * @layer domain
  * @unit config-foundation
  * @work-item-id WI-212
+ * @work-item-id WI-320
  *
  * ProjectConfig値オブジェクト - プロジェクト名とPresetを保持する
  */
-import { ConfigValidationError } from '../errors/config-validation-error.js';
-import { Preset } from './preset.js';
+import { ConfigValidationError } from "../errors/config-validation-error.js";
+import { Preset } from "./preset.js";
 
 interface ProjectConfigProps {
   readonly name: string;
@@ -17,29 +18,37 @@ interface ProjectConfigProps {
 export class ProjectConfig {
   readonly name: string;
   readonly preset: Preset;
-  readonly languages: readonly string[];
+  /**
+   * WI-320 (github#39): 宣言があった場合のみ保持する（undefined = 未宣言）。
+   * default `['typescript']` の注入をやめ、「未宣言」シグナルを下流
+   * （validator-system のファイルシステム言語検出、WI-319）まで生存させる。
+   */
+  readonly languages?: readonly string[];
 
   constructor(props: ProjectConfigProps) {
-    if (!props.name || props.name.trim() === '') {
-      throw new ConfigValidationError('ProjectConfig: name must not be empty');
-    }
-    const sourceLanguages = props.languages ?? ['typescript'];
-    if (sourceLanguages.length === 0) {
-      throw new ConfigValidationError('ProjectConfig: languages must not be empty');
+    if (!props.name || props.name.trim() === "") {
+      throw new ConfigValidationError("ProjectConfig: name must not be empty");
     }
-    const normalizedLanguages = sourceLanguages.map((language) => language.trim()).filter((language) => language.length > 0);
-    if (normalizedLanguages.length !== sourceLanguages.length) {
-      throw new ConfigValidationError('ProjectConfig: languages must contain non-empty strings');
+    if (props.languages !== undefined) {
+      if (props.languages.length === 0) {
+        throw new ConfigValidationError("ProjectConfig: languages must not be empty");
+      }
+      const normalizedLanguages = props.languages
+        .map((language) => language.trim())
+        .filter((language) => language.length > 0);
+      if (normalizedLanguages.length !== props.languages.length) {
+        throw new ConfigValidationError("ProjectConfig: languages must contain non-empty strings");
+      }
+      this.languages = Object.freeze([...normalizedLanguages]);
     }
     this.name = props.name;
     this.preset = props.preset;
-    this.languages = Object.freeze([...normalizedLanguages]);
     Object.freeze(this);
   }
 
   static create(raw: { name: string; preset: string; languages?: readonly string[] }): ProjectConfig {
     const preset = Preset.create(raw.preset);
-    return new ProjectConfig({ name: raw.name, preset, languages: raw.languages ?? ['typescript'] });
+    return new ProjectConfig({ name: raw.name, preset, languages: raw.languages });
   }
 
   rename(name: string): ProjectConfig {
@@ -51,9 +60,16 @@ export class ProjectConfig {
   }
 
   equals(other: ProjectConfig): boolean {
-    return this.name === other.name
-      && this.preset.equals(other.preset)
-      && this.languages.length === other.languages.length
-      && this.languages.every((language, index) => language === other.languages[index]);
+    return this.name === other.name && this.preset.equals(other.preset) && this.languagesEqual(other.languages);
+  }
+
+  private languagesEqual(otherLanguages: readonly string[] | undefined): boolean {
+    if (this.languages === undefined || otherLanguages === undefined) {
+      return this.languages === otherLanguages;
+    }
+    return (
+      this.languages.length === otherLanguages.length &&
+      this.languages.every((language, index) => language === otherLanguages[index])
+    );
   }
 }

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/config-foundation/infrastructure/repositories/file-system-config-repository.ts

@@ -2,19 +2,19 @@
  * @layer infrastructure
  * @unit config-foundation
  */
-import * as fs from 'node:fs/promises';
-import path from 'node:path';
-import type { ConfigRepositoryPort } from '../../domain/ports/config-repository-port.js';
+import * as fs from "node:fs/promises";
+import path from "node:path";
+import type { ConfigRepositoryPort } from "../../domain/ports/config-repository-port.js";
 
-const DEFAULT_CONFIG_FILE_NAME = 'phasegate.config.json';
-const PERSONAL_CONFIG_PATH = path.join('.phasegate-local', DEFAULT_CONFIG_FILE_NAME);
+const DEFAULT_CONFIG_FILE_NAME = "phasegate.config.json";
+const PERSONAL_CONFIG_PATH = path.join(".phasegate-local", DEFAULT_CONFIG_FILE_NAME);
 
 export class ConfigNotFoundError extends Error {
   readonly configPath: string;
 
   constructor(configPath: string) {
     super(`Config file not found: ${configPath}`);
-    this.name = 'ConfigNotFoundError';
+    this.name = "ConfigNotFoundError";
     this.configPath = configPath;
     Object.setPrototypeOf(this, new.target.prototype);
   }
@@ -24,15 +24,28 @@ export class ConfigPersistenceError extends Error {
   readonly configPath: string;
   readonly cause?: unknown;
 
-  constructor(configPath: string, cause?: unknown) {
-    super(`Failed to persist config: ${configPath}`);
-    this.name = 'ConfigPersistenceError';
+  constructor(configPath: string, cause?: unknown, message?: string) {
+    super(message ?? `Failed to persist config: ${configPath}`);
+    this.name = "ConfigPersistenceError";
     this.configPath = configPath;
     this.cause = cause;
     Object.setPrototypeOf(this, new.target.prototype);
   }
 }
 
+/**
+ * WI-325: load 経路の JSON パース失敗専用エラー。
+ * ConfigPersistenceError のサブクラスなので、既存の
+ * `instanceof ConfigPersistenceError` fail-open 判定（main.ts / WI-314）は無変更で動く。
+ */
+export class ConfigParseError extends ConfigPersistenceError {
+  constructor(configPath: string, cause?: unknown) {
+    super(configPath, cause, `Failed to parse config JSON: ${configPath}`);
+    this.name = "ConfigParseError";
+    Object.setPrototypeOf(this, new.target.prototype);
+  }
+}
+
 async function exists(targetPath: string): Promise<boolean> {
   try {
     await fs.access(targetPath);
@@ -69,14 +82,10 @@ async function findNearestConfig(startDirectory: string): Promise<string | null>
 
 export class FileSystemConfigRepository implements ConfigRepositoryPort {
   async load(configPath?: string): Promise<{ path: string; document: unknown }> {
-    const resolvedPath = configPath
-      ? path.resolve(configPath)
-      : await findNearestConfig(process.cwd());
+    const resolvedPath = configPath ? path.resolve(configPath) : await findNearestConfig(process.cwd());
 
     if (!resolvedPath) {
-      throw new ConfigNotFoundError(
-        path.resolve(process.cwd(), DEFAULT_CONFIG_FILE_NAME),
-      );
+      throw new ConfigNotFoundError(path.resolve(process.cwd(), DEFAULT_CONFIG_FILE_NAME));
     }
 
     if (!(await exists(resolvedPath))) {
@@ -84,7 +93,7 @@ export class FileSystemConfigRepository implements ConfigRepositoryPort {
     }
 
     try {
-      const raw = await fs.readFile(resolvedPath, 'utf8');
+      const raw = await fs.readFile(resolvedPath, "utf8");
 
       return {
         path: resolvedPath,
@@ -92,7 +101,7 @@ export class FileSystemConfigRepository implements ConfigRepositoryPort {
       };
     } catch (error) {
       if (error instanceof SyntaxError) {
-        throw new ConfigPersistenceError(resolvedPath, error);
+        throw new ConfigParseError(resolvedPath, error);
       }
 
       throw error;
@@ -104,7 +113,7 @@ export class FileSystemConfigRepository implements ConfigRepositoryPort {
 
     try {
       const serialized = `${JSON.stringify(document, null, 2)}\n`;
-      await fs.writeFile(resolvedPath, serialized, 'utf8');
+      await fs.writeFile(resolvedPath, serialized, "utf8");
     } catch (error) {
       throw new ConfigPersistenceError(resolvedPath, error);
     }

package/scripts/harness/config-foundation/infrastructure/schemas/harness-config-v2.schema.json

@@ -3,14 +3,7 @@
   "type": "object",
   "additionalProperties": false,
   "required": [
-    "project",
-    "layers",
-    "quickMode",
-    "phaseDependencies",
-    "planningMode",
-    "harnesses",
-    "paths",
-    "reporting"
+    "project"
   ],
   "properties": {
     "project": {

package/scripts/harness/config-foundation/infrastructure/schemas/harness-config-v3.schema.json

@@ -3,14 +3,7 @@
   "type": "object",
   "additionalProperties": false,
   "required": [
-    "project",
-    "layers",
-    "quickMode",
-    "phaseDependencies",
-    "planningMode",
-    "harnesses",
-    "paths",
-    "reporting"
+    "project"
   ],
   "$defs": {
     "effectCapability": {

package/scripts/harness/harness-api/domain/ports/config-query-port.ts

@@ -1,10 +1,20 @@
 // @layer domain
 // config-query-port.ts
 
-import type { PresetInfo, ConfigSummary, PhaseGateSummary } from '../value-objects/harness-status-summary.js';
+import type {
+  ConfigSummary,
+  LanguageInfo,
+  PhaseGateSummary,
+  PresetInfo,
+} from "../value-objects/harness-status-summary.js";
 
 export interface ConfigQueryPort {
   getPresetInfo(): Promise<PresetInfo>;
   getConfigSummary(): Promise<ConfigSummary>;
   getPhaseGateSummary(): Promise<PhaseGateSummary>;
+  /**
+   * WI-328: 実効言語リストと出所（declared / detected / fallback）を返す。
+   * 後方互換のため optional — 未実装アダプタでは status に languages を出さない。
+   */
+  getLanguageInfo?(): Promise<LanguageInfo>;
 }