Skip to content

Modify vscodeignore file to restore clean 'vsce package' builds - #14

Closed
Arfeen-Yousuf wants to merge 1 commit into
Dart-Code:masterfrom
Arfeen-Yousuf:fix/update-vscodeignore
Closed

Modify vscodeignore file to restore clean 'vsce package' builds#14
Arfeen-Yousuf wants to merge 1 commit into
Dart-Code:masterfrom
Arfeen-Yousuf:fix/update-vscodeignore

Conversation

@Arfeen-Yousuf

Copy link
Copy Markdown

Description
When attempting to build the Flutter extension .vsix package locally using vsce package, the process fails with an Extension entrypoint(s) missing error.

Root Cause
The existing .vscodeignore file was configured to ignore the out/ directory. Because package.json points to ./out/src/extension.js as the main entry point, vsce strips the required compiled files during packaging, causing the build to fail.

Click to expand error log
 ERROR  Extension entrypoint(s) missing. Make sure these files exist and aren't ignored by '.vscodeignore':
  extension/out/src/extension.js

DanTup added a commit that referenced this pull request Aug 24, 2026
out/src is ignored because for the Dart extension it's where the unbundled code goes. To keep things more consistent between the two, also build this into out/dist (eventually if we add more code here, we would compile to out/src and bundle to out/dist the same).

This prevents errors when running `vsce package` on the latest version, which errors if the entrypoint is ignored.

See #14
@DanTup

DanTup commented Aug 24, 2026

Copy link
Copy Markdown
Member

@Arfeen-Yousuf thanks for the PR! Oddly this seemed to build for me on an older version of vsce - I suspect the warning was added more recently (after updating, I also see it).

My intention is to try and keep some of these files/config more consistent with the Dart extension, so rather than this change, I have updated to build the extension into out/dist to match Dart. This keeps the ignore files consistent, and if we start bundling this one (like we do for Dart), we will compile to out/src and then bundle into out/dist the same.

Thanks!

@DanTup DanTup closed this Aug 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants