@usequark/quark-create-app 1.24.1 → 1.24.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@usequark/quark-create-app",
-  "version": "1.24.1",
+  "version": "1.24.2",
   "description": "CLI to scaffold new Quark projects with Next.js, Prisma, BullMQ, and Railway deployment",
   "keywords": [
     "quark",
@@ -56,7 +56,7 @@
     "sync-templates:check": "node scripts/sync-templates.js --check",
     "regen-migration": "node scripts/regenerate-template-migration.js",
     "smoke:published": "node scripts/smoke-published.js",
-    "test": "node test-cli.js && node --test src/utils.test.js && node --test src/prepare-hooks.test.js && node --test src/scaffold-output.test.js && node --test src/template-scripts.test.js && node --test src/scaffold-guards.test.js && node --test src/worker-dockerfile.test.js && node --test src/dependabot-config.test.js && node --test 'src/deploy/**/*.test.js' && node --test --test-concurrency=1 'src/deploy/**/*.integration.test.js'",
+    "test": "node test-cli.js && node --test src/utils.test.js && node --test src/prepare-hooks.test.js && node --test src/scaffold-output.test.js && node --test src/template-scripts.test.js && node --test src/scaffold-guards.test.js && node --test src/template-config.test.js && node --test src/worker-dockerfile.test.js && node --test src/dependabot-config.test.js && node --test 'src/deploy/**/*.test.js' && node --test --test-concurrency=1 'src/deploy/**/*.integration.test.js'",
     "test:build": "node test-build.js",
     "test:e2e": "node test-e2e.js",
     "test:e2e:full": "node test-e2e-full.js",

package/templates/base-project/biome.json

@@ -47,6 +47,5 @@
 				"organizeImports": "on"
 			}
 		}
-	},
-	"root": false
+	}
 }

package/templates/worker/src/index.js

@@ -131,6 +131,16 @@ export async function waitForRedis(
 		intervalMs = parseInt(process.env.WORKER_HEALTH_INTERVAL_MS || "1000", 10),
 	} = config;
 
+	// See waitForDatabase: reject a config object in the healthCheck slot
+	// rather than failing with an opaque TypeError.
+	if (typeof healthCheck !== "function") {
+		throw new AppError(
+			"waitForRedis expects a health check function as its first argument.",
+			500,
+			"REDIS_HEALTH_CHECK_INVALID",
+		);
+	}
+
 	const reportThrottledError = throttledError(logger, 3000);
 
 	for (let attempt = 1; attempt <= maxRetries; attempt++) {
@@ -190,6 +200,18 @@ export async function waitForDatabase(
 		intervalMs = parseInt(process.env.WORKER_DB_INTERVAL_MS || "2000", 10),
 	} = config;
 
+	// Guard against the config object being passed in the healthCheck slot:
+	// that surfaces as an opaque "healthCheck is not a function" TypeError and,
+	// because a TypeError is neither a schema nor a connection error, it
+	// rethrows on the first attempt so the retry config never applies.
+	if (typeof healthCheck !== "function") {
+		throw new AppError(
+			"waitForDatabase expects a health check function as its first argument.",
+			500,
+			"DATABASE_HEALTH_CHECK_INVALID",
+		);
+	}
+
 	const reportThrottledError = throttledError(logger, 5000);
 
 	for (let attempt = 1; attempt <= maxRetries; attempt++) {
@@ -379,7 +401,12 @@ async function startWorker() {
 
 		// Wait for database schema to be ready (migrations may still be running)
 		logger.info("Checking database schema readiness...");
-		await waitForDatabase(isDevMode ? { maxRetries: 5, intervalMs: 1000 } : {});
+		// healthCheck takes the first argument; the config object must not be
+		// passed in its slot or the call throws "healthCheck is not a function".
+		await waitForDatabase(
+			undefined,
+			isDevMode ? { maxRetries: 5, intervalMs: 1000 } : {},
+		);
 		logger.info("Database schema ready");
 
 		// Register a worker for each queue