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

Demo: Console log suddenly full of errors #259

Closed
kaisalmen opened this issue Nov 24, 2023 · 16 comments · Fixed by #260
Closed

Demo: Console log suddenly full of errors #259

kaisalmen opened this issue Nov 24, 2023 · 16 comments · Fixed by #260

Comments

@kaisalmen
Copy link
Collaborator

Latest main. I built it on my Ubuntu WSL as usual. And I just checked the deployed version same there. Are you aware of this already?
image
image

@CGNonofr
Copy link
Contributor

I wasn't! will check

@CGNonofr
Copy link
Contributor

Are you able to reproduce locally? I'm not

@kaisalmen
Copy link
Collaborator Author

kaisalmen commented Nov 24, 2023

Yes the screenshot is from my local env, but I see the same errors with the deployed version

@CGNonofr
Copy link
Contributor

Are you able to reproduce in the demo here?

@kaisalmen
Copy link
Collaborator Author

kaisalmen commented Nov 24, 2023

Are you able to reproduce in the demo here?

I don't understand that question ⬇️ I did a hard-reload/kill-cache
image

@CGNonofr
Copy link
Contributor

locally in the demo here I meant :)

@kaisalmen
Copy link
Collaborator Author

The same

@kaisalmen
Copy link
Collaborator Author

kaisalmen commented Nov 24, 2023

I build the main here (a726df1). The PR commit of #258. Not the one from #252 and this change won't alter the result, I think.

@CompuIves
Copy link
Collaborator

I just noticed this too with an upgrade. For some reason disabling the snippet service made it work for me here, I can also help debug this end of day.

@CompuIves
Copy link
Collaborator

CompuIves commented Nov 24, 2023

I've a theory, for me some issues were resolved yesterday after including https://github.com/CodinGame/monaco-vscode-api/blob/main/demo/vite.config.ts#L51-L64. I wonder if there's duplicate code.

@CGNonofr
Copy link
Contributor

There is a limit while trying to inject dependencies to prevent infinite loop which is limited arbitrary to 1000. Changing it to 2000 fixes the issue so it looks like a race condition here

@CompuIves
Copy link
Collaborator

🤯

@CGNonofr
Copy link
Contributor

Ok I'm able to reproduce locally, I wasn't up to date with the remote main. me = dumb

@CGNonofr
Copy link
Contributor

I've created a PR on the VSCode side: microsoft/vscode#199037

and the PR here: #260

@CompuIves
Copy link
Collaborator

Oh! Very interesting to see that this was on the VSCode side!

@kaisalmen
Copy link
Collaborator Author

@CGNonofr superb! Thank you.

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 a pull request may close this issue.

3 participants