Skip to content

feat(nodeup): standardize actionable error messages with hints#293

Merged
kdy1 merged 1 commit intomainfrom
kdy1/nodeup-error-message-overhaul
Mar 22, 2026
Merged

feat(nodeup): standardize actionable error messages with hints#293
kdy1 merged 1 commit intomainfrom
kdy1/nodeup-error-message-overhaul

Conversation

@kdy1
Copy link
Copy Markdown
Contributor

@kdy1 kdy1 commented Mar 22, 2026

Summary

  • standardize all NodeupError messages to <cause>. Hint: <next action>
  • keep JSON error envelope schema stable (kind, message, exit_code) while improving message text
  • migrate nodeup command/runtime error paths to actionable hints (toolchain/run/show/which/self/installer/release-index/etc.)
  • update integration tests for new message wording and hint presence
  • update nodeup docs/contracts/public docs to reflect the new error-message behavior

Validation

  • cargo fmt --all
  • cargo test -p nodeup
  • cargo test

Notes

  • clap-generated parser errors are intentionally unchanged

@kdy1 kdy1 changed the title nodeup: standardize actionable error messages with hints feat(nodeup): standardize actionable error messages with hints Mar 22, 2026
@kdy1 kdy1 merged commit 500ae11 into main Mar 22, 2026
14 checks passed
@kdy1 kdy1 deleted the kdy1/nodeup-error-message-overhaul branch March 22, 2026 01:58
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