Skip to content

Add dependency checks with helpful error messages#4

Merged
robdimarco-atxp merged 1 commit intomainfrom
robdimarco/atxp-244-give-helpful-error-messages-if-pre-requisites-do-not-exist
Sep 9, 2025
Merged

Add dependency checks with helpful error messages#4
robdimarco-atxp merged 1 commit intomainfrom
robdimarco/atxp-244-give-helpful-error-messages-if-pre-requisites-do-not-exist

Conversation

@robdimarco-atxp
Copy link
Copy Markdown
Contributor

Summary

  • Add dependency checking utility for git and npm before running CLI commands
  • Show friendly error messages with installation instructions when dependencies are missing
  • Add comprehensive tests for dependency checking functionality

Changes

  • New utility: check-dependencies.ts - handles checking for external dependencies (git, npm)
  • Enhanced CLI: Modified index.ts to check dependencies before running demo/create commands
  • User experience: Clear error messages with installation links instead of cryptic spawn errors
  • Tests: Added unit tests to verify dependency checking works correctly

Test plan

  • Build and typecheck passes
  • All existing tests pass
  • New dependency checking tests pass
  • Lint checks pass
  • Manual testing confirms dependency checks work for demo command
  • Help command works without dependency checks (as expected)

Fixes: https://linear.app/novellum/issue/ATXP-244/give-helpful-error-messages-if-pre-requisites-do-not-exist

🤖 Generated with Claude Code

- Add dependency checking utility for git and npm
- Check dependencies before running demo and create commands
- Show friendly error messages with installation instructions
- Add tests for dependency checking functionality

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@linear
Copy link
Copy Markdown

linear bot commented Sep 9, 2025

ATXP-244 Give helpful error messages if pre-requisites do not exist

We require tools such as git to be able to run the demo. We should check to see if they exist, and if not, prompt the user for it.

@robdimarco-atxp robdimarco-atxp merged commit d1dc63a into main Sep 9, 2025
1 check passed
@robdimarco-atxp robdimarco-atxp deleted the robdimarco/atxp-244-give-helpful-error-messages-if-pre-requisites-do-not-exist branch September 9, 2025 19:25
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.

1 participant