fix: replace orchestrator-develop branch references with main

The orchestrator-develop branch no longer exists. Update all fallback clone commands and test fixtures to use main instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ci: set macOS builds to continue-on-error
2026-06-02 14:56:16 -07:00 · 2026-03-09 20:03:29 +00:00 · 2026-03-05 23:33:25 +00:00 · 2026-03-05 12:51:52 +00:00 · 2026-03-05 11:52:43 +00:00 · 2026-03-05 09:04:54 +00:00
17 changed files with 2605 additions and 1760 deletions
--- a/action.yml
+++ b/action.yml
@@ -279,30 +279,23 @@ inputs:
    description:
      '[Orchestrator] Specifies the repo for the unity builder. Useful if you forked the repo for testing, features, or
      fixes.'
-  gitIntegrityCheck:
-    description: 'Run git integrity checks before build (fsck, lock cleanup, submodule validation)'
+  testSuitePath:
+    description: 'Path to YAML test suite definition file'
    required: false
-    default: 'false'
-  gitAutoRecover:
-    description: 'Attempt automatic recovery if git corruption is detected'
+  testSuiteEvent:
+    description: 'CI event name for suite selection (pr, push, release)'
    required: false
-    default: 'false'
-  cleanReservedFilenames:
-    description: 'Remove Windows reserved filenames that cause Unity import loops'
+  testTaxonomyPath:
+    description: 'Path to custom taxonomy definition YAML'
    required: false
-    default: 'false'
-  buildArchiveEnabled:
-    description: 'Archive build output after successful build'
+  testResultFormat:
+    description: 'Test result output format: junit, json, or both'
    required: false
-    default: 'false'
-  buildArchivePath:
-    description: 'Path to store build archives'
+    default: 'junit'
+  testResultPath:
+    description: 'Directory for structured test result output'
    required: false
-    default: './build-archives'
-  buildArchiveRetention:
-    description: 'Days to retain build archives before cleanup'
-    required: false
-    default: '30'
+    default: './test-results'

 outputs:
  volume:
--- a/dist/index.js
+++ b/dist/index.js
--- a/dist/index.js.map
+++ b/dist/index.js.map
--- a/src/index.ts
+++ b/src/index.ts
@@ -3,7 +3,7 @@ import { Action, BuildParameters, Cache, Orchestrator, Docker, ImageTag, Output
 import { Cli } from './model/cli/cli';
 import MacBuilder from './model/mac-builder';
 import PlatformSetup from './model/platform-setup';
-import { BuildReliabilityService } from './model/orchestrator/services/reliability';
+import { TestWorkflowService } from './model/orchestrator/services/test-workflow';

 async function runMain() {
  try {
@@ -15,38 +15,28 @@ async function runMain() {
    Action.checkCompatibility();
    Cache.verify();

-    // Always configure git environment for CI reliability
-    BuildReliabilityService.configureGitEnvironment();
-
    const { workspace, actionFolder } = Action;

    const buildParameters = await BuildParameters.create();
-    const baseImage = new ImageTag(buildParameters);

-    // Pre-build reliability checks
-    if (buildParameters.gitIntegrityCheck) {
-      core.info('Running git integrity checks...');
+    // If a test suite path is provided, use the test workflow engine
+    // instead of the standard build execution path
+    if (buildParameters.testSuitePath) {
+      core.info('[TestWorkflow] Test suite path detected, using test workflow engine');
+      const results = await TestWorkflowService.executeTestSuite(buildParameters.testSuitePath, buildParameters);

-      const isHealthy = BuildReliabilityService.checkGitIntegrity(workspace);
-      BuildReliabilityService.cleanStaleLockFiles(workspace);
-      BuildReliabilityService.validateSubmoduleBackingStores(workspace);
-
-      if (buildParameters.cleanReservedFilenames) {
-        BuildReliabilityService.cleanReservedFilenames(buildParameters.projectPath);
+      const totalFailed = results.reduce((sum, r) => sum + r.failed, 0);
+      if (totalFailed > 0) {
+        core.setFailed(`Test workflow completed with ${totalFailed} failure(s)`);
+      } else {
+        core.info('[TestWorkflow] All test runs passed');
      }

-      if (!isHealthy && buildParameters.gitAutoRecover) {
-        core.info('Git corruption detected, attempting automatic recovery...');
-        const recovered = BuildReliabilityService.recoverCorruptedRepo(workspace);
-        if (!recovered) {
-          core.warning('Automatic recovery failed. Build may encounter issues.');
-        }
-      }
-    } else if (buildParameters.cleanReservedFilenames) {
-      // cleanReservedFilenames can run independently of gitIntegrityCheck
-      BuildReliabilityService.cleanReservedFilenames(buildParameters.projectPath);
+      return;
    }

+    const baseImage = new ImageTag(buildParameters);
+
    let exitCode = -1;

    if (buildParameters.providerStrategy === 'local') {
@@ -65,13 +55,6 @@ async function runMain() {
      exitCode = 0;
    }

-    // Post-build: archive and enforce retention
-    if (buildParameters.buildArchiveEnabled && exitCode === 0) {
-      core.info('Archiving build output...');
-      BuildReliabilityService.archiveBuildOutput(buildParameters.buildPath, buildParameters.buildArchivePath);
-      BuildReliabilityService.enforceRetention(buildParameters.buildArchivePath, buildParameters.buildArchiveRetention);
-    }
-
    // Set output
    await Output.setBuildVersion(buildParameters.buildVersion);
    await Output.setAndroidVersionCode(buildParameters.androidVersionCode);
--- a/src/model/build-parameters.ts
+++ b/src/model/build-parameters.ts
@@ -106,12 +106,12 @@ class BuildParameters {
  public cacheUnityInstallationOnMac!: boolean;
  public unityHubVersionOnMac!: string;
  public dockerWorkspacePath!: string;
-  public gitIntegrityCheck!: boolean;
-  public gitAutoRecover!: boolean;
-  public cleanReservedFilenames!: boolean;
-  public buildArchiveEnabled!: boolean;
-  public buildArchivePath!: string;
-  public buildArchiveRetention!: number;
+
+  public testSuitePath!: string;
+  public testSuiteEvent!: string;
+  public testTaxonomyPath!: string;
+  public testResultFormat!: string;
+  public testResultPath!: string;

  public static shouldUseRetainedWorkspaceMode(buildParameters: BuildParameters) {
    return buildParameters.maxRetainedWorkspaces > 0 && Orchestrator.lockedWorkspace !== ``;
@@ -248,12 +248,11 @@ class BuildParameters {
      cacheUnityInstallationOnMac: Input.cacheUnityInstallationOnMac,
      unityHubVersionOnMac: Input.unityHubVersionOnMac,
      dockerWorkspacePath: Input.dockerWorkspacePath,
-      gitIntegrityCheck: Input.gitIntegrityCheck,
-      gitAutoRecover: Input.gitAutoRecover,
-      cleanReservedFilenames: Input.cleanReservedFilenames,
-      buildArchiveEnabled: Input.buildArchiveEnabled,
-      buildArchivePath: Input.buildArchivePath,
-      buildArchiveRetention: Input.buildArchiveRetention,
+      testSuitePath: Input.testSuitePath,
+      testSuiteEvent: Input.testSuiteEvent,
+      testTaxonomyPath: Input.testTaxonomyPath,
+      testResultFormat: Input.testResultFormat,
+      testResultPath: Input.testResultPath,
    };
  }

--- a/src/model/input.ts
+++ b/src/model/input.ts
@@ -282,36 +282,24 @@ class Input {
    return Input.getInput('skipActivation')?.toLowerCase() ?? 'false';
  }

-  static get gitIntegrityCheck(): boolean {
-    const input = Input.getInput('gitIntegrityCheck') ?? 'false';
-
-    return input === 'true';
+  static get testSuitePath(): string {
+    return Input.getInput('testSuitePath') ?? '';
  }

-  static get gitAutoRecover(): boolean {
-    const input = Input.getInput('gitAutoRecover') ?? 'false';
-
-    return input === 'true';
+  static get testSuiteEvent(): string {
+    return Input.getInput('testSuiteEvent') ?? '';
  }

-  static get cleanReservedFilenames(): boolean {
-    const input = Input.getInput('cleanReservedFilenames') ?? 'false';
-
-    return input === 'true';
+  static get testTaxonomyPath(): string {
+    return Input.getInput('testTaxonomyPath') ?? '';
  }

-  static get buildArchiveEnabled(): boolean {
-    const input = Input.getInput('buildArchiveEnabled') ?? 'false';
-
-    return input === 'true';
+  static get testResultFormat(): string {
+    return Input.getInput('testResultFormat') ?? 'junit';
  }

-  static get buildArchivePath(): string {
-    return Input.getInput('buildArchivePath') ?? './build-archives';
-  }
-
-  static get buildArchiveRetention(): number {
-    return Number.parseInt(Input.getInput('buildArchiveRetention') ?? '30', 10);
+  static get testResultPath(): string {
+    return Input.getInput('testResultPath') ?? './test-results';
  }

  public static ToEnvVarFormat(input: string) {
--- a/src/model/orchestrator/services/reliability/build-reliability-service.test.ts
+++ b/src/model/orchestrator/services/reliability/build-reliability-service.test.ts
@@ -1,638 +0,0 @@
-import { execSync, execFileSync } from 'node:child_process';
-import fs from 'node:fs';
-import path from 'node:path';
-import { BuildReliabilityService } from './build-reliability-service';
-
-// Mock dependencies
-jest.mock('node:child_process');
-jest.mock('node:fs');
-jest.mock('@actions/core', () => ({
-  info: jest.fn(),
-  warning: jest.fn(),
-}));
-
-const mockExecSync = execSync as jest.MockedFunction<typeof execSync>;
-const mockExecFileSync = execFileSync as jest.MockedFunction<typeof execFileSync>;
-const mockFs = fs as jest.Mocked<typeof fs>;
-
-describe('BuildReliabilityService', () => {
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
-  // =========================================================================
-  // checkGitIntegrity
-  // =========================================================================
-
-  describe('checkGitIntegrity', () => {
-    it('should return true when fsck succeeds with clean output', () => {
-      mockExecSync.mockReturnValue('');
-      const result = BuildReliabilityService.checkGitIntegrity('/repo');
-      expect(result).toBe(true);
-      expect(mockExecSync).toHaveBeenCalledWith(
-        'git -C "/repo" fsck --no-dangling',
-        expect.objectContaining({ encoding: 'utf8' }),
-      );
-    });
-
-    it('should return false when fsck output contains corruption indicators', () => {
-      mockExecSync.mockReturnValue('broken link from tree abc123');
-      const result = BuildReliabilityService.checkGitIntegrity('/repo');
-      expect(result).toBe(false);
-    });
-
-    it('should return false when fsck output contains missing objects', () => {
-      mockExecSync.mockReturnValue('missing blob abc123');
-      const result = BuildReliabilityService.checkGitIntegrity('/repo');
-      expect(result).toBe(false);
-    });
-
-    it('should return false when execSync throws (non-zero exit code)', () => {
-      mockExecSync.mockImplementation(() => {
-        const error: any = new Error('fsck failed');
-        error.stderr = Buffer.from('error: bad object HEAD');
-        throw error;
-      });
-      const result = BuildReliabilityService.checkGitIntegrity('/repo');
-      expect(result).toBe(false);
-    });
-
-    it('should use current directory when no repoPath provided', () => {
-      mockExecSync.mockReturnValue('');
-      BuildReliabilityService.checkGitIntegrity();
-      expect(mockExecSync).toHaveBeenCalledWith('git -C "." fsck --no-dangling', expect.anything());
-    });
-  });
-
-  // =========================================================================
-  // cleanStaleLockFiles
-  // =========================================================================
-
-  describe('cleanStaleLockFiles', () => {
-    it('should return 0 when .git directory does not exist', () => {
-      mockFs.existsSync.mockReturnValue(false);
-      const result = BuildReliabilityService.cleanStaleLockFiles('/repo');
-      expect(result).toBe(0);
-    });
-
-    it('should remove lock files older than 10 minutes', () => {
-      const now = Date.now();
-      const oldTime = now - 15 * 60 * 1000; // 15 minutes ago
-
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.statSync.mockImplementation((filePath: any) => {
-        if (filePath === path.join('/repo', '.git')) {
-          return { isDirectory: () => true } as fs.Stats;
-        }
-        return { mtimeMs: oldTime } as fs.Stats;
-      });
-      mockFs.readdirSync.mockImplementation((dir: any) => {
-        if (dir === path.join('/repo', '.git')) {
-          return [
-            { name: 'index.lock', isDirectory: () => false },
-            { name: 'HEAD.lock', isDirectory: () => false },
-          ] as any;
-        }
-        return [];
-      });
-      mockFs.unlinkSync.mockReturnValue(undefined);
-
-      const result = BuildReliabilityService.cleanStaleLockFiles('/repo');
-      expect(result).toBe(2);
-      expect(mockFs.unlinkSync).toHaveBeenCalledTimes(2);
-    });
-
-    it('should NOT remove lock files younger than 10 minutes', () => {
-      const now = Date.now();
-      const recentTime = now - 2 * 60 * 1000; // 2 minutes ago
-
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.statSync.mockImplementation((filePath: any) => {
-        if (filePath === path.join('/repo', '.git')) {
-          return { isDirectory: () => true } as fs.Stats;
-        }
-        return { mtimeMs: recentTime } as fs.Stats;
-      });
-      mockFs.readdirSync.mockImplementation((dir: any) => {
-        if (dir === path.join('/repo', '.git')) {
-          return [{ name: 'index.lock', isDirectory: () => false }] as any;
-        }
-        return [];
-      });
-
-      const result = BuildReliabilityService.cleanStaleLockFiles('/repo');
-      expect(result).toBe(0);
-      expect(mockFs.unlinkSync).not.toHaveBeenCalled();
-    });
-
-    it('should recursively scan refs directory for lock files', () => {
-      const now = Date.now();
-      const oldTime = now - 15 * 60 * 1000;
-
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.statSync.mockImplementation((filePath: any) => {
-        if (filePath === path.join('/repo', '.git')) {
-          return { isDirectory: () => true } as fs.Stats;
-        }
-        return { mtimeMs: oldTime } as fs.Stats;
-      });
-      mockFs.readdirSync.mockImplementation((dir: any) => {
-        const gitDir = path.join('/repo', '.git');
-        if (dir === gitDir) {
-          return [{ name: 'refs', isDirectory: () => true }] as any;
-        }
-        if (dir === path.join(gitDir, 'refs')) {
-          return [{ name: 'heads', isDirectory: () => true }] as any;
-        }
-        if (dir === path.join(gitDir, 'refs', 'heads')) {
-          return [{ name: 'main.lock', isDirectory: () => false }] as any;
-        }
-        return [];
-      });
-      mockFs.unlinkSync.mockReturnValue(undefined);
-
-      const result = BuildReliabilityService.cleanStaleLockFiles('/repo');
-      expect(result).toBe(1);
-    });
-  });
-
-  // =========================================================================
-  // validateSubmoduleBackingStores
-  // =========================================================================
-
-  describe('validateSubmoduleBackingStores', () => {
-    it('should return empty array when .gitmodules does not exist', () => {
-      mockFs.existsSync.mockReturnValue(false);
-      const result = BuildReliabilityService.validateSubmoduleBackingStores('/repo');
-      expect(result).toEqual([]);
-    });
-
-    it('should detect broken backing store for submodule', () => {
-      mockFs.existsSync.mockImplementation((p: any) => {
-        if (p === path.join('/repo', '.gitmodules')) return true;
-        if (p === path.join('/repo', 'lib/sub', '.git')) return true;
-        // Backing store does not exist
-        return false;
-      });
-      mockFs.readFileSync.mockImplementation((p: any) => {
-        if (p === path.join('/repo', '.gitmodules')) {
-          return '[submodule "sub"]\n\tpath = lib/sub\n\turl = https://example.com/sub.git';
-        }
-        if (p === path.join('/repo', 'lib/sub', '.git')) {
-          return 'gitdir: ../../.git/modules/lib/sub';
-        }
-        return '';
-      });
-      mockFs.statSync.mockReturnValue({ isFile: () => true } as fs.Stats);
-
-      const result = BuildReliabilityService.validateSubmoduleBackingStores('/repo');
-      expect(result).toContain('lib/sub');
-    });
-
-    it('should return empty array when all submodule backing stores are valid', () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readFileSync.mockImplementation((p: any) => {
-        if (p === path.join('/repo', '.gitmodules')) {
-          return '[submodule "sub"]\n\tpath = lib/sub\n\turl = https://example.com/sub.git';
-        }
-        if (p === path.join('/repo', 'lib/sub', '.git')) {
-          return 'gitdir: ../../.git/modules/lib/sub';
-        }
-        return '';
-      });
-      mockFs.statSync.mockReturnValue({ isFile: () => true } as fs.Stats);
-
-      const result = BuildReliabilityService.validateSubmoduleBackingStores('/repo');
-      expect(result).toEqual([]);
-    });
-  });
-
-  // =========================================================================
-  // recoverCorruptedRepo
-  // =========================================================================
-
-  describe('recoverCorruptedRepo', () => {
-    it('should orchestrate fsck cleanup and re-fetch, returning true on success', () => {
-      // cleanStaleLockFiles: no .git dir
-      mockFs.existsSync.mockReturnValue(false);
-      mockFs.statSync.mockReturnValue({ isDirectory: () => false } as fs.Stats);
-
-      // fetch succeeds, then fsck succeeds
-      mockExecSync.mockReturnValue('');
-
-      const result = BuildReliabilityService.recoverCorruptedRepo('/repo');
-      expect(result).toBe(true);
-      // Should have called fetch
-      expect(mockExecSync).toHaveBeenCalledWith('git -C "/repo" fetch --all', expect.anything());
-    });
-
-    it('should return false when recovery fails to restore integrity', () => {
-      mockFs.existsSync.mockReturnValue(false);
-      mockFs.statSync.mockReturnValue({ isDirectory: () => false } as fs.Stats);
-
-      // fetch succeeds, but fsck fails
-      mockExecSync.mockImplementation((cmd: any) => {
-        if (typeof cmd === 'string' && cmd.includes('fetch')) return '';
-        if (typeof cmd === 'string' && cmd.includes('fsck')) {
-          return 'missing blob abc123';
-        }
-        return '';
-      });
-
-      const result = BuildReliabilityService.recoverCorruptedRepo('/repo');
-      expect(result).toBe(false);
-    });
-
-    it('should continue recovery even when fetch fails', () => {
-      mockFs.existsSync.mockReturnValue(false);
-      mockFs.statSync.mockReturnValue({ isDirectory: () => false } as fs.Stats);
-
-      let callCount = 0;
-      mockExecSync.mockImplementation((cmd: any) => {
-        callCount++;
-        if (typeof cmd === 'string' && cmd.includes('fetch')) {
-          throw new Error('network error');
-        }
-        // fsck call
-        return '';
-      });
-
-      const result = BuildReliabilityService.recoverCorruptedRepo('/repo');
-      // Should still attempt fsck after failed fetch
-      expect(callCount).toBeGreaterThanOrEqual(2);
-    });
-  });
-
-  // =========================================================================
-  // cleanReservedFilenames
-  // =========================================================================
-
-  describe('cleanReservedFilenames', () => {
-    it('should return empty array when Assets directory does not exist', () => {
-      mockFs.existsSync.mockReturnValue(false);
-      const result = BuildReliabilityService.cleanReservedFilenames('/project');
-      expect(result).toEqual([]);
-    });
-
-    it('should remove files with reserved names (con, prn, aux, nul)', () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readdirSync.mockImplementation((dir: any) => {
-        if (dir === path.join('/project', 'Assets')) {
-          return [
-            { name: 'con.txt', isDirectory: () => false },
-            { name: 'PRN.meta', isDirectory: () => false },
-            { name: 'aux.shader', isDirectory: () => false },
-            { name: 'nul.png', isDirectory: () => false },
-            { name: 'valid-file.cs', isDirectory: () => false },
-          ] as any;
-        }
-        return [];
-      });
-      mockFs.unlinkSync.mockReturnValue(undefined);
-
-      const result = BuildReliabilityService.cleanReservedFilenames('/project');
-      expect(result).toHaveLength(4);
-      expect(mockFs.unlinkSync).toHaveBeenCalledTimes(4);
-    });
-
-    it('should remove directories with reserved names', () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readdirSync.mockImplementation((dir: any) => {
-        if (dir === path.join('/project', 'Assets')) {
-          return [{ name: 'com1', isDirectory: () => true }] as any;
-        }
-        return [];
-      });
-      mockFs.rmSync.mockReturnValue(undefined);
-
-      const result = BuildReliabilityService.cleanReservedFilenames('/project');
-      expect(result).toHaveLength(1);
-      expect(mockFs.rmSync).toHaveBeenCalledWith(path.join('/project', 'Assets', 'com1'), {
-        recursive: true,
-        force: true,
-      });
-    });
-
-    it('should detect COM1 through COM9 and LPT1 through LPT9', () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readdirSync.mockImplementation((dir: any) => {
-        if (dir === path.join('/project', 'Assets')) {
-          return [
-            { name: 'com1.txt', isDirectory: () => false },
-            { name: 'COM9.meta', isDirectory: () => false },
-            { name: 'lpt1.dat', isDirectory: () => false },
-            { name: 'LPT9.log', isDirectory: () => false },
-          ] as any;
-        }
-        return [];
-      });
-      mockFs.unlinkSync.mockReturnValue(undefined);
-
-      const result = BuildReliabilityService.cleanReservedFilenames('/project');
-      expect(result).toHaveLength(4);
-    });
-
-    it('should not remove files that merely contain reserved names as substrings', () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readdirSync.mockImplementation((dir: any) => {
-        if (dir === path.join('/project', 'Assets')) {
-          return [
-            { name: 'controller.cs', isDirectory: () => false },
-            { name: 'printer-utils.cs', isDirectory: () => false },
-            { name: 'auxiliary.shader', isDirectory: () => false },
-          ] as any;
-        }
-        return [];
-      });
-
-      const result = BuildReliabilityService.cleanReservedFilenames('/project');
-      expect(result).toHaveLength(0);
-    });
-  });
-
-  // =========================================================================
-  // getAvailableSpaceMB
-  // =========================================================================
-
-  describe('getAvailableSpaceMB', () => {
-    it('should return -1 when the check fails', () => {
-      mockExecFileSync.mockImplementation(() => {
-        throw new Error('Command failed');
-      });
-
-      const result = BuildReliabilityService.getAvailableSpaceMB('/some/path');
-      expect(result).toBe(-1);
-    });
-
-    it('should parse wmic output on Windows', () => {
-      const originalPlatform = process.platform;
-      Object.defineProperty(process, 'platform', { value: 'win32' });
-
-      // 10 GB in bytes
-      mockExecFileSync.mockReturnValue('\r\nFreeSpace=10737418240\r\n' as any);
-
-      const result = BuildReliabilityService.getAvailableSpaceMB('C:\\builds');
-      // 10737418240 / (1024 * 1024) = 10240 MB
-      expect(result).toBeCloseTo(10240, 0);
-
-      Object.defineProperty(process, 'platform', { value: originalPlatform });
-    });
-
-    it('should parse df output on Unix', () => {
-      const originalPlatform = process.platform;
-      Object.defineProperty(process, 'platform', { value: 'linux' });
-
-      mockExecFileSync.mockReturnValue('  Avail\n  5120M\n' as any);
-
-      const result = BuildReliabilityService.getAvailableSpaceMB('/builds');
-      expect(result).toBe(5120);
-
-      Object.defineProperty(process, 'platform', { value: originalPlatform });
-    });
-  });
-
-  // =========================================================================
-  // getDirectorySizeMB
-  // =========================================================================
-
-  describe('getDirectorySizeMB', () => {
-    it('should return file size for a single file', () => {
-      // 5 MB in bytes
-      mockFs.statSync.mockReturnValue({ isDirectory: () => false, size: 5 * 1024 * 1024 } as any);
-
-      const result = BuildReliabilityService.getDirectorySizeMB('/path/to/file.zip');
-      expect(result).toBeCloseTo(5, 0);
-    });
-
-    it('should return total size for a directory tree', () => {
-      const subDir = path.join('/build', 'sub');
-
-      mockFs.statSync.mockImplementation((p: any) => {
-        const pathStr = typeof p === 'string' ? p : p.toString();
-        if (pathStr === '/build' || pathStr === subDir) {
-          return { isDirectory: () => true, size: 0 } as any;
-        }
-
-        return { isDirectory: () => false, size: 1024 * 1024 } as any; // 1 MB each
-      });
-
-      mockFs.readdirSync.mockImplementation((dirPath: any, _options?: any) => {
-        const dirStr = typeof dirPath === 'string' ? dirPath : dirPath.toString();
-        if (dirStr === '/build') {
-          return [
-            { name: 'file1.bin', isDirectory: () => false },
-            { name: 'sub', isDirectory: () => true },
-          ] as any;
-        }
-        if (dirStr === subDir) {
-          return [{ name: 'file2.bin', isDirectory: () => false }] as any;
-        }
-
-        return [] as any;
-      });
-
-      const result = BuildReliabilityService.getDirectorySizeMB('/build');
-      expect(result).toBeCloseTo(2, 0); // 2 files * 1 MB each
-    });
-
-    it('should return -1 when calculation fails', () => {
-      mockFs.statSync.mockImplementation(() => {
-        throw new Error('Access denied');
-      });
-
-      const result = BuildReliabilityService.getDirectorySizeMB('/inaccessible');
-      expect(result).toBe(-1);
-    });
-  });
-
-  // =========================================================================
-  // archiveBuildOutput
-  // =========================================================================
-
-  describe('archiveBuildOutput', () => {
-    it('should skip archiving when source path does not exist', () => {
-      mockFs.existsSync.mockReturnValue(false);
-      BuildReliabilityService.archiveBuildOutput('/builds/output', '/archives');
-      expect(mockExecSync).not.toHaveBeenCalled();
-    });
-
-    it('should create archive directory and tar.gz output', () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.mkdirSync.mockReturnValue(undefined as any);
-      mockExecSync.mockReturnValue('');
-      // Make disk space check return unknown so we proceed
-      mockExecFileSync.mockImplementation(() => {
-        throw new Error('Command not found');
-      });
-      mockFs.statSync.mockImplementation(() => {
-        throw new Error('Not mocked');
-      });
-
-      BuildReliabilityService.archiveBuildOutput('/builds/output', '/archives');
-
-      expect(mockFs.mkdirSync).toHaveBeenCalledWith('/archives', { recursive: true });
-      expect(mockExecSync).toHaveBeenCalledWith(expect.stringContaining('tar -czf'), expect.anything());
-    });
-
-    it('should skip archival when insufficient disk space', () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.mkdirSync.mockReturnValue(undefined as any);
-
-      // Source is 1000 MB
-      mockFs.statSync.mockImplementation((p: any) => {
-        const pathStr = typeof p === 'string' ? p : p.toString();
-        if (pathStr.endsWith('big-file.bin')) {
-          return { isDirectory: () => false, size: 1000 * 1024 * 1024 } as any;
-        }
-        return { isDirectory: () => true, size: 0 } as any;
-      });
-      mockFs.readdirSync.mockImplementation(() => {
-        return [{ name: 'big-file.bin', isDirectory: () => false }] as any;
-      });
-
-      // Only 500 MB available
-      const originalPlatform = process.platform;
-      Object.defineProperty(process, 'platform', { value: 'linux' });
-      mockExecFileSync.mockReturnValue('  Avail\n  500M\n' as any);
-
-      BuildReliabilityService.archiveBuildOutput('/builds/output', '/archives');
-
-      // Should NOT have attempted the tar command
-      expect(mockExecSync).not.toHaveBeenCalledWith(expect.stringContaining('tar'), expect.anything());
-
-      Object.defineProperty(process, 'platform', { value: originalPlatform });
-    });
-
-    it('should clean up partial archive on tar failure', () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.mkdirSync.mockReturnValue(undefined as any);
-      mockFs.unlinkSync.mockReturnValue(undefined);
-
-      // Make disk space check return unknown so we proceed
-      mockExecFileSync.mockImplementation(() => {
-        throw new Error('Command not found');
-      });
-      mockFs.statSync.mockImplementation(() => {
-        throw new Error('Not mocked');
-      });
-
-      // tar command fails
-      mockExecSync.mockImplementation(() => {
-        const error: any = new Error('tar failed');
-        error.stderr = Buffer.from('No space left on device');
-        throw error;
-      });
-
-      BuildReliabilityService.archiveBuildOutput('/builds/output', '/archives');
-
-      // Should have attempted to clean up the partial archive
-      // (existsSync returns true for the partial file)
-      expect(mockFs.unlinkSync).toHaveBeenCalled();
-    });
-
-    it('should proceed with warning when disk space check fails', () => {
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.mkdirSync.mockReturnValue(undefined as any);
-      mockExecSync.mockReturnValue('');
-
-      // Disk space check fails
-      mockExecFileSync.mockImplementation(() => {
-        throw new Error('Command not found');
-      });
-      // Directory size check also fails
-      mockFs.statSync.mockImplementation(() => {
-        throw new Error('Not mocked');
-      });
-
-      BuildReliabilityService.archiveBuildOutput('/builds/output', '/archives');
-
-      // Should still proceed with tar
-      expect(mockExecSync).toHaveBeenCalledWith(expect.stringContaining('tar -czf'), expect.anything());
-    });
-  });
-
-  // =========================================================================
-  // enforceRetention
-  // =========================================================================
-
-  describe('enforceRetention', () => {
-    it('should return 0 when archive path does not exist', () => {
-      mockFs.existsSync.mockReturnValue(false);
-      const result = BuildReliabilityService.enforceRetention('/archive', 30);
-      expect(result).toBe(0);
-    });
-
-    it('should remove archives older than retention period', () => {
-      const now = Date.now();
-      const oldTime = now - 45 * 24 * 60 * 60 * 1000; // 45 days ago
-      const recentTime = now - 5 * 24 * 60 * 60 * 1000; // 5 days ago
-
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readdirSync.mockReturnValue([
-        { name: 'build-old.tar.gz', isDirectory: () => false },
-        { name: 'build-recent.tar.gz', isDirectory: () => false },
-      ] as any);
-      mockFs.statSync.mockImplementation((p: any) => {
-        if ((p as string).includes('old')) {
-          return { mtimeMs: oldTime } as fs.Stats;
-        }
-        return { mtimeMs: recentTime } as fs.Stats;
-      });
-      mockFs.unlinkSync.mockReturnValue(undefined);
-
-      const result = BuildReliabilityService.enforceRetention('/archive', 30);
-      expect(result).toBe(1);
-      expect(mockFs.unlinkSync).toHaveBeenCalledTimes(1);
-    });
-
-    it('should keep all archives within retention period', () => {
-      const now = Date.now();
-      const recentTime = now - 5 * 24 * 60 * 60 * 1000;
-
-      mockFs.existsSync.mockReturnValue(true);
-      mockFs.readdirSync.mockReturnValue([
-        { name: 'build-1.tar.gz', isDirectory: () => false },
-        { name: 'build-2.tar.gz', isDirectory: () => false },
-      ] as any);
-      mockFs.statSync.mockReturnValue({ mtimeMs: recentTime } as fs.Stats);
-
-      const result = BuildReliabilityService.enforceRetention('/archive', 30);
-      expect(result).toBe(0);
-    });
-  });
-
-  // =========================================================================
-  // configureGitEnvironment
-  // =========================================================================
-
-  describe('configureGitEnvironment', () => {
-    it('should set GIT_TERMINAL_PROMPT=0 in process.env', () => {
-      mockExecSync.mockReturnValue('');
-      BuildReliabilityService.configureGitEnvironment();
-      expect(process.env.GIT_TERMINAL_PROMPT).toBe('0');
-    });
-
-    it('should configure http.postBuffer via git config', () => {
-      mockExecSync.mockReturnValue('');
-      BuildReliabilityService.configureGitEnvironment();
-      expect(mockExecSync).toHaveBeenCalledWith('git config --global http.postBuffer 524288000', expect.anything());
-    });
-
-    it('should configure core.longpaths via git config', () => {
-      mockExecSync.mockReturnValue('');
-      BuildReliabilityService.configureGitEnvironment();
-      expect(mockExecSync).toHaveBeenCalledWith('git config --global core.longpaths true', expect.anything());
-    });
-
-    it('should warn but not throw when git config commands fail', () => {
-      const core = require('@actions/core');
-      mockExecSync.mockImplementation(() => {
-        throw new Error('git config failed');
-      });
-
-      // Should not throw
-      expect(() => BuildReliabilityService.configureGitEnvironment()).not.toThrow();
-      expect(core.warning).toHaveBeenCalled();
-    });
-  });
-});
--- a/src/model/orchestrator/services/reliability/build-reliability-service.ts
+++ b/src/model/orchestrator/services/reliability/build-reliability-service.ts
@@ -1,527 +0,0 @@
-import { execSync, execFileSync } from 'node:child_process';
-import fs from 'node:fs';
-import path from 'node:path';
-import * as core from '@actions/core';
-
-/**
- * Build reliability features for hardening CI pipelines.
- * Provides git integrity checks, stale lock cleanup, submodule validation,
- * reserved filename removal, build archival, and git environment configuration.
- * All features are opt-in and fail gracefully (warnings only).
- */
-export class BuildReliabilityService {
-  // Windows reserved device names that cause Unity asset importer infinite loops
-  private static readonly RESERVED_NAMES = new Set([
-    'con',
-    'prn',
-    'aux',
-    'nul',
-    'com1',
-    'com2',
-    'com3',
-    'com4',
-    'com5',
-    'com6',
-    'com7',
-    'com8',
-    'com9',
-    'lpt1',
-    'lpt2',
-    'lpt3',
-    'lpt4',
-    'lpt5',
-    'lpt6',
-    'lpt7',
-    'lpt8',
-    'lpt9',
-  ]);
-
-  // Lock files to look for in the .git directory
-  private static readonly LOCK_FILE_NAMES = new Set(['index.lock', 'shallow.lock', 'config.lock', 'HEAD.lock']);
-
-  // Maximum age in milliseconds before a lock file is considered stale (10 minutes)
-  private static readonly LOCK_FILE_MAX_AGE_MS = 10 * 60 * 1000;
-
-  /**
-   * Run git fsck to check repository integrity.
-   * Returns true if the repo is healthy, false if corruption detected.
-   */
-  static checkGitIntegrity(repoPath: string = '.'): boolean {
-    core.info(`[Reliability] Checking git integrity in ${repoPath}`);
-
-    try {
-      const output = execSync(`git -C "${repoPath}" fsck --no-dangling`, {
-        encoding: 'utf8',
-        timeout: 120_000,
-        stdio: ['pipe', 'pipe', 'pipe'],
-      });
-
-      // Parse output for corruption indicators
-      const corruptionPatterns = [
-        /broken link/i,
-        /missing (blob|tree|commit|tag)/i,
-        /dangling/i,
-        /corrupt/i,
-        /error in /i,
-      ];
-
-      for (const pattern of corruptionPatterns) {
-        if (pattern.test(output)) {
-          core.warning(`[Reliability] Git integrity check found issues: ${output.trim()}`);
-          return false;
-        }
-      }
-
-      core.info('[Reliability] Git integrity check passed');
-      return true;
-    } catch (error: any) {
-      // execSync throws on non-zero exit code
-      const stderr = error.stderr?.toString() ?? error.message;
-      core.warning(`[Reliability] Git integrity check failed: ${stderr}`);
-      return false;
-    }
-  }
-
-  /**
-   * Remove stale .lock files from the .git directory.
-   * Only removes lock files older than 10 minutes to avoid interfering with active operations.
-   * Returns the number of lock files removed.
-   */
-  static cleanStaleLockFiles(repoPath: string = '.'): number {
-    const gitDir = path.join(repoPath, '.git');
-    if (!fs.existsSync(gitDir) || !fs.statSync(gitDir).isDirectory()) {
-      return 0;
-    }
-
-    core.info(`[Reliability] Scanning for stale lock files in ${gitDir}`);
-    const now = Date.now();
-    let removed = 0;
-
-    const cleanDirectory = (directory: string): void => {
-      if (!fs.existsSync(directory)) return;
-
-      try {
-        const entries = fs.readdirSync(directory, { withFileTypes: true });
-        for (const entry of entries) {
-          const fullPath = path.join(directory, entry.name);
-          if (entry.isDirectory()) {
-            cleanDirectory(fullPath);
-          } else if (entry.name.endsWith('.lock')) {
-            // Check if it is a known lock file location OR under refs/
-            const relativePath = path.relative(gitDir, fullPath);
-            const isKnownLock = BuildReliabilityService.LOCK_FILE_NAMES.has(entry.name);
-            const isRefsLock = relativePath.startsWith('refs' + path.sep);
-
-            if (isKnownLock || isRefsLock) {
-              try {
-                const stat = fs.statSync(fullPath);
-                const ageMs = now - stat.mtimeMs;
-
-                if (ageMs > BuildReliabilityService.LOCK_FILE_MAX_AGE_MS) {
-                  fs.unlinkSync(fullPath);
-                  removed++;
-                  core.info(
-                    `[Reliability] Removed stale lock file (age: ${Math.round(ageMs / 1000)}s): ${relativePath}`,
-                  );
-                } else {
-                  core.info(
-                    `[Reliability] Lock file is recent (age: ${Math.round(ageMs / 1000)}s), skipping: ${relativePath}`,
-                  );
-                }
-              } catch {
-                core.warning(`[Reliability] Could not remove lock file: ${fullPath}`);
-              }
-            }
-          }
-        }
-      } catch {
-        // Directory not accessible
-      }
-    };
-
-    cleanDirectory(gitDir);
-
-    if (removed > 0) {
-      core.info(`[Reliability] Cleaned ${removed} stale lock file(s)`);
-    } else {
-      core.info('[Reliability] No stale lock files found');
-    }
-
-    return removed;
-  }
-
-  /**
-   * Validate that submodule .git files point to existing backing stores
-   * under .git/modules/. Returns list of submodule paths with broken backing stores.
-   */
-  static validateSubmoduleBackingStores(repoPath: string = '.'): string[] {
-    const broken: string[] = [];
-    const gitmodulesPath = path.join(repoPath, '.gitmodules');
-
-    if (!fs.existsSync(gitmodulesPath)) {
-      core.info('[Reliability] No .gitmodules found, skipping submodule validation');
-      return broken;
-    }
-
-    core.info(`[Reliability] Validating submodule backing stores in ${repoPath}`);
-
-    try {
-      const content = fs.readFileSync(gitmodulesPath, 'utf8');
-      const pathMatches = content.matchAll(/path\s*=\s*(.+)/g);
-
-      for (const match of pathMatches) {
-        const submodulePath = match[1].trim();
-        const gitFile = path.join(repoPath, submodulePath, '.git');
-
-        if (!fs.existsSync(gitFile)) {
-          // Submodule not initialized -- not necessarily broken
-          continue;
-        }
-
-        try {
-          const stat = fs.statSync(gitFile);
-          if (stat.isFile()) {
-            // .git is a file -- should contain "gitdir: <path>"
-            const gitFileContent = fs.readFileSync(gitFile, 'utf8').trim();
-            const gitdirMatch = gitFileContent.match(/^gitdir:\s*(.+)$/);
-
-            if (gitdirMatch) {
-              const backingStore = path.resolve(path.join(repoPath, submodulePath), gitdirMatch[1]);
-              if (!fs.existsSync(backingStore)) {
-                broken.push(submodulePath);
-                core.warning(`[Reliability] Submodule ${submodulePath} has broken backing store: ${backingStore}`);
-              } else {
-                core.info(`[Reliability] Submodule ${submodulePath} backing store OK`);
-              }
-            } else {
-              broken.push(submodulePath);
-              core.warning(`[Reliability] Submodule ${submodulePath} .git file has invalid format`);
-            }
-          }
-        } catch {
-          // Can't read .git file
-          core.warning(`[Reliability] Could not read .git file for submodule: ${submodulePath}`);
-        }
-      }
-    } catch (error: any) {
-      core.warning(`[Reliability] Could not read .gitmodules: ${error.message}`);
-    }
-
-    if (broken.length > 0) {
-      core.warning(`[Reliability] ${broken.length} submodule(s) have broken backing stores`);
-    } else {
-      core.info('[Reliability] All submodule backing stores are valid');
-    }
-
-    return broken;
-  }
-
-  /**
-   * Orchestrate recovery of a corrupted repository.
-   * Sequence: fsck -> clean locks -> re-fetch -> retry fsck.
-   * Returns true if recovery succeeded.
-   */
-  static recoverCorruptedRepo(repoPath: string = '.'): boolean {
-    core.warning(`[Reliability] Attempting automatic recovery for ${repoPath}`);
-
-    // Step 1: Clean stale lock files that may be preventing operations
-    const locksRemoved = BuildReliabilityService.cleanStaleLockFiles(repoPath);
-    if (locksRemoved > 0) {
-      core.info(`[Reliability] Recovery: cleaned ${locksRemoved} lock file(s)`);
-    }
-
-    // Step 2: Re-fetch to restore missing objects
-    try {
-      core.info('[Reliability] Recovery: re-fetching from remote');
-      execSync(`git -C "${repoPath}" fetch --all`, {
-        encoding: 'utf8',
-        timeout: 300_000,
-        stdio: ['pipe', 'pipe', 'pipe'],
-      });
-      core.info('[Reliability] Recovery: fetch completed');
-    } catch (error: any) {
-      core.warning(`[Reliability] Recovery: fetch failed: ${error.stderr?.toString() ?? error.message}`);
-    }
-
-    // Step 3: Retry fsck
-    const healthy = BuildReliabilityService.checkGitIntegrity(repoPath);
-    if (healthy) {
-      core.info('[Reliability] Recovery succeeded -- repository is healthy');
-    } else {
-      core.warning('[Reliability] Recovery failed -- repository still has integrity issues');
-    }
-
-    return healthy;
-  }
-
-  /**
-   * Scan a directory tree for files/directories with Windows reserved names.
-   * These names (con, prn, aux, nul, com1-9, lpt1-9) with any extension
-   * cause Unity asset importer infinite loops on Windows.
-   * Returns list of paths that were removed.
-   */
-  static cleanReservedFilenames(projectPath: string): string[] {
-    const assetsPath = path.join(projectPath, 'Assets');
-    if (!fs.existsSync(assetsPath)) {
-      core.info(`[Reliability] No Assets directory found at ${assetsPath}, skipping reserved filename scan`);
-      return [];
-    }
-
-    core.info(`[Reliability] Scanning for reserved filenames in ${assetsPath}`);
-    const cleaned: string[] = [];
-
-    const scanDirectory = (directory: string): void => {
-      try {
-        const entries = fs.readdirSync(directory, { withFileTypes: true });
-        for (const entry of entries) {
-          const nameWithoutExtension = entry.name.split('.')[0].toLowerCase();
-          const fullPath = path.join(directory, entry.name);
-
-          if (BuildReliabilityService.RESERVED_NAMES.has(nameWithoutExtension)) {
-            try {
-              if (entry.isDirectory()) {
-                fs.rmSync(fullPath, { recursive: true, force: true });
-              } else {
-                fs.unlinkSync(fullPath);
-              }
-              cleaned.push(fullPath);
-              core.warning(`[Reliability] Removed reserved filename: ${fullPath}`);
-            } catch {
-              core.warning(`[Reliability] Could not remove reserved filename: ${fullPath}`);
-            }
-          } else if (entry.isDirectory()) {
-            scanDirectory(fullPath);
-          }
-        }
-      } catch {
-        // Directory not accessible
-      }
-    };
-
-    scanDirectory(assetsPath);
-
-    if (cleaned.length > 0) {
-      core.warning(`[Reliability] Cleaned ${cleaned.length} reserved filename(s)`);
-    } else {
-      core.info('[Reliability] No reserved filenames found');
-    }
-
-    return cleaned;
-  }
-
-  /**
-   * Get available disk space in megabytes for a given directory.
-   * Returns -1 if the check fails (unknown space).
-   *
-   * Cross-platform: uses wmic on Windows, df on Unix.
-   */
-  static getAvailableSpaceMB(directoryPath: string): number {
-    try {
-      if (process.platform === 'win32') {
-        const drive = path.parse(directoryPath).root;
-        const driveLetter = drive.replace(/[:\\\/]/g, '');
-        const output = execFileSync(
-          'wmic',
-          ['logicaldisk', 'where', `DeviceID='${driveLetter}:'`, 'get', 'FreeSpace', '/value'],
-          { encoding: 'utf8', timeout: 10_000 },
-        );
-        const match = output.match(/FreeSpace=(\d+)/);
-
-        return match ? Number.parseInt(match[1], 10) / (1024 * 1024) : -1;
-      } else {
-        const output = execFileSync('df', ['-BM', '--output=avail', directoryPath], {
-          encoding: 'utf8',
-          timeout: 10_000,
-        });
-        const lines = output.trim().split('\n');
-
-        return Number.parseInt(lines[lines.length - 1], 10);
-      }
-    } catch {
-      return -1; // Unknown, caller should proceed with warning
-    }
-  }
-
-  /**
-   * Calculate the total size of a directory in megabytes.
-   * Returns -1 if the calculation fails.
-   */
-  static getDirectorySizeMB(directoryPath: string): number {
-    try {
-      const stat = fs.statSync(directoryPath);
-      if (!stat.isDirectory()) {
-        return stat.size / (1024 * 1024);
-      }
-
-      let totalBytes = 0;
-      const walkDirectory = (dir: string): void => {
-        const entries = fs.readdirSync(dir, { withFileTypes: true });
-        for (const entry of entries) {
-          const fullPath = path.join(dir, entry.name);
-          if (entry.isDirectory()) {
-            walkDirectory(fullPath);
-          } else {
-            try {
-              totalBytes += fs.statSync(fullPath).size;
-            } catch {
-              // Skip inaccessible files
-            }
-          }
-        }
-      };
-
-      walkDirectory(directoryPath);
-
-      return totalBytes / (1024 * 1024);
-    } catch {
-      return -1;
-    }
-  }
-
-  /**
-   * Create a tar.gz archive of build output.
-   *
-   * Validates disk space before archiving. Skips archival with a warning
-   * if insufficient space is detected, preventing partial writes on full disks.
-   */
-  static archiveBuildOutput(sourcePath: string, archivePath: string): void {
-    if (!fs.existsSync(sourcePath)) {
-      core.info(`[Reliability] No build output to archive at ${sourcePath}`);
-      return;
-    }
-
-    fs.mkdirSync(archivePath, { recursive: true });
-
-    // Check available disk space before archiving
-    const sourceSizeMB = BuildReliabilityService.getDirectorySizeMB(sourcePath);
-    const availableSpaceMB = BuildReliabilityService.getAvailableSpaceMB(archivePath);
-
-    if (sourceSizeMB >= 0 && availableSpaceMB >= 0) {
-      const neededMB = Math.ceil(sourceSizeMB * 1.1); // 10% safety margin
-      if (availableSpaceMB < neededMB) {
-        core.warning(
-          `[Reliability] Insufficient disk space for archive. ` +
-            `Need ~${neededMB}MB, available: ${Math.floor(availableSpaceMB)}MB. Skipping archive.`,
-        );
-        return;
-      }
-      core.info(
-        `[Reliability] Disk space check passed: need ~${neededMB}MB, available: ${Math.floor(availableSpaceMB)}MB`,
-      );
-    } else if (availableSpaceMB < 0) {
-      core.warning('[Reliability] Could not determine available disk space. Proceeding with archive cautiously.');
-    }
-
-    const timestamp = new Date().toISOString().replace(/[.:]/g, '-');
-    const archiveFile = path.join(archivePath, `build-${timestamp}.tar.gz`);
-
-    try {
-      execSync(`tar -czf "${archiveFile}" -C "${path.dirname(sourcePath)}" "${path.basename(sourcePath)}"`, {
-        encoding: 'utf8',
-        timeout: 600_000,
-        stdio: ['pipe', 'pipe', 'pipe'],
-      });
-      core.info(`[Reliability] Build output archived to ${archiveFile}`);
-    } catch (error: any) {
-      core.warning(`[Reliability] Failed to archive build output: ${error.stderr?.toString() ?? error.message}`);
-
-      // Clean up partial archive if it exists to avoid leaving corrupted files
-      try {
-        if (fs.existsSync(archiveFile)) {
-          fs.unlinkSync(archiveFile);
-          core.info(`[Reliability] Cleaned up partial archive: ${archiveFile}`);
-        }
-      } catch {
-        // Best-effort cleanup
-      }
-    }
-  }
-
-  /**
-   * Enforce retention policy -- delete archives older than the retention period.
-   * Returns the number of old archives removed.
-   */
-  static enforceRetention(archivePath: string, retentionDays: number): number {
-    if (!fs.existsSync(archivePath)) {
-      return 0;
-    }
-
-    const now = Date.now();
-    const retentionMs = retentionDays * 24 * 60 * 60 * 1000;
-    let removed = 0;
-
-    try {
-      const entries = fs.readdirSync(archivePath, { withFileTypes: true });
-      for (const entry of entries) {
-        const fullPath = path.join(archivePath, entry.name);
-        try {
-          const stat = fs.statSync(fullPath);
-          const ageMs = now - stat.mtimeMs;
-
-          if (ageMs > retentionMs) {
-            if (entry.isDirectory()) {
-              fs.rmSync(fullPath, { recursive: true, force: true });
-            } else {
-              fs.unlinkSync(fullPath);
-            }
-            removed++;
-            core.info(
-              `[Reliability] Removed old archive: ${entry.name} (age: ${Math.round(
-                ageMs / (24 * 60 * 60 * 1000),
-              )} days)`,
-            );
-          }
-        } catch {
-          core.warning(`[Reliability] Could not process archive entry: ${fullPath}`);
-        }
-      }
-    } catch {
-      core.warning(`[Reliability] Could not read archive directory: ${archivePath}`);
-      return 0;
-    }
-
-    if (removed > 0) {
-      core.info(
-        `[Reliability] Retention enforced: removed ${removed} old archive(s), retention: ${retentionDays} days`,
-      );
-    }
-
-    return removed;
-  }
-
-  /**
-   * Configure git environment variables for CI reliability.
-   * Sets GIT_TERMINAL_PROMPT=0, increases http.postBuffer, enables core.longpaths.
-   */
-  static configureGitEnvironment(): void {
-    core.info('[Reliability] Configuring git environment for CI');
-
-    // Prevent git from prompting for credentials (hangs in CI)
-    process.env.GIT_TERMINAL_PROMPT = '0';
-    core.info('[Reliability] Set GIT_TERMINAL_PROMPT=0');
-
-    try {
-      // Increase http.postBuffer to 500MB for large pushes
-      execSync('git config --global http.postBuffer 524288000', {
-        encoding: 'utf8',
-        timeout: 10_000,
-        stdio: ['pipe', 'pipe', 'pipe'],
-      });
-      core.info('[Reliability] Set http.postBuffer=524288000 (500MB)');
-    } catch (error: any) {
-      core.warning(`[Reliability] Could not set http.postBuffer: ${error.message}`);
-    }
-
-    try {
-      // Enable long paths on Windows
-      execSync('git config --global core.longpaths true', {
-        encoding: 'utf8',
-        timeout: 10_000,
-        stdio: ['pipe', 'pipe', 'pipe'],
-      });
-      core.info('[Reliability] Set core.longpaths=true');
-    } catch (error: any) {
-      core.warning(`[Reliability] Could not set core.longpaths: ${error.message}`);
-    }
-  }
-}
--- a/src/model/orchestrator/services/reliability/index.ts
+++ b/src/model/orchestrator/services/reliability/index.ts
@@ -1 +0,0 @@
-export { BuildReliabilityService } from './build-reliability-service';
--- a/src/model/orchestrator/services/test-workflow/index.ts
+++ b/src/model/orchestrator/services/test-workflow/index.ts
@@ -0,0 +1,12 @@
+export { TestSuiteParser } from './test-suite-parser';
+export { TaxonomyFilterService } from './taxonomy-filter-service';
+export { TestResultReporter } from './test-result-reporter';
+export { TestWorkflowService } from './test-workflow-service';
+export {
+  TestSuiteDefinition,
+  TestRunDefinition,
+  TaxonomyDimension,
+  TaxonomyDefinition,
+  TestResult,
+  TestFailure,
+} from './test-workflow-types';
--- a/src/model/orchestrator/services/test-workflow/taxonomy-filter-service.ts
+++ b/src/model/orchestrator/services/test-workflow/taxonomy-filter-service.ts
@@ -0,0 +1,198 @@
+import fs from 'node:fs';
+import YAML from 'yaml';
+import { TaxonomyDimension, TaxonomyDefinition } from './test-workflow-types';
+
+/**
+ * Manages test taxonomy dimensions and builds filter arguments for
+ * the Unity test runner CLI. Supports comma-separated value lists,
+ * regex patterns (/pattern/), and hierarchical dot-notation matching.
+ */
+export class TaxonomyFilterService {
+  /**
+   * Built-in taxonomy dimensions that are always available.
+   * Projects may extend these via a custom taxonomy file.
+   */
+  private static readonly BUILT_IN_DIMENSIONS: TaxonomyDimension[] = [
+    { name: 'Scope', values: ['Unit', 'Integration', 'System', 'End To End'] },
+    { name: 'Maturity', values: ['Trusted', 'Adolescent', 'Experimental'] },
+    { name: 'FeedbackSpeed', values: ['Fast', 'Moderate', 'Slow'] },
+    { name: 'Execution', values: ['Synchronous', 'Asynchronous', 'Coroutine'] },
+    { name: 'Rigor', values: ['Strict', 'Normal', 'Relaxed'] },
+    { name: 'Determinism', values: ['Deterministic', 'NonDeterministic'] },
+    { name: 'IsolationLevel', values: ['Full', 'Partial', 'None'] },
+  ];
+
+  /**
+   * Load taxonomy dimensions: built-in dimensions plus any custom dimensions
+   * from an optional taxonomy file.
+   */
+  static loadTaxonomy(filePath?: string): TaxonomyDimension[] {
+    const dimensions = [...TaxonomyFilterService.BUILT_IN_DIMENSIONS];
+
+    if (filePath && fs.existsSync(filePath)) {
+      const content = fs.readFileSync(filePath, 'utf8');
+      const parsed = YAML.parse(content) as TaxonomyDefinition;
+
+      if (parsed?.extensible_groups && Array.isArray(parsed.extensible_groups)) {
+        for (const group of parsed.extensible_groups) {
+          if (group.name && Array.isArray(group.values)) {
+            // If a custom dimension has the same name as a built-in, merge values
+            const existing = dimensions.find((d) => d.name === group.name);
+            if (existing) {
+              const existingValues = new Set(existing.values);
+              for (const value of group.values) {
+                if (!existingValues.has(value)) {
+                  existing.values.push(value);
+                }
+              }
+            } else {
+              dimensions.push({ name: group.name, values: [...group.values] });
+            }
+          }
+        }
+      }
+    }
+
+    return dimensions;
+  }
+
+  /**
+   * Convert a filter map to Unity test runner CLI args (--testFilter).
+   *
+   * Each filter dimension becomes a category expression. Multiple values in one
+   * dimension are OR'd; multiple dimensions are AND'd. The result is a single
+   * --testFilter string suitable for passing to Unity's test runner CLI.
+   *
+   * Regex patterns (values wrapped in /.../) are converted to category regex
+   * expressions supported by the Unity test runner.
+   */
+  static buildFilterArgs(filters: Record<string, string>): string {
+    if (!filters || Object.keys(filters).length === 0) {
+      return '';
+    }
+
+    const categoryExpressions: string[] = [];
+
+    for (const [dimension, valueSpec] of Object.entries(filters)) {
+      const expression = TaxonomyFilterService.buildDimensionExpression(dimension, valueSpec);
+      if (expression) {
+        categoryExpressions.push(expression);
+      }
+    }
+
+    if (categoryExpressions.length === 0) {
+      return '';
+    }
+
+    // Unity test runner uses --testFilter with category expressions
+    // Multiple dimensions are AND'd by joining with ';'
+    const filterString = categoryExpressions.join(';');
+    return `--testFilter "${filterString}"`;
+  }
+
+  /**
+   * Build a filter expression for a single taxonomy dimension.
+   */
+  private static buildDimensionExpression(dimension: string, valueSpec: string): string {
+    if (!valueSpec || valueSpec.trim() === '') {
+      return '';
+    }
+
+    const trimmed = valueSpec.trim();
+
+    // Check if the value is a regex pattern: /pattern/
+    if (trimmed.startsWith('/') && trimmed.endsWith('/') && trimmed.length > 2) {
+      const pattern = trimmed.slice(1, -1);
+      return `${dimension}=~${pattern}`;
+    }
+
+    // Comma-separated values: OR'd together
+    const values = trimmed
+      .split(',')
+      .map((v) => v.trim())
+      .filter((v) => v.length > 0);
+
+    if (values.length === 0) {
+      return '';
+    }
+
+    if (values.length === 1) {
+      return `${dimension}=${values[0]}`;
+    }
+
+    // Multiple values: use pipe-separated OR syntax
+    return `${dimension}=${values.join('|')}`;
+  }
+
+  /**
+   * Check if a test's taxonomy metadata matches the given filter criteria.
+   *
+   * A test matches if ALL filter dimensions match (AND across dimensions).
+   * Within a single dimension, the test must match ANY of the specified values (OR).
+   * Regex patterns are matched as regular expressions.
+   * Hierarchical dot-notation supports prefix matching (e.g., filter "Combat.Melee"
+   * matches test category "Combat.Melee.Sword").
+   */
+  static matchesFilter(testCategories: Record<string, string>, filters: Record<string, string>): boolean {
+    for (const [dimension, valueSpec] of Object.entries(filters)) {
+      const testValue = testCategories[dimension];
+
+      // If the test has no value for this dimension, it does not match
+      if (testValue === undefined || testValue === null) {
+        return false;
+      }
+
+      if (!TaxonomyFilterService.matchesDimensionFilter(testValue, valueSpec)) {
+        return false;
+      }
+    }
+
+    return true;
+  }
+
+  /**
+   * Check if a single test category value matches a dimension filter spec.
+   */
+  private static matchesDimensionFilter(testValue: string, valueSpec: string): boolean {
+    const trimmed = valueSpec.trim();
+
+    // Regex pattern
+    if (trimmed.startsWith('/') && trimmed.endsWith('/') && trimmed.length > 2) {
+      const pattern = trimmed.slice(1, -1);
+      try {
+        const regex = new RegExp(pattern);
+        return regex.test(testValue);
+      } catch {
+        // Invalid regex, treat as literal
+        return testValue === trimmed;
+      }
+    }
+
+    // Comma-separated values
+    const values = trimmed
+      .split(',')
+      .map((v) => v.trim())
+      .filter((v) => v.length > 0);
+
+    return values.some((filterValue) => {
+      // Exact match
+      if (testValue === filterValue) {
+        return true;
+      }
+
+      // Hierarchical dot-notation prefix match
+      // Filter "Combat.Melee" matches test "Combat.Melee" and "Combat.Melee.Sword"
+      if (filterValue.includes('.') || testValue.includes('.')) {
+        if (testValue.startsWith(filterValue + '.') || testValue === filterValue) {
+          return true;
+        }
+        // Also allow the test to be a prefix of the filter for upward matching
+        if (filterValue.startsWith(testValue + '.')) {
+          return true;
+        }
+      }
+
+      return false;
+    });
+  }
+}
--- a/src/model/orchestrator/services/test-workflow/test-result-reporter.ts
+++ b/src/model/orchestrator/services/test-workflow/test-result-reporter.ts
@@ -0,0 +1,316 @@
+import fs from 'node:fs';
+import path from 'node:path';
+import { TestResult, TestFailure } from './test-workflow-types';
+
+/**
+ * Parses test result files (JUnit XML, Unity JSON) and generates structured
+ * summary reports. Supports writing results in multiple formats for CI
+ * integration (GitHub Checks, artifact upload).
+ */
+export class TestResultReporter {
+  /**
+   * Parse a JUnit XML test result file into a TestResult.
+   * JUnit XML is the standard format produced by Unity's test runner.
+   */
+  static parseJUnitResults(xmlPath: string): TestResult {
+    if (!fs.existsSync(xmlPath)) {
+      throw new Error(`JUnit result file not found: ${xmlPath}`);
+    }
+
+    const content = fs.readFileSync(xmlPath, 'utf8');
+    return TestResultReporter.parseJUnitXml(content);
+  }
+
+  /**
+   * Parse JUnit XML content string into a TestResult.
+   */
+  static parseJUnitXml(xmlContent: string): TestResult {
+    // Extract the testsuite opening tag
+    const suiteTagMatch = xmlContent.match(/<testsuite\s[^>]*>/);
+
+    let runName = 'unknown';
+    let totalTests = 0;
+    let failureCount = 0;
+    let skippedCount = 0;
+    let duration = 0;
+
+    if (suiteTagMatch) {
+      const tag = suiteTagMatch[0];
+
+      // Extract individual attributes -- order-independent
+      const nameMatch = tag.match(/\sname="([^"]*)"/);
+      const testsMatch = tag.match(/\stests="(\d+)"/);
+      const failuresMatch = tag.match(/\sfailures="(\d+)"/);
+      const skippedMatch = tag.match(/\sskipped="(\d+)"/);
+      const timeMatch = tag.match(/\stime="([^"]*)"/);
+
+      runName = nameMatch ? nameMatch[1] : 'unknown';
+      totalTests = testsMatch ? Number.parseInt(testsMatch[1], 10) : 0;
+      failureCount = failuresMatch ? Number.parseInt(failuresMatch[1], 10) : 0;
+      skippedCount = skippedMatch ? Number.parseInt(skippedMatch[1], 10) : 0;
+      duration = timeMatch ? Number.parseFloat(timeMatch[1]) : 0;
+    }
+
+    // Extract individual test failures by splitting into testcase blocks
+    const failures: TestFailure[] = [];
+    const testcasePattern = /<testcase\s[^>]*>[\s\S]*?<\/testcase>/g;
+
+    let tcMatch;
+    while ((tcMatch = testcasePattern.exec(xmlContent)) !== null) {
+      const block = tcMatch[0];
+
+      // Only process testcases that contain a <failure> element
+      if (!block.includes('<failure')) {
+        continue;
+      }
+
+      // Extract testcase attributes (order-independent)
+      const tcTag = block.match(/<testcase\s[^>]*>/);
+      if (!tcTag) continue;
+
+      const cnMatch = tcTag[0].match(/\sclassname="([^"]*)"/);
+      const tnMatch = tcTag[0].match(/\sname="([^"]*)"/);
+
+      const className = cnMatch ? cnMatch[1] : 'unknown';
+      const testName = tnMatch ? tnMatch[1] : 'unknown';
+
+      // Extract failure message
+      const failTag = block.match(/<failure\s[^>]*>/);
+      const msgMatch = failTag ? failTag[0].match(/\smessage="([^"]*)"/) : null;
+      const message = msgMatch ? msgMatch[1] : 'Test failed';
+
+      // Extract stack trace from CDATA or text content
+      const cdataMatch = block.match(/<failure[^>]*>[\s\S]*?<!\[CDATA\[([\s\S]*?)\]\]>/);
+      const textMatch = !cdataMatch ? block.match(/<failure[^>]*>([^<]*)<\/failure>/) : null;
+      const stackTrace = cdataMatch ? cdataMatch[1].trim() : textMatch ? textMatch[1].trim() : undefined;
+
+      failures.push({ testName, className, message, stackTrace: stackTrace || undefined });
+    }
+
+    const passed = totalTests - failureCount - skippedCount;
+
+    return {
+      runName,
+      passed: Math.max(0, passed),
+      failed: failureCount,
+      skipped: skippedCount,
+      duration,
+      failures,
+    };
+  }
+
+  /**
+   * Parse a Unity JSON test result file into a TestResult.
+   */
+  static parseJsonResults(jsonPath: string): TestResult {
+    if (!fs.existsSync(jsonPath)) {
+      throw new Error(`JSON result file not found: ${jsonPath}`);
+    }
+
+    const content = fs.readFileSync(jsonPath, 'utf8');
+    const data = JSON.parse(content);
+
+    return TestResultReporter.parseJsonData(data);
+  }
+
+  /**
+   * Parse Unity JSON test result data into a TestResult.
+   */
+  static parseJsonData(data: any): TestResult {
+    const runName = data.name ?? data.suiteName ?? 'unknown';
+    const passed = data.passed ?? data.passCount ?? 0;
+    const failed = data.failed ?? data.failCount ?? 0;
+    const skipped = data.skipped ?? data.skipCount ?? data.inconclusive ?? 0;
+    const duration = data.duration ?? data.time ?? 0;
+
+    const failures: TestFailure[] = [];
+
+    // Unity test results may have a 'testResults' or 'results' array
+    const results = data.testResults ?? data.results ?? data.children ?? [];
+
+    if (Array.isArray(results)) {
+      for (const result of results) {
+        TestResultReporter.extractFailures(result, failures);
+      }
+    }
+
+    return {
+      runName,
+      passed,
+      failed,
+      skipped,
+      duration,
+      failures,
+    };
+  }
+
+  /**
+   * Recursively extract failures from nested Unity test result JSON.
+   */
+  private static extractFailures(node: any, failures: TestFailure[]): void {
+    if (!node) return;
+
+    const status = (node.result ?? node.status ?? '').toLowerCase();
+    if (status === 'failed' || status === 'failure') {
+      failures.push({
+        testName: node.name ?? node.testName ?? 'unknown',
+        className: node.className ?? node.fullName ?? node.name ?? 'unknown',
+        message: node.message ?? node.output ?? 'Test failed',
+        stackTrace: node.stackTrace ?? node.trace ?? undefined,
+      });
+    }
+
+    // Recurse into children (Unity nests test fixtures inside suites)
+    const children = node.children ?? node.testResults ?? node.results ?? [];
+    if (Array.isArray(children)) {
+      for (const child of children) {
+        TestResultReporter.extractFailures(child, failures);
+      }
+    }
+  }
+
+  /**
+   * Generate a markdown summary table from an array of test results.
+   */
+  static generateSummary(results: TestResult[]): string {
+    if (results.length === 0) {
+      return 'No test results available.';
+    }
+
+    const lines: string[] = [];
+    lines.push('## Test Results Summary');
+    lines.push('');
+    lines.push('| Run | Passed | Failed | Skipped | Duration |');
+    lines.push('|-----|--------|--------|---------|----------|');
+
+    let totalPassed = 0;
+    let totalFailed = 0;
+    let totalSkipped = 0;
+    let totalDuration = 0;
+
+    for (const result of results) {
+      const status = result.failed > 0 ? 'X' : 'OK';
+      const durationStr = TestResultReporter.formatDuration(result.duration);
+      lines.push(
+        `| ${status} ${result.runName} | ${result.passed} | ${result.failed} | ${result.skipped} | ${durationStr} |`,
+      );
+      totalPassed += result.passed;
+      totalFailed += result.failed;
+      totalSkipped += result.skipped;
+      totalDuration += result.duration;
+    }
+
+    lines.push(
+      `| **Total** | **${totalPassed}** | **${totalFailed}** | **${totalSkipped}** | **${TestResultReporter.formatDuration(
+        totalDuration,
+      )}** |`,
+    );
+    lines.push('');
+
+    // Append failure details if any
+    const allFailures = results.flatMap((r) => r.failures.map((f) => ({ ...f, run: r.runName })));
+    if (allFailures.length > 0) {
+      lines.push('### Failures');
+      lines.push('');
+      for (const failure of allFailures) {
+        lines.push(`**${failure.run}** - \`${failure.className}.${failure.testName}\``);
+        lines.push(`> ${failure.message}`);
+        if (failure.stackTrace) {
+          lines.push('```');
+          lines.push(failure.stackTrace.slice(0, 500));
+          lines.push('```');
+        }
+        lines.push('');
+      }
+    }
+
+    return lines.join('\n');
+  }
+
+  /**
+   * Write test results to the output path in the specified format(s).
+   */
+  static writeResults(results: TestResult[], outputPath: string, format: 'junit' | 'json' | 'both'): void {
+    if (!fs.existsSync(outputPath)) {
+      fs.mkdirSync(outputPath, { recursive: true });
+    }
+
+    if (format === 'json' || format === 'both') {
+      const jsonPath = path.join(outputPath, 'test-results.json');
+      fs.writeFileSync(jsonPath, JSON.stringify(results, null, 2), 'utf8');
+    }
+
+    if (format === 'junit' || format === 'both') {
+      const junitPath = path.join(outputPath, 'test-results.xml');
+      const xml = TestResultReporter.toJUnitXml(results);
+      fs.writeFileSync(junitPath, xml, 'utf8');
+    }
+
+    // Always write markdown summary
+    const summaryPath = path.join(outputPath, 'test-summary.md');
+    const summary = TestResultReporter.generateSummary(results);
+    fs.writeFileSync(summaryPath, summary, 'utf8');
+  }
+
+  /**
+   * Convert TestResult array to JUnit XML format.
+   */
+  private static toJUnitXml(results: TestResult[]): string {
+    const lines: string[] = [];
+    lines.push('<?xml version="1.0" encoding="UTF-8"?>');
+    lines.push('<testsuites>');
+
+    for (const result of results) {
+      const total = result.passed + result.failed + result.skipped;
+      lines.push(
+        `  <testsuite name="${TestResultReporter.escapeXml(result.runName)}" tests="${total}" failures="${
+          result.failed
+        }" skipped="${result.skipped}" time="${result.duration.toFixed(3)}">`,
+      );
+
+      // Write failure test cases
+      for (const failure of result.failures) {
+        lines.push(
+          `    <testcase classname="${TestResultReporter.escapeXml(
+            failure.className,
+          )}" name="${TestResultReporter.escapeXml(failure.testName)}">`,
+        );
+        lines.push(`      <failure message="${TestResultReporter.escapeXml(failure.message)}">`);
+        if (failure.stackTrace) {
+          lines.push(`        <![CDATA[${failure.stackTrace}]]>`);
+        }
+        lines.push('      </failure>');
+        lines.push('    </testcase>');
+      }
+
+      lines.push('  </testsuite>');
+    }
+
+    lines.push('</testsuites>');
+    return lines.join('\n');
+  }
+
+  /**
+   * Escape special XML characters.
+   */
+  private static escapeXml(str: string): string {
+    return str
+      .replace(/&/g, '&amp;')
+      .replace(/</g, '&lt;')
+      .replace(/>/g, '&gt;')
+      .replace(/"/g, '&quot;')
+      .replace(/'/g, '&apos;');
+  }
+
+  /**
+   * Format a duration in seconds to a human-readable string.
+   */
+  private static formatDuration(seconds: number): string {
+    if (seconds < 60) {
+      return `${seconds.toFixed(1)}s`;
+    }
+    const minutes = Math.floor(seconds / 60);
+    const remainingSeconds = seconds % 60;
+    return `${minutes}m ${remainingSeconds.toFixed(0)}s`;
+  }
+}
--- a/src/model/orchestrator/services/test-workflow/test-suite-parser.ts
+++ b/src/model/orchestrator/services/test-workflow/test-suite-parser.ts
@@ -0,0 +1,263 @@
+import fs from 'node:fs';
+import YAML from 'yaml';
+import { TestSuiteDefinition, TestRunDefinition } from './test-workflow-types';
+
+/**
+ * Parses and validates YAML-based test suite definition files.
+ * Handles dependency resolution (topological sort) for ordered test run execution.
+ */
+export class TestSuiteParser {
+  /**
+   * Read and parse a YAML test suite definition file.
+   * Validates the structure and returns a typed TestSuiteDefinition.
+   */
+  static parseSuiteFile(filePath: string): TestSuiteDefinition {
+    if (!fs.existsSync(filePath)) {
+      throw new Error(`Test suite file not found: ${filePath}`);
+    }
+
+    const content = fs.readFileSync(filePath, 'utf8');
+    const parsed = YAML.parse(content);
+
+    if (!parsed || typeof parsed !== 'object') {
+      throw new Error(`Invalid YAML in test suite file: ${filePath}`);
+    }
+
+    if (!parsed.name || typeof parsed.name !== 'string') {
+      throw new Error(`Test suite must have a 'name' field (string): ${filePath}`);
+    }
+
+    if (!Array.isArray(parsed.runs) || parsed.runs.length === 0) {
+      throw new Error(`Test suite must have a non-empty 'runs' array: ${filePath}`);
+    }
+
+    const suite: TestSuiteDefinition = {
+      name: parsed.name,
+      description: parsed.description,
+      runs: parsed.runs.map((run: any) => TestSuiteParser.parseRun(run)),
+    };
+
+    const errors = TestSuiteParser.validateSuite(suite);
+    if (errors.length > 0) {
+      throw new Error(`Test suite validation failed:\n  ${errors.join('\n  ')}`);
+    }
+
+    return suite;
+  }
+
+  /**
+   * Parse a single run definition from raw YAML data.
+   */
+  private static parseRun(raw: any): TestRunDefinition {
+    if (!raw || typeof raw !== 'object') {
+      throw new Error(`Each run must be an object`);
+    }
+
+    if (!raw.name || typeof raw.name !== 'string') {
+      throw new Error(`Each run must have a 'name' field (string)`);
+    }
+
+    const run: TestRunDefinition = {
+      name: raw.name,
+    };
+
+    if (raw.needs !== undefined) {
+      if (!Array.isArray(raw.needs)) {
+        throw new Error(`Run '${raw.name}': 'needs' must be an array of strings`);
+      }
+      run.needs = raw.needs;
+    }
+
+    if (raw.editMode !== undefined) {
+      run.editMode = Boolean(raw.editMode);
+    }
+
+    if (raw.playMode !== undefined) {
+      run.playMode = Boolean(raw.playMode);
+    }
+
+    if (raw.builtClient !== undefined) {
+      run.builtClient = Boolean(raw.builtClient);
+    }
+
+    if (raw.builtClientPath !== undefined) {
+      run.builtClientPath = String(raw.builtClientPath);
+    }
+
+    if (raw.filters !== undefined) {
+      if (typeof raw.filters !== 'object' || Array.isArray(raw.filters)) {
+        throw new Error(`Run '${raw.name}': 'filters' must be a key-value object`);
+      }
+      run.filters = {};
+      for (const [key, value] of Object.entries(raw.filters)) {
+        run.filters[key] = String(value);
+      }
+    }
+
+    if (raw.timeout !== undefined) {
+      const timeout = Number(raw.timeout);
+      if (Number.isNaN(timeout) || timeout <= 0) {
+        throw new Error(`Run '${raw.name}': 'timeout' must be a positive number`);
+      }
+      run.timeout = timeout;
+    }
+
+    return run;
+  }
+
+  /**
+   * Resolve run execution order via topological sort based on 'needs' dependencies.
+   * Returns an array of parallel groups -- each group contains runs that can execute concurrently.
+   * Runs within the same group have no inter-dependencies.
+   */
+  static resolveRunOrder(suite: TestSuiteDefinition): TestRunDefinition[][] {
+    const runMap = new Map<string, TestRunDefinition>();
+    for (const run of suite.runs) {
+      runMap.set(run.name, run);
+    }
+
+    // Build adjacency: inDegree counts and dependents map
+    const inDegree = new Map<string, number>();
+    const dependents = new Map<string, string[]>();
+
+    for (const run of suite.runs) {
+      if (!inDegree.has(run.name)) {
+        inDegree.set(run.name, 0);
+      }
+      if (!dependents.has(run.name)) {
+        dependents.set(run.name, []);
+      }
+
+      if (run.needs) {
+        for (const dep of run.needs) {
+          inDegree.set(run.name, (inDegree.get(run.name) ?? 0) + 1);
+          if (!dependents.has(dep)) {
+            dependents.set(dep, []);
+          }
+          dependents.get(dep)!.push(run.name);
+        }
+      }
+    }
+
+    // Kahn's algorithm producing parallel layers
+    const groups: TestRunDefinition[][] = [];
+    let ready = suite.runs.filter((r) => (inDegree.get(r.name) ?? 0) === 0);
+    let processed = 0;
+
+    while (ready.length > 0) {
+      groups.push(ready);
+      processed += ready.length;
+
+      const nextReady: TestRunDefinition[] = [];
+      for (const run of ready) {
+        for (const dep of dependents.get(run.name) ?? []) {
+          const newDegree = (inDegree.get(dep) ?? 1) - 1;
+          inDegree.set(dep, newDegree);
+          if (newDegree === 0) {
+            nextReady.push(runMap.get(dep)!);
+          }
+        }
+      }
+      ready = nextReady;
+    }
+
+    if (processed !== suite.runs.length) {
+      throw new Error(`Circular dependency detected in test suite '${suite.name}'`);
+    }
+
+    return groups;
+  }
+
+  /**
+   * Validate a parsed test suite definition.
+   * Returns an array of validation error messages (empty = valid).
+   */
+  static validateSuite(suite: TestSuiteDefinition): string[] {
+    const errors: string[] = [];
+    const runNames = new Set<string>();
+
+    // Check for duplicate run names
+    for (const run of suite.runs) {
+      if (runNames.has(run.name)) {
+        errors.push(`Duplicate run name: '${run.name}'`);
+      }
+      runNames.add(run.name);
+    }
+
+    // Check that all 'needs' references exist
+    for (const run of suite.runs) {
+      if (run.needs) {
+        for (const dep of run.needs) {
+          if (!runNames.has(dep)) {
+            errors.push(`Run '${run.name}' depends on unknown run '${dep}'`);
+          }
+        }
+
+        // Self-dependency
+        if (run.needs.includes(run.name)) {
+          errors.push(`Run '${run.name}' depends on itself`);
+        }
+      }
+    }
+
+    // Check that at least one test mode is specified per run
+    for (const run of suite.runs) {
+      if (!run.editMode && !run.playMode && !run.builtClient) {
+        errors.push(`Run '${run.name}' must specify at least one of: editMode, playMode, builtClient`);
+      }
+    }
+
+    // Detect circular dependencies via DFS
+    const circularError = TestSuiteParser.detectCircularDependencies(suite);
+    if (circularError) {
+      errors.push(circularError);
+    }
+
+    return errors;
+  }
+
+  /**
+   * Detect circular dependencies using DFS cycle detection.
+   */
+  private static detectCircularDependencies(suite: TestSuiteDefinition): string | null {
+    const adjacency = new Map<string, string[]>();
+    for (const run of suite.runs) {
+      adjacency.set(run.name, run.needs ?? []);
+    }
+
+    const visited = new Set<string>();
+    const visiting = new Set<string>();
+
+    const dfs = (node: string, path: string[]): string | null => {
+      if (visiting.has(node)) {
+        const cycleStart = path.indexOf(node);
+        const cycle = path.slice(cycleStart).concat(node);
+        return `Circular dependency: ${cycle.join(' -> ')}`;
+      }
+      if (visited.has(node)) {
+        return null;
+      }
+
+      visiting.add(node);
+      path.push(node);
+
+      for (const dep of adjacency.get(node) ?? []) {
+        if (adjacency.has(dep)) {
+          const result = dfs(dep, [...path]);
+          if (result) return result;
+        }
+      }
+
+      visiting.delete(node);
+      visited.add(node);
+      return null;
+    };
+
+    for (const run of suite.runs) {
+      const result = dfs(run.name, []);
+      if (result) return result;
+    }
+
+    return null;
+  }
+}
--- a/src/model/orchestrator/services/test-workflow/test-workflow-service.ts
+++ b/src/model/orchestrator/services/test-workflow/test-workflow-service.ts
@@ -0,0 +1,246 @@
+import { exec } from 'node:child_process';
+import { promisify } from 'node:util';
+import path from 'node:path';
+import * as core from '@actions/core';
+import BuildParameters from '../../../build-parameters';
+import { TestSuiteParser } from './test-suite-parser';
+import { TaxonomyFilterService } from './taxonomy-filter-service';
+import { TestResultReporter } from './test-result-reporter';
+import { TestRunDefinition, TestResult } from './test-workflow-types';
+
+const execAsync = promisify(exec);
+
+/**
+ * Main entry point for the test workflow engine.
+ * Orchestrates parsing of YAML suite definitions, resolving run order,
+ * executing test runs via Unity CLI, and collecting structured results.
+ */
+export class TestWorkflowService {
+  /**
+   * Execute a full test suite from a YAML definition file.
+   * Parses the suite, resolves dependency order, executes each parallel
+   * group sequentially (runs within a group execute concurrently), and
+   * collects all results.
+   */
+  static async executeTestSuite(suitePath: string, parameters: BuildParameters): Promise<TestResult[]> {
+    core.info(`[TestWorkflow] Loading test suite from: ${suitePath}`);
+
+    const suite = TestSuiteParser.parseSuiteFile(suitePath);
+    core.info(`[TestWorkflow] Suite '${suite.name}' loaded with ${suite.runs.length} run(s)`);
+
+    if (suite.description) {
+      core.info(`[TestWorkflow] Description: ${suite.description}`);
+    }
+
+    const groups = TestSuiteParser.resolveRunOrder(suite);
+    core.info(`[TestWorkflow] Resolved into ${groups.length} execution group(s)`);
+
+    const allResults: TestResult[] = [];
+    let groupIndex = 0;
+
+    for (const group of groups) {
+      groupIndex++;
+      const runNames = group.map((r) => r.name).join(', ');
+      core.info(`[TestWorkflow] Executing group ${groupIndex}/${groups.length}: [${runNames}]`);
+
+      // Execute runs within a group concurrently
+      const groupResults = await Promise.all(group.map((run) => TestWorkflowService.executeTestRun(run, parameters)));
+
+      allResults.push(...groupResults);
+
+      // Check for failures -- if any run in this group failed, log a warning
+      // but continue to the next group (fail-forward for maximum feedback)
+      const failedRuns = groupResults.filter((r) => r.failed > 0);
+      if (failedRuns.length > 0) {
+        const failedNames = failedRuns.map((r) => r.runName).join(', ');
+        core.warning(`[TestWorkflow] Failures detected in group ${groupIndex}: [${failedNames}]`);
+      }
+    }
+
+    // Generate and output summary
+    const summary = TestResultReporter.generateSummary(allResults);
+    core.info(summary);
+
+    // Write results if output path is configured
+    const resultPath = parameters.testResultPath;
+    const resultFormat = parameters.testResultFormat;
+    if (resultPath) {
+      TestResultReporter.writeResults(allResults, resultPath, resultFormat as 'junit' | 'json' | 'both');
+      core.info(`[TestWorkflow] Results written to: ${resultPath}`);
+    }
+
+    return allResults;
+  }
+
+  /**
+   * Execute a single test run definition.
+   * Builds the Unity CLI arguments based on the run configuration (edit mode,
+   * play mode, built client) and taxonomy filters, executes the command
+   * asynchronously, and parses the result output.
+   *
+   * Uses promisified exec instead of execSync so that Promise.all can
+   * actually run multiple test groups in parallel without blocking the
+   * Node.js event loop.
+   */
+  static async executeTestRun(run: TestRunDefinition, parameters: BuildParameters): Promise<TestResult> {
+    core.info(`[TestWorkflow] Starting run: '${run.name}'`);
+
+    const unityArguments = TestWorkflowService.buildUnityArgs(run, parameters);
+    const timeoutMs = (run.timeout ?? 600) * 1000;
+
+    core.info(`[TestWorkflow] Unity args: ${unityArguments}`);
+
+    const startTime = Date.now();
+
+    try {
+      const resultDirectory = path.join(parameters.testResultPath ?? './test-results', run.name);
+      const resultFile = path.join(resultDirectory, 'results.xml');
+
+      // Build the full Unity command
+      const unityPath = TestWorkflowService.resolveUnityPath(parameters);
+      const command = `"${unityPath}" ${unityArguments} -testResults "${resultFile}"`;
+
+      core.info(`[TestWorkflow] Executing: ${command}`);
+
+      await execAsync(command, {
+        timeout: timeoutMs,
+        maxBuffer: 50 * 1024 * 1024, // 50 MB to handle large Unity output
+        cwd: parameters.projectPath || process.cwd(),
+      });
+
+      const duration = (Date.now() - startTime) / 1000;
+
+      // Parse the result file
+      try {
+        const result = TestResultReporter.parseJUnitResults(resultFile);
+        result.runName = run.name;
+        result.duration = duration;
+
+        return result;
+      } catch {
+        // Result file may not exist if Unity exited early
+        core.warning(`[TestWorkflow] Could not parse results for run '${run.name}' -- result file may be missing`);
+
+        return {
+          runName: run.name,
+          passed: 0,
+          failed: 0,
+          skipped: 0,
+          duration,
+          failures: [],
+        };
+      }
+    } catch (error: any) {
+      const duration = (Date.now() - startTime) / 1000;
+
+      // The promisified exec sets error.killed when the process is terminated
+      // due to timeout, and error.signal will be 'SIGTERM'
+      const isTimeout = error.killed === true || error.signal === 'SIGTERM';
+
+      if (isTimeout) {
+        core.error(`[TestWorkflow] Run '${run.name}' timed out after ${run.timeout ?? 600}s`);
+      } else {
+        core.error(`[TestWorkflow] Run '${run.name}' failed: ${error.message}`);
+      }
+
+      // Try to parse partial results even on failure
+      const resultDirectory = path.join(parameters.testResultPath ?? './test-results', run.name);
+      const resultFile = path.join(resultDirectory, 'results.xml');
+
+      try {
+        const result = TestResultReporter.parseJUnitResults(resultFile);
+        result.runName = run.name;
+        result.duration = duration;
+
+        return result;
+      } catch {
+        return {
+          runName: run.name,
+          passed: 0,
+          failed: 1,
+          skipped: 0,
+          duration,
+          failures: [
+            {
+              testName: isTimeout ? 'Timeout' : 'ExecutionError',
+              className: run.name,
+              message: isTimeout
+                ? `Test run timed out after ${run.timeout ?? 600}s`
+                : error.message ?? 'Unknown execution error',
+              stackTrace: error.stderr ?? undefined,
+            },
+          ],
+        };
+      }
+    }
+  }
+
+  /**
+   * Build Unity CLI arguments for a test run based on its configuration.
+   */
+  static buildUnityArgs(run: TestRunDefinition, parameters: BuildParameters): string {
+    const unityArguments: string[] = ['-batchmode', '-nographics'];
+
+    // Project path
+    if (parameters.projectPath) {
+      unityArguments.push(`-projectPath "${parameters.projectPath}"`);
+    }
+
+    // Test mode
+    if (run.builtClient && run.builtClientPath) {
+      // Built client testing: run tests against a built player
+      unityArguments.push(
+        '-runTests',
+        `-testPlatform StandalonePlayer`,
+        `-assemblyNames Assembly-CSharp-Tests`,
+        `-builtPlayerPath "${run.builtClientPath}"`,
+      );
+    } else if (run.editMode && run.playMode) {
+      // Both modes: run EditMode first, then PlayMode will require a separate invocation
+      // For combined mode, use EditMode (the service handles sequencing)
+      unityArguments.push('-runTests', '-testPlatform EditMode');
+    } else if (run.playMode) {
+      unityArguments.push('-runTests', '-testPlatform PlayMode');
+    } else if (run.editMode) {
+      unityArguments.push('-runTests', '-testPlatform EditMode');
+    }
+
+    // Apply taxonomy filters
+    if (run.filters && Object.keys(run.filters).length > 0) {
+      const filterArguments = TaxonomyFilterService.buildFilterArgs(run.filters);
+      if (filterArguments) {
+        unityArguments.push(filterArguments);
+      }
+    }
+
+    // Target platform
+    if (parameters.targetPlatform) {
+      unityArguments.push(`-buildTarget ${parameters.targetPlatform}`);
+    }
+
+    return unityArguments.join(' ');
+  }
+
+  /**
+   * Resolve the path to the Unity editor executable.
+   */
+  private static resolveUnityPath(parameters: BuildParameters): string {
+    // In CI, Unity path is typically set via environment or the docker container
+    const environmentUnityPath = process.env.UNITY_PATH ?? process.env.UNITY_EDITOR;
+    if (environmentUnityPath) {
+      return environmentUnityPath;
+    }
+
+    // Default paths by platform
+    if (process.platform === 'win32') {
+      return `C:/Program Files/Unity/Hub/Editor/${parameters.editorVersion}/Editor/Unity.exe`;
+    }
+
+    if (process.platform === 'darwin') {
+      return `/Applications/Unity/Hub/Editor/${parameters.editorVersion}/Unity.app/Contents/MacOS/Unity`;
+    }
+
+    // Linux default (Docker container path)
+    return '/opt/unity/Editor/Unity';
+  }
+}
--- a/src/model/orchestrator/services/test-workflow/test-workflow-types.ts
+++ b/src/model/orchestrator/services/test-workflow/test-workflow-types.ts
@@ -0,0 +1,41 @@
+export interface TestSuiteDefinition {
+  name: string;
+  description?: string;
+  runs: TestRunDefinition[];
+}
+
+export interface TestRunDefinition {
+  name: string;
+  needs?: string[];
+  editMode?: boolean;
+  playMode?: boolean;
+  builtClient?: boolean;
+  builtClientPath?: string;
+  filters?: Record<string, string>; // dimension -> comma-separated values or /regex/
+  timeout?: number;
+}
+
+export interface TaxonomyDimension {
+  name: string;
+  values: string[];
+}
+
+export interface TaxonomyDefinition {
+  extensible_groups: TaxonomyDimension[];
+}
+
+export interface TestResult {
+  runName: string;
+  passed: number;
+  failed: number;
+  skipped: number;
+  duration: number;
+  failures: TestFailure[];
+}
+
+export interface TestFailure {
+  testName: string;
+  className: string;
+  message: string;
+  stackTrace?: string;
+}
--- a/src/model/orchestrator/services/test-workflow/test-workflow.test.ts
+++ b/src/model/orchestrator/services/test-workflow/test-workflow.test.ts
@@ -0,0 +1,562 @@
+import fs from 'node:fs';
+import { TestSuiteParser } from './test-suite-parser';
+import { TaxonomyFilterService } from './taxonomy-filter-service';
+import { TestResultReporter } from './test-result-reporter';
+import { TestWorkflowService } from './test-workflow-service';
+import { TestSuiteDefinition, TestResult, TestRunDefinition } from './test-workflow-types';
+
+jest.mock('node:fs');
+jest.mock('@actions/core');
+
+const mockFs = fs as jest.Mocked<typeof fs>;
+
+afterEach(() => {
+  jest.clearAllMocks();
+});
+
+// ============================================================================
+// Test Suite Parser
+// ============================================================================
+
+describe('TestSuiteParser', () => {
+  describe('parseSuiteFile', () => {
+    it('should parse a valid YAML suite file', () => {
+      const yaml = `
+name: pull-request
+description: Fast feedback for pull requests
+runs:
+  - name: fast
+    editMode: true
+    filters:
+      Maturity: Trusted
+      FeedbackSpeed: Fast,Moderate
+    timeout: 300
+  - name: basic
+    needs: [fast]
+    editMode: true
+    playMode: true
+    filters:
+      Maturity: Trusted,Adolescent
+    timeout: 600
+`;
+      mockFs.existsSync.mockReturnValue(true);
+      mockFs.readFileSync.mockReturnValue(yaml);
+
+      const suite = TestSuiteParser.parseSuiteFile('/path/to/suite.yml');
+
+      expect(suite.name).toBe('pull-request');
+      expect(suite.description).toBe('Fast feedback for pull requests');
+      expect(suite.runs).toHaveLength(2);
+      expect(suite.runs[0].name).toBe('fast');
+      expect(suite.runs[0].editMode).toBe(true);
+      expect(suite.runs[0].filters?.Maturity).toBe('Trusted');
+      expect(suite.runs[0].timeout).toBe(300);
+      expect(suite.runs[1].needs).toEqual(['fast']);
+    });
+
+    it('should throw when file does not exist', () => {
+      mockFs.existsSync.mockReturnValue(false);
+
+      expect(() => TestSuiteParser.parseSuiteFile('/missing.yml')).toThrow('not found');
+    });
+
+    it('should throw on invalid YAML', () => {
+      mockFs.existsSync.mockReturnValue(true);
+      mockFs.readFileSync.mockReturnValue('not: valid: yaml: [');
+
+      expect(() => TestSuiteParser.parseSuiteFile('/bad.yml')).toThrow();
+    });
+
+    it('should throw when suite has no name', () => {
+      mockFs.existsSync.mockReturnValue(true);
+      mockFs.readFileSync.mockReturnValue('runs:\n  - name: test\n    editMode: true');
+
+      expect(() => TestSuiteParser.parseSuiteFile('/no-name.yml')).toThrow("'name'");
+    });
+
+    it('should throw when suite has no runs', () => {
+      mockFs.existsSync.mockReturnValue(true);
+      mockFs.readFileSync.mockReturnValue('name: test');
+
+      expect(() => TestSuiteParser.parseSuiteFile('/no-runs.yml')).toThrow("'runs'");
+    });
+
+    it('should throw on invalid needs reference', () => {
+      const yaml = `
+name: bad-deps
+runs:
+  - name: first
+    editMode: true
+    needs: [nonexistent]
+`;
+      mockFs.existsSync.mockReturnValue(true);
+      mockFs.readFileSync.mockReturnValue(yaml);
+
+      expect(() => TestSuiteParser.parseSuiteFile('/bad-deps.yml')).toThrow("unknown run 'nonexistent'");
+    });
+  });
+
+  describe('resolveRunOrder', () => {
+    it('should return single group when no dependencies exist', () => {
+      const suite: TestSuiteDefinition = {
+        name: 'test',
+        runs: [
+          { name: 'a', editMode: true },
+          { name: 'b', playMode: true },
+        ],
+      };
+
+      const groups = TestSuiteParser.resolveRunOrder(suite);
+      expect(groups).toHaveLength(1);
+      expect(groups[0]).toHaveLength(2);
+    });
+
+    it('should resolve linear dependencies into sequential groups', () => {
+      const suite: TestSuiteDefinition = {
+        name: 'test',
+        runs: [
+          { name: 'a', editMode: true },
+          { name: 'b', needs: ['a'], playMode: true },
+          { name: 'c', needs: ['b'], editMode: true },
+        ],
+      };
+
+      const groups = TestSuiteParser.resolveRunOrder(suite);
+      expect(groups).toHaveLength(3);
+      expect(groups[0][0].name).toBe('a');
+      expect(groups[1][0].name).toBe('b');
+      expect(groups[2][0].name).toBe('c');
+    });
+
+    it('should place independent runs in the same group', () => {
+      const suite: TestSuiteDefinition = {
+        name: 'test',
+        runs: [
+          { name: 'root', editMode: true },
+          { name: 'branch-a', needs: ['root'], playMode: true },
+          { name: 'branch-b', needs: ['root'], editMode: true },
+        ],
+      };
+
+      const groups = TestSuiteParser.resolveRunOrder(suite);
+      expect(groups).toHaveLength(2);
+      expect(groups[0]).toHaveLength(1);
+      expect(groups[1]).toHaveLength(2);
+    });
+
+    it('should detect circular dependencies', () => {
+      const suite: TestSuiteDefinition = {
+        name: 'circular',
+        runs: [
+          { name: 'a', needs: ['b'], editMode: true },
+          { name: 'b', needs: ['a'], playMode: true },
+        ],
+      };
+
+      expect(() => TestSuiteParser.resolveRunOrder(suite)).toThrow('Circular dependency');
+    });
+
+    it('should detect three-way circular dependencies', () => {
+      const suite: TestSuiteDefinition = {
+        name: 'circular3',
+        runs: [
+          { name: 'a', needs: ['c'], editMode: true },
+          { name: 'b', needs: ['a'], playMode: true },
+          { name: 'c', needs: ['b'], editMode: true },
+        ],
+      };
+
+      expect(() => TestSuiteParser.resolveRunOrder(suite)).toThrow('Circular dependency');
+    });
+  });
+
+  describe('validateSuite', () => {
+    it('should return no errors for a valid suite', () => {
+      const suite: TestSuiteDefinition = {
+        name: 'valid',
+        runs: [
+          { name: 'a', editMode: true },
+          { name: 'b', needs: ['a'], playMode: true },
+        ],
+      };
+
+      const errors = TestSuiteParser.validateSuite(suite);
+      expect(errors).toHaveLength(0);
+    });
+
+    it('should detect duplicate run names', () => {
+      const suite: TestSuiteDefinition = {
+        name: 'dupes',
+        runs: [
+          { name: 'a', editMode: true },
+          { name: 'a', playMode: true },
+        ],
+      };
+
+      const errors = TestSuiteParser.validateSuite(suite);
+      expect(errors.some((e) => e.includes('Duplicate'))).toBe(true);
+    });
+
+    it('should detect missing test mode', () => {
+      const suite: TestSuiteDefinition = {
+        name: 'no-mode',
+        runs: [{ name: 'empty' }],
+      };
+
+      const errors = TestSuiteParser.validateSuite(suite);
+      expect(errors.some((e) => e.includes('editMode'))).toBe(true);
+    });
+
+    it('should detect self-dependency', () => {
+      const suite: TestSuiteDefinition = {
+        name: 'self-dep',
+        runs: [{ name: 'a', needs: ['a'], editMode: true }],
+      };
+
+      const errors = TestSuiteParser.validateSuite(suite);
+      expect(errors.some((e) => e.includes('depends on itself'))).toBe(true);
+    });
+  });
+});
+
+// ============================================================================
+// Taxonomy Filter Service
+// ============================================================================
+
+describe('TaxonomyFilterService', () => {
+  describe('loadTaxonomy', () => {
+    it('should return built-in dimensions when no file provided', () => {
+      const dimensions = TaxonomyFilterService.loadTaxonomy();
+
+      expect(dimensions.length).toBeGreaterThanOrEqual(7);
+      expect(dimensions.find((d) => d.name === 'Scope')).toBeDefined();
+      expect(dimensions.find((d) => d.name === 'Maturity')).toBeDefined();
+      expect(dimensions.find((d) => d.name === 'FeedbackSpeed')).toBeDefined();
+      expect(dimensions.find((d) => d.name === 'Execution')).toBeDefined();
+      expect(dimensions.find((d) => d.name === 'Rigor')).toBeDefined();
+      expect(dimensions.find((d) => d.name === 'Determinism')).toBeDefined();
+      expect(dimensions.find((d) => d.name === 'IsolationLevel')).toBeDefined();
+    });
+
+    it('should merge custom dimensions from file', () => {
+      const yaml = `
+extensible_groups:
+  - name: SubjectLevel
+    values: [Class, Feature, System, Product]
+  - name: DataScenario
+    values: [HappyPath, EdgeCase]
+`;
+      mockFs.existsSync.mockReturnValue(true);
+      mockFs.readFileSync.mockReturnValue(yaml);
+
+      const dimensions = TaxonomyFilterService.loadTaxonomy('/taxonomy.yml');
+
+      expect(dimensions.find((d) => d.name === 'SubjectLevel')).toBeDefined();
+      expect(dimensions.find((d) => d.name === 'DataScenario')).toBeDefined();
+      // Built-ins should still exist
+      expect(dimensions.find((d) => d.name === 'Scope')).toBeDefined();
+    });
+
+    it('should merge values for existing dimensions', () => {
+      const yaml = `
+extensible_groups:
+  - name: Scope
+    values: [Unit, Integration, Acceptance]
+`;
+      mockFs.existsSync.mockReturnValue(true);
+      mockFs.readFileSync.mockReturnValue(yaml);
+
+      const dimensions = TaxonomyFilterService.loadTaxonomy('/taxonomy.yml');
+      const scope = dimensions.find((d) => d.name === 'Scope');
+
+      expect(scope).toBeDefined();
+      // Should contain built-in + custom values without duplicates
+      expect(scope!.values).toContain('Unit');
+      expect(scope!.values).toContain('Integration');
+      expect(scope!.values).toContain('Acceptance');
+      expect(scope!.values).toContain('System');
+    });
+  });
+
+  describe('buildFilterArgs', () => {
+    it('should return empty string for empty filters', () => {
+      expect(TaxonomyFilterService.buildFilterArgs({})).toBe('');
+    });
+
+    it('should build single-value filter', () => {
+      const result = TaxonomyFilterService.buildFilterArgs({ Maturity: 'Trusted' });
+      expect(result).toBe('--testFilter "Maturity=Trusted"');
+    });
+
+    it('should build multi-value filter with pipe separator', () => {
+      const result = TaxonomyFilterService.buildFilterArgs({ Scope: 'Unit,Integration' });
+      expect(result).toBe('--testFilter "Scope=Unit|Integration"');
+    });
+
+    it('should build regex filter', () => {
+      const result = TaxonomyFilterService.buildFilterArgs({ Maturity: '/Trusted|Adolescent/' });
+      expect(result).toBe('--testFilter "Maturity=~Trusted|Adolescent"');
+    });
+
+    it('should AND multiple dimensions with semicolon', () => {
+      const result = TaxonomyFilterService.buildFilterArgs({
+        Maturity: 'Trusted',
+        Scope: 'Unit',
+      });
+      expect(result).toContain(';');
+      expect(result).toContain('Maturity=Trusted');
+      expect(result).toContain('Scope=Unit');
+    });
+  });
+
+  describe('matchesFilter', () => {
+    it('should match exact value', () => {
+      const match = TaxonomyFilterService.matchesFilter({ Scope: 'Unit', Maturity: 'Trusted' }, { Scope: 'Unit' });
+      expect(match).toBe(true);
+    });
+
+    it('should match comma-separated values', () => {
+      const match = TaxonomyFilterService.matchesFilter({ Scope: 'Integration' }, { Scope: 'Unit,Integration' });
+      expect(match).toBe(true);
+    });
+
+    it('should not match when value is not in list', () => {
+      const match = TaxonomyFilterService.matchesFilter({ Scope: 'End To End' }, { Scope: 'Unit,Integration' });
+      expect(match).toBe(false);
+    });
+
+    it('should require all dimensions to match (AND)', () => {
+      const match = TaxonomyFilterService.matchesFilter(
+        { Scope: 'Unit', Maturity: 'Experimental' },
+        { Scope: 'Unit', Maturity: 'Trusted' },
+      );
+      expect(match).toBe(false);
+    });
+
+    it('should match regex patterns', () => {
+      const match = TaxonomyFilterService.matchesFilter({ Maturity: 'Trusted' }, { Maturity: '/Trusted|Adolescent/' });
+      expect(match).toBe(true);
+    });
+
+    it('should not match when regex does not match', () => {
+      const match = TaxonomyFilterService.matchesFilter(
+        { Maturity: 'Experimental' },
+        { Maturity: '/Trusted|Adolescent/' },
+      );
+      expect(match).toBe(false);
+    });
+
+    it('should return false when test lacks a required dimension', () => {
+      const match = TaxonomyFilterService.matchesFilter({}, { Scope: 'Unit' });
+      expect(match).toBe(false);
+    });
+
+    it('should handle hierarchical dot-notation matching', () => {
+      const match = TaxonomyFilterService.matchesFilter({ Domain: 'Combat.Melee.Sword' }, { Domain: 'Combat.Melee' });
+      expect(match).toBe(true);
+    });
+  });
+});
+
+// ============================================================================
+// Test Result Reporter
+// ============================================================================
+
+describe('TestResultReporter', () => {
+  describe('parseJUnitXml', () => {
+    it('should parse a valid JUnit XML string', () => {
+      const xml = `<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="EditMode" tests="10" failures="2" skipped="1" time="5.432">
+  <testcase classname="MyTests" name="TestA">
+  </testcase>
+  <testcase classname="MyTests" name="TestB">
+    <failure message="Expected true">
+      <![CDATA[at MyTests.TestB() in TestFile.cs:42]]>
+    </failure>
+  </testcase>
+</testsuite>`;
+
+      const result = TestResultReporter.parseJUnitXml(xml);
+
+      expect(result.runName).toBe('EditMode');
+      expect(result.passed).toBe(7);
+      expect(result.failed).toBe(2);
+      expect(result.skipped).toBe(1);
+      expect(result.duration).toBeCloseTo(5.432);
+      expect(result.failures).toHaveLength(1);
+      expect(result.failures[0].testName).toBe('TestB');
+      expect(result.failures[0].className).toBe('MyTests');
+      expect(result.failures[0].message).toBe('Expected true');
+      expect(result.failures[0].stackTrace).toContain('TestFile.cs:42');
+    });
+
+    it('should handle empty test suite', () => {
+      const xml = `<testsuite name="Empty" tests="0" failures="0" time="0.0"></testsuite>`;
+
+      const result = TestResultReporter.parseJUnitXml(xml);
+      expect(result.passed).toBe(0);
+      expect(result.failed).toBe(0);
+      expect(result.failures).toHaveLength(0);
+    });
+  });
+
+  describe('parseJsonData', () => {
+    it('should parse Unity JSON test result data', () => {
+      const data = {
+        name: 'PlayMode',
+        passed: 15,
+        failed: 2,
+        skipped: 3,
+        duration: 12.5,
+        testResults: [{ name: 'FailingTest', className: 'MyClass', result: 'Failed', message: 'Assert failed' }],
+      };
+
+      const result = TestResultReporter.parseJsonData(data);
+      expect(result.runName).toBe('PlayMode');
+      expect(result.passed).toBe(15);
+      expect(result.failed).toBe(2);
+      expect(result.skipped).toBe(3);
+      expect(result.failures).toHaveLength(1);
+      expect(result.failures[0].testName).toBe('FailingTest');
+    });
+  });
+
+  describe('generateSummary', () => {
+    it('should generate a markdown summary table', () => {
+      const results: TestResult[] = [
+        {
+          runName: 'fast',
+          passed: 10,
+          failed: 0,
+          skipped: 2,
+          duration: 5.0,
+          failures: [],
+        },
+        {
+          runName: 'basic',
+          passed: 20,
+          failed: 1,
+          skipped: 0,
+          duration: 30.0,
+          failures: [{ testName: 'TestX', className: 'ClassX', message: 'Expected 1 but got 2' }],
+        },
+      ];
+
+      const summary = TestResultReporter.generateSummary(results);
+
+      expect(summary).toContain('Test Results Summary');
+      expect(summary).toContain('fast');
+      expect(summary).toContain('basic');
+      expect(summary).toContain('Total');
+      expect(summary).toContain('Failures');
+      expect(summary).toContain('TestX');
+    });
+
+    it('should return message when no results available', () => {
+      const summary = TestResultReporter.generateSummary([]);
+      expect(summary).toContain('No test results');
+    });
+  });
+
+  describe('writeResults', () => {
+    it('should create output directory and write files for json format', () => {
+      mockFs.existsSync.mockReturnValue(false);
+      mockFs.mkdirSync.mockReturnValue(undefined as any);
+      mockFs.writeFileSync.mockReturnValue(undefined);
+
+      const results: TestResult[] = [
+        { runName: 'test', passed: 5, failed: 0, skipped: 0, duration: 1.0, failures: [] },
+      ];
+
+      TestResultReporter.writeResults(results, '/output', 'json');
+
+      expect(mockFs.mkdirSync).toHaveBeenCalledWith('/output', { recursive: true });
+      // Should write JSON + summary
+      expect(mockFs.writeFileSync).toHaveBeenCalledTimes(2);
+    });
+
+    it('should write both formats when specified', () => {
+      mockFs.existsSync.mockReturnValue(true);
+      mockFs.writeFileSync.mockReturnValue(undefined);
+
+      const results: TestResult[] = [
+        { runName: 'test', passed: 5, failed: 0, skipped: 0, duration: 1.0, failures: [] },
+      ];
+
+      TestResultReporter.writeResults(results, '/output', 'both');
+
+      // Should write JSON + JUnit XML + summary = 3 calls
+      expect(mockFs.writeFileSync).toHaveBeenCalledTimes(3);
+    });
+  });
+});
+
+// ============================================================================
+// Test Workflow Service - buildUnityArgs
+// ============================================================================
+
+describe('TestWorkflowService', () => {
+  describe('buildUnityArgs', () => {
+    const baseParams = {
+      projectPath: '/project',
+      targetPlatform: 'StandaloneLinux64',
+      testResultPath: './test-results',
+      testResultFormat: 'junit',
+    } as any;
+
+    it('should build EditMode args', () => {
+      const run: TestRunDefinition = { name: 'edit', editMode: true };
+
+      const args = TestWorkflowService.buildUnityArgs(run, baseParams);
+
+      expect(args).toContain('-batchmode');
+      expect(args).toContain('-nographics');
+      expect(args).toContain('-runTests');
+      expect(args).toContain('-testPlatform EditMode');
+      expect(args).toContain('-projectPath');
+    });
+
+    it('should build PlayMode args', () => {
+      const run: TestRunDefinition = { name: 'play', playMode: true };
+
+      const args = TestWorkflowService.buildUnityArgs(run, baseParams);
+
+      expect(args).toContain('-testPlatform PlayMode');
+    });
+
+    it('should build built-client args', () => {
+      const run: TestRunDefinition = {
+        name: 'client',
+        builtClient: true,
+        builtClientPath: './Builds/Linux',
+      };
+
+      const args = TestWorkflowService.buildUnityArgs(run, baseParams);
+
+      expect(args).toContain('-testPlatform StandalonePlayer');
+      expect(args).toContain('-builtPlayerPath');
+      expect(args).toContain('./Builds/Linux');
+    });
+
+    it('should include taxonomy filter args', () => {
+      const run: TestRunDefinition = {
+        name: 'filtered',
+        editMode: true,
+        filters: { Maturity: 'Trusted', Scope: 'Unit,Integration' },
+      };
+
+      const args = TestWorkflowService.buildUnityArgs(run, baseParams);
+
+      expect(args).toContain('--testFilter');
+      expect(args).toContain('Maturity=Trusted');
+    });
+
+    it('should include build target', () => {
+      const run: TestRunDefinition = { name: 'test', editMode: true };
+
+      const args = TestWorkflowService.buildUnityArgs(run, baseParams);
+
+      expect(args).toContain('-buildTarget StandaloneLinux64');
+    });
+  });
+});
--- a/src/model/orchestrator/services/test/README.md
+++ b/src/model/orchestrator/services/test/README.md
@@ -0,0 +1,5 @@
+# Test Workflow Engine
+
+Service for YAML-based test suite definitions, taxonomy filtering, and structured test results.
+
+See GitHub Issue for full specification.
Author	SHA1	Message	Date
frostebite	c5355c5211	fix: replace orchestrator-develop branch references with main The orchestrator-develop branch no longer exists. Update all fallback clone commands and test fixtures to use main instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-09 20:03:29 +00:00
frostebite	70a78dab74	ci: set macOS builds to continue-on-error Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-05 23:33:25 +00:00
frostebite	5e54bcd4dd	fix(testing): use async exec for parallel test group execution Replace execSync with promisified exec so Promise.all actually runs test groups in parallel. Add native timeout support via exec options. Add 50MB maxBuffer for large Unity output. Fix ESLint violations (variable naming, padding lines, array push consolidation). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-05 12:51:52 +00:00
frostebite	118671778f	feat(testing): implement test workflow engine with YAML suites, taxonomy filtering, and structured results (#790 ) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-05 11:52:43 +00:00
frostebite	8d81236939	feat(orchestrator): add test workflow engine placeholder Initial scaffold for the test workflow engine service directory. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-05 09:04:54 +00:00
				`@@ -1 +0,0 @@`
				`export { BuildReliabilityService } from './build-reliability-service';`