Add a TypeScript step to create-astro

.changeset/ninety-planets-work.md

@@ -0,0 +1,5 @@
+---
+'create-astro': minor
+---
+
+Add a step to configure how strict TypeScript should be

packages/create-astro/package.json

@@ -25,7 +25,8 @@
   },
   "files": [
     "dist",
-    "create-astro.js"
+    "create-astro.js",
+    "tsconfigs"
   ],
   "dependencies": {
     "chalk": "^5.0.1",

packages/create-astro/src/index.ts

@@ -7,6 +7,7 @@ import ora from 'ora';
 import os from 'os';
 import path from 'path';
 import prompts from 'prompts';
+import url from 'url';
 import detectPackageManager from 'which-pm-runs';
 import yargs from 'yargs-parser';
 import { loadWithRocketGradient, rocketAscii } from './gradient.js';
@@ -284,10 +285,61 @@ export async function main() {
 		ora().info(dim(`--dry-run enabled, skipping.`));
 	} else if (gitResponse.git) {
 		await execaCommand('git init', { cwd });
+		ora().succeed('Git repository created!');
 	} else {
 		ora().info(dim(`Sounds good! You can come back and run ${cyan(`git init`)} later.`));
 	}
 
+	const tsResponse = await prompts(
+		{
+			type: 'select',
+			name: 'typescript',
+			message:
+				'Astro is a TypeScript-first framework. How strict would you like the typechecking to be?',
+			choices: [
+				{
+					title: 'Relaxed',
+					description: "Not a big fan of TypeScript? No issues, we'll keep this kind",
+					value: 'default',
+				},
+				{
+					title: 'Strict (recommended)',
+					description: 'Will enable `strict`',
+					value: 'strict',
+				},
+				{
+					title: 'Stricter',
+					description: 'Will enable `strict` and some stricter settings',
+					value: 'stricter',
+				},
+			],
+		},
+		{
+			onCancel: () => {
+				ora().info(
+					dim(
+						'Operation cancelled. No worries, your project folder has already been created and TypeScript settings have been set to "Relaxed"'
+					)
+				);
+				process.exit(1);
+			},
+		}
+	);
+
+	if (args.dryRun) {
+		ora().info(dim(`--dry-run enabled, skipping.`));
+	} else if (tsResponse.typescript) {
+		fs.copyFileSync(
+			path.join(
+				url.fileURLToPath(new URL('..', import.meta.url)),
+				'tsconfigs',
+				`tsconfig.${tsResponse.typescript}.json`
+			),
+			path.join(cwd, 'tsconfig.json')
+		);
+		ora().succeed('TypeScript settings applied!');
+	}
+
 	ora().succeed('Setup complete.');
 	ora({ text: green('Ready for liftoff!') }).succeed();
 	await wait(300);

packages/create-astro/tsconfigs/tsconfig.strict.json

@@ -0,0 +1,19 @@
+{
+  "compilerOptions": {
+    // Enable top-level await, and other modern ESM features.
+    "target": "ESNext",
+    "module": "ESNext",
+    // Enable node-style module resolution, for things like npm package imports.
+    "moduleResolution": "node",
+    // Enable JSON imports.
+    "resolveJsonModule": true,
+    // Enable stricter transpilation for better output.
+    "isolatedModules": true,
+    // Astro will directly run your TypeScript code, no transpilation needed.
+    "noEmit": true,
+    // Enable strict type checking.
+    "strict": true,
+    // Error when a value import is only used as a type.
+    "importsNotUsedAsValues": "error"
+  }
+}

packages/create-astro/tsconfigs/tsconfig.stricter.json

@@ -0,0 +1,33 @@
+{
+  "compilerOptions": {
+    // Enable top-level await, and other modern ESM features.
+    "target": "ESNext",
+    "module": "ESNext",
+    // Enable node-style module resolution, for things like npm package imports.
+    "moduleResolution": "node",
+    // Enable JSON imports.
+    "resolveJsonModule": true,
+    // Enable stricter transpilation for better output.
+    "isolatedModules": true,
+    // Astro will directly run your TypeScript code, no transpilation needed.
+    "noEmit": true,
+    // Enable strict type checking.
+    "strict": true,
+    // Error when a value import is only used as a type.
+    "importsNotUsedAsValues": "error",
+    // Report errors for fallthrough cases in switch statements
+    "noFallthroughCasesInSwitch": true,
+    // Force functions designed to override their parent class to be specified as `override`.
+    "noImplicitOverride": true,
+    // Force functions to specify that they can return `undefined` if a possibe code path does not return a value.
+    "noImplicitReturns": true,
+    // Report an error when a variable is declared but never used.
+    "noUnusedLocals": true,
+    // Report an error when a parameter is declared but never used.
+    "noUnusedParameters": true,
+    // Force the usage of the indexed syntax to access fields declared using an index signature.
+    "noUncheckedIndexedAccess": true,
+    // Report an error when the value `undefined` is given to an optional property that doesn't specify `undefined` as a valid value.
+    "exactOptionalPropertyTypes": true
+  }
+}