feat(wrangler): Drop worker prefix when creating KV namespaces

.changeset/serious-teachers-marry.md

@@ -0,0 +1,8 @@
+---
+"wrangler": major
+---
+
+Remove worker name prefix from KV namespace create
+
+BREAKING CHANGE: when running `wrangler kv namespace create <name>`, previously the name of the namespace was automatically prefixed with the worker title, or `worker-` when running outside the context of a worker.
+After this change, KV namespaces will no longer get prefixed, and the name used is the name supplied, verbatim.

packages/wrangler/src/__tests__/__snapshots__/kv.test.ts.snap

@@ -1,7 +1,7 @@
 // Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
 exports[`wrangler > kv namespace > create > wrangler.json > should create a namespace 1`] = `
-"🌀 Creating namespace with title \\"worker-UnitTestNamespace\\"
+"🌀 Creating namespace with title \\"UnitTestNamespace\\"
 ✨ Success!
 Add the following to your configuration file in your kv_namespaces array:
 {
@@ -15,7 +15,7 @@ Add the following to your configuration file in your kv_namespaces array:
 `;
 
 exports[`wrangler > kv namespace > create > wrangler.json > should create a namespace in an environment if configured to do so 1`] = `
-"🌀 Creating namespace with title \\"worker-customEnv-UnitTestNamespace\\"
+"🌀 Creating namespace with title \\"customEnv-UnitTestNamespace\\"
 ✨ Success!
 Add the following to your configuration file in your kv_namespaces array under [env.customEnv]:
 {
@@ -29,7 +29,7 @@ Add the following to your configuration file in your kv_namespaces array under [
 `;
 
 exports[`wrangler > kv namespace > create > wrangler.json > should create a namespace using configured worker name 1`] = `
-"🌀 Creating namespace with title \\"other-worker-UnitTestNamespace\\"
+"🌀 Creating namespace with title \\"UnitTestNamespace\\"
 ✨ Success!
 Add the following to your configuration file in your kv_namespaces array:
 {
@@ -43,7 +43,7 @@ Add the following to your configuration file in your kv_namespaces array:
 `;
 
 exports[`wrangler > kv namespace > create > wrangler.json > should create a preview namespace if configured to do so 1`] = `
-"🌀 Creating namespace with title \\"worker-UnitTestNamespace_preview\\"
+"🌀 Creating namespace with title \\"UnitTestNamespace_preview\\"
 ✨ Success!
 Add the following to your configuration file in your kv_namespaces array:
 {
@@ -57,7 +57,7 @@ Add the following to your configuration file in your kv_namespaces array:
 `;
 
 exports[`wrangler > kv namespace > create > wrangler.toml > should create a namespace 1`] = `
-"🌀 Creating namespace with title \\"worker-UnitTestNamespace\\"
+"🌀 Creating namespace with title \\"UnitTestNamespace\\"
 ✨ Success!
 Add the following to your configuration file in your kv_namespaces array:
 [[kv_namespaces]]
@@ -67,7 +67,7 @@ id = \\"some-namespace-id\\"
 `;
 
 exports[`wrangler > kv namespace > create > wrangler.toml > should create a namespace in an environment if configured to do so 1`] = `
-"🌀 Creating namespace with title \\"worker-customEnv-UnitTestNamespace\\"
+"🌀 Creating namespace with title \\"customEnv-UnitTestNamespace\\"
 ✨ Success!
 Add the following to your configuration file in your kv_namespaces array under [env.customEnv]:
 [[kv_namespaces]]
@@ -77,7 +77,7 @@ id = \\"some-namespace-id\\"
 `;
 
 exports[`wrangler > kv namespace > create > wrangler.toml > should create a namespace using configured worker name 1`] = `
-"🌀 Creating namespace with title \\"other-worker-UnitTestNamespace\\"
+"🌀 Creating namespace with title \\"UnitTestNamespace\\"
 ✨ Success!
 Add the following to your configuration file in your kv_namespaces array:
 [[kv_namespaces]]
@@ -87,7 +87,7 @@ id = \\"some-namespace-id\\"
 `;
 
 exports[`wrangler > kv namespace > create > wrangler.toml > should create a preview namespace if configured to do so 1`] = `
-"🌀 Creating namespace with title \\"worker-UnitTestNamespace_preview\\"
+"🌀 Creating namespace with title \\"UnitTestNamespace_preview\\"
 ✨ Success!
 Add the following to your configuration file in your kv_namespaces array:
 [[kv_namespaces]]

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

@@ -189,23 +189,23 @@ describe("wrangler", () => {
 			describe.each(["wrangler.json", "wrangler.toml"])("%s", (configPath) => {
 				it("should create a namespace", async () => {
 					writeWranglerConfig({ name: "worker" }, configPath);
-					mockCreateRequest("worker-UnitTestNamespace");
+					mockCreateRequest("UnitTestNamespace");
 					await runWrangler("kv namespace create UnitTestNamespace");
 					expect(std.out).toMatchSnapshot();
 				});
 
 				it("should create a preview namespace if configured to do so", async () => {
 					writeWranglerConfig({ name: "worker" }, configPath);
 
-					mockCreateRequest("worker-UnitTestNamespace_preview");
+					mockCreateRequest("UnitTestNamespace_preview");
 					await runWrangler("kv namespace create UnitTestNamespace --preview");
 					expect(std.out).toMatchSnapshot();
 				});
 
 				it("should create a namespace using configured worker name", async () => {
 					writeWranglerConfig({ name: "other-worker" }, configPath);
 
-					mockCreateRequest("other-worker-UnitTestNamespace");
+					mockCreateRequest("UnitTestNamespace");
 					await runWrangler("kv namespace create UnitTestNamespace");
 					expect(std.out).toMatchSnapshot();
 				});
@@ -223,7 +223,7 @@ describe("wrangler", () => {
 						configPath
 					);
 
-					mockCreateRequest("worker-customEnv-UnitTestNamespace");
+					mockCreateRequest("customEnv-UnitTestNamespace");
 					await runWrangler(
 						"kv namespace create UnitTestNamespace --env customEnv"
 					);

packages/wrangler/src/kv/index.ts

@@ -88,16 +88,9 @@ defineCommand({
 
 	async handler(args) {
 		const config = readConfig(args.config, args);
-		if (!config.name) {
-			logger.warn(
-				"No configured name present, using `worker` as a prefix for the title"
-			);
-		}
-
-		const name = config.name || "worker";
-		const environment = args.env ? `-${args.env}` : "";
+		const environment = args.env ? `${args.env}-` : "";
 		const preview = args.preview ? "_preview" : "";
-		const title = `${name}${environment}-${args.namespace}${preview}`;
+		const title = `${environment}${args.namespace}${preview}`;
 
 		const accountId = await requireAuth(config);