Skip to content

fix(cli): improve missing build output error message#11

Merged
NJX-njx merged 1 commit into
NJX-njx:mainfrom
Mahakbajpai:fix-cli-missing-build-message
Feb 15, 2026
Merged

fix(cli): improve missing build output error message#11
NJX-njx merged 1 commit into
NJX-njx:mainfrom
Mahakbajpai:fix-cli-missing-build-message

Conversation

@Mahakbajpai

Copy link
Copy Markdown
Contributor

Summary

Improves the CLI error message shown when the build output (dist/entry.js) is missing during onboarding.
Instead of throwing a generic error, the CLI now provides a clear explanation and actionable steps to fix the issue.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring (no functional changes)

Related Issues

Fixes #7

Changes Made

  • Replaced generic thrown error with a user-friendly error message
  • Clearly explains that build output is missing
  • Suggests running pnpm install and pnpm build
  • Exits with proper status code after displaying guidance

Checklist

  • My code follows the project's coding style
  • I have run pnpm check and pnpm test locally
  • I have updated documentation if needed
  • My changes generate no new warnings or errors

@github-actions github-actions 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.

👋 Welcome to OpenSoul! Thanks for your first pull request.

A maintainer will review your changes soon. Please make sure:

  • You've run pnpm check locally and it passes
  • Your changes follow our Contributing Guide
  • You've filled out the PR template

Thank you for contributing! 🎉

@NJX-njx NJX-njx merged commit 86ab474 into NJX-njx:main Feb 15, 2026
20 of 22 checks passed
@NJX-njx

NJX-njx commented Feb 15, 2026

Copy link
Copy Markdown
Owner

@Mahakbajpai Thank you very much, and you are welcome to continue making some interesting contributions

NJX-njx added a commit that referenced this pull request Feb 27, 2026
fix(cli): improve missing build output error message
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.

Improve error messages in CLI onboarding

2 participants