@platforma-sdk/block-tools 2.16.13 → 2.16.14

package/dist/structure/rules/root-changeset-config.d.ts

@@ -1,8 +1,9 @@
+import { RunContext } from '../engine/api';
 /** Seeded so the scaffold's own PR passes the `require-package-path-bump` gate
  *  without bumping: `changeset version` consumes it and leaves the facade at
  *  `INITIAL_MODULE_VERSION`, so the first version a user sees is 1.0.0 rather
  *  than 1.0.1. Seeded (not managed) — a refresh must never resurrect it once
  *  the first release has consumed it. */
 export declare const EMPTY_CHANGESET_SEED = "---\n---\n\nInitial release.\n";
-export declare function rootChangesetConfigInitial(): Record<string, unknown>;
-export declare function rootChangesetConfigRules(): void;
+export declare function rootChangesetConfigInitial(ctx: RunContext): Record<string, unknown>;
+export declare function rootChangesetConfigRules(ctx: RunContext): void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platforma-sdk/block-tools",
-  "version": "2.16.13",
+  "version": "2.16.14",
   "description": "Utility to manipulate Platforma Blocks and Block Registry",
   "license": "UNLICENSED",
   "bin": {
@@ -34,13 +34,13 @@
     "undici": "~7.16.0",
     "yaml": "^2.8.0",
     "zod": "~3.25.76",
-    "@milaboratories/pl-http": "1.2.4",
-    "@milaboratories/pl-model-backend": "1.4.36",
-    "@milaboratories/pl-model-middle-layer": "1.34.2",
     "@milaboratories/pl-model-common": "1.53.0",
-    "@milaboratories/ts-helpers": "1.8.6",
+    "@milaboratories/pl-model-backend": "1.4.36",
+    "@milaboratories/pl-http": "1.2.4",
     "@milaboratories/resolve-helper": "1.1.3",
+    "@milaboratories/pl-model-middle-layer": "1.34.2",
     "@platforma-sdk/blocks-deps-updater": "2.2.0",
+    "@milaboratories/ts-helpers": "1.8.6",
     "@platforma-sdk/package-builder-lib": "1.4.1"
   },
   "devDependencies": {

package/src/structure/__tests__/root-changeset-config.test.ts

@@ -19,6 +19,13 @@ const VARS: BlockVars = {
 
 const PATH = ".changeset/config.json";
 
+const FIXED_GROUP = [
+  "@platforma-open/test-org.demo.block",
+  "@platforma-open/test-org.demo.model",
+  "@platforma-open/test-org.demo.ui",
+  "@platforma-open/test-org.demo.workflow",
+];
+
 function refresh(fs: MemoryFileSystem, isSdkInternal = false) {
   return engineRun(STRUCTURE, fs, discoverRunContext({ fs, isSdkInternal }), {
     templates: defaultTemplateProvider(),
@@ -65,6 +72,46 @@ describe("root .changeset/config.json", () => {
     expect(after.access).toBe("restricted");
   });
 
+  test("init fixes the facade to its model/ui/workflow siblings", () => {
+    const { fs } = simulateInit({ vars: VARS });
+    expect(JSON.parse(fs.read(PATH)).fixed).toEqual([FIXED_GROUP]);
+  });
+
+  test("refresh adds the group to a block that predates it", () => {
+    const { fs } = simulateInit({ vars: VARS });
+    fs.write(PATH, JSON.stringify({ ...JSON.parse(fs.read(PATH)), fixed: [] }, null, 2));
+
+    refresh(fs);
+    expect(JSON.parse(fs.read(PATH)).fixed).toEqual([FIXED_GROUP]);
+  });
+
+  test("refresh merges into the author group that overlaps and keeps the others", () => {
+    const { fs } = simulateInit({ vars: VARS });
+    const authored = {
+      ...JSON.parse(fs.read(PATH)),
+      fixed: [
+        ["@platforma-open/test-org.demo.software-a", "@platforma-open/test-org.demo.software-b"],
+        ["@platforma-open/test-org.demo.extra", "@platforma-open/test-org.demo.ui"],
+        ["@platforma-open/test-org.demo.model"],
+      ],
+    };
+    fs.write(PATH, JSON.stringify(authored, null, 2));
+
+    refresh(fs);
+    expect(JSON.parse(fs.read(PATH)).fixed).toEqual([
+      ["@platforma-open/test-org.demo.software-a", "@platforma-open/test-org.demo.software-b"],
+      [...FIXED_GROUP, "@platforma-open/test-org.demo.extra"],
+    ]);
+  });
+
+  test("refresh of a canonical config changes nothing", () => {
+    const { fs } = simulateInit({ vars: VARS });
+    const before = fs.read(PATH);
+
+    refresh(fs);
+    expect(fs.read(PATH)).toBe(before);
+  });
+
   test("sdk-internal blocks get no config — the monorepo owns it", () => {
     const { fs } = simulateInit({ vars: VARS, isSdkInternal: true });
     expect(fs.exists(PATH)).toBe(false);

package/src/structure/rules/root-changeset-config.ts

@@ -5,9 +5,17 @@
 //
 // Managed, not fixed: `$schema` is whatever `changeset init` last wrote (seven
 // versions are in use across the blocks) and `fixed`/`linked`/`ignore` are
-// author-owned lists. Only the fields CI depends on are re-asserted.
+// author-owned lists. Only the fields CI depends on are re-asserted, plus the
+// block's own `fixed` group.
 
-import { ensureField, enforceFieldOrder } from "../engine/api";
+import {
+  ensureField,
+  enforceFieldOrder,
+  transformAt,
+  type RunContext,
+  type Scope,
+} from "../engine/api";
+import { findModules } from "../engine/ctx";
 
 /** Seeded so the scaffold's own PR passes the `require-package-path-bump` gate
  *  without bumping: `changeset version` consumes it and leaves the facade at
@@ -20,6 +28,33 @@ export const EMPTY_CHANGESET_SEED = `---
 Initial release.
 `;
 
+export function rootChangesetConfigInitial(ctx: RunContext): Record<string, unknown> {
+  return {
+    $schema: "https://unpkg.com/@changesets/config@3.1.1/schema.json",
+    changelog: CHANGELOG,
+    commit: false,
+    fixed: [blockFixedGroup(ctx)],
+    linked: [],
+    access: ACCESS,
+    baseBranch: BASE_BRANCH,
+    updateInternalDependencies: UPDATE_INTERNAL_DEPENDENCIES,
+    ignore: [],
+  };
+}
+
+export function rootChangesetConfigRules(ctx: RunContext): void {
+  ensureField("changelog", CHANGELOG);
+  ensureField("commit", false);
+  ensureField("access", ACCESS);
+  ensureField("baseBranch", BASE_BRANCH);
+  ensureField("updateInternalDependencies", UPDATE_INTERNAL_DEPENDENCIES);
+  const group = blockFixedGroup(ctx);
+  transformAt<unknown>("fixed", (current) => withFixedGroup(current, group));
+  enforceFieldOrder(FIELD_ORDER);
+}
+
+// Internals
+
 const CHANGELOG = "@changesets/cli/changelog";
 const ACCESS = "restricted";
 const BASE_BRANCH = "main";
@@ -37,25 +72,35 @@ const FIELD_ORDER = [
   "ignore",
 ];
 
-export function rootChangesetConfigInitial(): Record<string, unknown> {
-  return {
-    $schema: "https://unpkg.com/@changesets/config@3.1.1/schema.json",
-    changelog: CHANGELOG,
-    commit: false,
-    fixed: [],
-    linked: [],
-    access: ACCESS,
-    baseBranch: BASE_BRANCH,
-    updateInternalDependencies: UPDATE_INTERNAL_DEPENDENCIES,
-    ignore: [],
-  };
+/** Packages released under one version. The facade lists its siblings only as
+ *  devDependencies, and changesets never bumps a package for a devDependency
+ *  release — without this group a model/ui/workflow change ships no new block.
+ *  The kind and software stay out: they are runtime dependencies of the model
+ *  and the workflow, so their bumps already reach the group. */
+const FIXED_SCOPES: Scope[] = ["block", "model", "ui", "workflow"];
+
+function blockFixedGroup(ctx: RunContext): string[] {
+  return FIXED_SCOPES.flatMap((scope) => findModules(ctx, scope).map((m) => m.name));
 }
 
-export function rootChangesetConfigRules(): void {
-  ensureField("changelog", CHANGELOG);
-  ensureField("commit", false);
-  ensureField("access", ACCESS);
-  ensureField("baseBranch", BASE_BRANCH);
-  ensureField("updateInternalDependencies", UPDATE_INTERNAL_DEPENDENCIES);
-  enforceFieldOrder(FIELD_ORDER);
+/** Merges `group` into the first author group that shares a package with it,
+ *  keeping that group's extra packages after the block's own; drops the
+ *  block's packages from every other group, since changesets rejects a package
+ *  listed in two fixed groups. Appends `group` when no author group overlaps. */
+function withFixedGroup(current: unknown, group: string[]): string[][] {
+  const owned = new Set(group);
+  const groups = Array.isArray(current) ? current.filter(isStringArray) : [];
+  const hostIndex = groups.findIndex((g) => g.some((name) => owned.has(name)));
+  const result: string[][] = [];
+  groups.forEach((g, i) => {
+    const extras = g.filter((name) => !owned.has(name));
+    if (i === hostIndex) result.push([...group, ...extras]);
+    else if (extras.length > 0) result.push(extras);
+  });
+  if (hostIndex === -1) result.push(group);
+  return result;
+}
+
+function isStringArray(value: unknown): value is string[] {
+  return Array.isArray(value) && value.every((v) => typeof v === "string");
 }

package/src/structure/rules/root.ts

@@ -50,9 +50,9 @@ export function rootRules(): void {
 
         managed(
           ".changeset/config.json",
-          generate(() => rootChangesetConfigInitial()),
-          () => {
-            rootChangesetConfigRules();
+          generate((ctx) => rootChangesetConfigInitial(ctx)),
+          (ctx) => {
+            rootChangesetConfigRules(ctx);
           },
         );
         seed(".changeset/scaffold.md", text(EMPTY_CHANGESET_SEED));