@tiens.nguyen/gu-cli 1.0.765 → 1.0.767

package/client-expert-api.mjs

@@ -23,6 +23,7 @@ import { spawnSurviving, pidOnPort } from "./platform-tools.mjs";
 import { DEFAULT_API_BASE as HOSTED_API_BASE } from "./device-login.mjs";
 import { spawn } from "node:child_process";
 import { writeFile, mkdir, readFile, access } from "node:fs/promises";
+import { readFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { join } from "node:path";
 
@@ -32,6 +33,29 @@ export const LOCAL_API_BASE = `http://127.0.0.1:${LOCAL_API_PORT}`;
 export const LOCAL_MONGO_URI = process.env.GONEXT_LOCAL_MONGO_URI || "mongodb://127.0.0.1:27017";
 export const LOCAL_DB_NAME = process.env.GONEXT_LOCAL_DB_NAME || "gu";
 
+/**
+ * THE URI THIS MACHINE'S MONGODB ACTUALLY ACCEPTS — credentials included (task #15).
+ *
+ * Setup asks for a username and password when 27017 already belongs to a MongoDB with its own
+ * admin, and saves the working URI as localMongoUri in client-expert.json. Every later caller
+ * used LOCAL_MONGO_URI instead — the empty pair — so on such a machine the model seed failed
+ * with "Command find requires authentication" (reported on a fresh Ubuntu box: coder "—", "agent
+ * model is not set"), and every API restart started a server that could not read its own
+ * database. Read here, once, as every default, so no caller can forget it.
+ *
+ * GONEXT_LOCAL_MONGO_URI still wins: setup treats a forced URI as deliberate and never saves one.
+ */
+export function localMongoUri({ env = process.env, home = homedir() } = {}) {
+  if (env.GONEXT_LOCAL_MONGO_URI) return env.GONEXT_LOCAL_MONGO_URI;
+  try {
+    const st = JSON.parse(readFileSync(join(home, ".gonext", "client-expert.json"), "utf8"));
+    if (typeof st?.localMongoUri === "string" && st.localMongoUri) return st.localMongoUri;
+  } catch {
+    /* no setup yet, or unreadable — the default is the only thing there is to try */
+  }
+  return LOCAL_MONGO_URI;
+}
+
 const GONEXT_DIR = join(homedir(), ".gonext");
 export const API_PID_FILE = join(GONEXT_DIR, "local-api.pid");
 export const API_LOG_FILE = join(GONEXT_DIR, "local-api.log");
@@ -62,7 +86,7 @@ export function generateLocalUserId() {
  */
 export function apiEnv({
   port = LOCAL_API_PORT,
-  mongoUri = LOCAL_MONGO_URI,
+  mongoUri = localMongoUri(),
   dbName = LOCAL_DB_NAME,
   base = {},
   hostedBase = hostedApiBase(),
@@ -201,7 +225,7 @@ export function installApiDeps(apiDir, { onLine } = {}) {
  */
 export async function seedDatabase({
   apiDir,
-  mongoUri = LOCAL_MONGO_URI,
+  mongoUri = localMongoUri(),
   dbName = LOCAL_DB_NAME,
   userId,
   workerKey,
@@ -245,7 +269,7 @@ export async function seedDatabase({
 export async function startLocalApi({
   apiDir,
   port = LOCAL_API_PORT,
-  mongoUri = LOCAL_MONGO_URI,
+  mongoUri = localMongoUri(),
   dbName = LOCAL_DB_NAME,
 } = {}) {
   await mkdir(GONEXT_DIR, { recursive: true });
@@ -700,7 +724,7 @@ export function settingsPatch({ existing = {}, models = CLIENT_EXPERT_MODELS } =
  */
 export async function modelsAlreadyConfigured({
   apiDir,
-  mongoUri = LOCAL_MONGO_URI,
+  mongoUri = localMongoUri(),
   dbName = LOCAL_DB_NAME,
   userId,
 } = {}) {
@@ -721,7 +745,7 @@ export async function modelsAlreadyConfigured({
 
 export async function seedSettings({
   apiDir,
-  mongoUri = LOCAL_MONGO_URI,
+  mongoUri = localMongoUri(),
   dbName = LOCAL_DB_NAME,
   userId,
   // The box the user chose at setup (task #220), or the shared-Ollama defaults when the

package/client-expert-run.mjs

@@ -894,7 +894,7 @@ export async function runClientExpertSetup({ ask }) {
     // machine alone (that rule protects anyone who set their models up by hand), so on a re-run
     // of this step the answer would be collected and silently discarded. Point at `/model`
     // instead — it is the one path that CAN change a machine that is already set up.
-    const already = await modelsAlreadyConfigured({ apiDir, userId: creds.userId });
+    const already = await modelsAlreadyConfigured({ apiDir, mongoUri, userId: creds.userId });
     if (already) {
       console.log("");
       console.log(dim("  Models are already configured on this machine — leaving them alone."));
@@ -946,6 +946,7 @@ export async function runClientExpertSetup({ ask }) {
   try {
     const written = await seedSettings({
       apiDir,
+      mongoUri,
       userId: creds.userId,
       ...(chosenModels ? { models: chosenModels } : {}),
     });

package/gu-cli.mjs

@@ -1229,7 +1229,7 @@ if (args[0] === "clear" || args[0] === "reset") {
     try {
       const ce = await import("./client-expert-api.mjs");
       const { MongoClient } = ce.loadMongoDriver(apiDirForDrop);
-      const client = new MongoClient(ce.LOCAL_MONGO_URI, { serverSelectionTimeoutMS: 8000 });
+      const client = new MongoClient(ce.localMongoUri(), { serverSelectionTimeoutMS: 8000 });
       await client.connect();
       await client.db(ce.LOCAL_DB_NAME).dropDatabase();
       await client.close();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gu-cli",
-  "version": "1.0.765",
+  "version": "1.0.767",
   "description": "Gu CLI — the gu terminal plus the local worker that runs agent / OCR / PDF / embedding jobs on your Mac (Ollama / MLX / OpenAI-compatible).",
   "type": "module",
   "license": "MIT",