Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JSON error when running <myTreeViewId>.focus-command #63638

Closed
jrieken opened this issue Nov 22, 2018 · 3 comments
Closed

JSON error when running <myTreeViewId>.focus-command #63638

jrieken opened this issue Nov 22, 2018 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug tree-views Extension tree view issues verified Verification succeeded
Milestone

Comments

@jrieken
Copy link
Member

jrieken commented Nov 22, 2018

  • have a custom view with id foo
  • from the extension host run the foo.focus command
  • on the main side a silent error occurs (run with break on all errors)

screenshot 2018-11-22 at 12 30 36

@sandy081 sandy081 added bug Issue identified by VS Code Team member as probable bug tree-views Extension tree view issues labels Nov 22, 2018
@sandy081 sandy081 added this to the November 2018 milestone Nov 22, 2018
@jrieken
Copy link
Member Author

jrieken commented Nov 22, 2018

@alexdima
Copy link
Member

I have to ignore them... Extensions constantly make RPC calls which involve recursive datastructures... This was always showing up in error telemetry.

This was also one of the root causes for leakages, e.g. a command invocation would never complete if serializing its result would throw.

Long story short, the "contract" is that calling with a non serializable object leads to the UI side getting "null". IMHO this is not so bad. The errors don't go to error telemetry because they then reach me and I have nothing to do with them.

@jrieken
Copy link
Member Author

jrieken commented Nov 23, 2018

How about a console.warn-statement when in dev-mode process.env.VSCODE_DEV?

@roblourens roblourens added the verified Verification succeeded label Dec 10, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug tree-views Extension tree view issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants