Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 33 additions & 0 deletions src/__tests__/extension.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,11 @@ vi.mock("@dotenvx/dotenvx", () => ({
config: vi.fn(),
}))

// Mock fs so the extension module can safely check for optional .env.
vi.mock("fs", () => ({
existsSync: vi.fn().mockReturnValue(false),
}))

const mockBridgeOrchestratorDisconnect = vi.fn().mockResolvedValue(undefined)

const mockCloudServiceInstance = {
Expand Down Expand Up @@ -238,6 +243,34 @@ describe("extension.ts", () => {
authStateChangedHandler = undefined
})

test("does not call dotenvx.config when optional .env does not exist", async () => {
vi.resetModules()

const fs = await import("fs")
vi.mocked(fs.existsSync).mockReturnValue(false)

const dotenvx = await import("@dotenvx/dotenvx")

const { activate } = await import("../extension")
await activate(mockContext)

expect(dotenvx.config).not.toHaveBeenCalled()
})

test("calls dotenvx.config when optional .env exists", async () => {
vi.resetModules()

const fs = await import("fs")
vi.mocked(fs.existsSync).mockReturnValue(true)

const dotenvx = await import("@dotenvx/dotenvx")

const { activate } = await import("../extension")
await activate(mockContext)

expect(dotenvx.config).toHaveBeenCalledTimes(1)
})

test("authStateChangedHandler calls BridgeOrchestrator.disconnect when logged-out event fires", async () => {
const { CloudService, BridgeOrchestrator } = await import("@roo-code/cloud")

Expand Down
14 changes: 9 additions & 5 deletions src/extension.ts
Original file line number Diff line number Diff line change
@@ -1,15 +1,19 @@
import * as vscode from "vscode"
import * as dotenvx from "@dotenvx/dotenvx"
import * as fs from "fs"
import * as path from "path"

// Load environment variables from .env file
try {
// Specify path to .env file in the project root directory
// The extension-level .env is optional (not shipped in production builds).
// Avoid calling dotenvx when the file doesn't exist, otherwise dotenvx emits
// a noisy [MISSING_ENV_FILE] error to the extension host console.
const envPath = path.join(__dirname, "..", ".env")
dotenvx.config({ path: envPath })
} catch (e) {
// Silently handle environment loading errors
console.warn("Failed to load environment variables:", e)
if (fs.existsSync(envPath)) {
dotenvx.config({ path: envPath })
}
} catch {
// Best-effort only: never fail extension activation due to optional env loading.
}

import type { CloudUserInfo, AuthState } from "@roo-code/types"
Expand Down
Loading