Fix exported names from a dep (close #750)

server/build.go

@@ -534,16 +534,12 @@ rebuild:
 						}
 
 						if isLocalSpecifier(specifier) {
-							// is sub-module of current package and non-dynamic import
+							// sub-module of current package and non-dynamic import
 							if strings.HasPrefix(fullFilepath, task.realWd) && args.Kind != api.ResolveJSDynamicImport {
 								// splits modules based on the `exports` defines in package.json,
 								// see https://nodejs.org/api/packages.html
 								if om, ok := npm.PkgExports.(*orderedMap); ok {
 									modName := "." + strings.TrimPrefix(fullFilepath, path.Join(task.installDir, "node_modules", npm.Name))
-									// bundles _internal_ modules
-									if strings.Contains(modName, "/internal/") {
-										return api.OnResolveResult{}, nil
-									}
 									if strings.HasSuffix(modName, ".mjs") {
 										modName = strings.TrimSuffix(specifier, ".mjs")
 									} else {
@@ -586,7 +582,31 @@ rebuild:
 										}
 									}
 								}
-								return api.OnResolveResult{}, nil
+
+								// externalize the module that is an alias of a dependency
+								// means this file just include a single line(js): `export * from "dep"`
+								isAlias := false
+								fi, ioErr := os.Lstat(fullFilepath)
+								if ioErr == nil && fi.Size() < 256 {
+									data, ioErr := os.ReadFile(fullFilepath)
+									if ioErr == nil {
+										out, esbErr := minify(string(data), api.ESNext, api.LoaderJS)
+										if esbErr == nil {
+											p := bytes.Split(out, []byte("\""))
+											if len(p) == 3 && string(p[0]) == "export*from" && string(p[2]) == ";\n" {
+												url := string(p[1])
+												if !isLocalSpecifier(url) {
+													isAlias = true
+													specifier = url
+												}
+											}
+										}
+									}
+								}
+
+								if !isAlias {
+									return api.OnResolveResult{}, nil
+								}
 							}
 							specifier = strings.TrimPrefix(fullFilepath, filepath.Join(task.installDir, "node_modules")+"/")
 						}

test/issue-750/issue-750.test.ts

@@ -0,0 +1,19 @@
+import { assert } from "https://deno.land/[email protected]/testing/asserts.ts";
+
+import * as element from "http://localhost:8080/@lume/[email protected]?target=esnext";
+import * as variable from "http://localhost:8080/@lume/[email protected]";
+
+Deno.test("issue #750", () => {
+  assert(checkKeys(element, variable));
+});
+
+// check keys of an object in another object
+function checkKeys(a: object, b: object) {
+  for (const key in b) {
+    if (!(key in a)) {
+      console.error(`key ${key} is missing in b`);
+      return false;
+    }
+  }
+  return true;
+}