treat "name" in `wrangler init [name] as a path (#877)

Treat the "name" parameter in wrangler init as a path.

This means that running wrangler init . will create a worker in the current directory,
and the worker's name will be the name of the current directory.

You can also run wrangler init path/to/my-worker and a worker will be created at
[CWD]/path/to/my-worker with the name my-worker,

.changeset/dry-walls-buy.md

@@ -0,0 +1,11 @@
+---
+"wrangler": patch
+---
+
+Treat the "name" parameter in `wrangler init` as a path.
+
+This means that running `wrangler init .` will create a worker in the current directory,
+and the worker's name will be the name of the current directory.
+
+You can also run `wrangler init path/to/my-worker` and a worker will be created at
+`[CWD]/path/to/my-worker` with the name `my-worker`,

packages/wrangler/src/__tests__/init.test.ts

@@ -1,5 +1,6 @@
 import * as fs from "node:fs";
 import * as fsp from "node:fs/promises";
+import path from "node:path";
 import * as TOML from "@iarna/toml";
 import { parseConfigFileTextToJson } from "typescript";
 import { version as wranglerVersion } from "../../package.json";
@@ -777,4 +778,56 @@ describe("init", () => {
       expect(fs.existsSync("./src/index.ts")).toBe(false);
     });
   });
+
+  describe("worker names", () => {
+    it("should create a worker with a given name", async () => {
+      await runWrangler("init my-worker -y");
+
+      const parsed = TOML.parse(
+        await fsp.readFile("./my-worker/wrangler.toml", "utf-8")
+      );
+
+      expect(typeof parsed.compatibility_date).toBe("string");
+      expect(parsed.name).toBe("my-worker");
+    });
+
+    it('should create a worker with the name of the current directory if "name" is .', async () => {
+      await runWrangler("init . -y");
+
+      const parsed = TOML.parse(await fsp.readFile("wrangler.toml", "utf-8"));
+
+      expect(typeof parsed.compatibility_date).toBe("string");
+      expect(parsed.name).toBe(path.basename(process.cwd()).toLowerCase());
+      expect(fs.existsSync("./my-worker/package.json")).toBe(false);
+      expect(fs.existsSync("./my-worker/tsconfig.json")).toBe(false);
+    });
+
+    it('should create a worker in a nested directory if "name" is path/to/worker', async () => {
+      await runWrangler("init path/to/worker -y");
+
+      const parsed = TOML.parse(
+        await fsp.readFile("path/to/worker/wrangler.toml", "utf-8")
+      );
+
+      expect(typeof parsed.compatibility_date).toBe("string");
+      expect(parsed.name).toBe("worker");
+      expect(fs.existsSync("./my-worker/package.json")).toBe(false);
+      expect(fs.existsSync("./my-worker/tsconfig.json")).toBe(false);
+    });
+
+    it("should normalize characters that aren't lowercase alphanumeric, underscores, or dashes", async () => {
+      await runWrangler("init WEIRD_w0rkr_N4m3.js.tsx.tar.gz -y");
+      const parsed = TOML.parse(
+        await fsp.readFile(
+          "WEIRD_w0rkr_N4m3.js.tsx.tar.gz/wrangler.toml",
+          "utf-8"
+        )
+      );
+
+      expect(typeof parsed.compatibility_date).toBe("string");
+      expect(parsed.name).toBe("weird_w0rkr_n4m3-js-tsx-tar-gz");
+      expect(fs.existsSync("./my-worker/package.json")).toBe(false);
+      expect(fs.existsSync("./my-worker/tsconfig.json")).toBe(false);
+    });
+  });
 });

packages/wrangler/src/index.tsx

@@ -311,6 +311,10 @@ export async function main(argv: string[]): Promise<void> {
       // TODO: make sure args.name is a valid identifier for a worker name
 
       const creationDirectory = path.join(process.cwd(), args.name ?? "");
+      const workerName = path
+        .basename(creationDirectory)
+        .toLowerCase()
+        .replaceAll(/[^a-z0-9\-_]/gm, "-");
 
       const packageManager = await getPackageManager(creationDirectory);
 
@@ -322,8 +326,7 @@ export async function main(argv: string[]): Promise<void> {
         "./wrangler.toml"
       );
       let justCreatedWranglerToml = false;
-      const workerName =
-        args.name || path.basename(path.resolve(process.cwd()));
+
       if (fs.existsSync(wranglerTomlDestination)) {
         logger.warn(`${wranglerTomlDestination} file already exists!`);
         const shouldContinue = await confirm(
@@ -335,6 +338,7 @@ export async function main(argv: string[]): Promise<void> {
       } else {
         await mkdir(creationDirectory, { recursive: true });
         const compatibilityDate = new Date().toISOString().substring(0, 10);
+
         try {
           await writeFile(
             wranglerTomlDestination,
@@ -343,6 +347,7 @@ export async function main(argv: string[]): Promise<void> {
               compatibility_date: compatibilityDate,
             }) + "\n"
           );
+
           logger.log(`✨ Successfully created wrangler.toml`);
           justCreatedWranglerToml = true;
         } catch (err) {
@@ -379,6 +384,7 @@ export async function main(argv: string[]): Promise<void> {
               "  "
             ) + "\n"
           );
+
           await packageManager.install();
           logger.log(`✨ Created package.json`);
           pathToPackageJson = path.join(creationDirectory, "package.json");