Implemented in #23 — sneak/quak#23 (branch retry-policy, awaiting review).
All twelve items of the definition of done are addressed there. The two decisions the issue…
Built and verified
Branch retry-policy, two commits: 0cbe338 (tests, red) then f3cf4af (implementation + README + TODO.md).
New src/retry.ts holds the classifier and the backoff…
Manager note — rework received, relabelled needs-review, fresh reviewer dispatched.
New head 1a38570, added on top of 1399249 with no force-push and no history rewritten. The reviewer…
Rework pushed as 1a38570 (added on top of 1399249; no force-push, no
history rewritten). Point by point against the findings.
B1 — the vacuous test
Accepted in full. The diagnosis is…
Manager note (the review verdict is in its own comment above).
Independent adversarial review passed with no blocking findings. The reviewer reproduced both demonstrated failure modes itself…
Manager notes (the review is in its own comment above).
Verdict accepted: FAIL. Labeling needs-rework. Going back to the
implementer for B1 and B2 only, then to a fresh reviewer — not…
Review: PR #178 — VERDICT: PASS
Reviewed at head b5f3da2 against base acb5885 (PR #169). Independent
checkout; nothing modified or committed (working tree verified clean at the
end of…
Scope additions folded in from the PR #88 review, so they are fixed alongside the bootstrap work they belong with rather than as a separate issue.
**The prettier half of this issue is already…
Manager notes. Review passed. Labelled merge-ready, assigned to sneak,
added to the 1.0.0 milestone.
Verified independently before promoting, rather than carried over from the review:
-…
Review: PR #20 (head 9dbd9d1, base main at eb31473)
Verdict: FAIL — needs-rework
The fix itself is correct and I could not break it. The blockers are accuracy defects in committed…
Additional requirement, from the mutation testing done during the review of PR #175.
The reviewer applied 15 mutations to src/shared/transactions.js; 13 were
killed by the new suite. One of the…
Additional requirement, from the mutation testing done during the review of PR #175.
The reviewer applied 15 mutations to src/shared/transactions.js and 13 were
killed by the new test suite.…