Skip to content

fix(supabase): skip Node warning in Deno#2541

Merged
mandarini merged 2 commits into
supabase:masterfrom
7ttp:fix/deno-deprecation-warning
Jul 20, 2026
Merged

fix(supabase): skip Node warning in Deno#2541
mandarini merged 2 commits into
supabase:masterfrom
7ttp:fix/deno-deprecation-warning

Conversation

@7ttp

@7ttp 7ttp commented Jul 20, 2026

Copy link
Copy Markdown
Member

TL;DR

avoid showing the Node 20 deprecation warning in Edge Functions
Edge Runtime exposes a Node 20 compatibility shim alongside Deno so the SDK treated it as a real Node runtime

basically this skips the warning when Deno is present while keeping it
unchanged for actual Node 20 and below..

Ref

@7ttp
7ttp requested a review from a team as a code owner July 20, 2026 07:51
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 24da473d-c2a2-4fcf-9da5-b618d27606d8

📥 Commits

Reviewing files that changed from the base of the PR and between 7419f67 and 26a82a0.

📒 Files selected for processing (1)
  • packages/core/supabase-js/test/unit/deprecation.test.ts
🚧 Files skipped from review as they are similar to previous changes (1)
  • packages/core/supabase-js/test/unit/deprecation.test.ts

📝 Walkthrough

Summary by CodeRabbit

  • Bug Fixes
    • Deprecation warnings are no longer displayed when running in Deno-like environments, even if Node-style version information is available.
    • Improved runtime environment detection to ensure warning behavior matches the current platform.
  • Tests
    • Expanded the deprecation warning test suite to simulate a Deno-like environment and verify warnings are suppressed accordingly.

Walkthrough

The deprecation-warning environment check now treats the presence of globalThis.Deno like a browser environment and suppresses the Node version warning. Unit-test setup and teardown preserve the original Deno global, and a test verifies suppression when Deno exposes a Node-compatible process version.

Assessment against linked issues

Objective Addressed Explanation
Suppress the Node deprecation warning in Deno-compatible Edge Runtime environments [#2540, #2539]

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install timed out. The project may have too many dependencies for the sandbox.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@pkg-pr-new

pkg-pr-new Bot commented Jul 20, 2026

Copy link
Copy Markdown

Open in StackBlitz

@supabase/auth-js

npm i https://pkg.pr.new/@supabase/auth-js@2541

@supabase/functions-js

npm i https://pkg.pr.new/@supabase/functions-js@2541

@supabase/postgrest-js

npm i https://pkg.pr.new/@supabase/postgrest-js@2541

@supabase/realtime-js

npm i https://pkg.pr.new/@supabase/realtime-js@2541

@supabase/storage-js

npm i https://pkg.pr.new/@supabase/storage-js@2541

@supabase/supabase-js

npm i https://pkg.pr.new/@supabase/supabase-js@2541

commit: 26a82a0

@coveralls

coveralls commented Jul 20, 2026

Copy link
Copy Markdown

Coverage Status

Coverage is 93.275%7ttp:fix/deno-deprecation-warning into supabase:master. No base build found for supabase:master.

@mandarini
mandarini merged commit 01dbb53 into supabase:master Jul 20, 2026
27 of 29 checks passed
@7ttp
7ttp deleted the fix/deno-deprecation-warning branch July 20, 2026 11:11
mandarini pushed a commit to supabase/supabase that referenced this pull request Jul 22, 2026
This PR updates @supabase/*-js libraries to version 2.110.8.

**Source**: supabase-js-stable-release

**Changes**:
- Updated @supabase/supabase-js to 2.110.8
- Updated @supabase/auth-js to 2.110.8
- Updated @supabase/realtime-js to 2.110.8
- Updated @supabase/postgest-js to 2.110.8
- Refreshed pnpm-lock.yaml

---

## Release Notes

## v2.110.8

## 2.110.8 (2026-07-21)

### 🩹 Fixes

- **auth:** downgrade aborted/transient fetch failures from
console.error to warn
([#2544](supabase/supabase-js#2544))
- **functions:** clean up cross-signal abort listener on invoke() return
([#2487](supabase/supabase-js#2487))
- **functions:** match response Content-Type case-insensitively
([#2515](supabase/supabase-js#2515))
- **storage:** url-encode object key in CDN purge methods
([#2545](supabase/supabase-js#2545))
- **supabase:** skip Node warning in Deno
([#2541](supabase/supabase-js#2541))

### ❤️ Thank You

- Franco Kaddour @FrancoKaddour
- Katerina Skroumpelou @mandarini
- Pedro Henrique
- Vaibhav @7ttp
## v2.110.7

## 2.110.7 (2026-07-16)

### 🩹 Fixes

- **postgrest:** correct self-reference inference
([#2525](supabase/supabase-js#2525))
- **realtime:** trigger set auth on INITIAL_SESSION event
([#2531](supabase/supabase-js#2531))
- **realtime:** update phoenix to fix presence issue
([#2532](supabase/supabase-js#2532))

### ❤️ Thank You

- Eduardo Gurgel
- Filipe Cabaço @filipecabaco
- Vaibhav @7ttp

This PR was created automatically.

Co-authored-by: supabase-workflow-trigger[bot] <266661614+supabase-workflow-trigger[bot]@users.noreply.github.com>
mandarini pushed a commit to supabase/ssr that referenced this pull request Jul 22, 2026
This PR updates `@supabase/supabase-js` to v2.110.8.

**Source**: supabase-js-stable-release

---

## Release Notes

## v2.110.8

## 2.110.8 (2026-07-21)

### 🩹 Fixes

- **auth:** downgrade aborted/transient fetch failures from
console.error to warn
([#2544](supabase/supabase-js#2544))
- **functions:** clean up cross-signal abort listener on invoke() return
([#2487](supabase/supabase-js#2487))
- **functions:** match response Content-Type case-insensitively
([#2515](supabase/supabase-js#2515))
- **storage:** url-encode object key in CDN purge methods
([#2545](supabase/supabase-js#2545))
- **supabase:** skip Node warning in Deno
([#2541](supabase/supabase-js#2541))

### ❤️ Thank You

- Franco Kaddour @FrancoKaddour
- Katerina Skroumpelou @mandarini
- Pedro Henrique
- Vaibhav @7ttp

This PR was created automatically.

Co-authored-by: supabase-workflow-trigger[bot] <266661614+supabase-workflow-trigger[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants