Skip to content

docs(skills): restore agent-browser prefix in interact bash examples - #195

Merged
hmishra2250 merged 1 commit into
firecrawl:mainfrom
chenxin-yan:skills/interact-agent-browser-prefix
Aug 18, 2026
Merged

docs(skills): restore agent-browser prefix in interact bash examples#195
hmishra2250 merged 1 commit into
firecrawl:mainfrom
chenxin-yan:skills/interact-agent-browser-prefix

Conversation

@chenxin-yan

@chenxin-yan chenxin-yan commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #193, addressing post-merge review feedback

interact sends --code literally to the API with no client-side prefixing (src/commands/interact.ts:82-98), and the server-side bash environment requires invoking agent-browser explicitly. #191/#193 correctly removed the nonexistent --language flag but wrongly dropped the agent-browser prefix — an over-generalization from browser execute's auto-prefix logic (src/index.ts:1851-1859), which never runs for interact.

Note for maintainers: the interact help text shows firecrawl interact -c "snapshot" --bash without the prefix (src/index.ts:~1993) — if the prefix is required, that example may warrant the same fix in a code PR.

interact sends --code literally to the API with no client-side
prefixing, and the server-side bash environment requires invoking
agent-browser explicitly (per maintainer review). The earlier fix
correctly removed the nonexistent --language flag but wrongly dropped
the prefix, over-generalizing from browser execute's auto-prefix
logic, which never runs for interact.

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 1 file

Re-trigger cubic

@hmishra2250
hmishra2250 self-requested a review August 18, 2026 21:07

@hmishra2250 hmishra2250 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.

LGTM!

@hmishra2250
hmishra2250 merged commit 253abde into firecrawl:main Aug 18, 2026
8 checks passed
@chenxin-yan
chenxin-yan deleted the skills/interact-agent-browser-prefix branch August 18, 2026 21:27
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.

2 participants