Skip to content

v2.4.0: sync parity + colorless job verbs, and dataset snapshot error reporting#40

Open
karaposu wants to merge 3 commits into
brightdata:devfrom
karaposu:dev
Open

v2.4.0: sync parity + colorless job verbs, and dataset snapshot error reporting#40
karaposu wants to merge 3 commits into
brightdata:devfrom
karaposu:dev

Conversation

@karaposu

Copy link
Copy Markdown
Contributor

Summary

  • Two changes + version bump 2.3.2 → 2.4.0.

  • SyncBrightDataClient now mirrors the async surface: adds datasets, the TikTok/YouTube/Reddit/Perplexity/DigiKey scrapers,
    TikTok/YouTube search verticals, Pinterest trigger/status/fetch, and two Instagram-search methods (profiles, reels_all).

  • BaseWebScraper + DiscoverService gain id-driven status/wait/fetch/to_result — a triggered job is drivable without
    async-only job methods (colorless pattern, like the crawler); existing job methods unchanged.

  • Contract note: sync discover_trigger() now returns a colorless DiscoverSnapshot (drive via
    discover_status/discover_fetch(task_id)) instead of the async-only DiscoverJob, which couldn't be used from sync.

Tests

  • Full unit suite green locally (248 passed); new tests cover the colorless verbs, sync parity, and the dataset error path.

karaposu added 3 commits June 16, 2026 15:39
…one"

SnapshotStatus keeps the raw API response and matches more reason keys; download() now reports the reason (or raw) and the snapshot_id.
- SyncBrightDataClient mirrors the async surface: adds `datasets`, the TikTok/YouTube/Reddit/Perplexity/DigiKey scrapers, TikTok/YouTube search verticals, Pinterest trigger/status/fetch, and two Instagram-search methods (profiles, reels_all).
- BaseWebScraper and DiscoverService gain id-driven status/wait/fetch/to_result, so a triggered job is drivable without async-only job methods (colorless pattern, like the crawler); existing job methods unchanged.
- sync discover_trigger() now returns a colorless DiscoverSnapshot instead of the async-only DiscoverJob.
- adds the first synchronous tests.
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