Skip to content

Remove CLI binary from sdk-functions package#37

Open
shrey150 wants to merge 1 commit intomainfrom
shrey/remove-cli-binary
Open

Remove CLI binary from sdk-functions package#37
shrey150 wants to merge 1 commit intomainfrom
shrey/remove-cli-binary

Conversation

@shrey150
Copy link

Summary

  • Removed the bin field from package.json so this package no longer registers a bb binary
  • Removed CLI-only dependencies: commander, chalk, dotenv, archiver, tsx, @types/archiver
  • Removed the CLI build config from tsup.config.ts (the second entry that built src/cli/main.ts and copied templates)
  • Kept all runtime library dependencies (@browserbasehq/sdk, zod) — verified these are the only ones imported by non-CLI source files

The CLI functionality (bb init, bb dev, bb publish) has been moved to @browserbasehq/cli which uses subcommands (bb functions init, etc). Having both packages register a bb binary causes conflicts on install.

The src/cli/ directory is intentionally left in place — source cleanup will follow in a separate PR.

Test plan

  • pnpm install succeeds with reduced dependency set
  • pnpm run build succeeds — produces dist/index.js, dist/index.cjs, and type declarations
  • Verify npm pack / pnpm pack output no longer includes cli.js
  • Confirm installing this package no longer creates a bb binary in node_modules/.bin

🤖 Generated with Claude Code

The CLI has been moved to @browserbasehq/cli. This package should
only ship the runtime library (defineFn, etc). Having both packages
register a 'bb' binary causes conflicts.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant