fix: the http://0.0.0.0:port cant visit in windows, we shouldnt set p…

…ublicPath as //0.0.0.0:${port}/

.changeset/blue-kiwis-raise.md

@@ -0,0 +1,6 @@
+---
+'@modern-js/builder-shared': patch
+---
+
+fix: The http://0.0.0.0:port can't visit in windows, we shouldn't set publicPath as `//0.0.0.0:${port}/`;
+fix: 在 windows 里不能正常访问 http://0.0.0.0:port，我们不应该将 publicPath 设置成 `//0.0.0.0:${port}`

packages/builder/builder-shared/src/apply/output.ts

@@ -82,7 +82,13 @@ function getPublicPath({
   } else if (dev.assetPrefix === true) {
     const hostname = context.devServer?.hostname || DEFAULT_DEV_HOST;
     const port = context.devServer?.port || DEFAULT_PORT;
-    publicPath = `//${hostname}:${port}/`;
+    if (hostname !== DEFAULT_DEV_HOST) {
+      // If user not specify the hostname, it would use DEFAULT_DEV_HOST(0.0.0.0)
+      // The http://0.0.0.0:port can't visit in windows, so we shouldn't set publicPath as `//0.0.0.0:${port}/`;
+      // Relative to docs:
+      // - https://github.com/quarkusio/quarkus/issues/12246
+      publicPath = `//${hostname}:${port}/`;
+    }
   }
 
   return addTrailingSlash(publicPath);

tests/integration/asset-prefix/test/index.test.ts

@@ -1,7 +1,6 @@
 import path from 'path';
 import { readFileSync } from 'fs';
 import { Page } from 'puppeteer';
-import { DEFAULT_DEV_HOST } from '@modern-js/utils';
 import { launchApp, killApp } from '../../../utils/modernTestUtils';
 
 declare const page: Page;
@@ -18,7 +17,7 @@ describe('asset prefix', () => {
       path.join(appDir, 'dist/html/main/index.html'),
       'utf-8',
     );
-    expect(HTML.includes(`//${DEFAULT_DEV_HOST}:3333/static/js/`)).toBeTruthy();
+    expect(HTML.includes(`/static/js/`)).toBeTruthy();
 
     killApp(app);
   });
@@ -27,7 +26,7 @@ describe('asset prefix', () => {
     const appDir = path.resolve(fixtures, 'dev-asset-prefix');
 
     const app = await launchApp(appDir);
-    const expected = `//${DEFAULT_DEV_HOST}:3333`;
+    const expected = '';
 
     const mainJs = readFileSync(
       path.join(appDir, 'dist/static/js/main.js'),
@@ -38,7 +37,7 @@ describe('asset prefix', () => {
       mainJs.includes(`window.__assetPrefix__ = '${expected}';`),
     ).toBeTruthy();
 
-    await page.goto(`http:${expected}`);
+    await page.goto(`http://localhost:3333`);
 
     const assetPrefix = await page.evaluate(() => {
       // eslint-disable-next-line @typescript-eslint/ban-ts-comment