Skip to content
This repository has been archived by the owner on Aug 16, 2021. It is now read-only.

feat: remote debugger #1404

Merged
merged 39 commits into from
Oct 14, 2020
Merged

feat: remote debugger #1404

merged 39 commits into from
Oct 14, 2020

Conversation

menduz
Copy link
Member

@menduz menduz commented Oct 7, 2020

What?

Generate a new runtime (scene system) to run scenes in node.js and enable debugging in VSCode

Changes:

  • The scene.system.ts file content was moved almost entirely to DecentralandRuntime.ts
  • Added cli.scene.system.ts to run in a constrained node runtime.
  • Preview scenes will now work connected to the CLI via WebSocket when the ?WS_SCENE=1 is added to the URL.

Important: This is an opt-in feature. This is NOT the default behavior.

Why?

Remove limitations of the web platform to develop and test scenes. Node.js will enable not only debugging but also testing tools and quicker development loops.

decentraland/cli#523 <- cli counterpart
https://github.com/decentraland/node-runtime <- runtime for node

menduz and others added 30 commits August 29, 2020 02:02
@github-actions
Copy link

github-actions bot commented Oct 7, 2020

This branch can be previewed at https://explorer.decentraland.zone/branch/remote-debugger/index.html

@menduz menduz changed the title Remote debugger feat: remote debugger Oct 7, 2020
Copy link
Contributor

@nchamo nchamo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

I love the idea of creating the concept of SceneRuntime

kernel/packages/entryPoints/preview.ts Show resolved Hide resolved
@menduz menduz merged commit d120cc4 into master Oct 14, 2020
@menduz menduz deleted the remote-debugger branch October 14, 2020 17:33
This was referenced Oct 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants