docs: mark F11 accepted and log manager decision
This commit is contained in:
parent
871182d6d5
commit
3d9ed8de51
2 changed files with 10 additions and 2 deletions
|
|
@ -216,7 +216,7 @@
|
|||
"Journal compaction under the run lock retains unresolved rows plus the last 1,000 recovered rows, so the file cannot grow without bound.",
|
||||
"Targeted unit tests cover journal append/replay/idempotency, backoff schedule, max-attempt exhaustion, due-retry selection, and the failure endpoint response shape."
|
||||
],
|
||||
"passes": false,
|
||||
"passes": true,
|
||||
"blocked_by": [
|
||||
"F08"
|
||||
],
|
||||
|
|
@ -224,7 +224,8 @@
|
|||
"Root cause: `/ops` is an in-memory 200-entry ring buffer per PM2 worker; after a long run the 17 failure details were unrecoverable.",
|
||||
"Do not reuse `/ops` for persistence; keep it as the live view. Journal is append-only for cluster safety; compact only under `run.lock`.",
|
||||
"Retry policy starts with constants; expose settings only if tuning is needed after the first real run.",
|
||||
"No user-facing notifications; logs and the read-only API/module table are the observability surface."
|
||||
"No user-facing notifications; logs and the read-only API/module table are the observability surface.",
|
||||
"Accepted 2026-10-30 on code + test evidence (79/79, tsc clean, build OK) after three review rounds; browser module render and live PM2 scheduler/retry tick were waived by the user and remain unverified."
|
||||
]
|
||||
},
|
||||
{
|
||||
|
|
|
|||
|
|
@ -305,3 +305,10 @@ Not verified locally:
|
|||
- Added deterministic successor-replacement test via guard hooks; both appends survive and the successor guard is restored.
|
||||
- `npx tsc --noEmit` 0; `npm test` 79/79 pass; `npm run build` emits both bundles. Logs: `.opencode/tmp/runs/f11-guard3-{test,build}.log` and `f11-guard3-final-*`.
|
||||
- Remaining manual checks: live PM2 multi-worker append/compaction, live scheduler retry tick, browser module rendering.
|
||||
|
||||
## 2026-10-30 — Manager: F11 accepted, F12 host fix recorded
|
||||
|
||||
- Decision: squash-merged `feat/F11` into `goal/image-previews` after reviewer APPROVE (79/79 tests, tsc clean, build OK) and flipped F11 passes=true. User waived the manual browser/live-PM2 checks; recorded as unverified in the F11 note.
|
||||
- Context: three review rounds hardened replay attempts, compaction-vs-append locking, exhausted-retry handling, and reclaim-guard identity. Residual fs-lock scheduling windows are documented and accepted.
|
||||
- Outcome: accepted. F12 passes=true came from the user's own host verification (2,836 null rows backfilled, column default set) and was committed unchanged. Extension repo still owns no DB code for F12.
|
||||
- Learning: journal locking is the hard part, not the retry logic — plan the lock protocol before writing the journal. Browser + live-cluster checks remain the only gaps; deploy requires a human.
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue