@stacksjs/actions 0.70.155 → 0.70.157

package/dist/bump.js

@@ -2,7 +2,7 @@ import { execSync, log, parseOptions } from "@stacksjs/cli";
 import { path as p } from "@stacksjs/path";
 import { versionBump } from "@stacksjs/bumpx";
 import { generateChangelog, loadLogsmithConfig } from "@stacksjs/logsmith";
-import { existsSync, readFileSync, writeFileSync } from "node:fs";
+import { existsSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
 import { join, relative } from "node:path";
 const options = parseOptions(), allowedBumps = new Set(["patch", "minor", "major", "prepatch", "preminor", "premajor", "prerelease"]), rawBump = options?.bump?.toString(), bumpArg = rawBump ? allowedBumps.has(rawBump) || /^\d+\.\d+\.\d+(?:-[\w.]+)?$/.test(rawBump) ? rawBump : null : null;
 if (rawBump && !bumpArg)
@@ -126,11 +126,19 @@ async function writeChangelog() {
 await writeChangelog();
 if (!isDryRun && isFrameworkRelease)
   pinMetaCoreDeps(nextVersion);
-if (!isDryRun && existsSync(p.projectPath("bun.lock")))
-  await execSync(["bun", "install", "--lockfile-only"], {
-    cwd: p.projectPath(),
-    stdin: "inherit"
-  });
+if (!isDryRun && existsSync(p.projectPath("bun.lock"))) {
+  const lockPath = p.projectPath("bun.lock"), previousLock = readFileSync(lockPath);
+  unlinkSync(lockPath);
+  try {
+    await execSync(["bun", "install", "--lockfile-only"], {
+      cwd: p.projectPath(),
+      stdin: "inherit"
+    });
+  } catch (error) {
+    writeFileSync(lockPath, previousLock);
+    throw error;
+  }
+}
 function pinMetaCoreDeps(version) {
   const metaPath = p.frameworkPath("core/package.json"), meta = JSON.parse(readFileSync(metaPath, "utf-8"));
   let pinned = 0;

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.js

@@ -25,7 +25,11 @@ import {
   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");
@@ -125,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);
@@ -395,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;
   }
@@ -407,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;
@@ -416,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.155",
+  "version": "0.70.157",
   "description": "The Stacks actions.",
   "author": "Chris Breuer",
   "contributors": [
@@ -62,7 +62,7 @@
     "prepublishOnly": "bun run build"
   },
   "dependencies": {
-    "@stacksjs/config": "0.70.155",
+    "@stacksjs/config": "0.70.157",
     "@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.155",
-    "@stacksjs/cli": "0.70.155",
-    "@stacksjs/config": "0.70.155",
-    "@stacksjs/database": "0.70.155",
+    "@stacksjs/api": "0.70.157",
+    "@stacksjs/cli": "0.70.157",
+    "@stacksjs/config": "0.70.157",
+    "@stacksjs/database": "0.70.157",
     "@stacksjs/tlsx": "^0.13.2",
     "better-dx": "^0.2.17",
-    "@stacksjs/dns": "0.70.155",
-    "@stacksjs/enums": "0.70.155",
-    "@stacksjs/env": "0.70.155",
-    "@stacksjs/error-handling": "0.70.155",
-    "@stacksjs/logging": "0.70.155",
-    "@stacksjs/path": "0.70.155",
-    "@stacksjs/security": "0.70.155",
-    "@stacksjs/storage": "0.70.155",
-    "@stacksjs/strings": "0.70.155",
-    "@stacksjs/utils": "0.70.155",
-    "@stacksjs/validation": "0.70.155"
+    "@stacksjs/dns": "0.70.157",
+    "@stacksjs/enums": "0.70.157",
+    "@stacksjs/env": "0.70.157",
+    "@stacksjs/error-handling": "0.70.157",
+    "@stacksjs/logging": "0.70.157",
+    "@stacksjs/path": "0.70.157",
+    "@stacksjs/security": "0.70.157",
+    "@stacksjs/storage": "0.70.157",
+    "@stacksjs/strings": "0.70.157",
+    "@stacksjs/utils": "0.70.157",
+    "@stacksjs/validation": "0.70.157"
   },
   "peerDependencies": {
     "pickier": "^0.1.35"