test: stabilize shared mocks

src/cli/install.test.ts

@@ -120,8 +120,10 @@ mock.module('./background-subagents', () => {
     ...actualBackgroundSubagents,
     isBackgroundSubagentsEnabled: (env?: string) =>
       enableInstallMocks ? true : originalIsBackgroundSubagentsEnabled(env),
-    detectBackgroundSubagentsTarget: () =>
-      enableInstallMocks ? '/path' : originalDetectBackgroundSubagentsTarget(),
+    detectBackgroundSubagentsTarget: (env?: NodeJS.ProcessEnv) =>
+      enableInstallMocks
+        ? '/path'
+        : originalDetectBackgroundSubagentsTarget(env),
     expandHomePath: (p: string) =>
       enableInstallMocks ? p : originalExpandHomePath(p),
     getBackgroundSubagentsBlock: (target: string) =>

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

@@ -984,7 +984,7 @@ describe('auto-update-checker/index', () => {
     const hook = createAutoUpdateCheckerHook(ctx as never);
     hook.event({ event: { type: 'session.created', properties: {} } });
 
-    await waitForCalls(logMock, 1);
+    await waitForCalls(skillSyncMocks.syncBundledSkillsFromPackage, 1);
 
     expect(skillSyncMocks.syncBundledSkillsFromPackage).toHaveBeenCalledWith(
       '/tmp/opencode',

src/utils/env.test.ts

@@ -10,12 +10,17 @@ describe('isTruthyEnvValue', () => {
     expect(isTruthyEnvValue(value)).toBe(true);
   });
 
-  test.each([undefined, '', '0', 'false', 'no', 'off', 'anything'])(
-    '%p is not truthy',
-    (value) => {
-      expect(isTruthyEnvValue(value)).toBe(false);
-    },
-  );
+  test.each([
+    undefined,
+    '',
+    '0',
+    'false',
+    'no',
+    'off',
+    'anything',
+  ])('%p is not truthy', (value) => {
+    expect(isTruthyEnvValue(value)).toBe(false);
+  });
 });
 
 describe('isPluginDisabledByEnv', () => {