agentic-orchestrator 0.1.6 → 0.1.8

package/apps/control-plane/src/core/gates.ts

@@ -73,7 +73,7 @@ class LcovCoverageParser implements CoverageParserStrategy {
     return {
       line: linesFound > 0 ? linesHit / linesFound : 1,
       branch: branchesFound > 0 ? branchesHit / branchesFound : 1,
-      details: { linesFound, linesHit, branchesFound, branchesHit }
+      details: { linesFound, linesHit, branchesFound, branchesHit },
     };
   }
 }
@@ -103,7 +103,7 @@ class JacocoXmlCoverageParser implements CoverageParserStrategy {
     return {
       line: lineCovered + lineMissed > 0 ? lineCovered / (lineCovered + lineMissed) : 1,
       branch: branchCovered + branchMissed > 0 ? branchCovered / (branchCovered + branchMissed) : 1,
-      details: { lineMissed, lineCovered, branchMissed, branchCovered }
+      details: { lineMissed, lineCovered, branchMissed, branchCovered },
     };
   }
 }
@@ -117,7 +117,7 @@ class CoberturaXmlCoverageParser implements CoverageParserStrategy {
     return {
       line: lineRate,
       branch: branchRate,
-      details: { lineRate, branchRate }
+      details: { lineRate, branchRate },
     };
   }
 }
@@ -145,10 +145,22 @@ export class CoverageParserRegistry {
 export class CoverageThresholdPolicy {
   merge(policy: PolicyShape, profileThresholds: Partial<GateThresholds> = {}): GateThresholds {
     return {
-      coverage_line_min: Math.max(policy.testing.coverage.minimums.line, profileThresholds.coverage_line_min ?? 0),
-      coverage_branch_min: Math.max(policy.testing.coverage.minimums.branch, profileThresholds.coverage_branch_min ?? 0),
-      coverage_line_target: Math.max(policy.testing.coverage.targets.line, profileThresholds.coverage_line_target ?? 0),
-      coverage_branch_target: Math.max(policy.testing.coverage.targets.branch, profileThresholds.coverage_branch_target ?? 0)
+      coverage_line_min: Math.max(
+        policy.testing.coverage.minimums.line,
+        profileThresholds.coverage_line_min ?? 0,
+      ),
+      coverage_branch_min: Math.max(
+        policy.testing.coverage.minimums.branch,
+        profileThresholds.coverage_branch_min ?? 0,
+      ),
+      coverage_line_target: Math.max(
+        policy.testing.coverage.targets.line,
+        profileThresholds.coverage_line_target ?? 0,
+      ),
+      coverage_branch_target: Math.max(
+        policy.testing.coverage.targets.branch,
+        profileThresholds.coverage_branch_target ?? 0,
+      ),
     };
   }
 }
@@ -163,7 +175,10 @@ function pickAllowedEnv(envAllowlist: string[] = []): NodeJS.ProcessEnv {
   return env;
 }
 
-export async function parseCoverage(parserConfig: ParserConfig | undefined, worktreePath: string): Promise<CoverageMetrics | null> {
+export async function parseCoverage(
+  parserConfig: ParserConfig | undefined,
+  worktreePath: string,
+): Promise<CoverageMetrics | null> {
   if (!parserConfig || parserConfig.type === 'none') {
     return null;
   }
@@ -221,17 +236,25 @@ export class GateExecutionService {
       env: NodeJS.ProcessEnv;
       worktreePath: string;
       logDirectory: string;
-    }
+    },
   ): Promise<Record<string, unknown>> {
-    const timeoutSeconds = Number(step.timeout_seconds ?? ctx.policy.execution.default_step_timeout_seconds);
+    const timeoutSeconds = Number(
+      step.timeout_seconds ?? ctx.policy.execution.default_step_timeout_seconds,
+    );
     const stepLogPath = path.join(
       ctx.logDirectory,
-      `${Date.now()}-${ctx.mode}-${step.name.replaceAll(/[^a-zA-Z0-9_-]/g, '_')}.log`
+      `${Date.now()}-${ctx.mode}-${step.name.replaceAll(/[^a-zA-Z0-9_-]/g, '_')}.log`,
     );
 
     const retryPolicy = ctx.policy.execution.retry_policy;
     let attempts = 0;
-    let lastResult: CommandResult = { code: 1, signal: null, stdout: '', stderr: '', timeout: false };
+    let lastResult: CommandResult = {
+      code: 1,
+      signal: null,
+      stdout: '',
+      stderr: '',
+      timeout: false,
+    };
     const maxAttempts = retryPolicy.transient_max_retries + 1;
 
     while (attempts < maxAttempts) {
@@ -239,7 +262,7 @@ export class GateExecutionService {
       const result = await runCommand(step.cmd[0], step.cmd.slice(1), {
         cwd: step.cwd ? path.join(ctx.worktreePath, step.cwd) : ctx.worktreePath,
         env: { ...ctx.env, ...step.env },
-        timeoutMs: timeoutSeconds * 1000
+        timeoutMs: timeoutSeconds * 1000,
       });
 
       lastResult = result;
@@ -252,7 +275,7 @@ export class GateExecutionService {
         result.stdout,
         '',
         '## stderr',
-        result.stderr
+        result.stderr,
       ].join('\n');
       await fs.writeFile(stepLogPath, `${logText}\n`, 'utf8');
 
@@ -272,7 +295,7 @@ export class GateExecutionService {
       attempts,
       exit_code: lastResult.code,
       timeout: lastResult.timeout,
-      log_path: stepLogPath
+      log_path: stepLogPath,
     };
   }
 
@@ -284,7 +307,7 @@ export class GateExecutionService {
     policy,
     worktreePath,
     logDirectory,
-    evidenceDirectory
+    evidenceDirectory,
   }: {
     featureId: string;
     mode: string;
@@ -335,7 +358,7 @@ export class GateExecutionService {
             skip_reason: 'dependency_failed',
             parallel_group: step.parallel_group,
             started_at: nowIso(),
-            finished_at: nowIso()
+            finished_at: nowIso(),
           });
         }
 
@@ -348,13 +371,13 @@ export class GateExecutionService {
               policy,
               env,
               worktreePath,
-              logDirectory
+              logDirectory,
             });
             stepResult['parallel_group'] = step.parallel_group;
             stepResult['started_at'] = startedAt;
             stepResult['finished_at'] = nowIso();
             return stepResult;
-          })
+          }),
         );
 
         for (const result of results) {
@@ -379,7 +402,7 @@ export class GateExecutionService {
           policy,
           env,
           worktreePath,
-          logDirectory
+          logDirectory,
         });
 
         stepResults.push({ ...result, started_at: startedAt, finished_at: nowIso() });
@@ -397,14 +420,17 @@ export class GateExecutionService {
     let coverageStatus = 'na';
     let coverageFailure: Record<string, unknown> | null = null;
     if (coverage) {
-      if (coverage.line < thresholds.coverage_line_min || coverage.branch < thresholds.coverage_branch_min) {
+      if (
+        coverage.line < thresholds.coverage_line_min ||
+        coverage.branch < thresholds.coverage_branch_min
+      ) {
         coverageStatus = 'fail';
         overall = 'fail';
         coverageFailure = {
           line: coverage.line,
           branch: coverage.branch,
           min_line: thresholds.coverage_line_min,
-          min_branch: thresholds.coverage_branch_min
+          min_branch: thresholds.coverage_branch_min,
         };
       } else {
         coverageStatus = 'pass';
@@ -421,7 +447,7 @@ export class GateExecutionService {
       coverage_failure: coverageFailure,
       thresholds,
       step_results: stepResults,
-      finished_at: nowIso()
+      finished_at: nowIso(),
     };
 
     const evidencePath = path.join(evidenceDirectory, `gate-${mode}-${Date.now()}.json`);
@@ -436,7 +462,7 @@ export class GateExecutionService {
       coverage_status: coverageStatus,
       evidence_path: evidencePath,
       latest_path: latestPath,
-      thresholds
+      thresholds,
     };
   }
 }
@@ -445,7 +471,7 @@ const defaultCoverageParserRegistry = new CoverageParserRegistry([
   new LcovCoverageParser(),
   new JunitXmlCoverageParser(),
   new JacocoXmlCoverageParser(),
-  new CoberturaXmlCoverageParser()
+  new CoberturaXmlCoverageParser(),
 ]);
 const defaultGateExecutionService = new GateExecutionService(new CoverageThresholdPolicy());
 
@@ -457,7 +483,7 @@ export async function runGateMode({
   policy,
   worktreePath,
   logDirectory,
-  evidenceDirectory
+  evidenceDirectory,
 }: {
   featureId: string;
   mode: string;
@@ -476,6 +502,6 @@ export async function runGateMode({
     policy,
     worktreePath,
     logDirectory,
-    evidenceDirectory
+    evidenceDirectory,
   });
 }

package/apps/control-plane/src/core/git.ts

@@ -32,7 +32,7 @@ export class NodeCommandRunner implements CommandRunner {
         child = spawn(cmd, args, {
           cwd: options.cwd,
           env: options.env,
-          stdio: ['pipe', 'pipe', 'pipe']
+          stdio: ['pipe', 'pipe', 'pipe'],
         });
       } catch (err) {
         return resolve({
@@ -40,7 +40,7 @@ export class NodeCommandRunner implements CommandRunner {
           signal: null,
           stdout: '',
           stderr: (err as Error).message,
-          timeout: false
+          timeout: false,
         });
       }
 
@@ -82,7 +82,7 @@ export class NodeCommandRunner implements CommandRunner {
           signal,
           stdout,
           stderr,
-          timeout: killedByTimeout
+          timeout: killedByTimeout,
         });
       });
     });
@@ -96,7 +96,11 @@ export class NodeGitClient implements GitClient {
     this.commandRunner = commandRunner;
   }
 
-  async runGit(repoRoot: string, args: string[], options: RunCommandOptions = {}): Promise<CommandResult> {
+  async runGit(
+    repoRoot: string,
+    args: string[],
+    options: RunCommandOptions = {},
+  ): Promise<CommandResult> {
     return await this.commandRunner.run('git', args, { ...options, cwd: options.cwd ?? repoRoot });
   }
 }
@@ -104,10 +108,18 @@ export class NodeGitClient implements GitClient {
 export const commandRunner: CommandRunner = new NodeCommandRunner();
 export const gitClient: GitClient = new NodeGitClient(commandRunner);
 
-export function runCommand(cmd: string, args: string[], options: RunCommandOptions = {}): Promise<CommandResult> {
+export function runCommand(
+  cmd: string,
+  args: string[],
+  options: RunCommandOptions = {},
+): Promise<CommandResult> {
   return commandRunner.run(cmd, args, options);
 }
 
-export async function runGit(repoRoot: string, args: string[], options: RunCommandOptions = {}): Promise<CommandResult> {
+export async function runGit(
+  repoRoot: string,
+  args: string[],
+  options: RunCommandOptions = {},
+): Promise<CommandResult> {
   return await gitClient.runGit(repoRoot, args, options);
 }