@h-rig/runtime 0.0.6-alpha.10 → 0.0.6-alpha.12

package/dist/bin/rig-agent-dispatch.js

@@ -2951,45 +2951,6 @@ async function readSourceAwareTaskStatus(projectRoot, taskId, options = {}) {
     return null;
   }
 }
-function updateGithubIssueTaskBySourceIssueId(sourceIssueId, taskId, update, options = {}) {
-  const parsed = sourceIssueId?.trim().match(/^([^/]+)\/([^#]+)#(\d+)$/);
-  if (!parsed || parsed[3] !== taskId) {
-    return false;
-  }
-  applyGithubIssueUpdate(options.ghBinary ?? "gh", options.spawn ?? spawnSync, parsed[3], {
-    sourceIssueId: sourceIssueId.trim(),
-    taskSource: { kind: "github-issues", owner: parsed[1], repo: parsed[2] }
-  }, update);
-  return true;
-}
-function updateSourceAwareTaskConfigTask(projectRoot, taskId, update, options = {}) {
-  const configPath = options.configPath ?? resolve13(projectRoot, ".rig", "task-config.json");
-  const rawEntry = readRawTaskEntry(configPath, taskId);
-  if (!rawEntry) {
-    const configuredFilesPath = readConfiguredFilesTaskSourcePath(projectRoot);
-    return configuredFilesPath ? updateFileBackedTask(projectRoot, configuredFilesPath, taskId, update) : false;
-  }
-  const metadata = readMaterializedTaskMetadata(rawEntry);
-  const source = metadata.taskSource;
-  if (source?.kind === "github-issues") {
-    applyGithubIssueUpdate(options.ghBinary ?? "gh", options.spawn ?? spawnSync, taskId, metadata, update);
-    return true;
-  }
-  if (source?.kind === "files" && source.path) {
-    return updateFileBackedTask(projectRoot, source.path, taskId, update);
-  }
-  if (source?.kind && source.kind !== "files" && source.kind !== "legacy-task-config") {
-    return false;
-  }
-  if (!source && options.allowLocalTaskConfigStatusFallback === false) {
-    return false;
-  }
-  if (typeof update.status !== "string" || update.status.trim().length === 0) {
-    return false;
-  }
-  writeLegacyTaskStatus(configPath, taskId, update.status);
-  return true;
-}
 function readMaterializedTaskMetadata(entry) {
   const rawRig = entry._rig;
   if (!isPlainRecord2(rawRig)) {
@@ -3063,46 +3024,6 @@ function stripLegacyTaskConfigMetadata2(raw) {
   const { validation_descriptions: _legacyDescriptions, _meta, ...tasks } = raw;
   return tasks;
 }
-function writeLegacyTaskStatus(configPath, taskId, status) {
-  const rawConfig = readRawTaskConfig(configPath);
-  if (!rawConfig) {
-    return;
-  }
-  const entry = rawConfig[taskId];
-  if (!isPlainRecord2(entry)) {
-    return;
-  }
-  entry.status = status;
-  writeFileSync5(configPath, `${JSON.stringify(rawConfig, null, 2)}
-`, "utf8");
-}
-function updateFileBackedTask(projectRoot, sourcePath, taskId, update) {
-  const directory = resolve13(projectRoot, sourcePath);
-  const file = findFileBackedTaskFile(directory, taskId);
-  if (!file) {
-    return false;
-  }
-  const raw = JSON.parse(readFileSync6(file, "utf8"));
-  if (!isPlainRecord2(raw)) {
-    return false;
-  }
-  if (update.status)
-    raw.status = update.status;
-  if (update.title !== undefined)
-    raw.title = update.title;
-  if (update.body !== undefined)
-    raw.body = update.body;
-  if (update.comment?.trim()) {
-    const existing = Array.isArray(raw.comments) ? raw.comments : [];
-    raw.comments = [
-      ...existing,
-      { body: update.comment, createdAt: new Date().toISOString(), source: "rig" }
-    ];
-  }
-  writeFileSync5(file, `${JSON.stringify(raw, null, 2)}
-`, "utf8");
-  return true;
-}
 function listFileBackedTasks(projectRoot, sourcePath) {
   const directory = resolve13(projectRoot, sourcePath);
   if (!existsSync13(directory)) {
@@ -3170,26 +3091,6 @@ function readGithubIssueTask(bin, spawnFn, id, metadata, rawEntry) {
   ], spawnFn);
   return githubIssueToTask(issue, source, rawEntry);
 }
-function applyGithubIssueUpdate(bin, spawnFn, id, metadata, update) {
-  const source = requireGithubIssueSource(metadata, id);
-  const repo = `${source.owner}/${source.repo}`;
-  if (update.status) {
-    applyGithubIssueStatus(bin, repo, spawnFn, id, update.status);
-  }
-  if (typeof update.comment === "string" && update.comment.trim().length > 0) {
-    runGhVoid(bin, ["issue", "comment", String(id), "--repo", repo, "--body", update.comment], spawnFn);
-  }
-  const editArgs = ["issue", "edit", String(id), "--repo", repo];
-  if (typeof update.title === "string" && update.title.trim().length > 0) {
-    editArgs.push("--title", update.title.trim());
-  }
-  if (typeof update.body === "string") {
-    editArgs.push("--body", update.body);
-  }
-  if (editArgs.length > 5) {
-    runGhVoid(bin, editArgs, spawnFn);
-  }
-}
 function requireGithubIssueSource(metadata, id) {
   const source = metadata.taskSource;
   if (source?.kind === "github-issues" && source.owner && source.repo) {
@@ -3249,73 +3150,6 @@ function githubStatusFor(issue) {
     return "cancelled";
   return "open";
 }
-function applyGithubIssueStatus(bin, repo, spawnFn, id, status) {
-  if (status === "closed") {
-    runGhVoid(bin, ["issue", "close", String(id), "--repo", repo], spawnFn);
-    return;
-  }
-  const targetLabel = statusLabelFor(status);
-  for (const label of STATUS_LABELS) {
-    if (targetLabel !== null && label === targetLabel) {
-      continue;
-    }
-    try {
-      runGhVoid(bin, ["issue", "edit", String(id), "--repo", repo, "--remove-label", label], spawnFn);
-    } catch {}
-  }
-  if (targetLabel !== null) {
-    try {
-      runGhVoid(bin, ["issue", "edit", String(id), "--repo", repo, "--add-label", targetLabel], spawnFn);
-    } catch (error) {
-      const message = error instanceof Error ? error.message : String(error);
-      if (!/not found/i.test(message)) {
-        throw error;
-      }
-      ensureStatusLabel(bin, repo, spawnFn, targetLabel);
-      runGhVoid(bin, ["issue", "edit", String(id), "--repo", repo, "--add-label", targetLabel], spawnFn);
-    }
-  }
-}
-function statusLabelFor(status) {
-  switch (status) {
-    case "in_progress":
-      return "in-progress";
-    case "blocked":
-      return "blocked";
-    case "ready":
-      return "ready";
-    case "under_review":
-      return "under-review";
-    case "failed":
-      return "failed";
-    case "cancelled":
-      return "cancelled";
-    case "open":
-      return null;
-    default:
-      throw new Error(`unsupported status: ${status}`);
-  }
-}
-function ensureStatusLabel(bin, repo, spawnFn, label) {
-  try {
-    runGhVoid(bin, [
-      "label",
-      "create",
-      label,
-      "--repo",
-      repo,
-      "--color",
-      "6f42c1",
-      "--description",
-      "Task status managed by Rig"
-    ], spawnFn);
-  } catch (error) {
-    const message = error instanceof Error ? error.message : String(error);
-    if (!/already exists/i.test(message)) {
-      throw error;
-    }
-  }
-}
 function selectedGitHubEnv() {
   const token = process.env.RIG_GITHUB_SELECTED_TOKEN?.trim() || process.env.RIG_GITHUB_TOKEN?.trim() || "";
   return { GH_TOKEN: token, GITHUB_TOKEN: token, RIG_GITHUB_TOKEN: token };
@@ -3331,10 +3165,6 @@ function runGh(bin, args, spawnFn) {
   }
   return JSON.parse(res.stdout);
 }
-function runGhVoid(bin, args, spawnFn) {
-  const res = spawnFn(bin, [...args], ghSpawnOptions());
-  assertGhSuccess(args, res);
-}
 function assertGhSuccess(args, res) {
   if (res.error) {
     const msg = res.error.message ?? String(res.error);
@@ -3372,19 +3202,6 @@ function isPlainRecord2(candidate) {
 function hasRunnableTaskSource(source) {
   return Boolean(source && typeof source === "object" && !Array.isArray(source));
 }
-function cleanString(value) {
-  return typeof value === "string" && value.trim().length > 0 ? value.trim() : null;
-}
-function taskIdFromSourceIssueId(value) {
-  const raw = cleanString(value);
-  if (!raw)
-    return null;
-  const issueNumber = raw.match(/#([^#\s]+)$/)?.[1];
-  return issueNumber ?? raw;
-}
-function resolveSourceTaskId(taskId, sourceTask) {
-  return cleanString(sourceTask?.id) ?? taskIdFromSourceIssueId(sourceTask?.sourceIssueId) ?? taskIdFromSourceIssueId(sourceTask?.source_issue_id) ?? taskId;
-}
 async function getPluginTask(projectRoot, taskId) {
   const ctx = await buildPluginHostContext(projectRoot);
   const [source] = ctx?.taskSourceRegistry.list() ?? [];
@@ -3409,91 +3226,6 @@ async function readConfiguredTaskSourceTask(projectRoot, taskId) {
     task
   };
 }
-async function updatePluginTaskSourceTask(projectRoot, taskId, update) {
-  const ctx = await buildPluginHostContext(projectRoot);
-  const [source] = ctx?.taskSourceRegistry.list() ?? [];
-  if (!hasRunnableTaskSource(source)) {
-    return ctx ? { taskId, updated: false, source: "none", sourceKind: null, status: null } : null;
-  }
-  if (source.updateTask) {
-    await source.updateTask(taskId, update);
-  } else if (update.status && source.updateStatus) {
-    await source.updateStatus(taskId, update.status);
-  } else {
-    return {
-      taskId,
-      updated: false,
-      source: "plugin",
-      sourceKind: source.kind,
-      status: null
-    };
-  }
-  const status = source.get ? (await source.get(taskId))?.status ?? update.status ?? null : update.status ?? null;
-  return {
-    taskId,
-    updated: true,
-    source: "plugin",
-    sourceKind: source.kind,
-    status
-  };
-}
-async function updateConfiguredTaskSourceTask(projectRoot, input) {
-  const taskId = resolveSourceTaskId(input.taskId, input.sourceTask);
-  let pluginResult = null;
-  try {
-    pluginResult = await updatePluginTaskSourceTask(projectRoot, taskId, input.update);
-  } catch (error) {
-    const fallbackUpdated = updateSourceAwareTaskConfigTask(projectRoot, taskId, input.update, {
-      allowLocalTaskConfigStatusFallback: false
-    });
-    if (!fallbackUpdated) {
-      throw error;
-    }
-    return {
-      taskId,
-      updated: true,
-      source: "compat",
-      sourceKind: null,
-      status: await readSourceAwareTaskStatus(projectRoot, taskId)
-    };
-  }
-  if (pluginResult) {
-    return pluginResult;
-  }
-  const updated = updateSourceAwareTaskConfigTask(projectRoot, taskId, input.update);
-  return {
-    taskId,
-    updated,
-    source: updated ? "compat" : "none",
-    sourceKind: null,
-    status: await readSourceAwareTaskStatus(projectRoot, taskId)
-  };
-}
-function buildTaskRunLifecycleComment(input) {
-  const lines = [
-    "<!-- rig:status-comment -->",
-    `### Rig status: ${input.status}`,
-    "",
-    input.summary,
-    "",
-    `- Run: ${input.runId}`,
-    `- Status: ${input.status}`
-  ];
-  if (input.errorText?.trim()) {
-    lines.push(`- Error: ${input.errorText.trim()}`);
-  }
-  if (input.runtimeWorkspace?.trim()) {
-    lines.push(`- Runtime workspace: ${input.runtimeWorkspace.trim()}`);
-  }
-  if (input.logsDir?.trim()) {
-    lines.push(`- Logs: ${input.logsDir.trim()}`);
-  }
-  if (input.sessionDir?.trim()) {
-    lines.push(`- Session: ${input.sessionDir.trim()}`);
-  }
-  return lines.join(`
-`);
-}
 
 // packages/runtime/src/control-plane/native/task-state.ts
 import { existsSync as existsSync17, readFileSync as readFileSync8, readdirSync as readdirSync2, statSync as statSync4, writeFileSync as writeFileSync6 } from "fs";
@@ -9319,15 +9051,13 @@ async function runAgentWrapper(options = {}) {
     throw error;
   }
   const serverManagedRun = Boolean(process.env.RIG_SERVER_RUN_ID?.trim());
-  if (exitCode === 0 && serverManagedRun) {
-    await updateTaskSourceAfterRun(monorepoRoot, taskId, runtime);
-  }
   const taskClosed = await isTaskClosed(monorepoRoot, taskId);
   const finalExitCode = resolveFinalProviderExitCode({
     providerExitCode: exitCode,
     taskClosed,
     serverManagedRun
   });
+  const handoffRequired = exitCode !== 0 || !taskClosed && !serverManagedRun;
   emitWrapperEvent("provider.completed", {
     provider,
     runtimeId: runtime.id,
@@ -9336,24 +9066,21 @@ async function runAgentWrapper(options = {}) {
     providerExitCode: exitCode,
     taskClosed,
     serverManagedRun,
-    handoffRequired: exitCode !== 0 || !taskClosed
+    handoffRequired
   });
-  if (exitCode !== 0 || !taskClosed) {
+  if (handoffRequired) {
     recordRuntimeHandoff(projectRoot, runtime, taskId, exitCode);
   }
-  if (exitCode === 0 && !taskClosed && serverManagedRun) {
-    console.error(`[rig-agent] Server-managed task run cannot finish successfully while ${taskId} is still open.`);
-  }
   return finalExitCode;
 }
 function resolveFinalProviderExitCode(input) {
   if (input.providerExitCode !== 0) {
     return input.providerExitCode;
   }
-  if (input.taskClosed) {
+  if (input.taskClosed || input.serverManagedRun) {
     return 0;
   }
-  return input.serverManagedRun ? 2 : 0;
+  return 0;
 }
 function shouldBypassProviderSandboxOnPlatform(provider, platform) {
   if (platform !== "darwin") {
@@ -9584,41 +9311,6 @@ async function readPluginTaskStatus(projectRoot, taskId) {
     return readSourceAwareTaskStatus(projectRoot, taskId);
   }
 }
-async function updateTaskSourceAfterRun(projectRoot, taskId, runtime) {
-  const comment = buildTaskRunLifecycleComment({
-    runId: process.env.RIG_SERVER_RUN_ID || "(unknown)",
-    status: "closed",
-    summary: "Rig task run completed and closed this task.",
-    runtimeWorkspace: runtime.workspaceDir,
-    logsDir: runtime.logsDir,
-    sessionDir: runtime.sessionDir
-  });
-  try {
-    const result = await updateConfiguredTaskSourceTask(projectRoot, {
-      taskId,
-      update: { status: "closed", comment }
-    });
-    if (result.updated) {
-      return;
-    }
-    throw new Error(result.source === "plugin" || result.sourceKind ? `configured task source${result.sourceKind ? ` (${result.sourceKind})` : ""} does not support lifecycle updates` : "no source-aware task source is configured");
-  } catch (error) {
-    let fallbackUpdated = false;
-    try {
-      const sourceIssueId = loadRuntimeContextFromEnv()?.sourceTask?.sourceIssueId;
-      fallbackUpdated = updateGithubIssueTaskBySourceIssueId(sourceIssueId, taskId, { status: "closed", comment });
-      fallbackUpdated = updateSourceAwareTaskConfigTask(projectRoot, taskId, { status: "closed", comment }) || fallbackUpdated;
-    } catch (fallbackError) {
-      console.error(`[rig-agent] Source-aware compatibility update also failed for ${taskId}: ${fallbackError instanceof Error ? fallbackError.message : String(fallbackError)}`);
-    }
-    const message = `[rig-agent] Failed to update task source for ${taskId}${fallbackUpdated ? "; applied source-aware compatibility update" : ""}: ${error instanceof Error ? error.message : String(error)}`;
-    if (fallbackUpdated) {
-      console.log(message);
-    } else {
-      console.error(message);
-    }
-  }
-}
 function recordRuntimeHandoff(hostProjectRoot, runtime, taskId, exitCode) {
   const handoffDir = resolve35(hostProjectRoot, ".rig/runtime/handoffs");
   mkdirSync19(handoffDir, { recursive: true });

package/dist/bin/rig-agent.js

@@ -7278,8 +7278,9 @@ function defaultPrCloseoutLine(taskId, repoNameWithOwner) {
   const sourceIssueId = loadRuntimeContextFromEnv()?.sourceTask?.sourceIssueId;
   if (sourceIssueId) {
     const match = sourceIssueId.match(/^([^#]+)#(\d+)$/);
-    if (match) {
-      const [, sourceRepo, issueNumber] = match;
+    if (match?.[1] && match[2]) {
+      const sourceRepo = match[1];
+      const issueNumber = match[2];
       return sourceRepo.toLowerCase() === repoNameWithOwner.toLowerCase() ? `Closes #${issueNumber}` : `Closes ${sourceRepo}#${issueNumber}`;
     }
   }

package/dist/src/control-plane/agent-wrapper.js

@@ -9319,15 +9319,13 @@ async function runAgentWrapper(options = {}) {
     throw error;
   }
   const serverManagedRun = Boolean(process.env.RIG_SERVER_RUN_ID?.trim());
-  if (exitCode === 0 && serverManagedRun) {
-    await updateTaskSourceAfterRun(monorepoRoot, taskId, runtime);
-  }
   const taskClosed = await isTaskClosed(monorepoRoot, taskId);
   const finalExitCode = resolveFinalProviderExitCode({
     providerExitCode: exitCode,
     taskClosed,
     serverManagedRun
   });
+  const handoffRequired = exitCode !== 0 || !taskClosed && !serverManagedRun;
   emitWrapperEvent("provider.completed", {
     provider,
     runtimeId: runtime.id,
@@ -9336,24 +9334,21 @@ async function runAgentWrapper(options = {}) {
     providerExitCode: exitCode,
     taskClosed,
     serverManagedRun,
-    handoffRequired: exitCode !== 0 || !taskClosed
+    handoffRequired
   });
-  if (exitCode !== 0 || !taskClosed) {
+  if (handoffRequired) {
     recordRuntimeHandoff(projectRoot, runtime, taskId, exitCode);
   }
-  if (exitCode === 0 && !taskClosed && serverManagedRun) {
-    console.error(`[rig-agent] Server-managed task run cannot finish successfully while ${taskId} is still open.`);
-  }
   return finalExitCode;
 }
 function resolveFinalProviderExitCode(input) {
   if (input.providerExitCode !== 0) {
     return input.providerExitCode;
   }
-  if (input.taskClosed) {
+  if (input.taskClosed || input.serverManagedRun) {
     return 0;
   }
-  return input.serverManagedRun ? 2 : 0;
+  return 0;
 }
 function shouldBypassProviderSandboxOnPlatform(provider, platform) {
   if (platform !== "darwin") {
@@ -9587,8 +9582,8 @@ async function readPluginTaskStatus(projectRoot, taskId) {
 async function updateTaskSourceAfterRun(projectRoot, taskId, runtime) {
   const comment = buildTaskRunLifecycleComment({
     runId: process.env.RIG_SERVER_RUN_ID || "(unknown)",
-    status: "closed",
-    summary: "Rig task run completed and closed this task.",
+    status: "completed",
+    summary: "Rig task run completed; source closeout waits for approved PR merge.",
     runtimeWorkspace: runtime.workspaceDir,
     logsDir: runtime.logsDir,
     sessionDir: runtime.sessionDir
@@ -9596,7 +9591,7 @@ async function updateTaskSourceAfterRun(projectRoot, taskId, runtime) {
   try {
     const result = await updateConfiguredTaskSourceTask(projectRoot, {
       taskId,
-      update: { status: "closed", comment }
+      update: { status: "completed", comment }
     });
     if (result.updated) {
       return;
@@ -9606,8 +9601,8 @@ async function updateTaskSourceAfterRun(projectRoot, taskId, runtime) {
     let fallbackUpdated = false;
     try {
       const sourceIssueId = loadRuntimeContextFromEnv()?.sourceTask?.sourceIssueId;
-      fallbackUpdated = updateGithubIssueTaskBySourceIssueId(sourceIssueId, taskId, { status: "closed", comment });
-      fallbackUpdated = updateSourceAwareTaskConfigTask(projectRoot, taskId, { status: "closed", comment }) || fallbackUpdated;
+      fallbackUpdated = updateGithubIssueTaskBySourceIssueId(sourceIssueId, taskId, { status: "completed", comment });
+      fallbackUpdated = updateSourceAwareTaskConfigTask(projectRoot, taskId, { status: "completed", comment }) || fallbackUpdated;
     } catch (fallbackError) {
       console.error(`[rig-agent] Source-aware compatibility update also failed for ${taskId}: ${fallbackError instanceof Error ? fallbackError.message : String(fallbackError)}`);
     }