agentic-orchestrator 0.1.6 → 0.1.8

package/apps/control-plane/src/application/services/gate-service.ts

@@ -13,7 +13,7 @@ type AnyRecord = Record<string, any>;
 
 type StateUpdater = (
   frontMatter: AnyRecord,
-  body: string
+  body: string,
 ) => Promise<{ frontMatter?: AnyRecord; body?: string }>;
 
 type GateRunResult = {
@@ -33,7 +33,11 @@ function normalizeRepoPathForState(repoRoot: string, absolutePath: string): stri
   return relative;
 }
 
-function deriveStatusFromGateResult(currentStatus: string, mode: string, overall: 'pass' | 'fail'): string {
+function deriveStatusFromGateResult(
+  currentStatus: string,
+  mode: string,
+  overall: 'pass' | 'fail',
+): string {
   if (overall === 'fail') {
     return STATUS.BLOCKED;
   }
@@ -54,7 +58,11 @@ export interface GateServicePort {
   getGatesConfig(): AnyRecord;
   getPolicySnapshot(): AnyRecord;
   readState(featureId: string): Promise<{ frontMatter: AnyRecord }>;
-  updateState(featureId: string, expectedVersion: number | null, updater: StateUpdater): Promise<AnyRecord>;
+  updateState(
+    featureId: string,
+    expectedVersion: number | null,
+    updater: StateUpdater,
+  ): Promise<AnyRecord>;
   worktreePath(featureId: string): string;
   logsPath(featureId: string): string;
   evidencePath(featureId: string): string;
@@ -73,15 +81,15 @@ export class GateService {
       return {
         data: {
           profile: profileName,
-          config: gatesConfig.profiles[profileName] ?? null
-        }
+          config: gatesConfig.profiles[profileName] ?? null,
+        },
       };
     }
 
     return {
       data: {
-        profiles: gatesConfig.profiles
-      }
+        profiles: gatesConfig.profiles,
+      },
     };
   }
 
@@ -89,12 +97,16 @@ export class GateService {
     const profile = this.port.getGatesConfig().profiles[profileName];
     if (!profile || !profile.modes?.[mode]) {
       throw {
-        normalizedResponse: fail(ERROR_CODES.UNKNOWN_GATE_PROFILE_OR_MODE, 'Unknown gate profile or mode', {
-          profile: profileName,
-          mode,
-          retryable: false,
-          requires_human: true
-        })
+        normalizedResponse: fail(
+          ERROR_CODES.UNKNOWN_GATE_PROFILE_OR_MODE,
+          'Unknown gate profile or mode',
+          {
+            profile: profileName,
+            mode,
+            retryable: false,
+            requires_human: true,
+          },
+        ),
       };
     }
     return profile;
@@ -103,7 +115,7 @@ export class GateService {
   async gatesRun(
     featureId: string,
     profileName: string | null,
-    mode: string
+    mode: string,
   ): Promise<{
     data: {
       feature_id: string;
@@ -134,14 +146,14 @@ export class GateService {
     if (isIncrementalMode(mode)) {
       const interpolatedSteps: GateStep[] = originalModeSteps.map((step) => ({
         ...step,
-        cmd: interpolateGateCommands(step.cmd, { base_branch: baseBranch, feature_id: featureId })
+        cmd: interpolateGateCommands(step.cmd, { base_branch: baseBranch, feature_id: featureId }),
       }));
       effectiveProfile = {
         ...profile,
         modes: {
           ...(profileAsGate.modes ?? {}),
-          [mode]: interpolatedSteps
-        }
+          [mode]: interpolatedSteps,
+        },
       };
     }
 
@@ -153,28 +165,33 @@ export class GateService {
       policy: policySnapshot as Parameters<typeof runGateMode>[0]['policy'],
       worktreePath: this.port.worktreePath(featureId),
       logDirectory: this.port.logsPath(featureId),
-      evidenceDirectory: this.port.evidencePath(featureId)
+      evidenceDirectory: this.port.evidencePath(featureId),
     })) as GateRunResult;
 
-    const nextStateStatus = deriveStatusFromGateResult(state.frontMatter.status, mode, runResult.overall);
+    const nextStateStatus = deriveStatusFromGateResult(
+      state.frontMatter.status,
+      mode,
+      runResult.overall,
+    );
 
     await this.port.updateState(featureId, state.frontMatter.version, (frontMatter, body) =>
       Promise.resolve({
         frontMatter: {
           status: nextStateStatus,
-          status_reason: runResult.overall === 'fail' ? `Gate mode ${mode} failed` : frontMatter.status_reason,
+          status_reason:
+            runResult.overall === 'fail' ? `Gate mode ${mode} failed` : frontMatter.status_reason,
           gates: {
             ...frontMatter.gates,
-            [mode]: runResult.overall === 'pass' ? GATE_RESULT.PASS : GATE_RESULT.FAIL
+            [mode]: runResult.overall === 'pass' ? GATE_RESULT.PASS : GATE_RESULT.FAIL,
           },
           evidence: {
             last_gate_mode: mode,
             last_gate_evidence_path: normalizeRepoPathForState(repoRoot, runResult.evidence_path),
-            coverage_status: runResult.coverage_status
-          }
+            coverage_status: runResult.coverage_status,
+          },
         },
-        body
-      })
+        body,
+      }),
     );
 
     if (hasBaseBranchTemplate) {
@@ -182,7 +199,11 @@ export class GateService {
       for (const evidenceFile of [runResult.evidence_path, runResult.latest_path]) {
         const raw = await fs.readFile(evidenceFile, 'utf8');
         const parsed = JSON.parse(raw) as Record<string, unknown>;
-        await fs.writeFile(evidenceFile, `${JSON.stringify({ ...parsed, ...skippedInfo }, null, 2)}\n`, 'utf8');
+        await fs.writeFile(
+          evidenceFile,
+          `${JSON.stringify({ ...parsed, ...skippedInfo }, null, 2)}\n`,
+          'utf8',
+        );
       }
     }
 
@@ -197,12 +218,12 @@ export class GateService {
             coverage: runResult.coverage,
             thresholds: runResult.thresholds,
             retryable: true,
-            requires_human: false
+            requires_human: false,
           },
           {
-            log_path: normalizeRepoPathForState(repoRoot, runResult.latest_path)
-          }
-        )
+            log_path: normalizeRepoPathForState(repoRoot, runResult.latest_path),
+          },
+        ),
       };
     }
 
@@ -213,23 +234,25 @@ export class GateService {
         profile: effectiveProfileName,
         overall: runResult.overall,
         coverage: runResult.coverage,
-        evidence_path: normalizeRepoPathForState(repoRoot, runResult.evidence_path)
+        evidence_path: normalizeRepoPathForState(repoRoot, runResult.evidence_path),
       },
       evidence: {
-        log_path: normalizeRepoPathForState(repoRoot, runResult.latest_path)
-      }
+        log_path: normalizeRepoPathForState(repoRoot, runResult.latest_path),
+      },
     };
   }
 
-  async evidenceLatest(featureId: string): Promise<{ data: { feature_id: string; latest: AnyRecord | null; path?: string } }> {
+  async evidenceLatest(
+    featureId: string,
+  ): Promise<{ data: { feature_id: string; latest: AnyRecord | null; path?: string } }> {
     const repoRoot = this.port.getRepoRoot();
     const evidenceDir = this.port.evidencePath(featureId);
     if (!(await pathExists(evidenceDir))) {
       return {
         data: {
           feature_id: featureId,
-          latest: null
-        }
+          latest: null,
+        },
       };
     }
 
@@ -241,8 +264,8 @@ export class GateService {
       return {
         data: {
           feature_id: featureId,
-          latest: null
-        }
+          latest: null,
+        },
       };
     }
 
@@ -250,7 +273,7 @@ export class GateService {
       files.map(async (file) => {
         const stat = await fs.stat(file);
         return { file, mtimeMs: stat.mtimeMs };
-      })
+      }),
     );
 
     withStats.sort((a, b) => b.mtimeMs - a.mtimeMs);
@@ -261,8 +284,8 @@ export class GateService {
       data: {
         feature_id: featureId,
         latest,
-        path: normalizeRepoPathForState(repoRoot, latestFile)
-      }
+        path: normalizeRepoPathForState(repoRoot, latestFile),
+      },
     };
   }
 }

package/apps/control-plane/src/application/services/instance-isolation-service.ts

@@ -14,5 +14,5 @@ export function computeInstanceId(configPath: string): string {
  * Used when no explicit --config flag is provided.
  */
 export function defaultConfigPath(repoRoot: string): string {
-  return `${repoRoot}/agentic/orchestrator/policy.yaml`;
+  return `${repoRoot}/config/agentic/orchestrator/policy.yaml`;
 }

package/apps/control-plane/src/application/services/issue-tracker-service.ts

@@ -26,7 +26,10 @@ export interface IssueTrackerConfig {
 // --- GitHub adapter (via gh CLI) ---
 
 export type GhRunner = (args: string[]) => Promise<{ stdout: string; exitCode: number }>;
-export type HttpRunner = (url: string, init: RequestInit) => Promise<{ status: number; ok: boolean; body: string }>;
+export type HttpRunner = (
+  url: string,
+  init: RequestInit,
+) => Promise<{ status: number; ok: boolean; body: string }>;
 
 export function createGhRunner(fn?: GhRunner): GhRunner {
   if (fn !== undefined) {
@@ -57,7 +60,7 @@ export function createHttpRunner(fn?: HttpRunner): HttpRunner {
       return {
         status: response.status,
         ok: response.ok,
-        body: await response.text()
+        body: await response.text(),
       };
     } catch {
       return { status: 0, ok: false, body: '' };
@@ -92,9 +95,12 @@ export class GitHubIssueTracker implements IssueTracker {
   async getIssue(issueId: string): Promise<Issue> {
     const repoArgs = this.repo ? ['--repo', this.repo] : [];
     const result = await this.ghRunner([
-      'issue', 'view', issueId,
+      'issue',
+      'view',
+      issueId,
       ...repoArgs,
-      '--json', 'number,title,body,state,url'
+      '--json',
+      'number,title,body,state,url',
     ]);
     if (result.exitCode !== 0 || !result.stdout) {
       return { id: issueId, title: '', body: '', status: 'unknown', url: '' };
@@ -111,7 +117,7 @@ export class GitHubIssueTracker implements IssueTracker {
       title: parsed.title,
       body: parsed.body,
       status: parsed.state.toLowerCase(),
-      url: parsed.url
+      url: parsed.url,
     };
   }
 
@@ -147,7 +153,10 @@ function lowerCaseStatus(value: string): string {
 }
 
 function normalizeAopStatus(status: string): string {
-  return status.trim().toLowerCase().replace(/[^a-z0-9]+/g, '_');
+  return status
+    .trim()
+    .toLowerCase()
+    .replace(/[^a-z0-9]+/g, '_');
 }
 
 function mapDefaultLinearStatus(status: string): string | null {
@@ -198,9 +207,12 @@ export class LinearIssueTracker implements IssueTracker {
     this.httpRunner = createHttpRunner(httpRunner);
   }
 
-  private async graphQl<TData>(query: string, variables: Record<string, unknown>): Promise<TData | null> {
+  private async graphQl<TData>(
+    query: string,
+    variables: Record<string, unknown>,
+  ): Promise<TData | null> {
     const headers: Record<string, string> = {
-      'Content-Type': 'application/json'
+      'Content-Type': 'application/json',
     };
     if (this.token) {
       headers['Authorization'] = `Bearer ${this.token}`;
@@ -208,7 +220,7 @@ export class LinearIssueTracker implements IssueTracker {
     const result = await this.httpRunner(this.baseUrl, {
       method: 'POST',
       headers,
-      body: JSON.stringify({ query, variables })
+      body: JSON.stringify({ query, variables }),
     });
     if (!result.ok || !result.body) {
       return null;
@@ -242,16 +254,16 @@ export class LinearIssueTracker implements IssueTracker {
       state:
         stateRaw && typeof stateRaw === 'object'
           ? {
-              name: readString((stateRaw as Record<string, unknown>)['name'])
+              name: readString((stateRaw as Record<string, unknown>)['name']),
             }
-          : null
+          : null,
     };
   }
 
   private async resolveIssueNode(issueId: string): Promise<LinearIssueNode | null> {
     const byIdentifier = await this.graphQl<{ issueByIdentifier?: unknown }>(
       'query AopIssueByIdentifier($identifier: String!) { issueByIdentifier(identifier: $identifier) { id identifier title description url state { name } } }',
-      { identifier: issueId }
+      { identifier: issueId },
     );
     const identifierIssue = LinearIssueTracker.parseIssueNode(byIdentifier?.issueByIdentifier);
     if (identifierIssue) {
@@ -260,7 +272,7 @@ export class LinearIssueTracker implements IssueTracker {
 
     const byId = await this.graphQl<{ issue?: unknown }>(
       'query AopIssueById($id: String!) { issue(id: $id) { id identifier title description url state { name } } }',
-      { id: issueId }
+      { id: issueId },
     );
     return LinearIssueTracker.parseIssueNode(byId?.issue);
   }
@@ -289,7 +301,7 @@ export class LinearIssueTracker implements IssueTracker {
       title: issue.title,
       body: issue.description,
       status: lowerCaseStatus(issue.state?.name ?? ''),
-      url: issue.url
+      url: issue.url,
     };
   }
 
@@ -304,7 +316,7 @@ export class LinearIssueTracker implements IssueTracker {
     }
     await this.graphQl(
       'mutation AopIssueUpdate($id: String!, $stateId: String!) { issueUpdate(id: $id, input: { stateId: $stateId }) { success } }',
-      { id: issue.id, stateId }
+      { id: issue.id, stateId },
     );
   }
 
@@ -315,7 +327,7 @@ export class LinearIssueTracker implements IssueTracker {
     }
     await this.graphQl(
       'mutation AopCommentCreate($issueId: String!, $body: String!) { commentCreate(input: { issueId: $issueId, body: $body }) { success } }',
-      { issueId: issue.id, body: comment }
+      { issueId: issue.id, body: comment },
     );
   }
 }
@@ -359,13 +371,13 @@ export class JiraIssueTracker implements IssueTracker {
   private async requestJson<T>(
     endpoint: string,
     method: 'GET' | 'POST',
-    body?: Record<string, unknown>
+    body?: Record<string, unknown>,
   ): Promise<T | null> {
     if (!this.baseUrl) {
       return null;
     }
     const headers: Record<string, string> = {
-      Accept: 'application/json'
+      Accept: 'application/json',
     };
     const authHeader = this.authHeader();
     if (authHeader) {
@@ -377,7 +389,7 @@ export class JiraIssueTracker implements IssueTracker {
     const result = await this.httpRunner(`${this.baseUrl}${endpoint}`, {
       method,
       headers,
-      body: body === undefined ? undefined : JSON.stringify(body)
+      body: body === undefined ? undefined : JSON.stringify(body),
     });
     if (!result.ok || !result.body) {
       return null;
@@ -415,36 +427,31 @@ export class JiraIssueTracker implements IssueTracker {
       title: readString(fields.summary),
       body: toJiraDescription(fields.description),
       status: lowerCaseStatus(readString(fields.status?.name)),
-      url: this.baseUrl ? `${this.baseUrl}/browse/${issueId}` : ''
+      url: this.baseUrl ? `${this.baseUrl}/browse/${issueId}` : '',
     };
   }
 
   async updateIssueStatus(issueId: string, status: string): Promise<void> {
-    const transitionsPayload = await this.requestJson<{ transitions?: Array<{ id?: string; name?: string }> }>(
-      `/rest/api/2/issue/${encodeURIComponent(issueId)}/transitions`,
-      'GET'
-    );
+    const transitionsPayload = await this.requestJson<{
+      transitions?: Array<{ id?: string; name?: string }>;
+    }>(`/rest/api/2/issue/${encodeURIComponent(issueId)}/transitions`, 'GET');
     const transitions = transitionsPayload?.transitions ?? [];
     const targetName = this.resolveTransitionName(status);
     const transition = transitions.find(
-      (item) => typeof item.name === 'string' && item.name.toLowerCase() === targetName
+      (item) => typeof item.name === 'string' && item.name.toLowerCase() === targetName,
     );
     if (!transition?.id) {
       return;
     }
-    await this.requestJson(
-      `/rest/api/2/issue/${encodeURIComponent(issueId)}/transitions`,
-      'POST',
-      { transition: { id: transition.id } }
-    );
+    await this.requestJson(`/rest/api/2/issue/${encodeURIComponent(issueId)}/transitions`, 'POST', {
+      transition: { id: transition.id },
+    });
   }
 
   async addComment(issueId: string, comment: string): Promise<void> {
-    await this.requestJson(
-      `/rest/api/2/issue/${encodeURIComponent(issueId)}/comment`,
-      'POST',
-      { body: comment }
-    );
+    await this.requestJson(`/rest/api/2/issue/${encodeURIComponent(issueId)}/comment`, 'POST', {
+      body: comment,
+    });
   }
 }
 
@@ -452,10 +459,14 @@ export class JiraIssueTracker implements IssueTracker {
 export function createIssueTracker(
   config: IssueTrackerConfig | undefined,
   ghRunner?: GhRunner,
-  httpRunner?: HttpRunner
+  httpRunner?: HttpRunner,
 ): IssueTracker | undefined {
-  if (!config) {return undefined;}
-  if (config.enabled === false) {return undefined;}
+  if (!config) {
+    return undefined;
+  }
+  if (config.enabled === false) {
+    return undefined;
+  }
   switch (config.type) {
     case 'github':
       return new GitHubIssueTracker(config.config ?? {}, ghRunner);