fix(tests): LEVELCODE_HOME override + per-test isolated home preload

Root fix for the CI test matrix: bun caches os.homedir() at first use, so
HOME/USERPROFILE swapping per test did not move user-data paths after the
first caller — common's team tests leaked fixtures into the runner's real
home and collided across files (ENOTEMPTY on archive renames).

- getUserHomeDir() (common/src/utils/home-dir.ts): LEVELCODE_HOME wins over
  the cached os.homedir(); env reads are live
- all user-data paths routed through it: team-fs, team-discovery,
  team-maintenance, hooks loader, common getConfigDir
- test/setup-isolated-home.ts preload (wired into root + sdk bunfig):
  every test gets a fresh isolated home, cleaned up automatically
- agents-console test seeds through getTeamsDir()/getTasksDir() instead of
  literal home paths

Author: yethikrishna

bunfig.toml

@@ -7,4 +7,4 @@ linkWorkspacePackages = true
 [test]
 # Exclude test repositories, integration tests, and Playwright e2e tests from test execution by default
 exclude = ["evals/test-repos/**", "**/*.integration.test.*", "web/src/__tests__/e2e/**"]
-preload = ["./sdk/test/setup-env.ts", "./test/setup-bigquery-mocks.ts"]
+preload = ["./test/setup-isolated-home.ts", "./sdk/test/setup-env.ts", "./test/setup-bigquery-mocks.ts"]

cli/src/agents-console/__tests__/agents-console.test.ts

@@ -9,7 +9,7 @@ import {
   plainTheme,
 } from '../agents-console'
 import { collectAgentsData, renderAgentsCommand } from '../run-agents'
-import { getTeamsDir } from '@levelcode/common/utils/team-fs'
+import { getTeamsDir, getTasksDir } from '@levelcode/common/utils/team-fs'
 
 import type { TeamConfig, TeamMember, TeamTask } from '@levelcode/common/types/team-config'
 
@@ -148,18 +148,9 @@ describe('agents command (disk-backed)', () => {
     }
     process.env.HOME = tmpHome
     process.env.USERPROFILE = tmpHome
-    // bun may cache os.homedir() on first use; clean the cached location.
-    const configRoot = path.dirname(path.dirname(getTeamsDir()))
-    if (fs.existsSync(configRoot)) {
-      fs.rmSync(configRoot, { recursive: true, force: true })
-    }
   })
 
   afterEach(() => {
-    const configRoot = path.dirname(path.dirname(getTeamsDir()))
-    if (fs.existsSync(configRoot)) {
-      fs.rmSync(configRoot, { recursive: true, force: true })
-    }
     for (const [key, value] of Object.entries(origHome)) {
       if (value === undefined) delete process.env[key]
       else process.env[key] = value
@@ -168,7 +159,9 @@ describe('agents command (disk-backed)', () => {
   })
 
   function seedTeam(name: string): void {
-    const teamDir = path.join(tmpHome, '.config', 'levelcode', 'teams', name)
+    // Seed through getTeamsDir()/getTasksDir(): with the isolated-home
+    // preload these resolve to a fresh LEVELCODE_HOME per test.
+    const teamDir = path.join(getTeamsDir(), name)
     fs.mkdirSync(path.join(teamDir, 'inboxes'), { recursive: true })
     const config = makeConfig({
       name,
@@ -180,7 +173,7 @@ describe('agents command (disk-backed)', () => {
       JSON.stringify(config),
       'utf-8',
     )
-    const tasksDir = path.join(tmpHome, '.config', 'levelcode', 'tasks', name)
+    const tasksDir = getTasksDir(name)
     fs.mkdirSync(tasksDir, { recursive: true })
     fs.writeFileSync(
       path.join(tasksDir, '1.json'),

common/src/hooks/loader.ts

@@ -1,7 +1,8 @@
 import fs from 'fs'
-import os from 'os'
 import path from 'path'
 
+import { getUserHomeDir } from '../utils/home-dir'
+
 import { hooksConfigSchema } from './types'
 
 import type { HookMatcher, HookEventName, HooksConfig } from './types'
@@ -33,7 +34,7 @@ function configEnvSuffix(): string {
 
 /** Candidate settings.json paths in load order. */
 export function getHookConfigPaths(projectRoot: string): string[] {
-  const home = os.homedir()
+  const home = getUserHomeDir()
   const commonBase =
     process.env.LEVELCODE_DIR || path.join(home, '.levelcode')
   return [

common/src/utils/auth.ts

@@ -1,12 +1,13 @@
 import path from 'path'
-import os from 'os'
+
+import { getUserHomeDir } from './home-dir'
 
 // ============================================================================
 // Config Directory
 // ============================================================================
 
 export function getConfigDir(): string {
-  const home = os.homedir()
+  const home = getUserHomeDir()
   // Use .levelcode in home directory
   const baseDir = process.env.LEVELCODE_DIR || path.join(home, '.levelcode')
   return baseDir

common/src/utils/home-dir.ts

@@ -0,0 +1,14 @@
+import * as os from 'os'
+
+/**
+ * Resolvable user home directory.
+ *
+ * Bun caches `os.homedir()` at first use, which makes per-test HOME
+ * swapping unreliable (the first caller freezes the value for the whole
+ * process). User-data paths across the monorepo go through this helper
+ * instead, so tests and sandboxes can relocate the home reliably with
+ * `LEVELCODE_HOME` (env reads are always live).
+ */
+export function getUserHomeDir(): string {
+  return process.env.LEVELCODE_HOME || os.homedir()
+}

common/src/utils/team-discovery.ts

@@ -1,4 +1,5 @@
 import * as fs from 'fs'
+import { getUserHomeDir } from './home-dir'
 import * as path from 'path'
 import * as os from 'os'
 import { getTeamsDir, loadTeamConfig, validateTeamName } from './team-fs'
@@ -32,7 +33,7 @@ export interface TeamSummary {
 // agentStepId changes every call) can still locate the team.
 
 function getLastActiveTeamPath(): string {
-  return path.join(os.homedir(), '.config', 'levelcode', 'teams', '.last-active-team')
+  return path.join(getUserHomeDir(), '.config', 'levelcode', 'teams', '.last-active-team')
 }
 
 /**

common/src/utils/team-fs.ts

@@ -1,4 +1,5 @@
 import * as fs from 'fs'
+import { getUserHomeDir } from './home-dir'
 import * as path from 'path'
 import * as os from 'os'
 import type { TeamConfig, TeamMember, TeamTask } from '../types/team-config'
@@ -71,7 +72,7 @@ function assertPathContained(resolvedPath: string, expectedParent: string): void
 // --- Path helpers ---
 
 function getConfigRoot(): string {
-  return path.join(os.homedir(), '.config', 'levelcode')
+  return path.join(getUserHomeDir(), '.config', 'levelcode')
 }
 
 export function getTeamsDir(): string {

common/src/utils/team-maintenance.ts

@@ -1,12 +1,13 @@
 import * as fs from 'fs'
+import { getUserHomeDir } from './home-dir'
 import * as path from 'path'
 import * as os from 'os'
 import type { TeamConfig, TeamTask, AgentStatus } from '../types/team-config'
 import { teamConfigSchema, teamTaskSchema } from '../types/team-config-schemas'
 import { loadTeamConfig, listTasks, getTeamsDir, getTasksDir, validateTeamName } from './team-fs'
 
 function getConfigRoot(): string {
-  return path.join(os.homedir(), '.config', 'levelcode')
+  return path.join(getUserHomeDir(), '.config', 'levelcode')
 }
 
 function getTeamDir(teamName: string): string {

sdk/bunfig.toml

@@ -6,4 +6,4 @@ linkWorkspacePackages = true
 [test]
 # Exclude large test repositories from execution
 exclude = ["../evals/test-repos/**"]
-preload = ["./test/setup-env.ts", "../test/setup-bigquery-mocks.ts"]
+preload = ["../test/setup-isolated-home.ts", "./test/setup-env.ts", "../test/setup-bigquery-mocks.ts"]

test/setup-isolated-home.ts

@@ -0,0 +1,51 @@
+/**
+ * Global test preload: give every test a fresh, isolated home directory
+ * via LEVELCODE_HOME.
+ *
+ * Why: bun caches os.homedir() at first use, so swapping HOME/USERPROFILE
+ * per test does not move user-data paths after the first caller — tests
+ * leaked state into each other (and on CI, into the runner's real home).
+ * Every user-data path in the monorepo goes through
+ * `getUserHomeDir()` (common/src/utils/home-dir.ts), which prefers
+ * LEVELCODE_HOME. Env reads are live, so this per-test override is reliable
+ * on every platform.
+ *
+ * Tests that want a specific home can set LEVELCODE_HOME themselves; the
+ * preload restores the previous value afterwards. Temp dirs are cleaned up.
+ */
+
+import { beforeEach, afterEach } from 'bun:test'
+import * as fs from 'fs'
+import * as os from 'os'
+import * as path from 'path'
+
+let currentHome: string | null = null
+let savedValue: string | undefined
+
+beforeEach(() => {
+  savedValue = process.env.LEVELCODE_HOME
+  try {
+    currentHome = fs.mkdtempSync(path.join(os.tmpdir(), 'levelcode-home-'))
+  } catch {
+    currentHome = null // Fall back to whatever homedir resolves to
+  }
+  if (currentHome) {
+    process.env.LEVELCODE_HOME = currentHome
+  }
+})
+
+afterEach(() => {
+  if (savedValue !== undefined) {
+    process.env.LEVELCODE_HOME = savedValue
+  } else {
+    delete process.env.LEVELCODE_HOME
+  }
+  if (currentHome) {
+    try {
+      fs.rmSync(currentHome, { recursive: true, force: true })
+    } catch {
+      // Best-effort cleanup
+    }
+    currentHome = null
+  }
+})