create-mercato-app 0.6.3-develop.3876.1.d40fe4ec2d → 0.6.3-develop.3881.1.0b590ac4eb

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-mercato-app",
-  "version": "0.6.3-develop.3876.1.d40fe4ec2d",
+  "version": "0.6.3-develop.3881.1.0b590ac4eb",
   "type": "module",
   "description": "Create a new Open Mercato application",
   "main": "./dist/index.js",

package/template/scripts/dev-orchestration-log-policy.mjs

@@ -66,6 +66,17 @@ export function isIgnorableTurboShutdownLine(line) {
     || /^command (interrupted|cancelled|canceled)/i.test(plain)
 }
 
+export function isIgnorableConsolidatedWatchLine(line) {
+  if (typeof line !== 'string') return false
+  const plain = normalize(line)
+  if (!plain.startsWith('[watch]')) return false
+  return /^\[watch\] consolidated watcher: /.test(plain)
+    || /^\[watch\] [^:]+: rebuilding\.\.\.$/.test(plain)
+    || /^\[watch\] [^:]+: rebuild complete$/.test(plain)
+    || /^\[watch\] [^:]+: no source files found, skipping rebuild$/.test(plain)
+    || /^\[watch\] no workspace packages with a `watch` script /.test(plain)
+}
+
 const FAILURE_NOISE_PREDICATES = [
   isIgnorableBoxDrawingLine,
   isIgnorableEnvInjectionLine,
@@ -89,6 +100,7 @@ const TURBO_NOISE_PREDICATES = [
   isIgnorableTurboCacheCancellationLine,
   isIgnorableTurboShutdownLine,
   isIgnorableBoxDrawingLine,
+  isIgnorableConsolidatedWatchLine,
 ]
 
 export function isIgnorableTurboLine(line) {

package/template/scripts/dev-shutdown-utils.mjs

@@ -0,0 +1,41 @@
+import spawn from 'cross-spawn'
+
+// Windows does not propagate signals from a parent process down to grandchildren
+// the way POSIX does: child.kill('SIGTERM'/'SIGKILL') only terminates the direct
+// child, leaving any further descendants (next dev, mercato generate watch, etc.)
+// alive and still holding ports. taskkill with /T (tree) /F (force) is the
+// platform-blessed way to terminate the whole descendant tree. On POSIX we keep
+// child.kill so the existing graceful-then-forced two-phase shutdown is preserved.
+export function killProcessTree(child, signal, options = {}) {
+  if (!child) return false
+
+  const platform = options.platform ?? process.platform
+  const spawnImpl = options.spawn ?? spawn
+
+  if (platform === 'win32') {
+    const pid = child.pid
+    if (typeof pid !== 'number' || Number.isNaN(pid) || pid <= 0) {
+      return false
+    }
+    try {
+      const killer = spawnImpl('taskkill', ['/pid', String(pid), '/T', '/F'], {
+        stdio: 'ignore',
+        windowsHide: true,
+      })
+      if (killer && typeof killer.on === 'function') {
+        killer.on('error', () => { /* best-effort: taskkill may not exist on stripped images */ })
+      }
+      return true
+    } catch {
+      return false
+    }
+  }
+
+  if (child.killed) return false
+  try {
+    child.kill(signal)
+    return true
+  } catch {
+    return false
+  }
+}

package/template/scripts/dev.mjs

@@ -23,6 +23,7 @@ import {
   stripAnsi,
 } from './dev-splash-helpers.mjs'
 import { purgeAppBuildCaches } from './dev-cache-purge.mjs'
+import { killProcessTree } from './dev-shutdown-utils.mjs'
 import { resolveSpawnCommand } from './dev-spawn-utils.mjs'
 import { createDevSplashCodingFlow } from './dev-splash-coding-flow.mjs'
 import { createDevSplashGitRepoFlow } from './dev-splash-git-repo-flow.mjs'
@@ -1023,13 +1024,13 @@ function shutdown(exitCode = 0) {
   }
 
   for (const child of alive) {
-    child.kill('SIGTERM')
+    killProcessTree(child, 'SIGTERM')
   }
 
   setTimeout(() => {
     for (const child of children) {
       if (!child.killed) {
-        child.kill('SIGKILL')
+        killProcessTree(child, 'SIGKILL')
       }
     }
     closeDevLogSession()
@@ -1492,10 +1493,21 @@ async function runPassthroughStage(label, commandArgs, options = {}) {
   console.log(`✅ ${formatProgressLine(label, stageCurrent, stageTotal, resolveProgressPercent(stageCurrent, stageTotal))} in ${formatDuration(Date.now() - startedAt)}`)
 }
 
+function resolveWatchPackagesScript() {
+  // `OM_WATCH_PACKAGES_MODE=legacy` falls back to the Turbo per-package
+  // fan-out for developers who need the old behavior (debugging, or pairing
+  // with `OM_PACKAGE_WATCH_MODE=persistent` for hot rebuilds at the cost of
+  // ~1 GB more idle RSS). Default is the consolidated single-process watcher.
+  const raw = String(process.env.OM_WATCH_PACKAGES_MODE ?? '').trim().toLowerCase()
+  return raw === 'legacy' ? 'watch:packages:legacy' : 'watch:packages'
+}
+
 function startPackageWatch() {
+  const watchScript = resolveWatchPackagesScript()
+
   if (classic) {
-    const child = spawnCommand(yarnCommand, ['watch:packages'], {
-      label: 'watch:packages',
+    const child = spawnCommand(yarnCommand, [watchScript], {
+      label: watchScript,
       logFile: getDevRunnerLog(),
       mirrorOutput: true,
     })
@@ -1529,16 +1541,7 @@ function startPackageWatch() {
     activity: 'Workspace package watch started',
   })
 
-  const child = spawnCommand(yarnCommand, [
-    'turbo',
-    'run',
-    'watch',
-    '--filter=./packages/*',
-    '--concurrency=32',
-    '--output-logs=errors-only',
-    '--log-order=grouped',
-    '--log-prefix=none',
-  ], {
+  const child = spawnCommand(yarnCommand, [watchScript], {
     label: 'Watching workspace packages',
     logFile: getDevRunnerLog(),
     mirrorOutput: verbose,