Skip to content

Fix: wire Firecrawl API key and add retry handling for timeouts#1

Open
isaacnaor wants to merge 1 commit into
firecrawl:mainfrom
isaacnaor:main
Open

Fix: wire Firecrawl API key and add retry handling for timeouts#1
isaacnaor wants to merge 1 commit into
firecrawl:mainfrom
isaacnaor:main

Conversation

@isaacnaor

Copy link
Copy Markdown

This PR addresses issues with Firecrawl integration in the AI Readiness API route.

Changes:

  • Explicitly pass FIRECRAWL_API_KEY into the Firecrawl client to fix "API key required" errors.
  • Add a minimal retry wrapper around firecrawl.scrape to handle upstream timeouts gracefully.
  • Pin runtime to nodejs to avoid env variable quirks under the Edge runtime.
  • Remove duplicate Firecrawl client instantiation and debug logs.

Why it helps:

  • Ensures the app reliably detects and uses the Firecrawl API key.
  • Makes timeouts less disruptive by retrying once before failing.
  • Improves stability and clarity for contributors running the app locally.

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