@inteeka/task-cli 0.1.10 → 0.1.12

package/dist/cli.js

@@ -2170,7 +2170,11 @@ import { homedir as homedir5 } from "os";
 import { join as join7 } from "path";
 var FIX_PROMPT_JSON_SCHEMA = {
   type: "object",
-  required: ["summary", "suspected_files", "proposed_changes", "confidence"],
+  // Phase 3 — confidence_reason is REQUIRED unconditionally so the
+  // server-side refine() (which mandates ≥20 chars on low/medium) never
+  // rejects a fresh submission. The model emits it for high too; that's
+  // cheap (≤1500 chars) and doubles as documentation for reviewers.
+  required: ["summary", "suspected_files", "proposed_changes", "confidence", "confidence_reason"],
   additionalProperties: false,
   properties: {
     summary: { type: "string", minLength: 1, maxLength: 2e3 },
@@ -2209,11 +2213,11 @@ var FIX_PROMPT_JSON_SCHEMA = {
     },
     risk_notes: { type: "string", maxLength: 2e3 },
     confidence: { type: "string", enum: ["low", "medium", "high"] },
-    // Phase 3 — explicit reasoning for the confidence rating. Required at
-    // ≥20 chars for low/medium ratings; the dashboard's Zod refine() rejects
-    // anything shorter on `/submit`. Optional on `high` so the model can
-    // omit it when the evidence is overwhelming.
-    confidence_reason: { type: "string", maxLength: 1500 }
+    // Phase 3 — explicit reasoning for the confidence rating. Always
+    // required (see `required` array above). minLength=20 matches the
+    // dashboard's Zod refine() so low/medium submissions never fail
+    // server-side validation; high also emits it (cheap, useful).
+    confidence_reason: { type: "string", minLength: 20, maxLength: 1500 }
   }
 };
 var LlmGenerationError = class extends Error {
@@ -2232,7 +2236,9 @@ async function generateFixPromptJson(args) {
     "",
     args.ticketBlock,
     "",
-    "Return JSON only matching the supplied schema. Do not include explanatory prose, markdown fences, or commentary."
+    "Return JSON only matching the supplied schema. Do not include explanatory prose, markdown fences, or commentary.",
+    "",
+    "IMPORTANT: confidence_reason is REQUIRED for every rating (\u226520 chars). For low/medium, name which investigation axes lacked signal. For high, briefly state which axes corroborated the proposal."
   ].join("\n");
   const cliArgs = [
     "--print",
@@ -2705,6 +2711,143 @@ function clampInt(raw, min, max, fallback) {
   return Math.min(v, max);
 }
 
+// src/commands/pr-test.ts
+import { execFileSync as execFileSync6 } from "child_process";
+function registerPrTest(program2) {
+  program2.command("pr-test").description(
+    "Dry-run the full PR pipeline \u2014 cuts a throwaway branch, opens a real PR via the dashboard, then cleans up. Use this to verify your git integration before running task work on real tickets."
+  ).option(
+    "--keep",
+    "Leave the PR open and the branches in place after the test (default: clean up)"
+  ).option("--silent", "Suppress per-step progress output").action(async (opts) => {
+    await runPrTest(opts);
+  });
+}
+async function runPrTest(opts) {
+  const cwd = findRepoRoot();
+  const project = await readProjectConfig(cwd);
+  if (!project) {
+    throw new CliError(
+      CLI_EXIT_CODES.MISCONFIGURATION,
+      "No project link in this repo",
+      "Run 'task link' first."
+    );
+  }
+  const baseBranch = project.cli_base_branch ?? "development";
+  const silent = !!opts.silent;
+  if (!silent) process.stdout.write(`${c.dim(`Step 1/6: assertBaseBranch (${baseBranch})\u2026`)}
+`);
+  assertBaseBranch(cwd, baseBranch);
+  const timestamp = Date.now();
+  const branchName = `task/pr-test-${timestamp}`;
+  const prTitle = `[task pr-test] CLI dry-run probe ${new Date(timestamp).toISOString()}`;
+  if (!silent) process.stdout.write(`${c.dim(`Step 2/6: createTicketBranch (${branchName})\u2026`)}
+`);
+  createTicketBranch(cwd, branchName, baseBranch);
+  let pushSucceeded = false;
+  const recover = () => {
+    try {
+      checkoutBranch(cwd, baseBranch);
+    } catch {
+    }
+    deleteLocalBranch(cwd, branchName);
+  };
+  try {
+    if (!silent) process.stdout.write(`${c.dim("Step 3/6: empty commit\u2026")}
+`);
+    execFileSync6(
+      "git",
+      ["commit", "--allow-empty", "-m", `task pr-test: connectivity probe ${timestamp}`],
+      { cwd, stdio: ["ignore", "pipe", "pipe"] }
+    );
+    if (!silent) process.stdout.write(`${c.dim(`Step 4/6: pushBranch (${branchName})\u2026`)}
+`);
+    pushBranch(cwd, branchName);
+    pushSucceeded = true;
+    if (!silent) process.stdout.write(`${c.dim("Step 5/6: open PR via dashboard\u2026")}
+`);
+    const pr = await apiCallOrThrow("POST", `/api/v1/cli/me/git/pr-test`, {
+      body: {
+        project_id: project.project_id,
+        source_branch: branchName,
+        base_branch: baseBranch,
+        title: prTitle
+      }
+    });
+    process.stdout.write(
+      `
+${c.ok("\u2713 PR opened")} ${c.cyan(pr.pr_url)}
+  repo:   ${pr.repo}
+  branch: ${pr.source_branch} \u2192 ${pr.base_branch}
+  number: #${pr.pr_number}
+
+`
+    );
+    if (opts.keep) {
+      process.stdout.write(
+        `${c.warn("--keep")} ${c.dim("flag set \u2014 PR and branches preserved. Manual cleanup:")}
+  ${c.cyan(`task pr-test-cleanup ${pr.pr_number} ${branchName}`)}
+  (or close on GitHub + ${c.cyan(`git push origin --delete ${branchName}`)} + ${c.cyan(`git branch -D ${branchName}`)})
+`
+      );
+      checkoutBranch(cwd, baseBranch);
+      return;
+    }
+    if (!silent) process.stdout.write(`${c.dim("Step 6/6: cleanup\u2026")}
+`);
+    const cleanup = await apiCall("POST", `/api/v1/cli/me/git/pr-test/cleanup`, {
+      body: {
+        project_id: project.project_id,
+        pr_number: pr.pr_number,
+        source_branch: branchName
+      }
+    });
+    checkoutBranch(cwd, baseBranch);
+    deleteLocalBranch(cwd, branchName);
+    if (cleanup.ok && cleanup.data) {
+      const { pr_closed, branch_deleted } = cleanup.data;
+      process.stdout.write(
+        `${c.ok("\u2713 Cleanup complete")} \u2014 PR ${pr_closed ? "closed" : c.warn("NOT closed")}, remote branch ${branch_deleted ? "deleted" : c.warn("NOT deleted")}, local branch deleted.
+`
+      );
+      if (!pr_closed || !branch_deleted) {
+        process.stdout.write(
+          c.dim(`  Manually close at: ${pr.pr_url}
+`) + c.dim(`  Manually delete remote: git push origin --delete ${branchName}
+`)
+        );
+      }
+    } else {
+      process.stdout.write(
+        `${c.warn("Cleanup endpoint failed \u2014 branches & PR may need manual cleanup:")}
+  ${c.cyan(pr.pr_url)}
+  ${c.cyan(`git push origin --delete ${branchName}`)}
+`
+      );
+    }
+    process.stdout.write(
+      `
+${c.ok("\u2713 pr-test passed")} \u2014 the full CLI \u2192 dashboard \u2192 GitHub PR pipeline works.
+`
+    );
+  } catch (err) {
+    process.stdout.write(`
+${c.err("\u2717 pr-test failed")}: ${err.message}
+`);
+    if (pushSucceeded) {
+      process.stdout.write(
+        c.dim(
+          `  The branch was pushed but the PR step failed. Manually clean up:
+    git push origin --delete ${branchName}
+`
+        )
+      );
+    }
+    recover();
+    throw err;
+  }
+}
+
 // src/commands/scheduled-task.ts
 import { randomUUID as randomUUID3 } from "crypto";
 
@@ -2715,7 +2858,7 @@ import { platform as platform2 } from "os";
 import { mkdir as mkdir7, readFile as readFile5, writeFile as writeFile8, unlink as unlink3, readdir } from "fs/promises";
 import { homedir as homedir6 } from "os";
 import { join as join8 } from "path";
-import { execFileSync as execFileSync6, spawn as spawn4 } from "child_process";
+import { execFileSync as execFileSync7, spawn as spawn4 } from "child_process";
 
 // src/scheduler/cron-translate.ts
 function translateToLaunchd(cron) {
@@ -2883,17 +3026,17 @@ var launchdAdapter = {
     const path = plistPath(entry.id);
     await writeFile8(path, buildPlist(entry));
     try {
-      execFileSync6("launchctl", ["bootout", bootstrapDomain(), path], { stdio: "ignore" });
+      execFileSync7("launchctl", ["bootout", bootstrapDomain(), path], { stdio: "ignore" });
     } catch {
     }
     if (entry.enabled) {
-      execFileSync6("launchctl", ["bootstrap", bootstrapDomain(), path]);
+      execFileSync7("launchctl", ["bootstrap", bootstrapDomain(), path]);
     }
   },
   async remove(id) {
     const path = plistPath(id);
     try {
-      execFileSync6("launchctl", ["bootout", bootstrapDomain(), path], { stdio: "ignore" });
+      execFileSync7("launchctl", ["bootout", bootstrapDomain(), path], { stdio: "ignore" });
     } catch {
     }
     try {
@@ -2958,10 +3101,10 @@ var launchdAdapter = {
       xml = xml.replace(/\s*<key>Disabled<\/key>\s*<true\/>/, "");
       await writeFile8(path, xml);
       try {
-        execFileSync6("launchctl", ["bootout", bootstrapDomain(), path], { stdio: "ignore" });
+        execFileSync7("launchctl", ["bootout", bootstrapDomain(), path], { stdio: "ignore" });
       } catch {
       }
-      execFileSync6("launchctl", ["bootstrap", bootstrapDomain(), path]);
+      execFileSync7("launchctl", ["bootstrap", bootstrapDomain(), path]);
     } else {
       if (!/<key>Disabled<\/key>/.test(xml)) {
         xml = xml.replace(
@@ -2971,7 +3114,7 @@ var launchdAdapter = {
         await writeFile8(path, xml);
       }
       try {
-        execFileSync6("launchctl", ["bootout", bootstrapDomain(), path], { stdio: "ignore" });
+        execFileSync7("launchctl", ["bootout", bootstrapDomain(), path], { stdio: "ignore" });
       } catch {
       }
     }
@@ -2979,7 +3122,7 @@ var launchdAdapter = {
 };
 
 // src/scheduler/cron.ts
-import { execFileSync as execFileSync7, spawn as spawn5 } from "child_process";
+import { execFileSync as execFileSync8, spawn as spawn5 } from "child_process";
 
 // src/scheduler/safe-command.ts
 var FORBIDDEN = /[;&|`$()<>\\]/;
@@ -3034,7 +3177,7 @@ var MARK_OPEN = (id) => `# task-cli:${id}:start`;
 var MARK_CLOSE = (id) => `# task-cli:${id}:end`;
 function readCrontab() {
   try {
-    return execFileSync7("crontab", ["-l"], { encoding: "utf8" });
+    return execFileSync8("crontab", ["-l"], { encoding: "utf8" });
   } catch {
     return "";
   }
@@ -3149,7 +3292,7 @@ var cronAdapter = {
 };
 
 // src/scheduler/windows.ts
-import { execFileSync as execFileSync8, spawn as spawn6 } from "child_process";
+import { execFileSync as execFileSync9, spawn as spawn6 } from "child_process";
 var TASK_PREFIX = "TaskCLI_";
 function taskName(id) {
   return `${TASK_PREFIX}${id.replace(/[^A-Za-z0-9_-]/g, "_")}`;
@@ -3214,22 +3357,22 @@ function pad(v) {
 var windowsAdapter = {
   async upsert(entry) {
     const args = buildSchtasksArgs(entry, entry.command);
-    execFileSync8("schtasks.exe", args, { stdio: "ignore" });
+    execFileSync9("schtasks.exe", args, { stdio: "ignore" });
     if (!entry.enabled) {
-      execFileSync8("schtasks.exe", ["/Change", "/TN", taskName(entry.id), "/DISABLE"], {
+      execFileSync9("schtasks.exe", ["/Change", "/TN", taskName(entry.id), "/DISABLE"], {
         stdio: "ignore"
       });
     }
   },
   async remove(id) {
     try {
-      execFileSync8("schtasks.exe", ["/Delete", "/TN", taskName(id), "/F"], { stdio: "ignore" });
+      execFileSync9("schtasks.exe", ["/Delete", "/TN", taskName(id), "/F"], { stdio: "ignore" });
     } catch {
     }
   },
   async list() {
     try {
-      const csv = execFileSync8("schtasks.exe", ["/Query", "/FO", "CSV", "/V"], {
+      const csv = execFileSync9("schtasks.exe", ["/Query", "/FO", "CSV", "/V"], {
         encoding: "utf8"
       });
       const lines = csv.split(/\r?\n/);
@@ -3279,7 +3422,7 @@ var windowsAdapter = {
   },
   async setEnabled(id, enabled) {
     try {
-      execFileSync8(
+      execFileSync9(
         "schtasks.exe",
         ["/Change", "/TN", taskName(id), enabled ? "/ENABLE" : "/DISABLE"],
         { stdio: "ignore" }
@@ -3687,7 +3830,7 @@ function registerConfig(program2) {
 }
 
 // src/commands/doctor.ts
-import { execFileSync as execFileSync9 } from "child_process";
+import { execFileSync as execFileSync10 } from "child_process";
 import { request as request5 } from "undici";
 function registerDoctor(program2) {
   program2.command("doctor").description("Diagnose your CLI setup").action(async () => {
@@ -3735,7 +3878,7 @@ function registerDoctor(program2) {
       });
     }
     try {
-      const dirty = execFileSync9("git", ["status", "--porcelain"], {
+      const dirty = execFileSync10("git", ["status", "--porcelain"], {
         cwd: root,
         encoding: "utf8"
       }).trim();
@@ -3759,7 +3902,7 @@ function registerDoctor(program2) {
 }
 function checkBinary(name, command) {
   try {
-    const out = execFileSync9(command, ["--version"], { encoding: "utf8" }).trim();
+    const out = execFileSync10(command, ["--version"], { encoding: "utf8" }).trim();
     return { name, ok: true, detail: out.split("\n")[0] ?? out };
   } catch {
     return { name, ok: false, detail: `'${command}' not found on PATH` };
@@ -3767,7 +3910,7 @@ function checkBinary(name, command) {
 }
 
 // src/commands/version.ts
-var CLI_VERSION = true ? "0.1.10" : "0.0.0-dev";
+var CLI_VERSION = true ? "0.1.12" : "0.0.0-dev";
 function registerVersion(program2) {
   program2.command("version").description("Print the CLI version").action(() => {
     process.stdout.write(CLI_VERSION + "\n");
@@ -3791,6 +3934,7 @@ registerTickets(program);
 registerTicket(program);
 registerWork(program);
 registerScan(program);
+registerPrTest(program);
 registerScheduledTask(program);
 registerRuns(program);
 registerConfig(program);