Skip to content

Merge branch 'main' into 42-i-cant-manage-to-extract-the-contents-of-…

06b6137
Select commit
Loading
Failed to load commit list.
Merged

feat: add cookies integration #45

Merge branch 'main' into 42-i-cant-manage-to-extract-the-contents-of-…
06b6137
Select commit
Loading
Failed to load commit list.
Cursor / Cursor BugBot completed Jul 21, 2025 in 3m 3s

BugBot Review

BugBot Analysis Progress (3m 4s elapsed)

✅ Gathered PR context (2s)
✅ Analyzed code changes (1s)
✅ Completed bug detection — 1 potential bug found (3m 0s)
✅ Validation and filtering completed (0s)
✅ Posted analysis results — 1 bug reported (2s)
✅ Analysis completed successfully (0s)

Final Result: BugBot completed review and found 1 potential issue

Request ID: serverGenReqId_a755ad71-9cb9-4e01-a8c8-4cd6084fedfa

Details

Bug: SmartScraper Cookie Validation Fails

The cookies parameter validation in smartScraper is incomplete. The check typeof cookies === 'object' && cookies !== null incorrectly accepts arrays, functions, and other non-plain objects, even though the API expects a plain object with key-value pairs. This can lead to unexpected behavior or API errors when invalid types are passed.

scrapegraph-js/src/smartScraper.js#L31-L38

https://github.com/ScrapeGraphAI/scrapegraph-sdk/blob/06b61379b2f6eca2437c2462449a7b86f40d8db5/scrapegraph-js/src/smartScraper.js#L31-L38

Fix in CursorFix in Web


Was this report helpful? Give feedback by reacting with 👍 or 👎