[waleedlatif1]

Summary

• Finish Firecrawl v2 response handling while preserving workflow outputs: search returns its web result array and map returns URL strings.
• Preserve legacy crawl path-depth limits with v2 URL filters and keep discovery depth independent. Retain the v1 search default of five results.
• Refresh generated documentation and tool metadata. All Firecrawl endpoints already target v2; no SDK upgrade is needed.

Type of Change

• Bug fix

Testing

• Demonstrated the original 10 regressions and five combined-filter index-page regressions failing before their fixes. All 49 Firecrawl tests pass.
• Checked 361 URL paths at three depth limits against Rust's regex engine, including index-file exceptions.
• Ran root VITEST_MAX_WORKERS=4 bun run test, bun run lint, bun run type-check, bun run check:audits, and the block-registry check against origin/staging.
• Verified request and response changes against the official v2 migration guide. Live provider requests were not run.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing (new tests pass the test-audit authoring gate)
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
docs	Ready	Preview	Nov 10, 2026 7:52pm UTC

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; no new actionable issues were found.

Summary

This PR updates Firecrawl v2 calls while keeping workflow outputs stable.

• Firecrawl crawl limits URL depth without changing discovery depth.
• Firecrawl search returns web results and defaults to five.
• Firecrawl map gives workflows a list of URL strings.

_{Reviews (2) · Last reviewed commit: "fix(firecrawl): preserve index pages wit..." · Reviewed by Greptile}

[cubic-dev-ai]

1 issue found across 9 files

Confidence score: 3/5

• In apps/sim/tools/firecrawl/crawl.ts, the depth exclusion also drops /index.html and /index.php with maxDepth: 1 and includePaths: ['^/docs'], so otherwise allowed pages won't be crawled. Preserve the index-file exception when applying the exclusion.

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="apps/sim/tools/firecrawl/crawl.ts">

<violation number="1" location="apps/sim/tools/firecrawl/crawl.ts:166">
P2: Preserve the `/index.html` and `/index.php` exception when applying this depth exclusion; with `maxDepth: 1` and `includePaths: ['^/docs']`, this pattern excludes those otherwise allowed pages.</violation>
</file>

<sub>Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

All reported issues were addressed across 9 files

<sub>Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic</sub>