js/esbuild: Don't try to resolve packages in /assets marked as external

Fixes #13183

1 files changed, 11 insertions, 0 deletions

diff --git a/internal/js/esbuild/resolve.go b/internal/js/esbuild/resolve.go
index ac0010da9..8ceec97ef 100644
--- a/internal/js/esbuild/resolve.go
+++ b/internal/js/esbuild/resolve.go
@@ -167,6 +167,17 @@ func createBuildPlugins(rs *resources.Spec, assetsResolver *fsResolver, depsMana
 			}
 		}
 
+		for _, ext := range opts.Externals {
+			// ESBuild will do a more thorough check for packages resolved in node_modules,
+			// but we need to make sure that we don't try to resolve these in the /assets folder.
+			if ext == impPath {
+				return api.OnResolveResult{
+					Path:     impPath,
+					External: true,
+				}, nil
+			}
+		}
+
 		if opts.ImportOnResolveFunc != nil {
 			if s := opts.ImportOnResolveFunc(impPath, args); s != "" {
 				return api.OnResolveResult{Path: s, Namespace: NsHugoImportResolveFunc}, nil