@stacksjs/actions 0.70.154 → 0.70.156

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

@@ -1,3 +1,16 @@
+/**
+ * The first upgrade process records the real file changes, then restarts itself
+ * after replacing its own framework code. The restarted process sees the
+ * already-synced tree as unchanged, but it still owns the post-sync work that
+ * the first process intentionally deferred.
+ */
+export declare function shouldRunPostSyncHooks(changeCount: number, alreadyRestarted: boolean): boolean;
+/**
+ * A restarted process cannot reconstruct whether a package manifest changed
+ * during the parent sync. Refreshing dependencies is the safe, idempotent
+ * choice whenever post-sync work crosses that process boundary.
+ */
+export declare function shouldRefreshPostSyncDependencies(corePackageChanged: boolean, alreadyRestarted: boolean): boolean;
 /**
  * Run the migration hook after a vendored framework upgrade.
  *

package/dist/upgrade/framework-hooks.js

@@ -1,3 +1,9 @@
+export function shouldRunPostSyncHooks(changeCount, alreadyRestarted) {
+  return changeCount > 0 || alreadyRestarted;
+}
+export function shouldRefreshPostSyncDependencies(corePackageChanged, alreadyRestarted) {
+  return corePackageChanged || alreadyRestarted;
+}
 export async function runPostSyncMigration(options) {
   const code = await options.spawn({
     cmd: [options.bunExecutable, options.migrateScript, "migrate", "--force"],

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

@@ -43,6 +43,14 @@ export declare function readSyncedVersion(versionFilePath: string): SyncedVersio
  * Persist the last-synced channel + sha to `.stacks-version`.
  */
 export declare function writeSyncedVersion(versionFilePath: string, ch: 'stable' | 'canary', sha: string): void;
+/**
+ * Decide whether the upgrade must protect framework-managed paths from local
+ * edits. The initial process always owns this preflight unless the user passed
+ * `--force`. An internal restart runs only after that validated parent has
+ * intentionally replaced those paths, so checking again would reject the
+ * upgrade's own writes as if they belonged to the user.
+ */
+export declare function shouldCheckDirtyManagedPaths(args: { force: boolean, alreadyRestarted: boolean }): boolean;
 /**
  * Pure decision for the "already up to date" short-circuit. Returns true only
  * when a sync can be safely skipped: no override flags, not a pinned version,

package/dist/upgrade/framework-utils.js

@@ -53,6 +53,9 @@ export function writeSyncedVersion(versionFilePath, ch, sha) {
     writeFileSync(versionFilePath, `${ch} ${sha}`, "utf-8");
   } catch {}
 }
+export function shouldCheckDirtyManagedPaths(args) {
+  return !args.force && !args.alreadyRestarted;
+}
 export function shouldShortCircuit(args) {
   const { force, targetVersion, remoteSha, synced, channel } = args;
   if (force || targetVersion)

package/dist/upgrade/framework.js

@@ -19,12 +19,17 @@ import {
   resolveUpgradeContext,
   resolveUpgradeMessage,
   shouldAutoDetectLocalStacks,
+  shouldCheckDirtyManagedPaths,
   shouldShortCircuit,
   snapshotTree,
   writeChannel,
   writeSyncedVersion
 } from "./framework-utils";
-import { runPostSyncMigration } from "./framework-hooks";
+import {
+  runPostSyncMigration,
+  shouldRefreshPostSyncDependencies,
+  shouldRunPostSyncHooks
+} from "./framework-hooks";
 const options = parseOptions(), projectRoot = p.projectPath();
 if (!existsSync(p.projectPath("storage/framework/core"))) {
   const { upgradeStacksPackages } = await import("./packages");
@@ -44,7 +49,8 @@ if (options.dryRun) {
   await runDryRunPreview();
   process.exit(ExitCode.Success);
 }
-if (!options.force) {
+const alreadyRestarted = process.argv.includes("--__restarted");
+if (shouldCheckDirtyManagedPaths({ force: !!options.force, alreadyRestarted })) {
   const dirty = await detectDirtyManagedPaths(projectRoot);
   if (dirty.length > 0) {
     console.warn(`
@@ -91,7 +97,7 @@ try {
   console.error("\u2718 Failed to upgrade Stacks framework:", err?.message || err);
   process.exit(ExitCode.FatalError);
 }
-const newVersion = readVersion(corePkgPath), ownHashAfter = await hashFileOrZero(ownPath), alreadyRestarted = process.argv.includes("--__restarted");
+const newVersion = readVersion(corePkgPath), ownHashAfter = await hashFileOrZero(ownPath);
 if (ownHashBefore !== 0n && ownHashAfter !== 0n && ownHashBefore !== ownHashAfter && !alreadyRestarted) {
   console.log(`Upgrade pulled a newer version of the upgrade action \u2014 re-running once to pick up new sync paths and hooks.
 `);
@@ -123,7 +129,7 @@ for (const { managed, summary } of perPath)
 const runHooks = !options.noPostinstall && options.postinstall !== !1;
 if (runHooks)
   try {
-    await runPostSyncHooks({ aggregate, perPath, projectRoot });
+    await runPostSyncHooks({ aggregate, perPath, projectRoot, alreadyRestarted });
   } catch (err) {
     console.error(`Post-sync hooks failed: ${err?.message || err}`);
     process.exit(ExitCode.FatalError);
@@ -393,8 +399,8 @@ async function syncRootFilesFromGitHub(ref) {
   }
 }
 async function runPostSyncHooks(args) {
-  const { aggregate, perPath, projectRoot } = args;
-  if (aggregate.added + aggregate.changed + aggregate.removed === 0) {
+  const { aggregate, perPath, projectRoot, alreadyRestarted } = args, changeCount = aggregate.added + aggregate.changed + aggregate.removed;
+  if (!shouldRunPostSyncHooks(changeCount, alreadyRestarted)) {
     console.log("Nothing changed \u2014 skipping post-sync hooks.");
     return;
   }
@@ -405,7 +411,7 @@ async function runPostSyncHooks(args) {
   } catch (err) {
     console.warn(`  auto-imports failed (non-fatal): ${err?.message || err}`);
   }
-  if (perPath.some((entry) => {
+  if (shouldRefreshPostSyncDependencies(perPath.some((entry) => {
     const { managed, summary } = entry;
     if (managed.isFile)
       return !1;
@@ -414,7 +420,7 @@ async function runPostSyncHooks(args) {
     if (summary.added + summary.changed === 0)
       return !1;
     return pkgJsonInTree(join(projectRoot, managed.localPath));
-  })) {
+  }), alreadyRestarted)) {
     console.log("Running `bun install` to refresh dependencies...");
     try {
       const code = await Bun.spawn({

package/package.json

@@ -2,7 +2,7 @@
   "name": "@stacksjs/actions",
   "type": "module",
   "sideEffects": false,
-  "version": "0.70.154",
+  "version": "0.70.156",
   "description": "The Stacks actions.",
   "author": "Chris Breuer",
   "contributors": [
@@ -62,7 +62,7 @@
     "prepublishOnly": "bun run build"
   },
   "dependencies": {
-    "@stacksjs/config": "0.70.154",
+    "@stacksjs/config": "0.70.156",
     "@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.154",
-    "@stacksjs/cli": "0.70.154",
-    "@stacksjs/config": "0.70.154",
-    "@stacksjs/database": "0.70.154",
+    "@stacksjs/api": "0.70.156",
+    "@stacksjs/cli": "0.70.156",
+    "@stacksjs/config": "0.70.156",
+    "@stacksjs/database": "0.70.156",
     "@stacksjs/tlsx": "^0.13.2",
     "better-dx": "^0.2.17",
-    "@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"
+    "@stacksjs/dns": "0.70.156",
+    "@stacksjs/enums": "0.70.156",
+    "@stacksjs/env": "0.70.156",
+    "@stacksjs/error-handling": "0.70.156",
+    "@stacksjs/logging": "0.70.156",
+    "@stacksjs/path": "0.70.156",
+    "@stacksjs/security": "0.70.156",
+    "@stacksjs/storage": "0.70.156",
+    "@stacksjs/strings": "0.70.156",
+    "@stacksjs/utils": "0.70.156",
+    "@stacksjs/validation": "0.70.156"
   },
   "peerDependencies": {
     "pickier": "^0.1.35"