docs(ignition): predicate re-bind ratification + cadence + delegated-merge amendment#1052
docs(ignition): predicate re-bind ratification + cadence + delegated-merge amendment#1052kyle-sexton wants to merge 5 commits into
Conversation
…, delegated-merge amendment Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Claude finished @kyle-sexton's task in 1m 7s —— View job PR Review —
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5ea3420c5d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
🔒 babysit-prs lane claiming this PR for on-branch fix work this cycle (amendment-round: 16, safe tier). Will fix clear branch-owned findings and push; will not resolve threads or merge (safe tier). |
|
Claude finished @kyle-sexton's task in 2m 32s —— View job PR Review —
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4c269aa649
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Two Codex review findings on #1052, both valid: - Eligibility figure ~2026-07-28/29 omitted the 48h maturity lag on the span's closing edge; corrected to ~2026-07-30/31 with the derivation. - The delegated-merge amendment's "definition unaffected" claim now also applies the definition to counted runs: scratch#24's delegated pre-merge linkage fix disqualifies it as an autonomous completion. Hand-excluded until the mechanical filter lands (scratch#60). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
|
Warning Automated review did not complete — this is an infrastructure failure, not a review verdict. Treat any Claude comment on this PR (including a placeholder like "I'll analyze this and get back to you") as incomplete, not "no findings."
Re-running the job, or pushing a new commit, will retry the review. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 410221205e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
External review: the delegated-merge record read as standing authorization for session-initiated merges, which would exercise the C2 auto-merge cell before the flip earns it (circular evidence). The record is now explicitly a historical caveat; forward policy requires a per-PR human action or explicit per-PR human instruction. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c5c265310b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| > completion must itself be merged ≥48h before evaluation — the maturity guard lags the | ||
| > closing edge by ~2 days on top of the span. Preconditions before any | ||
| > flip: the mechanical demotion watcher (scratch#41) replaces ad-hoc verification. | ||
| > Same change: fire cadence hourly → **15-min grid** (slot attestation generalized, |
There was a problem hiding this comment.
Reconfigure the missed-fire detector for the 15-minute cadence
When the Desktop task moves from hourly to a 15-minute grid, the existing Phase 2 missed-fire detector is still specified to compare run-history timestamps against the hourly cadence (PLAN.md lines 152-153). That leaves the availability signal unable to detect individual missed 15-minute slots (and potentially misclassifies the new schedule), so the schedule change should explicitly update and revalidate that detector before relying on the Phase 4 failure tracker.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
| # | Finding | Classification | Evidence | Reacted |
|---|---|---|---|---|
| 1 | Phase 2 missed-fire detector still specced against the hourly cadence after the 15-min rebind | VALID — surfaced to operator | Confirmed at head a0c407e7 (PLAN.md ~L152–153): the detector is "comparing run history timestamps against the hourly cadence" — unchanged while the Phase 4 amendment ratifies the 15-min grid |
👍 |
Disposition (safe tier): not editing and not resolving. This PR carries needs-human, and reconfiguring/revalidating the Phase 2 detector is beyond this PR's stated docs-status-amendment scope — an operator call, not a clear branch-owned mechanical fix.
Note: this and the L280 thread are two loci of one operator decision — propagate the ratified 15-min/7-day rebind into the live Brief / operator-decision table / parent roadmap / Phase 2 detector, or deliberately keep those as design-time records with the Phase 4 amendment layered on top. Surfaced to @kyle-sexton to decide and, if propagating, amend; resolve this thread to ratify keeping it as-is.
|
Claude encountered an error after 1s —— View job I'll analyze this and get back to you. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a0c407e7a2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| > Same change: fire cadence hourly → **15-min grid** (slot attestation generalized, | ||
| > tolerance 480s; quantified residual — accidental Run-now attests within ~53% of the hour | ||
| > vs ~17% before, accepted against the accidental-kick threat model + transcript-join | ||
| > primacy + watcher; historic negative re-classification under the new grid is inert | ||
| > today — the only manual runs are pre-window or failed); lease TTL 1h stopgap | ||
| > (TODO plugins#1034 sub-hour protocol); label-kick event-driven dispatch filed | ||
| > (scratch#42) to retire polling waste. Desktop task schedule flip is the operator's | ||
| > single remaining action, gated on the merged attestation code (done). |
There was a problem hiding this comment.
Update the active schedule and predicate summaries
This rebind makes the 15-minute/7-day policy current, but the active Brief still defines an hourly drain and a 14-day predicate (docs/topics/autonomy-ignition/PLAN.md:9-13), its operator-decision table still directs an hourly preset (:29), and the parent roadmap still scopes Phase I as hourly (docs/topics/ladder-climb-roadmap/PLAN.md:95-96). Those conflicting operational instructions can cause the remaining Desktop change or a later promotion review to follow the old cadence and threshold rather than the ratified policy; update the live summaries alongside this status amendment.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
| # | Finding | Classification | Evidence | Reacted |
|---|---|---|---|---|
| 1 | Live summaries still encode the pre-rebind policy: Brief defines hourly drain + ≥14d predicate, operator-decision table directs an hourly preset, parent roadmap scopes Phase I as hourly | VALID — surfaced to operator | Confirmed at head a0c407e7: Brief (PLAN.md ~L12) "hourly C2 drain" and predicate "≥14 days"; operator-decision table Schedule row (~L29) "Hourly preset"; parent ladder-climb-roadmap/PLAN.md:95–96 still hourly. The Phase 4 amendment ratifies 15-min/7-day, so the live summaries conflict |
👍 |
Disposition (safe tier): not editing and not resolving. This PR carries needs-human; rewriting the authoritative Brief + operator-decision table, and editing a second document (ladder-climb-roadmap/PLAN.md) not in this PR's diff, is beyond the stated docs-status-amendment scope — an operator/design call, not a clear branch-owned fix.
Note: this and the L273 thread are two loci of one operator decision — propagate the ratified 15-min/7-day rebind into the live Brief / operator-decision table / parent roadmap / Phase 2 detector, or deliberately keep those as design-time records with the Phase 4 amendment layered on top. Surfaced to @kyle-sexton to decide and, if propagating, amend; resolve this thread to ratify keeping it as-is.
|
Claude encountered an error after 1s —— View job I'll analyze this and get back to you. |
Dated Phase-4 notes: operator re-bound the org-bindable predicate thresholds in a reviewed change (scratch#43) — span 7d with a 48h merge-maturity guard on both count and span; simulation rejected; demotion watcher (scratch#41) is a flip precondition; cadence moves to a 15-min slot grid with quantified attestation residual; lease TTL 1h stopgap pending #1034; label-kick follow-up scratch#42. Also records the delegated-merge amendment describing the actual merge path during accumulation. Stale 14-day standing-lane reference updated.
Related
🤖 Generated with Claude Code