@valbuild/server 0.136.8 → 0.136.10

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # @valbuild/server
 
+## 0.136.10
+
+### Patch Changes
+
+- [#743](https://github.com/valbuild/val/pull/743) [`fef29b0`](https://github.com/valbuild/val/commit/fef29b0d9e826e4e7e63a3ae61553137a3f9d280) Thanks [@freekh](https://github.com/freekh)! - Saving no longer gets stuck on "Changes cannot be saved: something else keeps changing them first" when another editor has an unpublished change and later work has been published.
+
+  A new change is now written on top of the latest change the content service has registered, which it reports alongside the list of pending changes, instead of the last change this deployment was shown. Since changes can be published independently, those two can differ for good: an unpublished change can sit before published ones that the running deployment already contains. The Studio and the MCP tools both name the reported head, and fall back to the old behaviour against a content service that does not report one.
+
+- Updated dependencies [[`fef29b0`](https://github.com/valbuild/val/commit/fef29b0d9e826e4e7e63a3ae61553137a3f9d280)]:
+  - @valbuild/ui@0.136.10
+  - @valbuild/shared@0.136.10
+
+## 0.136.9
+
+### Patch Changes
+
+- Updated dependencies [[`c2bd7e7`](https://github.com/valbuild/val/commit/c2bd7e7e537f00a269a54b72274fe5efb81c0ae9)]:
+  - @valbuild/ui@0.136.9
+
 ## 0.136.8
 
 ### Patch Changes

package/dist/declarations/src/ValOps.d.ts

@@ -141,6 +141,11 @@ export declare abstract class ValOps {
         commitSha?: CommitSha;
         sourcesSha: SourcesSha;
         patches: PatchId[];
+        /**
+         * The head of the chain, when the store reports one. See
+         * {@link OrderedPatches.headPatchId}.
+         */
+        headPatchId?: PatchId | null;
     } | {
         type: "error";
         error: GenericErrorMessage;
@@ -851,6 +856,16 @@ export type OrderedPatches = {
         } | null;
     }[];
     commits?: ValCommit[];
+    /**
+     * The head of the patch chain: the last patch registered on the branch,
+     * published or not. `null` when there is none.
+     *
+     * What a new patch names as its parent, and NOT the last of `patches` — see
+     * `chainHeadOf` in `@valbuild/shared` for why. Absent means the store does
+     * not report one: `ValOpsFS` (whose ordering log defines order, and which
+     * ignores the parent) and a content service that predates the field.
+     */
+    headPatchId?: PatchId | null;
     error?: GenericErrorMessage;
     errors?: PatchReadError[];
     unauthorized?: boolean;
@@ -862,6 +877,8 @@ export type OrderedPatchesMetadata = {
     })[];
     commits?: ValCommit[];
     deployments?: ValDeployment[];
+    /** See {@link OrderedPatches.headPatchId}. */
+    headPatchId?: OrderedPatches["headPatchId"];
     error?: GenericErrorMessage;
     errors?: OrderedPatches["errors"];
     unauthorized?: boolean;

package/dist/declarations/src/ValOpsHttp.d.ts

@@ -256,6 +256,8 @@ export declare class ValOpsHttp extends ValOps {
         patches: PatchId[];
         /** Of `patches`, the ones that have shipped. See the implementation. */
         appliedPatches: PatchId[];
+        /** The head of the chain. See {@link OrderedPatchesMetadata.headPatchId}. */
+        headPatchId?: PatchId | null;
         /** The newest commit, which is the publish head. */
         headCommitSha?: string;
     } | {

package/dist/valbuild-server.cjs.dev.js

@@ -7706,6 +7706,11 @@ const GetApplicablePatches = z.z.object({
       commitSha: z.z.string()
     }).nullable()
   })),
+  /**
+   * The head of the chain. See `OrderedPatches.headPatchId`. Optional because
+   * a content service that predates it sends nothing.
+   */
+  headPatchId: internal.PatchId.nullable().optional(),
   commits: z.z.array(z.z.object({
     commitSha: z.z.string(),
     /*
@@ -8431,6 +8436,13 @@ class ValOpsHttp extends ValOps {
       deployments: allPatchData.deployments || [],
       patches,
       appliedPatches,
+      /*
+       * The CHAIN head, not the publish head below. Spread: absent is a content
+       * service that does not report it, which is not the same as `null`.
+       */
+      ...(allPatchData.headPatchId !== undefined ? {
+        headPatchId: allPatchData.headPatchId
+      } : {}),
       /*
        * The PUBLISH head, which is not `commitSha`.
        *
@@ -8555,6 +8567,9 @@ class ValOpsHttp extends ValOps {
      * the same treatment and a home on `OrderedPatches` first.
      */
     let commits;
+    // A fact about the branch too, so taken from the first chunk that has it
+    // for the same reason `commits` is.
+    let headPatchId;
     if (patchIds === undefined || patchIds.length === 0) {
       return this.fetchPatchesInternal({
         patchIds: patchIds,
@@ -8575,6 +8590,9 @@ class ValOpsHttp extends ValOps {
       if (commits === undefined && res.commits !== undefined) {
         commits = res.commits;
       }
+      if (headPatchId === undefined && res.headPatchId !== undefined) {
+        headPatchId = res.headPatchId;
+      }
     }
     // Chunking is a query-string-length workaround, NOT a filter: the content
     // api returns every applicable patch per request regardless of which
@@ -8607,6 +8625,9 @@ class ValOpsHttp extends ValOps {
       // readers do — sees the same shape the unchunked path gives it.
       ...(commits !== undefined ? {
         commits
+      } : {}),
+      ...(headPatchId !== undefined ? {
+        headPatchId
       } : {})
     };
   }
@@ -8702,7 +8723,10 @@ class ValOpsHttp extends ValOps {
             commits,
             deployments,
             patches,
-            errors
+            errors,
+            ...(data.headPatchId !== undefined ? {
+              headPatchId: data.headPatchId
+            } : {})
           };
         }
         console.error("Could not parse patches response. Error: " + zodValidationError.fromError(parsed.error));

package/dist/valbuild-server.cjs.prod.js

@@ -7706,6 +7706,11 @@ const GetApplicablePatches = z.z.object({
       commitSha: z.z.string()
     }).nullable()
   })),
+  /**
+   * The head of the chain. See `OrderedPatches.headPatchId`. Optional because
+   * a content service that predates it sends nothing.
+   */
+  headPatchId: internal.PatchId.nullable().optional(),
   commits: z.z.array(z.z.object({
     commitSha: z.z.string(),
     /*
@@ -8431,6 +8436,13 @@ class ValOpsHttp extends ValOps {
       deployments: allPatchData.deployments || [],
       patches,
       appliedPatches,
+      /*
+       * The CHAIN head, not the publish head below. Spread: absent is a content
+       * service that does not report it, which is not the same as `null`.
+       */
+      ...(allPatchData.headPatchId !== undefined ? {
+        headPatchId: allPatchData.headPatchId
+      } : {}),
       /*
        * The PUBLISH head, which is not `commitSha`.
        *
@@ -8555,6 +8567,9 @@ class ValOpsHttp extends ValOps {
      * the same treatment and a home on `OrderedPatches` first.
      */
     let commits;
+    // A fact about the branch too, so taken from the first chunk that has it
+    // for the same reason `commits` is.
+    let headPatchId;
     if (patchIds === undefined || patchIds.length === 0) {
       return this.fetchPatchesInternal({
         patchIds: patchIds,
@@ -8575,6 +8590,9 @@ class ValOpsHttp extends ValOps {
       if (commits === undefined && res.commits !== undefined) {
         commits = res.commits;
       }
+      if (headPatchId === undefined && res.headPatchId !== undefined) {
+        headPatchId = res.headPatchId;
+      }
     }
     // Chunking is a query-string-length workaround, NOT a filter: the content
     // api returns every applicable patch per request regardless of which
@@ -8607,6 +8625,9 @@ class ValOpsHttp extends ValOps {
       // readers do — sees the same shape the unchunked path gives it.
       ...(commits !== undefined ? {
         commits
+      } : {}),
+      ...(headPatchId !== undefined ? {
+        headPatchId
       } : {})
     };
   }
@@ -8702,7 +8723,10 @@ class ValOpsHttp extends ValOps {
             commits,
             deployments,
             patches,
-            errors
+            errors,
+            ...(data.headPatchId !== undefined ? {
+              headPatchId: data.headPatchId
+            } : {})
           };
         }
         console.error("Could not parse patches response. Error: " + zodValidationError.fromError(parsed.error));

package/dist/valbuild-server.esm.js

@@ -8,7 +8,7 @@ import path__default from 'path';
 import fs from 'fs';
 import vm from 'node:vm';
 import { Module } from 'node:module';
-import { isSchemaSourceFixError, resolveSchemaSourceFixForError, Patch, getErrorMessageFromUnknownJson, JSONValue, PatchGroup, newestCommitSha, SerializedSchema, VAL_ENABLE_COOKIE_NAME, VAL_STATE_COOKIE, VAL_SESSION_COOKIE, Api } from '@valbuild/shared/internal';
+import { isSchemaSourceFixError, resolveSchemaSourceFixForError, Patch, getErrorMessageFromUnknownJson, PatchId as PatchId$1, JSONValue, PatchGroup, newestCommitSha, SerializedSchema, VAL_ENABLE_COOKIE_NAME, VAL_STATE_COOKIE, VAL_SESSION_COOKIE, Api } from '@valbuild/shared/internal';
 import { createUIRequestHandler } from '@valbuild/ui/server';
 import crypto$1 from 'crypto';
 import z$1, { z } from 'zod';
@@ -7672,6 +7672,11 @@ const GetApplicablePatches = z.object({
       commitSha: z.string()
     }).nullable()
   })),
+  /**
+   * The head of the chain. See `OrderedPatches.headPatchId`. Optional because
+   * a content service that predates it sends nothing.
+   */
+  headPatchId: PatchId$1.nullable().optional(),
   commits: z.array(z.object({
     commitSha: z.string(),
     /*
@@ -8397,6 +8402,13 @@ class ValOpsHttp extends ValOps {
       deployments: allPatchData.deployments || [],
       patches,
       appliedPatches,
+      /*
+       * The CHAIN head, not the publish head below. Spread: absent is a content
+       * service that does not report it, which is not the same as `null`.
+       */
+      ...(allPatchData.headPatchId !== undefined ? {
+        headPatchId: allPatchData.headPatchId
+      } : {}),
       /*
        * The PUBLISH head, which is not `commitSha`.
        *
@@ -8521,6 +8533,9 @@ class ValOpsHttp extends ValOps {
      * the same treatment and a home on `OrderedPatches` first.
      */
     let commits;
+    // A fact about the branch too, so taken from the first chunk that has it
+    // for the same reason `commits` is.
+    let headPatchId;
     if (patchIds === undefined || patchIds.length === 0) {
       return this.fetchPatchesInternal({
         patchIds: patchIds,
@@ -8541,6 +8556,9 @@ class ValOpsHttp extends ValOps {
       if (commits === undefined && res.commits !== undefined) {
         commits = res.commits;
       }
+      if (headPatchId === undefined && res.headPatchId !== undefined) {
+        headPatchId = res.headPatchId;
+      }
     }
     // Chunking is a query-string-length workaround, NOT a filter: the content
     // api returns every applicable patch per request regardless of which
@@ -8573,6 +8591,9 @@ class ValOpsHttp extends ValOps {
       // readers do — sees the same shape the unchunked path gives it.
       ...(commits !== undefined ? {
         commits
+      } : {}),
+      ...(headPatchId !== undefined ? {
+        headPatchId
       } : {})
     };
   }
@@ -8668,7 +8689,10 @@ class ValOpsHttp extends ValOps {
             commits,
             deployments,
             patches,
-            errors
+            errors,
+            ...(data.headPatchId !== undefined ? {
+              headPatchId: data.headPatchId
+            } : {})
           };
         }
         console.error("Could not parse patches response. Error: " + fromError(parsed.error));

package/package.json

@@ -16,7 +16,7 @@
     "./package.json": "./package.json"
   },
   "types": "dist/valbuild-server.cjs.d.ts",
-  "version": "0.136.8",
+  "version": "0.136.10",
   "devDependencies": {
     "@prettier/sync": "^0.6.1",
     "@types/jest": "^30.0.0",
@@ -31,8 +31,8 @@
     "zod": "^4.4.3",
     "zod-validation-error": "^5.0.0",
     "@valbuild/core": "0.136.8",
-    "@valbuild/ui": "0.136.8",
-    "@valbuild/shared": "0.136.8"
+    "@valbuild/shared": "0.136.10",
+    "@valbuild/ui": "0.136.10"
   },
   "engines": {
     "node": "^20.19.0 || >=22"