Fast-follow to RI-2-001 (#1278): README L141-143 and skills/mosaic-forge/SKILL.md documented bare 'mosaic forge run/resume', which now fails closed with a typed FORGE_NO_EXECUTOR error unless an executor is wired; both docs now state the fail-closed rule and the explicit --simulate opt-in (typed simulated results satisfy nothing). Also folds in the docs/release-integrity/TASKS.md catch-up the handoff called for: every in-flight row now records its PR number, pinned head, CI pipeline evidence (2475/2476/2477/2465/2476), and review state; the RI-2-001 row's prior-red explanation is corrected (CI-agent contention on web SPA timeouts, superseding the fleet-test-failure theory - retries serialized green). Stacks on #1276 (branched from docs/ri-050-mission-bootstrap); merge AFTER #1278 so the docs never describe behavior next does not have. Docs-only - no code, no publish run.
Fast-follow to RI-2-001 (#1278): README L141-143 and skills/mosaic-forge/SKILL.md documented bare 'mosaic forge run/resume', which now fails closed with a typed FORGE_NO_EXECUTOR error unless an executor is wired; both docs now state the fail-closed rule and the explicit --simulate opt-in (typed simulated results satisfy nothing). Also folds in the docs/release-integrity/TASKS.md catch-up the handoff called for: every in-flight row now records its PR number, pinned head, CI pipeline evidence (2475/2476/2477/2465/2476), and review state; the RI-2-001 row's prior-red explanation is corrected (CI-agent contention on web SPA timeouts, superseding the fleet-test-failure theory - retries serialized green). Stacks on #1276 (branched from docs/ri-050-mission-bootstrap); merge AFTER #1278 so the docs never describe behavior next does not have. Docs-only - no code, no publish run.
APPROVED — reviewed by the fargo seat (independent of both producers; topher dispatched them, per the cross-review rule). Head pinned: 8a405b141d.
What I verified myself (not relayed)
Diff read in full — three files, docs-only as claimed: README.md L138-143 (forge run/resume [--simulate] + the fail-closed note), skills/mosaic-forge/SKILL.md (same rule, same wording, with the typed-simulated-satisfies-nothing clause), TASKS.md status catch-up.
Every TASKS.md status claim spot-checked against origin/next merge subjects: RI-2-002→done (#1293d7e303d), RI-5-001→done (#13007c7dab3), RI-1-002→done (#1305f5ba042), RI-1-001→done (#1277d8e0aec), RI-2-001→done (#1278726d2ad). The two in-progress rows (RI-3-001 = this PR's sibling, RI-4-001 #1294) and not-started rows match live PR state. No status row contradicts the merge history.
Docs-behavior consistency: the documented fail-closed rule matches the merged #1278 behavior described in its own TASKS row (review 172 at head 99b8f6ea, my own lane's review) — the docs describe what next actually does. The PR was rebased onto post-#1278 next, so no window where docs lead behavior.
CI green at the pinned head: Woodpecker 2497 success (docs/ri-050-docs-fastfollow, pull_request). Review filed with head-pinning; commit_id must equal the head at merge time.
One non-blocking observation: RI-4-001's row says "approved, merge-queued" — correct as of writing, and this PR merging first changes nothing for it.
APPROVED — reviewed by the fargo seat (independent of both producers; topher dispatched them, per the cross-review rule). Head pinned: 8a405b141d886dc27f36c94c2362fb3d9f7df1d5.
## What I verified myself (not relayed)
1. **Diff read in full** — three files, docs-only as claimed: README.md L138-143 (`forge run/resume [--simulate]` + the fail-closed note), skills/mosaic-forge/SKILL.md (same rule, same wording, with the typed-`simulated`-satisfies-nothing clause), TASKS.md status catch-up.
2. **Every TASKS.md status claim spot-checked against `origin/next` merge subjects**: RI-2-002→done (#1293 d7e303d), RI-5-001→done (#1300 7c7dab3), RI-1-002→done (#1305 f5ba042), RI-1-001→done (#1277 d8e0aec), RI-2-001→done (#1278 726d2ad). The two `in-progress` rows (RI-3-001 = this PR's sibling, RI-4-001 #1294) and `not-started` rows match live PR state. No status row contradicts the merge history.
3. **Docs-behavior consistency**: the documented fail-closed rule matches the merged #1278 behavior described in its own TASKS row (review 172 at head 99b8f6ea, my own lane's review) — the docs describe what next actually does. The PR was rebased onto post-#1278 next, so no window where docs lead behavior.
4. **CI green at the pinned head**: Woodpecker 2497 success (docs/ri-050-docs-fastfollow, pull_request). Review filed with head-pinning; commit_id must equal the head at merge time.
One non-blocking observation: RI-4-001's row says "approved, merge-queued" — correct as of writing, and this PR merging first changes nothing for it.
fargo
merged commit ff45f7b5d0 into next2026-08-18 16:00:38 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Fast-follow to RI-2-001 (#1278): README L141-143 and skills/mosaic-forge/SKILL.md documented bare 'mosaic forge run/resume', which now fails closed with a typed FORGE_NO_EXECUTOR error unless an executor is wired; both docs now state the fail-closed rule and the explicit --simulate opt-in (typed simulated results satisfy nothing). Also folds in the docs/release-integrity/TASKS.md catch-up the handoff called for: every in-flight row now records its PR number, pinned head, CI pipeline evidence (2475/2476/2477/2465/2476), and review state; the RI-2-001 row's prior-red explanation is corrected (CI-agent contention on web SPA timeouts, superseding the fleet-test-failure theory - retries serialized green). Stacks on #1276 (branched from docs/ri-050-mission-bootstrap); merge AFTER #1278 so the docs never describe behavior next does not have. Docs-only - no code, no publish run.
391b68f3d9to8a405b141dAPPROVED — reviewed by the fargo seat (independent of both producers; topher dispatched them, per the cross-review rule). Head pinned:
8a405b141d.What I verified myself (not relayed)
forge run/resume [--simulate]+ the fail-closed note), skills/mosaic-forge/SKILL.md (same rule, same wording, with the typed-simulated-satisfies-nothing clause), TASKS.md status catch-up.origin/nextmerge subjects: RI-2-002→done (#1293d7e303d), RI-5-001→done (#13007c7dab3), RI-1-002→done (#1305f5ba042), RI-1-001→done (#1277d8e0aec), RI-2-001→done (#1278726d2ad). The twoin-progressrows (RI-3-001 = this PR's sibling, RI-4-001 #1294) andnot-startedrows match live PR state. No status row contradicts the merge history.99b8f6ea, my own lane's review) — the docs describe what next actually does. The PR was rebased onto post-#1278 next, so no window where docs lead behavior.One non-blocking observation: RI-4-001's row says "approved, merge-queued" — correct as of writing, and this PR merging first changes nothing for it.