Update dependency deno to v2.7.14#8531
Conversation
8d2411b to
93f02ee
Compare
9519850 to
4ec9de6
Compare
|
@copilot please analyze and propose a fix or improvement for this issue/PR. DetailsImportant Reference (RxDB docs for LLMs)Use the official RxDB docs here to align with architecture, plugins, storage abstractions, replication patterns, typings, and coding conventions: What to do (pick the right action based on what’s present)Bug report with no test case
Bug report with a test case already present but no fix
Regression (something used to work)
Dependency version bump required
Types/TS errors
Docs-only or README mismatch
Output requirementsPlease provide:
🚨 Author mention ruleIf your solution:
then mention the original issue/PR author at the end of your response: @ please review the proposed changes. |
4ec9de6 to
a364e7e
Compare
…ble-byonm flags BroadcastChannel was stabilized in Deno 2.6.0, making --unstable-broadcast-channel unnecessary. --unstable-byonm is also deprecated as BYONM is now the default in Deno 2.x. Co-authored-by: pubkey <8926560+pubkey@users.noreply.github.com>
... Two improvements were needed alongside the version bump:
Both flags have been removed from all |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
|
@copilot the CI for this fails, please inspect the logs and fix. |
…cache is evicted by GC Co-authored-by: pubkey <8926560+pubkey@users.noreply.github.com>
The CI failure was in Root cause: Fix (commit |
This PR contains the following updates:
2.6.5→2.7.14Additional fixes for Deno 2.7 compatibility
The following deprecated CLI flags have been removed from the
test:deno:*scripts inpackage.json:--unstable-broadcast-channel:BroadcastChannelwas stabilized in Deno 2.6.0, making this flag unnecessary. In Deno 2.7.x it causes anunknown unstable flagerror.--unstable-byonm: "Bring Your Own node_modules" is now the default behavior in Deno 2.x, so this flag is deprecated and no longer needed.The
--unstable-kvflag is retained for the DenoKV storage tests, as Deno KV remains an unstable API.Fix:
upsertLocal()conflict under aggressive GCDeno 2.7 runs the garbage collector more aggressively between async operations.
upsertLocalpreviously relied solely on the doc cache to determine whether a local document existed. When the GC collected a returnedRxDocument(whose only strong reference was viaWeakRefin the cache), theFinalizationRegistrycallback evicted the entire cache entry. On the next call toupsertLocal, the empty cache caused it to fall through toinsertLocal, which then failed with a 409 conflict because the document still existed in storage.upsertLocalnow falls back to agetSingleDocumentstorage read when the cache has no entry for the given id. If the document is found in storage and is not deleted, it is updated viaincrementalModifywith the correct previous revision instead of attempting a conflicting insert.Release Notes