feat(eve): productionize Browserbase extension 0.2 - #2821
Conversation
🦋 Changeset detectedLatest commit: 87a393b The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
||
| export default defineAgent({ | ||
| model: openai(process.env.EVE_STAGEHAND_MODEL ?? "gpt-5.6-luna"), | ||
| model: openai(process.env.EVE_STAGEHAND_MODEL ?? "gpt-5.4-mini"), |
There was a problem hiding this comment.
any reason this was changed to 5.4 mini? would prefer 5.6 luna
There was a problem hiding this comment.
Addressed in aacd83be9 — restored the example default to gpt-5.6-luna while retaining the environment override.
There was a problem hiding this comment.
Follow-up: the private example workspace has now been removed entirely in 8441da9, so this package no longer owns an Eve agent or model default.
Signed-off-by: Shrey Pandya <shrey@browserbase.com>
Addresses feedback from @shrey150 Signed-off-by: Shrey Pandya <shrey@browserbase.com>
Signed-off-by: Shrey Pandya <shrey@browserbase.com>
Signed-off-by: Shrey Pandya <shrey@browserbase.com>
edca842 to
65a7123
Compare
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Signed-off-by: Shrey Pandya <shrey@browserbase.com>
…t/productionize-eve-0-2 Signed-off-by: Shrey Pandya <shrey@browserbase.com>
There was a problem hiding this comment.
2 issues found and verified against the latest diff
Confidence score: 4/5
- In
scripts/release/consolidate-changelogs.ts, Eve is now included inpackageChangelogs, but the release tests do not cover its package path or label, so Eve-generated changelogs could be silently omitted or mishandled—add a focused consolidation test. - In
packages/integrations/eve/extension/tools/snapshot.ts,snapshotTool.executelacks focused valid- and rejected-input coverage, leaving delegated-result regressions undetected—add tests for both execution outcomes.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="packages/integrations/eve/extension/tools/snapshot.ts">
<violation number="1" location="packages/integrations/eve/extension/tools/snapshot.ts:13">
P3: The new snapshot execution path has no focused test; the current suite only checks its metadata. Add valid-input and rejected-input tests that exercise `snapshotTool.execute` and verify the delegated result.
(Based on your team's feedback about adding unit tests for new behavior.)</violation>
</file>
<file name="scripts/release/consolidate-changelogs.ts">
<violation number="1" location="scripts/release/consolidate-changelogs.ts:30">
P2: Adding Eve to `packageChangelogs` changes the release consolidation set, but the release tests never exercise this package path or label. Add a focused test for an Eve-generated changelog so a future path or label regression cannot omit Eve release notes from the root changelog.
(Based on your team's feedback about tests for new behavior.) .</violation>
</file>
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
| }, | ||
| { | ||
| label: "Eve Extension", | ||
| path: path.join(repositoryRoot, "packages/integrations/eve/CHANGELOG.md"), |
There was a problem hiding this comment.
P2: Adding Eve to packageChangelogs changes the release consolidation set, but the release tests never exercise this package path or label. Add a focused test for an Eve-generated changelog so a future path or label regression cannot omit Eve release notes from the root changelog.
(Based on your team's feedback about tests for new behavior.) .
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At scripts/release/consolidate-changelogs.ts, line 30:
<comment>Adding Eve to `packageChangelogs` changes the release consolidation set, but the release tests never exercise this package path or label. Add a focused test for an Eve-generated changelog so a future path or label regression cannot omit Eve release notes from the root changelog.
(Based on your team's feedback about tests for new behavior.) .</comment>
<file context>
@@ -25,6 +25,10 @@ const packageChangelogs = [
},
+ {
+ label: "Eve Extension",
+ path: path.join(repositoryRoot, "packages/integrations/eve/CHANGELOG.md"),
+ },
];
</file context>
There was a problem hiding this comment.
Addressed in 4016cf563: exported the consolidation source list and added a focused test for the exact Eve changelog path, label, and generated root heading.
| export default defineTool({ | ||
| description: SNAPSHOT_TOOL_DESCRIPTION, | ||
| inputSchema: SNAPSHOT_INPUT_SCHEMA, | ||
| async execute(rawInput) { |
There was a problem hiding this comment.
P3: The new snapshot execution path has no focused test; the current suite only checks its metadata. Add valid-input and rejected-input tests that exercise snapshotTool.execute and verify the delegated result.
(Based on your team's feedback about adding unit tests for new behavior.)
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/integrations/eve/extension/tools/snapshot.ts, line 13:
<comment>The new snapshot execution path has no focused test; the current suite only checks its metadata. Add valid-input and rejected-input tests that exercise `snapshotTool.execute` and verify the delegated result.
(Based on your team's feedback about adding unit tests for new behavior.) </comment>
<file context>
@@ -0,0 +1,17 @@
+export default defineTool({
+ description: SNAPSHOT_TOOL_DESCRIPTION,
+ inputSchema: SNAPSHOT_INPUT_SCHEMA,
+ async execute(rawInput) {
+ const input = SnapshotInputSchema.parse(rawInput);
+ return stagehandSession.run(({ tools }) => tools.snapshot(input));
</file context>
There was a problem hiding this comment.
Addressed in 4016cf563: added execution coverage proving valid snapshot input delegates with the parsed options and invalid input rejects before the session is opened.
…t/productionize-eve-0-2 Signed-off-by: Shrey Pandya <shrey@browserbase.com>
Signed-off-by: Shrey Pandya <shrey@browserbase.com>
Signed-off-by: Shrey Pandya <shrey@browserbase.com>
…t/productionize-eve-0-2 Signed-off-by: Shrey Pandya <shrey@browserbase.com>
…t/productionize-eve-0-2 Signed-off-by: Shrey Pandya <shrey@browserbase.com>
Summary
@browserbasehq/eveextension from this repositorybrowserbase__run,browserbase__snapshot, andbrowserbase__screenshotThis is stacked on #2824, which propagates browser-side
browser.close()requests to the host lifecycle owner. Version 0.2 replaces the legacy focused Browserbase tools with the Stagehand V4 Code Mode contract. Eve built-inweb_searchandweb_fetchremain unchanged; #2822 adds optional Browserbase overrides.Release setup
Before the first publish, configure npm trusted publishing for
@browserbasehq/evewith repositorybrowserbase/stagehandand workflow.github/workflows/release.yml. The included minor changeset publishes this as 0.2.0.After 0.2.0 is available, the duplicate package in
browserbase/integrationscan be retired and its pending 0.2 release should not be published.Validation
publintand has no private Core runtime dependencystagehand-extension.zipand exposes exactly the three namespaced Code Mode toolsbrowser.close()all passedCOMPLETED, with zero sessions left running