fix: fix no-entry bug

Author: imtaotao

dev/react/public/a.js

@@ -1 +1 @@
-// console.log('########', require);
+console.log('########', require);

packages/browser-vm/src/pluginify.ts

@@ -67,7 +67,7 @@ function rewriteAppAndSandbox(
       {
         // For application of environment variables
         ...env,
-        ...app.getExecScriptEnv(false),
+        ...app.getExecScriptEnv(options?.noEntry),
       },
       url,
       options,

packages/core/src/module/app.ts

@@ -1,12 +1,14 @@
 import { StyleManager, TemplateManager } from '@garfish/loader';
 import {
+  Text,
   warn,
   assert,
+  hasOwn,
   remove,
-  Text,
   isJs,
   isObject,
   isPromise,
+  toBoolean,
   findTarget,
   evalWithEnv,
   transformUrl,
@@ -181,14 +183,14 @@ export class App {
       code,
       true,
       url,
-      options.async,
+      options?.async,
     );
     code += url ? `\n//# sourceURL=${url}\n` : '';
 
-    if (!env['window']) {
+    if (!hasOwn(env, 'window')) {
       env = {
-        window: this.global,
         ...env,
+        window: this.global,
       };
     }
 
@@ -298,6 +300,7 @@ export class App {
     if (this.esModule) return {};
     if (noEntry) {
       return {
+        require: this.cjsModules.require,
         [__GARFISH_EXPORTS__]: this.customExports,
         [__GARFISH_GLOBAL_ENV__]: this.globalEnvVariables,
       };
@@ -488,7 +491,7 @@ export class App {
           const { url, scriptCode } = jsManager;
           this.execScript(scriptCode, {}, url || this.appInfo.entry, {
             async: false,
-            noEntry: Boolean(entryManager.findAttributeValue(node, 'no-entry')),
+            noEntry: toBoolean(entryManager.findAttributeValue(node, 'no-entry')),
           });
         } else if (__DEV__) {
           const async = entryManager.findAttributeValue(node, 'async');

packages/utils/__tests__/runtimeUtils.spec.ts

@@ -49,8 +49,9 @@ describe('Garfish shared runtimeUtils', () => {
   });
 
   it('toBoolean', () => {
+    expect(toBoolean('')).toBe(true);
     expect(toBoolean('false')).toBe(false);
-    [0, 1, '', 'a', , false, true, null, undefined, {}, () => {}].forEach(
+    [0, 1, 'a', false, true, null, undefined, {}, () => {}].forEach(
       (val) => {
         expect(toBoolean(val)).toBe(Boolean(val));
       },

packages/utils/src/utils.ts

@@ -163,7 +163,9 @@ export function assert(condition: any, msg?: string | Error) {
 }
 
 export function toBoolean(val: any) {
-  return val === 'false' ? false : Boolean(val);
+  if (val === '') return true;
+  if (val === 'false') return false;
+  return Boolean(val);
 }
 
 export function remove<T>(list: Array<T> | Set<T>, el: T) {