Ari Perkkiö

Ok, this makes sense now. This is not exactly a bug as it works exactly as intended, but I have no idea why it was built like this in the...

> it's a special behaviour of `default` reporter when you work with a single test in watch mode, for example, to see how tests are running. It exists for convenience...

We are considering removing all non-Vitest related defaults from `coverage.exclude`, see: - https://github.com/vitest-dev/vitest/issues/6956

The `Toggle terminal` button cannot be replaced with custom buttons. However you can hide the button by specifying `previews: false`, and provide your own `Run tests` button in the lesson...

I'm not sure if Astro supports this out-of-the-box (cc. @Nemikolh, ???). But one thing you could do is convert the `content.md` into `content.mdx`, import the script contents as raw `string`...

@iliakan could you describe your use case a bit more? What does the `landing.js` do? Why does it need to be run outside webcontainer as well?

Internal packages should always use scoped package names. This is a security feature. But if that convention hasn't been followed elsewhere, let's not follow it here either.

> ```diff > - import { resetProcessFactory, setProcessFactory } from '@tutorialkit/test-utils'; > + import { resetProcessFactory, setProcessFactory } from 'test-utils'; > ``` This could import `test-utils` npm package, not our...

I recommend to read this one as well if you haven't seen it earlier: https://medium.com/@alex.birsan/dependency-confusion-4a5d60fec610 It's a bit longer post but really good. 💯

> like starting with `~` or something. That would work. I've seen `~`, `@/` and `#` prefixes used a lot for this. The `#` seems to be related to [sub...