Skip to content

Conversation

sevetseh28
Copy link

@sevetseh28 sevetseh28 commented Aug 27, 2025

Description

Adds a new console log when build-packages.json fails. I was getting an error that had empty stderr but stdout was non-empty but not being shown. Example:

❌ config-types (build): npm run build failed with code 2
stderr: 
stdout: 
> @continuedev/[email protected] build
> tsc
error TS2688: Cannot find type definition file for 'node'.
  The file is in the program because:
    Entry point of type library 'node' specified in compilerOptions

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-general-review or @continue-detailed-review

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Log stdout on package build failures to improve debugging when stderr is empty. build-packages.js now prints both stderr and stdout after a non-zero exit.

@sevetseh28 sevetseh28 requested a review from a team as a code owner August 27, 2025 08:54
@sevetseh28 sevetseh28 requested review from tomasz-stefaniak and removed request for a team August 27, 2025 08:54
Copy link

github-actions bot commented Aug 27, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 27, 2025
@sevetseh28
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant