Deprecate the url patch

No more needed after wrangler 3.77+

Author: vicb

TODO.txt

@@ -2,14 +2,14 @@
 
 ## TODO
 
-- wrangler alias
 - move the wrangler.toml to the templates
-- dependency graph
 
 DONE:
 
 - figure out the assets
 - copy the template folders
+- remove the need for wrangler aliases
+- remove url patch - unblocked by wrangler 3.77+
 
 ## Install:
 
@@ -30,8 +30,6 @@ const nextConfig = {
 export default nextConfig;
 ```
 
-- add "node-url": "npm:url@^0.11.4" to the package.json
-
 - add a wrangler.toml int the generated app
 
 ```

builder/src/build/build-worker/index.ts

@@ -4,7 +4,7 @@ import { readdirSync, readFileSync, writeFileSync } from "node:fs";
 import { cp, readFile, writeFile } from "node:fs/promises";
 
 import { patchRequire } from "./patches/investigated/patchRequire";
-import { patchUrl } from "./patches/investigated/patchUrl";
+import { patchUrl } from "./patches/deprecated/patchUrl";
 import { copyTemplates } from "./patches/investigated/copyTemplates";
 
 import { patchReadFile } from "./patches/to-investigate/patchReadFile";
@@ -126,7 +126,6 @@ async function updateWorkerBundledCode(
 
 	patchedCode = patchRequire(patchedCode);
 	patchedCode = patchReadFile(patchedCode, nextjsAppPaths);
-	patchedCode = patchUrl(patchedCode);
 	patchedCode = inlineNextRequire(patchedCode, nextjsAppPaths);
 	patchedCode = patchFindDir(patchedCode, nextjsAppPaths);
 	patchedCode = inlineEvalManifest(patchedCode, nextjsAppPaths);

builder/src/build/build-worker/patches/investigated/patchUrl.ts renamed to builder/src/build/build-worker/patches/deprecated/patchUrl.ts

@@ -1,4 +1,6 @@
 /**
+ * No more needed with wrangler 3.77+
+ *
  * This solves the fact that the workerd URL parsing is not compatible with the node.js one
  * VERY IMPORTANT: this required the following dependency to be part of the application!!!! (this is very bad!!!)
  *    "node-url": "npm:url@^0.11.4"