Skip to content

Demo (broken thread): close the debt task without adding the proof #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
rdryfoos wants to merge 2 commits into
base: main
Choose a base branch
from
+1 −4
Open

Demo (broken thread): close the debt task without adding the proof #2

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
644af64
example-app demo (broken): close the debt task without the proof → GAP
claude Aug 10, 2026
09e3460
Merge branch 'main' into demo/break-sync-gap
claude Aug 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 1 addition & 4 deletions examples/example-app/specs/sync/tasks.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,7 @@ excused from the silent-gap check while the TODO stays open.
- [x] T001 Reconcile queued ops on reconnect and fan the merged log out — **Carries**: FR-SYNC-01, AC-SYNC-01
- [x] T002 Queue edits locally while offline and apply them optimistically — **Carries**: AC-OFFL-01
- [x] T013 Keyboard reachability + assistive-tech announcements for list actions — **Carries**: AC-A11Y-01

## Tracked debt

- [ ] T005 Field-level merge test for disjoint edits — **Carries**: AC-SYNC-02 — deferred to the sync milestone
- [x] T005 Field-level merge test for disjoint edits — **Carries**: AC-SYNC-02

@coderabbitai coderabbitai Bot Aug 10, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Do not mark T005 completed without the AC-SYNC-02 proof.

examples/example-app/tests/test_sync.py:1-11 states that AC-SYNC-02 is not proved and remains tracked debt under T005. This line creates a false completion state and a silent acceptance gap. Add the promised disjoint-field merge test and remove the open TODO, or keep T005 under tracked debt until the proof exists.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@examples/example-app/specs/sync/tasks.md` at line 12, Align T005’s completion
status with the AC-SYNC-02 proof: add the promised disjoint-field merge test and
remove the related open TODO, or uncheck T005 and keep it tracked as incomplete
until that proof exists.


## Planning altitude (backlog — referenced so the registry set stays closed)

Expand Down
Loading