Skip to content

⬆️ Support sharedb@8 - #189

Merged
alecgibson merged 1 commit into
mainfrom
sharedb-8
Nov 11, 2026
Merged

alecgibson merged 1 commit into
mainfrom
sharedb-8

Conversation

@alecgibson

Copy link
Copy Markdown
Contributor

The only breaking change is no longer polling multi-channel queries every second by default (share/sharedb#780).

Also collapse the || chain into a single range that accepts the same versions.

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

The only breaking change is no longer polling multi-channel queries
every second by default (share/sharedb#780).

Also collapse the `||` chain into a single range that accepts the same
versions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

CI still installs only ShareDB v6, leaving the newly claimed v8 compatibility untested.

1 open issue
What changed in this PR

This PR extends the MongoDB adapter’s declared compatibility to ShareDB 8 while preserving support for previously accepted stable versions.

Changes:

  • Adds ShareDB 8 to the supported dependency range.
  • Replaces the chained version ranges with >=1.9.1 <9.
File Description
package.json Broadens and simplifies the ShareDB dependency range.

Review effort: Balanced

Comment thread package.json Resolved
@alecgibson
alecgibson requested a balanced review from Copilot November 11, 2026 16:20
@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 94.712%. remained the same — sharedb-8 into main

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The ShareDB dependency-range update has no active review issues.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

@alecgibson
alecgibson marked this pull request as ready for review November 11, 2026 16:31
@alecgibson
alecgibson merged commit d3cfb73 into main Nov 11, 2026
16 checks 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