carlos / platform Public

Clone
git clone https://amadan.net/carlos/platform

Plain git — no account needed to clone.

Download

Download this file

1| Round 3 finding | Rev 4 sufficient? | Why |
2|---|---|---|
3| 10. Rollback affects unrelated processes | Yes | §2.3 now groups by the same non-empty upstream, with `Socket` taking precedence over `Addr`. |
4| 11. Shared-host stamps and refusal coverage | Yes | §2.3 restores links and registry stamps even for siblings advanced by an interrupted pass, and records refusals for every group host. |
5| 13. Re-promoting does not trigger retry | Yes | §2.5 introduces `promote --retry` through both interfaces, explicitly requiring a fresh `PromotedAt`. |
6
7New findings at `924d2bf8`:
8
914. **P2 — Proof-waiting routes outside canary channels can wait until the hourly pass.** [§4.5](/home/paulca/amadan.net/carlos/platform-design-console-release-defences/docs/superpowers/specs/2026-10-06-console-release-defences-design.md:439) relies on the two-second fast loop to reconsider a missing proof after its 30-second cache expires. However, [`fastMembers`](/home/paulca/amadan.net/carlos/platform-design-console-release-defences/internal/host/agent.go:1550) excludes non-canary routes that already have a live version. A promotion’s serial bump can run the gate once, find no proof, and leave the old version live; subsequent ticks then exclude that route. §4.4 specifies uploading the proof but no wake-up mechanism for its consumer. Require pending-proof routes to remain eligible for bounded rechecks, or specify a proof-arrival trigger that reaches them.
10
1115. **P2 — Readiness probes can reject a healthy socket-backed unit.** [§2.4](/home/paulca/amadan.net/carlos/platform-design-console-release-defences/docs/superpowers/specs/2026-10-06-console-release-defences-design.md:197) always sends a configured health probe to `rt.Addr`, although [`Route.Socket`](/home/paulca/amadan.net/carlos/platform-design-console-release-defences/internal/registry/registry.go:40) takes precedence, and [`unitRail`](/home/paulca/amadan.net/carlos/platform-design-console-release-defences/cli.go:1624) permits named units on socket-backed routes. With a configured probe and an empty `Addr`, a healthy unit serving its Unix socket never passes readiness: D1 rolls it back, the restored build fails the same check, and the refusal blocks further adoption. With both fields populated, D1 checks an upstream the route does not use. Make readiness follow the effective upstream, including Unix-socket HTTP probes.
12
13Verdict: accept with fixes.