diff --git a/src/features/boulder-state/storage.test.ts b/src/features/boulder-state/storage.test.ts index e52174cef..a624773b2 100644 --- a/src/features/boulder-state/storage.test.ts +++ b/src/features/boulder-state/storage.test.ts @@ -351,7 +351,7 @@ describe("boulder-state", () => { expect(progress.isComplete).toBe(true) }) - test("should return isComplete true for empty plan", () => { + test("should return isComplete false for plan with content but no checkboxes", () => { // given - plan with no checkboxes const planPath = join(TEST_DIR, "empty-plan.md") writeFileSync(planPath, "# Plan\nNo tasks here") @@ -361,7 +361,7 @@ describe("boulder-state", () => { // then expect(progress.total).toBe(0) - expect(progress.isComplete).toBe(true) + expect(progress.isComplete).toBe(false) }) test("should handle non-existent file", () => { diff --git a/src/features/boulder-state/storage.ts b/src/features/boulder-state/storage.ts index c9ac83993..fd4a2d094 100644 --- a/src/features/boulder-state/storage.ts +++ b/src/features/boulder-state/storage.ts @@ -133,7 +133,7 @@ export function getPlanProgress(planPath: string): PlanProgress { return { total, completed, - isComplete: total === 0 || completed === total, + isComplete: total > 0 && completed === total, } } catch { return { total: 0, completed: 0, isComplete: true }