fix: use original block values upon retry

Author: andrew-polk

src/pull.ts

@@ -292,7 +292,7 @@ export async function executeWithRateLimitAndRetries<T>(
         warning(
           `While doing "${label}", got error "${
             e.message as string
-          }". Will retry after  ${secondsToWait}s...`
+          }". Will retry after ${secondsToWait}s...`
         );
         await new Promise(resolve => setTimeout(resolve, 1000 * secondsToWait));
       } else {

src/transform.ts

@@ -163,7 +163,13 @@ async function doNotionToMarkdown(
     "notionToMarkdown.blocksToMarkdown",
     async () => {
       mdBlocks = await docunotionContext.notionToMarkdown.blocksToMarkdown(
-        blocks
+        // We need to provide a copy of blocks.
+        // Calling blocksToMarkdown can modify the values in the blocks. If it does, and then
+        // we have to retry, we end up retrying with the modified values, which
+        // causes various issues (like using the transformed image url instead of the original one).
+        // Note, currently, we don't do anything else with blocks after this.
+        // If that changes, we'll need to figure out a more sophisticated approach.
+        JSON.parse(JSON.stringify(blocks))
       );
     }
   );