You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Follow-up to the Phase 12 stack (PR #801, epic #534 D1-019).
What and why
id world apply currently reconciles grants, ceiling and profile against a live world, and id world diff reports drift. The remaining manifest fields are parsed but not yet applied over the wire: a module hash/source (install/compile) and schedules (set/cancel). The model already plans these steps; they need the server-side apply path.
Acceptance
id world apply installs (or compiles) the manifest module and applies schedule set/cancel, journaling each step.
A manifest naming only a module + schedules applies idempotently.
Acceptance criteria
The body's ## Acceptance bullets hold: id world apply installs (or compiles) the manifest module and applies schedule set/cancel, journaling each step.
A manifest naming only a module + schedules applies idempotently (second apply is a no-op).
id world diff after apply reports no drift for module or schedules.
Related: Phase 12 stack PR #801 (merged); epic #534 (D1-019); siblings #805 (F12.11) and #806 (F12.13). PR #860 touched world apply reporting for #818, not this wiring. No merged PR resolves this (verified 2026-10-26).
Notes: The body already had an ## Acceptance list; the criteria above restate it in the standard section so queue/triage tooling can find it. Effort l: server-side apply path for module install/compile plus schedules reconciliation.
Risks: Apply-path changes are world-mutating; idempotency and journaling are the safety properties to test first.
Suggested approach: Extend the existing grants/ceiling/profile apply path with module and schedule steps, each journaled, with a diff-after-apply test.
Agent note — verified merge-ready except infra clippy + the merge-queue stall; parked
Queue task 2661 resumed by Lane B. PR #1820 verified fresh at 569abd29: MERGEABLE, all review threads resolved (7/7), latest Codex summary at this head reports no findings. CI: every check SUCCESS except clippy-lint (and its aggregate nix flake check).
Lane B re-checked the clippy failure from the run log (run 40894646360, 20:06Z attempt): it is infrastructure, not code — repeated HTTP error 418 downloads with “GitHub Actions Cache throttled Magic Nix Cache” and a Twirp ResourceExhausted rate-limit error. The PR diff does not touch src/tags.rs (the on-main clippy item in #2401). A --failed rerun has been requested (attempt 4 now in progress).
Resume: when the rerun is green and the merge-queue stall #2403 is fixed, enqueue PR #1820, verify it merges, and complete task 2661.
Queue task 2661 completed by Lane C, 2026-10-30T01:04Z. PR #1820 (id(world): wire manifest apply — module install + schedules over the wire) MERGED at 2026-10-30T01:00:10Z, merge commit 17c4a0de5a34b7b919312c4b1fcf7766b3006657, after the merge-queue stall (#2403) was fixed. The PR body says "Closes the apply-wiring gap in #807" (not the auto-close keyword), so this issue is being closed manually with that evidence. The pre-existing clippy debt at src/tags.rs:1484,1499 remains tracked separately in #2401.
Activity
feat(id): wire id world diff and id world apply (F12.1)
docs(id): Phase 14 plan — close-out: finish, measure, verify, open th…
dezren39 commentedon Oct 26, 2026
Intake review
Related: Phase 12 stack PR #801 (merged); epic #534 (D1-019); siblings #805 (F12.11) and #806 (F12.13). PR #860 touched world apply reporting for #818, not this wiring. No merged PR resolves this (verified 2026-10-26).
Notes: The body already had an
## Acceptancelist; the criteria above restate it in the standard section so queue/triage tooling can find it. Effort l: server-side apply path for module install/compile plus schedules reconciliation.Risks: Apply-path changes are world-mutating; idempotency and journaling are the safety properties to test first.
Suggested approach: Extend the existing grants/ceiling/profile apply path with module and schedule steps, each journaled, with a diff-after-apply test.
358 remaining items
dezren39 commentedon Oct 29, 2026
🤖 Agent note · parked
dezren39 commentedon Oct 29, 2026
Agent note — verified merge-ready except infra clippy + the merge-queue stall; parked
Queue task 2661 resumed by Lane B. PR #1820 verified fresh at
569abd29: MERGEABLE, all review threads resolved (7/7), latest Codex summary at this head reports no findings. CI: every check SUCCESS exceptclippy-lint(and its aggregatenix flake check).Lane B re-checked the clippy failure from the run log (run 40894646360, 20:06Z attempt): it is infrastructure, not code — repeated
HTTP error 418downloads with “GitHub Actions Cache throttled Magic Nix Cache” and a TwirpResourceExhaustedrate-limit error. The PR diff does not touchsrc/tags.rs(the on-main clippy item in #2401). A--failedrerun has been requested (attempt 4 now in progress).Resume: when the rerun is green and the merge-queue stall #2403 is fixed, enqueue PR #1820, verify it merges, and complete task 2661.
dezren39 commentedon Oct 29, 2026
🤖 Agent note · parked
clippy::expect_usedat src/tags.rs:1484 and :1499. That is the pre-existing id: clippy-lint red on main — expect_used at src/tags.rs:1484,1499 (clippy 1.99) #2401 failure, not this PR's code (all other checks SUCCESS, Codex clean at head per prior shift). No further rerun is useful until id: clippy-lint red on main — expect_used at src/tags.rs:1484,1499 (clippy 1.99) #2401 merges; the merge queue stall ci: merge queue ejects every PR — no merge since #2233 (10:57Z), entries dropped 2-6 min into AWAITING_CHECKS #2403 also remains open (main frozen at 5a34fa7). Re-parking.dezren39 commentedon Oct 30, 2026
🤖 Agent note ·
agent-note:v1· done — PR mergedQueue task 2661 completed by Lane C, 2026-10-30T01:04Z. PR #1820 (
id(world): wire manifest apply — module install + schedules over the wire) MERGED at 2026-10-30T01:00:10Z, merge commit17c4a0de5a34b7b919312c4b1fcf7766b3006657, after the merge-queue stall (#2403) was fixed. The PR body says "Closes the apply-wiring gap in #807" (not the auto-close keyword), so this issue is being closed manually with that evidence. The pre-existing clippy debt atsrc/tags.rs:1484,1499remains tracked separately in #2401.