duckworth: row 300 round 2 approve (static) on db5122920 (db8be52c), headers fix only, round 1 miss corrected, C1 carried (duckworth-0268)

This commit is contained in:
jason.woltje committed 2026-10-23 07:08:00 -05:00
1 parent 3bc2793ccd
commit 65318da249
2 files changed
+35

No files matched your search

@@ -0,0 +1,34 @@
# Row 300 (R6-N1, #1807) round 2: delta review (duckworth-0268)
Reviewer: Duckworth. Builder: Launchpad. Request: sage-3412. Round 1: ROUND1-6a1d53066.md (comment
31150, not recorded).
Candidate: launchpad/row300 at db5122920 on 6a1d53066. Manifest digest
db8be52ce895603e687985acfae29e53e3e8782ae945c3cded6cca36141bfa3b, one path.
Method: git reads only. No node, no tests.
## Verdict: APPROVE, with C1 carried
- `git diff 6a1d53066 db5122920 -- packages` is one line in the test's call helper.
`resolve({ status, body })` becomes `resolve({ status, headers: res.headers, body })`. That is
granted-user.test.mjs:40's headers field, without its JSON parse. No assertion changes, and the
rest of the commit is BUILD.md's run 1 section plus the re-hashed manifest.
- The body stays text. That fits this file: :144 prints it, and :149's doesNotMatch reads it as a
string.
- MANIFEST.sha256 has one line, 03b5d60f…75cf4, equal to the blob at db5122920. The file hashes to
db8be52c…fa3b.
- Under mos-1272 this is a first-run red on a new test, placed by the builder. It doesn't need Mos
first.
- M12's kill path is unchanged. The fix only lets signIn read set-cookie and location.
## Correction to round 1
Round 1 missed this. signIn (:108-115) reads `start.headers.location` and `setCookies(r)`, which is
`r.headers['set-cookie']`, but call() never resolved `headers`. The file couldn't pass as written.
I read both functions and didn't trace the field across them.
## Condition
- **C1 (carried).** On Sage's CLEAR, the new file runs whole, then the webui suite whole, and M12 is
killed, all on db5122920.
Round 1's N1 (no recorder on `upstream`) goes to the landing note as deferred (sage-3405).
+1
View File
@@ -1556,3 +1556,4 @@ are never rewritten or removed; corrections are new entries.
2026-10-23T06:36Z | Duckworth (Opus 5.5) | row 294 C1 check on run 4 record (24b745447, launchpad-0087) | C1 met: M1 red only G4 at :241 (502 access-refused), G5 ok 5 same run; round 1 403 status corrected to 502; C2 met by run 4 (duckworth-0258) |
2026-10-23T07:13Z | Duckworth (Opus 5.5) | row 303 round 1 diff review (88ffecaf7, digest 3c3be5d3, launchpad-0089) | approve: one added R9 guard line, matches brief and mos-1422 pt 2, nothing else changed; C1 whole-file gate (duckworth-0260) |
2026-10-23T11:49Z | Duckworth (Opus 5.5) | rows 300/301 round 1 manifest reviews (6a1d53066 62023a00, 04a179740 def28197, launchpad-0098) | 300 approve C1 gate+M12, N1 upstream unrecorded; 301 approve C1 rebase recheck C2 gate, fold is a pure move, pin 15e8123f->39fcb27c (duckworth-0263) |
2026-10-23T12:08Z | Duckworth (Opus 5.5) | row 300 round 2 delta review (db5122920, digest db8be52c, sage-3412) | approve: call() resolves headers again, no assertion change; round 1 miss of the missing headers field recorded; C1 carried (duckworth-0268) |