osborn 0.9.20 → 0.9.22

package/Dockerfile.sandbox

@@ -16,8 +16,17 @@ RUN apt-get update -qq && \
     python-is-python3 && \
     rm -rf /var/lib/apt/lists/*
 
-# Install osborn + claude-code globally
-RUN npm install -g osborn@latest @anthropic-ai/claude-code
+# Pin osborn to an explicit version so Docker layer cache invalidates whenever
+# the version changes. Earlier `RUN npm install -g osborn@latest` looked correct
+# but it never invalidated: `latest` is a tag resolved by npm at install time,
+# but Docker's layer cache keys on the literal RUN command string. So every
+# rebuild reused the cached layer from when "latest" was a previous version,
+# producing images labeled :0.9.21 that actually contained osborn 0.9.19 inside.
+#
+# Pass --build-arg OSBORN_VERSION=X.Y.Z when building. image-build-check.ts
+# does this automatically using the npm-registry-resolved latest version.
+ARG OSBORN_VERSION=latest
+RUN npm install -g "osborn@${OSBORN_VERSION}" @anthropic-ai/claude-code
 
 # Persistent workspace + claude config dirs
 RUN mkdir -p /workspace /root/.claude

package/dist/index.js

@@ -397,22 +397,18 @@ function startApiServer(workingDir, port) {
                 const destSlug = join(projectsDir, effectiveSlug);
                 mkdirSync(destSlug, { recursive: true });
                 const sourceSlugPath = join(effectiveSource, sourceSlug);
-                // Always rewrite EVERY `"cwd":"<anything>"` to the target workspace dir
-                // when targetWorkDir is provided. Earlier version tried to derive the
-                // source cwd from the slug name via dash-to-slash substitution, but
-                // slug encoding is ambiguous (hyphens in path components like
-                // `codespaces-blank` get expanded into directory boundaries). That
-                // version only caught the canonical sprite cwd and missed laptop +
-                // codespaces references inside the JSONL bodies. Migration test on
-                // bj2zgt → fly machine found 12 files still pointing at /Users/...
-                // or /workspaces/codespaces-blank after the old rewrite. The brute-
-                // force regex below is bulletproof: any `cwd` reference becomes the
-                // runtime workspace dir, period.
-                const destCwd = targetWorkDir ?? slugToCwd(effectiveSlug);
-                const needsCwdRewrite = !!targetWorkDir;
-                void sourceSlugPath; // silence linter — used below in walk
-                const cwdRewriteRe = /"cwd":"[^"]*"/g;
-                const cwdReplacement = `"cwd":"${destCwd}"`;
+                // NO content mutation. Earlier versions rewrote the embedded `"cwd":"..."`
+                // field inside JSONL entries to match the destination workspace. That was
+                // wrong on two counts:
+                //   1. The cwd field is documentary metadata, not how Claude Code resolves
+                //      a session at resume time — resume uses the slug directory name.
+                //   2. Mutating contents breaks roundtripability (laptop → cloud → laptop
+                //      ends up with /workspace cwd on laptop), corrupting historical data
+                //      across environment hops.
+                // What's actually needed is just the slug rename (handled by `effectiveSlug`
+                // below). File contents stay byte-exact across every transfer direction.
+                void sourceSlugPath;
+                void targetWorkDir;
                 // Walk the source slug directory and copy files individually so we can:
                 //   (a) skip AppleDouble per-file too (in case nested)
                 //   (b) rewrite cwd inside .jsonl files when remapping across workspaces
@@ -444,17 +440,10 @@ function startApiServer(workingDir, port) {
                             catch { /* dst doesn't exist — write it */ }
                             if (!shouldWrite)
                                 continue;
-                            if (needsCwdRewrite && e.name.endsWith('.jsonl')) {
-                                // Regex-rewrite every `"cwd":"<anything>"` → target. Catches all
-                                // possible source cwds in one pass (laptop, codespaces, sprite,
-                                // etc.) without depending on slug-derived heuristics.
-                                const content = readFileSync(sp, 'utf8');
-                                const rewritten = content.replace(cwdRewriteRe, cwdReplacement);
-                                writeFileSync(dp, rewritten);
-                            }
-                            else {
-                                cpSync(sp, dp, { force: true });
-                            }
+                            // Copy byte-exact — no content mutation. The slug rename above is
+                            // the only structural change; file contents are immutable historical
+                            // record and must roundtrip cleanly between environments.
+                            cpSync(sp, dp, { force: true });
                             filesWritten++;
                         }
                         // skip symlinks, sockets, etc.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "osborn",
-  "version": "0.9.20",
+  "version": "0.9.22",
   "description": "Voice AI coding assistant - local agent that connects to Osborn frontend",
   "type": "module",
   "bin": {