rusackas opened a new pull request, #41206: URL: https://github.com/apache/superset/pull/41206
### SUMMARY The **Embedded SDK Release** workflow has been failing on every push to `master` since #40991 bumped `@superset-ui/embedded-sdk` to `0.4.0`. While the version sat at `0.3.0` (already on npm), `release-if-necessary.js` always took the "version already exists, exiting" path and the job stayed green. The bump made it actually run `npm publish` for the first time since November 2025, and the publish fails. The real failure is invisible in CI: the catch block only prints `err.stdout`, but `npm publish` writes its failure details (auth/permission/registry errors) to **stderr**. So the logs show only: ``` [embedded-sdk-release] build successful, publishing [embedded-sdk-release] Encountered an error, details should be above ##[error]Process completed with exit code 1. ``` …with nothing above it. This PR prints `err.stderr` as well so the actual npm error becomes visible. Note: this is a **diagnostics** fix. The underlying publish failure is almost certainly an expired/invalid `NPM_TOKEN` (or a newer npm 2FA/provenance requirement) and must be resolved by a maintainer with npm org access — `release-if-necessary.js` only checks that the token is non-empty, not that it's valid. Once this lands, the next master run will reveal the exact error. This workflow triggers only on `push` to `master`/release branches, so it does **not** gate PR merges — but it leaves a red ❌ on every master commit and means `0.4.0` is not being published. ### TESTING INSTRUCTIONS The change is in the error-handling path of the release script. To verify the swallowing behavior, run the script with a command whose error goes to stderr; previously only stdout was echoed, now both streams are printed. The real verification is the next `Embedded SDK Release` run on master, which will now log the actual npm publish error. ### ADDITIONAL INFORMATION - [ ] Has associated issue: - [ ] Required feature flags: - [ ] Changes UI - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [ ] Introduces new feature or API - [ ] Removes existing feature or API 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
