Skip to content

feat(browser): warn that firecrawl browser is deprecated - #374

Open
mogery wants to merge 2 commits into
mainfrom
capy/cli-browser-deprecation-warning
Open

mogery wants to merge 2 commits into
mainfrom
capy/cli-browser-deprecation-warning

Conversation

@mogery

@mogery mogery commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

firecrawl browser has been hidden from help and documented as replaced by firecrawl scrape plus firecrawl interact, but running it gave no sign of that. This makes the deprecation visible before the command is removed.

  • Every firecrawl browser invocation, including launch-session, execute, list and close, now prints one line to stderr:

    Warning: `firecrawl browser` is deprecated and will be removed in a future release. Use `firecrawl scrape <url>`, then `firecrawl interact "<prompt>"` or `firecrawl interact -c "<code>"`. See `firecrawl interact --help`.
    
  • Stdout, --json output and exit codes are unchanged, so scripts that read the command's output keep working.

  • The warning is a preAction hook on the browser command, so Commander runs it for the command and every subcommand.

  • cli-argv.test.ts covers the shorthand and a subcommand against the built CLI.

No version bump; it ships with the next release.

Open in Capy


Summary by cubic

Every firecrawl browser invocation now prints a deprecation warning to stderr, pointing users to firecrawl scrape and firecrawl interact as replacements before the command is removed.

  • The warning runs for the browser command and all its subcommands via a preAction hook, and also appears in browser and browser subcommand help output.
  • Stdout, --json output, and exit codes are unchanged, so existing scripts keep working.

Written for commit acc3be9. Summary will update on new commits.

View guided diff Turn on auto-fix

Every firecrawl browser invocation, including its subcommands, now
prints one line on stderr naming firecrawl scrape and firecrawl
interact as the replacement. Stdout and exit codes are unchanged, so
scripts and --json output keep working.

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 2 files

Shadow auto-approve: would auto-approve with 2 open P3 issues. Adds a stderr deprecation warning for firecrawl browser and its subcommands via a preAction hook, leaving stdout, JSON output, and exit codes unchanged; tests cover both the command and a subcommand.

View guided diff | Turn on auto-fix | Re-trigger cubic

Comment thread src/index.ts Resolved
Comment thread src/__tests__/cli-argv.test.ts Resolved
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0 issues found across 2 files (changes from recent commits).

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Shadow auto-approve: would auto-approve. Adds a stderr deprecation warning and help banner to the deprecated firecrawl browser command without changing stdout, JSON output, or exit codes; tests cover the command, subcommands, and help output.

View guided diff | Turn on auto-fix | Re-trigger cubic

This branch has not been deployed

No deployments
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