fix(auto-update-checker): skip the check when auto_update is false

packages/plugin/src/hooks/auto-update-checker/index.test.ts

@@ -187,6 +187,44 @@ describe("auto-update-checker/index", () => {
         // Even after waiting longer than initDelayMs would be, no check fires
         await new Promise((r) => setTimeout(r, 50));
         expect(showToast).not.toHaveBeenCalled();
+        expect(checkerMocks.getLatestVersion).not.toHaveBeenCalled();
+    });
+
+    test("auto_update: false makes no npm request (#854)", async () => {
+        checkerMocks.findPluginEntry.mockImplementation(() => ({
+            entry: "@cortexkit/opencode-magic-context",
+            pinnedVersion: null,
+            isPinned: false,
+            configPath: "/config/opencode.jsonc",
+        }));
+        checkerMocks.getCachedVersion.mockImplementation(() => "0.15.5");
+        checkerMocks.getLatestVersion.mockImplementation(async () => "0.15.6");
+        const { autoUpdateCheckerOptionsFromConfig, createAutoUpdateCheckerHook } =
+            await freshIndexImport();
+        const { ctx, showToast } = createCtx();
+
+        createAutoUpdateCheckerHook(ctx as Parameters<typeof createAutoUpdateCheckerHook>[0], {
+            ...autoUpdateCheckerOptionsFromConfig(false),
+            showStartupToast: false,
+            initDelayMs: 0,
+            storageDir: makeTempStorageDir(),
+        });
+
+        await new Promise((r) => setTimeout(r, 50));
+        expect(checkerMocks.getLatestVersion).not.toHaveBeenCalled();
+        expect(showToast).not.toHaveBeenCalled();
+    });
+
+    test("auto_update unset or true keeps the check enabled", async () => {
+        const { autoUpdateCheckerOptionsFromConfig } = await freshIndexImport();
+        expect(autoUpdateCheckerOptionsFromConfig(undefined)).toEqual({
+            enabled: true,
+            autoUpdate: true,
+        });
+        expect(autoUpdateCheckerOptionsFromConfig(true)).toEqual({
+            enabled: true,
+            autoUpdate: true,
+        });
     });
 
     test("on-disk timestamp dedupes concurrent plugin instances", async () => {

packages/plugin/src/hooks/auto-update-checker/index.ts

@@ -46,6 +46,18 @@ function warn(message: string): void {
     log(`WARN: ${message}`);
 }
 
+/**
+ * Maps the plugin's `auto_update` setting to checker options. `false` turns
+ * the checker off entirely so no npm request is made, matching the host
+ * updater's disable behavior.
+ */
+export function autoUpdateCheckerOptionsFromConfig(
+    autoUpdateSetting: boolean | undefined,
+): Pick<AutoUpdateCheckerOptions, "enabled" | "autoUpdate"> {
+    const enabled = autoUpdateSetting !== false;
+    return { enabled, autoUpdate: enabled };
+}
+
 /**
  * Auto-update checker.
  *

packages/plugin/src/index.ts

@@ -38,7 +38,10 @@ import {
 } from "./features/magic-context/storage-db";
 import { recordToolDefinition } from "./features/magic-context/tool-definition-tokens";
 import { runDeferredV22Backfill } from "./features/magic-context/v22-deferred-backfill";
-import { createAutoUpdateCheckerHook } from "./hooks/auto-update-checker";
+import {
+    autoUpdateCheckerOptionsFromConfig,
+    createAutoUpdateCheckerHook,
+} from "./hooks/auto-update-checker";
 import {
     COMPARTMENT_AGENT_SYSTEM_PROMPT,
     COMPARTMENT_STRUCTURAL_SYSTEM_PROMPT,
@@ -881,7 +884,8 @@ const server: Plugin = async (ctx) => {
                 },
             },
             autoUpdateChecker: createAutoUpdateCheckerHook(ctx, {
-                autoUpdate: pluginConfig.auto_update !== false,
+                // auto_update: false skips the checker entirely: no npm request.
+                ...autoUpdateCheckerOptionsFromConfig(pluginConfig.auto_update),
                 signal: autoUpdateAbort.signal,
                 // Multi-project plugin reloads coordinate via this on-disk
                 // timestamp so npm gets hit at most once per check window