(superset) branch master updated: fix(embedded-sdk): surface npm publish stderr in release script (#41206)

[ru...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

rusackas pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 60a78041932 fix(embedded-sdk): surface npm publish stderr in release script (#41206)
60a78041932 is described below

commit 60a780419320a822a8ad253da5a0c3b531ab9a2f
Author: Evan Rusackas <ev...@preset.io>
AuthorDate: Thu Jun 18 13:16:15 2026 -0700

    fix(embedded-sdk): surface npm publish stderr in release script (#41206)
    
    Co-authored-by: Claude Opus 4.8 <no...@anthropic.com>
---
 superset-embedded-sdk/release-if-necessary.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/superset-embedded-sdk/release-if-necessary.js b/superset-embedded-sdk/release-if-necessary.js
index 83196ad821d..69878b8a46c 100644
--- a/superset-embedded-sdk/release-if-necessary.js
+++ b/superset-embedded-sdk/release-if-necessary.js
@@ -47,7 +47,11 @@ function logError(...args) {
       execSync('npm publish --access public', { stdio: 'pipe' });
       log(`published ${version} to npm`);
     } catch (err) {
-      console.error(String(err.stdout));
+      // npm writes failure details to stderr (auth/permission/registry
+      // errors in particular), so surface both streams to avoid masking
+      // the real cause in CI logs.
+      if (err.stdout) console.error(String(err.stdout));
+      if (err.stderr) console.error(String(err.stderr));
       logError('Encountered an error, details should be above');
       process.exitCode = 1;
     }