Ellyse/ct 580 gmail and rss importers have failures clean (#1424)

* fixed the path generation when setting array length to 0
* fixed array detection in retry logic
* cherrypick bernis change for handling for root writes with value property

Author: ellyxir

packages/runner/src/data-updating.ts

@@ -335,7 +335,7 @@ export function normalizeAndDiff(
           i++
         ) {
           changes.push({
-            location: { ...link, path: [...link.path, i.toString()] },
+            location: { ...link, path: [...link.path.slice(0, -1), i.toString()] },
             value: undefined,
           });
         }

packages/runner/src/storage/transaction-shim.ts

@@ -312,6 +312,12 @@ class TransactionWriter extends TransactionReader
       throw new Error(`Failed to get or create document: ${address.id}`);
     }
 
+    // Rewrite creating new documents as setting the value
+    if (address.path.length === 0 && isObject(value) && "value" in value) {
+      address = { ...address, path: ["value"] };
+      value = value.value;
+    }
+
     // Path-based logic
     if (!address.path.length) {
       const notFoundError: INotFoundError = new Error(
@@ -588,7 +594,7 @@ export class ExtendedStorageTransaction implements IExtendedStorageTransaction {
       for (const key of remainingPath) {
         nextValue =
           nextValue[key] =
-            (typeof Number(key) === "number" ? [] : {}) as typeof nextValue;
+            (!Number.isNaN(Number(key)) ? [] : {}) as typeof nextValue;
       }
       nextValue[lastKey] = value;
       const parentAddress = { ...address, path: lastValidPath ?? [] };