Skip to content

feat(cli): suppress update notifier - #6143

Open
7ttp wants to merge 27 commits into
supabase:developfrom
7ttp:fix/suppress-upgrade-notice
Open

feat(cli): suppress update notifier#6143
7ttp wants to merge 27 commits into
supabase:developfrom
7ttp:fix/suppress-upgrade-notice

Conversation

@7ttp

@7ttp 7ttp commented Aug 10, 2026

Copy link
Copy Markdown
Member

TL;DR

Adds SUPABASE_NO_UPDATE_NOTIFIER to suppress the
A new version of Supabase CLI is available notice, following the same convention...

What's introduced?

  • A native port of the upgrade notice to the legacy TypeScript shell: same message bytes, same supabase/.temp/cli-latest ten hour cache, --workdir aware, with --debug/SUPABASE_DEBUG diagnostics for failed fetches and cache reads.
  • One switch for both sides: the Go sidecar honors the same variable, and the legacy shell suppresses its Go children so the notice prints exactly once.
  • Hardening: a symlink anywhere in the cache path disables caching instead of being written through, color is gated for NO_COLOR and piped stderr, and a Ctrl-C or CI timeout during the bounded trailing check no longer turns a successful command into exit 130, through the compiled binary and the npm shim alike.

Why is it needed?

The notice only came from the bundled Go sidecar, so it appeared on delegated commands, was missing from native ones, and could not be turned off. Users who pin the CLI version on purpose (Nix, lockfiles, CI images) had no way to silence it.

ref:

@7ttp
7ttp requested a review from a team as a code owner August 10, 2026 23:56
@7ttp 7ttp self-assigned this Aug 10, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9ca756d584

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
Comment thread apps/cli/src/shared/cli/bin.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
@7ttp
7ttp force-pushed the fix/suppress-upgrade-notice branch from 9ca756d to e4e43ca Compare August 11, 2026 00:14

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f05dabfaae

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/shared/cli/run.ts Outdated
Comment thread apps/cli/src/shared/cli/bin.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts
@7ttp
7ttp marked this pull request as draft August 11, 2026 00:59

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5465aebde7

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/shared/cli/bin.ts
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
Comment thread apps/cli/src/shared/cli/run.ts Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 858af0ee50

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/shared/cli/run.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
Comment thread apps/cli/src/shared/cli/run.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ffddcf7ad7

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/shared/cli/bin.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
Comment thread apps/cli/src/shared/cli/bin.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5e245c0312

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7bc14c3880

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/shared/cli/run.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
@7ttp

7ttp commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a975ecc4ff

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts
Comment thread apps/cli/src/shared/cli/run.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
@7ttp

7ttp commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

@codex review

@7ttp

7ttp commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9e24e01b63

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/shared/cli/bin.ts
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
Comment thread apps/cli/src/shared/cli/bin.ts
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
Comment thread apps/cli/src/legacy/shared/legacy-upgrade-notice.ts Outdated
@7ttp

7ttp commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 978c6a8350

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli/src/shared/cli/run.ts Outdated
Comment thread apps/cli/src/shared/cli/run.ts Outdated
Comment thread apps/cli/src/shared/cli/run.ts Outdated
@7ttp
7ttp marked this pull request as ready for review August 11, 2026 11:10

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7806e6cb55

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +778 to +781
yield* Effect.scoped(
processControl
.holdSignals(["SIGINT", "SIGTERM", "SIGHUP"])
.pipe(Effect.andThen(Fiber.interrupt(cliFiber))),

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Hold signals before the once-listener is removed

When the npm shim is launched from a Unix terminal for a globally wrapped command such as start or db start, Ctrl-C is delivered to the compiled CLI and also forwarded by the shim as a second SIGINT. Fresh evidence in the current code is that ProcessControl.awaitSignal removes its once-listener before this hold is acquired (process-control.layer.ts:29-31), so the forwarded duplicate can land in that gap and default-terminate the process before Fiber.interrupt runs command finalizers/rollback; install the no-op hold before awaiting/replacing the signal listener, or avoid forwarding signals that were already sent to the foreground group.

AGENTS.md reference: apps/cli/AGENTS.md:L250-L256

Useful? React with 👍 / 👎.

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.

Support suppressing "A new version of Supabase CLI is available" message

1 participant