[WRONG BRANCH] release: 2.78.0-preview.20261005

bin/ocx.mjs

@@ -52,7 +52,7 @@ import {
 } from "../src/update/npm-cache-preflight.mjs";
 import { handoffWindowsTrayForUpdate, planWindowsTrayUpdate } from "../src/update/tray-update-plan.mjs";
 import { bootRestoreProbe, launcherUsableAfterNpmUpdate, transactionalNpmUpdate } from "../src/update/transactional-install.mjs";
-import { npmUpdateFailureGuidance } from "../src/update/update-failure-guidance.mjs";
+import { manualUpdateFailureGuidance, npmUpdateFailureGuidance } from "../src/update/update-failure-guidance.mjs";
 import {
   CODEX_CLI_VERSION_MANAGER_ROOT_ENV_SLOTS,
   isCodexCliUpdateInspectionArgv,
@@ -816,14 +816,11 @@ function runPackageManagerSelfUpdate(manager) {
       // legacy in-place install (which deletes live first) is exactly the wrong rescue —
       // it recreates the #1849 destruction path. Report and stop; the boot probe and the
       // recovery marker cover the swap-window states.
-      const manual = manager === "pnpm"
-        ? `pnpm add -g --allow-build=bun ${PKG}@${tag}`
-        : `npm install -g --allow-scripts=bun ${PKG}@${tag}`;
       // An unexpected exception leaves the active package path unproven for either manager.
       // Do not run service/tray/proxy recovery through a possibly half-swapped tree.
       postUpdateLauncherUsable = false;
       console.error(`opencodex: ${manager} update failed unexpectedly (${error?.message ?? error}). ` +
-        `The live install was not knowingly modified; run 'ocx update' again or reinstall with ${manual}.`);
+        `Run 'ocx update' again or follow the manual recovery steps below.`);
       res = { status: 1 };
     }
     if (res.status !== 0) recoverStoppedRuntimeAfterFailure("update failed");
@@ -861,13 +858,17 @@ function runPackageManagerSelfUpdate(manager) {
     // Phase-specific next step (#5624): whether the previous version is still in place decides
     // between "retry" and "restore", and a bare reinstall must follow a stop (#5496).
     const guidance = npmUpdateFailureGuidance({ ...npmFailure, pkgName: PKG, version: latest || undefined, tag });
-    console.error(`\nUpdate failed (npm ${npmFailure.phase}). ${guidance.lines.join(" ")}`);
+    console.error(`\nUpdate failed (npm ${npmFailure.phase}). ${guidance.lines.join("\n")}`);
     process.exit(1);
   }
-  const manual = manager === "pnpm"
-    ? `pnpm add -g --allow-build=bun ${PKG}@${tag}`
-    : `npm install -g --allow-scripts=bun ${PKG}@${tag}`;
-  console.error(`\nUpdate failed (${manager} exit ${res.status ?? "?"}). Try manually:  ${manual}`);
+  const guidance = manualUpdateFailureGuidance({
+    bin: manager,
+    args: manager === "pnpm"
+      ? ["add", "-g", "--allow-build=bun", `${PKG}@${latest || tag}`]
+      : ["install", "-g", "--allow-scripts=bun", `${PKG}@${latest || tag}`],
+    owner,
+  });
+  console.error(`\nUpdate failed (${manager} exit ${res.status ?? "?"}). ${guidance.join("\n")}`);
   process.exit(1);
 }
 

desktop/src-tauri/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "opencodex-desktop"
-version = "2.76.0-preview.20261003"
+version = "2.78.0-preview.20261005"
 description = "OpenCodex desktop shell"
 authors = ["OpenCodex contributors"]
 license = "MIT"
@@ -38,6 +38,10 @@ tokio = { version = "=1.45.1", features = ["sync", "time"] }
 [target.'cfg(target_os = "linux")'.dependencies]
 dbus = "=0.9.12"
 
+# Already present in the lock graph; only Windows writes its quoted Run command.
+[target.'cfg(target_os = "windows")'.dependencies]
+winreg = "=0.55.0"
+
 [profile.release]
 codegen-units = 1
 lto = "thin"

desktop/src-tauri/src/first_run.rs

@@ -6,8 +6,13 @@ use tauri_plugin_autostart::ManagerExt;
 const MARKER: &str = "start-at-login-claimed";
 
 /// Marker file recording that the login item names the launch-origin argument.
+#[cfg(not(target_os = "windows"))]
 const ORIGIN_MARKER: &str = "start-at-login-origin-flag";
 
+// Windows must revisit registrations claimed by the earlier, unquoted writer.
+#[cfg(target_os = "windows")]
+const ORIGIN_MARKER: &str = "start-at-login-quoted-origin-flag";
+
 /// What the one-time Start at Login decision did on this launch.
 #[derive(Clone, Copy, Debug, PartialEq, Eq)]
 pub enum StartAtLogin {
@@ -63,7 +68,7 @@ pub fn apply_start_at_login_default(app: &AppHandle) -> StartAtLogin {
     if app.autolaunch().is_enabled().unwrap_or(false) {
         return StartAtLogin::AlreadyDecided;
     }
-    match app.autolaunch().enable() {
+    match crate::login_autostart::enable(app) {
         Ok(()) => StartAtLogin::Enabled,
         Err(_) => StartAtLogin::Unavailable,
     }
@@ -99,7 +104,7 @@ pub fn adopt_launch_origin_argument(app: &AppHandle) {
     // leave a login launch showing its window forever.
     match app.autolaunch().is_enabled() {
         Ok(true) => {
-            if app.autolaunch().enable().is_err() {
+            if crate::login_autostart::enable(app).is_err() {
                 return;
             }
         }

desktop/src-tauri/src/lib.rs

@@ -9,6 +9,9 @@ mod first_run;
 mod formatting;
 mod identity;
 mod logging;
+mod login_autostart;
+#[cfg(any(target_os = "windows", test))]
+mod windows_autostart_command;
 // macOS only: it exists to replace one item in a menu no other platform installs. Compiling it
 // elsewhere would leave its contents unreachable, which -D warnings rejects.
 #[cfg(target_os = "macos")]

desktop/src-tauri/src/login_autostart.rs

@@ -0,0 +1,45 @@
+use tauri::AppHandle;
+
+/// All shell-owned enables use this entry point; the plugin still owns status and disable.
+#[cfg(not(target_os = "windows"))]
+pub fn enable(app: &AppHandle) -> Result<(), String> {
+    use tauri_plugin_autostart::ManagerExt;
+    app.autolaunch().enable().map_err(|error| error.to_string())
+}
+
+#[cfg(target_os = "windows")]
+pub fn enable(app: &AppHandle) -> Result<(), String> {
+    use winreg::enums::{HKEY_CURRENT_USER, KEY_SET_VALUE, REG_BINARY};
+    use winreg::{RegKey, RegValue};
+
+    let executable = std::env::current_exe().map_err(|error| error.to_string())?;
+    let executable = executable
+        .to_str()
+        .ok_or("autostart executable path is not Unicode")?;
+    let command =
+        crate::windows_autostart_command::command(executable, crate::startup::AUTOSTART_FLAG)?;
+    let name = &app.package_info().name;
+    let hkcu = RegKey::predef(HKEY_CURRENT_USER);
+    hkcu.open_subkey_with_flags(
+        r"SOFTWARE\Microsoft\Windows\CurrentVersion\Run",
+        KEY_SET_VALUE,
+    )
+    .and_then(|key| key.set_value(name, &command))
+    .map_err(|error| error.to_string())?;
+
+    // Match the plugin's explicit-enable behavior when Task Manager has an override.
+    if let Ok(key) = hkcu.open_subkey_with_flags(
+        r"SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\StartupApproved\Run",
+        KEY_SET_VALUE,
+    ) {
+        key.set_raw_value(
+            name,
+            &RegValue {
+                vtype: REG_BINARY,
+                bytes: vec![2, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0],
+            },
+        )
+        .map_err(|error| error.to_string())?;
+    }
+    Ok(())
+}

desktop/src-tauri/src/tray.rs

@@ -208,7 +208,7 @@ pub fn install(app: &AppHandle) -> tauri::Result<()> {
                 if enabled {
                     let _ = app.autolaunch().disable();
                 } else {
-                    let _ = app.autolaunch().enable();
+                    let _ = crate::login_autostart::enable(app);
                 }
             }
             "stop-proxy" => {

desktop/src-tauri/src/windows_autostart_command.rs

@@ -0,0 +1,41 @@
+/// Build the Run value without letting spaces split the executable path.
+pub fn command(executable: &str, argument: &str) -> Result<String, &'static str> {
+    if executable.is_empty() || executable.contains(['"', '\0']) {
+        return Err("invalid executable path for Windows autostart");
+    }
+    Ok(format!("\"{executable}\" {argument}"))
+}
+
+#[cfg(test)]
+mod tests {
+    use super::command;
+
+    #[test]
+    fn quotes_program_files_path_and_keeps_launch_origin() {
+        assert_eq!(
+            command(
+                r"C:\Program Files\OpenCodex\opencodex-desktop.exe",
+                "--autostart"
+            )
+            .unwrap(),
+            r#""C:\Program Files\OpenCodex\opencodex-desktop.exe" --autostart"#
+        );
+    }
+
+    #[test]
+    fn quotes_paths_without_spaces_and_preserves_unicode() {
+        for path in [r"C:\OpenCodex\app.exe", r"C:\用户\OpenCodex\app.exe"] {
+            assert_eq!(
+                command(path, "--autostart").unwrap(),
+                format!("\"{path}\" --autostart")
+            );
+        }
+    }
+
+    #[test]
+    fn rejects_paths_that_cannot_be_represented_in_a_run_command() {
+        for path in ["", "bad\"path.exe", "bad\0path.exe"] {
+            assert!(command(path, "--autostart").is_err());
+        }
+    }
+}

desktop/src-tauri/tauri.conf.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://schema.tauri.app/config/2",
   "productName": "OpenCodex",
-  "version": "2.76.0-preview.20261003",
+  "version": "2.78.0-preview.20261005",
   "identifier": "com.opencodex.desktop",
   "build": {
     "frontendDist": "../ui",

devlog/_fin/261003_antigravity_effort_families/000_plan.md

@@ -0,0 +1,49 @@
+# Antigravity discovered effort families
+
+New Claude 5.5 Opus and Sonnet tiers currently appear as separate models because discovery only collapses families listed in the bundled catalog. This unit makes complete discovered low/medium/high families one model with selectable effort, including future versions, while preserving explicit suffix requests.
+
+- Class: C3, one cohesive PABCD work-phase (wp1); satisfy-spec.
+- Trigger/goal: group the new Antigravity models and retain grouping during automatic refresh; publish and merge one ordinary PR into dev.
+- Non-goals: upstream API invention, authentication changes, release, deployment, running-service restart, unrelated UI changes.
+- Scope: current managed worktree; existing GitHub identity for authorized PR publication/integration. No user token/cost/wall-clock cap was set.
+- Verifiers: focused parser/wire/catalog/new-model-policy regressions, typecheck, structure/privacy/layout gates, docs build, exact-head hosted CI and merge receipt. Existing wire baseline: 61 pass, 0 fail (`bun test tests/adapters/google/google-antigravity-wire.test.ts`). Tests name source arguments directly; typecheck includes src through tsconfig; Bun executes the targeted test files.
+- Stop: implementation and independent review complete, relevant checks green, PR merged into dev. DONE needs these proofs; unresolved external checks remain unmet, never passed by assumption.
+- Evidence: this unit and local .codexclaw receipts. Escalate only new authority or genuine unresolved design blockers.
+
+## Findings and decisions
+
+`src/providers/antigravity-models.ts:127` and `:207` require a bundled picker ID before recognizing a full suffix family. `src/codex/catalog/provider-models.ts:756` then publishes an empty effort ladder for newly discovered models even when their wire map is exact. Existing base-URL scoped discovery mappings already route effort and are generation-fenced.
+
+Reuse those owners. No new provider registry version list or GUI grouping implementation is needed. Saved suffix IDs remain wire-authoritative. Incomplete ladders stay directly routable; unknown single-wire/tiered semantics remain unknown. Existing explicit configuration overrides retain precedence.
+
+## Consultation
+
+Architect proposal/reflection pending from handle 01a10104-3fca-79a2-8f9c-a410eb64d57e (V1 logical read-only architect, inherited model). Main owns integration; independent reviewer will audit the final plan before implementation.
+
+Architect D01/D02/D03/D05/D06 accepted. D04 amended: carry exact discovered family evidence with CatalogModel; normalize only discovery baseline and base disabled state through the existing reconciliation/persistence path. Keep provider selectedModels unchanged; a read-only shared visibility projection recognizes selected suffixes. Preserve raw default/combo references. No provider configuration migration or extra persistence surface. All-disabled suffixes transfer disabled status once; any enabled known suffix preserves an enabled base. A previously known base and explicit base disable win on later refreshes.
+
+Reflection: D01/D02/D03/D05/D06 ALIGNED; D04 gap (retained suffix IDs reappearing as new arrivals) folded into final plan by normalizing both policy input and baseline. Same architect recheck requested; no implementation before resolution and independent A audit.
+
+Final same-architect reflection: ALIGNED for D01-D06, no remaining material design gap. Baseline additionally passed typecheck and 29 listing/policy tests. Proceed to independent A audit.
+
+A round 1: FAIL, one accepted blocker: final merge independently filters raw selectedModels. Added retained-sync and convergence consumers and final-merge regression. No other material design blocker. Same reviewer re-audit requested.
+
+## Build and verification
+
+Implemented the audited plan. A bounded worker owned the family helper/policy/tests; main integrated parser, catalog, both final-merge selection callers and management projection. Discovery tests first failed 8/8 on the original source and passed after the fix. Existing future-family expectations were updated to assert the requested grouping. OpenCodex's existing synthetic ultra orchestration level remains separate from upstream low/medium/high; no unsupported synthetic max is added to the discovered ladder by default.
+
+Focused verification: 43 isolated test files, 830 pass / 0 fail, covering all Google adapter tests, family/policy/persistence, management and final merge, layout, file-size and optional-Lab import boundaries. The initial combined-process run had 803 pass / 2 fail: new test names disagreed with seed ownership (fixed by provider-prefixed names); an unchanged gemini-web-search mock replaced the listing test's OAuth token across files (each file passes in its own process). Full local suite is disproportionate for this bounded catalog change and shared workstation; repository resource exception uses these affected regressions, with full platform coverage left to exact-head hosted CI. Local logs stay in ignored scratch.
+
+Typecheck passed. Docs build passed (561 pages and 77,923 internal links). Structure and privacy scans passed. Structure catalog was already at its 600-line budget, so the shared family contract lives in providers-and-adapters with a link from the existing catalog paragraph.
+
+## Reviewed implementation outcome
+
+Independent implementation reviewer checked commit `916bd9d600723375c3c5667f4fe7b085af962c4d`, found no concrete regression, and independently ran all 28 new family tests with zero failures. The parser, cache, compatibility, policy projection, both final merge filters, and public list callers are covered. No live upstream model request was made; tests exercise the supplied wire-ID contract with controlled CCA discovery fixtures.
+
+Implementation is complete and published in PR #6501: https://github.com/lidge-jun/opencodex/pull/6501. The PR's live Verification section carries the final hosted-CI and authorized maintainer-integration receipt. Final goal completion additionally requires that exact-head gate and verified dev merge; neither publication nor this archived implementation record claims those external steps already passed.
+
+## External-review corrections
+
+The initial PR head passed hosted CI, but external review found a restart/outage defect that the in-memory tests missed. Accepted and corrected: a bounded exact-family snapshot now precedes synthetic catalog publication, restores routing after restart, and cannot revive after generation invalidation or an authoritative empty/partial replacement. Private no-follow atomic replacement and destination hashes keep URLs and credentials out of the snapshot. Write failure retains prior coherent discovery state. Overlapping family identities are preserved in public rows, policy and original-only selection projection.
+
+The same architect confirmed the revised D05 design ALIGNED. Independent code/security re-review closed both findings and passed 35 then-current focused tests. Main subsequently added two boundary assertions and confirmed 839 tests across 44 affected files (per-file isolation plus the final focused additions), with typecheck and structure checks passing. Restart tests use separate processes and the actual Google adapter while discovery throws, including each tier, medium default, corruption, size bounds, destination/home separation, tombstones and write-failure publication. Process restart is verified; power-loss durability is not claimed. Final updated-head hosted CI and merge receipt remain in the PR's Verification section.