@byline/host-tanstack-start 3.2.1 → 3.3.0

package/dist/admin-shell/collections/edit.js

@@ -4,7 +4,7 @@ import { FormRenderer } from "@byline/admin/react";
 import { getDefaultStatus, getWorkflowStatuses } from "@byline/core";
 import { useTranslation } from "@byline/i18n/react";
 import { Container, Section, useToastManager } from "@byline/ui/react";
-import { copyDocumentToLocale, deleteDocument, deleteDocumentLocale, duplicateCollectionDocument, unpublishDocument, updateCollectionDocumentWithPatches, updateDocumentStatus } from "../../server-fns/collections/index.js";
+import { copyDocumentToLocale, deleteDocument, deleteDocumentLocale, duplicateCollectionDocument, unpublishDocument, updateCollectionDocumentSystemFields, updateCollectionDocumentWithPatches, updateDocumentStatus } from "../../server-fns/collections/index.js";
 import { useNavigate } from "../chrome/loose-router.js";
 import { useTanStackNavigationGuard } from "./tanstack-navigation-guard.js";
 import { ViewMenu } from "./view-menu.js";
@@ -409,23 +409,30 @@ const EditView = ({ collectionDefinition, adminConfig, initialData, locale, cont
             });
         }
     };
-    const handleSubmit = async ({ data: _data, patches, systemPath, systemAvailableLocales })=>{
+    const handleSubmit = async ({ data: _data, patches, contentDirty, pathDirty, systemPath, availableLocalesDirty, systemAvailableLocales })=>{
         try {
-            await updateCollectionDocumentWithPatches({
+            if (pathDirty || availableLocalesDirty) await updateCollectionDocumentSystemFields({
                 data: {
                     collection: path,
                     id: String(initialData.id),
-                    patches,
-                    versionId: initialData.versionId,
                     locale,
-                    ...systemPath ? {
-                        path: systemPath
+                    ...pathDirty ? {
+                        path: systemPath ?? null
                     } : {},
-                    ...systemAvailableLocales ? {
-                        availableLocales: systemAvailableLocales
+                    ...availableLocalesDirty ? {
+                        availableLocales: systemAvailableLocales ?? []
                     } : {}
                 }
             });
+            if (contentDirty) await updateCollectionDocumentWithPatches({
+                data: {
+                    collection: path,
+                    id: String(initialData.id),
+                    patches,
+                    versionId: initialData.versionId,
+                    locale
+                }
+            });
             const description = t("collections.edit.updatedDescription", {
                 label: singularLower
             });

package/dist/server-fns/collections/update.d.ts

@@ -12,15 +12,30 @@ export declare const updateCollectionDocumentWithPatches: import("@tanstack/reac
     patches: DocumentPatch[];
     versionId?: string;
     locale?: string;
-    path?: string;
-    availableLocales?: string[];
 }) => {
     collection: string;
     id: string;
     patches: DocumentPatch[];
     versionId?: string;
     locale?: string;
-    path?: string;
+}, Promise<{
+    status: "ok";
+}>>;
+export declare const updateCollectionDocumentSystemFields: import("@tanstack/react-start").RequiredFetcher<undefined, (input: {
+    collection: string;
+    id: string;
+    locale?: string;
+    /** Path override; `null`/omitted means no path write. */
+    path?: string | null;
+    /** Editorial advertised-locale set; omitted means no advertised-locale write. */
+    availableLocales?: string[];
+}) => {
+    collection: string;
+    id: string;
+    locale?: string;
+    /** Path override; `null`/omitted means no path write. */
+    path?: string | null;
+    /** Editorial advertised-locale set; omitted means no advertised-locale write. */
     availableLocales?: string[];
 }, Promise<{
     status: "ok";

package/dist/server-fns/collections/update.js

@@ -1,12 +1,12 @@
 import { createServerFn } from "@tanstack/react-start";
 import { ERR_NOT_FOUND, getLogger, getServerConfig } from "@byline/core";
-import { updateDocumentWithPatches } from "@byline/core/services";
+import { updateDocumentSystemFields, updateDocumentWithPatches } from "@byline/core/services";
 import { getAdminRequestContext } from "../../auth/auth-context.js";
 import { ensureCollection } from "../../integrations/api-utils.js";
 const updateCollectionDocumentWithPatches = createServerFn({
     method: 'POST'
 }).inputValidator((input)=>input).handler(async ({ data: input })=>{
-    const { collection: path, id, patches, versionId, locale, path: explicitPath, availableLocales } = input;
+    const { collection: path, id, patches, versionId, locale } = input;
     const logger = getLogger();
     const config = await ensureCollection(path);
     if (!config) throw ERR_NOT_FOUND({
@@ -31,6 +31,38 @@ const updateCollectionDocumentWithPatches = createServerFn({
         documentId: id,
         patches,
         documentVersionId: versionId,
+        locale: locale ?? serverConfig.i18n.content.defaultLocale
+    });
+    return {
+        status: 'ok'
+    };
+});
+const updateCollectionDocumentSystemFields = createServerFn({
+    method: 'POST'
+}).inputValidator((input)=>input).handler(async ({ data: input })=>{
+    const { collection: path, id, locale, path: explicitPath, availableLocales } = input;
+    const logger = getLogger();
+    const config = await ensureCollection(path);
+    if (!config) throw ERR_NOT_FOUND({
+        message: 'Collection not found',
+        details: {
+            collectionPath: path
+        }
+    }).log(logger);
+    const serverConfig = getServerConfig();
+    const ctx = {
+        db: serverConfig.db,
+        definition: config.definition,
+        collectionId: config.collection.id,
+        collectionVersion: config.collection.version,
+        collectionPath: path,
+        logger,
+        defaultLocale: serverConfig.i18n.content.defaultLocale,
+        slugifier: serverConfig.slugifier,
+        requestContext: await getAdminRequestContext()
+    };
+    await updateDocumentSystemFields(ctx, {
+        documentId: id,
         locale: locale ?? serverConfig.i18n.content.defaultLocale,
         path: explicitPath,
         availableLocales
@@ -39,4 +71,4 @@ const updateCollectionDocumentWithPatches = createServerFn({
         status: 'ok'
     };
 });
-export { updateCollectionDocumentWithPatches };
+export { updateCollectionDocumentSystemFields, updateCollectionDocumentWithPatches };

package/package.json

@@ -3,7 +3,7 @@
   "private": false,
   "type": "module",
   "license": "MPL-2.0",
-  "version": "3.2.1",
+  "version": "3.3.0",
   "engines": {
     "node": ">=20.9.0"
   },
@@ -115,13 +115,13 @@
     "react-swipeable": "^7.0.2",
     "uuid": "^14.0.0",
     "zod": "^4.4.3",
-    "@byline/client": "3.2.1",
-    "@byline/auth": "3.2.1",
-    "@byline/core": "3.2.1",
-    "@byline/admin": "3.2.1",
-    "@byline/ai": "3.2.1",
-    "@byline/i18n": "3.2.1",
-    "@byline/ui": "3.2.1"
+    "@byline/admin": "3.3.0",
+    "@byline/auth": "3.3.0",
+    "@byline/i18n": "3.3.0",
+    "@byline/core": "3.3.0",
+    "@byline/ui": "3.3.0",
+    "@byline/client": "3.3.0",
+    "@byline/ai": "3.3.0"
   },
   "peerDependencies": {
     "@tanstack/react-router": "^1.167.0",

package/src/admin-shell/collections/edit.tsx

@@ -21,6 +21,7 @@ import {
   deleteDocumentLocale,
   duplicateCollectionDocument,
   unpublishDocument,
+  updateCollectionDocumentSystemFields,
   updateCollectionDocumentWithPatches,
   updateDocumentStatus,
 } from '../../server-fns/collections/index.js'
@@ -386,28 +387,57 @@ export const EditView = ({
   const handleSubmit = async ({
     data: _data,
     patches,
+    contentDirty,
+    pathDirty,
     systemPath,
+    availableLocalesDirty,
     systemAvailableLocales,
   }: {
     // biome-ignore lint/suspicious/noExplicitAny: data is collection-specific
     data: any
     // biome-ignore lint/suspicious/noExplicitAny: patches list shape
     patches: any[]
+    /** Document field data / patches changed → versioned write. */
+    contentDirty: boolean
+    /** Path widget changed → non-versioned direct write. */
+    pathDirty: boolean
     systemPath?: string | null
+    /** Available-locales widget changed → non-versioned direct write. */
+    availableLocalesDirty: boolean
     systemAvailableLocales?: string[]
   }) => {
     try {
-      await updateCollectionDocumentWithPatches({
-        data: {
-          collection: path,
-          id: String(initialData.id),
-          patches,
-          versionId: initialData.versionId as string | undefined,
-          locale,
-          ...(systemPath ? { path: systemPath } : {}),
-          ...(systemAvailableLocales ? { availableLocales: systemAvailableLocales } : {}),
-        },
-      })
+      // Document-grain system fields write first via their own non-versioned
+      // path — so a path conflict surfaces before we mint a content version,
+      // and these immediate writes never reset workflow status. See
+      // docs/I18N.md.
+      if (pathDirty || availableLocalesDirty) {
+        await updateCollectionDocumentSystemFields({
+          data: {
+            collection: path,
+            id: String(initialData.id),
+            locale,
+            ...(pathDirty ? { path: systemPath ?? null } : {}),
+            ...(availableLocalesDirty ? { availableLocales: systemAvailableLocales ?? [] } : {}),
+          },
+        })
+      }
+
+      // Content (field data / patches) follows the normal versioned path —
+      // mints a new draft version. Skipped entirely when only the system
+      // fields changed, so a path/advertising edit never creates an empty
+      // content version.
+      if (contentDirty) {
+        await updateCollectionDocumentWithPatches({
+          data: {
+            collection: path,
+            id: String(initialData.id),
+            patches,
+            versionId: initialData.versionId as string | undefined,
+            locale,
+          },
+        })
+      }
 
       const description = t('collections.edit.updatedDescription', { label: singularLower })
       toastManager.add({

package/src/server-fns/collections/update.ts

@@ -11,13 +11,18 @@ import { createServerFn } from '@tanstack/react-start'
 import { ERR_NOT_FOUND, getLogger, getServerConfig } from '@byline/core'
 import type { DocumentPatch } from '@byline/core/patches'
 import type { DocumentLifecycleContext } from '@byline/core/services'
-import { updateDocumentWithPatches } from '@byline/core/services'
+import { updateDocumentSystemFields, updateDocumentWithPatches } from '@byline/core/services'
 
 import { getAdminRequestContext } from '../../auth/auth-context.js'
 import { ensureCollection } from '../../integrations/api-utils.js'
 
 // ---------------------------------------------------------------------------
 // Apply patches (patch-based update — creates a new immutable version)
+//
+// Document-grain system fields (`path`, `availableLocales`) are deliberately
+// NOT handled here: they are written through their own non-versioned path
+// (`updateCollectionDocumentSystemFields` below) so that editing them does not
+// mint a new version or reset workflow status. See docs/I18N.md.
 // ---------------------------------------------------------------------------
 
 export const updateCollectionDocumentWithPatches = createServerFn({ method: 'POST' })
@@ -28,20 +33,10 @@ export const updateCollectionDocumentWithPatches = createServerFn({ method: 'POS
       patches: DocumentPatch[]
       versionId?: string
       locale?: string
-      path?: string
-      availableLocales?: string[]
     }) => input
   )
   .handler(async ({ data: input }) => {
-    const {
-      collection: path,
-      id,
-      patches,
-      versionId,
-      locale,
-      path: explicitPath,
-      availableLocales,
-    } = input
+    const { collection: path, id, patches, versionId, locale } = input
     const logger = getLogger()
     const config = await ensureCollection(path)
     if (!config) {
@@ -69,6 +64,58 @@ export const updateCollectionDocumentWithPatches = createServerFn({ method: 'POS
       patches,
       documentVersionId: versionId,
       locale: locale ?? serverConfig.i18n.content.defaultLocale,
+    })
+
+    return { status: 'ok' as const }
+  })
+
+// ---------------------------------------------------------------------------
+// System-managed, document-grain fields (path + advertised locales)
+//
+// Non-versioned, immediate write — does NOT create a new version or change
+// workflow status. Backs the admin path / available-locales widgets'
+// direct-write Save (the `direct-write` and `both` dirty-reason cases).
+// ---------------------------------------------------------------------------
+
+export const updateCollectionDocumentSystemFields = createServerFn({ method: 'POST' })
+  .inputValidator(
+    (input: {
+      collection: string
+      id: string
+      locale?: string
+      /** Path override; `null`/omitted means no path write. */
+      path?: string | null
+      /** Editorial advertised-locale set; omitted means no advertised-locale write. */
+      availableLocales?: string[]
+    }) => input
+  )
+  .handler(async ({ data: input }) => {
+    const { collection: path, id, locale, path: explicitPath, availableLocales } = input
+    const logger = getLogger()
+    const config = await ensureCollection(path)
+    if (!config) {
+      throw ERR_NOT_FOUND({
+        message: 'Collection not found',
+        details: { collectionPath: path },
+      }).log(logger)
+    }
+
+    const serverConfig = getServerConfig()
+    const ctx: DocumentLifecycleContext = {
+      db: serverConfig.db,
+      definition: config.definition,
+      collectionId: config.collection.id,
+      collectionVersion: config.collection.version,
+      collectionPath: path,
+      logger,
+      defaultLocale: serverConfig.i18n.content.defaultLocale,
+      slugifier: serverConfig.slugifier,
+      requestContext: await getAdminRequestContext(),
+    }
+
+    await updateDocumentSystemFields(ctx, {
+      documentId: id,
+      locale: locale ?? serverConfig.i18n.content.defaultLocale,
       path: explicitPath,
       availableLocales,
     })