@stacksjs/actions 0.70.153 → 0.70.154

package/dist/upgrade/framework-hooks.d.ts

@@ -0,0 +1,25 @@
+/**
+ * Run the migration hook after a vendored framework upgrade.
+ *
+ * The upgrade command is already an explicit request to install and activate
+ * the new framework. Its nested migration must therefore never open a second
+ * confirmation prompt. Ignoring stdin makes that contract structural, while
+ * `--force` bypasses both migration confirmation gates. A nonzero exit is a
+ * real upgrade failure so callers cannot report success with an unapplied
+ * schema.
+ */
+export declare function runPostSyncMigration(options: PostSyncMigrationOptions): Promise<void>;
+declare interface PostSyncSpawnOptions {
+  cmd: string[]
+  cwd: string
+  stdin: 'ignore'
+  stdout: 'inherit'
+  stderr: 'inherit'
+}
+declare interface PostSyncMigrationOptions {
+  bunExecutable: string
+  migrateScript: string
+  projectRoot: string
+  spawn: PostSyncSpawn
+}
+declare type PostSyncSpawn = (options: PostSyncSpawnOptions) => { exited: Promise<number> }

package/dist/upgrade/framework-hooks.js

@@ -0,0 +1,11 @@
+export async function runPostSyncMigration(options) {
+  const code = await options.spawn({
+    cmd: [options.bunExecutable, options.migrateScript, "migrate", "--force"],
+    cwd: options.projectRoot,
+    stdin: "ignore",
+    stdout: "inherit",
+    stderr: "inherit"
+  }).exited;
+  if (code !== 0)
+    throw Error(`Post-upgrade migration exited with code ${code}.`);
+}

package/dist/upgrade/framework.js

@@ -24,6 +24,7 @@ import {
   writeChannel,
   writeSyncedVersion
 } from "./framework-utils";
+import { runPostSyncMigration } from "./framework-hooks";
 const options = parseOptions(), projectRoot = p.projectPath();
 if (!existsSync(p.projectPath("storage/framework/core"))) {
   const { upgradeStacksPackages } = await import("./packages");
@@ -121,7 +122,12 @@ for (const { managed, summary } of perPath)
     console.log(`  ${managed.label.padEnd(10)} +${summary.added} ~${summary.changed} -${summary.removed} (${summary.unchanged} unchanged)`);
 const runHooks = !options.noPostinstall && options.postinstall !== !1;
 if (runHooks)
-  await runPostSyncHooks({ aggregate, perPath, projectRoot });
+  try {
+    await runPostSyncHooks({ aggregate, perPath, projectRoot });
+  } catch (err) {
+    console.error(`Post-sync hooks failed: ${err?.message || err}`);
+    process.exit(ExitCode.FatalError);
+  }
 if (ctx.channel === "canary" && !ctx.targetVersion)
   console.warn("\u26A0 Canary builds may contain bugs or breaking changes. Not recommended for production.");
 process.exit(ExitCode.Success);
@@ -426,18 +432,13 @@ async function runPostSyncHooks(args) {
   const migrationsDir = join(projectRoot, "database", "migrations");
   if (existsSync(migrationsDir)) {
     console.log("Running pending migrations...");
-    try {
-      const migrateScript = p.frameworkPath("core/buddy/src/cli.ts"), code = await Bun.spawn({
-        cmd: [process.argv[0] || "bun", migrateScript, "migrate"],
-        cwd: projectRoot,
-        stdout: "inherit",
-        stderr: "inherit"
-      }).exited;
-      if (code !== 0)
-        console.warn(`  migrate exited with code ${code} (non-fatal \u2014 DB may not be set up yet)`);
-    } catch (err) {
-      console.warn(`  migrate failed (non-fatal): ${err?.message || err}`);
-    }
+    const migrateScript = p.frameworkPath("core/buddy/src/cli.ts");
+    await runPostSyncMigration({
+      bunExecutable: process.argv[0] || "bun",
+      migrateScript,
+      projectRoot,
+      spawn: (spawnOptions) => Bun.spawn(spawnOptions)
+    });
   }
 }
 function pkgJsonInTree(dir) {

package/package.json

@@ -2,7 +2,7 @@
   "name": "@stacksjs/actions",
   "type": "module",
   "sideEffects": false,
-  "version": "0.70.153",
+  "version": "0.70.154",
   "description": "The Stacks actions.",
   "author": "Chris Breuer",
   "contributors": [
@@ -62,7 +62,7 @@
     "prepublishOnly": "bun run build"
   },
   "dependencies": {
-    "@stacksjs/config": "0.70.153",
+    "@stacksjs/config": "0.70.154",
     "@stacksjs/bumpx": "^0.2.6",
     "@stacksjs/bunpress": "^0.1.12",
     "@stacksjs/logsmith": "^0.2.3",
@@ -70,23 +70,23 @@
     "@stacksjs/ts-md": "^0.1.1"
   },
   "devDependencies": {
-    "@stacksjs/api": "0.70.153",
-    "@stacksjs/cli": "0.70.153",
-    "@stacksjs/config": "0.70.153",
-    "@stacksjs/database": "0.70.153",
+    "@stacksjs/api": "0.70.154",
+    "@stacksjs/cli": "0.70.154",
+    "@stacksjs/config": "0.70.154",
+    "@stacksjs/database": "0.70.154",
     "@stacksjs/tlsx": "^0.13.2",
     "better-dx": "^0.2.17",
-    "@stacksjs/dns": "0.70.153",
-    "@stacksjs/enums": "0.70.153",
-    "@stacksjs/env": "0.70.153",
-    "@stacksjs/error-handling": "0.70.153",
-    "@stacksjs/logging": "0.70.153",
-    "@stacksjs/path": "0.70.153",
-    "@stacksjs/security": "0.70.153",
-    "@stacksjs/storage": "0.70.153",
-    "@stacksjs/strings": "0.70.153",
-    "@stacksjs/utils": "0.70.153",
-    "@stacksjs/validation": "0.70.153"
+    "@stacksjs/dns": "0.70.154",
+    "@stacksjs/enums": "0.70.154",
+    "@stacksjs/env": "0.70.154",
+    "@stacksjs/error-handling": "0.70.154",
+    "@stacksjs/logging": "0.70.154",
+    "@stacksjs/path": "0.70.154",
+    "@stacksjs/security": "0.70.154",
+    "@stacksjs/storage": "0.70.154",
+    "@stacksjs/strings": "0.70.154",
+    "@stacksjs/utils": "0.70.154",
+    "@stacksjs/validation": "0.70.154"
   },
   "peerDependencies": {
     "pickier": "^0.1.35"