@companyhelm/cli 0.0.7 → 0.0.8

package/dist/commands/root.js

@@ -1467,10 +1467,11 @@ async function hasConfiguredSdks(cfg) {
         client.close();
     }
 }
-async function clearSdkModels(cfg, sdkName) {
+async function countSdkModels(cfg, sdkName) {
     const { db, client } = await (0, db_js_1.initDb)(cfg.state_db_path);
     try {
-        await db.delete(schema_js_1.llmModels).where((0, drizzle_orm_1.eq)(schema_js_1.llmModels.sdkName, sdkName));
+        const models = await db.select({ name: schema_js_1.llmModels.name }).from(schema_js_1.llmModels).where((0, drizzle_orm_1.eq)(schema_js_1.llmModels.sdkName, sdkName)).all();
+        return models.length;
     }
     finally {
         client.close();
@@ -1483,9 +1484,12 @@ async function refreshCodexModelsForRegistration(cfg, logger) {
         logger.info(`Refreshed Codex models from container app-server (${modelCount} models).`);
     }
     catch (error) {
-        logger.warn(`Failed to refresh Codex models from container app-server: ${toErrorMessage(error)}. ` +
-            "Registering runner with an empty Codex model list.");
-        await clearSdkModels(cfg, "codex");
+        const cachedModelCount = await countSdkModels(cfg, "codex");
+        const failureMessage = (0, refresh_models_js_1.formatSdkModelRefreshFailure)("codex", error);
+        if (cachedModelCount === 0) {
+            throw new Error(`${failureMessage} Runner startup aborted because no cached Codex models are available; refusing to register zero models.`);
+        }
+        logger.warn(`${failureMessage} Using ${cachedModelCount} cached Codex model(s) from local state instead of registering zero models.`);
     }
 }
 async function resolveThreadAuthMode(cfg) {

package/dist/commands/sdk/refresh-models.js

@@ -4,7 +4,14 @@ exports.runSdkRefreshModelsCommand = runSdkRefreshModelsCommand;
 exports.registerSdkRefreshModelsCommand = registerSdkRefreshModelsCommand;
 const refresh_models_js_1 = require("../../service/sdk/refresh_models.js");
 async function runSdkRefreshModelsCommand(options) {
-    const results = await (0, refresh_models_js_1.refreshSdkModels)({ sdk: options.sdk });
+    let results;
+    try {
+        results = await (0, refresh_models_js_1.refreshSdkModels)({ sdk: options.sdk });
+    }
+    catch (error) {
+        const sdkName = options.sdk ?? "configured";
+        throw new Error((0, refresh_models_js_1.formatSdkModelRefreshFailure)(sdkName, error));
+    }
     for (const result of results) {
         console.log(`Refreshed ${result.modelCount} models for SDK '${result.sdk}'.`);
     }

package/dist/service/docker/app_server_container.js

@@ -56,12 +56,46 @@ class AppServerContainerService {
         this.child = null;
         this.containerName = null;
         this.running = false;
+        this.recentStderrLines = [];
+        this.lastExitCode = null;
+        this.lastExitSignal = null;
         this.docker = options?.docker ?? new dockerode_1.default();
         this.imageStatusReporter = options?.imageStatusReporter;
     }
     reportImageStatus(message) {
         this.imageStatusReporter?.(message);
     }
+    recordStderr(chunk) {
+        const lines = chunk
+            .split(/\r?\n/)
+            .map((line) => line.trim())
+            .filter((line) => line.length > 0);
+        if (lines.length === 0) {
+            return;
+        }
+        this.recentStderrLines.push(...lines);
+        if (this.recentStderrLines.length > 8) {
+            this.recentStderrLines.splice(0, this.recentStderrLines.length - 8);
+        }
+    }
+    buildContainerStoppedErrorMessage() {
+        const details = [];
+        if (this.containerName) {
+            details.push(`container ${this.containerName}`);
+        }
+        if (this.lastExitCode !== null) {
+            details.push(`exit code ${this.lastExitCode}`);
+        }
+        else if (this.lastExitSignal) {
+            details.push(`signal ${this.lastExitSignal}`);
+        }
+        if (this.recentStderrLines.length > 0) {
+            details.push(`stderr: ${this.recentStderrLines.join(" | ")}`);
+        }
+        return details.length > 0
+            ? `App server container is not running (${details.join(", ")})`
+            : "App server container is not running";
+    }
     static isImageNotFound(error) {
         if (typeof error !== "object" || error === null) {
             return false;
@@ -159,6 +193,9 @@ class AppServerContainerService {
         if (this.running) {
             throw new Error("App server container is already running");
         }
+        this.recentStderrLines = [];
+        this.lastExitCode = null;
+        this.lastExitSignal = null;
         const cfg = config_js_1.config.parse({});
         await this.ensureImageAvailable(cfg.runtime_image);
         const { db, client } = await (0, db_js_1.initDb)(cfg.state_db_path);
@@ -218,15 +255,27 @@ class AppServerContainerService {
             this.messageQueue.push({ type: "stdout", payload: chunk });
         });
         child.stderr.on("data", (chunk) => {
-            this.messageQueue.push({ type: "stderr", payload: chunk.toString("utf8") });
+            const payload = chunk.toString("utf8");
+            this.recordStderr(payload);
+            this.messageQueue.push({ type: "stderr", payload });
         });
         child.on("error", (err) => {
+            this.recordStderr(err.message);
             this.messageQueue.push({ type: "error", reason: `docker process error: ${err.message}` });
             this.running = false;
             this.messageQueue.close();
         });
-        child.on("exit", () => {
+        child.on("exit", (code, signal) => {
+            const wasRunning = this.running;
+            this.lastExitCode = code;
+            this.lastExitSignal = signal;
             this.running = false;
+            if (wasRunning) {
+                this.messageQueue.push({
+                    type: "error",
+                    reason: this.buildContainerStoppedErrorMessage(),
+                });
+            }
             this.messageQueue.close();
         });
         this.child = child;
@@ -253,7 +302,7 @@ class AppServerContainerService {
     }
     async sendRaw(payload) {
         if (!this.running || !this.child || !this.child.stdin) {
-            throw new Error("App server container is not running");
+            throw new Error(this.buildContainerStoppedErrorMessage());
         }
         this.child.stdin.write(payload);
     }

package/dist/service/sdk/refresh_models.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.formatSdkModelRefreshFailure = formatSdkModelRefreshFailure;
 exports.refreshSdkModels = refreshSdkModels;
 const drizzle_orm_1 = require("drizzle-orm");
 const config_js_1 = require("../../config.js");
@@ -7,6 +8,13 @@ const db_js_1 = require("../../state/db.js");
 const schema_js_1 = require("../../state/schema.js");
 const app_server_js_1 = require("../app_server.js");
 const app_server_container_js_1 = require("../docker/app_server_container.js");
+function toErrorMessage(error) {
+    return error instanceof Error ? error.message : String(error);
+}
+function formatSdkModelRefreshFailure(sdk, error) {
+    return (`Failed to refresh ${sdk} models from the local Codex app-server: ${toErrorMessage(error)}. ` +
+        "Verify the runner image can start Codex app-server with valid auth, then retry.");
+}
 async function fetchCodexModelsFromAppServer(clientName, logger, imageStatusReporter) {
     const transport = new app_server_container_js_1.AppServerContainerService({ imageStatusReporter });
     const appServer = new app_server_js_1.AppServerService(transport, clientName, logger);

package/dist/templates/app_server_bootstrap.sh.j2

@@ -6,6 +6,12 @@ AGENT_UID={{ agent_uid }}
 AGENT_GID={{ agent_gid }}
 CODEX_AUTH_PATH={{ codex_auth_path }}
 APP_SERVER_COMMAND={{ app_server_command }}
+EXISTING_UID_USER=""
+
+if getent passwd "$AGENT_UID" >/dev/null 2>&1; then
+  EXISTING_UID_USER="$(getent passwd "$AGENT_UID" | cut -d: -f1)"
+  AGENT_USER="$EXISTING_UID_USER"
+fi
 
 AGENT_GROUP="$AGENT_USER"
 if getent group "$AGENT_GID" >/dev/null 2>&1; then
@@ -19,7 +25,7 @@ else
 fi
 
 if id -u "$AGENT_USER" >/dev/null 2>&1; then
-  usermod -u "$AGENT_UID" -g "$AGENT_GROUP" -d "$AGENT_HOME" "$AGENT_USER" || true
+  usermod -u "$AGENT_UID" -g "$AGENT_GROUP" -d "$AGENT_HOME" -s /bin/bash "$AGENT_USER" || true
 else
   useradd -m -d "$AGENT_HOME" -u "$AGENT_UID" -g "$AGENT_GROUP" -s /bin/bash "$AGENT_USER"
 fi

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@companyhelm/cli",
-  "version": "0.0.7",
+  "version": "0.0.8",
   "description": "Run coding agents in fully isolated Docker sandboxes, locally.",
   "license": "Apache-2.0",
   "repository": {