remove --polyfill-node / config.polyfill_node (#100)

* remove `polyfill-node` / `polyfill_node` The quality of the polyfills is quite bad, so I'm removing this implementation (introduced in #72) * add a changeset * Update package-lock.json
cloudflare · Dec 13, 2021 · 2f760f5 · 2f760f5
1 parent 7858ca2
commit 2f760f5
Show file tree

Hide file tree

Showing 9 changed files with 10 additions and 231 deletions.
diff --git a/.changeset/neat-timers-search.md b/.changeset/neat-timers-search.md
@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+remove `--polyfill-node`
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/wrangler/package.json b/packages/wrangler/package.json
@@ -40,9 +40,7 @@
     "esbuild": "0.14.1",
     "miniflare": "2.0.0-rc.3",
     "semiver": "^1.1.0",
-    "serve": "^13.0.2",
-    "@esbuild-plugins/node-globals-polyfill": "^0.1.1",
-    "@esbuild-plugins/node-modules-polyfill": "^0.1.2"
+    "serve": "^13.0.2"
   },
   "optionalDependencies": {
     "fsevents": "~2.3.2"

diff --git a/packages/wrangler/scripts/bundle.ts b/packages/wrangler/scripts/bundle.ts
@@ -12,14 +12,7 @@ async function run() {
     platform: "node",
     format: "cjs",
     // minify: true, // TODO: enable this again
-    external: [
-      "fsevents",
-      "esbuild",
-      "miniflare",
-      "@miniflare/core",
-      "@esbuild-plugins/node-globals-polyfill",
-      "@esbuild-plugins/node-modules-polyfill",
-    ], // todo - bundle miniflare too
+    external: ["fsevents", "esbuild", "miniflare", "@miniflare/core"], // todo - bundle miniflare too
     sourcemap: true,
     inject: [path.join(__dirname, "../import_meta_url.js")],
     define: {

diff --git a/packages/wrangler/src/config.ts b/packages/wrangler/src/config.ts
@@ -81,7 +81,6 @@ type Env = {
   site?: Site;
   jsx_factory?: string; // inherited
   jsx_fragment?: string; // inherited
-  polyfill_node?: boolean; //inherited
   // we should use typescript to parse cron patterns
   triggers?: { crons: Cron[] }; // inherited
   vars?: Vars;
@@ -108,7 +107,6 @@ export type Config = {
   webpack_config?: string; // inherited
   jsx_factory?: string; // inherited
   jsx_fragment?: string; // inherited
-  polyfill_node?: boolean; //inherited
   vars?: Vars;
   migrations?: DOMigration[];
   durable_objects?: { bindings: DurableObject[] };

diff --git a/packages/wrangler/src/dev.tsx b/packages/wrangler/src/dev.tsx
@@ -22,8 +22,6 @@ import commandExists from "command-exists";
 import assert from "assert";
 import { getAPIToken } from "./user";
 import fetch from "node-fetch";
-import NodeModulesPolyfills from "@esbuild-plugins/node-modules-polyfill";
-import NodeGlobalsPolyfills from "@esbuild-plugins/node-globals-polyfill";
 
 type CfScriptFormat = void | "modules" | "service-worker";
 
@@ -42,7 +40,6 @@ type Props = {
   compatibilityDate: void | string;
   compatibilityFlags: void | string[];
   usageModel: void | "bundled" | "unbound";
-  polyfillNode: void | boolean;
 };
 
 export function Dev(props: Props): JSX.Element {
@@ -61,7 +58,6 @@ export function Dev(props: Props): JSX.Element {
     staticRoot: props.public,
     jsxFactory: props.jsxFactory,
     jsxFragment: props.jsxFragment,
-    polyfillNode: props.polyfillNode,
   });
   if (bundle && bundle.type === "commonjs" && !props.format && props.public) {
     throw new Error(
@@ -355,26 +351,15 @@ function useEsbuild(props: {
   staticRoot: void | string;
   jsxFactory: string | void;
   jsxFragment: string | void;
-  polyfillNode: boolean | void;
 }): EsbuildBundle | void {
-  const {
-    entry,
-    destination,
-    staticRoot,
-    jsxFactory,
-    jsxFragment,
-    polyfillNode,
-  } = props;
+  const { entry, destination, staticRoot, jsxFactory, jsxFragment } = props;
   const [bundle, setBundle] = useState<EsbuildBundle>();
   useEffect(() => {
     let result: esbuild.BuildResult;
     async function build() {
       if (!destination) return;
       result = await esbuild.build({
         entryPoints: [entry],
-        define: {
-          ...(polyfillNode && { global: "globalThis" }),
-        },
         bundle: true,
         outdir: destination,
         metafile: true,
@@ -383,9 +368,6 @@ function useEsbuild(props: {
         loader: {
           ".js": "jsx",
         },
-        plugins: polyfillNode
-          ? [NodeGlobalsPolyfills({ buffer: true }), NodeModulesPolyfills()]
-          : undefined,
         ...(jsxFactory && { jsxFactory }),
         ...(jsxFragment && { jsxFragment }),
         external: ["__STATIC_CONTENT_MANIFEST"],
@@ -418,7 +400,7 @@ function useEsbuild(props: {
     return () => {
       result?.stop();
     };
-  }, [entry, destination, staticRoot, jsxFactory, jsxFragment, polyfillNode]);
+  }, [entry, destination, staticRoot, jsxFactory, jsxFragment]);
   return bundle;
 }
 

diff --git a/packages/wrangler/src/dialogs.tsx b/packages/wrangler/src/dialogs.tsx
@@ -43,7 +43,7 @@ type PromptProps = {
   onSubmit: (text: string) => void;
 };
 
-export function Prompt(props: PromptProps) {
+function Prompt(props: PromptProps) {
   const [value, setValue] = useState("");
 
   return (