changing config for build

Author: Kylejeong2

config.d.ts

@@ -2,6 +2,14 @@ import type { Cookie } from "playwright-core";
 import type { AvailableModelSchema } from "@browserbasehq/stagehand";
 
 export type Config = {
+  /**
+   * Browserbase API Key to authenticate requests
+   */
+  browserbaseApiKey: string;
+  /**
+   * Browserbase Project ID associated with the API key
+   */
+  browserbaseProjectId: string;
   /**
    * Whether or not to use Browserbase proxies
    * https://docs.browserbase.com/features/proxies

package.json

@@ -40,7 +40,6 @@
   "dependencies": {
     "@browserbasehq/stagehand": "^2.4.0",
     "@modelcontextprotocol/sdk": "^1.13.1",
-    "@playwright/test": "^1.49.0",
     "@smithery/cli": "^1.2.15",
     "commander": "^14.0.0",
     "dotenv": "^16.4.6",

pnpm-lock.yaml

@@ -6,8 +6,6 @@ export type ToolCapability = "core" | string;
 
 // Define Command Line Options Structure
 export type CLIOptions = {
-  browserbaseApiKey?: string;
-  browserbaseProjectId?: string;
   proxies?: boolean;
   advancedStealth?: boolean;
   contextId?: string;
@@ -23,6 +21,8 @@ export type CLIOptions = {
 
 // Default Configuration Values
 const defaultConfig: Config = {
+  browserbaseApiKey: process.env.BROWSERBASE_API_KEY ?? "",
+  browserbaseProjectId: process.env.BROWSERBASE_PROJECT_ID ?? "",
   proxies: false,
   server: {
     port: undefined,
@@ -50,18 +50,19 @@ export async function resolveConfig(cliOptions: CLIOptions): Promise<Config> {
   // --------------------------------
 
   // Basic validation for Browserbase keys - provide dummy values if not set
-  if (!cliOptions.browserbaseApiKey) {
+  if (!mergedConfig.browserbaseApiKey) {
     console.warn(
       "Warning: BROWSERBASE_API_KEY environment variable not set. Using dummy value.",
     );
-    cliOptions.browserbaseApiKey = "dummy-browserbase-api-key";
+    mergedConfig.browserbaseApiKey = "dummy-browserbase-api-key";
   }
-  if (!cliOptions.browserbaseProjectId) {
+  if (!mergedConfig.browserbaseProjectId) {
     console.warn(
       "Warning: BROWSERBASE_PROJECT_ID environment variable not set. Using dummy value.",
     );
-    cliOptions.browserbaseProjectId = "dummy-browserbase-project-id";
+    mergedConfig.browserbaseProjectId = "dummy-browserbase-project-id";
   }
+
   if (!mergedConfig.modelApiKey) {
     console.warn(
       "Warning: MODEL_API_KEY environment variable not set. Using dummy value.",
@@ -77,6 +78,8 @@ export async function configFromCLIOptions(
   cliOptions: CLIOptions,
 ): Promise<Config> {
   return {
+    browserbaseApiKey: process.env.BROWSERBASE_API_KEY ?? "",
+    browserbaseProjectId: process.env.BROWSERBASE_PROJECT_ID ?? "",
     server: {
       port: cliOptions.port,
       host: cliOptions.host,

src/config.ts

@@ -69,10 +69,7 @@ program
     const config = await resolveConfig(options);
     const serverList = new ServerList(async () =>
       createServerFunction({
-        config: config as Required<
-          Pick<typeof config, "browserbaseApiKey" | "browserbaseProjectId">
-        > &
-          typeof config,
+        config: config,
       }),
     );
     setupExitWatchdog(serverList);