Migrate Cursor browse plugin to the browse CLI; remove functions#2
Open
justinemach wants to merge 1 commit into
Open
Migrate Cursor browse plugin to the browse CLI; remove functions#2justinemach wants to merge 1 commit into
browse CLI; remove functions#2justinemach wants to merge 1 commit into
Conversation
- Repoint plugins/browse off the deprecated @browserbasehq/browse-cli@^0.1.4 onto the current `browse`@^0.8.4 (npm deprecation explicitly says migrate). Regenerated package-lock.json. Binary still resolves at node_modules/.bin/browse, so the MCP server + CLI wrapper are unchanged functionally. - Update stale `browse-cli` references in source comments + one error string. - Remove the `functions` plugin (per Ziray): drop plugins/functions, its marketplace entry, and the README row. - Bump versions: browse plugin 2.0.0 -> 2.1.0, marketplace 1.0.0 -> 1.1.0. MCP tools are untouched — existing Cursor installs keep working (additive). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit ca4ab57. Configure here.
| }, | ||
| "node_modules/browse": { | ||
| "version": "0.8.4", | ||
| "resolved": "https://socket-firewall.tail929132.ts.net/npm/browse/-/browse-0.8.4.tgz", |
There was a problem hiding this comment.
Lockfile uses private registry URLs
High Severity
Regenerated package-lock.json records many resolved tarball URLs on socket-firewall.tail929132.ts.net, including the browse dependency itself, instead of the public npm registry. npm ci in plugins/browse (including release npm ci) fetches those exact URLs, so installs fail anywhere that host is unreachable.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit ca4ab57. Configure here.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


What
Per Ziray — slight update + new release for the Cursor plugin.
browseCLI. The plugin pinned the deprecated@browserbasehq/browse-cli@^0.1.4(npm: "now published as 'browse'… please migrate"). Swapped tobrowse@^0.8.4and regeneratedpackage-lock.json. The binary still resolves atnode_modules/.bin/browse, sosrc/cli.tsandsrc/mcp-server.tsare functionally unchanged.functionsplugin — droppedplugins/functions/, its.cursor-plugin/marketplace.jsonentry, and the README row.browse-cli→browsein source comments + one error string.2.0.0 → 2.1.0, marketplace1.0.0 → 1.1.0.Additive / non-breaking
MCP tools (
browser_navigate,browser_click, …) are untouched, so existing Cursor installs keep working. This only updates the engine underneath + removesfunctions.Validation
node scripts/validate-template.mjs→ passespackage-lock.jsonregenerated; 0 remaining refs to the old packageRelease
After merge: tag
v2.1.0to triggerrelease.yml(build → prune dev deps → validate → tarball).Notes for review
functionsis subtractive — anyone who installed the Cursorfunctionsplugin loses it on update. Ziray approved; flagging for release notes.browsesurface (sessions, network, tabs, cloud) — kept this a version bump. Can follow up if we want tool parity.🤖 Generated with Claude Code
Note
Medium Risk
Swapping the automation engine and pulling a much larger dependency tree could affect install size and runtime behavior; removing the functions plugin is a breaking change for anyone who relied on it.
Overview
Migrates the Cursor marketplace to the current
browsepackage and drops the separate cloud-functions plugin.The browse plugin dependency moves from deprecated
@browserbasehq/browse-clitobrowse@^0.8.4(package.json/ lockfile). ThebrowserCLI and MCP server still invokenode_modules/.bin/browse; only comments and install error text now say browse instead of browse-cli. Browse plugin is 2.1.0; marketplace metadata is 1.1.0.The
functionsplugin is removed entirely (plugins/functions/, marketplace entry, README table row). The marketplace now ships only browse.MCP tool surface is unchanged in this PR; lockfile growth reflects the new
browsedependency tree (e.g. Stagehand / AI-related packages).Reviewed by Cursor Bugbot for commit ca4ab57. Bugbot is set up for automated code reviews on this repo. Configure here.