From 0fb2e9ce623be3f7588e9615a558996a2e633b3c Mon Sep 17 00:00:00 2001 From: Alan Agius <17563226+alan-agius4@users.noreply.github.com> Date: Tue, 12 May 2026 18:15:48 +0000 Subject: [PATCH 1/2] fix(ng-dev/release): make package deprecation errors non-fatal Catches and logs errors that occur during `npm deprecate` rather than allowing them to bubble up and cause the release process to fail. This serves as a temporary workaround for issues where package deprecation requests might fail unexpectedly. --- ng-dev/release/versioning/npm-command.ts | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/ng-dev/release/versioning/npm-command.ts b/ng-dev/release/versioning/npm-command.ts index b16fb74e5..7df3754ae 100644 --- a/ng-dev/release/versioning/npm-command.ts +++ b/ng-dev/release/versioning/npm-command.ts @@ -11,6 +11,7 @@ import semver from 'semver'; import {ChildProcess} from '../../utils/child-process.js'; import {NpmDistTag} from './npm-registry.js'; +import {Log} from '../../utils/logging.js'; export abstract class NpmCommand { /** @@ -47,7 +48,13 @@ export abstract class NpmCommand { args.push('--registry', registryUrl); } - await ChildProcess.spawn('npm', args, {mode: 'silent'}); + try { + await ChildProcess.spawn('npm', args, {mode: 'silent'}); + } catch (e) { + // TODO(alanagius): remove try/catch block once https://buganizer.corp.google.com/u/1/issues/512428441 is fixed. + Log.error(e); + Log.error(` ✘ An error occurred while deprecating "${packageName}".`); + } } /** From 28c12b2c36a5f07eb2cbe92c20128220b437524c Mon Sep 17 00:00:00 2001 From: Alan Agius <17563226+alan-agius4@users.noreply.github.com> Date: Tue, 12 May 2026 18:39:36 +0000 Subject: [PATCH 2/2] fixup! fix(ng-dev/release): make package deprecation errors non-fatal Co-authored-by: Joey Perrott Signed-off-by: Alan Agius --- ng-dev/release/versioning/npm-command.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ng-dev/release/versioning/npm-command.ts b/ng-dev/release/versioning/npm-command.ts index 7df3754ae..2f287e57c 100644 --- a/ng-dev/release/versioning/npm-command.ts +++ b/ng-dev/release/versioning/npm-command.ts @@ -52,8 +52,10 @@ export abstract class NpmCommand { await ChildProcess.spawn('npm', args, {mode: 'silent'}); } catch (e) { // TODO(alanagius): remove try/catch block once https://buganizer.corp.google.com/u/1/issues/512428441 is fixed. - Log.error(e); + Log.error(Array(80).join('#')); Log.error(` ✘ An error occurred while deprecating "${packageName}".`); + Log.error(e); + Log.error(Array(80).join('#')); } }