Handle Node.js Debug Messages and Improve Port Termination #99
+97
−11
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
This PR addresses two issues with Node.js communication:
Node.js Debug Messages: Fixes "unexpected message in handle_info/2" errors when Node.js emits debugging information (like "Debugger listening on ws://..." messages).
Terminal Reset Safety: Adds error handling around port operations during termination to prevent crashes when ports are in an invalid state.
Implementation Details
handle_info/2
callback to handle messages from Node.js processesdebug_mode
configuration option that controls whether Node.js stdout/stderr messages are loggedreset_terminal/1
to gracefully handle invalid ports during terminationUsage
When working with Node.js applications that emit debug output:
Benefits
Original Contribution
This PR builds on the work by @francois-codes, with refinements to follow best practices for configuration handling and error management.