@sequoialabs/payload-plugin-reversia 0.1.2 → 0.1.4

package/dist/endpoints/resources-insert.js

@@ -1,5 +1,32 @@
 import { unauthorizedResponse, validateApiKey } from '../utils/auth.js';
 import { deflatePopulatedRelationships, deserializeFieldValue, findLocalizedFields, } from '../utils/fields.js';
+const WRITE_CONFLICT_MAX_RETRIES = 3;
+const WRITE_CONFLICT_BASE_DELAY_MS = 50;
+function isWriteConflict(error) {
+    if (!error || typeof error !== 'object') {
+        return false;
+    }
+    const message = 'message' in error && typeof error.message === 'string'
+        ? error.message
+        : '';
+    return message.includes('Write conflict') || message.includes('write conflict');
+}
+async function withRetry(fn) {
+    for (let attempt = 0;; attempt++) {
+        try {
+            return await fn();
+        }
+        catch (error) {
+            if (attempt < WRITE_CONFLICT_MAX_RETRIES && isWriteConflict(error)) {
+                const delay = WRITE_CONFLICT_BASE_DELAY_MS * 2 ** attempt +
+                    Math.random() * WRITE_CONFLICT_BASE_DELAY_MS;
+                await new Promise((resolve) => setTimeout(resolve, delay));
+                continue;
+            }
+            throw error;
+        }
+    }
+}
 function stringifyDiffValue(value) {
     if (typeof value === 'string') {
         return value;
@@ -29,15 +56,53 @@ function applyTranslations({ allowedFields, data, sourceDoc, previousDoc, }) {
     // populated `{ id, filename, url }` objects that Payload's update validator
     // rejects.
     const cleanSource = sourceDoc ? deflatePopulatedRelationships(sourceDoc) : null;
-    // Build updateData ONLY from fields Reversia actually sent translations for.
-    // Never pre-clone containers that aren't being translated — writing them
-    // back would overwrite other locales' existing values with source-language
-    // content, because Payload's update replaces the entire localized slot.
+    // Build updateData from fields Reversia sent, plus any required fields that
+    // are empty in the target locale. Payload validates ALL required fields on
+    // every update (not just the ones in `data`), so an empty required field in
+    // the target locale would block the entire write. We seed those from source.
     const updateData = {};
     const diff = {};
     const acceptedFields = [];
     const previous = previousDoc ?? null;
     const source = cleanSource ?? null;
+    const dataKeys = new Set(Object.keys(data));
+    // Seed required fields not being translated so Payload's whole-document
+    // validation doesn't reject the update.
+    //
+    // Scalars: only seed when the target locale has no value at all (null /
+    // undefined / empty string). If the target already has something, leave it.
+    //
+    // Containers: always seed from source when not in `data`. The target might
+    // have a "structurally present but content-empty" value (e.g. Payload
+    // auto-creates 9 default block items from `defaultValue` but every block's
+    // required inner richText is null). A shallow null-check misses this; a
+    // deep check is impractical. Seeding writes source-locale content as a
+    // placeholder — once Reversia translates the field and sends it in `data`,
+    // the seed is overwritten with the real translation.
+    for (const field of allowedFields) {
+        if (!field.hasRequiredLeaf) {
+            continue;
+        }
+        if (dataKeys.has(field.name)) {
+            continue; // Reversia is sending it — will be handled below
+        }
+        const sourceValue = source ? source[field.name] : undefined;
+        if (sourceValue === undefined || sourceValue === null) {
+            continue; // Source is also empty — nothing we can do
+        }
+        if (field.isContainer) {
+            // Always seed containers from source — inner required subfields may be
+            // empty even when the container itself is non-null in the target.
+            updateData[field.name] = deflatePopulatedRelationships(structuredClone(sourceValue));
+        }
+        else {
+            // Scalars: only seed when truly empty in target.
+            const targetValue = previous ? previous[field.name] : undefined;
+            if (targetValue === undefined || targetValue === null || targetValue === '') {
+                updateData[field.name] = sourceValue;
+            }
+        }
+    }
     for (const [fieldName, translatedValue] of Object.entries(data)) {
         const field = fieldByName.get(fieldName);
         if (!field) {
@@ -118,12 +183,12 @@ export function createResourcesInsertEndpoint(pluginConfig, collectionsMap, glob
                             response.errors.push(`Item ${index}: no recognised fields in data (keys: ${Object.keys(item.data).join(', ')})`);
                             continue;
                         }
-                        await req.payload.updateGlobal({
+                        await withRetry(() => req.payload.updateGlobal({
                             slug: globalSlug,
                             locale: item.targetLocale,
                             data: updateData,
                             context: { reversiaInsertion: true },
-                        });
+                        }));
                         response[index] = { index, type: item.type, id: globalSlug, diff };
                         continue;
                     }
@@ -141,6 +206,7 @@ export function createResourcesInsertEndpoint(pluginConfig, collectionsMap, glob
                         response.errors.push(`Item ${index}: id is required for collection resources`);
                         continue;
                     }
+                    const itemId = item.id;
                     const allowedFields = findLocalizedFields(collection.fields);
                     const [sourceDoc, previousDoc] = await Promise.all([
                         req.payload.findByID({
@@ -166,14 +232,14 @@ export function createResourcesInsertEndpoint(pluginConfig, collectionsMap, glob
                         response.errors.push(`Item ${index}: no recognised fields in data (keys: ${Object.keys(item.data).join(', ')})`);
                         continue;
                     }
-                    await req.payload.update({
+                    await withRetry(() => req.payload.update({
                         collection: slug,
-                        id: item.id,
+                        id: itemId,
                         locale: item.targetLocale,
                         data: updateData,
                         context: { reversiaInsertion: true },
-                    });
-                    response[index] = { index, type: item.type, id: item.id, diff };
+                    }));
+                    response[index] = { index, type: item.type, id: itemId, diff };
                 }
                 catch (error) {
                     recordInsertionFailure(req, response, error, index, item);

package/dist/types.d.ts

@@ -110,6 +110,10 @@ export interface TranslatableFieldConfig {
     behavior?: ReversiaFieldBehavior;
     type?: ReversiaFieldType;
     selected?: boolean;
+    rules?: {
+        maxLength?: number;
+        minLength?: number;
+    };
 }
 export interface ResourceDefinition {
     type: string;
@@ -168,6 +172,17 @@ export interface LocalizedFieldInfo {
     reversia?: ReversiaFieldCustom;
     /** One descriptor per localized leaf inside the container. */
     leaves: LocalizedLeaf[];
+    /** Payload `maxLength` on the field (text / textarea / code). */
+    maxLength?: number;
+    /** Payload `minLength` on the field (text / textarea / code). */
+    minLength?: number;
+    /**
+     * True when Payload marks this field (or any required inner leaf) as
+     * `required`. Used during insertion to seed empty target-locale slots from
+     * the source locale so Payload's whole-document validation doesn't reject
+     * the update.
+     */
+    hasRequiredLeaf?: boolean;
 }
 export interface ResourceItem {
     id: string;

package/dist/utils/fields.js

@@ -2,9 +2,6 @@ import { ReversiaFieldType } from '../types.js';
 import { applyByKeys, compileKeyMatcher, DEFAULT_RICHTEXT_KEYS, extractByKeys, } from './json-extract.js';
 import { resolveStaticLabel } from './labels.js';
 import { applyTranslationsToContainer, joinPointers, resolveLeafLocations, } from './path-resolver.js';
-/* -------------------------------------------------------------------------- */
-/*  Schema introspection                                                      */
-/* -------------------------------------------------------------------------- */
 function getFieldLabel(field) {
     if (!('name' in field)) {
         return '';
@@ -59,6 +56,14 @@ const SCALAR_PAYLOAD_TYPES = new Set([
 function isScalarPayloadType(type) {
     return SCALAR_PAYLOAD_TYPES.has(type);
 }
+function getNumericProp(field, prop) {
+    const val = field[prop];
+    return typeof val === 'number' && Number.isFinite(val) ? val : undefined;
+}
+function getBoolProp(field, prop) {
+    const val = field[prop];
+    return typeof val === 'boolean' ? val : undefined;
+}
 /* -------------------------------------------------------------------------- */
 /*  Discovery: top-level localized fields                                     */
 /* -------------------------------------------------------------------------- */
@@ -126,6 +131,9 @@ function describeTopLevelField(field) {
     // Top-level scalar that is itself localized → non-container, value shipped
     // as a primitive.
     if (isLocalized(field) && isScalarPayloadType(payloadFieldType)) {
+        const maxLength = getNumericProp(field, 'maxLength');
+        const minLength = getNumericProp(field, 'minLength');
+        const required = getBoolProp(field, 'required');
         return {
             name,
             label,
@@ -140,6 +148,9 @@ function describeTopLevelField(field) {
                     reversia,
                 },
             ],
+            ...(maxLength !== undefined && { maxLength }),
+            ...(minLength !== undefined && { minLength }),
+            ...(required && { hasRequiredLeaf: true }),
         };
     }
     // Top-level richText / json (localized or not localized but containing
@@ -163,6 +174,7 @@ function describeTopLevelField(field) {
                     reversia,
                 },
             ],
+            ...(getBoolProp(field, 'required') && { hasRequiredLeaf: true }),
         };
     }
     // Structured wrapper: group / array / blocks. Walk inside to collect
@@ -192,6 +204,11 @@ function describeTopLevelField(field) {
         isContainer: true,
         reversia,
         leaves: innerLeaves,
+        // Containers (arrays, blocks, groups) almost always have required inner
+        // subfields we can't cheaply enumerate. Marking hasRequiredLeaf ensures we
+        // always seed empty target-locale containers from source so Payload's
+        // whole-document required-field validation doesn't reject the update.
+        hasRequiredLeaf: true,
     };
 }
 function collectLeaves(fields, parentSegments, out) {
@@ -272,6 +289,27 @@ function resolveAsLabel(field, hasLabelAlready) {
     }
     return false;
 }
+function resolveRules(field) {
+    // Rules only make sense for non-container scalars — containers ship as JSON
+    // pointer maps where the concept of "max length" doesn't apply at the
+    // top-level entry.
+    if (field.isContainer) {
+        return undefined;
+    }
+    const maxLength = field.maxLength;
+    const minLength = field.minLength;
+    if (maxLength === undefined && minLength === undefined) {
+        return undefined;
+    }
+    const rules = {};
+    if (maxLength !== undefined) {
+        rules.maxLength = maxLength;
+    }
+    if (minLength !== undefined) {
+        rules.minLength = minLength;
+    }
+    return rules;
+}
 function humanize(segment) {
     const spaced = segment
         .replace(/[_-]+/g, ' ')
@@ -309,6 +347,10 @@ export function buildTranslatableConfiguration(localizedFields) {
         if (field.reversia?.selected === false) {
             fieldConfig.selected = false;
         }
+        const rules = resolveRules(field);
+        if (rules) {
+            fieldConfig.rules = rules;
+        }
         config[field.name] = fieldConfig;
     }
     return config;
@@ -417,9 +459,6 @@ function resolveTranslatableKeys(leaf) {
     }
     return null;
 }
-/* -------------------------------------------------------------------------- */
-/*  Deserialization                                                           */
-/* -------------------------------------------------------------------------- */
 /**
  * Reverses `serializeField` for one top-level field.
  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sequoialabs/payload-plugin-reversia",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "author": {
     "name": "Jean Walrave",
     "email": "contact@reversia.tech",