Skip to content

fix: make REPL errors non-silent#2313

Open
igorlfs wants to merge 1 commit intomicrosoft:mainfrom
igorlfs:repl-errors-silent
Open

fix: make REPL errors non-silent#2313
igorlfs wants to merge 1 commit intomicrosoft:mainfrom
igorlfs:repl-errors-silent

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 Never mind, figured this out!

@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

@igorlfs
Copy link
Contributor Author

igorlfs commented Feb 4, 2026

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.

Could that be made more clear into the specification? As it stands, "ignoring" showUser does not sound correct.

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