Skip to content

Conversation

@mdehsan873
Copy link
Collaborator

@mdehsan873 mdehsan873 commented Nov 14, 2025

Note

Replaces Python SmartCrawler usage from smartcrawler() to crawl(), including async and schema examples.

  • Docs (services/smartcrawler.mdx):
    • Python examples:
      • Quick Start: client.smartcrawler(...)client.crawl(...).
      • Custom Schema Example: client.smartcrawler(...)client.crawl(...).
      • Async Support: await client.smartcrawler(...)await client.crawl(...).

Written by Cursor Bugbot for commit 398b46e. This will update automatically on new commits. Configure here.

client = Client(api_key="your-api-key")

response = client.smartcrawler(
response = client.crawl(
Copy link

Choose a reason for hiding this comment

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

Bug: Fix Inconsistent JavaScript Documentation

The Python examples were updated to use client.crawl(), but the JavaScript Quick Start still imports and calls smartCrawler(). This creates an inconsistency where line 127 correctly uses crawl() for JavaScript, but lines 40 and 49 still use the old smartCrawler() function name, making the documentation inconsistent across examples.

Fix in Cursor Fix in Web

features: list[str] = Field(description="Features")

response = client.smartcrawler(
response = client.crawl(
Copy link

Choose a reason for hiding this comment

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

Bug: Standardize JavaScript Example Crawl Function

The JavaScript schema example still imports and calls smartCrawler() while the corresponding Python example was updated to use client.crawl(). This is inconsistent with the JavaScript markdown mode example at line 127 which correctly uses crawl(), creating confusion about the correct function name to use.

Fix in Cursor Fix in Web

@VinciGit00 VinciGit00 merged commit 26a810e into main Nov 14, 2025
3 checks passed
@VinciGit00 VinciGit00 deleted the fix-the-smartcrawler branch November 14, 2025 16:26
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.

3 participants