Better server build

[shadowfax92]

No description provided.

[greptile-apps]

Greptile Overview

Greptile Summary

Refactored server build process with a new centralized build script that supports multiple targets and environment modes.

• Created scripts/build_server.ts to consolidate all platform-specific build commands
• Added prod/dev mode distinction with clean environment handling for production builds
• Improved codex binary path resolution with explicit executability validation
• Changed development environment file from .env to .env.dev
• Added dotenv dependency for explicit environment file parsing (note: CLAUDE.md states Bun auto-loads .env, but this is needed for custom env file names)

The build script provides better control over environment variables during compilation, which is important for creating reproducible production builds.

Confidence Score: 5/5

• Safe to merge - well-structured refactoring with improved build process
• The changes are primarily infrastructure improvements with no risky logic changes. The new build script is well-documented, handles errors appropriately, and improves the build process. The codex binary resolution improvements add robustness. The only consideration is ensuring .env.dev and .env.prod files exist in deployment environments, but this is a configuration concern rather than a code issue.
• Verify .env.dev and .env.prod files exist in all environments where builds occur

Important Files Changed

File Analysis

Filename	Score	Overview
bun.lock	5/5	Added dotenv@17.2.3 dependency for explicit environment file parsing
package.json	5/5	Version bump to 0.0.8, refactored build scripts to use build_server.ts, changed env file from .env to .env.dev for development
packages/agent/src/agent/CodexSDKAgent.ts	5/5	Improved codex binary resolution with explicit executability checks and better error handling for invalid paths
scripts/build_server.ts	4/5	New build script with clean environment handling for prod builds and support for multiple targets; requires .env.prod and .env.dev files

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Script as build_server.ts
    participant Env as .env.dev/.env.prod
    participant Bun as Bun Build
    participant Patch as patch-windows-exe.ts
    participant Output as dist/server/

    Dev->>Script: bun scripts/build_server.ts --mode=prod --target=all
    Script->>Script: parseArgs() - validate mode and targets
    Script->>Env: loadEnvFile() - read .env.prod or .env.dev
    Env-->>Script: environment variables
    
    alt Production Mode
        Script->>Script: createCleanEnv() - minimal system vars + env file
    else Development Mode
        Script->>Script: merge shell env + env file
    end
    
    loop For each target
        Script->>Bun: bun build --compile with target config
        Bun-->>Output: compiled binary
        
        alt Windows target
            Script->>Patch: patch-windows-exe.ts
            Patch-->>Output: patched .exe
        end
    end
    
    Script-->>Dev: All builds completed

Loading

_{3 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}