fixup! add loose-vars option to wrangler types

rename `loose-vars` to `strict-vars`

Author: dario-piotrowicz

.changeset/unlucky-timers-sort.md

@@ -2,12 +2,12 @@
 "wrangler": minor
 ---
 
-add `loose-vars` option to `wrangler types`
+add `strict-vars` option to `wrangler types`
 
-add a new `--loose-vars` option to `wrangler types` that developers can use to get more
-loose types for their variables instead of literal and union types
+add a new `--strict-vars` option to `wrangler types` that developers can use to enable/disable
+more strict/literal types for their variables
 
-these more loose types can be useful when developers change often their `vars` values,
+opting out of strict variables can be useful when developers change often their `vars` values,
 even more so when multiple environments are involved
 
 ## Example
@@ -33,7 +33,7 @@ interface Env {
 }
 ```
 
-while `wrangler types --loose-vars` would instead generate:
+while `wrangler types --strict-vars=false` would instead generate:
 
 ```
 interface Env {

packages/wrangler/src/__tests__/type-generation.test.ts

@@ -602,7 +602,7 @@ describe("generateTypes()", () => {
 	`);
 	});
 
-	it("should respect the loose-vars option", async () => {
+	it("should allow opting out of strict-vars", async () => {
 		fs.writeFileSync(
 			"./wrangler.toml",
 			TOML.stringify({
@@ -616,7 +616,7 @@ describe("generateTypes()", () => {
 			"utf-8"
 		);
 
-		await runWrangler("types --loose-vars");
+		await runWrangler("types --strict-vars=false");
 
 		expect(std.out).toMatchInlineSnapshot(`
 		"Generating project types...
@@ -710,8 +710,8 @@ describe("generateTypes()", () => {
 		`);
 		});
 
-		it("should produce loose types for variables (with --loose-vars)", async () => {
-			await runWrangler("types --loose-vars=true");
+		it("should produce non-strict types for variables (with --strict-vars=false)", async () => {
+			await runWrangler("types --strict-vars=false");
 
 			expect(std.out).toMatchInlineSnapshot(`
 			"Generating project types...

packages/wrangler/src/type-generation/index.ts

@@ -39,11 +39,10 @@ export function typesOptions(yargs: CommonYargsArgv) {
 			describe: "The path of the generated runtime types file",
 			demandOption: false,
 		})
-		.option("loose-vars", {
+		.option("strict-vars", {
 			type: "boolean",
-			default: false,
-			describe:
-				'Generate "loose"/generic types instead of literal and union types for variables',
+			default: true,
+			describe: "Generate literal and union types for variables",
 		});
 }
 
@@ -154,7 +153,7 @@ export async function typesHandler(
 		config,
 		envInterface,
 		outputPath,
-		args.looseVars
+		args.strictVars
 	);
 }
 
@@ -253,7 +252,7 @@ async function generateTypes(
 	config: Config,
 	envInterface: string,
 	outputPath: string,
-	looseVars: boolean
+	strictVars: boolean
 ) {
 	const configContainsEntrypoint =
 		config.main !== undefined || !!config.site?.["entry-point"];
@@ -304,7 +303,7 @@ async function generateTypes(
 				varInfo
 					.map(({ value }) => value)
 					.map((varValue) => {
-						if (looseVars) {
+						if (!strictVars) {
 							if (Array.isArray(varValue)) {
 								const typesInArray = [
 									...new Set(varValue.map((item) => typeof item)),