Hi @HardlyDifficult ! This repo was forwarded to me by @nearnshaw to take a look at it in order to use it for a tutorial/video. First of all, great job! I really like the approach you took to build this game, I just wanted to give you some feedback around some minor issues that I think can be improved:

• The name of the ScriptableScene class is still HouseScene (link)

• The name of the list of enemies/creeps is entities (i.e. getState().entities), I think this name might be a little misleading since entity is a term used to refer to the 3D entities in the scene like <entity>, maybe a name like enemies or creeps or blobs would be more clear? Same goes for the <Entity> component (link)

• I see this pattern repeated in the code:

// do something
setTimeout(() => {
  // do something else later
}, 1000)

This pattern becomes a little hard to follow when there are many nested timeouts like here, but if you add a helper like this:

const sleep = (ms: number): Promise<void> => new Promise(resolve => setTimeout(resolve, ms))

Then you can use a pattern like this:

// do something
await sleep(100)
// do something else later
await sleep(1000)
// do something else after that

Which is easier to follow, you just have to make sure to make the function you are executing that helper in an async function, like converting this:

this.eventSubscriber.on('event', () => {
  // do something
  setTimeout(() => {
    // do something else
  }, 1000)
})

into this:

this.eventSubscriber.on('event', async () => {
  // do something
  await sleep(1000)
  // do something else
})

notice the async

- this.eventSubscriber.on('event', () => {
+ this.eventSubscriber.on('event', async () => {

• And finally, just as a suggestion (it's probably out of the scope for this tutorial) I see that the game starts as soon as the server starts, but there's no concept of a "match". If this were to be deployed to decentraland it would be a never ending match (unless the server restarts). It would be great if the game had some event/condition to make it start (ie clicking on a button) and then an "end" condition (time/score/lives/etc) to make it end the game and go back to initial state. Again, that's just a suggestion if you are planning to deploy this to DCL (which I believe you should totally do!) but I don't think is necessary in order to use this as a tutorial/video.