@platforma-sdk/block-tools 2.14.2 → 2.14.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platforma-sdk/block-tools",
-  "version": "2.14.2",
+  "version": "2.14.3",
   "description": "Utility to manipulate Platforma Blocks and Block Registry",
   "license": "UNLICENSED",
   "bin": {
@@ -35,13 +35,13 @@
     "yaml": "^2.8.0",
     "zod": "~3.25.76",
     "@milaboratories/pl-model-backend": "1.4.21",
+    "@milaboratories/pl-model-common": "1.48.0",
     "@milaboratories/pl-http": "1.2.4",
     "@milaboratories/ts-helpers": "1.8.6",
-    "@milaboratories/pl-model-common": "1.48.0",
-    "@milaboratories/pl-model-middle-layer": "1.32.0",
-    "@milaboratories/resolve-helper": "1.1.3",
     "@platforma-sdk/blocks-deps-updater": "2.2.0",
-    "@platforma-sdk/package-builder-lib": "1.3.0"
+    "@milaboratories/pl-model-middle-layer": "1.32.0",
+    "@platforma-sdk/package-builder-lib": "1.3.0",
+    "@milaboratories/resolve-helper": "1.1.3"
   },
   "devDependencies": {
     "@rollup/plugin-node-resolve": "^16.0.1",
@@ -56,8 +56,8 @@
     "vite": "^8.0.6",
     "vite-plugin-dts": "^4.5.3",
     "vitest": "^4.1.3",
-    "@milaboratories/ts-builder": "1.7.0",
     "@milaboratories/build-configs": "2.0.0",
+    "@milaboratories/ts-builder": "1.7.0",
     "@milaboratories/ts-configs": "1.4.0"
   },
   "scripts": {

package/src/structure/__tests__/facade-end-state.test.ts

@@ -1,7 +1,7 @@
 // Facade end-state coverage — the slim `block/` the structurer emits and
 // enforces: block-scope detection by the top-level `block` field, the four-file
 // `src/` surface + facade `tsconfig.json`, the slim publishable `package.json`,
-// sibling privacy, and legacy-shim cleanup.
+// sibling privacy, legacy-shim cleanup, and author-script preservation.
 
 import { describe, test, expect } from "vitest";
 import type { BlockVars, RunContext } from "../engine/api";
@@ -241,6 +241,26 @@ describe("facade: sibling versions on refresh", () => {
   });
 });
 
+describe("facade: author scripts beyond the canonical set survive a refresh", () => {
+  test("a pre-existing `mark-stable` script is preserved", () => {
+    const { fs, ctx } = simulateInit({ vars: VARS });
+    const MARK_STABLE =
+      "block-tools mark-stable -r 's3://milab-euce1-prod-pkgs-s3-block-registry/pub/releases/?region=eu-central-1'";
+
+    // `.github/workflows/mark-stable.yaml` (engine-generated) runs
+    // `cd block && pnpm run mark-stable`, so the facade must keep the script.
+    const before = JSON.parse(fs.read("block/package.json"));
+    before.scripts["mark-stable"] = MARK_STABLE;
+    fs.write("block/package.json", `${JSON.stringify(before, null, 2)}\n`);
+
+    const refreshCtx: RunContext = { ...ctx, dryRun: false };
+    expect(() => engineRun(STRUCTURE, fs, refreshCtx, { templates: TEMPLATES })).not.toThrow();
+
+    const after = JSON.parse(fs.read("block/package.json"));
+    expect(after.scripts["mark-stable"]).toBe(MARK_STABLE);
+  });
+});
+
 describe("facade: legacy shim cleanup on refresh", () => {
   test("a refresh removes a pre-existing block/index.js + index.d.ts, then is idempotent", () => {
     const { fs, ctx } = simulateInit({ vars: VARS });

package/src/structure/rules/block-package-json.ts

@@ -11,7 +11,6 @@ import {
   ensureDevDep,
   ensureWorkspaceScopeDevDeps,
   requireField,
-  removeScript,
   enforceAlphabeticalOrder,
   enforceFieldOrder,
   type RunContext,
@@ -221,7 +220,10 @@ export function blockPackageJsonRules(ctx: RunContext): void {
 
   const scripts = blockScripts(v.npmOrg, ctx.isSdkInternal);
   for (const [name, command] of Object.entries(scripts)) ensureScript(name, command);
-  removeScript("mark-stable");
+  // Scripts beyond the canonical set are block-author territory — left as-is.
+  // `mark-stable` in particular is invoked by the engine-generated
+  // `.github/workflows/mark-stable.yaml` (which does `cd block && pnpm run
+  // mark-stable`), so stripping it broke that workflow.
 
   // Slim invariant: the facade carries ZERO runtime dependencies. The siblings
   // and the SDK build/publish deps are all devDeps, so `dependencies` ends