chore(module-tools): add comment for resolve false in browser alias (#…

…4818)

packages/solutions/module-tools/src/builder/esbuild/adapter.ts

@@ -190,7 +190,9 @@ export const adapterPlugin = (compiler: ICompiler): Plugin => {
           ? args.path
           : getResultPath(originalFilePath, dir, args.kind);
         if (resultPath === false) {
-          debugResolve('empty resolve:', args);
+          // https://github.com/defunctzombie/package-browser-field-spec
+          // we may get false when resolve browser field, in this case, we set it a empty object
+          debugResolve('resolve false:', args);
           return {
             path: '/empty-stub',
             sideEffects: false,

tests/integration/module/fixtures/build/resolve/false/index.ts

@@ -1,6 +1,3 @@
-// In some directory structure, we can not resolve './util.inspect' from 'index.js' in 'object-inspect',
-// For example, in bytedance internal monorepo solution, but here we can resolve it.
-
 import xxx from 'object-inspect';
 
 console.log('xxx:', xxx);

tests/integration/module/fixtures/build/resolve/false/modern.config.ts

@@ -3,5 +3,6 @@ import { defineConfig } from '@modern-js/module-tools/defineConfig';
 export default defineConfig({
   buildConfig: {
     input: ['./index.ts'],
+    platform: 'browser',
   },
 });