mcpx: ${mcpx.root} / ${mcpx.cwd} substitution at spawn + era identity on effective definition #960

Activity

1. github-actions

   assigned

   dezren39

   on Oct 26, 2026
2. dezren39

   added

   kind/featureNew feature or capability requestNew feature or capability request

   source/humanFiled directly by a humanFiled directly by a human

   status/blockedQueue or user escalation stateQueue or user escalation state

   priority/normalOrdinary priority (the default)Ordinary priority (the default)

   effort/mMedium: expected under 1 day of workMedium: expected under 1 day of work

   meta/enrichedIntake review done: acceptance criteria, links, and normalized labels addedIntake review done: acceptance criteria, links, and normalized labels added

   on Oct 26, 2026
3. dezren39 commented on Oct 26, 2026

   

   dezren39

   on Oct 26, 2026

   MemberAuthor

   More actions

   Intake review

   Still live (verified this run): no ${mcpx.root} / ${mcpx.cwd} substitution exists in pkgs/mcpx on main (grep clean), no ResolvedScope to source the root from, and no merged PR addresses it. Design record is in open PR #957.

   Related: #958 (slice 1, prerequisite) · PR #957 (decision 0004, open) · #961 (slice 4; does not require this slice) · design origin dezren39/darwin-nix#818.

   Notes / risks: Marked status/blocked at intake — depends on slice 1. Two easy-to-miss requirements: (1) substitution happens at spawn time only, never in the stored/merged config; (2) the era/instance identity must hash the substituted definition, or two instances differing only by root will collide in the pool cache. The empty-root loud failure (never substitute daemon cwd) is a safety property — keep it an error, not a warning.

   Suggested approach: After #958, add load-time placeholder validation (unknown/malformed ${mcpx.…} names the server + field), spawn-time substitution over Args/Env/Cwd, and switch pool.Identity to the effective definition; test each field plus the unknown-placeholder and empty-root errors.

   — Fleet monitor intake enrichment (acceptance criteria were already in the body; labels normalized, meta/enriched added).
4. dezren39 commented on Oct 29, 2026

   

   dezren39

   on Oct 29, 2026

   MemberAuthor

   More actions

   🤖 Agent note · handed off

   • Agent: queue worker (repo-worker-code), lane C shift
   • Session / ID: queue task 2897 (triage of mcpx: ${mcpx.root} / ${mcpx.cwd} substitution in Args, Env and Cwd at spawn time #1991)
   • Found via: queue claim (triage of mcpx: ${mcpx.root} / ${mcpx.cwd} substitution in Args, Env and Cwd at spawn time #1991)
   • Task: Two-way duplicate link: mcpx: ${mcpx.root} / ${mcpx.cwd} substitution in Args, Env and Cwd at spawn time #1991 closed as a duplicate of this issue.
   • Expectation: This issue remains the sole tracker for the ${mcpx.root}/${mcpx.cwd} substitution work.
   • Links: mcpx: ${mcpx.root} / ${mcpx.cwd} substitution in Args, Env and Cwd at spawn time #1991 (duplicate, closed), queue task 2897

   Duplicate link (from triage of #1991, task 2897): #1991 — the 2026-10-28 relocation of dezren39/darwin-nix#818 — duplicates this issue, which was filed 2026-10-26 as slice 2 of decision record 0004 and explicitly carries nix#818's acceptance criteria. #1991 is closed as a duplicate of this issue; nothing in #1991 goes beyond what is specified here (this issue's spec is the more precise one: load-time placeholder validation, loud spawn failure on an empty resolved root, pool.Identity hashing the effective definition). Implementation stays tracked here, gated on slice 1 (#958, still open).
5. dezren39

   added

   work/assignedQueue worker lifecycle stateQueue worker lifecycle state

   queue/claimedQueue worker lifecycle stateQueue worker lifecycle state

   status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

   on Nov 1, 2026
6. dezren39 commented on Nov 1, 2026

   

   dezren39

   on Nov 1, 2026

   MemberAuthor

   More actions

   🤖 Agent note · picked up

   • Agent: queue worker (repo-worker-code), lane B shift
   • Session / ID: queue task 2296 (triage)
   • Found via: queue claim
   • Task: Triage mcpx: ${mcpx.root} / ${mcpx.cwd} substitution at spawn + era identity on effective definition #960 (${mcpx.root}/${mcpx.cwd} spawn-time substitution, slice 2 of decision 0004) against current main: verify the feature's absence, check prerequisite slice 1 (mcpx: ResolvedScope plumbing + spawn cwd = resolved scope root #958), and route.
   • Expectation: s — verdict comment with fresh-HEAD evidence, follow-up route recorded, labels synced.
   • Links: queue task 2296
7. dezren39 commented on Nov 1, 2026

   

   dezren39

   on Nov 1, 2026

   MemberAuthor

   More actions

   🤖 Agent note · done

   • Agent: queue worker (repo-worker-code), lane B shift
   • Session / ID: queue task 2296 (triage)
   • Found via: queue claim
   • Task: Triage verdict for mcpx: ${mcpx.root} / ${mcpx.cwd} substitution at spawn + era identity on effective definition #960 (slice 2 of decision 0004: ${mcpx.root}/${mcpx.cwd} spawn-time substitution + era identity on the effective definition).
   • Expectation: Verdict with fresh-HEAD evidence + follow-up route — both delivered below.
   • Links: follow-up fix task 4874 (queued, gated on mcpx: ResolvedScope plumbing + spawn cwd = resolved scope root #958); packet: hidden_files/packet-960-fix.json

   Triage verdict: CONFIRMED — not implemented; implementation gated on slice 1 (#958)

   Evidence (fresh, current main b56ccda5c, 2026-10-31 21:06 CDT, laptop primary origin/main):

   • grep -rn 'mcpx\.root\|mcpx\.cwd' pkgs/mcpx --include='*.go' → no matches: no spawn-time substitution exists.
   • grep -rn 'ResolvedScope' pkgs/mcpx --include='*.go' → no matches: slice 1's plumbing has not landed either.
   • pkgs/mcpx/docs/decisions/0004-scope-resolved-roots.md exists on main — the spec (§4) is merged and available to implement against.

   Dependency state: #958 (slice 1, ResolvedScope plumbing) is still OPEN and, at this check, carries queue/claimed + work/assigned — another worker holds it. This slice cannot be built before it: there is no resolved root to substitute.

   Route: follow-up fix enqueued as queue task 4874 (kind small_fix, packet packet-960-fix.json) with the gate pinned in the packet: at claim time re-check #958 — if it has not landed, block the task naming the dependency rather than improvising a parallel ResolvedScope. The packet pins the full scope: load-time placeholder validation (unknown/malformed ${mcpx.*} names server + field), spawn-time-only substitution over Args/Env/Cwd, loud spawn failure on an empty resolved root (never daemon cwd), and pool.Identity hashing the effective substituted definition.

   This issue stays open with status/blocked (dependency) until #958 lands; the duplicate #1991 remains closed against this tracker.
8. dezren39

   removed

   status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

   on Nov 1, 2026
9. 38 remaining items

   Load more
10. dezren39

    added

    queue/parkedQueue task is parked (waiting on CI, time, or an event)Queue task is parked (waiting on CI, time, or an event)

    status/parkedDeliberately paused with a recorded resume conditionDeliberately paused with a recorded resume condition

    queue/claimedQueue worker lifecycle stateQueue worker lifecycle state

    status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

    and removed

    status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

    queue/claimedQueue worker lifecycle stateQueue worker lifecycle state

    status/parkedDeliberately paused with a recorded resume conditionDeliberately paused with a recorded resume condition

    queue/parkedQueue task is parked (waiting on CI, time, or an event)Queue task is parked (waiting on CI, time, or an event)

    on Nov 3, 2026
11. dezren39

    added 2 commits that reference this issue on Nov 3, 2026

    fix(mcpx): preserve caller scope across replacement, exec watch, sche…

    ...

    701a4d6

    feat(mcpx): spawn-time ${mcpx.root}/${mcpx.cwd} substitution + load v…

    ...

    5b0aeb5
12. dezren39 commented on Nov 3, 2026

    

    dezren39

    on Nov 3, 2026

    MemberAuthor

    More actions

    🤖 Agent note · done

    • Agent: repo queue worker lane code-a (repo-worker-code)
    • Session / ID: task 4874
    • Found via: queue claim (resume from awaiting_ci checkpoint)
    • Task: task 4874 complete — fixed_merged. PR feat(mcpx): spawn-time ${mcpx.root}/${mcpx.cwd} substitution + load validation (#960) #5135 squash-merged as 5b0aeb5: spawn-time ${mcpx.root}/${mcpx.cwd} substitution in Args/Env/Cwd on the effective definition only (stored config stays unsubstituted), load-time placeholder validation, and era identity over the substituted definition — every acceptance criterion on mcpx: ${mcpx.root} / ${mcpx.cwd} substitution at spawn + era identity on effective definition #960 is covered by the PR and its tests.
    • Codex gate: the PR's three Codex P2 findings were fixed in 701a4d6 (caller cwd preserved across Restart/exec-watch replacement/subscription monitor/handoff; exec watch stamps the effective definition; caller-scoped schema refresh via RefreshSchemasFor + Warm deferral) with replies on each thread (4460624033, 4460624728, 4460625345) and regression tests in pkgs/mcpx/internal/pool/codex5135_test.go. Laptop suites: pool, config, daemon all pass. The two basic PR checks stayed QUEUED (CI runs on main per the standing rule); main CI is the backstop.
    • Expectation: none — mcpx: ${mcpx.root} / ${mcpx.cwd} substitution at spawn + era identity on effective definition #960 closes with this note.
13. dezren39

    closed this as completedon Nov 3, 2026
14. dezren39

    removed

    status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

    queue/claimedQueue worker lifecycle stateQueue worker lifecycle state

    work/assignedQueue worker lifecycle stateQueue worker lifecycle state

    on Nov 3, 2026