fix(sync): Pass store parameter in recursive shouldMarkCompleted calls

The shouldMarkCompleted function recursively calls itself without passing
the store parameter, causing 3+ level task hierarchies to fail. Intermediate
tasks without commits are incorrectly treated as leaf tasks, preventing
parent issues from closing even when all actual leaf tasks have commits.

Fixes #110

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: dcramer

src/core/github/sync.ts

@@ -696,7 +696,9 @@ export class GitHubSyncService {
       const descendants = collectDescendants(store.tasks, task.id);
       if (descendants.length > 0) {
         // Parent is "completed" only when ALL descendants have verified commits on remote
-        return descendants.every((d) => this.shouldMarkCompleted(d.task));
+        return descendants.every((d) =>
+          this.shouldMarkCompleted(d.task, store),
+        );
       }
     }
 

src/core/shortcut/sync.ts

@@ -904,7 +904,9 @@ export class ShortcutSyncService {
       const descendants = collectDescendants(store.tasks, task.id);
       if (descendants.length > 0) {
         // Parent is "completed" only when ALL descendants have verified commits on remote
-        return descendants.every((d) => this.shouldMarkCompleted(d.task));
+        return descendants.every((d) =>
+          this.shouldMarkCompleted(d.task, store),
+        );
       }
     }