@staff0rd/assist 0.634.3 → 0.636.0

package/dist/index.js

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.634.3",
+  version: "0.636.0",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -677,6 +677,9 @@ var assistConfigShape = {
     draft: z3.boolean().default(false),
     readingWordsPerMinute: z3.number().int().positive().optional()
   }).optional(),
+  review: z3.strictObject({
+    codexModel: z3.string().optional()
+  }).optional(),
   readTime: z3.strictObject({
     wordsPerMinute: z3.number().int().positive().default(200)
   }).optional(),
@@ -3180,6 +3183,15 @@ var refactorConfigHelp = [
   }
 ];
 
+// src/commands/review/reviewConfigHelp.ts
+var reviewConfigHelp = [
+  {
+    key: "review.codexModel",
+    setter: "assist config set review.codexModel gpt-5-codex",
+    note: "optional; runs the codex reviewer on this model via the LiteLLM proxy (needs litellm.baseUrl and litellm.apiKey)"
+  }
+];
+
 // src/commands/roam/roamConfigHelp.ts
 var roamConfigHelp = [
   {
@@ -3506,6 +3518,7 @@ var configHelpEntries = [
   ...ravendbConfigHelp,
   ...readTimeConfigHelp,
   ...refactorConfigHelp,
+  ...reviewConfigHelp,
   ...roamConfigHelp,
   ...runConfigHelp,
   ...seqConfigHelp,
@@ -24120,15 +24133,24 @@ import chalk172 from "chalk";
 
 // src/commands/litellm/resolveLitellmConfig.ts
 import chalk171 from "chalk";
-function resolveLitellmConfig() {
+
+// src/commands/litellm/readLitellmConfig.ts
+function readLitellmConfig() {
   const litellm = loadConfig().litellm;
   const baseUrl = litellm?.baseUrl?.trim();
   const apiKey = litellm?.apiKey?.trim();
-  if (!baseUrl || !apiKey) {
-    const missing = [
-      ...baseUrl ? [] : ["litellm.baseUrl"],
-      ...apiKey ? [] : ["litellm.apiKey"]
-    ];
+  const missing = [
+    ...baseUrl ? [] : ["litellm.baseUrl"],
+    ...apiKey ? [] : ["litellm.apiKey"]
+  ];
+  if (!baseUrl || !apiKey) return { config: null, missing };
+  return { config: { baseUrl: baseUrl.replace(/\/+$/, ""), apiKey }, missing };
+}
+
+// src/commands/litellm/resolveLitellmConfig.ts
+function resolveLitellmConfig() {
+  const { config, missing } = readLitellmConfig();
+  if (!config) {
     console.error(chalk171.red("LiteLLM is not configured"));
     for (const key of missing) {
       const entry = litellmConfigHelp.find((help) => help.key === key);
@@ -24138,7 +24160,7 @@ function resolveLitellmConfig() {
     }
     process.exit(1);
   }
-  return { baseUrl: baseUrl.replace(/\/+$/, ""), apiKey };
+  return config;
 }
 
 // src/commands/litellm/listModels.ts
@@ -30867,7 +30889,8 @@ function writeStdinSafely(child, payload) {
 function startChild(spec) {
   const child = spawn7(spec.command, spec.args, {
     stdio: ["pipe", "pipe", "pipe"],
-    shell: process.platform === "win32"
+    shell: process.platform === "win32",
+    ...spec.env ? { env: { ...process.env, ...spec.env } } : {}
   });
   const flushPending = attachLineParser(child, spec.onLine);
   const stderr = attachStderrCollector(child);
@@ -30940,6 +30963,30 @@ function resolveClaude(args) {
 // src/commands/review/runCodexReviewer.ts
 import { existsSync as existsSync62, unlinkSync as unlinkSync18 } from "fs";
 
+// src/commands/review/buildCodexModelArgs.ts
+var API_KEY_ENV = "ASSIST_LITELLM_API_KEY";
+function buildCodexModelArgs() {
+  const model = loadConfig().review?.codexModel?.trim();
+  if (!model) return { args: [], env: {} };
+  const { config } = readLitellmConfig();
+  if (!config) return { args: [], env: {} };
+  return {
+    args: [
+      "-c",
+      `model_providers.litellm.base_url=${config.baseUrl}/v1`,
+      "-c",
+      `model_providers.litellm.env_key=${API_KEY_ENV}`,
+      "-c",
+      "model_providers.litellm.wire_api=chat",
+      "-c",
+      "model_provider=litellm",
+      "-m",
+      model
+    ],
+    env: { [API_KEY_ENV]: config.apiKey }
+  };
+}
+
 // src/commands/review/parseCodexEvent.ts
 function isItemStarted(value) {
   return typeof value === "object" && value !== null && value.type === "item.started";
@@ -30965,9 +31012,10 @@ function parseCodexEvent(line) {
 }
 
 // src/commands/review/runCodexReviewer.ts
-function codexArgs(outputPath) {
+function codexArgs(outputPath, modelArgs) {
   return [
     "exec",
+    ...modelArgs,
     "--sandbox",
     "read-only",
     "--json",
@@ -30978,12 +31026,14 @@ function codexArgs(outputPath) {
 async function runCodexReviewer(spec) {
   const { spinner } = spec;
   const command = "codex";
+  const override = buildCodexModelArgs();
   const result = await runStreamingChild({
     name: spec.name,
     command,
-    args: codexArgs(spec.outputPath),
+    args: codexArgs(spec.outputPath, override.args),
     stdin: spec.stdin,
     quiet: Boolean(spinner),
+    ...override.args.length > 0 ? { env: override.env } : {},
     onLine: (line) => {
       const event = parseCodexEvent(line);
       if (event.kind !== "tool_use") return;
@@ -31290,7 +31340,7 @@ async function review(options2 = {}) {
 
 // src/commands/registerReview.ts
 function registerReview(program2) {
-  program2.command("review").description(
+  const reviewCommand = program2.command("review").description(
     "Run Claude and Codex in parallel to review the current branch's PR, or check out a PR by number first when given; --checkout-only just checks the PR out and leaves an idle Claude session in the checkout tree"
   ).argument(
     "[number]",
@@ -31328,6 +31378,11 @@ function registerReview(program2) {
   ).action(
     (number, options2) => review({ ...options2, number })
   );
+  configHelp(
+    reviewCommand,
+    reviewConfigHelp,
+    "With review.codexModel unset the codex reviewer runs on the user's own codex auth; when it is set but the LiteLLM proxy is not configured, the reviewer falls back to that plain codex run."
+  );
 }
 
 // src/commands/rules/addRule.ts

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@staff0rd/assist",
-	"version": "0.634.3",
+	"version": "0.636.0",
 	"type": "module",
 	"main": "dist/index.js",
 	"bin": {