Skip to content

(js sdk)fix/ws deno#2476

Merged
rafaelsideguide merged 3 commits intomainfrom
(js-sdk)fix/ws-deno
Dec 3, 2025
Merged

(js sdk)fix/ws deno#2476
rafaelsideguide merged 3 commits intomainfrom
(js-sdk)fix/ws-deno

Conversation

@rafaelsideguide
Copy link
Copy Markdown
Collaborator

@rafaelsideguide rafaelsideguide commented Dec 2, 2025

Summary by cubic

Make the Watcher work across Deno, browsers, and Node without the ws package. Adds a polling fallback, robust message parsing, and prevents duplicate document events.

  • Bug Fixes

    • Load WebSocket from globalThis or node:undici; fall back to HTTP polling if unavailable.
    • Decode binary frames to UTF-8 before JSON parsing.
    • Deduplicate emitted documents across WS and polling.
    • Emit documents from snapshot responses and set binaryType to arraybuffer when supported.
  • Dependencies

    • Remove ws and @types/ws.
    • Add a type shim for node:undici WebSocket.
    • Bump @mendable/firecrawl-js to 4.8.1.

Written for commit 4c3aa6f. Summary will update automatically on new commits.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 5 files

@rafaelsideguide rafaelsideguide merged commit e4acbcd into main Dec 3, 2025
3 of 8 checks passed
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 5 files

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