@h-rig/cli 0.0.6-alpha.7 → 0.0.6-alpha.70

package/dist/src/commands/task-report-bug.js

@@ -7,12 +7,19 @@ import pc from "picocolors";
 
 // packages/cli/src/runner.ts
 import { EventBus } from "@rig/runtime/control-plane/runtime/events";
-import { CliError } from "@rig/runtime/control-plane/errors";
+import { CliError as RuntimeCliError } from "@rig/runtime/control-plane/errors";
 import { evaluate, loadPolicy, resolveAction } from "@rig/runtime/control-plane/runtime/guard";
-import { PluginManager } from "@rig/runtime/control-plane/runtime/plugins";
-import { loadRuntimeContextFromEnv } from "@rig/runtime/control-plane/runtime/context";
 import { buildBinary } from "@rig/runtime/control-plane/runtime/isolation";
-import { CliError as CliError2 } from "@rig/runtime/control-plane/errors";
+
+class CliError extends RuntimeCliError {
+  hint;
+  constructor(message, exitCode = 1, options = {}) {
+    super(message, exitCode);
+    if (options.hint?.trim()) {
+      this.hint = options.hint.trim();
+    }
+  }
+}
 function formatCommand(parts) {
   return parts.map((part) => /[^a-zA-Z0-9_./:-]/.test(part) ? JSON.stringify(part) : part).join(" ");
 }
@@ -36,7 +43,7 @@ function takeOption(args, option) {
     if (current === option) {
       const next = args[index + 1];
       if (!next || next.startsWith("-")) {
-        throw new CliError(`Missing value for ${option}`);
+        throw new CliError(`Missing value for ${option}`, 1, { hint: `Provide a value after ${option}, e.g. \`${option} <value>\`.` });
       }
       value = next;
       index += 1;
@@ -241,15 +248,15 @@ function buildBugReportMarkdown(input, browser, screenshots, assets) {
     ...input.issueId ? [
       `- Canonical task assets live under \`artifacts/${input.issueId}/bug-report/\`.`,
       `- Start with \`artifacts/${input.issueId}/bug-report/task.md\` and the files in \`artifacts/${input.issueId}/bug-report/assets/\`.`,
-      browserRequired ? `- Run \`bun run rig task info --task ${input.issueId}\` to confirm browser wiring before debugging.` : `- Run \`bun run rig task info --task ${input.issueId}\` to confirm scope and artifact links before debugging.`
+      browserRequired ? `- Run \`rig task info --task ${input.issueId}\` to confirm browser wiring before debugging.` : `- Run \`rig task info --task ${input.issueId}\` to confirm scope and artifact links before debugging.`
     ] : [
-      "- Draft-only report: convert this into a beads task before assigning it to an agent run."
+      "- Draft-only report: convert this into a Rig task before assigning it to an agent run."
     ],
     "",
     "## Validation",
     "",
     "```bash",
-    ...input.issueId ? [`bun run rig task validate --task ${input.issueId}`] : [],
+    ...input.issueId ? [`rig task validate --task ${input.issueId}`] : [],
     ...browserRequired ? [
       "bun run app:check:browser:hp-next",
       "bun run app:e2e:browser:hp-next"
@@ -344,7 +351,7 @@ async function executeTaskReportBug(context, args) {
   const noBrowserFlag = takeFlag(pending, "--no-browser");
   pending = noBrowserFlag.rest;
   if (browserFlag.value && noBrowserFlag.value) {
-    throw new CliError2("Pass only one of --browser or --no-browser.");
+    throw new CliError("Pass only one of --browser or --no-browser.", 1, { hint: "Re-run `rig task report-bug` with just one of the flags." });
   }
   const overwriteFlag = takeFlag(pending, "--overwrite");
   pending = overwriteFlag.rest;
@@ -396,7 +403,7 @@ async function executeTaskReportBug(context, args) {
   pending = outputRootResult.rest;
   const slugResult = takeOption(pending, "--slug");
   pending = slugResult.rest;
-  requireNoExtraArgs(pending, "bun run rig report-bug [--no-prompt] [--no-beads] [--browser|--no-browser] --title <text> --url <url> [--asset <dragged-file>]");
+  requireNoExtraArgs(pending, "rig report-bug [--no-prompt] [--no-beads] [--browser|--no-browser] --title <text> --url <url> [--asset <dragged-file>]");
   let draft = {
     outputRoot: outputRootResult.value || "",
     title: titleResult.value,
@@ -428,7 +435,7 @@ async function executeTaskReportBug(context, args) {
   const shouldPrompt = !noPromptFlag.value;
   if (shouldPrompt) {
     if (context.outputMode !== "text" || !process.stdin.isTTY || !process.stdout.isTTY) {
-      throw new CliError2("Interactive bug reporting requires a TTY in text mode. Pass --no-prompt with flags for automation.");
+      throw new CliError("Interactive bug reporting requires a TTY in text mode. Pass --no-prompt with flags for automation.");
     }
     draft = await promptForBugReportDetails(draft);
   }
@@ -501,7 +508,7 @@ async function executeTaskReportBug(context, args) {
     console.log(`Evidence assets: ${result.assetDir}`);
     if (taskConfigPath) {
       console.log(`Task config: ${taskConfigPath}`);
-      console.log(`Run: bun run rig task info --task ${issueId}`);
+      console.log(`Run: rig task info --task ${issueId}`);
     }
   }
   return {
@@ -532,7 +539,7 @@ function validateBugReportInputFiles(input) {
   for (const { path, label } of files.map((item) => ({ ...item, path: item.path.trim() })).filter((item) => item.path)) {
     const source = resolve3(input.projectRoot, path);
     if (!existsSync2(source)) {
-      throw new CliError2(`${label} does not exist: ${source}`);
+      throw new CliError(`${label} does not exist: ${source}`, 1, { hint: "Check the path and re-run `rig task report-bug` with an existing file." });
     }
   }
 }
@@ -572,7 +579,7 @@ function createBugReportBeadsTask(monorepoRoot, input, draft) {
   const issue = tryRunBrJson(monorepoRoot, command);
   if (!issue.ok) {
     if (!isRecoverableBeadsJsonlFailure(issue.error)) {
-      throw new CliError2(issue.error);
+      throw new CliError(issue.error);
     }
     return {
       issueId: generateBugReportIssueId(monorepoRoot, input.title),
@@ -583,7 +590,7 @@ function createBugReportBeadsTask(monorepoRoot, input, draft) {
   }
   const issueId = issue.value.id?.trim();
   if (!issueId) {
-    throw new CliError2("br create did not return an issue id.");
+    throw new CliError("br create did not return an issue id.", 1, { hint: "Re-run `rig task report-bug`; if it persists, file the bug via your task source directly." });
   }
   return {
     issueId,
@@ -690,12 +697,12 @@ ${serializedEntry}
   }
   const insertAt = raw.lastIndexOf("}");
   if (insertAt < 0) {
-    throw new CliError2(`Invalid task config JSON object: ${taskConfigPath}`);
+    throw new CliError(`Invalid task config JSON object: ${taskConfigPath}`, 1, { hint: "Fix or reset that file (it must be a JSON object), then re-run `rig task report-bug`." });
   }
   const before = raw.slice(0, insertAt).replace(/\s*$/, "");
   const after = raw.slice(insertAt + 1).trim();
   if (after) {
-    throw new CliError2(`Invalid trailing content in task config: ${taskConfigPath}`);
+    throw new CliError(`Invalid trailing content in task config: ${taskConfigPath}`, 1, { hint: "Remove the trailing content so the file is a single JSON object, then retry." });
   }
   const comma = before.trim() === "{" ? "" : ",";
   writeFileSync2(taskConfigPath, `${before}${comma}
@@ -763,7 +770,7 @@ function generateBugReportIssueId(monorepoRoot, title) {
       return id;
     }
   }
-  throw new CliError2("Could not generate a unique bug task id.");
+  throw new CliError("Could not generate a unique bug task id.", 1, { hint: "Re-run `rig task report-bug`; pass an explicit title to vary the generated id." });
 }
 function isRecoverableBeadsJsonlFailure(message) {
   return /Invalid JSON|missing field|JSONL is newer than DB|stale database/i.test(message);
@@ -771,7 +778,7 @@ function isRecoverableBeadsJsonlFailure(message) {
 function runBrJson(cwd, command) {
   const result = tryRunBrJson(cwd, command);
   if (!result.ok) {
-    throw new CliError2(result.error);
+    throw new CliError(result.error);
   }
   return result.value;
 }
@@ -969,7 +976,7 @@ async function promptBugSelect(message, options, initialValue) {
 function unwrapClackPrompt(result) {
   if (clack.isCancel(result)) {
     clack.cancel("Bug report cancelled.");
-    throw new CliError2("Bug report cancelled by user.");
+    throw new CliError("Bug report cancelled by user.");
   }
   return result;
 }
@@ -979,7 +986,7 @@ function normalizeBugEnvironmentChoice(environment) {
 function requireReportBugValue(value, option) {
   const normalized = value?.trim();
   if (!normalized) {
-    throw new CliError2(`Missing ${option}. Run interactively or pass ${option} <value>.`);
+    throw new CliError(`Missing ${option}. Run interactively or pass ${option} <value>.`);
   }
   return normalized;
 }
@@ -1060,7 +1067,7 @@ function takeRepeatedOption(args, option) {
     if (current === option) {
       const next = args[index + 1];
       if (!next || next.startsWith("-")) {
-        throw new CliError2(`Missing value for ${option}`);
+        throw new CliError(`Missing value for ${option}`, 1, { hint: `Provide a value after ${option}, e.g. \`${option} <value>\`.` });
       }
       values.push(next);
       index += 1;