diff --git a/src/cli/update-cli.ts b/src/cli/update-cli.ts index c6f3dbd62..2d2d8ddfe 100644 --- a/src/cli/update-cli.ts +++ b/src/cli/update-cli.ts @@ -62,6 +62,7 @@ import { formatCliCommand } from "./command-format.js"; import { installCompletion } from "./completion-cli.js"; import { runDaemonRestart } from "./daemon-cli.js"; import { formatHelpExamples } from "./help-format.js"; +import { suppressDeprecations } from "./update-cli/suppress-deprecations.js"; export type UpdateCommandOptions = { json?: boolean; @@ -672,8 +673,7 @@ function printResult(result: UpdateRunResult, opts: PrintResultOptions) { } export async function updateCommand(opts: UpdateCommandOptions): Promise { - process.noDeprecation = true; - process.env.NODE_NO_WARNINGS = "1"; + suppressDeprecations(); const timeoutMs = opts.timeout ? Number.parseInt(opts.timeout, 10) * 1000 : undefined; const shouldRestart = opts.restart !== false; diff --git a/src/cli/update-cli/suppress-deprecations.ts b/src/cli/update-cli/suppress-deprecations.ts new file mode 100644 index 000000000..8912b5285 --- /dev/null +++ b/src/cli/update-cli/suppress-deprecations.ts @@ -0,0 +1,16 @@ +/** + * Suppress Node.js deprecation warnings. + * + * On Node.js v23+ `process.noDeprecation` may be a read-only property + * (defined via a getter on the prototype with no setter), so the + * assignment can throw. We fall back to the environment variable which + * achieves the same effect. + */ +export function suppressDeprecations(): void { + try { + process.noDeprecation = true; + } catch { + // read-only on Node v23+; NODE_NO_WARNINGS below covers this case + } + process.env.NODE_NO_WARNINGS = "1"; +}