Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

Update MAX_MAP_LIMIT to 100,000 in OpenAPI documentation

Summary

This PR updates the maximum value for the limit parameter in the map endpoint from 30,000 to 100,000 across three OpenAPI specification files in the documentation repository. This brings the OpenAPI documentation in sync with the actual code implementation in the main firecrawl repository, where MAX_MAP_LIMIT is already set to 100,000 in both v1 and v2 types.ts files.

Files updated:

  • api-reference/v1-openapi.json - Updated limit maximum from 30,000 to 100,000
  • v1/api-reference/v1-openapi.json - Updated limit maximum from 30,000 to 100,000
  • v1/api-reference/v2-openapi.json - Updated limit maximum from 30,000 to 100,000

Note: The api-reference/v2-openapi.json file already had the correct value of 100,000 and did not need updating.

Review & Testing Checklist for Human

  • Verify code implementation: Confirm that the actual API code in firecrawl/firecrawl repo has MAX_MAP_LIMIT = 100000 set in apps/api/src/controllers/v1/types.ts and apps/api/src/controllers/v2/types.ts (I verified this, but please double-check)
  • Test map endpoint: Test that the map endpoint actually accepts and correctly handles limit values up to 100,000 without errors or performance issues
  • Check for other references: Search the documentation for any other places that might reference the old 30,000 limit that weren't caught in this update

Notes

@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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