@njdamstra/appwrite-utils-cli 1.11.7 → 1.11.8

package/dist/migrations/migrateStrings.js

@@ -250,6 +250,31 @@ export async function executeMigrationPlan(adapter, options) {
             skipped += entries.length;
             continue;
         }
+        // Temporarily set ALL required attributes in this collection to optional.
+        // Appwrite rejects partial updateRow calls if a required attribute is missing
+        // from the payload — even if the field exists in the document. This affects
+        // every data-copy step during migration.
+        const schemaRes = await tryAwaitWithRetry(() => adapter.getTable({ databaseId: first.databaseId, tableId: first.collectionId }));
+        const allAttrs = schemaRes?.data?.attributes || schemaRes?.data?.columns || [];
+        const originallyRequired = allAttrs
+            .filter((a) => a.required === true && a.status === "available")
+            .map((a) => a.key);
+        if (originallyRequired.length > 0) {
+            MessageFormatter.info(`  Temporarily setting ${originallyRequired.length} required attribute(s) to optional...`, { prefix: "Execute" });
+            for (const key of originallyRequired) {
+                try {
+                    await tryAwaitWithRetry(() => adapter.updateAttribute({
+                        databaseId: first.databaseId,
+                        tableId: first.collectionId,
+                        key,
+                        required: false,
+                    }));
+                }
+                catch {
+                    // Non-fatal — attribute might not support updating required
+                }
+            }
+        }
         // Migrate each attribute in this collection
         for (const entry of entries) {
             const cpEntry = getOrCreateCheckpointEntry(checkpoint, entry);
@@ -275,25 +300,23 @@ export async function executeMigrationPlan(adapter, options) {
                 MessageFormatter.error(`  ${entry.attributeKey}: FAILED — ${cpEntry.error}`, undefined, { prefix: "Execute" });
             }
         }
-        // Restore required flags after all attributes in this collection are done.
-        // This must happen AFTER all migrations to avoid partial-update validation
-        // errors (Appwrite rejects updateRow if a required attribute is missing
-        // from the payload, even for partial updates).
-        const completedRequired = entries.filter((e) => {
-            const cp = findCheckpointEntry(checkpoint, e);
-            return cp?.phase === "completed" && e.isRequired;
-        });
-        for (const entry of completedRequired) {
-            try {
-                await tryAwaitWithRetry(() => adapter.updateAttribute({
-                    databaseId: entry.databaseId,
-                    tableId: entry.collectionId,
-                    key: entry.attributeKey,
-                    required: true,
-                }));
-            }
-            catch {
-                MessageFormatter.info(`  Warning: could not set ${entry.attributeKey} back to required`, { prefix: "Execute" });
+        // Restore required flags for ALL originally-required attributes.
+        // This covers both migrated attributes (recreated as optional) and
+        // non-migrated attributes (temporarily set to optional above).
+        if (originallyRequired.length > 0) {
+            MessageFormatter.info(`  Restoring ${originallyRequired.length} required attribute(s)...`, { prefix: "Execute" });
+            for (const key of originallyRequired) {
+                try {
+                    await tryAwaitWithRetry(() => adapter.updateAttribute({
+                        databaseId: first.databaseId,
+                        tableId: first.collectionId,
+                        key,
+                        required: true,
+                    }));
+                }
+                catch {
+                    MessageFormatter.info(`  Warning: could not restore required flag for ${key}`, { prefix: "Execute" });
+                }
             }
         }
         // After collection completes, offer to update local YAML

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@njdamstra/appwrite-utils-cli",
   "description": "Appwrite Utility Functions to help with database management, data conversion, data import, migrations, and much more. Meant to be used as a CLI tool, I do not recommend installing this in frontend environments.",
-  "version": "1.11.7",
+  "version": "1.11.8",
   "main": "dist/main.js",
   "type": "module",
   "repository": {

package/src/migrations/migrateStrings.ts

@@ -338,6 +338,40 @@ export async function executeMigrationPlan(
       continue;
     }
 
+    // Temporarily set ALL required attributes in this collection to optional.
+    // Appwrite rejects partial updateRow calls if a required attribute is missing
+    // from the payload — even if the field exists in the document. This affects
+    // every data-copy step during migration.
+    const schemaRes = await tryAwaitWithRetry(() =>
+      adapter.getTable({ databaseId: first.databaseId, tableId: first.collectionId })
+    );
+    const allAttrs: any[] =
+      schemaRes?.data?.attributes || schemaRes?.data?.columns || [];
+    const originallyRequired = allAttrs
+      .filter((a: any) => a.required === true && a.status === "available")
+      .map((a: any) => a.key as string);
+
+    if (originallyRequired.length > 0) {
+      MessageFormatter.info(
+        `  Temporarily setting ${originallyRequired.length} required attribute(s) to optional...`,
+        { prefix: "Execute" }
+      );
+      for (const key of originallyRequired) {
+        try {
+          await tryAwaitWithRetry(() =>
+            adapter.updateAttribute({
+              databaseId: first.databaseId,
+              tableId: first.collectionId,
+              key,
+              required: false,
+            } as any)
+          );
+        } catch {
+          // Non-fatal — attribute might not support updating required
+        }
+      }
+    }
+
     // Migrate each attribute in this collection
     for (const entry of entries) {
       const cpEntry = getOrCreateCheckpointEntry(checkpoint, entry);
@@ -382,29 +416,30 @@ export async function executeMigrationPlan(
       }
     }
 
-    // Restore required flags after all attributes in this collection are done.
-    // This must happen AFTER all migrations to avoid partial-update validation
-    // errors (Appwrite rejects updateRow if a required attribute is missing
-    // from the payload, even for partial updates).
-    const completedRequired = entries.filter((e) => {
-      const cp = findCheckpointEntry(checkpoint, e);
-      return cp?.phase === "completed" && e.isRequired;
-    });
-    for (const entry of completedRequired) {
-      try {
-        await tryAwaitWithRetry(() =>
-          adapter.updateAttribute({
-            databaseId: entry.databaseId,
-            tableId: entry.collectionId,
-            key: entry.attributeKey,
-            required: true,
-          } as any)
-        );
-      } catch {
-        MessageFormatter.info(
-          `  Warning: could not set ${entry.attributeKey} back to required`,
-          { prefix: "Execute" }
-        );
+    // Restore required flags for ALL originally-required attributes.
+    // This covers both migrated attributes (recreated as optional) and
+    // non-migrated attributes (temporarily set to optional above).
+    if (originallyRequired.length > 0) {
+      MessageFormatter.info(
+        `  Restoring ${originallyRequired.length} required attribute(s)...`,
+        { prefix: "Execute" }
+      );
+      for (const key of originallyRequired) {
+        try {
+          await tryAwaitWithRetry(() =>
+            adapter.updateAttribute({
+              databaseId: first.databaseId,
+              tableId: first.collectionId,
+              key,
+              required: true,
+            } as any)
+          );
+        } catch {
+          MessageFormatter.info(
+            `  Warning: could not restore required flag for ${key}`,
+            { prefix: "Execute" }
+          );
+        }
       }
     }