@jphutchins/code-review 0.1.0-alpha.10 → 0.1.0-alpha.11

package/dist/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import { defineCommand, runMain } from 'citty';
 import { readFileSync, writeFileSync, readdirSync } from 'fs';
-import { resolve as resolve$1, join, dirname } from 'path';
+import { resolve as resolve$1, join, dirname, basename } from 'path';
 import { Eta } from 'eta';
 import parseDiff from 'parse-diff';
 import { Ajv2020 } from 'ajv/dist/2020.js';
@@ -463,8 +463,9 @@ var sumTranscriptUsage = (entries) => {
   };
 };
 var subagentFiles = (mainPath) => {
+  const dir = join(dirname(mainPath), basename(mainPath, ".jsonl"), "subagents");
   try {
-    return readdirSync(join(dirname(mainPath), "subagents")).filter((name) => name.endsWith(".jsonl")).map((name) => join(dirname(mainPath), "subagents", name));
+    return readdirSync(dir).filter((name) => name.endsWith(".jsonl")).map((name) => join(dir, name));
   } catch {
     return [];
   }
@@ -722,6 +723,7 @@ var stopHookSettings = (command) => ({
 });
 
 // src/budget.ts
+var DEADLINE_ENV = "CODE_REVIEW_DEADLINE_EPOCH";
 var DEFAULT_RESERVE = { frac: 0.15, flatUsd: 0.02, flatMs: 12e4 };
 var SOFT_MULTIPLE = 2;
 var costAxis = (i) => i.spentUsd !== null && i.budgetUsd !== null && i.budgetUsd > 0 ? { used: i.spentUsd, limit: i.budgetUsd, flat: i.reserve.flatUsd } : null;
@@ -810,6 +812,20 @@ var parseWallMs = (raw) => {
       return n * 36e5;
   }
 };
+var parseEpochSecMs = (raw) => {
+  if (raw === void 0) return null;
+  const t4 = raw.trim();
+  if (!/^\d+$/.test(t4)) return null;
+  const n = Number.parseInt(t4, 10);
+  return Number.isFinite(n) && n > 0 ? n * 1e3 : null;
+};
+var anchoredElapsedMs = (src) => {
+  if (src.deadlineMs !== null && src.wallMs !== null)
+    return Math.max(0, src.wallMs - (src.deadlineMs - src.nowMs));
+  if (src.firstTsMs !== null) return Math.max(0, src.nowMs - src.firstTsMs);
+  return null;
+};
+var deadlineEpochSec = (wallMs, nowMs) => Math.floor(nowMs / 1e3) + Math.ceil(wallMs / 1e3);
 var parseFraction = (raw, fallback) => {
   if (raw === void 0) return fallback;
   const n = Number.parseFloat(raw);
@@ -1362,7 +1378,7 @@ var post = async (input, ghApi = runGhApi) => {
 `
     );
   }
-  const botReviews = comments.length > 0 ? await fetchBotReviews(input.repo, prNumber, input.botLogin, ghApi) : [];
+  const botReviews = await fetchBotReviews(input.repo, prNumber, input.botLogin, ghApi);
   const alreadyReviewedThisSha = botReviews.some((r) => r.commitId === input.headSha);
   const initialDisposition = comments.length > 0 ? alreadyReviewedThisSha ? { kind: "suppressed-existing-review", sha: input.headSha } : void 0 : strays.length > 0 ? { kind: "none-in-diff" } : void 0;
   const commonRenderInput = {
@@ -1396,10 +1412,9 @@ var post = async (input, ghApi = runGhApi) => {
     renderBody(initialDisposition),
     ghApi
   );
-  if (comments.length === 0) return;
   if (alreadyReviewedThisSha) {
     process.stderr.write(
-      `A completed bot review already exists for ${input.headSha} \u2014 updated sticky only, no new inline review
+      `A completed bot review already exists for ${input.headSha} \u2014 updated sticky only, no new review
 `
     );
     return;
@@ -1418,11 +1433,11 @@ var post = async (input, ghApi = runGhApi) => {
     ghApi
   );
   process.stderr.write(
-    `Posted ${String(comments.length)} inline comments on PR #${String(prNumber)}
+    `Posted a review with ${String(comments.length)} inline comment(s) on PR #${String(prNumber)}
 `
   );
   await minimizeSupersededComments(input.repo, prNumber, input.headSha, input.botLogin, ghApi);
-  if (stickyRef !== null) {
+  if (comments.length > 0 && stickyRef !== null) {
     const confirmedDisposition = {
       kind: "posted",
       count: comments.length,
@@ -1776,16 +1791,34 @@ var withMeta = (base, meta) => ({
   ...meta.route ? { route: meta.route } : {},
   ...meta.effort ? { effort: meta.effort } : {}
 });
-var nativeTelemetry = (native, meta) => withMeta(
+var resolveTelemetry = (native, meta) => {
+  const fb = native.models.length === 0 ? meta.transcriptFallback?.() : void 0;
+  const useFallback = fb !== void 0 && fb.models.length > 0;
+  return withMeta(
+    useFallback ? {
+      models: [...fb.models],
+      turns: fb.turns,
+      duration_ms: fb.durationMs,
+      vendor_cost_usd: native.vendorCostUsd
+    } : {
+      models: native.models,
+      turns: native.turns,
+      duration_ms: native.durationMs,
+      vendor_cost_usd: native.vendorCostUsd
+    },
+    meta
+  );
+};
+var nativeTelemetry = (native, meta) => resolveTelemetry(
   {
     models: mapModelUsage(native.modelUsage),
     turns: native.num_turns,
-    duration_ms: native.duration_ms,
-    vendor_cost_usd: native.total_cost_usd ?? null
+    durationMs: native.duration_ms,
+    vendorCostUsd: native.total_cost_usd ?? null
   },
   meta
 );
-var absentTelemetry = (meta) => withMeta({ models: [], turns: 0, duration_ms: 0, vendor_cost_usd: null }, meta);
+var absentTelemetry = (meta) => resolveTelemetry({ models: [], turns: 0, durationMs: 0, vendorCostUsd: null }, meta);
 var buildEnvelope = (telemetry, native, agentFilePath) => {
   const outcome = findingsOutcome(native, agentFilePath);
   switch (outcome.kind) {
@@ -1908,6 +1941,16 @@ var unwrapAdapt = (either) => {
   }
   throw new Error("unreachable");
 };
+var transcriptFallbackFrom = (path) => {
+  const tree = readTranscriptTree(resolve$1(path));
+  if (tree.missing)
+    process.stderr.write(
+      `code-review adapt: transcript ${path} is unreadable \u2014 no telemetry fallback (issue #36)
+`
+    );
+  const usage = sumTranscriptUsage(tree.entries);
+  return { models: usage.models, turns: usage.turns, durationMs: usage.durationMs };
+};
 var bundledPath = (...segments) => resolve$1(import.meta.dirname, "..", ...segments);
 var packageVersion = JSON.parse(readFileSync(bundledPath("package.json"), "utf-8")).version;
 var resolveTemplatePath = (templateArg) => templateArg ? resolve$1(templateArg) : bundledPath("templates", "comment.eta");
@@ -2148,11 +2191,17 @@ var budgetHookCmd = defineCommand({
       const usage = tree ? sumTranscriptUsage(tree.entries) : void 0;
       const prices = args.prices ? tryReadPrices(args.prices) : null;
       const spentUsd = prices !== null && usage ? computeCost(usage.models, prices).totalCostUSD : null;
+      const wallMs = args.wall ? parseWallMs(args.wall) : null;
       const output = evaluateBudgetHook(input, {
         spentUsd,
         budgetUsd: parseBudgetUsd(args["budget-usd"]),
-        elapsedMs: usage?.firstTsMs != null ? Math.max(0, Date.now() - usage.firstTsMs) : null,
-        wallMs: args.wall ? parseWallMs(args.wall) : null,
+        elapsedMs: anchoredElapsedMs({
+          deadlineMs: parseEpochSecMs(process.env[DEADLINE_ENV]),
+          wallMs,
+          firstTsMs: usage?.firstTsMs ?? null,
+          nowMs: Date.now()
+        }),
+        wallMs,
         reserve: {
           frac: parseFraction(args["reserve-frac"], DEFAULT_RESERVE.frac),
           flatUsd: parseBudgetUsd(args["reserve-usd"]) ?? DEFAULT_RESERVE.flatUsd,
@@ -2252,6 +2301,28 @@ var printSettingsCmd = defineCommand({
 `);
   }
 });
+var deadlineCmd = defineCommand({
+  meta: {
+    name: "deadline",
+    description: "Print the run's absolute deadline as Unix epoch seconds (now + --wall). The review job exports this as CODE_REVIEW_DEADLINE_EPOCH right before `claude -p` so every budget hook \u2014 the main agent's and each fan-out subagent's \u2014 measures the SAME true remaining wall instead of its own transcript's start, which reads \u22480 in a fresh subagent and leaves the fan-out unsteered (issue #45)."
+  },
+  args: {
+    wall: {
+      type: "string",
+      description: "Wall-clock budget for the run (e.g. 24m, 1200s, 2h) \u2014 the deadline is now + this",
+      required: true
+    }
+  },
+  run: async ({ args }) => {
+    const wallMs = parseWallMs(args.wall);
+    if (wallMs === null) {
+      fail(`--wall must be a duration like 24m, 1200s, or 2h (got '${args.wall}')`);
+    } else {
+      process.stdout.write(`${String(deadlineEpochSec(wallMs, Date.now()))}
+`);
+    }
+  }
+});
 var derivedSchemaVersion = (kind, raw) => kind === "findings" ? declaredVersion(raw) : void 0;
 var validateCmd = defineCommand({
   meta: {
@@ -2319,13 +2390,20 @@ var adaptCmd = defineCommand({
     effort: {
       type: "string",
       description: 'Effort label to stamp into the envelope (e.g. "max" or "low")'
+    },
+    transcript: {
+      type: "string",
+      description: "Path to the session transcript (the main .jsonl); when the native envelope carries no per-model usage \u2014 a wall-clock kill leaves it empty (issue #39) \u2014 telemetry is recovered from the transcript tree (main + subagents) so cost is real, not $0.00 (issue #36)"
     }
   },
   run: async ({ args }) => {
     const envelope = unwrapAdapt(
       adapt(requireAdapterName(args.adapter), readJSONOrAbsent(args.native), args["agent-file"], {
         route: args.route,
-        effort: args.effort
+        effort: args.effort,
+        ...args.transcript ? {
+          transcriptFallback: () => transcriptFallbackFrom(args.transcript)
+        } : {}
       })
     );
     process.stdout.write(`${JSON.stringify(envelope, null, 2)}
@@ -2733,7 +2811,7 @@ var main = defineCommand({
   meta: {
     name: "code-review",
     version: packageVersion,
-    description: "Deterministic commenter for agentic PR review \u2014 gather, render, inline, post, adapt, extract, validate-patches, cost, check-cost, validate, stop-gate, budget-hook, and print-settings"
+    description: "Deterministic commenter for agentic PR review \u2014 gather, render, inline, post, adapt, extract, validate-patches, cost, check-cost, validate, stop-gate, budget-hook, print-settings, and deadline"
   },
   subCommands: {
     gather: gatherCmd,
@@ -2749,7 +2827,8 @@ var main = defineCommand({
     "print-schema": printSchemaCmd,
     "stop-gate": stopGateCmd,
     "budget-hook": budgetHookCmd,
-    "print-settings": printSettingsCmd
+    "print-settings": printSettingsCmd,
+    deadline: deadlineCmd
   }
 });
 if (!process.env["VITEST"]) {