update provider flag (#217)
* auto-release * fixes * fix secret name * update provider flag
This commit is contained in:
+10
-11
@@ -678,14 +678,16 @@ export class BrowserManager {
|
||||
|
||||
/**
|
||||
* Connect to Browserbase remote browser via CDP.
|
||||
* Returns true if connected, false if credentials not available.
|
||||
* Requires BROWSERBASE_API_KEY and BROWSERBASE_PROJECT_ID environment variables.
|
||||
*/
|
||||
private async connectToBrowserbase(): Promise<boolean> {
|
||||
private async connectToBrowserbase(): Promise<void> {
|
||||
const browserbaseApiKey = process.env.BROWSERBASE_API_KEY;
|
||||
const browserbaseProjectId = process.env.BROWSERBASE_PROJECT_ID;
|
||||
|
||||
if (!browserbaseApiKey || !browserbaseProjectId) {
|
||||
return false;
|
||||
throw new Error(
|
||||
'BROWSERBASE_API_KEY and BROWSERBASE_PROJECT_ID are required when using browserbase as a provider'
|
||||
);
|
||||
}
|
||||
|
||||
const response = await fetch('https://api.browserbase.com/v1/sessions', {
|
||||
@@ -727,8 +729,6 @@ export class BrowserManager {
|
||||
this.pages.push(page);
|
||||
this.activePageIndex = 0;
|
||||
this.setupPageTracking(page);
|
||||
|
||||
return true;
|
||||
} catch (error) {
|
||||
await this.closeBrowserbaseSession(session.id, browserbaseApiKey).catch((sessionError) => {
|
||||
console.error('Failed to close Browserbase session during cleanup:', sessionError);
|
||||
@@ -844,14 +844,13 @@ export class BrowserManager {
|
||||
return;
|
||||
}
|
||||
|
||||
// Try connecting to cloud browser providers if configured
|
||||
// Browserbase: auto-connects when BROWSERBASE_API_KEY and BROWSERBASE_PROJECT_ID are set
|
||||
if (await this.connectToBrowserbase()) {
|
||||
// Cloud browser providers require explicit opt-in via -p flag or AGENT_BROWSER_PROVIDER env var
|
||||
// -p flag takes precedence over env var
|
||||
const provider = options.provider ?? process.env.AGENT_BROWSER_PROVIDER;
|
||||
if (provider === 'browserbase') {
|
||||
await this.connectToBrowserbase();
|
||||
return;
|
||||
}
|
||||
|
||||
// Browser Use: requires explicit opt-in via -p browseruse flag or AGENT_BROWSER_PROVIDER=browseruse
|
||||
const provider = options.provider ?? process.env.AGENT_BROWSER_PROVIDER;
|
||||
if (provider === 'browseruse') {
|
||||
await this.connectToBrowserUse();
|
||||
return;
|
||||
|
||||
Reference in New Issue
Block a user