fix - skip arborist plugin deps install in tests; bump snapshot revert timeout

Root cause of all 7 test timeouts:

  config.ts fires a background @npmcli/arborist.reify() for @opencode-ai/plugin
  in every .opencode/ directory it discovers. plugin/index.ts and tool/registry.ts
  then call waitForDependencies() which joins that fiber before loading plugins
  or custom tools. On Blacksmith ARM64, a cold arborist fetch takes 10-30 s per
  test. Running in parallel across a 4-vCPU runner saturates CPU/IO and starves
  even unrelated tests (session.processor, snapshot.revert) past their 30 s limit.

Fixes:
  - flag.ts: add OPENCODE_DISABLE_PLUGIN_DEPS_INSTALL flag
  - config.ts: guard the npmSvc.install() + deps.push() block with the new flag
  - preload.ts: set OPENCODE_DISABLE_PLUGIN_DEPS_INSTALL=true for all tests
    (safe: bun resolves @opencode-ai/plugin from the workspace node_modules;
    tests do not author plugins that import the SDK at runtime)
  - snapshot.test.ts: raise timeout on the 280-file revert test to 60 s
    (git operations across 280 files can push past 30 s on ARM64)

Author: lgarceau768

packages/opencode/src/config/config.ts

@@ -542,28 +542,30 @@ export const layer = Layer.effect(
 
           yield* ensureGitignore(dir).pipe(Effect.orDie)
 
-          const dep = yield* npmSvc
-            .install(dir, {
-              add: [
-                {
-                  name: "@opencode-ai/plugin",
-                  version: InstallationLocal ? undefined : InstallationVersion,
-                },
-              ],
-            })
-            .pipe(
-              Effect.exit,
-              Effect.tap((exit) =>
-                Exit.isFailure(exit)
-                  ? Effect.sync(() => {
-                      log.warn("background dependency install failed", { dir, error: String(exit.cause) })
-                    })
-                  : Effect.void,
-              ),
-              Effect.asVoid,
-              Effect.forkDetach,
-            )
-          deps.push(dep)
+          if (!Flag.OPENCODE_DISABLE_PLUGIN_DEPS_INSTALL) {
+            const dep = yield* npmSvc
+              .install(dir, {
+                add: [
+                  {
+                    name: "@opencode-ai/plugin",
+                    version: InstallationLocal ? undefined : InstallationVersion,
+                  },
+                ],
+              })
+              .pipe(
+                Effect.exit,
+                Effect.tap((exit) =>
+                  Exit.isFailure(exit)
+                    ? Effect.sync(() => {
+                        log.warn("background dependency install failed", { dir, error: String(exit.cause) })
+                      })
+                    : Effect.void,
+                ),
+                Effect.asVoid,
+                Effect.forkDetach,
+              )
+            deps.push(dep)
+          }
 
           result.command = mergeDeep(result.command ?? {}, yield* Effect.promise(() => ConfigCommand.load(dir)))
           result.agent = mergeDeep(result.agent ?? {}, yield* Effect.promise(() => ConfigAgent.load(dir)))

packages/opencode/src/flag/flag.ts

@@ -44,6 +44,12 @@ export const Flag = {
   OPENCODE_DISABLE_AUTOCOMPACT: truthy("OPENCODE_DISABLE_AUTOCOMPACT"),
   OPENCODE_DISABLE_MODELS_FETCH: truthy("OPENCODE_DISABLE_MODELS_FETCH"),
   OPENCODE_DISABLE_MOUSE: truthy("OPENCODE_DISABLE_MOUSE"),
+  // Skip the background @npmcli/arborist install of @opencode-ai/plugin into
+  // .opencode/ directories. In production this ensures the plugin SDK is
+  // available for user-authored plugins. In test environments bun resolves
+  // @opencode-ai/plugin from the workspace node_modules directly, so the
+  // install is unnecessary and causes test timeouts on slower CI runners.
+  OPENCODE_DISABLE_PLUGIN_DEPS_INSTALL: truthy("OPENCODE_DISABLE_PLUGIN_DEPS_INSTALL"),
   OPENCODE_DISABLE_CLAUDE_CODE,
   OPENCODE_DISABLE_CLAUDE_CODE_PROMPT: OPENCODE_DISABLE_CLAUDE_CODE || truthy("OPENCODE_DISABLE_CLAUDE_CODE_PROMPT"),
   OPENCODE_DISABLE_CLAUDE_CODE_SKILLS,

packages/opencode/test/preload.ts

@@ -45,6 +45,12 @@ process.env["OPENCODE_TEST_HOME"] = testHome
 const testManagedConfigDir = path.join(dir, "managed")
 process.env["OPENCODE_TEST_MANAGED_CONFIG_DIR"] = testManagedConfigDir
 process.env["OPENCODE_DISABLE_DEFAULT_PLUGINS"] = "true"
+// Skip the background @npmcli/arborist install of @opencode-ai/plugin into
+// .opencode/ dirs. Tests don't need the runtime npm install because bun
+// resolves @opencode-ai/plugin from the monorepo workspace node_modules.
+// Without this, plugin/tool tests trigger a full npm network fetch per test,
+// consuming 10-30 s on Blacksmith ARM64 CI and causing timeouts.
+process.env["OPENCODE_DISABLE_PLUGIN_DEPS_INSTALL"] = "true"
 
 // Write the cache version file to prevent global/index.ts from clearing the cache
 const cacheDir = path.join(dir, "cache", "opencode")

packages/opencode/test/snapshot/snapshot.test.ts

@@ -1528,4 +1528,4 @@ test("revert handles large mixed batches across chunk boundaries", async () => {
       )
     },
   })
-})
+}, 60_000) // 280 files + multiple git operations can exceed 30 s on slower ARM64 CI runners