fix: skip .agents integration tests when CODEBUFF_API_KEY is not set

- Use describe.skip pattern to skip entire test suites when API key is unavailable
- Remove inline API key checks that threw errors mid-test
- Tests will now skip gracefully in CI when CODEBUFF_API_KEY secret is not configured

Author: brandonkachen

.agents/__tests__/context-pruner.integration.test.ts

@@ -1,5 +1,5 @@
 import { API_KEY_ENV_VAR } from '@codebuff/common/old-constants'
-import { describe, expect, it } from 'bun:test'
+import { describe, expect, it, beforeAll } from 'bun:test'
 
 import {
   CodebuffClient,
@@ -11,12 +11,17 @@ import {
   type JSONValue,
 } from '@codebuff/sdk'
 
+const apiKey = process.env[API_KEY_ENV_VAR]
+const describeWithApiKey = apiKey ? describe : describe.skip
+
 /**
  * Integration tests for the context-pruner agent.
  * These tests verify that context-pruner correctly prunes message history
  * while maintaining tool-call/tool-result pair integrity for Anthropic API compliance.
+ * 
+ * These tests require CODEBUFF_API_KEY to be set and are skipped otherwise.
  */
-describe('Context Pruner Agent Integration', () => {
+describeWithApiKey('Context Pruner Agent Integration', () => {
   // Helper to create a text message
   const createMessage = (
     role: 'user' | 'assistant',
@@ -58,10 +63,6 @@ describe('Context Pruner Agent Integration', () => {
   it(
     'should prune large message history and maintain tool-call/tool-result pairs',
     async () => {
-      const apiKey = process.env[API_KEY_ENV_VAR]
-      if (!apiKey) {
-        throw new Error('API key not found')
-      }
 
       // Create a test agent that spawns context-pruner and then does one more step
       const testAgent: AgentDefinition = {
@@ -205,10 +206,6 @@ Do not do anything else. Just spawn context-pruner and then report the result.`,
   it(
     'should prune context with small token limit and preserve tool pairs',
     async () => {
-      const apiKey = process.env[API_KEY_ENV_VAR]
-      if (!apiKey) {
-        throw new Error('API key not found')
-      }
 
       // Create a test agent that spawns context-pruner with very aggressive pruning
       const testAgent: AgentDefinition = {

.agents/__tests__/editor-best-of-n.integration.test.ts

@@ -5,22 +5,23 @@ import { CodebuffClient } from '@codebuff/sdk'
 
 import type { PrintModeEvent } from '@codebuff/common/types/print-mode'
 
+const apiKey = process.env[API_KEY_ENV_VAR]
+const describeWithApiKey = apiKey ? describe : describe.skip
+
 /**
  * Integration tests for the editor-best-of-n-max agent.
  * These tests verify that the best-of-n editor workflow works correctly:
  * 1. Spawns multiple implementor agents in parallel
  * 2. Collects their implementation proposals
  * 3. Uses a selector agent to choose the best implementation
  * 4. Applies the chosen implementation
+ * 
+ * These tests require CODEBUFF_API_KEY to be set and are skipped otherwise.
  */
-describe('Editor Best-of-N Max Agent Integration', () => {
+describeWithApiKey('Editor Best-of-N Max Agent Integration', () => {
   it.skip(
     'should generate and select the best implementation for a simple edit',
     async () => {
-      const apiKey = process.env[API_KEY_ENV_VAR]
-      if (!apiKey) {
-        throw new Error('API key not found')
-      }
 
       // Create mock project files with a simple TypeScript file to edit
       const projectFiles: Record<string, string> = {

.agents/__tests__/file-explorer.integration.test.ts

@@ -7,6 +7,9 @@ import fileListerDefinition from '../file-explorer/file-lister'
 
 import type { PrintModeEvent } from '@codebuff/common/types/print-mode'
 
+const apiKey = process.env[API_KEY_ENV_VAR]
+const describeWithApiKey = apiKey ? describe : describe.skip
+
 /**
  * Integration tests for agents that use the read_subtree tool.
  * These tests verify that the SDK properly initializes the session state
@@ -17,15 +20,13 @@ import type { PrintModeEvent } from '@codebuff/common/types/print-mode'
  * - file-lister directly uses the read_subtree tool
  * - file-picker spawns file-lister as a subagent, adding complexity
  * - Testing file-lister directly verifies the core functionality
+ * 
+ * These tests require CODEBUFF_API_KEY to be set and are skipped otherwise.
  */
-describe('File Lister Agent Integration - read_subtree tool', () => {
+describeWithApiKey('File Lister Agent Integration - read_subtree tool', () => {
   it(
     'should find relevant files using read_subtree tool',
     async () => {
-      const apiKey = process.env[API_KEY_ENV_VAR]
-      if (!apiKey) {
-        throw new Error('API key not found')
-      }
 
       // Create mock project files that the file-lister should be able to find
       const projectFiles: Record<string, string> = {
@@ -142,10 +143,6 @@ export interface User {
   it(
     'should use the file tree from session state',
     async () => {
-      const apiKey = process.env[API_KEY_ENV_VAR]
-      if (!apiKey) {
-        throw new Error('API key not found')
-      }
 
       // Create a different set of project files with a specific structure
       const projectFiles: Record<string, string> = {
@@ -196,10 +193,6 @@ export interface User {
   it(
     'should respect directories parameter',
     async () => {
-      const apiKey = process.env[API_KEY_ENV_VAR]
-      if (!apiKey) {
-        throw new Error('API key not found')
-      }
 
       // Create project with multiple top-level directories
       const projectFiles: Record<string, string> = {
@@ -252,19 +245,17 @@ export interface User {
  * Integration tests for the file-picker agent that spawns subagents.
  * The file-picker spawns file-lister as a subagent to find files.
  * This tests the spawn_agents tool functionality through the SDK.
+ * 
+ * These tests require CODEBUFF_API_KEY to be set and are skipped otherwise.
  */
-describe('File Picker Agent Integration - spawn_agents tool', () => {
+describeWithApiKey('File Picker Agent Integration - spawn_agents tool', () => {
   // Note: This test requires the local agent definitions to be used for both
   // file-picker AND its spawned file-lister subagent. Currently, the spawned
   // agent may resolve to the server version which has the old parsing bug.
   // Skip until we have a way to ensure spawned agents use local definitions.
   it.skip(
     'should spawn file-lister subagent and find relevant files',
     async () => {
-      const apiKey = process.env[API_KEY_ENV_VAR]
-      if (!apiKey) {
-        throw new Error('API key not found')
-      }
 
       // Create mock project files
       const projectFiles: Record<string, string> = {