phasegate 0.134.0 → 0.135.0

package/scripts/harness/validator-system/infrastructure/adapters/biome-ast-source-code-analyzer-adapter.ts

@@ -12,9 +12,23 @@ import { basename, join, relative, sep } from 'node:path';
 
 const HARNESS_ROOT = join(process.cwd(), 'scripts', 'harness');
 
+export interface BiomeAstSourceCodeAnalyzerAdapterOptions {
+  readonly sourceRoot?: string;
+  /** スキャン対象から除外するパスパターン（デフォルト: テストディレクトリを除外） */
+  readonly excludePattern?: RegExp;
+}
+
 export class BiomeAstSourceCodeAnalyzerAdapter implements SourceCodeAnalyzerPort {
+  private readonly sourceRoot: string;
+  private readonly excludePattern: RegExp;
+
+  constructor(options: BiomeAstSourceCodeAnalyzerAdapterOptions = {}) {
+    this.sourceRoot = options.sourceRoot ?? HARNESS_ROOT;
+    this.excludePattern = options.excludePattern ?? /__tests__\//;
+  }
+
   async analyzeExports(targetUnits?: readonly string[]): Promise<readonly SourceAnalysisResult[]> {
-    const targetFiles = await collectTargetFiles(targetUnits);
+    const targetFiles = await collectTargetFiles(this.sourceRoot, this.excludePattern, targetUnits);
     if (targetFiles.length === 0) return [];
 
     const program = ts.createProgram(targetFiles, {
@@ -30,7 +44,7 @@ export class BiomeAstSourceCodeAnalyzerAdapter implements SourceCodeAnalyzerPort
       const sourceFile = program.getSourceFile(filePath);
       if (!sourceFile) continue;
       results.push({
-        unitName: resolveUnitName(filePath),
+        unitName: resolveUnitName(this.sourceRoot, filePath),
         filePath,
         exports: extractExports(sourceFile),
         imports: extractImports(sourceFile),
@@ -134,26 +148,31 @@ function extractImports(sourceFile: ts.SourceFile): SourceAnalysisResult['import
   return imports;
 }
 
-async function collectTargetFiles(targetUnits?: readonly string[]): Promise<readonly string[]> {
+async function collectTargetFiles(
+  sourceRoot: string,
+  excludePattern: RegExp,
+  targetUnits?: readonly string[],
+): Promise<readonly string[]> {
   const roots = targetUnits && targetUnits.length > 0
-    ? targetUnits.map((unit) => join(HARNESS_ROOT, unit))
-    : [HARNESS_ROOT];
+    ? targetUnits.map((unit) => join(sourceRoot, unit))
+    : [sourceRoot];
 
   const files: string[] = [];
   for (const root of roots) {
-    files.push(...await walkTsFiles(root));
+    files.push(...await walkTsFiles(root, excludePattern));
   }
 
   return files;
 }
 
-async function walkTsFiles(root: string): Promise<string[]> {
+async function walkTsFiles(root: string, excludePattern: RegExp): Promise<string[]> {
   try {
     const entries = await readdir(root, { withFileTypes: true });
     const files = await Promise.all(entries.map(async (entry) => {
       const fullPath = join(root, entry.name);
+      if (excludePattern.test(fullPath)) return [];
       if (entry.isDirectory()) {
-        return walkTsFiles(fullPath);
+        return walkTsFiles(fullPath, excludePattern);
       }
       return fullPath.endsWith('.ts') ? [fullPath] : [];
     }));
@@ -163,8 +182,8 @@ async function walkTsFiles(root: string): Promise<string[]> {
   }
 }
 
-function resolveUnitName(filePath: string): string {
-  const relativePath = relative(HARNESS_ROOT, filePath);
+function resolveUnitName(sourceRoot: string, filePath: string): string {
+  const relativePath = relative(sourceRoot, filePath);
   const [firstSegment] = relativePath.split(sep);
   return firstSegment || basename(filePath);
 }

package/scripts/harness/validator-system/infrastructure/adapters/harness-config-validator-config-adapter.ts

@@ -6,6 +6,7 @@
  * HarnessConfigV2からLayerConfig VOを構築する
  */
 import { LayerConfig } from '../../domain/value-objects/layer-config.js';
+import { ValidatorId } from '../../domain/value-objects/validator-id.js';
 import type { ValidatorConfigPort } from '../../domain/ports/validator-config-port.js';
 
 export interface HarnessConfigLayers {
@@ -39,7 +40,7 @@ export class HarnessConfigValidatorConfigAdapter implements ValidatorConfigPort
     const defaultValidators: Record<string, string[]> = {
       L2: ['L2-001', 'L2-002', 'L2-003'],
       L3: ['L3-001', 'L3-002', 'L3-003', 'L3-004'],
-      L4: ['L4-001', 'L4-002', 'L4-003'],
+      L4: ['L4-001', 'L4-002', 'L4-003', 'L4-004', 'L4-005'],
     };
 
     const thresholds: Record<string, number> = {};
@@ -62,10 +63,18 @@ export class HarnessConfigValidatorConfigAdapter implements ValidatorConfigPort
     return LayerConfig.create({
       layer,
       enabled: layerData.enabled !== false,
-      validatorIds: layerData.validators ?? defaultValidators[layer],
+      validatorIds: (layerData.validators ?? defaultValidators[layer]).map((idOrName) => this.normalizeValidatorId(idOrName)),
       thresholds,
       strictOnly: layerData.strictOnly ?? strictOnly,
       preset,
     });
   }
+
+  private normalizeValidatorId(idOrName: string): string {
+    try {
+      return ValidatorId.create(idOrName).value;
+    } catch {
+      return ValidatorId.fromName(idOrName).value;
+    }
+  }
 }