fix: resolve dotenv loading in examples and tests

[shamsharoon]

why

Examples were failing with "OpenAI API key is missing" errors despite having OPENAI_API_KEY in the .env file. The issue was that dotenv.config() in stagehand.config.ts was looking for .env in the current working directory (process.cwd()), which pointed to examples/ when running via pnpm run example, not the project root where .env actually exists.

Related Issue: dotenv peerDependency

what changed

• stagehand.config.ts: Updated dotenv.config() to use path.resolve(__dirname, ".env") instead of relying on process.cwd(). This ensures .env is loaded relative to the config file's location, working correctly whether running from source or compiled code.

• examples/2048.ts: Updated to import and use StagehandConfig instead of hardcoding configuration, aligning with other examples in the repo.

• evals/deterministic/tests/Errors/apiKeyError.test.ts: Fixed tests that verify error handling when API keys are missing:

  • Added beforeAll/afterAll hooks to temporarily clear and restore OPENAI_API_KEY
  • Added config overrides (llmClient: undefined, modelClientOptions: undefined) to properly test error conditions
  • Ensures tests don't interfere with other test files

test plan

• ✅ Ran pnpm run example 2048 - example executes successfully without API key errors
• ✅ Ran pnpm run e2e:local - all 43 tests pass
• ✅ Specifically tested apiKeyError.test.ts - all 3 tests pass and correctly throw expected errors
• ✅ Verified afterAll hook restores environment for subsequent test files

[changeset-bot]

⚠️ No Changeset found

Latest commit: a2ad8d0

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[greptile-apps]

Greptile Overview

Summary

Fixed environment variable loading in examples and tests by updating stagehand.config.ts to use path.resolve(__dirname, ".env") instead of relying on process.cwd(), ensuring .env is loaded correctly regardless of execution context.

Key Changes:

• stagehand.config.ts: Changed dotenv.config() to use __dirname-based path resolution for reliable .env loading
• examples/2048.ts: Updated to import and use shared StagehandConfig for consistency
• evals/deterministic/tests/Errors/apiKeyError.test.ts: Added beforeAll/afterAll hooks and explicit config overrides (llmClient: undefined, modelClientOptions: undefined) to properly isolate tests and verify error handling

Impact:
Examples now run successfully from any directory without "OpenAI API key is missing" errors, and API key error tests are properly isolated and won't interfere with other test files.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The changes are well-tested (43 tests passing), address a clear bug with a straightforward fix, and improve test isolation without introducing new dependencies or complex logic
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
stagehand.config.ts	5/5	Fixed dotenv path resolution to use __dirname instead of process.cwd() for reliable .env loading
examples/2048.ts	5/5	Updated to use shared StagehandConfig for consistency with other examples
evals/deterministic/tests/Errors/apiKeyError.test.ts	5/5	Added proper test isolation with beforeAll/afterAll hooks and explicit config overrides to test error conditions

Sequence Diagram

sequenceDiagram
    participant User
    participant ExampleScript as examples/2048.ts
    participant Config as stagehand.config.ts
    participant Dotenv as dotenv
    participant FS as File System
    
    User->>ExampleScript: pnpm run example 2048
    Note over User,ExampleScript: CWD = examples/
    
    ExampleScript->>Config: import StagehandConfig
    activate Config
    
    Config->>Dotenv: dotenv.config({ path: path.resolve(__dirname, ".env") })
    Note over Config,Dotenv: __dirname points to project root<br/>(where config file is located)
    
    Dotenv->>FS: Read .env from project root
    FS-->>Dotenv: Environment variables loaded
    
    Dotenv-->>Config: Success
    Config-->>ExampleScript: Config with OPENAI_API_KEY
    deactivate Config
    
    ExampleScript->>ExampleScript: new Stagehand({ ...StagehandConfig })
    Note over ExampleScript: API key properly loaded

Loading

_{3 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}