fix bug when removing cards

skipping cards when iterating index
while also shortening array

Author: jonabc

dist/index.js

@@ -2270,8 +2270,8 @@ function run() {
             yield ensureCard(automationNoteCard, 0, targetColumn);
             // delete all cards in target column that do not exist in the source column,
             // except for the automation note
-            // start at index 1 to account for the automation note card
-            for (let index = 1; index < targetColumn.cards.nodes.length; index++) {
+            // don't iterate over index 0, to account for the automation note card
+            for (let index = targetColumn.cards.nodes.length - 1; index >= 1; index--) {
                 const targetCard = targetColumn.cards.nodes[index];
                 const [sourceCard] = findCard(targetCard, sourceColumn);
                 if (!sourceCard) {

src/main.ts

@@ -125,8 +125,8 @@ async function run(): Promise<void> {
 
     // delete all cards in target column that do not exist in the source column,
     // except for the automation note
-    // start at index 1 to account for the automation note card
-    for (let index = 1; index < targetColumn.cards.nodes.length; index++) {
+    // don't iterate over index 0, to account for the automation note card
+    for (let index = targetColumn.cards.nodes.length - 1; index >= 1; index--) {
       const targetCard = targetColumn.cards.nodes[index]
       const [sourceCard] = findCard(targetCard, sourceColumn)