@kidsinai/kids-client 0.0.24 → 0.0.26

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "@kidsinai/kids-client",
-  "version": "0.0.24",
+  "version": "0.0.26",
   "type": "module",
   "description": "Own-client TUI for Kids OpenCode — talks to local `opencode serve` via @opencode-ai/sdk v2 with kid-warm rendering, mission progress, permission dialog, and stderr-tail audit pipeline.",
   "license": "MIT",

package/src/core/models.ts

@@ -69,3 +69,28 @@ function pickModels(models: unknown): unknown[] {
   if (models && typeof models === "object") return Object.values(models)
   return []
 }
+
+/** True when an LLM error is about the *model itself* (not auth/network) — e.g.
+ *  a model the current ChatGPT-account auth isn't allowed to use. The fix is to
+ *  switch models, so callers should re-pick rather than retry or re-auth. */
+export function isModelUnavailable(msg: string): boolean {
+  const m = msg.toLowerCase()
+  return m.includes("not supported")
+    || m.includes("model_not_found")
+    || m.includes("does not have access")
+    || m.includes("not available")
+    || m.includes("unsupported model")
+}
+
+/** Pick a kid-safe default model: prefer the small/standard tiers that work
+ *  with ChatGPT-account auth; skip the `-pro` tiers (API-key only, rejected by
+ *  the Codex/OAuth path). Returns null only if the server reports no models. */
+export function pickDefaultModel(models: ModelChoice[]): ModelChoice | null {
+  const usable = models.filter((m) => !/-pro\b/i.test(m.id))
+  const prefer = ["gpt-5.4-mini", "gpt-5.4", "claude-3-5-sonnet", "sonnet", "gpt-5.5"]
+  for (const p of prefer) {
+    const hit = usable.find((m) => m.id.toLowerCase().includes(p))
+    if (hit) return hit
+  }
+  return usable[0] ?? models[0] ?? null
+}

package/src/core/serve-manager.ts

@@ -56,6 +56,16 @@ export interface ServeManagerOptions {
    */
   serverUsername: string
   opencodeBin: string
+  /**
+   * Inline opencode config (JSON string) passed to the spawned serve via
+   * OPENCODE_CONFIG_CONTENT. opencode otherwise reads the empty global config
+   * and falls back to a default model the ChatGPT-account auth can't use
+   * (gpt-5.5-pro) with permissions unset. Passing a curated `{ model,
+   * permission }` here pins a usable default model AND enforces the kid-safety
+   * "ask before write/edit" gate at the source. (OPENCODE_CONFIG as a file
+   * path is unreliable; the inline content var is what opencode honors.)
+   */
+  opencodeConfigContent?: string
   /** Max total wait for readiness in ms. Default {@link DEFAULT_READY_TIMEOUT_MS}. */
   readyTimeoutMs?: number
   /** Per-probe abort ceiling in ms. Default {@link DEFAULT_PROBE_TIMEOUT_MS}. */
@@ -101,6 +111,9 @@ export class ServeManager {
         ...process.env,
         OPENCODE_SERVER_PASSWORD: this.opts.serverPassword,
         OPENCODE_SERVER_USERNAME: this.opts.serverUsername,
+        ...(this.opts.opencodeConfigContent
+          ? { OPENCODE_CONFIG_CONTENT: this.opts.opencodeConfigContent }
+          : {}),
       },
       stdout: "pipe",
       stderr: "pipe",

package/src/index.tsx

@@ -20,6 +20,7 @@
 import React from "react"
 import { render } from "ink"
 import { join } from "node:path"
+import { readFileSync } from "node:fs"
 import { readEnv, validateEnv, type KidsClientEnv } from "./core/env.ts"
 import { ServeManager } from "./core/serve-manager.ts"
 import { createKidsClient, type OpencodeClient } from "./core/connection.ts"
@@ -31,7 +32,7 @@ import { listInstalledPacks, resolveContext } from "./core/course-pack.ts"
 import { readLastSession, writeLastSession } from "./core/last-session.ts"
 import { isCompletionTrigger, runCheck } from "./core/check-runner.ts"
 import { parseSlash, matchCommand } from "./core/commands.ts"
-import { listModels } from "./core/models.ts"
+import { listModels, isModelUnavailable, pickDefaultModel } from "./core/models.ts"
 import { findFiles } from "./core/files.ts"
 import { App } from "./render/ink/App.tsx"
 import { FREE_PLAY_PACK_ID } from "./render/ink/screens/CoursePackPicker.tsx"
@@ -344,6 +345,7 @@ async function bootServices(env: KidsClientEnv, store: Store): Promise<ServiceSe
     serverPassword: env.opencodeServerPassword,
     serverUsername: env.opencodeServerUsername,
     opencodeBin: env.opencodeBin,
+    opencodeConfigContent: buildServeConfig(env.configDir),
     onAuditLine: (event) => {
       audit.push(event)
       store.pushAudit(event)
@@ -652,10 +654,32 @@ function makeFullHandlers(
 
       store.update({ thinking: true })
       updateLastSession()
+      // Resolve a usable model up front. With no explicit pick, the server
+      // falls back to whatever model it last used — which may be one the
+      // current auth can't use (a ChatGPT-account login can't use gpt-5.5-pro).
+      // Pinning a known-good default makes the kid's first message just work.
+      let model = snap.selectedModel
+      if (!model) {
+        const def = pickDefaultModel(await listModels(client))
+        if (def) {
+          model = def.id
+          store.update({ selectedModel: def.id, selectedModelLabel: def.label })
+        }
+      }
       try {
-        await session.prompt(text, { model: snap.selectedModel ?? undefined })
+        await session.prompt(text, { model: model ?? undefined })
       } catch (err) {
         const detail = errMessage(err)
+        if (isModelUnavailable(detail)) {
+          // The model isn't usable on this account (e.g. a -pro model under a
+          // ChatGPT login). Clear it so the next message auto-picks a good
+          // default, and guide the kid to /model instead of a scary error.
+          store.update({ thinking: false, selectedModel: null, selectedModelLabel: null })
+          sysMessage(env.locale === "zh-Hans"
+            ? "这个 AI 模型在你的账号下用不了。直接再发一条消息会自动换成可用模型，或打 /model 自己选（推荐 gpt-5.4-mini）。"
+            : "That AI model isn't available on your account. Just send again to auto-switch to a usable one, or type /model to choose (try gpt-5.4-mini).")
+          return
+        }
         store.update({ thinking: false, screen: { kind: "error", variant: classifyLlmError(detail), detail } })
       }
     },
@@ -893,6 +917,34 @@ function errMessage(err: unknown): string {
   return String(err)
 }
 
+/**
+ * Build the inline opencode config passed to the spawned serve. opencode's
+ * own global config is effectively empty, so without this it defaults the
+ * openai provider to gpt-5.5-pro (rejected by ChatGPT-account auth) with no
+ * permission gate. We forward a curated, schema-clean subset of the kids
+ * preset: a usable default model + the kid-safety "ask before acting" gate.
+ *
+ * Deliberately NOT forwarded: the preset's `agent.tools` / `_comment` keys —
+ * they fail opencode's config schema (which silently drops the whole config).
+ * The tool whitelist is enforced by the kids plugin regardless, so nothing is
+ * lost. Returns undefined if the preset is missing/unreadable (serve then
+ * falls back to its own config — same as before this change).
+ */
+function buildServeConfig(configDir: string): string | undefined {
+  let model = "openai/gpt-5.4-mini"
+  try {
+    const preset = JSON.parse(readFileSync(join(configDir, "opencode.json"), "utf8")) as { model?: string }
+    if (typeof preset.model === "string" && preset.model.includes("/")) model = preset.model
+  } catch {
+    return undefined
+  }
+  return JSON.stringify({
+    $schema: "https://opencode.ai/config.json",
+    model,
+    permission: { edit: "ask", write: "ask", bash: "ask", webfetch: "ask" },
+  })
+}
+
 void main().catch((err) => {
   console.error("kids-client: fatal startup error:", err)
   process.exit(1)