Skip to content

fix(sdk): show nuxt install failures#5040

Open
cyq1017 wants to merge 2 commits into
daytonaio:mainfrom
cyq1017:codex/daytona-5008-nuxt-install-log-gpt55
Open

fix(sdk): show nuxt install failures#5040
cyq1017 wants to merge 2 commits into
daytonaio:mainfrom
cyq1017:codex/daytona-5008-nuxt-install-log-gpt55

Conversation

@cyq1017

@cyq1017 cyq1017 commented Jun 14, 2026

Copy link
Copy Markdown

Fixes #5008.

Problem:

  • The Nuxt runtime-compat harness ran both npm install steps with --silent, so transient registry/install failures could exit without the useful npm error text in CI logs.

Change:

  • Remove --silent from the Nuxt runtime install steps so npm failure details remain visible.
  • Add a shell-level regression test with a fake npm that verifies install failures from either install step are surfaced.

Verification:

  • bash libs/sdk-typescript/runtime-tests/test-nuxt-install-diagnostics.sh
  • git diff --check origin/main...HEAD
  • sensitive scan over changed files

Summary by cubic

Show npm error output during Nuxt runtime installs in CI by removing --silent, so transient registry/install failures are visible. Adds a regression test with a temporary fake npm to ensure failures from either install step surface their messages.

  • Bug Fixes
    • Removed --silent from both npm install calls in libs/sdk-typescript/runtime-tests/nuxt/run.sh to keep error output.
    • Added libs/sdk-typescript/runtime-tests/test-nuxt-install-diagnostics.sh that injects a fake npm via PATH and asserts the failure message appears when the first or second install fails.

Written for commit e495f12. Summary will update on new commits.

Review in cubic

Signed-off-by: cyq <15000851237@163.com>
@cyq1017 cyq1017 marked this pull request as ready for review June 19, 2026 06:56

@cubic-dev-ai cubic-dev-ai 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.

1 issue found across 2 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread libs/sdk-typescript/runtime-tests/test-nuxt-install-diagnostics.sh
Signed-off-by: cyq <15000851237@163.com>
@vedranjukic vedranjukic force-pushed the main branch 2 times, most recently from 2c15a7f to b40f732 Compare June 23, 2026 14:12
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.

ci: nuxt runtime-compat harness fails silently on transient npm errors

1 participant