update sessions/0066/SESSION-0066.0-TRANSCRIPT-2026-07-02T16-24--INPROGRESS.md
This commit is contained in:
@@ -19,6 +19,41 @@
|
||||
|
||||
```
|
||||
|
||||
## Plan
|
||||
|
||||
> Anchor: #70 (type/bug, P2 — ELIGIBLE R2b)
|
||||
|
||||
Fix the INV-5 reject-after-interior-edit gap: `revertInPlace` in
|
||||
`src/proposalController.ts` skips the restore when the exact-substring anchor
|
||||
is orphaned by an interior edit inside the pending range, yet still reports
|
||||
success — ✗ Reject silently leaves the (edited) proposed text in the buffer.
|
||||
Evaluate the issue's fix directions (a: tracked current range; b: hard
|
||||
failure; c: fuzzy resolve), pick one, TDD the fix, host-E2E the repro sketch
|
||||
(propose → optimistic-apply → type inside range → rejectById), branch → PR →
|
||||
merge per §5.4. Session isolated in worktree `fix-70-inv5-reject-gap`
|
||||
(concurrent sessions 0061/0062/0063 in flight; 0062 occupies the canonical
|
||||
checkout).
|
||||
|
||||
## Progress
|
||||
|
||||
- Plan written: `docs/superpowers/plans/2026-07-02-70-inv5-reject-after-interior-edit.md`
|
||||
(4 tasks). Fix directions evaluated per the issue: **(a)** tracked applied
|
||||
span (new `DocState.appliedSpans` bookkeeping: set at optimistic-apply,
|
||||
re-synced on exact resolve, shifted through interior-safe edits, distrusted
|
||||
on boundary-straddling edits) with **(b)** honest hard failure as the
|
||||
fallback (warn, keep proposal, return false); **(c)** fuzzy resolve rejected
|
||||
(INV-11: never guess).
|
||||
- All 4 tasks executed TDD on branch `worktree-fix-70-inv5-reject-gap`
|
||||
(commits a9f8dd0, 82cea74, ddd634b, f47dfbc): revertInPlace fallback,
|
||||
hard-failure lock test, rejectAll parity ({reverted, skipped} + batch skip
|
||||
report), fullLoop tweak-then-reject leg.
|
||||
- Verification: typecheck + 301 unit + 94/5 host E2E green at each step.
|
||||
- Execution note: first distrust-test cut (whole-buffer rewrite) failed —
|
||||
VS Code minimizes workspace edits, so the rewrite decomposed into interior
|
||||
hunks the span legitimately survives; switched to a boundary-straddling
|
||||
deletion (plan doc updated in place).
|
||||
- Workflow code review (high) running; then push → PR → merge → close #70.
|
||||
|
||||
## Deferred decisions
|
||||
|
||||
_Autonomous-mode low-confidence calls the driver made and would have
|
||||
|
||||
Reference in New Issue
Block a user