Skip to content

Integrate @decodo/sdk-ts and update CI for sibling checkout#2

Merged
paulius-krutkis-dcd merged 4 commits into
mainfrom
decodo-sdk-integration
Jun 2, 2026
Merged

Integrate @decodo/sdk-ts and update CI for sibling checkout#2
paulius-krutkis-dcd merged 4 commits into
mainfrom
decodo-sdk-integration

Conversation

@paulius-krutkis-dcd

@paulius-krutkis-dcd paulius-krutkis-dcd commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replace the CLI's custom HTTP client and web scraping wrapper with @decodo/sdk-ts (DecodoClient, SDK error types, typed scrape requests).
  • Remove duplicated platform code: HttpClient, local error classes, request/response types, and WebScrapingApi.
  • Update auth validation to call client.webScrapingApi.scrape() with Target.Universal (imported as ScrapeTarget to avoid a Biome duplicate-key lint).
  • Wire CI to check out and build Decodo/sdk-ts alongside the CLI until the SDK is published to npm; compile committed src/generated only (codegen needs gitignored decodo.ir.json).

Notes

  • x-integration is still "sdk-ts" in the SDK (SCR-3150); tests reflect that. INTEGRATION_HEADER = "cli" remains for when the SDK supports it.
  • Temporary CI/dev setup uses file:../sdk-ts and a separate sdk-ts checkout. TODOs mark what to remove once @decodo/sdk-ts is on npm.

Test plan

  • pnpm lint
  • pnpm typecheck
  • pnpm test
  • CI passes with dual checkout and compile-only sdk-ts build

@paulius-krutkis-dcd paulius-krutkis-dcd changed the title refactor: consolidate error handling and remove unused error classes;… Integrate @decodo/sdk-ts and update CI for sibling checkout Jun 2, 2026
@paulius-krutkis-dcd paulius-krutkis-dcd merged commit cb980a1 into main Jun 2, 2026
1 check passed
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