@geekmidas/cli 1.10.36 → 1.10.37

package/docs/dev-server-resilience-design.md

@@ -0,0 +1,189 @@
+# Dev Server Resilience Design
+
+Status: parked / not yet started
+Owner: TBD
+Scope: `packages/cli/src/dev/index.ts`, `packages/cli/src/generators/Generator.ts`
+
+## Background
+
+Running `gkm dev` against a real workspace (e.g. `rezgo`: api, auth, web, app/Expo)
+exposes three classes of failure. None are caused by user code; they are
+limitations of the current dev orchestrator and hot-reload implementation.
+
+## Issue 1 — Hot reload misses new files and serves stale transitive modules
+
+### Symptoms
+
+- Adding a new route file does not trigger a rebuild.
+- After adding a new export to a non-route file, route files that import it
+  fail with:
+  `SyntaxError: The requested module '~/.../foo.ts' does not provide an export named 'bar'`
+- Some changes "just don't get picked up" until the dev server is restarted.
+
+### Root causes
+
+1. **`add`/`unlink` events are ignored.** The chokidar watcher in `dev/index.ts`
+   only listens for `change`:
+   ```ts
+   watcher.on('change', async (path) => { ... });
+   ```
+   New or deleted files never trigger a rebuild.
+
+2. **Cache-bust is shallow.** `Generator.ts:load()` appends `?t=<now>` only to
+   the top-level globbed route file URL. Node's ESM cache is keyed by URL, so
+   any *transitive* import (services, domain modules, env, etc.) is served from
+   cache forever within the CLI process. When a transitive module is edited,
+   the route file is re-evaluated against the stale cached version → the
+   "does not provide an export named" error above.
+
+3. **Watch patterns are too narrow.** Only `routes`, `functions`, `crons`,
+   `subscribers` (plus envParser / logger / hooks files) are watched. Edits to
+   `src/domain/**`, `src/services/**`, etc. don't fire any rebuild even though
+   those files are pulled in transitively at openapi/build time.
+
+### Proposed fix
+
+- Subscribe to `add` and `unlink` on the watcher and re-resolve the glob set
+  before rebuilding.
+- Broaden the watch pattern to cover the app's `src/**/*.{ts,tsx}` (configurable),
+  not just construct entry directories.
+- Bust transitive caches. Two viable approaches:
+  - **Worker thread per rebuild** (preferred): move `EndpointGenerator.load` /
+    `FunctionGenerator.load` / etc. into a short-lived `worker_threads` worker
+    that gets recreated on each rebuild. Each worker has a fresh module graph,
+    so transitive imports are guaranteed fresh. Cost is one worker spawn per
+    rebuild (~50-100ms), but avoids the unbounded-memory leak of accumulating
+    `?t=N` URLs in the main process.
+  - **Custom loader hook**: install a Node loader that rewrites every resolved
+    URL to include the current rebuild generation. Cheaper per rebuild but
+    fragile (loader compatibility with tsx, source maps, etc.).
+
+Worker thread is the recommended path.
+
+## Issue 2 — Ctrl+C orphans `tsx`/`node` processes
+
+### Symptoms
+
+After pressing Ctrl+C in a workspace `gkm dev`, the shell returns but
+`tsx` and `node` processes remain alive. Workaround is `killall node`.
+
+### Process chain in workspace mode
+
+```
+shell
+  └── gkm dev (top-level, workspace)
+        └── pnpm turbo run dev          [spawned with detached: true]
+              ├── @rezgo/api: gkm dev   (per-app)
+              │     └── tsx              [spawned with detached: true]
+              │           └── node       (Hono server)
+              ├── @rezgo/auth: ...
+              └── @rezgo/app: ...
+```
+
+### Root cause
+
+The per-app `gkm dev` spawns `tsx` with `detached: true`
+(`dev/index.ts:1521`). That moves tsx into a *new process group* outside the
+turbo subtree. The workspace shutdown handler only sends SIGTERM/SIGKILL to
+turbo's pgid; tsx's pgid is never targeted. After SIGKILL hits turbo's group,
+the per-app gkm processes die *before* their async shutdown finishes calling
+`process.kill(-tsxPgid, 'SIGKILL')`. tsx and its node child are orphaned.
+
+The `detached: true` flag exists because the user's running Hono server may
+ignore SIGTERM, so we need to kill its whole group. The design conflict is:
+detaching tsx is necessary to forcibly kill it, but it also escapes the
+top-level reachability tree.
+
+### Proposed fix
+
+Track every tsx PID centrally:
+
+- Each `DevServer` registers its `tsx` pid into a shared `Set<number>` at the
+  workspace level (or writes to a state file under `.gkm/`).
+- Workspace shutdown iterates that set and issues `process.kill(-pid, 'SIGKILL')`
+  for each known tsx pgid *before* falling back to the turbo-pgid SIGKILL.
+- As belt-and-suspenders, drain the registry on per-app `gkm dev` exit so dead
+  pids are not retargeted.
+
+This makes the kill reachable regardless of process-group escape.
+
+## Issue 3 — One app's crash brings down the whole workspace
+
+### Symptoms
+
+`@rezgo/app` (Expo) crashed with a Metro `DependencyGraph._onHasteChange`
+TypeError. Within seconds, `@rezgo/api`, `@rezgo/auth`, `@rezgo/web` all
+received SIGTERM and shut down. The user lost 21 minutes of warmed dev state.
+
+### Root causes
+
+1. **Turbo's persistent-task semantics.** When a persistent task in
+   `turbo run dev` exits non-zero, turbo cancels sibling persistent tasks. There
+   is no flag (as of turbo 2.x) that keeps survivors running through a sibling
+   crash. `--continue` only affects scheduling of *queued* tasks, not running
+   persistent ones.
+2. **gkm's orchestrator treats turbo's non-zero exit as fatal.** In
+   `workspaceDevCommand`:
+   ```ts
+   turboProcess.on('exit', (code) => {
+     if (code !== null && code !== 0) {
+       reject(new Error(`Turbo exited with code ${code}`));
+     }
+   });
+   ```
+   The reject fires the shutdown handler, which kills any survivors that
+   turbo hadn't already torn down.
+
+### Proposed fix
+
+Stop using `turbo run dev` for the persistent dev loop. Build an in-process
+supervisor inside `workspaceDevCommand`:
+
+- Resolve dependency order via the existing `getAppBuildOrder` (we still use
+  turbo's task graph indirectly via workspace config).
+- For each app, `spawn` its dev command directly (`pnpm dev` in that app's
+  cwd, or for backend apps invoke the per-app gkm dev codepath directly without
+  going through pnpm/turbo).
+- Supervise each child:
+  - `on('exit', code)`: log the crash, do **not** propagate, schedule a restart
+    with exponential backoff (start at 1s, cap at 30s, reset after a 60s clean
+    run).
+  - Stream stdout/stderr to the parent with an app-name prefix so output stays
+    readable.
+- Centralize the PID registry from Issue 2 here.
+- Ctrl+C handler: iterate registry, SIGKILL each child group, then exit.
+
+Turbo remains the build-time orchestrator (`gkm build`, dependency-ordered
+package builds). Only the dev persistent loop is moved off it.
+
+### Why not fix turbo's behavior instead
+
+Tried first. `--continue=dependencies-successful` and friends don't apply to
+sibling persistent tasks. Filing it upstream is fine, but the dev experience
+needs to be fixed now and the supervisor pattern is also what makes Issues 1
+and 2 solvable cleanly.
+
+## Implementation order (when unparked)
+
+1. Land the in-process supervisor (Issue 3 fix). This gives us a single owner
+   of the dev process tree.
+2. Move tsx PID registration into the supervisor (Issue 2 fix).
+3. Move hot-reload (worker thread + broadened watcher + `add`/`unlink`) into
+   the per-app branch of the supervisor (Issue 1 fix).
+
+Each step is shippable independently. Order matters: 1 unblocks the cleanest
+form of 2 and 3.
+
+## Out of scope
+
+- Replacing `tsx` (e.g. with `node --experimental-strip-types`).
+- Native Bun runtime for dev — separate decision.
+- Replacing chokidar.
+
+## Open questions
+
+- Should the supervisor expose a `--no-restart` flag for CI-like local runs?
+- Restart backoff thresholds — should they be configurable per app in
+  workspace config, or fixed?
+- Should we surface a small status line ("3 of 4 apps running") when one app
+  is dead but the rest are healthy?

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geekmidas/cli",
-  "version": "1.10.36",
+  "version": "1.10.37",
   "description": "CLI tools for building Lambda handlers, server applications, and generating OpenAPI specs",
   "private": false,
   "type": "module",
@@ -56,11 +56,11 @@
     "prompts": "~2.4.2",
     "tsx": "~4.20.3",
     "yaml": "~2.8.2",
-    "@geekmidas/constructs": "~3.0.9",
-    "@geekmidas/errors": "~1.0.0",
-    "@geekmidas/schema": "~1.0.0",
+    "@geekmidas/constructs": "~3.0.10",
     "@geekmidas/envkit": "~1.0.6",
-    "@geekmidas/logger": "~1.0.1"
+    "@geekmidas/errors": "~1.0.0",
+    "@geekmidas/logger": "~1.0.1",
+    "@geekmidas/schema": "~1.0.1"
   },
   "devDependencies": {
     "@types/lodash.kebabcase": "^4.1.9",

package/src/build/__tests__/workspace-build.spec.ts

@@ -103,13 +103,13 @@ describe('Workspace Build Command', () => {
 		it('should have correct structure for failed build', () => {
 			const result: AppBuildResult = {
 				appName: 'web',
-				type: 'frontend',
+				type: 'web',
 				success: false,
 				error: 'Build failed',
 			};
 
 			expect(result.appName).toBe('web');
-			expect(result.type).toBe('frontend');
+			expect(result.type).toBe('web');
 			expect(result.success).toBe(false);
 			expect(result.error).toBe('Build failed');
 			expect(result.outputPath).toBeUndefined();
@@ -137,14 +137,14 @@ describe('Workspace Build Command', () => {
 						routes: './src/**/*.ts',
 					},
 					web: {
-						type: 'frontend',
+						type: 'web',
 						path: 'apps/web',
 						port: 3002,
 						dependencies: ['api', 'auth'],
 						framework: 'nextjs',
 					},
 					admin: {
-						type: 'frontend',
+						type: 'web',
 						path: 'apps/admin',
 						port: 3003,
 						dependencies: ['api'],
@@ -159,7 +159,7 @@ describe('Workspace Build Command', () => {
 
 			const apps = Object.entries(workspace.apps);
 			const backendApps = apps.filter(([, app]) => app.type === 'backend');
-			const frontendApps = apps.filter(([, app]) => app.type === 'frontend');
+			const frontendApps = apps.filter(([, app]) => app.type === 'web');
 
 			expect(backendApps).toHaveLength(2);
 			expect(frontendApps).toHaveLength(2);
@@ -181,7 +181,7 @@ describe('Workspace Build Command', () => {
 						dependencies: [],
 					},
 					web: {
-						type: 'frontend',
+						type: 'web',
 						path: 'apps/web',
 						port: 3001,
 						dependencies: [],
@@ -207,7 +207,7 @@ describe('Workspace Build Command', () => {
 			const frontendOutputPath = join(
 				workspace.root,
 				frontendApp.path,
-				frontendApp.type === 'frontend' ? '.next' : '.gkm',
+				frontendApp.type === 'web' ? '.next' : '.gkm',
 			);
 			expect(frontendOutputPath).toBe('/workspace/apps/web/.next');
 		});

package/src/build/index.ts

@@ -362,7 +362,7 @@ async function buildForProvider(
  */
 export interface AppBuildResult {
 	appName: string;
-	type: 'backend' | 'frontend';
+	type: 'backend' | 'web' | 'mobile';
 	success: boolean;
 	outputPath?: string;
 	error?: string;
@@ -415,7 +415,7 @@ export async function workspaceBuildCommand(
 	const results: AppBuildResult[] = [];
 	const apps = Object.entries(workspace.apps);
 	const backendApps = apps.filter(([, app]) => app.type === 'backend');
-	const frontendApps = apps.filter(([, app]) => app.type === 'frontend');
+	const frontendApps = apps.filter(([, app]) => app.type === 'web');
 
 	logger.log(`\n🏗️  Building workspace: ${workspace.name}`);
 	logger.log(
@@ -486,7 +486,12 @@ export async function workspaceBuildCommand(
 		// Summary
 		logger.log(`\n📋 Build Summary:`);
 		for (const result of results) {
-			const icon = result.type === 'backend' ? '⚙️' : '🌐';
+			const icon =
+				result.type === 'backend'
+					? '⚙️'
+					: result.type === 'mobile'
+						? '📱'
+						: '🌐';
 			logger.log(
 				`   ${icon} ${result.appName}: ${result.outputPath || 'built'}`,
 			);
@@ -522,13 +527,24 @@ function getAppOutputPath(
 ): string {
 	const appPath = join(workspace.root, app.path);
 
-	if (app.type === 'frontend') {
-		// Next.js standalone output
-		return join(appPath, '.next');
-	} else {
-		// Backend .gkm output
-		return join(appPath, '.gkm');
+	if (app.type === 'mobile') {
+		// Mobile builds are produced by the framework's own toolchain
+		// (e.g. EAS Build) — no local output path.
+		return '';
+	}
+	if (app.type === 'web') {
+		switch (app.framework) {
+			case 'vite':
+			case 'tanstack-start':
+				return join(appPath, 'dist');
+			case 'remix':
+				return join(appPath, 'build');
+			default:
+				return join(appPath, '.next');
+		}
 	}
+	// Backend .gkm output
+	return join(appPath, '.gkm');
 }
 
 /**

package/src/credentials/__tests__/helpers.ts

@@ -76,7 +76,7 @@ export function createWorkspaceConfig(
 			const deps = app.dependencies
 				? `dependencies: [${app.dependencies.map((d) => `'${d}'`).join(', ')}],`
 				: '';
-			if (app.type === 'frontend') {
+			if (app.type === 'web') {
 				const framework = app.framework ?? 'nextjs';
 				return `    ${name}: {
       type: '${app.type}',

package/src/credentials/__tests__/workspaceCredentials.spec.ts

@@ -45,7 +45,7 @@ describe('workspace credentials', () => {
 				{
 					api: { type: 'backend', path: 'apps/api', port: 3001 },
 					web: {
-						type: 'frontend',
+						type: 'web',
 						path: 'apps/web',
 						port: 3002,
 						dependencies: ['api'],

package/src/deploy/__tests__/domain.spec.ts

@@ -55,7 +55,7 @@ describe('resolveHost', () => {
 	});
 
 	it('should return base domain for main frontend app', () => {
-		const app = createApp({ type: 'frontend' });
+		const app = createApp({ type: 'web' });
 		const host = resolveHost('web', app, 'production', dokployConfig, true);
 		expect(host).toBe('myapp.com');
 	});
@@ -100,7 +100,7 @@ describe('resolveHost', () => {
 });
 
 describe('isMainFrontendApp', () => {
-	const createApp = (type: 'backend' | 'frontend'): NormalizedAppConfig => ({
+	const createApp = (type: 'backend' | 'web'): NormalizedAppConfig => ({
 		type,
 		path: 'apps/test',
 		port: 3000,
@@ -111,7 +111,7 @@ describe('isMainFrontendApp', () => {
 	it('should return false for backend apps', () => {
 		const apps = {
 			api: createApp('backend'),
-			web: createApp('frontend'),
+			web: createApp('web'),
 		};
 		expect(isMainFrontendApp('api', apps.api, apps)).toBe(false);
 	});
@@ -119,8 +119,8 @@ describe('isMainFrontendApp', () => {
 	it('should return true for app named "web" if it is frontend', () => {
 		const apps = {
 			api: createApp('backend'),
-			web: createApp('frontend'),
-			admin: createApp('frontend'),
+			web: createApp('web'),
+			admin: createApp('web'),
 		};
 		expect(isMainFrontendApp('web', apps.web, apps)).toBe(true);
 	});
@@ -128,8 +128,8 @@ describe('isMainFrontendApp', () => {
 	it('should return true for first frontend app when no "web" app', () => {
 		const apps = {
 			api: createApp('backend'),
-			dashboard: createApp('frontend'),
-			admin: createApp('frontend'),
+			dashboard: createApp('web'),
+			admin: createApp('web'),
 		};
 		expect(isMainFrontendApp('dashboard', apps.dashboard, apps)).toBe(true);
 		expect(isMainFrontendApp('admin', apps.admin, apps)).toBe(false);
@@ -138,22 +138,22 @@ describe('isMainFrontendApp', () => {
 	it('should return false for non-first frontend when no "web" app', () => {
 		const apps = {
 			api: createApp('backend'),
-			dashboard: createApp('frontend'),
-			admin: createApp('frontend'),
+			dashboard: createApp('web'),
+			admin: createApp('web'),
 		};
 		expect(isMainFrontendApp('admin', apps.admin, apps)).toBe(false);
 	});
 
 	it('should return false when no frontend apps in allApps (edge case)', () => {
 		// Edge case: checking a frontend app against an empty allApps
-		const frontendApp = createApp('frontend');
+		const frontendApp = createApp('web');
 		expect(isMainFrontendApp('myapp', frontendApp, {})).toBe(false);
 	});
 });
 
 describe('generatePublicUrlBuildArgs', () => {
 	const createApp = (dependencies: string[]): NormalizedAppConfig => ({
-		type: 'frontend',
+		type: 'web',
 		path: 'apps/web',
 		port: 3001,
 		dependencies,
@@ -214,7 +214,7 @@ describe('generatePublicUrlBuildArgs', () => {
 
 describe('getPublicUrlArgNames', () => {
 	const createApp = (dependencies: string[]): NormalizedAppConfig => ({
-		type: 'frontend',
+		type: 'web',
 		path: 'apps/web',
 		port: 3001,
 		dependencies,

package/src/deploy/__tests__/env-resolver.spec.ts

@@ -738,7 +738,7 @@ describe('Docker build arg extraction', () => {
 		overrides: Partial<EnvResolverContext> = {},
 	): EnvResolverContext => ({
 		app: {
-			type: 'frontend',
+			type: 'web',
 			path: 'apps/web',
 			port: 3001,
 			dependencies: ['api', 'auth'],

package/src/deploy/__tests__/index.spec.ts

@@ -430,7 +430,7 @@ describe('workspaceDeployCommand', () => {
 					resolvedDeployTarget: 'dokploy',
 				},
 				web: {
-					type: 'frontend',
+					type: 'web',
 					path: 'apps/web',
 					port: 3001,
 					dependencies: ['api'],
@@ -527,7 +527,7 @@ describe('workspaceDeployCommand', () => {
 						resolvedDeployTarget: 'dokploy',
 					},
 					web: {
-						type: 'frontend',
+						type: 'web',
 						path: 'apps/web',
 						port: 3002,
 						dependencies: ['api', 'auth'],
@@ -566,7 +566,7 @@ describe('workspaceDeployCommand', () => {
 						resolvedDeployTarget: 'dokploy',
 					},
 					web: {
-						type: 'frontend',
+						type: 'web',
 						path: 'apps/web',
 						port: 3002,
 						dependencies: ['api'],
@@ -614,7 +614,7 @@ describe('workspaceDeployCommand', () => {
 			};
 
 			const app = {
-				type: 'frontend' as const,
+				type: 'web' as const,
 				path: 'apps/web',
 				port: 3000,
 				dependencies: ['api', 'auth'],
@@ -645,7 +645,7 @@ describe('workspaceDeployCommand', () => {
 			};
 
 			const app = {
-				type: 'frontend' as const,
+				type: 'web' as const,
 				path: 'apps/web',
 				port: 3000,
 				dependencies: ['api', 'auth'], // wants both api and auth
@@ -719,7 +719,7 @@ describe('workspaceDeployCommand', () => {
 		it('should inject DATABASE_URL for backend apps', () => {
 			const workspaceName = 'test-workspace';
 			const hasPostgres = true;
-			const appType: 'backend' | 'frontend' = 'backend';
+			const appType: 'backend' | 'web' = 'backend';
 
 			const envVars: string[] = [];
 
@@ -735,7 +735,7 @@ describe('workspaceDeployCommand', () => {
 
 		it('should not inject DATABASE_URL for frontend apps', () => {
 			const hasPostgres = true;
-			const appType = 'frontend' as 'backend' | 'frontend';
+			const appType = 'web' as 'backend' | 'web';
 
 			const envVars: string[] = [];
 
@@ -794,7 +794,7 @@ describe('workspaceDeployCommand', () => {
 		it('should have correct structure for failed AppDeployResult', () => {
 			const failedResult = {
 				appName: 'web',
-				type: 'frontend' as const,
+				type: 'web' as const,
 				success: false,
 				error: 'Build failed',
 			};
@@ -809,7 +809,7 @@ describe('workspaceDeployCommand', () => {
 					{ appName: 'api', type: 'backend' as const, success: true },
 					{
 						appName: 'web',
-						type: 'frontend' as const,
+						type: 'web' as const,
 						success: false,
 						error: 'Failed',
 					},
@@ -834,7 +834,7 @@ describe('workspaceDeployCommand', () => {
 			};
 
 			const webApp = {
-				type: 'frontend' as const,
+				type: 'web' as const,
 				path: 'apps/web',
 				port: 3000,
 				dependencies: ['api', 'auth'],
@@ -873,7 +873,7 @@ describe('workspaceDeployCommand', () => {
 			};
 
 			const webApp = {
-				type: 'frontend' as const,
+				type: 'web' as const,
 				path: 'apps/web',
 				port: 3001,
 				dependencies: ['api', 'payments'],
@@ -922,7 +922,7 @@ describe('workspaceDeployCommand', () => {
 			};
 
 			const webApp = {
-				type: 'frontend' as const,
+				type: 'web' as const,
 				path: 'apps/web',
 				port: 3001,
 				dependencies: ['api', 'auth'],
@@ -997,7 +997,7 @@ describe('workspaceDeployCommand', () => {
 
 			// Step 2: Deploy web (depends on api)
 			const webApp = {
-				type: 'frontend' as const,
+				type: 'web' as const,
 				path: 'apps/web',
 				port: 3001,
 				dependencies: ['api'],
@@ -1048,7 +1048,7 @@ describe('workspaceDeployCommand', () => {
 			};
 
 			const webApp = {
-				type: 'frontend' as const,
+				type: 'web' as const,
 				path: 'apps/web',
 				port: 3000,
 				dependencies: ['api', 'auth', 'notifications'],

package/src/deploy/__tests__/sniffer.spec.ts

@@ -32,7 +32,7 @@ describe('sniffAppEnvironment', () => {
 
 	describe('frontend apps', () => {
 		it('should return empty env vars for frontend apps with no dependencies', async () => {
-			const app = createApp({ type: 'frontend', dependencies: [] });
+			const app = createApp({ type: 'web', dependencies: [] });
 
 			const result = await sniffAppEnvironment(app, 'web', workspacePath);
 
@@ -42,7 +42,7 @@ describe('sniffAppEnvironment', () => {
 
 		it('should return NEXT_PUBLIC_{DEP}_URL for frontend dependencies', async () => {
 			const app = createApp({
-				type: 'frontend',
+				type: 'web',
 				dependencies: ['api', 'auth'],
 			});
 
@@ -56,7 +56,7 @@ describe('sniffAppEnvironment', () => {
 
 		it('should generate uppercase dep names in NEXT_PUBLIC_{DEP}_URL', async () => {
 			const app = createApp({
-				type: 'frontend',
+				type: 'web',
 				dependencies: ['payments-service', 'notification_api'],
 			});
 
@@ -73,7 +73,7 @@ describe('sniffAppEnvironment', () => {
 		describe('config sniffing', () => {
 			it('should sniff env vars from config.client path', async () => {
 				const app = createApp({
-					type: 'frontend',
+					type: 'web',
 					path: fixturesPath,
 					dependencies: [],
 					config: {
@@ -90,7 +90,7 @@ describe('sniffAppEnvironment', () => {
 
 			it('should sniff env vars from config.server path', async () => {
 				const app = createApp({
-					type: 'frontend',
+					type: 'web',
 					path: fixturesPath,
 					dependencies: [],
 					config: {
@@ -107,7 +107,7 @@ describe('sniffAppEnvironment', () => {
 
 			it('should combine vars from both config.client and config.server', async () => {
 				const app = createApp({
-					type: 'frontend',
+					type: 'web',
 					path: fixturesPath,
 					dependencies: ['api'],
 					config: {
@@ -129,7 +129,7 @@ describe('sniffAppEnvironment', () => {
 
 			it('should deduplicate vars from both config files', async () => {
 				const app = createApp({
-					type: 'frontend',
+					type: 'web',
 					path: fixturesPath,
 					dependencies: [],
 					config: {
@@ -306,7 +306,7 @@ describe('sniffAppEnvironment', () => {
 
 		it('returns empty optionalEnvVars for frontend apps', async () => {
 			const app = createApp({
-				type: 'frontend',
+				type: 'web',
 				dependencies: ['api'],
 			});
 
@@ -339,7 +339,7 @@ describe('sniffAllApps', () => {
 				// No entry, routes, or envParser - will return empty
 			},
 			web: {
-				type: 'frontend',
+				type: 'web',
 				path: 'apps/web',
 				port: 3001,
 				dependencies: ['api', 'auth'],