chore: ignore node_modules when using vsce for bundling

[bethatguyad]

Closed #160

[bethatguyad]

Before:

After

[bethatguyad]

@sdankel friendly pin, could you please have a look?

[sdankel]

@bethatguyad thank you for the contribution! However, I'm pretty sure the dependencies need to be part of the bundle for it to work properly. I'll test this out today. devDependencies are ignored automatically:

https://code.visualstudio.com/api/working-with-extensions/publishing-extension#using-.vscodeignore

We should be ignoring .ts files, as the docs suggest, if you'd like to make that change.

[sdankel]

When I tested it, the plugin does not work without node_modules in the bundle.

[bethatguyad]

When I tested it, the plugin does not work without node_modules in the bundle.

Can you try again? Now it works, I use esbuild bundle the client before vsce package.
Now it is only

[sdankel]

When I tested it, the plugin does not work without node_modules in the bundle.

Can you try again? Now it works, I use esbuild bundle the client before vsce package. Now it is only

Nice! It works now. With the ts files ignored it's even smaller: 413.96KB

[sdankel]

Great work! Happy to approve once my comments are addressed. This seems to make the extension run much faster 🔥