@gotcos/glasses-server 6.39.0 → 6.39.2

package/CHANGELOG.md

@@ -1,3 +1,34 @@
+## 6.39.2
+
+Every Claude fork failed; the copy it left behind was real.
+
+`buildClaudeForkArgs` still carried `--tools ''`, the third read-only layer the
+attached path retired on 2026-08-16. On the current Claude CLI that flag makes
+a fork-resume attempt a context compaction that fails (`too_few_groups`) and
+then report a synthetic 400 "Prompt is too long" with zero input tokens -- so
+the route classified every claude fork `fork_orphan_possible` even though the
+forked transcript sat on disk complete except for the failed turn. Found the
+hour COS Control 0.5.76 put a Fork button in front of the route; isolated by
+single-flag bisection (plan-only completes, allowedTools-only completes,
+`--tools ''` alone reproduces).
+
+The fork keeps its read-only stance -- `--permission-mode plan` plus an empty
+`--allowedTools` -- and drops only the flag the CLI can no longer carry.
+Pinned by a regression test that goes red if `--tools` returns to the argv.
+
+## 6.39.1
+
+Durable jobs now remember they ran on Ollama.
+
+FIXED: 6.39.0 allowed `provider: 'ollama'` in the durable-job types and the
+runtime stamped it, but the journal sanitizer (`safeLinkage`) and the replay
+reader (`applyLinkage`) both stripped it -- so a live or replayed Ollama job
+came back providerless and the phone could never acknowledge which engine
+answered. Both now persist `ollama` and `ollamaRunId`; unknown providers still
+strip. Pinned by journal round-trip tests, mutation-verified.
+
+No other changes; pairs with COS Glasses 6.8.436 and COS Control 0.5.74.
+
 ## 6.39.0
 
 Ollama is a fourth G2 picker, hidden unless a local daemon answers.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotcos/glasses-server",
-  "version": "6.39.0",
+  "version": "6.39.2",
   "description": "COS Glasses \u2014 self-hosted AI heads-up-display server for Even G2 smart glasses, powered by Claude Code, Codex, Cursor Agent CLI, or local Ollama",
   "type": "module",
   "bin": {

package/server/lib/fork-thread.ts

@@ -211,11 +211,18 @@ export function forkOrphanPossible(result: ForkResult): boolean {
  * `claude -p --resume <id> --fork-session`, read-only, prompt on stdin.
  *
  * `--fork-session` is documented by the installed CLI as "When resuming, create a
- * new session ID" and is only meaningful alongside `--resume`. The read-only pair
- * (`--permission-mode plan` plus the empty tool lists) is carried over from the
- * attached path unchanged: a fork runs a real model turn, and it does so against a
- * workspace the user did not explicitly hand us, so it gets no more authority than
- * a continuation does.
+ * new session ID" and is only meaningful alongside `--resume`. The fork keeps a
+ * read-only stance — a fork runs a real model turn against a workspace the user
+ * did not explicitly hand us — but that stance is now `--permission-mode plan`
+ * plus an empty `--allowedTools` only. The third layer this used to carry,
+ * `--tools ''`, is GONE: on the current CLI an empty --tools on a fork-resume
+ * triggers a spurious context compaction (`too_few_groups`) followed by a
+ * synthetic 400 "Prompt is too long" with zero input tokens — every claude fork
+ * failed as orphan_possible while the transcript copy sat there complete.
+ * Isolated by single-flag bisection on 2026-08-26: plan-only completed,
+ * allowedTools-only completed, `--tools ''` alone reproduced the failure. (The
+ * attached path dropped all three layers on 2026-08-16 by explicit decision;
+ * the fork deliberately keeps the two that still work.)
  *
  * `stream-json` requires `--verbose`; without it the CLI refuses and we would never
  * see the id we are required to verify.
@@ -228,7 +235,6 @@ export function buildClaudeForkArgs(nativeThreadId: string): string[] {
     '--resume', nativeThreadId,
     '--fork-session',
     '--permission-mode', 'plan',
-    '--tools', '',
     '--allowedTools', '',
   ]
 }

package/server/lib/query-job-store.ts

@@ -550,11 +550,11 @@ export class QueryJobStore {
   }
 
   private applyLinkage(snapshot: QueryJobSnapshot, raw: Record<string, unknown>): void {
-    const provider = raw.provider === 'claude' || raw.provider === 'codex' || raw.provider === 'cursor'
+    const provider = raw.provider === 'claude' || raw.provider === 'codex' || raw.provider === 'cursor' || raw.provider === 'ollama'
       ? raw.provider
       : undefined
     if (provider) snapshot.provider = provider
-    const fields = ['resolvedModel', 'cliSessionId', 'claudeRunId', 'codexRunId', 'codexThreadId', 'cursorRunId'] as const
+    const fields = ['resolvedModel', 'cliSessionId', 'claudeRunId', 'codexRunId', 'codexThreadId', 'cursorRunId', 'ollamaRunId'] as const
     for (const field of fields) {
       const value = safeOptional(raw[field])
       if (value) snapshot[field] = value
@@ -886,7 +886,11 @@ export class QueryJobStore {
 
   private safeLinkage(linkage: QueryJobProviderLinkage): QueryJobProviderLinkage {
     return {
-      ...(linkage.provider === 'claude' || linkage.provider === 'codex' || linkage.provider === 'cursor'
+      // 'ollama' joined this allowlist in 6.39.1. The 6.39.0 types allowed it and
+      // query-job-runtime stamped it, but this sanitizer silently dropped it on
+      // persist -- so phone acknowledgement of a live or replayed Ollama job could
+      // never see its provider. Unknown providers still strip.
+      ...(linkage.provider === 'claude' || linkage.provider === 'codex' || linkage.provider === 'cursor' || linkage.provider === 'ollama'
         ? { provider: linkage.provider }
         : {}),
       ...(safeOptional(linkage.resolvedModel, 64) ? { resolvedModel: safeOptional(linkage.resolvedModel, 64) } : {}),
@@ -895,6 +899,7 @@ export class QueryJobStore {
       ...(safeOptional(linkage.codexRunId) ? { codexRunId: safeOptional(linkage.codexRunId) } : {}),
       ...(safeOptional(linkage.codexThreadId) ? { codexThreadId: safeOptional(linkage.codexThreadId) } : {}),
       ...(safeOptional(linkage.cursorRunId) ? { cursorRunId: safeOptional(linkage.cursorRunId) } : {}),
+      ...(safeOptional(linkage.ollamaRunId) ? { ollamaRunId: safeOptional(linkage.ollamaRunId) } : {}),
     }
   }