chore: bump deps and improve test harness

Author: brandonkachen

bun.lock

@@ -7,4 +7,4 @@ linkWorkspacePackages = true
 [test]
 # Exclude test repositories from test execution to prevent timeouts
 exclude = ["evals/test-repos/**"]
-preload = ["./sdk/test/setup-env.ts"]
+preload = ["./sdk/test/setup-env.ts", "./test/setup-bigquery-mocks.ts"]

bunfig.toml

@@ -33,13 +33,13 @@
     "@codebuff/sdk": "workspace:*",
     "@opentui/core": "^0.1.63",
     "@opentui/react": "^0.1.63",
-    "@tanstack/react-query": "^5.62.8",
+    "@tanstack/react-query": "^5.90.12",
     "commander": "^14.0.1",
     "immer": "^10.1.3",
     "jimp": "^1.6.0",
     "open": "^10.1.0",
     "pino": "9.4.0",
-    "posthog-node": "4.17.2",
+    "posthog-node": "^5.8.0",
     "react": "^19.0.0",
     "react-reconciler": "^0.32.0",
     "remark-breaks": "^4.0.0",
@@ -50,7 +50,7 @@
     "ts-pattern": "^5.9.0",
     "unified": "^11.0.0",
     "yoga-layout": "^3.2.1",
-    "zod": "^4.0.0",
+    "zod": "^4.2.1",
     "zustand": "^5.0.8"
   },
   "devDependencies": {

cli/package.json

@@ -37,12 +37,68 @@ export const sleep = (ms: number) =>
 
 let cachedEnv: Record<string, string> | null = null
 
+const TEST_CLIENT_ENV_DEFAULTS: Record<string, string> = {
+  NEXT_PUBLIC_CB_ENVIRONMENT: 'test',
+  NEXT_PUBLIC_CODEBUFF_APP_URL: 'http://localhost:3000',
+  NEXT_PUBLIC_SUPPORT_EMAIL: 'support@codebuff.com',
+  NEXT_PUBLIC_POSTHOG_API_KEY: 'test-posthog-key',
+  NEXT_PUBLIC_POSTHOG_HOST_URL: 'https://us.i.posthog.com',
+  NEXT_PUBLIC_STRIPE_PUBLISHABLE_KEY: 'pk_test_placeholder',
+  NEXT_PUBLIC_STRIPE_CUSTOMER_PORTAL:
+    'https://billing.stripe.com/p/login/test_placeholder',
+  NEXT_PUBLIC_GOOGLE_SITE_VERIFICATION_ID: 'test-verification',
+  NEXT_PUBLIC_WEB_PORT: '3000',
+}
+const TEST_SERVER_ENV_DEFAULTS: Record<string, string> = {
+  OPEN_ROUTER_API_KEY: 'test',
+  OPENAI_API_KEY: 'test',
+  LINKUP_API_KEY: 'test',
+  PORT: '4242',
+  DATABASE_URL: 'postgres://user:pass@localhost:5432/db',
+  CODEBUFF_GITHUB_ID: 'test-id',
+  CODEBUFF_GITHUB_SECRET: 'test-secret',
+  NEXTAUTH_SECRET: 'test-secret',
+  STRIPE_SECRET_KEY: 'sk_test_dummy',
+  STRIPE_WEBHOOK_SECRET_KEY: 'whsec_dummy',
+  STRIPE_USAGE_PRICE_ID: 'price_test',
+  STRIPE_TEAM_FEE_PRICE_ID: 'price_test',
+  LOOPS_API_KEY: 'test',
+  DISCORD_PUBLIC_KEY: 'test',
+  DISCORD_BOT_TOKEN: 'test',
+  DISCORD_APPLICATION_ID: 'test',
+}
+
+function ensureCliEnvDefaults(): void {
+  if (!process.env.NODE_ENV) {
+    process.env.NODE_ENV = 'test'
+  }
+  if (!process.env.BUN_ENV) {
+    process.env.BUN_ENV = 'test'
+  }
+  if (process.env.CI !== 'true' && process.env.CI !== '1') {
+    process.env.CI = 'true'
+  }
+
+  for (const [key, value] of Object.entries(TEST_CLIENT_ENV_DEFAULTS)) {
+    if (!process.env[key]) {
+      process.env[key] = value
+    }
+  }
+
+  for (const [key, value] of Object.entries(TEST_SERVER_ENV_DEFAULTS)) {
+    if (!process.env[key]) {
+      process.env[key] = value
+    }
+  }
+}
+
 function loadCliEnv(): Record<string, string> {
   if (cachedEnv) {
     return cachedEnv
   }
 
   try {
+    ensureCliEnvDefaults()
     // NOTE: Inline require() is used for lazy loading - the env module depends on
     // Infisical secrets which may not be available at module load time in test environments
     const { env } = require('../../../packages/internal/src/env') as {

cli/src/__tests__/test-utils.ts

@@ -1,6 +1,7 @@
 import { describe, test, expect, afterEach } from 'bun:test'
 
-import { getCliEnv, createTestCliEnv } from '../../utils/env'
+import { getCliEnv } from '../../utils/env'
+import { createTestCliEnv } from '../../testing/env'
 
 describe('cli/utils/env', () => {
   describe('getCliEnv', () => {

cli/src/__tests__/utils/env.test.ts

@@ -1,25 +1,33 @@
+import React from 'react'
 import { describe, test, expect, beforeEach, afterEach, mock } from 'bun:test'
 
 import { useTimeout } from '../use-timeout'
 
 /**
  * Tests for useTimeout hook
  *
- * NOTE: These tests are currently skipped due to React 19 + Bun compatibility issues.
- * The renderHook utility from React Testing Library returns null results in this environment.
- * See cli/knowledge.md "React Testing Library + React 19 + Bun Incompatibility" section.
- *
- * The hook implementation follows the spec exactly and will be tested through integration
- * tests when used in actual components (e.g., in Part 5 when used for reconnection messages).
+ * NOTE: Tests install a minimal React dispatcher so hooks can run without a renderer.
  */
 
 describe('useTimeout', () => {
+  const reactInternals = (React as any)
+    .__CLIENT_INTERNALS_DO_NOT_USE_OR_WARN_USERS_THEY_CANNOT_UPGRADE
   let originalSetTimeout: typeof setTimeout
   let originalClearTimeout: typeof clearTimeout
   let timers: { id: number; ms: number; fn: Function; cleared: boolean }[]
   let nextId: number
+  let originalDispatcher: any
 
   beforeEach(() => {
+    originalDispatcher = reactInternals.H
+    reactInternals.H = {
+      useRef: (value: any) => ({ current: value }),
+      useCallback: (callback: any) => callback,
+      useEffect: (effect: any) => {
+        effect()
+      },
+    }
+
     timers = []
     nextId = 1
     originalSetTimeout = globalThis.setTimeout
@@ -40,11 +48,12 @@ describe('useTimeout', () => {
   })
 
   afterEach(() => {
+    reactInternals.H = originalDispatcher
     globalThis.setTimeout = originalSetTimeout
     globalThis.clearTimeout = originalClearTimeout
   })
 
-  test.skip('setTimeout schedules a timeout with correct delay', () => {
+  test('setTimeout schedules a timeout with correct delay', () => {
     const { setTimeout } = useTimeout()
     const callback = mock(() => {})
 
@@ -55,7 +64,7 @@ describe('useTimeout', () => {
     expect(timers[0].cleared).toBe(false)
   })
 
-  test.skip('timeout callback executes when invoked', () => {
+  test('timeout callback executes when invoked', () => {
     const { setTimeout } = useTimeout()
     const callback = mock(() => {})
 
@@ -69,7 +78,7 @@ describe('useTimeout', () => {
     expect(callback).toHaveBeenCalledTimes(1)
   })
 
-  test.skip('clearTimeout marks the timeout as cleared', () => {
+  test('clearTimeout marks the timeout as cleared', () => {
     const { setTimeout, clearTimeout } = useTimeout()
     const callback = mock(() => {})
 
@@ -80,7 +89,7 @@ describe('useTimeout', () => {
     expect(timers[0].cleared).toBe(true)
   })
 
-  test.skip('clearTimeout prevents callback from being used', () => {
+  test('clearTimeout prevents callback from being used', () => {
     const { setTimeout, clearTimeout } = useTimeout()
     const callback = mock(() => {})
 
@@ -92,7 +101,7 @@ describe('useTimeout', () => {
     expect(timers[0].cleared).toBe(true)
   })
 
-  test.skip('replacing timeout with same key clears the previous one', () => {
+  test('replacing timeout with same key clears the previous one', () => {
     const { setTimeout } = useTimeout()
     const callback1 = mock(() => {})
     const callback2 = mock(() => {})
@@ -108,7 +117,7 @@ describe('useTimeout', () => {
     expect(timers[1].ms).toBe(2000)
   })
 
-  test.skip('clearTimeout when no timeout is active does nothing', () => {
+  test('clearTimeout when no timeout is active does nothing', () => {
     const { clearTimeout } = useTimeout()
 
     // Should not throw
@@ -117,7 +126,7 @@ describe('useTimeout', () => {
     expect(timers.length).toBe(0)
   })
 
-  test.skip('multiple setTimeout calls with different keys work independently', () => {
+  test('multiple setTimeout calls with different keys work independently', () => {
     const { setTimeout } = useTimeout()
     const callbacks = [mock(() => {}), mock(() => {}), mock(() => {})]
 
@@ -131,7 +140,7 @@ describe('useTimeout', () => {
     expect(timers[2].cleared).toBe(false)
   })
 
-  test.skip('setTimeout after clearTimeout works correctly', () => {
+  test('setTimeout after clearTimeout works correctly', () => {
     const { setTimeout, clearTimeout } = useTimeout()
     const callback1 = mock(() => {})
     const callback2 = mock(() => {})
@@ -145,7 +154,7 @@ describe('useTimeout', () => {
     expect(timers[1].cleared).toBe(false)
   })
 
-  test.skip('hook returns stable setTimeout and clearTimeout functions', () => {
+  test('hook returns distinct setTimeout and clearTimeout functions', () => {
     const result1 = useTimeout()
     const result2 = useTimeout()
 
@@ -154,7 +163,7 @@ describe('useTimeout', () => {
     expect(result1.clearTimeout).not.toBe(result2.clearTimeout)
   })
 
-  test.skip('clearTimeout without key clears all timeouts', () => {
+  test('clearTimeout without key clears all timeouts', () => {
     const { setTimeout, clearTimeout } = useTimeout()
     const callbacks = [mock(() => {}), mock(() => {}), mock(() => {})]
 
@@ -174,7 +183,7 @@ describe('useTimeout', () => {
     expect(timers[2].cleared).toBe(true)
   })
 
-  test.skip('clearTimeout with specific key only clears that timeout', () => {
+  test('clearTimeout with specific key only clears that timeout', () => {
     const { setTimeout, clearTimeout } = useTimeout()
     const callbacks = [mock(() => {}), mock(() => {}), mock(() => {})]
 
@@ -189,7 +198,7 @@ describe('useTimeout', () => {
     expect(timers[2].cleared).toBe(false)
   })
 
-  test.skip('timeout auto-cleans up after execution', () => {
+  test('timeout auto-cleans up after execution', () => {
     const { setTimeout } = useTimeout()
     const callback = mock(() => {})
 
@@ -203,7 +212,7 @@ describe('useTimeout', () => {
     // but the implementation removes the key from the Map after execution
   })
 
-  test.skip('can reuse same key after timeout executes', () => {
+  test('can reuse same key after timeout executes', () => {
     const { setTimeout } = useTimeout()
     const callback1 = mock(() => {})
     const callback2 = mock(() => {})
@@ -222,7 +231,7 @@ describe('useTimeout', () => {
     expect(timers[1].cleared).toBe(false)
   })
 
-  test.skip('multiple timeouts can execute independently', () => {
+  test('multiple timeouts can execute independently', () => {
     const { setTimeout } = useTimeout()
     const callback1 = mock(() => {})
     const callback2 = mock(() => {})
@@ -242,7 +251,7 @@ describe('useTimeout', () => {
     expect(callback3).toHaveBeenCalledTimes(1)
   })
 
-  test.skip('replacing timeout before execution prevents old callback', () => {
+  test('replacing timeout before execution prevents old callback', () => {
     const { setTimeout } = useTimeout()
     const oldCallback = mock(() => {})
     const newCallback = mock(() => {})
@@ -261,7 +270,7 @@ describe('useTimeout', () => {
     expect(newCallback).toHaveBeenCalledTimes(1)
   })
 
-  test.skip('clearTimeout on executed timeout does nothing', () => {
+  test('clearTimeout on executed timeout does nothing', () => {
     const { setTimeout, clearTimeout } = useTimeout()
     const callback = mock(() => {})
 
@@ -276,7 +285,7 @@ describe('useTimeout', () => {
     expect(timers.length).toBe(1)
   })
 
-  test.skip('mixing set and clear operations maintains correct state', () => {
+  test('mixing set and clear operations maintains correct state', () => {
     const { setTimeout, clearTimeout } = useTimeout()
 
     setTimeout(