Skip to content

Conversation

@igorlfs
Copy link
Contributor

@igorlfs igorlfs commented Jan 30, 2026

Closes #2283

Note: I also wanna make some other errors explicit, if that's okay

Edit: notably, I'm interested in the following line:

throw new ProtocolError(errors.createSilentError(text));

@connor4312
Copy link
Member

This is intentional, since clients get that error in response to a REPL request and generally will already show that error as the result of the evaluation in their REPL/debug console. This differs from errors in other more implicit requests that have semantics that would not always require the client to show some output/error

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.

RFC: some errors should have showUser set to true

2 participants