Prevent spaces in names (#5738)

* Prevent spaces in names

* Create few-icons-refuse.md

* fix test

.changeset/few-icons-refuse.md

@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+fix: Prevent spaces in names when validating

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

@@ -1207,7 +1207,7 @@ describe("normalizeAndValidateConfig()", () => {
 		describe("name", () => {
 			it("should error on invalid `name` value with spaces", () => {
 				const expectedConfig: RawEnvironment = {
-					name: "NCC 1701 D",
+					name: "this has spaces",
 				} as unknown as RawEnvironment;
 
 				const { config, diagnostics } = normalizeAndValidateConfig(
@@ -1218,10 +1218,11 @@ describe("normalizeAndValidateConfig()", () => {
 
 				expect(config).toEqual(expect.objectContaining(expectedConfig));
 				expect(diagnostics.hasWarnings()).toBe(false);
+
 				expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
-			          "Processing wrangler configuration:
-			            - Expected \\"name\\" to be of type string, alphanumeric and lowercase with dashes only but got \\"NCC 1701 D\\"."
-		        `);
+					"Processing wrangler configuration:
+					  - Expected \\"name\\" to be of type string, alphanumeric and lowercase with dashes only but got \\"this has spaces\\"."
+				`);
 			});
 
 			it("should be valid `name` with underscores", () => {

packages/wrangler/src/config/validation-helpers.ts

@@ -257,7 +257,7 @@ export const isNonEmptyString: ValidatorFn = (
  */
 export const isValidName: ValidatorFn = (diagnostics, field, value) => {
 	if (
-		(typeof value === "string" && /^$|^[a-z0-9_ ][a-z0-9-_ ]*$/.test(value)) ||
+		(typeof value === "string" && /^$|^[a-z0-9_][a-z0-9-_]*$/.test(value)) ||
 		value === undefined
 	) {
 		return true;