.vscode-test-web folder is published to npm

[F3n67u]

Describe the bug

a 23M .vscode-test-web dir is published to npm, make vscode-webview-ui-toolkit's unpack size huge(24.3M current)

https://unpkg.com/browse/@vscode/[email protected]/

To reproduce

$ mkdir toolkit-test 
$ cd toolkit-test
$ yarn add @vscode/webview-ui-toolkit 
$ cd node_modules/@vscode/webview-ui-toolkit
$ du -sh .vscode-test-web 
 23M    .vscode-test-web

Expected behavior

.vscode-test-web is ignored when published to npm

Current behavior

.vscode-test-web dir is published to npm

BTW

should we also ignore react dir?

[hawkticehurst]

Wowow 😅🤦🏻‍♂️ thank you so much for catching this (and for the subsequent PR)!

I'll go ahead and merge that right now and publish a new patched version of the package.

Also, no the react folder in the root directory should stay. It's the (annoyingly janky) way that we have to structure the NPM build at this time so that toolkit users can import the toolkit React components via the @vscode/webview-ui-toolkit/react path.

It will eventually be removed/restructured once TypeScript releases support for the package.json exports field. You can read more about it here if you're curious.