Skip to content

Fix: ensure splitting is disabled in esbuild configuration#1355

Merged
alexweininger merged 5 commits intomainfrom
alex/plum-junglefowl
Feb 4, 2026
Merged

Fix: ensure splitting is disabled in esbuild configuration#1355
alexweininger merged 5 commits intomainfrom
alex/plum-junglefowl

Conversation

@alexweininger
Copy link
Member

Seeing if this fixes #1352

@alexweininger alexweininger requested a review from a team as a code owner February 3, 2026 20:26
Copilot AI review requested due to automatic review settings February 3, 2026 20:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses issue #1352 where the Azure panel fails to load after installing the Azure Resources extension with an Azure Functions project open. The fix explicitly disables code splitting in the esbuild configuration to prevent VS Code from encountering issues when loading the extension bundles.

Changes:

  • Added splitting: false to the esbuild configuration to prevent esbuild from creating separate code chunks

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
bwateratmsft
bwateratmsft previously approved these changes Feb 3, 2026
Copy link
Contributor

@bwateratmsft bwateratmsft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, but are we sure it's the splitting that's causing the problem?

In any case, there will be splitting done with the telemetry bundle, so it can't be avoided altogether.

@alexweininger
Copy link
Member Author

Approved, but are we sure it's the splitting that's causing the problem?

In any case, there will be splitting done with the telemetry bundle, so it can't be avoided altogether.

I think it is caused by splitting. But it's a VS Code bug that allows the extension to start loading before everything from the VSIX is unpacked... speculation here

@alexweininger
Copy link
Member Author

I'm going to have Jing/CTI test this out before merging to see if this fixes it.

@bwateratmsft
Copy link
Contributor

Sounds good. We should have CTI check the folder to see if that file exists after the error has appeared, or if it's still gone...

@alexweininger
Copy link
Member Author

Sounds good. We should have CTI check the folder to see if that file exists after the error has appeared, or if it's still gone...

Good idea

],
// Disable code splitting to avoid VS Code extension loading issues (see #1352)
splitting: false,
format: 'cjs',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is strange...the format should be 'cjs' by default...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very strange. The splitting false option didn't work unless this was also set. Not sure why but Copilot suggested it and it works :)

@alexweininger alexweininger merged commit e744e4d into main Feb 4, 2026
3 checks passed
@alexweininger alexweininger deleted the alex/plum-junglefowl branch February 4, 2026 18:03
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.

The Azure panel fails to load after installing Azure Resources extension with an Azure Functions project open

3 participants