[STG-1692] refactor: rewrite cookie-sync skill to use bb sync CLI#58
Open
[STG-1692] refactor: rewrite cookie-sync skill to use bb sync CLI#58
Conversation
Replaces the standalone Node.js script with the new `bb sync` command from @browserbasehq/cli, eliminating the need for npm install and direct Stagehand/SDK dependencies in the skill. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
shrey150
changed the title
Contributor
Author
|
Companion CLI PR with the |
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
1 participant
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.
Summary
cookie-sync.mjsscript (308 lines + Stagehand/SDK dependencies) with the newbb synccommand from@browserbasehq/clipackage.json,package-lock.json, andscripts/directory — no morenpm installneededbb syncflags and workflowsWhy
The
bb synccommand was added to the CLI (@browserbasehq/cli) and does everything the standalone script did (local Chrome CDP discovery, cookie export, context creation, cookie injection) but as a first-class CLI command. This means:bbinstallednode_modulesor dependency managementbbcommandsTest plan
bb sync --domains github.com— exported 6,153 cookies, filtered to 15, injected into contextbb sync --domains google.com,github.com— filtered to 119 cookies, injected into contextjq🤖 Generated with Claude Code
Note
Low Risk
Low risk because changes are limited to the
cookie-syncskill docs and removal of bundled Node script/dependencies; main risk is users now depend on@browserbasehq/cliavailability and itsbb syncbehavior/output.Overview
Replaces the
cookie-syncskill’s standalone Node/Stagehand implementation with the Browserbase CLI’sbb synccommand, updating all examples/workflows to usebb syncJSON output and flags (e.g.,--domains,--context-id,--stealth,--proxy).Removes the skill-local implementation and npm artifacts (
scripts/cookie-sync.mjs,package.json,package-lock.json), and refreshesSKILL.md/REFERENCE.mdtroubleshooting and setup guidance to assume a globalbbinstall.Written by Cursor Bugbot for commit 8adb29a. This will update automatically on new commits. Configure here.