freebuff 0.0.196 → 0.0.197

package/index.js

@@ -29,6 +29,7 @@ const launcher = createLauncher({
   wrapperVersion: packageJson.version,
   binaryChecksums: packageJson.binaryChecksums ?? null,
   telemetryEvent: 'cli.update_freebuff_failed',
+  deferUpdatesUntilExit: true,
 })
 
 module.exports = launcher

package/launcher.js

@@ -139,6 +139,9 @@ function createLauncher(productConfig) {
     startupBanner = [],
     telemetryEvent = 'cli.update_codebuff_failed',
     telemetryProperties = {},
+    // Freebuff sessions remain open between turns. The launcher cannot know
+    // when a restart is safe, so keep the process alive until the user exits.
+    deferUpdatesUntilExit = false,
     tempDownloadDirName = `.${packageName}-download-temp`,
     // Tests only. os.homedir() ignores $HOME under `bun test`, so pointing HOME
     // at a temp dir is not enough to keep a test off the real ~/.config.
@@ -267,7 +270,14 @@ function createLauncher(productConfig) {
       binaryName,
       binaryPath: path.join(configDir, binaryName),
       metadataPath: path.join(configDir, `${packageName}-metadata.json`),
-      tempDownloadDir: path.join(configDir, tempDownloadDirName),
+      // A deferred download may wait here for an hour. Another invocation
+      // must not replace its staged binary (or delete it while downloading).
+      tempDownloadDir: path.join(
+        configDir,
+        deferUpdatesUntilExit
+          ? `${tempDownloadDirName}-${process.pid}`
+          : tempDownloadDirName,
+      ),
       userAgent: `${packageName}-cli`,
       requestTimeout: 20000,
       downloadRequestTimeout: 120000,
@@ -276,6 +286,7 @@ function createLauncher(productConfig) {
   }
 
   const CONFIG = createConfig(packageName)
+  const pendingUpdates = new WeakMap()
   const { downloadFile, httpGet, withRetries } = createReleaseHttpClient({
     env: process.env,
     userAgent: CONFIG.userAgent,
@@ -1085,7 +1096,10 @@ function createLauncher(productConfig) {
     }
   }
 
-  function installStagedBinary({ tempBinaryPath, version, targetKey }) {
+  function installStagedBinary(
+    { tempBinaryPath, version, targetKey },
+    { quiet = false } = {},
+  ) {
     const replacements = []
     const metadataTempPath = `${CONFIG.metadataPath}.new.${process.pid}`
 
@@ -1133,8 +1147,10 @@ function createLauncher(productConfig) {
       }
     }
 
-    term.clearLine()
-    console.log(`Download complete! Starting ${displayName}...`)
+    if (!quiet) {
+      term.clearLine()
+      console.log(`Download complete! Starting ${displayName}...`)
+    }
   }
 
   async function downloadBinary(
@@ -1240,7 +1256,11 @@ function createLauncher(productConfig) {
     })
   }
 
-  async function checkForUpdates(runningProcess, exitListener) {
+  async function checkForUpdates(
+    runningProcess,
+    exitListener,
+    fetchLatestRelease = getLatestRelease,
+  ) {
     // main() schedules this 100ms after launch, so the binary it was handed can
     // already be dead — a startup crash that handed off to the baseline
     // fallback, most of all. Updating around a corpse would race that
@@ -1259,9 +1279,14 @@ function createLauncher(productConfig) {
     try {
       const currentVersion = getCurrentVersion()
 
-      const latestRelease = await getLatestRelease()
+      const latestRelease = await fetchLatestRelease()
       const latestVersion = latestRelease?.version ?? null
       if (!latestVersion) return
+      if (
+        runningProcess.exitCode !== null ||
+        runningProcess.signalCode !== null
+      )
+        return
 
       if (
         // Download new version if current version is unknown or outdated.
@@ -1274,6 +1299,19 @@ function createLauncher(productConfig) {
           { quiet: true, binaryChecksums: latestRelease.binaryChecksums },
         )
 
+        if (deferUpdatesUntilExit) {
+          // No signals, terminal output, or binary/WASM replacement while the
+          // CLI owns the session. Even an idle model picker can admit a session
+          // while the download is in flight. A download that loses the race
+          // with exit must not install into a fallback child's running files.
+          if (
+            runningProcess.exitCode === null &&
+            runningProcess.signalCode === null
+          )
+            pendingUpdates.set(runningProcess, stagedBinary)
+          return
+        }
+
         term.clearLine()
 
         runningProcess.removeListener('exit', exitListener)
@@ -1313,6 +1351,33 @@ function createLauncher(productConfig) {
     }
   }
 
+  function installPendingUpdate(child) {
+    const staged = pendingUpdates.get(child)
+    pendingUpdates.delete(child)
+    if (!staged) return
+
+    try {
+      // A second launcher may have installed a newer release while this
+      // session was open. Never roll its installation back on our exit.
+      const currentVersion = getCurrentVersion()
+      if (
+        currentVersion &&
+        compareVersions(currentVersion, staged.version) >= 0
+      ) {
+        fs.rmSync(CONFIG.tempDownloadDir, { recursive: true, force: true })
+        return
+      }
+      installStagedBinary(staged, { quiet: true })
+      console.log(
+        `Updated ${displayName} to ${staged.version}. Ready for your next launch.`,
+      )
+    } catch (error) {
+      // Installation rolls back on failure. Updating must not change the
+      // user's exit code or resurrect a CLI they just closed.
+      console.error(`Could not install ${displayName} update: ${error.message}`)
+    }
+  }
+
   /**
    * Make captured output safe to print back to the terminal.
    *
@@ -1693,6 +1758,7 @@ function createLauncher(productConfig) {
         exitAlternateScreen: shouldExitAlternateScreen(code, signal),
       })
       printCrashDiagnostics(code, signal, { msAlive, stderrTail })
+      installPendingUpdate(child)
       process.exit(signal ? 1 : code || 0)
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "freebuff",
-  "version": "0.0.196",
+  "version": "0.0.197",
   "description": "The world's strongest free coding agent",
   "license": "MIT",
   "bin": {
@@ -41,13 +41,13 @@
     "provenance": false
   },
   "binaryChecksums": {
-    "linux-x64": "43a19d7def718fbd3ab316ae9dcf90b0839e50c4ed9688c9b99794a7b258d2aa",
-    "linux-x64-baseline": "2f2b277ba60369535828c5945561ea3d64c0de4d7cbca29389899197f3d86520",
-    "linux-arm64": "481d8becd750d13ce5da9e41e33029bfae63d417a29752cbdd36b9c5fef42f8b",
-    "darwin-x64": "cf82f805d6646d00551f47b9d25fa0691062d815480902d0d66e83feeddf8767",
-    "darwin-x64-baseline": "b0e2dc5bfee7eccc2fbc861cfcbd28e0a0a806e8ce84ad2d0628f463118ccfae",
-    "darwin-arm64": "31481e33ebd1ce657871e638814a0cf40983ee34826f320948e7c3a27c1aa606",
-    "win32-x64": "a3915eb379f9c5e7d001fd209831aa2ff9bd610f2e7a33f6f66b0d8d7f46cda7",
-    "win32-x64-baseline": "6562e33a4842207a7be9a0c69dda3a621ed412a0be50d96177771f34330da52c"
+    "linux-x64": "d7c809028c05beedda432b6bedb8defbdaa63f9ab189ca212306a539d351ca1f",
+    "linux-x64-baseline": "5e22f749d0ab0dda6fab36d39280824833109919c7ce4895d595e5ee6588a1fe",
+    "linux-arm64": "22d7607e57d215e3f1a290111c6e2bb1c71bbc5b84136e6661b626437fecad99",
+    "darwin-x64": "0b55ac5383f5ed8f2e4b1739428d435be1b6d729fb3f56c05b1564dd2dd97333",
+    "darwin-x64-baseline": "9588e4c8428ca996004fb57dbe021027d54abb1d69c937f830874c0585c4a815",
+    "darwin-arm64": "f732bf1a71a57e312a973264a835ca68eb55ce5e56371f052c5894576df2b016",
+    "win32-x64": "fa7309df307038cd474b6ba248754fdea011b245d1cffa2ca42d9f79f3ba71b4",
+    "win32-x64-baseline": "7fce9f5abe69b7b3babb63dc338b0c349dffbafbe9d0ccb731ef9c2bed8af89d"
   }
 }