Fix encodeBuildArgsPrefix function

server/build_args.go

@@ -94,12 +94,15 @@ func encodeBuildArgsPrefix(args BuildArgs, pkg Pkg, isDts bool) string {
 			for name := range info.PeerDependencies {
 				pkgDeps.Add(name)
 			}
+			if pkg.SubPath != "" {
+				pkgDeps.Add(pkg.Name)
+			}
 		}
 	}
 	if len(args.alias) > 0 && pkgDeps.Len() > 0 {
 		var ss sort.StringSlice
 		for from, to := range args.alias {
-			if from != pkg.Name {
+			if from != pkg.Name || pkg.SubPath != "" {
 				ss = append(ss, fmt.Sprintf("%s:%s", from, to))
 			}
 		}
@@ -111,7 +114,7 @@ func encodeBuildArgsPrefix(args BuildArgs, pkg Pkg, isDts bool) string {
 	if len(args.deps) > 0 && pkgDeps.Len() > 0 {
 		var ss sort.StringSlice
 		for _, p := range args.deps {
-			if p.Name != pkg.Name {
+			if p.Name != pkg.Name || pkg.SubPath != "" {
 				ss = append(ss, fmt.Sprintf("%s@%s", p.Name, p.Version))
 			}
 		}
@@ -123,7 +126,7 @@ func encodeBuildArgsPrefix(args BuildArgs, pkg Pkg, isDts bool) string {
 	if args.external.Len() > 0 && (args.external.Has("*") || pkgDeps.Len() > 0) {
 		var ss sort.StringSlice
 		for _, name := range args.external.Values() {
-			if name != pkg.Name {
+			if name != pkg.Name || pkg.SubPath != "" {
 				ss = append(ss, name)
 			}
 		}

test/external-all/preact.test.tsx

@@ -30,5 +30,8 @@ Deno.test("external all", () => {
 Deno.test("external all 2", async () => {
   const res = await fetch("http://localhost:8080/*[email protected]/jsx-runtime");
   const code = await res.text();
-  assertStringIncludes(code, "[email protected]/X-ZS8q/");
+  assertStringIncludes(code, "[email protected]/X-ZS8q/denonext/");
+  const res2 = await fetch("http://localhost:8080/*[email protected]");
+  const code2 = await res2.text();
+  assertStringIncludes(code2, "[email protected]/denonext/preact.mjs");
 });

test/external/preact.test.tsx

@@ -1,4 +1,4 @@
-import { assert } from "https://deno.land/[email protected]/testing/asserts.ts";
+import { assert, assertStringIncludes } from "https://deno.land/[email protected]/testing/asserts.ts";
 
 import { h } from "preact";
 import render from "preact-render-to-string";
@@ -26,3 +26,13 @@ Deno.test("external", () => {
   const html = render(<App />);
   assert(html == "<main><p>just now</p></main>");
 });
+
+Deno.test("external 2", async () => {
+  const res = await fetch("http://localhost:8080/[email protected]/hooks?external=preact");
+  const code = await res.text();
+  assertStringIncludes(code, "[email protected]/X-ZS9wcmVhY3Q/");
+
+  const res2 = await fetch("http://localhost:8080/[email protected]?external=preact&target=es2022");
+  const code2 = await res2.text();
+  assertStringIncludes(code2, "[email protected]/es2022/preact.mjs");
+});