cleanup

Author: kentcdodds

exercises/01.building-an-mvp/04.solution.implementation/.env

@@ -1,14 +1,2 @@
-# Example environment variables for local dev, test, and preview deploys.
-# Copy this to `.env` for local development and local tests.
-
-# Required
-COOKIE_SECRET=79c7d07db29567bc273192c55b1a613bfdc73097c134ef00573a372c51c2ae78
-
 # Optional (preview uses request origin by default)
 APP_BASE_URL=
-
-# Resend (optional; preview deploys overwrite these to point at the mock services)
-RESEND_API_BASE_URL=
-RESEND_API_KEY=
-RESEND_FROM_EMAIL=reset@epic-scheduler.dev
-

exercises/01.building-an-mvp/04.solution.implementation/.env.example

@@ -1,14 +1,6 @@
 # Example environment variables for local dev, test, and preview deploys.
 # Copy this to `.env` for local development and local tests.
 
-# Required
-COOKIE_SECRET=LOCAL_AND_PREVIEW_COOKIE_SECRET_32_CHARS_MINIMUM
-
 # Optional (preview uses request origin by default)
 APP_BASE_URL=
 
-# Resend (optional; preview deploys overwrite these to point at the mock services)
-RESEND_API_BASE_URL=
-RESEND_API_KEY=
-RESEND_FROM_EMAIL=reset@epic-scheduler.dev
-

exercises/01.building-an-mvp/04.solution.implementation/AGENTS.md

@@ -24,5 +24,4 @@ This file is intentionally brief. Detailed instructions live in focused docs:
 - Architecture references:
   - [docs/architecture/index.md](./docs/architecture/index.md)
   - [docs/architecture/request-lifecycle.md](./docs/architecture/request-lifecycle.md)
-  - [docs/architecture/authentication.md](./docs/architecture/authentication.md)
   - [docs/architecture/data-storage.md](./docs/architecture/data-storage.md)

exercises/01.building-an-mvp/04.solution.implementation/cli.ts

@@ -1,14 +1,10 @@
 import { spawn, type ChildProcess } from 'node:child_process'
-import { randomUUID } from 'node:crypto'
 import { platform } from 'node:os'
 import readline from 'node:readline'
 import { setTimeout as delay } from 'node:timers/promises'
 import getPort, { clearLockedPorts } from 'get-port'
 
 const defaultWorkerPort = 3742
-const defaultMockPort = 8788
-const mockReadyTimeoutMs = 10_000
-const mockReadyPollMs = 200
 
 const ansiReset = '\x1b[0m'
 const ansiBright = '\x1b[1m'
@@ -48,8 +44,6 @@ const extraArgs = process.argv.slice(2)
 let shutdown: (() => void) | null = null
 let devChildren: Array<ChildProcess> = []
 let workerOrigin = ''
-let mockResendProcess: ChildProcess | null = null
-let mockEnvOverrides: Record<string, string> = {}
 
 void startDev()
 
@@ -61,7 +55,7 @@ async function startDev() {
 	})
 	shutdown = setupShutdown(
 		() => devChildren,
-		() => mockResendProcess,
+		() => null,
 	)
 }
 
@@ -230,7 +224,6 @@ async function restartDev(
 	{ announce }: { announce: boolean } = { announce: true },
 ) {
 	await stopChildren(devChildren)
-	const mockEnv = await ensureMockServers()
 	const desiredPort = Number.parseInt(
 		process.env.PORT ?? String(defaultWorkerPort),
 		10,
@@ -253,7 +246,7 @@ async function restartDev(
 	const worker = runNpmScript(
 		'dev:worker',
 		extraArgs,
-		{ PORT: String(workerPort), ...mockEnv },
+		{ PORT: String(workerPort) },
 		{ outputFilter: 'worker' },
 	)
 	devChildren = [client, worker]
@@ -264,87 +257,6 @@ async function restartDev(
 	}
 }
 
-function hasEnvValue(value: string | undefined) {
-	return typeof value === 'string' && value.trim().length > 0
-}
-
-async function isMockReady(baseUrl: string) {
-	try {
-		const response = await fetch(`${baseUrl}/__mocks/meta`)
-		await response.body?.cancel()
-		return response.ok
-	} catch {
-		return false
-	}
-}
-
-async function waitForMockReady(baseUrl: string, child: ChildProcess) {
-	const start = Date.now()
-	while (Date.now() - start < mockReadyTimeoutMs) {
-		if (child.killed || child.exitCode !== null) {
-			return false
-		}
-		if (await isMockReady(baseUrl)) {
-			return true
-		}
-		await delay(mockReadyPollMs)
-	}
-	return false
-}
-
-async function ensureMockServers() {
-	if (
-		mockResendProcess &&
-		!mockResendProcess.killed &&
-		mockResendProcess.exitCode === null
-	) {
-		return mockEnvOverrides
-	}
-	const desiredPort = Number.parseInt(
-		process.env.MOCK_API_PORT ?? String(defaultMockPort),
-		10,
-	)
-	const portRange = Array.from(
-		{ length: 10 },
-		(_, index) => desiredPort + index,
-	)
-	const mockPort = await getPort({ port: portRange })
-	const baseUrl = `http://127.0.0.1:${mockPort}`
-	const apiToken = `mock-resend-${randomUUID()}`
-	const child = runNpmScript(
-		'dev:mock-resend',
-		[
-			'--port',
-			String(mockPort),
-			'--ip',
-			'127.0.0.1',
-			'--var',
-			`MOCK_API_TOKEN:${apiToken}`,
-		],
-		{},
-	)
-	mockResendProcess = child
-	child.once('exit', () => {
-		mockResendProcess = null
-	})
-	mockEnvOverrides = {
-		RESEND_API_BASE_URL: baseUrl,
-		RESEND_API_KEY: apiToken,
-	}
-	if (!hasEnvValue(process.env.RESEND_FROM_EMAIL)) {
-		mockEnvOverrides.RESEND_FROM_EMAIL = 'reset@epic-scheduler.dev'
-	}
-	const didStart = await waitForMockReady(baseUrl, child)
-	if (!didStart) {
-		console.warn(
-			`Mock API worker did not become ready within ${mockReadyTimeoutMs}ms.`,
-		)
-	}
-	console.log(dim(`Mock API worker running at ${baseUrl}`))
-	console.log(dim(`Resend mock base URL ${baseUrl}`))
-	return mockEnvOverrides
-}
-
 async function stopChildren(children: Array<ChildProcess>) {
 	await Promise.all(children.map((child) => stopChild(child)))
 }