@byline/host-tanstack-start 3.16.1 → 3.17.1

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

@@ -25,7 +25,7 @@ function parseUploadFormData(data) {
         fields
     };
 }
-function resolveUploadFieldName(definition, collectionPath, requested) {
+function resolveUploadField(definition, collectionPath, requested) {
     const candidates = getUploadFields(definition);
     if (requested) {
         const match = candidates.find((f)=>f.name === requested);
@@ -37,9 +37,9 @@ function resolveUploadFieldName(definition, collectionPath, requested) {
                 available: candidates.map((f)=>f.name)
             }
         });
-        return match.name;
+        return match;
     }
-    if (1 === candidates.length) return candidates[0].name;
+    if (1 === candidates.length) return candidates[0];
     if (0 === candidates.length) throw ERR_VALIDATION({
         message: `Collection '${collectionPath}' has no upload-capable image/file field. Add an 'upload' block to one of its image/file fields.`,
         details: {
@@ -72,10 +72,9 @@ const uploadCollectionField = createServerFn({
             }
         }, uploadCollectionField).log(logger);
         const serverConfig = getServerConfig();
-        const resolvedFieldName = resolveUploadFieldName(config.definition, collectionPath, fieldName);
-        const targetField = config.definition.fields.find((f)=>f.name === resolvedFieldName);
+        const targetField = resolveUploadField(config.definition, collectionPath, fieldName);
         const storage = targetField.upload?.storage ?? serverConfig.storage;
-        if (!storage) throw new Error(`No storage provider configured for field '${resolvedFieldName}' on collection '${collectionPath}'. Set either field.upload.storage or the site-wide ServerConfig.storage.`);
+        if (!storage) throw new Error(`No storage provider configured for field '${targetField.name}' on collection '${collectionPath}'. Set either field.upload.storage or the site-wide ServerConfig.storage.`);
         let buffer;
         try {
             buffer = Buffer.from(await file.arrayBuffer());
@@ -92,7 +91,7 @@ const uploadCollectionField = createServerFn({
             collectionId: config.collection.id,
             collectionVersion: config.collection.version,
             collectionPath,
-            fieldName: resolvedFieldName,
+            fieldName: targetField.name,
             storage,
             logger,
             defaultLocale: serverConfig.i18n.content.defaultLocale,

package/package.json

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

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

@@ -84,16 +84,17 @@ function parseUploadFormData(data: FormData): UploadDocumentInput {
  * wins; otherwise default to the unique upload-capable field. Throws
  * `ERR_VALIDATION` when the request is ambiguous.
  *
- * Walks `definition.fields` only (not nested `group` / `array` /
- * `blocks`) — top-level upload fields are the supported case for the
- * transport today. Nested-field uploads remain available via the core
- * service when wired into a custom transport.
+ * Candidates come from `getUploadFields`, which recurses into `group` /
+ * `array` / `blocks` — upload fields nested in repeating structures are
+ * addressed by their leaf field name. Note the ambiguity fallback counts
+ * nested fields too: a collection whose only extra upload field is
+ * nested still requires an explicit `field` selector.
  */
-function resolveUploadFieldName(
+function resolveUploadField(
   definition: CollectionDefinition,
   collectionPath: string,
   requested: string | null
-): string {
+): ImageField | FileField {
   const candidates = getUploadFields(definition)
 
   if (requested) {
@@ -110,10 +111,10 @@ function resolveUploadFieldName(
         },
       })
     }
-    return match.name
+    return match
   }
 
-  if (candidates.length === 1) return candidates[0].name
+  if (candidates.length === 1) return candidates[0]
 
   if (candidates.length === 0) {
     throw ERR_VALIDATION({
@@ -167,19 +168,12 @@ export const uploadCollectionField = createServerFn({ method: 'POST' })
         }
 
         const serverConfig = getServerConfig()
-        const resolvedFieldName = resolveUploadFieldName(
-          config.definition,
-          collectionPath,
-          fieldName
-        )
-        const targetField = config.definition.fields.find((f) => f.name === resolvedFieldName) as
-          | ImageField
-          | FileField
+        const targetField = resolveUploadField(config.definition, collectionPath, fieldName)
         // Per-field storage routing falls through to the site-wide default.
         const storage = targetField.upload?.storage ?? serverConfig.storage
         if (!storage) {
           throw new Error(
-            `No storage provider configured for field '${resolvedFieldName}' on collection ` +
+            `No storage provider configured for field '${targetField.name}' on collection ` +
               `'${collectionPath}'. Set either field.upload.storage or the site-wide ` +
               'ServerConfig.storage.'
           )
@@ -199,7 +193,7 @@ export const uploadCollectionField = createServerFn({ method: 'POST' })
           collectionId: config.collection.id,
           collectionVersion: config.collection.version,
           collectionPath,
-          fieldName: resolvedFieldName,
+          fieldName: targetField.name,
           storage,
           logger,
           defaultLocale: serverConfig.i18n.content.defaultLocale,