Compare commits
1
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
7a2639d0f4 |
+1
-15
@@ -1,19 +1,5 @@
|
|||||||
# Tasks — MVP (Top-Level Rollup)
|
# Tasks — MVP (Top-Level Rollup)
|
||||||
|
|
||||||
> ---
|
|
||||||
>
|
|
||||||
> **STATUS: SUPERSEDED — 2026-08-20.** kind `tracking` · superseded by `docs/fleet/NORTH_STAR.yaml`
|
|
||||||
>
|
|
||||||
> This file is the pre-backlog tracking mechanism. `NS-2` in the north star declares the
|
|
||||||
> replacement: every backlog item is a Mosaic Backlog card projected from the YAML. That
|
|
||||||
> model replaced this one and nobody retired the old file, so it kept reading as
|
|
||||||
> authoritative while going stale.
|
|
||||||
>
|
|
||||||
> **Do not trust a status in this file.** Verified 2026-08-20: it was already behind the
|
|
||||||
> code when it froze five weeks ago.
|
|
||||||
>
|
|
||||||
> Kept as a record of what was believed. Do not update it; update the YAML.
|
|
||||||
|
|
||||||
> Single-writer: orchestrator only. Workers read but never modify.
|
> Single-writer: orchestrator only. Workers read but never modify.
|
||||||
>
|
>
|
||||||
> **Mission:** mvp-20260312
|
> **Mission:** mvp-20260312
|
||||||
@@ -122,7 +108,7 @@ Active workstream is **W1 — Federation v1**. Workers should:
|
|||||||
|
|
||||||
## north-star doctrine consolidation — doc PR — feat/north-star-doctrine
|
## north-star doctrine consolidation — doc PR — feat/north-star-doctrine
|
||||||
|
|
||||||
- Status: applied Mos's consolidated merge-map to docs/fleet/FLEET-DOCTRINE.md (budget governance + control plane/central register + 200k cap + delegation + unified-identity Fleet + role-based naming + tmux security + drift re-captures). Doctrine only; #622/#623/#625/#628 out-of-scope. Conflict checklist green. Detail: scratchpads/north-star-doctrine.md.
|
- Status: applied Mos's consolidated merge-map to docs/fleet/north-star.md (budget governance + control plane/central register + 200k cap + delegation + unified-identity Fleet + role-based naming + tmux security + drift re-captures). Doctrine only; #622/#623/#625/#628 out-of-scope. Conflict checklist green. Detail: scratchpads/north-star-doctrine.md.
|
||||||
|
|
||||||
## #631 — re-seed preserves user fleet data (CRITICAL) — fix/631-reseed-preserves-fleet-data
|
## #631 — re-seed preserves user fleet data (CRITICAL) — fix/631-reseed-preserves-fleet-data
|
||||||
|
|
||||||
|
|||||||
@@ -1,21 +1,5 @@
|
|||||||
# Tasks — Federation v1
|
# Tasks — Federation v1
|
||||||
|
|
||||||
> ---
|
|
||||||
>
|
|
||||||
> **STATUS: SUPERSEDED — 2026-08-20.** kind `tracking` · superseded by `docs/fleet/NORTH_STAR.yaml`
|
|
||||||
>
|
|
||||||
> This file is the pre-backlog tracking mechanism. `NS-2` in the north star declares the
|
|
||||||
> replacement: every backlog item is a Mosaic Backlog card projected from the YAML. That
|
|
||||||
> model replaced this one and nobody retired the old file, so it kept reading as
|
|
||||||
> authoritative while going stale.
|
|
||||||
>
|
|
||||||
> **Do not trust a status in this file.** Verified 2026-08-20: it was already behind the
|
|
||||||
> code when it froze five weeks ago. `FED-M3-06` is marked not-started and `get.controller.ts`
|
|
||||||
> has existed for eight weeks; `FED-M3-10/11` claim no tests exist while fifteen spec files
|
|
||||||
> do. `FED-M2-DEPLOY-IMG-FIX` names `apps/gateway/Dockerfile`, which is not in the repo.
|
|
||||||
>
|
|
||||||
> Kept as a record of what was believed. Do not update it; update the YAML.
|
|
||||||
|
|
||||||
> Single-writer: orchestrator only. Workers read but never modify.
|
> Single-writer: orchestrator only. Workers read but never modify.
|
||||||
>
|
>
|
||||||
> **Mission:** federation-v1-20260419
|
> **Mission:** federation-v1-20260419
|
||||||
|
|||||||
+25
-45
@@ -3,7 +3,7 @@
|
|||||||
> **Generated file — do not edit by hand.**
|
> **Generated file — do not edit by hand.**
|
||||||
> Projected deterministically from [`NORTH_STAR.yaml`](./NORTH_STAR.yaml) by the pure
|
> Projected deterministically from [`NORTH_STAR.yaml`](./NORTH_STAR.yaml) by the pure
|
||||||
> generator in `packages/mosaic/src/commands/fleet.ts` (`renderNorthStarMarkdown`).
|
> generator in `packages/mosaic/src/commands/fleet.ts` (`renderNorthStarMarkdown`).
|
||||||
> Edit the YAML, then regenerate. Self-contained Mosaic.
|
> Edit the YAML, then regenerate. Self-contained Mosaic — no Hermes dependency.
|
||||||
|
|
||||||
## Mission
|
## Mission
|
||||||
|
|
||||||
@@ -11,7 +11,7 @@ A self-driving Mosaic system that 24/7 unattended converts a machine-readable go
|
|||||||
|
|
||||||
## Substrate
|
## Substrate
|
||||||
|
|
||||||
The Mosaic Backlog is the backlog of record + dispatch engine, built on Mosaic's native Postgres storage service (@mosaicstack/db drizzle; PGlite-embedded by default, full Postgres by config).
|
The Mosaic Backlog is the backlog of record + dispatch engine, built on Mosaic's native Postgres storage service (@mosaicstack/db drizzle; PGlite-embedded by default, full Postgres by config). NOT Hermes.
|
||||||
|
|
||||||
## Standing objectives
|
## Standing objectives
|
||||||
|
|
||||||
@@ -24,18 +24,16 @@ The Mosaic Backlog is the backlog of record + dispatch engine, built on Mosaic's
|
|||||||
- **NS-7** — Meta-loop (session-review + enhancer) continuously proposes small fleet-improvement PRs.
|
- **NS-7** — Meta-loop (session-review + enhancer) continuously proposes small fleet-improvement PRs.
|
||||||
- **NS-8** — Single operator-flippable PAUSE kill-switch (fleet/run/PAUSED) honored before every dispatch and every merge.
|
- **NS-8** — Single operator-flippable PAUSE kill-switch (fleet/run/PAUSED) honored before every dispatch and every merge.
|
||||||
- **NS-9** — Mosaic is a general-purpose multi-agent system: the user declares the SYSTEM TYPE to run (e.g. software delivery, personal assistant, research, business/operations) and the orchestrator provisions the matching persona roster and org structure from a cross-domain baseline persona library; the delivery/coding fleet is one profile among many.
|
- **NS-9** — Mosaic is a general-purpose multi-agent system: the user declares the SYSTEM TYPE to run (e.g. software delivery, personal assistant, research, business/operations) and the orchestrator provisions the matching persona roster and org structure from a cross-domain baseline persona library; the delivery/coding fleet is one profile among many.
|
||||||
- **NS-10** — An adoption is not complete until the mechanism it replaces is removed. Two live conventions for one concern is the defect, not a transition state. Measured 2026-08-20: brain-home adopted by 9 modules and not 10; MOSAIC_HOME honored in 4 places, each re-deriving it; backlog cards declared while TASKS.md files stayed authoritative. Every one was decided correctly and left half-applied.
|
|
||||||
|
|
||||||
## Success criteria
|
## Success criteria
|
||||||
|
|
||||||
- **AC-NS-0** (tier 0) — The operator launches an agent on any configured harness with one command, observes its state and sends it work without attaching to a terminal multiplexer.
|
- **AC-NS-1** — The supervisor keeps a two-agent floor (1 orchestrator + >=1 enhancer) healthy across reboot.
|
||||||
- **AC-NS-1** (tier 1) — The supervisor keeps a two-agent floor (1 orchestrator + >=1 enhancer) healthy across reboot.
|
- **AC-NS-2** — A goal added to this YAML is decomposed to cards and either merged or escalated, with no human in the loop.
|
||||||
- **AC-NS-2** (tier 1) — A goal added to this YAML is decomposed to cards and either merged or escalated, with no human in the loop.
|
- **AC-NS-3** — No PR merges with failure/error/no-status/timeout CI, and none bypass pr-merge.sh.
|
||||||
- **AC-NS-3** (tier 1) — No PR merges with failure/error/no-status/timeout CI, and none bypass pr-merge.sh.
|
- **AC-NS-4** — TTL is enforced on claims; token caps remain advisory until a real meter exists.
|
||||||
- **AC-NS-4** (tier 1) — TTL is enforced on claims; token caps remain advisory until a real meter exists.
|
- **AC-NS-5** — Flipping fleet/run/PAUSED halts dispatch and merges within one tick.
|
||||||
- **AC-NS-5** (tier 1) — Flipping fleet/run/PAUSED halts dispatch and merges within one tick.
|
- **AC-NS-6** — A user can declare a system type and the fleet provisions the matching persona roster + topology from the baseline library, with no code change.
|
||||||
- **AC-NS-6** (tier 2) — A user can declare a system type and the fleet provisions the matching persona roster + topology from the baseline library, with no code change.
|
- **AC-NS-7** — A user-customized persona (edited or added via the orchestrator) survives mosaic update: baseline reseed never clobbers user overrides.
|
||||||
- **AC-NS-7** (tier 2) — A user-customized persona (edited or added via the orchestrator) survives mosaic update: baseline reseed never clobbers user overrides.
|
|
||||||
|
|
||||||
## Workstreams
|
## Workstreams
|
||||||
|
|
||||||
@@ -47,44 +45,26 @@ The Mosaic Backlog is the backlog of record + dispatch engine, built on Mosaic's
|
|||||||
| D | Merge-gate — single approver, pr-merge.sh after CI wait |
|
| D | Merge-gate — single approver, pr-merge.sh after CI wait |
|
||||||
| E | Meta-loop — session-review + enhancer improvement PRs |
|
| E | Meta-loop — session-review + enhancer improvement PRs |
|
||||||
| F | Safety-rails — TTL claims, advisory spend, PAUSE kill-switch |
|
| F | Safety-rails — TTL claims, advisory spend, PAUSE kill-switch |
|
||||||
| G | Kill-switch — operator PAUSE honored before dispatch and merge |
|
|
||||||
| H | Personas & system profiles — cross-domain library, system-type provisioning, update-surviving customization |
|
| H | Personas & system profiles — cross-domain library, system-type provisioning, update-surviving customization |
|
||||||
| I | Operator surface — launcher, fleet visibility, reliable steering (tier 0) |
|
|
||||||
| J | Web control plane — browser surface over the gateway (tier 1) |
|
|
||||||
| K | Clients — desktop and mobile over the same backend (tier 2) |
|
|
||||||
| L | Auth profiles — per-provider accounts, per-session selection (tier 2) |
|
|
||||||
|
|
||||||
## Goals (backlog projection)
|
## Goals (backlog projection)
|
||||||
|
|
||||||
| id | title | tier | phase | priority | depends_on |
|
| id | title | phase | priority | depends_on |
|
||||||
| --- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ---- | ----- | ----------- | -------------- |
|
| --- | ------------------------------------------------------------------------------------------------------------------------------------------------- | ----- | ----------- | ---------- |
|
||||||
| A1 | Machine-readable NORTH_STAR.yaml + Markdown projection | 1 | 1 | must-have | — |
|
| A1 | Machine-readable NORTH_STAR.yaml + Markdown projection | 1 | must-have | — |
|
||||||
| A2 | Mosaic Backlog schema + storage-service card store (drizzle/PGlite) | 1 | 1 | must-have | A1 |
|
| A2 | Mosaic Backlog schema + storage-service card store (drizzle/PGlite) | 1 | must-have | A1 |
|
||||||
| A3a | Card lifecycle — create/claim/release with stable ids + depends_on DAG | 1 | 1 | must-have | A2 |
|
| A3a | Card lifecycle — create/claim/release with stable ids + depends_on DAG | 1 | must-have | A2 |
|
||||||
| A3b | TTL-bounded claim enforcement (wall-clock) on cards | 1 | 1 | must-have | A3a |
|
| A3b | TTL-bounded claim enforcement (wall-clock) on cards | 1 | must-have | A3a |
|
||||||
| A4 | Advisory spend projection per card (degrades to TTL, no real meter) | 1 | 1 | should-have | A3a |
|
| A4 | Advisory spend projection per card (degrades to TTL, no real meter) | 1 | should-have | A3a |
|
||||||
| B1 | Supervisor tick — readiness scan, two-agent-floor health check | 1 | 2 | must-have | A3a |
|
| B1 | Supervisor tick — readiness scan, two-agent-floor health check | 2 | must-have | A3a |
|
||||||
| B2 | Native dispatch/claim — assign ready dependency-satisfied work | 1 | 2 | must-have | A3b, B1 |
|
| B2 | Native dispatch/claim — assign ready dependency-satisfied work | 2 | must-have | A3b, B1 |
|
||||||
| B3a | Planner decompose — goal added to YAML → cards | 1 | 2 | must-have | A2, B1 |
|
| B3a | Planner decompose — goal added to YAML → cards | 2 | must-have | A2, B1 |
|
||||||
| B3b | Replan request on empty backlog; escalate on no-decompose | 1 | 2 | should-have | B3a |
|
| B3b | Replan request on empty backlog; escalate on no-decompose | 2 | should-have | B3a |
|
||||||
| G1 | PAUSE kill-switch + merge-gate honored before dispatch and merge | 1 | 2 | must-have | B2 |
|
| G1 | PAUSE kill-switch + merge-gate honored before dispatch and merge | 2 | must-have | B2 |
|
||||||
| H1 | Cross-domain baseline persona library (exec, marketing, ops, research, assistant + engineering roles) | 2 | 1 | must-have | A1 |
|
| H1 | Cross-domain baseline persona library (exec, marketing, ops, research, assistant + engineering roles) | 1 | must-have | A1 |
|
||||||
| H2 | System-type profiles — declarative mapping of system type to persona roster + topology | 2 | 2 | must-have | H1 |
|
| H2 | System-type profiles — declarative mapping of system type to persona roster + topology | 2 | must-have | H1 |
|
||||||
| H3 | System-type provisioning — user declares type; orchestrator instantiates the matching roster + structure | 2 | 2 | must-have | H2 |
|
| H3 | System-type provisioning — user declares type; orchestrator instantiates the matching roster + structure | 2 | must-have | H2 |
|
||||||
| H4 | Update-surviving persona customization — ad-hoc edits/additions persisted in a PRESERVE-protected override layer (baseline merged with overrides) | 2 | 2 | must-have | H1 |
|
| H4 | Update-surviving persona customization — ad-hoc edits/additions persisted in a PRESERVE-protected override layer (baseline merged with overrides) | 2 | must-have | H1 |
|
||||||
| A5 | NORTH_STAR schema validator — every goal's workstream declared, every workstream has a goal, every depends_on id exists, every tier has a success criterion; runs in CI beside the Markdown regeneration check | 0 | 1 | must-have | A1 |
|
|
||||||
| I1 | One home resolver — a single function resolving MOSAIC_HOME with a sane default, adopted by every module. Today brain-home.ts is imported by 9 modules while 10 still use DEFAULT_MOSAIC_HOME, and MOSAIC_HOME is re-derived ad hoc in 4 places. NS-10 applies - finish the adoption and delete the second path | 0 | 1 | must-have | — |
|
|
||||||
| I2 | mosaic fleet ps sees the fleet that is actually running. Three measured blockers: the roster declares socket `mosaic-fleet` which does not exist, the 18 live sessions are on the default socket, and nothing writes a roster because seats are launched outside the CLI. Make the socket configurable and the roster written at launch, or make ps read tmux + seat dirs directly | 0 | 1 | must-have | I1 |
|
|
||||||
| I3 | Migrate fleet steering onto mosaic agent send --verify (exists, FLEET-OBS-005, spec FR-5) and retire tools/tmux/agent-send.sh, which forges the sender (D33) and returns an uninformative rc (D16, D34). FR-5 predates those defects by a month | 0 | 1 | must-have | I1 |
|
|
||||||
| I4 | mosaic fleet absorbs what launch-seat.sh does and launch-seat.sh is deprecated: compose the prompt file set, force the skill set, wire the style hook, fail closed on any unreadable input, export per-seat git identity, and register the seat in the roster. launch-seat.sh was a manual method; it is the reference implementation, not the destination | 0 | 1 | must-have | I1, I5 |
|
|
||||||
| I5 | Harness probe matrix — verify a working prompt-injection path for claude, codex, opencode and pi, and refuse any runtime whose path is unverified. AC-NS-0 clause 1 ("any configured harness") rests on this. The probe work in docs/plans/2026-08-19_launch-seat-multi-runtime.md (brain, untracked) transfers; its launch-seat.sh target does not | 0 | 1 | must-have | — |
|
|
||||||
| I6 | Finish the heartbeat responder (FLEET-OBS-002, the only Phase-2 task still in-progress; spec FR-2). Health must mean "answered a heartbeat", not "pane alive" — pane state measured two seats wrong on 2026-08-20 | 0 | 1 | must-have | — |
|
|
||||||
| I7 | Independent review and live-fleet dogfood of the Phase-2 verbs (FLEET-OBS-008), then land them (FLEET-OBS-009). Implementation is done and verification is not; "done" in a task file frozen five weeks is not evidence | 0 | 1 | must-have | I2, I3, I4, I6 |
|
|
||||||
| I8 | Neutralize misleading documentation — supersede headers on docs that state a stale status, a false blocker or a retired mechanism. Cheap, and it is tier 0 because a stale doc does not merely fail to help an agent, it actively misroutes one. Rebuilding the documentation is a separate and later job | 0 | 1 | must-have | — |
|
|
||||||
| I9 | Study t3code's agent-attach and multi-provider auth methods and record what transfers. Reference only — Mosaic implements its own within the stack, never adopts the code and never takes the dependency. Informs HOW I/J/K/L are built, not whether | 0 | 1 | should-have | — |
|
|
||||||
| J1 | Web control plane over the gateway — fleet visibility and steering in a browser, same data source as I2 | 1 | 3 | must-have | I2 |
|
|
||||||
| K1 | Desktop and mobile clients against the gateway, authenticated | 2 | 4 | must-have | J1 |
|
|
||||||
| L1 | Per-provider auth profiles with per-session selection | 2 | 4 | must-have | I4 |
|
|
||||||
|
|
||||||
## Assumptions (vetoable)
|
## Assumptions (vetoable)
|
||||||
|
|
||||||
|
|||||||
+2
-129
@@ -6,7 +6,7 @@
|
|||||||
# packages/mosaic/src/commands/fleet.ts (renderNorthStarMarkdown). Edit the YAML,
|
# packages/mosaic/src/commands/fleet.ts (renderNorthStarMarkdown). Edit the YAML,
|
||||||
# never the .md.
|
# never the .md.
|
||||||
#
|
#
|
||||||
# Self-contained Mosaic. The backlog of record is
|
# Self-contained Mosaic. NO Hermes runtime dependency. The backlog of record is
|
||||||
# the Mosaic Backlog on Mosaic's OWN native Postgres storage service.
|
# the Mosaic Backlog on Mosaic's OWN native Postgres storage service.
|
||||||
|
|
||||||
version: 1
|
version: 1
|
||||||
@@ -24,7 +24,7 @@ substrate:
|
|||||||
note: >-
|
note: >-
|
||||||
The Mosaic Backlog is the backlog of record + dispatch engine, built on
|
The Mosaic Backlog is the backlog of record + dispatch engine, built on
|
||||||
Mosaic's native Postgres storage service (@mosaicstack/db drizzle;
|
Mosaic's native Postgres storage service (@mosaicstack/db drizzle;
|
||||||
PGlite-embedded by default, full Postgres by config).
|
PGlite-embedded by default, full Postgres by config). NOT Hermes.
|
||||||
|
|
||||||
standing_objectives:
|
standing_objectives:
|
||||||
- id: NS-1
|
- id: NS-1
|
||||||
@@ -69,53 +69,32 @@ standing_objectives:
|
|||||||
business/operations) and the orchestrator provisions the matching persona
|
business/operations) and the orchestrator provisions the matching persona
|
||||||
roster and org structure from a cross-domain baseline persona library; the
|
roster and org structure from a cross-domain baseline persona library; the
|
||||||
delivery/coding fleet is one profile among many.
|
delivery/coding fleet is one profile among many.
|
||||||
- id: NS-10
|
|
||||||
text: >-
|
|
||||||
An adoption is not complete until the mechanism it replaces is removed.
|
|
||||||
Two live conventions for one concern is the defect, not a transition
|
|
||||||
state. Measured 2026-08-20: brain-home adopted by 9 modules and not 10;
|
|
||||||
MOSAIC_HOME honored in 4 places, each re-deriving it; backlog cards
|
|
||||||
declared while TASKS.md files stayed authoritative. Every one was decided
|
|
||||||
correctly and left half-applied.
|
|
||||||
|
|
||||||
success_criteria:
|
success_criteria:
|
||||||
- id: AC-NS-0
|
|
||||||
tier: 0
|
|
||||||
text: >-
|
|
||||||
The operator launches an agent on any configured harness with one
|
|
||||||
command, observes its state and sends it work without attaching to a
|
|
||||||
terminal multiplexer.
|
|
||||||
- id: AC-NS-1
|
- id: AC-NS-1
|
||||||
tier: 1
|
|
||||||
text: >-
|
text: >-
|
||||||
The supervisor keeps a two-agent floor (1 orchestrator + >=1 enhancer)
|
The supervisor keeps a two-agent floor (1 orchestrator + >=1 enhancer)
|
||||||
healthy across reboot.
|
healthy across reboot.
|
||||||
- id: AC-NS-2
|
- id: AC-NS-2
|
||||||
tier: 1
|
|
||||||
text: >-
|
text: >-
|
||||||
A goal added to this YAML is decomposed to cards and either merged or
|
A goal added to this YAML is decomposed to cards and either merged or
|
||||||
escalated, with no human in the loop.
|
escalated, with no human in the loop.
|
||||||
- id: AC-NS-3
|
- id: AC-NS-3
|
||||||
tier: 1
|
|
||||||
text: >-
|
text: >-
|
||||||
No PR merges with failure/error/no-status/timeout CI, and none bypass
|
No PR merges with failure/error/no-status/timeout CI, and none bypass
|
||||||
pr-merge.sh.
|
pr-merge.sh.
|
||||||
- id: AC-NS-4
|
- id: AC-NS-4
|
||||||
tier: 1
|
|
||||||
text: >-
|
text: >-
|
||||||
TTL is enforced on claims; token caps remain advisory until a real meter
|
TTL is enforced on claims; token caps remain advisory until a real meter
|
||||||
exists.
|
exists.
|
||||||
- id: AC-NS-5
|
- id: AC-NS-5
|
||||||
tier: 1
|
|
||||||
text: >-
|
text: >-
|
||||||
Flipping fleet/run/PAUSED halts dispatch and merges within one tick.
|
Flipping fleet/run/PAUSED halts dispatch and merges within one tick.
|
||||||
- id: AC-NS-6
|
- id: AC-NS-6
|
||||||
tier: 2
|
|
||||||
text: >-
|
text: >-
|
||||||
A user can declare a system type and the fleet provisions the matching
|
A user can declare a system type and the fleet provisions the matching
|
||||||
persona roster + topology from the baseline library, with no code change.
|
persona roster + topology from the baseline library, with no code change.
|
||||||
- id: AC-NS-7
|
- id: AC-NS-7
|
||||||
tier: 2
|
|
||||||
text: >-
|
text: >-
|
||||||
A user-customized persona (edited or added via the orchestrator) survives
|
A user-customized persona (edited or added via the orchestrator) survives
|
||||||
mosaic update: baseline reseed never clobbers user overrides.
|
mosaic update: baseline reseed never clobbers user overrides.
|
||||||
@@ -133,186 +112,80 @@ workstreams:
|
|||||||
title: Meta-loop — session-review + enhancer improvement PRs
|
title: Meta-loop — session-review + enhancer improvement PRs
|
||||||
- id: F
|
- id: F
|
||||||
title: Safety-rails — TTL claims, advisory spend, PAUSE kill-switch
|
title: Safety-rails — TTL claims, advisory spend, PAUSE kill-switch
|
||||||
- id: G
|
|
||||||
title: Kill-switch — operator PAUSE honored before dispatch and merge
|
|
||||||
- id: H
|
- id: H
|
||||||
title: Personas & system profiles — cross-domain library, system-type provisioning, update-surviving customization
|
title: Personas & system profiles — cross-domain library, system-type provisioning, update-surviving customization
|
||||||
- id: I
|
|
||||||
title: Operator surface — launcher, fleet visibility, reliable steering (tier 0)
|
|
||||||
- id: J
|
|
||||||
title: Web control plane — browser surface over the gateway (tier 1)
|
|
||||||
- id: K
|
|
||||||
title: Clients — desktop and mobile over the same backend (tier 2)
|
|
||||||
- id: L
|
|
||||||
title: Auth profiles — per-provider accounts, per-session selection (tier 2)
|
|
||||||
|
|
||||||
# NOTE: workstreams C, D, E and F are declared but currently project no goals.
|
|
||||||
# That is planning debt, not an editing error: their goals have not been written
|
|
||||||
# yet. The A5 validator below reports it rather than letting it stay invisible.
|
|
||||||
|
|
||||||
goals:
|
goals:
|
||||||
- id: A1
|
- id: A1
|
||||||
title: Machine-readable NORTH_STAR.yaml + Markdown projection
|
title: Machine-readable NORTH_STAR.yaml + Markdown projection
|
||||||
phase: 1
|
phase: 1
|
||||||
tier: 1
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: []
|
depends_on: []
|
||||||
- id: A2
|
- id: A2
|
||||||
title: Mosaic Backlog schema + storage-service card store (drizzle/PGlite)
|
title: Mosaic Backlog schema + storage-service card store (drizzle/PGlite)
|
||||||
phase: 1
|
phase: 1
|
||||||
tier: 1
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [A1]
|
depends_on: [A1]
|
||||||
- id: A3a
|
- id: A3a
|
||||||
title: Card lifecycle — create/claim/release with stable ids + depends_on DAG
|
title: Card lifecycle — create/claim/release with stable ids + depends_on DAG
|
||||||
phase: 1
|
phase: 1
|
||||||
tier: 1
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [A2]
|
depends_on: [A2]
|
||||||
- id: A3b
|
- id: A3b
|
||||||
title: TTL-bounded claim enforcement (wall-clock) on cards
|
title: TTL-bounded claim enforcement (wall-clock) on cards
|
||||||
phase: 1
|
phase: 1
|
||||||
tier: 1
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [A3a]
|
depends_on: [A3a]
|
||||||
- id: A4
|
- id: A4
|
||||||
title: Advisory spend projection per card (degrades to TTL, no real meter)
|
title: Advisory spend projection per card (degrades to TTL, no real meter)
|
||||||
phase: 1
|
phase: 1
|
||||||
tier: 1
|
|
||||||
priority: should-have
|
priority: should-have
|
||||||
depends_on: [A3a]
|
depends_on: [A3a]
|
||||||
- id: B1
|
- id: B1
|
||||||
title: Supervisor tick — readiness scan, two-agent-floor health check
|
title: Supervisor tick — readiness scan, two-agent-floor health check
|
||||||
phase: 2
|
phase: 2
|
||||||
tier: 1
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [A3a]
|
depends_on: [A3a]
|
||||||
- id: B2
|
- id: B2
|
||||||
title: Native dispatch/claim — assign ready dependency-satisfied work
|
title: Native dispatch/claim — assign ready dependency-satisfied work
|
||||||
phase: 2
|
phase: 2
|
||||||
tier: 1
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [A3b, B1]
|
depends_on: [A3b, B1]
|
||||||
- id: B3a
|
- id: B3a
|
||||||
title: Planner decompose — goal added to YAML → cards
|
title: Planner decompose — goal added to YAML → cards
|
||||||
phase: 2
|
phase: 2
|
||||||
tier: 1
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [A2, B1]
|
depends_on: [A2, B1]
|
||||||
- id: B3b
|
- id: B3b
|
||||||
title: Replan request on empty backlog; escalate on no-decompose
|
title: Replan request on empty backlog; escalate on no-decompose
|
||||||
phase: 2
|
phase: 2
|
||||||
tier: 1
|
|
||||||
priority: should-have
|
priority: should-have
|
||||||
depends_on: [B3a]
|
depends_on: [B3a]
|
||||||
- id: G1
|
- id: G1
|
||||||
title: PAUSE kill-switch + merge-gate honored before dispatch and merge
|
title: PAUSE kill-switch + merge-gate honored before dispatch and merge
|
||||||
phase: 2
|
phase: 2
|
||||||
tier: 1
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [B2]
|
depends_on: [B2]
|
||||||
- id: H1
|
- id: H1
|
||||||
title: Cross-domain baseline persona library (exec, marketing, ops, research, assistant + engineering roles)
|
title: Cross-domain baseline persona library (exec, marketing, ops, research, assistant + engineering roles)
|
||||||
phase: 1
|
phase: 1
|
||||||
tier: 2
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [A1]
|
depends_on: [A1]
|
||||||
- id: H2
|
- id: H2
|
||||||
title: System-type profiles — declarative mapping of system type to persona roster + topology
|
title: System-type profiles — declarative mapping of system type to persona roster + topology
|
||||||
phase: 2
|
phase: 2
|
||||||
tier: 2
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [H1]
|
depends_on: [H1]
|
||||||
- id: H3
|
- id: H3
|
||||||
title: System-type provisioning — user declares type; orchestrator instantiates the matching roster + structure
|
title: System-type provisioning — user declares type; orchestrator instantiates the matching roster + structure
|
||||||
phase: 2
|
phase: 2
|
||||||
tier: 2
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [H2]
|
depends_on: [H2]
|
||||||
- id: H4
|
- id: H4
|
||||||
title: Update-surviving persona customization — ad-hoc edits/additions persisted in a PRESERVE-protected override layer (baseline merged with overrides)
|
title: Update-surviving persona customization — ad-hoc edits/additions persisted in a PRESERVE-protected override layer (baseline merged with overrides)
|
||||||
phase: 2
|
phase: 2
|
||||||
tier: 2
|
|
||||||
priority: must-have
|
priority: must-have
|
||||||
depends_on: [H1]
|
depends_on: [H1]
|
||||||
- id: A5
|
|
||||||
title: NORTH_STAR schema validator — every goal's workstream declared, every workstream has a goal, every depends_on id exists, every tier has a success criterion; runs in CI beside the Markdown regeneration check
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: must-have
|
|
||||||
depends_on: [A1]
|
|
||||||
- id: I1
|
|
||||||
title: One home resolver — a single function resolving MOSAIC_HOME with a sane default, adopted by every module. Today brain-home.ts is imported by 9 modules while 10 still use DEFAULT_MOSAIC_HOME, and MOSAIC_HOME is re-derived ad hoc in 4 places. NS-10 applies - finish the adoption and delete the second path
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: must-have
|
|
||||||
depends_on: []
|
|
||||||
- id: I2
|
|
||||||
title: 'mosaic fleet ps sees the fleet that is actually running. Three measured blockers: the roster declares socket `mosaic-fleet` which does not exist, the 18 live sessions are on the default socket, and nothing writes a roster because seats are launched outside the CLI. Make the socket configurable and the roster written at launch, or make ps read tmux + seat dirs directly'
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: must-have
|
|
||||||
depends_on: [I1]
|
|
||||||
- id: I3
|
|
||||||
title: Migrate fleet steering onto mosaic agent send --verify (exists, FLEET-OBS-005, spec FR-5) and retire tools/tmux/agent-send.sh, which forges the sender (D33) and returns an uninformative rc (D16, D34). FR-5 predates those defects by a month
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: must-have
|
|
||||||
depends_on: [I1]
|
|
||||||
- id: I4
|
|
||||||
title: 'mosaic fleet absorbs what launch-seat.sh does and launch-seat.sh is deprecated: compose the prompt file set, force the skill set, wire the style hook, fail closed on any unreadable input, export per-seat git identity, and register the seat in the roster. launch-seat.sh was a manual method; it is the reference implementation, not the destination'
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: must-have
|
|
||||||
depends_on: [I1, I5]
|
|
||||||
- id: I5
|
|
||||||
title: Harness probe matrix — verify a working prompt-injection path for claude, codex, opencode and pi, and refuse any runtime whose path is unverified. AC-NS-0 clause 1 ("any configured harness") rests on this. The probe work in docs/plans/2026-08-19_launch-seat-multi-runtime.md (brain, untracked) transfers; its launch-seat.sh target does not
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: must-have
|
|
||||||
depends_on: []
|
|
||||||
- id: I6
|
|
||||||
title: Finish the heartbeat responder (FLEET-OBS-002, the only Phase-2 task still in-progress; spec FR-2). Health must mean "answered a heartbeat", not "pane alive" — pane state measured two seats wrong on 2026-08-20
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: must-have
|
|
||||||
depends_on: []
|
|
||||||
- id: I7
|
|
||||||
title: Independent review and live-fleet dogfood of the Phase-2 verbs (FLEET-OBS-008), then land them (FLEET-OBS-009). Implementation is done and verification is not; "done" in a task file frozen five weeks is not evidence
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: must-have
|
|
||||||
depends_on: [I2, I3, I4, I6]
|
|
||||||
- id: I8
|
|
||||||
title: Neutralize misleading documentation — supersede headers on docs that state a stale status, a false blocker or a retired mechanism. Cheap, and it is tier 0 because a stale doc does not merely fail to help an agent, it actively misroutes one. Rebuilding the documentation is a separate and later job
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: must-have
|
|
||||||
depends_on: []
|
|
||||||
- id: I9
|
|
||||||
title: Study t3code's agent-attach and multi-provider auth methods and record what transfers. Reference only — Mosaic implements its own within the stack, never adopts the code and never takes the dependency. Informs HOW I/J/K/L are built, not whether
|
|
||||||
phase: 1
|
|
||||||
tier: 0
|
|
||||||
priority: should-have
|
|
||||||
depends_on: []
|
|
||||||
- id: J1
|
|
||||||
title: Web control plane over the gateway — fleet visibility and steering in a browser, same data source as I2
|
|
||||||
phase: 3
|
|
||||||
tier: 1
|
|
||||||
priority: must-have
|
|
||||||
depends_on: [I2]
|
|
||||||
- id: K1
|
|
||||||
title: Desktop and mobile clients against the gateway, authenticated
|
|
||||||
phase: 4
|
|
||||||
tier: 2
|
|
||||||
priority: must-have
|
|
||||||
depends_on: [J1]
|
|
||||||
- id: L1
|
|
||||||
title: Per-provider auth profiles with per-session selection
|
|
||||||
phase: 4
|
|
||||||
tier: 2
|
|
||||||
priority: must-have
|
|
||||||
depends_on: [I4]
|
|
||||||
|
|
||||||
assumptions:
|
assumptions:
|
||||||
- id: ASM-1
|
- id: ASM-1
|
||||||
|
|||||||
@@ -1,7 +1,7 @@
|
|||||||
# PRD — Mosaic Fleet Suite (init, configure, operate)
|
# PRD — Mosaic Fleet Suite (init, configure, operate)
|
||||||
|
|
||||||
> **Workstream:** W-FLEET (Fleet) under mission `mvp-20260312` · **Phase:** 3→4 productization
|
> **Workstream:** W-FLEET (Fleet) under mission `mvp-20260312` · **Phase:** 3→4 productization
|
||||||
> **North star:** [docs/fleet/FLEET-DOCTRINE.md](./FLEET-DOCTRINE.md) · prior: Phase-2 observability (#579), durable launch (#581), real-agent enablement (#583/#584/#586), releases 0.0.35–0.0.37
|
> **North star:** [docs/fleet/north-star.md](./north-star.md) · prior: Phase-2 observability (#579), durable launch (#581), real-agent enablement (#583/#584/#586), releases 0.0.35–0.0.37
|
||||||
> **Lead:** Jarvis @ `w-jarvis`. **Collaborator:** coder agent @ `dragon-lin` ([email protected]:coder0-0).
|
> **Lead:** Jarvis @ `w-jarvis`. **Collaborator:** coder agent @ `dragon-lin` ([email protected]:coder0-0).
|
||||||
> Owner of this file: Fleet workstream lead. Does not modify MVP single-writer control-plane files.
|
> Owner of this file: Fleet workstream lead. Does not modify MVP single-writer control-plane files.
|
||||||
|
|
||||||
|
|||||||
+1
-1
@@ -1,7 +1,7 @@
|
|||||||
# PRD — Fleet Phase 2: Operator Observability
|
# PRD — Fleet Phase 2: Operator Observability
|
||||||
|
|
||||||
> **Workstream:** W-FLEET under `mvp-20260312` · **Phase:** 2
|
> **Workstream:** W-FLEET under `mvp-20260312` · **Phase:** 2
|
||||||
> **North star:** [docs/fleet/FLEET-DOCTRINE.md](./FLEET-DOCTRINE.md)
|
> **North star:** [docs/fleet/north-star.md](./north-star.md)
|
||||||
> **Source umbrella PRD:** [docs/PRD.md](../PRD.md) (Mosaic Stack v0.1.0)
|
> **Source umbrella PRD:** [docs/PRD.md](../PRD.md) (Mosaic Stack v0.1.0)
|
||||||
> **Tracks task:** `fleet-observability-1` — restore operator observability into fleet agent sessions.
|
> **Tracks task:** `fleet-observability-1` — restore operator observability into fleet agent sessions.
|
||||||
|
|
||||||
|
|||||||
+1
-16
@@ -1,25 +1,10 @@
|
|||||||
# Tasks — W-FLEET (Fleet) Phase 2: Observability
|
# Tasks — W-FLEET (Fleet) Phase 2: Observability
|
||||||
|
|
||||||
> ---
|
|
||||||
>
|
|
||||||
> **STATUS: SUPERSEDED — 2026-08-20.** kind `tracking` · superseded by `docs/fleet/NORTH_STAR.yaml`
|
|
||||||
>
|
|
||||||
> This file is the pre-backlog tracking mechanism. `NS-2` in the north star declares the
|
|
||||||
> replacement: every backlog item is a Mosaic Backlog card projected from the YAML. That
|
|
||||||
> model replaced this one and nobody retired the old file, so it kept reading as
|
|
||||||
> authoritative while going stale.
|
|
||||||
>
|
|
||||||
> **Do not trust a status in this file.** Verified 2026-08-20: it was already behind the
|
|
||||||
> code when it froze five weeks ago. The `FLEET-OBS` series was the one thing worth salvaging and
|
|
||||||
> is now carried as goals `I2`, `I3`, `I6` and `I7` at tier 0.
|
|
||||||
>
|
|
||||||
> Kept as a record of what was believed. Do not update it; update the YAML.
|
|
||||||
|
|
||||||
> Workstream task file for the Fleet. Single-writer: Fleet workstream lead (orchestrator).
|
> Workstream task file for the Fleet. Single-writer: Fleet workstream lead (orchestrator).
|
||||||
> Workers read but never modify. This is **not** the MVP rollup (`docs/TASKS.md`) — a
|
> Workers read but never modify. This is **not** the MVP rollup (`docs/TASKS.md`) — a
|
||||||
> rollup row is proposed to the MVP orchestrator, not written here.
|
> rollup row is proposed to the MVP orchestrator, not written here.
|
||||||
>
|
>
|
||||||
> Mission: `mvp-20260312` · PRD: [docs/fleet/PRD.md](./PRD.md) · North star: [docs/fleet/FLEET-DOCTRINE.md](./FLEET-DOCTRINE.md)
|
> Mission: `mvp-20260312` · PRD: [docs/fleet/PRD.md](./PRD.md) · North star: [docs/fleet/north-star.md](./north-star.md)
|
||||||
> Status: `not-started` | `in-progress` | `done` | `blocked` | `failed`
|
> Status: `not-started` | `in-progress` | `done` | `blocked` | `failed`
|
||||||
|
|
||||||
| id | status | description | depends_on | agent | pr | notes |
|
| id | status | description | depends_on | agent | pr | notes |
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
# F4 — Orchestrator chat connector + Matrix (local homeserver)
|
# F4 — Orchestrator chat connector + Matrix (local homeserver)
|
||||||
|
|
||||||
> **Issue:** #616 · **Doctrine:** `docs/fleet/FLEET-DOCTRINE.md` (#613) — orchestrator-chat-connector decision.
|
> **Issue:** #616 · **Doctrine:** `docs/fleet/north-star.md` (#613) — orchestrator-chat-connector decision.
|
||||||
> **Status:** Phase 1 (abstraction + scaffold) in this PR; Phase 2+ are follow-ups (below).
|
> **Status:** Phase 1 (abstraction + scaffold) in this PR; Phase 2+ are follow-ups (below).
|
||||||
|
|
||||||
## Goal
|
## Goal
|
||||||
|
|||||||
@@ -1,26 +1,9 @@
|
|||||||
---
|
# Mosaic Fleet — North Star
|
||||||
kind: spec
|
|
||||||
parent: docs/fleet/NORTH_STAR.yaml
|
|
||||||
status: active
|
|
||||||
---
|
|
||||||
|
|
||||||
# Mosaic Fleet — Doctrine
|
|
||||||
|
|
||||||
> **This is the WHY. `NORTH_STAR.yaml` is the WHAT and WHEN.**
|
|
||||||
> Renamed from `north-star.md` on 2026-08-20. It sat one character away from the
|
|
||||||
> generated `NORTH_STAR.md` in the same directory, and the two are read by different
|
|
||||||
> populations — the PRDs and TASKS files cite this one, while the agent role contracts
|
|
||||||
> and the generator spec cite the YAML pair. Same-name-different-thing was the confusion;
|
|
||||||
> the content was never in conflict.
|
|
||||||
>
|
|
||||||
> **Nothing here overrides `NORTH_STAR.yaml`.** Where this document states a plan item,
|
|
||||||
> the YAML is authoritative. Where it states a decision, a rationale, or a role
|
|
||||||
> definition, this document is the record and the YAML carries none of it.
|
|
||||||
>
|
|
||||||
> **Workstream:** W-FLEET (Fleet) under mission `mvp-20260312`
|
> **Workstream:** W-FLEET (Fleet) under mission `mvp-20260312`
|
||||||
> **Umbrella:** [docs/MISSION-MANIFEST.md](../MISSION-MANIFEST.md)
|
> **Umbrella:** [docs/MISSION-MANIFEST.md](../MISSION-MANIFEST.md) · [docs/PRD.md](../PRD.md) (Mosaic Stack v0.1.0)
|
||||||
> **Authored:** 2026-06-20. Owner: Fleet workstream lead.
|
> **Status:** doctrine — authored 2026-06-20. Owner of this file: Fleet workstream lead.
|
||||||
> This document does **not** modify the MVP rollup.
|
> This document does **not** modify the MVP rollup; a rollup row is proposed, not written here.
|
||||||
|
|
||||||
## Vision
|
## Vision
|
||||||
|
|
||||||
@@ -281,17 +264,15 @@ Dedicated Postgres **instance** vs. dedicated **schema** in the existing instanc
|
|||||||
Recommendation: dedicated schema, existing instance (a migration file, not new infra);
|
Recommendation: dedicated schema, existing instance (a migration file, not new infra);
|
||||||
re-evaluate if isolation or write-volume demands it.
|
re-evaluate if isolation or write-volume demands it.
|
||||||
|
|
||||||
## Phased roadmap — SUPERSEDED
|
## Phased roadmap
|
||||||
|
|
||||||
Superseded 2026-08-20 by [`NORTH_STAR.yaml`](./NORTH_STAR.yaml), whose `goals` carry both
|
| Phase | Outcome | Status |
|
||||||
a `phase` (build order) and a `tier` (which promise the goal delivers). The five-phase
|
| ---------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ | ------- |
|
||||||
table that stood here could not express those as separate axes, and its "Phase 2 —
|
| 0–1 | tmux PoC, hardening, published CLI v0.0.34 (#565–#568) | ✅ done |
|
||||||
Observability ▶ now" row stayed unfalsified for two months because a phase has no exit
|
| **2 — Observability** | fleet ps (host+tenant aware join), heartbeat protocol + dogfood stub answers it, agent watch (read-only), agent send --verify receipts | ▶ now |
|
||||||
test. Tiers do: see `AC-NS-0` through `AC-NS-7`.
|
| 3 — Real runtimes | claude/codex/pi/opencode answer heartbeat; **hybrid lifecycle** (core always-on: **orchestrator + enhancer**; ephemeral workers per lane) | planned |
|
||||||
|
| 4 — Unified definition | one agent schema in gateway; mosaic agent --new → materialized per-tenant session; uid-tenant provisioning; **`fleet` schema migration + `forge-exec` TaskExecutor adapter (forge → `agent-send.sh`)** | planned |
|
||||||
The phase-2 content itself is not lost — it is specified in
|
| 5 — Control plane | federation-backed cross-host × cross-tenant fleet view; **webUI** (surface chosen then) for MVP-X1 parity; **central register live (spend ledger, docs-as-projections, multi-host Kanban)** | planned |
|
||||||
[`PRD.md`](./PRD.md) (Fleet Phase 2: Operator Observability) and is now tracked as
|
|
||||||
goals `I1`–`I5` at tier 0.
|
|
||||||
|
|
||||||
## Decisions of record (2026-06-20, with Jason)
|
## Decisions of record (2026-06-20, with Jason)
|
||||||
|
|
||||||
@@ -4,16 +4,7 @@
|
|||||||
**Tracking:** [Mosaic Stack issue #751](https://git.mosaicstack.dev/mosaicstack/stack/issues/751)
|
**Tracking:** [Mosaic Stack issue #751](https://git.mosaicstack.dev/mosaicstack/stack/issues/751)
|
||||||
**Execution:** USC web1 only; collision-free GPT coder2/3/4/5 lanes
|
**Execution:** USC web1 only; collision-free GPT coder2/3/4/5 lanes
|
||||||
**Contract:** `SHARED-CONTRACT.md` + four `contracts/*.v1.ts` files
|
**Contract:** `SHARED-CONTRACT.md` + four `contracts/*.v1.ts` files
|
||||||
|
**Implementation hold:** no feature slice starts until the canon PR is merged to `main` with terminal-green CI; after merge, each slice remains held until every declared KBN prerequisite is complete.
|
||||||
> **HOLD LIFTED — 2026-08-20.** The stated condition ("no feature slice starts until the
|
|
||||||
> canon PR is merged to `main` with terminal-green CI") was **met on 2026-07-14** by
|
|
||||||
> `49e8a541` — _docs(#751): Publish native Kanban/SOT canon (#752)_ — which is on both
|
|
||||||
> `main` and `next`, and `docs/requirements/native-kanban-sot.md` is present on `main`.
|
|
||||||
> The line below was never updated, so for five weeks this file advertised a blocker that
|
|
||||||
> did not exist. The status line above ("PUBLICATION IN PROGRESS") is stale for the same
|
|
||||||
> reason. This file is a build plan, not a task tracker, and is NOT superseded.
|
|
||||||
|
|
||||||
**Implementation hold (SATISFIED — see note above):** no feature slice starts until the canon PR is merged to `main` with terminal-green CI; after merge, each slice remains held until every declared KBN prerequisite is complete.
|
|
||||||
|
|
||||||
> This publication file is not a runtime task authority. After cutover, repository `TASKS.md` is generated read-only and never imported.
|
> This publication file is not a runtime task authority. After cutover, repository `TASKS.md` is generated read-only and never imported.
|
||||||
|
|
||||||
|
|||||||
@@ -35,4 +35,4 @@ A standing panel of senior voices, each arguing from a fixed vantage. The board
|
|||||||
deliberately slow and adversarial — its value is catching the expensive mistake
|
deliberately slow and adversarial — its value is catching the expensive mistake
|
||||||
before a single agent-hour is spent on it.
|
before a single agent-hour is spent on it.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` ('board' role = forge BOD; role library).
|
> Doctrine: `docs/fleet/north-star.md` ('board' role = forge BOD; role library).
|
||||||
|
|||||||
@@ -33,4 +33,4 @@ The code role writes the change and opens the PR; it never touches the merge pat
|
|||||||
The focused builder. It takes one well-scoped card, drives it to green, opens a
|
The focused builder. It takes one well-scoped card, drives it to green, opens a
|
||||||
clean PR, and hands off — never reaching past the card it was given.
|
clean PR, and hands off — never reaching past the card it was given.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library).
|
> Doctrine: `docs/fleet/north-star.md` (role library).
|
||||||
|
|||||||
@@ -35,4 +35,4 @@ The work-breakdown specialist. It takes a phased plan and a DAG and emits a clea
|
|||||||
linked set of single-PR cards on the Mosaic backlog — then steps back and lets the
|
linked set of single-PR cards on the Mosaic backlog — then steps back and lets the
|
||||||
executors run.
|
executors run.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library); spend accounting is a process mandate.
|
> Doctrine: `docs/fleet/north-star.md` (role library); spend accounting is a process mandate.
|
||||||
|
|||||||
@@ -36,4 +36,4 @@ The scribe of record. It makes sure the docs and the north star's projections
|
|||||||
describe the system as it actually is, and it never lets two writers fight over one
|
describe the system as it actually is, and it never lets two writers fight over one
|
||||||
TASKS file.
|
TASKS file.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library).
|
> Doctrine: `docs/fleet/north-star.md` (role library).
|
||||||
|
|||||||
@@ -38,4 +38,4 @@ The orchestrator alone optimizes for _this_ delivery; the enhancer optimizes for
|
|||||||
delivery — self-healing the fleet's tools, skills, and harnesses, and routing real defects
|
delivery — self-healing the fleet's tools, skills, and harnesses, and routing real defects
|
||||||
upstream. Together they are the irreducible core; every other role is added on demand.
|
upstream. Together they are the irreducible core; every other role is added on demand.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (two-agent floor + role library).
|
> Doctrine: `docs/fleet/north-star.md` (two-agent floor + role library).
|
||||||
|
|||||||
@@ -46,4 +46,4 @@ The single, accountable gatekeeper. It waits for green CI (`pr-ci-wait.sh`),
|
|||||||
respects the pause switch, merges only through `pr-merge.sh`, and records every
|
respects the pause switch, merges only through `pr-merge.sh`, and records every
|
||||||
decision — so the fleet has exactly one trustworthy door to production.
|
decision — so the fleet has exactly one trustworthy door to production.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library); merge path: `pr-merge.sh` + `pr-ci-wait.sh`; forbidden paths: `pr-merge.sh` guard.
|
> Doctrine: `docs/fleet/north-star.md` (role library); merge path: `pr-merge.sh` + `pr-ci-wait.sh`; forbidden paths: `pr-merge.sh` guard.
|
||||||
|
|||||||
@@ -35,4 +35,4 @@ The on-call dispatcher. It makes sure every escalation is seen and re-seen until
|
|||||||
handled, and it holds the one switch that can stop the fleet when something is
|
handled, and it holds the one switch that can stop the fleet when something is
|
||||||
wrong.
|
wrong.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library); pause switch: `fleet/run/PAUSED`.
|
> Doctrine: `docs/fleet/north-star.md` (role library); pause switch: `fleet/run/PAUSED`.
|
||||||
|
|||||||
@@ -43,4 +43,4 @@ A lean, decisive coordinator. It thinks in readiness and throughput, dispatches
|
|||||||
next ready card the instant a dependency clears, and never lets an idle agent sit
|
next ready card the instant a dependency clears, and never lets an idle agent sit
|
||||||
while ready work exists — keeping its own context minimal so the loop never slows.
|
while ready work exists — keeping its own context minimal so the loop never slows.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (two-agent floor + role library).
|
> Doctrine: `docs/fleet/north-star.md` (two-agent floor + role library).
|
||||||
|
|||||||
@@ -38,4 +38,4 @@ The architect of the mission's shape. It thinks in phases and dependencies, hand
|
|||||||
a clean DAG to decomposition, and reports its plan back to the orchestrator that
|
a clean DAG to decomposition, and reports its plan back to the orchestrator that
|
||||||
dispatched it.
|
dispatched it.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (two-agent floor + role library).
|
> Doctrine: `docs/fleet/north-star.md` (two-agent floor + role library).
|
||||||
|
|||||||
@@ -34,4 +34,4 @@ The rebase role keeps PR branches fresh; it never approves or merges.
|
|||||||
The janitor of the merge queue. It quietly keeps branches current and re-runnable,
|
The janitor of the merge queue. It quietly keeps branches current and re-runnable,
|
||||||
and knows when a conflict is beyond a mechanical rebase and must be escalated.
|
and knows when a conflict is beyond a mechanical rebase and must be escalated.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library).
|
> Doctrine: `docs/fleet/north-star.md` (role library).
|
||||||
|
|||||||
@@ -35,4 +35,4 @@ The careful reader. It assumes nothing, checks the change against its card and i
|
|||||||
tests, and is willing to say "not yet" — its value is catching the wrong change
|
tests, and is willing to say "not yet" — its value is catching the wrong change
|
||||||
before it reaches the merge-gate.
|
before it reaches the merge-gate.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library).
|
> Doctrine: `docs/fleet/north-star.md` (role library).
|
||||||
|
|||||||
@@ -36,4 +36,4 @@ tree or the merge path.
|
|||||||
The adversary on your side. It reads every diff asking "how does this get exploited
|
The adversary on your side. It reads every diff asking "how does this get exploited
|
||||||
or leak?" — the second, security-focused pair of eyes before the merge-gate.
|
or leak?" — the second, security-focused pair of eyes before the merge-gate.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library); forbidden paths: `pr-merge.sh` guard.
|
> Doctrine: `docs/fleet/north-star.md` (role library); forbidden paths: `pr-merge.sh` guard.
|
||||||
|
|||||||
@@ -34,4 +34,4 @@ The retrospective analyst. It reads completed sessions and distills them into cl
|
|||||||
actionable signals — the raw material the enhancer uses to make the fleet better
|
actionable signals — the raw material the enhancer uses to make the fleet better
|
||||||
next time.
|
next time.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library); consumed by the enhancer role.
|
> Doctrine: `docs/fleet/north-star.md` (role library); consumed by the enhancer role.
|
||||||
|
|||||||
@@ -34,4 +34,4 @@ merge path.
|
|||||||
The skeptic who insists on running it. It trusts observed behavior over claimed
|
The skeptic who insists on running it. It trusts observed behavior over claimed
|
||||||
behavior, and turns "should work" into "verified works" — or a concrete bug report.
|
behavior, and turns "should work" into "verified works" — or a concrete bug report.
|
||||||
|
|
||||||
> Doctrine: `docs/fleet/FLEET-DOCTRINE.md` (role library).
|
> Doctrine: `docs/fleet/north-star.md` (role library).
|
||||||
|
|||||||
@@ -257,36 +257,8 @@ assert_owned_tmux_server() {
|
|||||||
fail "tmux server ownership or environment validation failed"
|
fail "tmux server ownership or environment validation failed"
|
||||||
}
|
}
|
||||||
|
|
||||||
# Lease-broker socket preflight (#1292). The gated runtime (`mosaic yolo …` →
|
# Validate exact server ownership before querying, cleaning, or creating any
|
||||||
# launch-runtime.py) registers with the broker or dies ~4 seconds in, with the
|
# managed session. An unmanaged or contaminated named socket is never repaired.
|
||||||
# diagnostic invisible because tmux destroys the dead pane. This check runs
|
|
||||||
# BEFORE any tmux effect — including the ownership probe below — so a host
|
|
||||||
# without a broker produces a named, surviving refusal instead of a doomed
|
|
||||||
# pane. Exit 75 (EX_TEMPFAIL), distinct from 64 (bad projection) and 69 (host
|
|
||||||
# not ready for other reasons); the agent@ unit is Type=oneshot with no
|
|
||||||
# Restart=, so the failed unit keeps its message instead of looping. Socket
|
|
||||||
# resolution matches launch.ts's defaultLeaseBrokerSocket precedence exactly.
|
|
||||||
# This preflight DETECTS and REFUSES — it never starts the broker (activation
|
|
||||||
# belongs to the fleet control plane; a component that both detects and fixes
|
|
||||||
# cannot be used to measure whether the fix worked).
|
|
||||||
broker_socket_path() {
|
|
||||||
if [ -n "${MOSAIC_LEASE_BROKER_SOCKET:-}" ]; then
|
|
||||||
printf '%s\n' "$MOSAIC_LEASE_BROKER_SOCKET"
|
|
||||||
return 0
|
|
||||||
fi
|
|
||||||
local runtime_dir="${XDG_RUNTIME_DIR:-/run/user/$(id -u)}"
|
|
||||||
printf '%s\n' "${runtime_dir}/mosaic-lease/broker.sock"
|
|
||||||
}
|
|
||||||
|
|
||||||
if [ "$MODE" = "launch" ]; then
|
|
||||||
_broker_socket=$(broker_socket_path)
|
|
||||||
if [ ! -S "$_broker_socket" ]; then
|
|
||||||
echo "[fleet] FAIL_LAUNCH broker-absent: lease broker socket ${_broker_socket} missing; runtime launch denied (#1292)." >&2
|
|
||||||
echo "[fleet] remedy: systemctl --user enable --now mosaic-lease-broker.service (or reinstall via: mosaic fleet install)" >&2
|
|
||||||
exit 75
|
|
||||||
fi
|
|
||||||
fi
|
|
||||||
|
|
||||||
assert_owned_tmux_server
|
assert_owned_tmux_server
|
||||||
|
|
||||||
if [ "$MODE" = interaction ]; then
|
if [ "$MODE" = interaction ]; then
|
||||||
|
|||||||
@@ -1,222 +0,0 @@
|
|||||||
#!/usr/bin/env bash
|
|
||||||
# CI-fit regression suite for the #1292 lease-broker socket preflight in
|
|
||||||
# start-agent-session.sh.
|
|
||||||
#
|
|
||||||
# WHY THIS SUITE IS CI-FIT WHERE test-start-agent-session.sh IS NOT (#1017/#1270
|
|
||||||
# context): that older suite's precondition is "the host does not have the pi
|
|
||||||
# binary", which a CI image that ships pi violates — its guard correctly
|
|
||||||
# refuses to report a pass there, so it is excluded from the chain. THIS suite
|
|
||||||
# controls its own preconditions instead of inheriting them from the host: a
|
|
||||||
# fake tmux on PATH, a fake mosaic on PATH, a real unix socket created in a
|
|
||||||
# tmpdir, a hermetic env (env -i, fake HOME, GIT_CONFIG_GLOBAL severed). It
|
|
||||||
# never depends on what the host has installed, so a green here means the same
|
|
||||||
# thing on every host. Anyone adding cases: keep that property — no case may
|
|
||||||
# depend on host state.
|
|
||||||
#
|
|
||||||
# The failure this suite is written down to catch (#1292): a seat launched on a
|
|
||||||
# host with no lease broker dies ~4 seconds in at registration, with the
|
|
||||||
# diagnostic invisible because tmux destroys the dead pane. The preflight runs
|
|
||||||
# BEFORE any tmux effect and refuses with a NAMED code (exit 75, EX_TEMPFAIL)
|
|
||||||
# so the message survives. The agent@ unit is Type=oneshot with no Restart=,
|
|
||||||
# so a failed unit keeps its output instead of looping.
|
|
||||||
#
|
|
||||||
# Cases:
|
|
||||||
# 1. absent socket -> exit 75, message names broker-absent + socket path +
|
|
||||||
# remedy, and NO tmux session was ever created (the doomed-pane half).
|
|
||||||
# 2. present socket (real unix socket in tmpdir) -> proceeds PAST the
|
|
||||||
# preflight (the suite then stops at the next precondition, proving the
|
|
||||||
# preflight was not the refusal).
|
|
||||||
# 3. explicit MOSAIC_LEASE_BROKER_SOCKET wins over XDG_RUNTIME_DIR default.
|
|
||||||
# 4. --stop mode does NOT require the broker (teardown must not be fenced on
|
|
||||||
# a component whose absence is exactly what teardown may follow).
|
|
||||||
#
|
|
||||||
# Sabotage control, run by the developer (not in-suite): remove the preflight
|
|
||||||
# block from start-agent-session.sh, re-run — case 1 fails (a tmux session is
|
|
||||||
# created / exit is not 75), cases 2-4 still pass; restore byte-identically.
|
|
||||||
|
|
||||||
set -euo pipefail
|
|
||||||
|
|
||||||
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
|
|
||||||
WORK_DIR="${MOSAIC_TEST_WORK_DIR:-$PWD/.mosaic-test-work/agent-session-broker-preflight}"
|
|
||||||
FAKE_HOME="$WORK_DIR/home"
|
|
||||||
BIN_DIR="$WORK_DIR/bin"
|
|
||||||
ENV_DIR="$WORK_DIR/env"
|
|
||||||
SOCK_DIR="$WORK_DIR/sockets"
|
|
||||||
LOG_FILE="$WORK_DIR/tmux-calls.log"
|
|
||||||
|
|
||||||
rm -rf "$WORK_DIR"
|
|
||||||
# The script asserts a managed directory tree under MOSAIC_HOME: mosaic/,
|
|
||||||
# mosaic/fleet/, mosaic/fleet/agents/ — private (0700/0750-style) modes, no
|
|
||||||
# symlinks — plus a per-agent env projection. Build the full tree the launcher
|
|
||||||
# expects so the suite reaches the BROKER preflight rather than dying at
|
|
||||||
# environment validation.
|
|
||||||
mkdir -p "$FAKE_HOME/.config/mosaic/fleet/agents" "$BIN_DIR" "$SOCK_DIR"
|
|
||||||
chmod 700 "$FAKE_HOME/.config/mosaic" "$FAKE_HOME/.config/mosaic/fleet/agents"
|
|
||||||
chmod 750 "$FAKE_HOME/.config/mosaic/fleet"
|
|
||||||
cat > "$FAKE_HOME/.config/mosaic/fleet/agents/preflight-test.env.generated" <<'ENVEOF'
|
|
||||||
MOSAIC_AGENT_NAME=preflight-test
|
|
||||||
MOSAIC_GIT_IDENTITY=preflight-test
|
|
||||||
MOSAIC_AGENT_CLASS=worker
|
|
||||||
MOSAIC_AGENT_RUNTIME=pi
|
|
||||||
MOSAIC_AGENT_MODEL=
|
|
||||||
MOSAIC_AGENT_REASONING=
|
|
||||||
MOSAIC_AGENT_TOOL_POLICY=code
|
|
||||||
MOSAIC_AGENT_WORKDIR=/tmp
|
|
||||||
MOSAIC_TMUX_SOCKET=mosaic-fleet
|
|
||||||
ENVEOF
|
|
||||||
chmod 600 "$FAKE_HOME/.config/mosaic/fleet/agents/preflight-test.env.generated"
|
|
||||||
|
|
||||||
# ─── Fake tmux: records every invocation; new-session marks the marker. ────
|
|
||||||
: > "$LOG_FILE"
|
|
||||||
cat > "$BIN_DIR/tmux" <<SH
|
|
||||||
#!/usr/bin/env bash
|
|
||||||
printf 'tmux %s\n' "\$*" >> "$LOG_FILE"
|
|
||||||
if [[ "\$*" == *new-session* ]]; then
|
|
||||||
echo "TMUX-NEW-SESSION-INVOKED" >> "$LOG_FILE"
|
|
||||||
fi
|
|
||||||
exit 0
|
|
||||||
SH
|
|
||||||
chmod +x "$BIN_DIR/tmux"
|
|
||||||
|
|
||||||
# ─── Fake mosaic/pi binaries so the script proceeds past its own lookups. ───
|
|
||||||
for bin in mosaic pi claude; do
|
|
||||||
printf '#!/usr/bin/env bash\nexit 0\n' > "$BIN_DIR/$bin"
|
|
||||||
chmod +x "$BIN_DIR/$bin"
|
|
||||||
done
|
|
||||||
|
|
||||||
# ─── Minimal launch environment the script expects. ────────────────────────
|
|
||||||
# (Enough for the preflight to be reached; later stages will still fail in
|
|
||||||
# case 2 — that is expected and asserted.)
|
|
||||||
run_session_script() {
|
|
||||||
local mode="$1"; shift
|
|
||||||
(
|
|
||||||
cd "$WORK_DIR"
|
|
||||||
env -i HOME="$FAKE_HOME" PATH="$BIN_DIR:/usr/bin:/bin" \
|
|
||||||
GIT_CONFIG_GLOBAL=/dev/null GIT_CONFIG_SYSTEM=/dev/null \
|
|
||||||
MOSAIC_HOME="$FAKE_HOME/.config/mosaic" \
|
|
||||||
AGENT_NAME=preflight-test \
|
|
||||||
"$@" \
|
|
||||||
bash "$SCRIPT_DIR/start-agent-session.sh" $mode preflight-test
|
|
||||||
)
|
|
||||||
}
|
|
||||||
|
|
||||||
fail=0
|
|
||||||
assert() {
|
|
||||||
local desc="$1" expected="$2" actual="$3"
|
|
||||||
if [[ "$expected" != "$actual" ]]; then
|
|
||||||
echo "FAIL: $desc — expected '$expected', got '$actual'" >&2
|
|
||||||
fail=1
|
|
||||||
fi
|
|
||||||
}
|
|
||||||
assert_contains() {
|
|
||||||
local desc="$1" haystack="$2" needle="$3"
|
|
||||||
[[ "$haystack" == *"$needle"* ]] || { echo "FAIL: $desc — missing '$needle' in: $haystack" >&2; fail=1; }
|
|
||||||
}
|
|
||||||
assert_not_contains() {
|
|
||||||
local desc="$1" haystack="$2" needle="$3"
|
|
||||||
if [[ "$haystack" == *"$needle"* ]]; then
|
|
||||||
echo "FAIL: $desc — must not contain '$needle'" >&2
|
|
||||||
fail=1
|
|
||||||
fi
|
|
||||||
return 0
|
|
||||||
}
|
|
||||||
|
|
||||||
# ─── 1. Absent socket → named refusal, NO tmux session. ────────────────────
|
|
||||||
: > "$LOG_FILE"
|
|
||||||
stderr_file="$WORK_DIR/stderr-1.tmp"
|
|
||||||
set +e
|
|
||||||
out=$(run_session_script "" MOSAIC_LEASE_BROKER_SOCKET="$SOCK_DIR/absent.sock" 2>"$stderr_file")
|
|
||||||
rc=$?
|
|
||||||
set -e
|
|
||||||
assert "absent socket exit code" "75" "$rc"
|
|
||||||
err=$(cat "$stderr_file")
|
|
||||||
assert_contains "absent socket names the failure" "$err" "FAIL_LAUNCH broker-absent"
|
|
||||||
assert_contains "absent socket names the socket path" "$err" "$SOCK_DIR/absent.sock"
|
|
||||||
assert_contains "absent socket names a remedy" "$err" "mosaic fleet install"
|
|
||||||
log1=$(cat "$LOG_FILE")
|
|
||||||
assert_not_contains "absent socket must not create a tmux session" "$log1" "TMUX-NEW-SESSION-INVOKED"
|
|
||||||
|
|
||||||
# ─── 2. Present socket → passes the preflight. ─────────────────────────────
|
|
||||||
# Expected: ownership/env checks AFTER the preflight may refuse (fixture is
|
|
||||||
# minimal by design); the assertion is only that the refusal is NOT
|
|
||||||
# broker-absent and the exit is NOT 75.
|
|
||||||
# Create a REAL unix socket: a detached python holder binds it and stays alive
|
|
||||||
# for the duration (bash cannot create sockets; a foreground python would
|
|
||||||
# close the socket on exit and -S on a closed-but-unlinked path fails). Written
|
|
||||||
# as a script file + setsid nohup so no job-control/heredoc interaction with
|
|
||||||
# set -e can silently kill the suite.
|
|
||||||
# AF_UNIX binds cap at 108 path bytes; the suite's workdir exceeds that, so
|
|
||||||
# the live socket lives at a SHORT path under /tmp (unique per run, cleaned
|
|
||||||
# with the suite). The preflight takes its socket path explicitly, so this
|
|
||||||
# stays fully controlled.
|
|
||||||
# A real unix socket at a SHORT absolute path (AF_UNIX limit is 108 bytes,
|
|
||||||
# so the repo-deep SOCK_DIR cannot host it). The name is composed, not
|
|
||||||
# `mktemp -u`: the CI image's mktemp dialect rejects that invocation
|
|
||||||
# (pipeline 2562: "mktemp: : Invalid argument"), and no pre-existing file is
|
|
||||||
# wanted anyway — the holder binds it fresh.
|
|
||||||
LIVE_SOCK="/tmp/mosaic-preflight-$RANDOM-$$.sock"
|
|
||||||
trap 'rm -f "$LIVE_SOCK"' EXIT
|
|
||||||
rm -f "$SOCK_DIR/live.sock" "$LIVE_SOCK"
|
|
||||||
cat > "$SOCK_DIR/holder.py" <<'PY'
|
|
||||||
import socket, sys, time
|
|
||||||
path = sys.argv[1]
|
|
||||||
s = socket.socket(socket.AF_UNIX, socket.SOCK_STREAM)
|
|
||||||
s.bind(path)
|
|
||||||
s.listen(1)
|
|
||||||
time.sleep(120)
|
|
||||||
PY
|
|
||||||
python3 "$SOCK_DIR/holder.py" "$LIVE_SOCK" >/dev/null 2>"$SOCK_DIR/holder.err" &
|
|
||||||
HOLDER_PID=$!
|
|
||||||
# Wait for the socket object to exist (bind is near-instant, but do not race it).
|
|
||||||
for _ in $(seq 1 50); do
|
|
||||||
[ -S "$LIVE_SOCK" ] && break
|
|
||||||
sleep 0.1
|
|
||||||
done
|
|
||||||
if [ ! -S "$LIVE_SOCK" ]; then
|
|
||||||
echo "FAIL: could not create live socket fixture (holder pid $HOLDER_PID)" >&2
|
|
||||||
ps -p "$HOLDER_PID" -o pid,stat,cmd --no-headers >&2 || echo "(holder exited)" >&2
|
|
||||||
cat "$SOCK_DIR/holder.err" >&2 || true
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
: > "$LOG_FILE"
|
|
||||||
set +e
|
|
||||||
out=$(run_session_script "" MOSAIC_LEASE_BROKER_SOCKET="$LIVE_SOCK" 2>"$WORK_DIR/stderr-2.tmp")
|
|
||||||
rc=$?
|
|
||||||
set -e
|
|
||||||
# The preflight PASSED if the failure (whatever later stage refused) is NOT
|
|
||||||
# the broker refusal, and tmux was reached or a later precondition named
|
|
||||||
# something else.
|
|
||||||
err2=$(cat "$WORK_DIR/stderr-2.tmp")
|
|
||||||
assert_not_contains "live socket must not refuse broker-absent" "$err2" "broker-absent"
|
|
||||||
if [[ "$rc" == "75" ]]; then
|
|
||||||
echo "FAIL: live socket — preflight still refused (exit 75) with a live socket" >&2
|
|
||||||
fail=1
|
|
||||||
fi
|
|
||||||
|
|
||||||
# ─── 3. Explicit socket env wins over XDG default. ─────────────────────────
|
|
||||||
set +e
|
|
||||||
out=$(run_session_script "" XDG_RUNTIME_DIR="$SOCK_DIR/no-runtime-here" MOSAIC_LEASE_BROKER_SOCKET="$SOCK_DIR/absent2.sock" 2>"$WORK_DIR/stderr-3.tmp")
|
|
||||||
rc=$?
|
|
||||||
set -e
|
|
||||||
assert "explicit env wins (exit 75)" "75" "$rc"
|
|
||||||
assert_contains "explicit env path named" "$(cat "$WORK_DIR/stderr-3.tmp")" "$SOCK_DIR/absent2.sock"
|
|
||||||
|
|
||||||
# ─── 4. --stop is not fenced on the broker. ────────────────────────────────
|
|
||||||
: > "$LOG_FILE"
|
|
||||||
set +e
|
|
||||||
out=$(run_session_script "--stop" MOSAIC_LEASE_BROKER_SOCKET="$SOCK_DIR/absent3.sock" 2>"$WORK_DIR/stderr-4.tmp")
|
|
||||||
rc=$?
|
|
||||||
set -e
|
|
||||||
err4=$(cat "$WORK_DIR/stderr-4.tmp")
|
|
||||||
assert_not_contains "--stop must not refuse broker-absent" "$err4" "broker-absent"
|
|
||||||
if [[ "$rc" == "75" ]]; then
|
|
||||||
echo "FAIL: --stop — exit 75 means teardown was fenced on the broker" >&2
|
|
||||||
fail=1
|
|
||||||
fi
|
|
||||||
|
|
||||||
kill "$HOLDER_PID" 2>/dev/null || true
|
|
||||||
|
|
||||||
if [[ "$fail" -eq 0 ]]; then
|
|
||||||
echo "start-agent-session lease-broker preflight regression passed"
|
|
||||||
fi
|
|
||||||
exit "$fail"
|
|
||||||
@@ -102,17 +102,9 @@ gitea_resolve_api_for_login() {
|
|||||||
return 1
|
return 1
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
# NO --login: the acting credential is this identity's own token and there
|
GITEA_API_TOKEN=$(get_gitea_token_for_login "$effective_login" "$host") \
|
||||||
# is deliberately no tea-config fallback. get_gitea_token_for_login matches
|
|| GITEA_API_TOKEN=$(get_gitea_token "$host") || {
|
||||||
# by login NAME and performs no authentication check, and with no --login
|
echo "Error: Gitea token not found for login '$effective_login' (comment write/read-back)" >&2
|
||||||
# that name was a HOST GUESS resolving to a shared account. A live shared
|
|
||||||
# token would therefore have authored every seat's comment as that
|
|
||||||
# account, making Gate-16 author-is-not-reviewer unenforceable fleet-wide;
|
|
||||||
# a dead one is only what made the defect visible. get_gitea_token fails
|
|
||||||
# loud on a fleet host when no identity resolves, and that refusal is the
|
|
||||||
# correct outcome, not a case to fall back from.
|
|
||||||
GITEA_API_TOKEN=$(get_gitea_token "$host") || {
|
|
||||||
echo "Error: no Gitea credential resolved for the acting identity on host '$host' (comment write/read-back). Set MOSAIC_GIT_IDENTITY=<agent-id>, or pass --login <name> to use a named tea credential." >&2
|
|
||||||
return 1
|
return 1
|
||||||
}
|
}
|
||||||
fi
|
fi
|
||||||
@@ -343,12 +335,15 @@ if [[ "$PLATFORM" == "github" ]]; then
|
|||||||
gh issue comment "$ISSUE_NUMBER" --body "$COMMENT"
|
gh issue comment "$ISSUE_NUMBER" --body "$COMMENT"
|
||||||
echo "Added comment to GitHub issue #$ISSUE_NUMBER"
|
echo "Added comment to GitHub issue #$ISSUE_NUMBER"
|
||||||
elif [[ "$PLATFORM" == "gitea" ]]; then
|
elif [[ "$PLATFORM" == "gitea" ]]; then
|
||||||
# A --login override selects a NAMED tea credential and is the only way to
|
# Resolve the login this comment should be attributed to: the --login
|
||||||
# reach the tea store. With no --login there is deliberately no guess: the
|
# override when given, otherwise the detected default for this repo's host.
|
||||||
# comment is attributed to this identity's own credential, resolved by
|
# A --login override always wins. Otherwise name this repo host's login only
|
||||||
# gitea_resolve_api_for_login. The guess this replaced named a SHARED
|
# as a best effort: the login name merely selects a per-login token, and
|
||||||
# account, selecting an identity the caller never asked to act as.
|
# gitea_resolve_api_for_login falls back to the host credential
|
||||||
|
# (get_gitea_token) when no tea login is named, so the default credential
|
||||||
|
# still resolves even when the host tea has no matching login entry.
|
||||||
EFFECTIVE_LOGIN="$LOGIN_OVERRIDE"
|
EFFECTIVE_LOGIN="$LOGIN_OVERRIDE"
|
||||||
|
[[ -n "$EFFECTIVE_LOGIN" ]] || EFFECTIVE_LOGIN=$(get_gitea_login 2>/dev/null || true)
|
||||||
|
|
||||||
# Bind the REST endpoint + token to the effective login, then derive the
|
# Bind the REST endpoint + token to the effective login, then derive the
|
||||||
# acting identity from that SAME credential (GET /user). The write below and
|
# acting identity from that SAME credential (GET /user). The write below and
|
||||||
|
|||||||
@@ -12,7 +12,7 @@
|
|||||||
# concurrent record cannot masquerade as this write and a no-op fails closed.
|
# concurrent record cannot masquerade as this write and a no-op fails closed.
|
||||||
#
|
#
|
||||||
# --login override: the default login is resolved from the local tea login list
|
# --login override: the default login is resolved from the local tea login list
|
||||||
# for this repo's host from the acting identity's own credential. Pass --login <name> to
|
# for this repo's host (get_gitea_login_for_host). Pass --login <name> to
|
||||||
# override it for this invocation only. The REST write, the /user identity read,
|
# override it for this invocation only. The REST write, the /user identity read,
|
||||||
# and every read-back are ALL performed with the token of the EFFECTIVE login,
|
# and every read-back are ALL performed with the token of the EFFECTIVE login,
|
||||||
# so the write and its verification bind to the same identity.
|
# so the write and its verification bind to the same identity.
|
||||||
@@ -372,17 +372,9 @@ gitea_resolve_api_for_login() {
|
|||||||
return 1
|
return 1
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
# NO --login: the acting credential is this identity's own token and there
|
GITEA_API_TOKEN=$(get_gitea_token_for_login "$effective_login" "$host") \
|
||||||
# is deliberately no tea-config fallback. get_gitea_token_for_login matches
|
|| GITEA_API_TOKEN=$(get_gitea_token "$host") || {
|
||||||
# by login NAME and performs no authentication check, and with no --login
|
echo "Error: Gitea token not found for login '$effective_login' (review write/read-back)" >&2
|
||||||
# that name was a HOST GUESS resolving to a shared account. A live shared
|
|
||||||
# token would therefore have authored every seat's review as that
|
|
||||||
# account, making Gate-16 author-is-not-reviewer unenforceable fleet-wide;
|
|
||||||
# a dead one is only what made the defect visible. get_gitea_token fails
|
|
||||||
# loud on a fleet host when no identity resolves, and that refusal is the
|
|
||||||
# correct outcome, not a case to fall back from.
|
|
||||||
GITEA_API_TOKEN=$(get_gitea_token "$host") || {
|
|
||||||
echo "Error: no Gitea credential resolved for the acting identity on host '$host' (review write/read-back). Set MOSAIC_GIT_IDENTITY=<agent-id>, or pass --login <name> to use a named tea credential." >&2
|
|
||||||
return 1
|
return 1
|
||||||
}
|
}
|
||||||
fi
|
fi
|
||||||
@@ -706,7 +698,7 @@ if [[ "$PLATFORM" == "github" ]]; then
|
|||||||
elif [[ "$PLATFORM" == "gitea" ]]; then
|
elif [[ "$PLATFORM" == "gitea" ]]; then
|
||||||
case $ACTION in
|
case $ACTION in
|
||||||
approve)
|
approve)
|
||||||
# Best-effort host for credential resolution only (gitea_resolve_api_for_login
|
# Best-effort host for the tea-login GUESS only (gitea_resolve_api_for_login
|
||||||
# below re-derives the real host from HOST_OVERRIDE/remote independently and
|
# below re-derives the real host from HOST_OVERRIDE/remote independently and
|
||||||
# is authoritative). Prefer an explicit -H/--host; otherwise best-effort
|
# is authoritative). Prefer an explicit -H/--host; otherwise best-effort
|
||||||
# git-remote inference, tolerating its ABSENCE (a bare `get_remote_host` here
|
# git-remote inference, tolerating its ABSENCE (a bare `get_remote_host` here
|
||||||
@@ -714,13 +706,15 @@ elif [[ "$PLATFORM" == "gitea" ]]; then
|
|||||||
# SILENTLY — exit 1, zero output — even though -r/-H are exactly the flags
|
# SILENTLY — exit 1, zero output — even though -r/-H are exactly the flags
|
||||||
# that support running with no usable origin at all).
|
# that support running with no usable origin at all).
|
||||||
host="${HOST_OVERRIDE:-$(get_remote_host 2>/dev/null || true)}"
|
host="${HOST_OVERRIDE:-$(get_remote_host 2>/dev/null || true)}"
|
||||||
# A --login override selects a NAMED tea credential and is the only
|
# A --login override always wins. Otherwise name this host's login
|
||||||
# way to reach the tea store. With no --login there is deliberately no
|
# only as a best effort: the login name merely selects a per-login
|
||||||
# guess: gitea_resolve_api_for_login resolves this identity's own token.
|
# token, and gitea_resolve_api_for_login falls back to the host
|
||||||
# The guess this replaced named a SHARED account, selecting an identity
|
# credential (get_gitea_token) when no tea login is named — so a host
|
||||||
# the caller never asked to act as. The single resolved token is then
|
# tea's login list need not enumerate exotic (e.g. ported) hosts for
|
||||||
# used for the write, the /user identity, and the read-back.
|
# the default credential to resolve. The single resolved token is
|
||||||
|
# then used for the write, the /user identity, and the read-back.
|
||||||
EFFECTIVE_LOGIN="$LOGIN_OVERRIDE"
|
EFFECTIVE_LOGIN="$LOGIN_OVERRIDE"
|
||||||
|
[[ -n "$EFFECTIVE_LOGIN" ]] || EFFECTIVE_LOGIN=$(get_gitea_login_for_host "$host" 2>/dev/null || true)
|
||||||
# Bind the REST endpoint + token to the effective login, then derive
|
# Bind the REST endpoint + token to the effective login, then derive
|
||||||
# the acting identity from that SAME credential so the review submit
|
# the acting identity from that SAME credential so the review submit
|
||||||
# and its read-back verify against the identity that performed them.
|
# and its read-back verify against the identity that performed them.
|
||||||
@@ -741,7 +735,7 @@ elif [[ "$PLATFORM" == "gitea" ]]; then
|
|||||||
echo "Error: Comment required for request-changes"
|
echo "Error: Comment required for request-changes"
|
||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
# Best-effort host for credential resolution only (gitea_resolve_api_for_login
|
# Best-effort host for the tea-login GUESS only (gitea_resolve_api_for_login
|
||||||
# below re-derives the real host from HOST_OVERRIDE/remote independently and
|
# below re-derives the real host from HOST_OVERRIDE/remote independently and
|
||||||
# is authoritative). Prefer an explicit -H/--host; otherwise best-effort
|
# is authoritative). Prefer an explicit -H/--host; otherwise best-effort
|
||||||
# git-remote inference, tolerating its ABSENCE (a bare `get_remote_host` here
|
# git-remote inference, tolerating its ABSENCE (a bare `get_remote_host` here
|
||||||
@@ -749,13 +743,15 @@ elif [[ "$PLATFORM" == "gitea" ]]; then
|
|||||||
# SILENTLY — exit 1, zero output — even though -r/-H are exactly the flags
|
# SILENTLY — exit 1, zero output — even though -r/-H are exactly the flags
|
||||||
# that support running with no usable origin at all).
|
# that support running with no usable origin at all).
|
||||||
host="${HOST_OVERRIDE:-$(get_remote_host 2>/dev/null || true)}"
|
host="${HOST_OVERRIDE:-$(get_remote_host 2>/dev/null || true)}"
|
||||||
# A --login override selects a NAMED tea credential and is the only
|
# A --login override always wins. Otherwise name this host's login
|
||||||
# way to reach the tea store. With no --login there is deliberately no
|
# only as a best effort: the login name merely selects a per-login
|
||||||
# guess: gitea_resolve_api_for_login resolves this identity's own token.
|
# token, and gitea_resolve_api_for_login falls back to the host
|
||||||
# The guess this replaced named a SHARED account, selecting an identity
|
# credential (get_gitea_token) when no tea login is named — so a host
|
||||||
# the caller never asked to act as. The single resolved token is then
|
# tea's login list need not enumerate exotic (e.g. ported) hosts for
|
||||||
# used for the write, the /user identity, and the read-back.
|
# the default credential to resolve. The single resolved token is
|
||||||
|
# then used for the write, the /user identity, and the read-back.
|
||||||
EFFECTIVE_LOGIN="$LOGIN_OVERRIDE"
|
EFFECTIVE_LOGIN="$LOGIN_OVERRIDE"
|
||||||
|
[[ -n "$EFFECTIVE_LOGIN" ]] || EFFECTIVE_LOGIN=$(get_gitea_login_for_host "$host" 2>/dev/null || true)
|
||||||
gitea_resolve_api_for_login "$EFFECTIVE_LOGIN" "${LOGIN_OVERRIDE:+explicit}" || exit 1
|
gitea_resolve_api_for_login "$EFFECTIVE_LOGIN" "${LOGIN_OVERRIDE:+explicit}" || exit 1
|
||||||
ACTING_LOGIN=$(gitea_authenticated_login) || exit 1
|
ACTING_LOGIN=$(gitea_authenticated_login) || exit 1
|
||||||
head_sha=$(gitea_pr_head_sha "$PR_NUMBER") || exit 1
|
head_sha=$(gitea_pr_head_sha "$PR_NUMBER") || exit 1
|
||||||
@@ -770,7 +766,7 @@ elif [[ "$PLATFORM" == "gitea" ]]; then
|
|||||||
echo "Error: Comment required"
|
echo "Error: Comment required"
|
||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
# Best-effort host for credential resolution only (gitea_resolve_api_for_login
|
# Best-effort host for the tea-login GUESS only (gitea_resolve_api_for_login
|
||||||
# below re-derives the real host from HOST_OVERRIDE/remote independently and
|
# below re-derives the real host from HOST_OVERRIDE/remote independently and
|
||||||
# is authoritative). Prefer an explicit -H/--host; otherwise best-effort
|
# is authoritative). Prefer an explicit -H/--host; otherwise best-effort
|
||||||
# git-remote inference, tolerating its ABSENCE (a bare `get_remote_host` here
|
# git-remote inference, tolerating its ABSENCE (a bare `get_remote_host` here
|
||||||
@@ -778,13 +774,15 @@ elif [[ "$PLATFORM" == "gitea" ]]; then
|
|||||||
# SILENTLY — exit 1, zero output — even though -r/-H are exactly the flags
|
# SILENTLY — exit 1, zero output — even though -r/-H are exactly the flags
|
||||||
# that support running with no usable origin at all).
|
# that support running with no usable origin at all).
|
||||||
host="${HOST_OVERRIDE:-$(get_remote_host 2>/dev/null || true)}"
|
host="${HOST_OVERRIDE:-$(get_remote_host 2>/dev/null || true)}"
|
||||||
# A --login override selects a NAMED tea credential and is the only
|
# A --login override always wins. Otherwise name this host's login
|
||||||
# way to reach the tea store. With no --login there is deliberately no
|
# only as a best effort: the login name merely selects a per-login
|
||||||
# guess: gitea_resolve_api_for_login resolves this identity's own token.
|
# token, and gitea_resolve_api_for_login falls back to the host
|
||||||
# The guess this replaced named a SHARED account, selecting an identity
|
# credential (get_gitea_token) when no tea login is named — so a host
|
||||||
# the caller never asked to act as. The single resolved token is then
|
# tea's login list need not enumerate exotic (e.g. ported) hosts for
|
||||||
# used for the write, the /user identity, and the read-back.
|
# the default credential to resolve. The single resolved token is
|
||||||
|
# then used for the write, the /user identity, and the read-back.
|
||||||
EFFECTIVE_LOGIN="$LOGIN_OVERRIDE"
|
EFFECTIVE_LOGIN="$LOGIN_OVERRIDE"
|
||||||
|
[[ -n "$EFFECTIVE_LOGIN" ]] || EFFECTIVE_LOGIN=$(get_gitea_login_for_host "$host" 2>/dev/null || true)
|
||||||
gitea_resolve_api_for_login "$EFFECTIVE_LOGIN" "${LOGIN_OVERRIDE:+explicit}" || exit 1
|
gitea_resolve_api_for_login "$EFFECTIVE_LOGIN" "${LOGIN_OVERRIDE:+explicit}" || exit 1
|
||||||
ACTING_LOGIN=$(gitea_authenticated_login) || exit 1
|
ACTING_LOGIN=$(gitea_authenticated_login) || exit 1
|
||||||
comment_id=$(gitea_create_comment_verified "$PR_NUMBER" "$COMMENT" "$ACTING_LOGIN") || {
|
comment_id=$(gitea_create_comment_verified "$PR_NUMBER" "$COMMENT" "$ACTING_LOGIN") || {
|
||||||
|
|||||||
@@ -54,22 +54,6 @@ def main(
|
|||||||
arguments = parser.parse_args(argv)
|
arguments = parser.parse_args(argv)
|
||||||
source_environment = os.environ if environ is None else environ
|
source_environment = os.environ if environ is None else environ
|
||||||
|
|
||||||
# D29: a session that never held a lease has nothing to revoke, and that is a
|
|
||||||
# SUCCESS, not a failed revocation. The block below is deliberately fail-closed
|
|
||||||
# for a broker that is unreachable, which is right — but it cannot distinguish
|
|
||||||
# "the broker is down" from "there was never a lease", so a bare-launched
|
|
||||||
# session was denied every lifecycle transition, including compaction. Denying
|
|
||||||
# compaction protects nothing there; it converts a recoverable context limit
|
|
||||||
# into a lost session.
|
|
||||||
#
|
|
||||||
# Absence must be TOTAL to qualify. If exactly one variable is present the
|
|
||||||
# session is half-provisioned, which is real misconfiguration, and it still
|
|
||||||
# takes the fail-closed path below.
|
|
||||||
lease_variables = ("MOSAIC_LEASE_BROKER_SOCKET", "MOSAIC_LEASE_SESSION_ID")
|
|
||||||
present = [name for name in lease_variables if source_environment.get(name)]
|
|
||||||
if not present:
|
|
||||||
return 0
|
|
||||||
|
|
||||||
try:
|
try:
|
||||||
if not arguments.reason or len(arguments.reason) > 128:
|
if not arguments.reason or len(arguments.reason) > 128:
|
||||||
raise ValueError("invalid revoke reason")
|
raise ValueError("invalid revoke reason")
|
||||||
|
|||||||
@@ -25,7 +25,7 @@
|
|||||||
"lint": "eslint src",
|
"lint": "eslint src",
|
||||||
"typecheck": "tsc --noEmit",
|
"typecheck": "tsc --noEmit",
|
||||||
"test": "vitest run --passWithNoTests && pnpm run test:framework-shell",
|
"test": "vitest run --passWithNoTests && pnpm run test:framework-shell",
|
||||||
"test:framework-shell": "bash framework/tools/quality/scripts/check-test-enumeration.sh && bash framework/tools/quality/scripts/test-check-test-enumeration.sh && python3 framework/tools/quality/scripts/test-framework-drift-check.py && bash framework/tools/quality/scripts/test-framework-drift-doctor.sh && bash framework/systemd/user/test-fleet-units.sh && python3 src/lease-broker/daemon_deadline_unittest.py && python3 src/lease-broker/normative_fragments_unittest.py && python3 src/lease-broker/promotion_binding_unittest.py && python3 src/lease-broker/promotion_trigger_unittest.py && python3 src/lease-broker/receipt_challenge_unittest.py && python3 src/lease-broker/context_recovery_unittest.py && python3 src/lease-broker/recovery_runtime_unittest.py && python3 src/lease-broker/recovery_b1_adversarial_unittest.py && python3 src/lease-broker/receipt_observer_client_unittest.py && python3 src/lease-broker/invariant_r_unittest.py && python3 src/lease-broker/framework_skill_portability_unittest.py && python3 src/lease-broker/revoke_noop_unittest.py && python3 src/mutator-gate/runtime_tools_unittest.py && python3 src/mutator-gate/runtime_launch_guard_unittest.py && python3 src/mutator-gate/version_coupling_unittest.py && python3 framework/tools/lease-broker/check-runtime-launches.py --root ../.. && bash framework/tools/codex/test-pr-diff-context.sh && bash framework/tools/qa/test-deps-preflight.sh && bash framework/tools/git/test-pr-edit.sh && bash framework/tools/git/test-pr-review-gitea-comment.sh && bash framework/tools/git/test-pr-review-repo-host-override.sh && bash framework/tools/git/test-ci-queue-wait-no-status.sh && bash framework/tools/git/test-ci-queue-wait-branch-absent.sh && bash framework/tools/git/test-ci-queue-wait-tristate.sh && bash framework/tools/git/test-ci-queue-wait-github-checks.sh && bash framework/tools/git/test-pr-merge-queue-branch.sh && bash framework/tools/git/test-pr-merge-fork-ci-status.sh && bash framework/tools/git/test-pr-merge-head-pin.sh && bash framework/tools/git/test-pr-merge-message-field.sh && bash framework/tools/git/test-git-credential-mosaic.sh && bash framework/tools/git/test-gitea-token-identity.sh && bash framework/tools/git/test-explain-diagnostic-status-neutral.sh && bash framework/tools/git/test-detect-platform-outside-repo.sh && bash framework/tools/woodpecker/test-terminal-green-contract.sh && bash framework/tools/_scripts/test-install-ordering-guard.sh && bash framework/tools/_scripts/test-mosaic-init-rce.sh && bash framework/tools/tmux/agent-send.test.sh && bash framework/tools/wake/test-wake-store-ack.sh && bash framework/tools/wake/test-wake-store-enqueue-race.sh && bash framework/tools/wake/test-wake-digest-hmac.sh && bash framework/tools/wake/test-wake-digest-quarantine.sh && bash framework/tools/wake/test-wake-detector.sh && bash framework/tools/wake/test-wake-fn-oracle.sh && bash framework/tools/wake/test-wake-reconcile.sh && bash framework/tools/wake/test-wake-beacon.sh && bash framework/tools/wake/test-wake-preimage.sh && bash framework/tools/wake/test-wake-install.sh && bash framework/tools/glpi/test-list-http-status.sh && bash framework/tools/orchestrator/test-board-roll.sh && bash framework/tools/woodpecker/test-ci-wait-exit-matrix.sh && bash framework/tools/_scripts/test-fleet-transport-check.sh && bash framework/tools/_scripts/test-brain-home-check.sh && bash framework/tools/fleet/test-agent-session-broker-preflight.sh"
|
"test:framework-shell": "bash framework/tools/quality/scripts/check-test-enumeration.sh && bash framework/tools/quality/scripts/test-check-test-enumeration.sh && python3 framework/tools/quality/scripts/test-framework-drift-check.py && bash framework/tools/quality/scripts/test-framework-drift-doctor.sh && bash framework/systemd/user/test-fleet-units.sh && python3 src/lease-broker/daemon_deadline_unittest.py && python3 src/lease-broker/normative_fragments_unittest.py && python3 src/lease-broker/promotion_binding_unittest.py && python3 src/lease-broker/promotion_trigger_unittest.py && python3 src/lease-broker/receipt_challenge_unittest.py && python3 src/lease-broker/context_recovery_unittest.py && python3 src/lease-broker/recovery_runtime_unittest.py && python3 src/lease-broker/recovery_b1_adversarial_unittest.py && python3 src/lease-broker/receipt_observer_client_unittest.py && python3 src/lease-broker/invariant_r_unittest.py && python3 src/lease-broker/framework_skill_portability_unittest.py && python3 src/mutator-gate/runtime_tools_unittest.py && python3 src/mutator-gate/runtime_launch_guard_unittest.py && python3 src/mutator-gate/version_coupling_unittest.py && python3 framework/tools/lease-broker/check-runtime-launches.py --root ../.. && bash framework/tools/codex/test-pr-diff-context.sh && bash framework/tools/qa/test-deps-preflight.sh && bash framework/tools/git/test-pr-edit.sh && bash framework/tools/git/test-pr-review-gitea-comment.sh && bash framework/tools/git/test-pr-review-repo-host-override.sh && bash framework/tools/git/test-ci-queue-wait-no-status.sh && bash framework/tools/git/test-ci-queue-wait-branch-absent.sh && bash framework/tools/git/test-ci-queue-wait-tristate.sh && bash framework/tools/git/test-ci-queue-wait-github-checks.sh && bash framework/tools/git/test-pr-merge-queue-branch.sh && bash framework/tools/git/test-pr-merge-fork-ci-status.sh && bash framework/tools/git/test-pr-merge-head-pin.sh && bash framework/tools/git/test-pr-merge-message-field.sh && bash framework/tools/git/test-git-credential-mosaic.sh && bash framework/tools/git/test-gitea-token-identity.sh && bash framework/tools/git/test-explain-diagnostic-status-neutral.sh && bash framework/tools/git/test-detect-platform-outside-repo.sh && bash framework/tools/woodpecker/test-terminal-green-contract.sh && bash framework/tools/_scripts/test-install-ordering-guard.sh && bash framework/tools/_scripts/test-mosaic-init-rce.sh && bash framework/tools/tmux/agent-send.test.sh && bash framework/tools/wake/test-wake-store-ack.sh && bash framework/tools/wake/test-wake-store-enqueue-race.sh && bash framework/tools/wake/test-wake-digest-hmac.sh && bash framework/tools/wake/test-wake-digest-quarantine.sh && bash framework/tools/wake/test-wake-detector.sh && bash framework/tools/wake/test-wake-fn-oracle.sh && bash framework/tools/wake/test-wake-reconcile.sh && bash framework/tools/wake/test-wake-beacon.sh && bash framework/tools/wake/test-wake-preimage.sh && bash framework/tools/wake/test-wake-install.sh && bash framework/tools/glpi/test-list-http-status.sh && bash framework/tools/orchestrator/test-board-roll.sh && bash framework/tools/woodpecker/test-ci-wait-exit-matrix.sh && bash framework/tools/_scripts/test-fleet-transport-check.sh && bash framework/tools/_scripts/test-brain-home-check.sh"
|
||||||
},
|
},
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"@mosaicstack/brain": "workspace:*",
|
"@mosaicstack/brain": "workspace:*",
|
||||||
|
|||||||
@@ -147,11 +147,11 @@ describe('renderNorthStarMarkdown', () => {
|
|||||||
'standing_objectives:',
|
'standing_objectives:',
|
||||||
' - { id: NS-1, text: t }',
|
' - { id: NS-1, text: t }',
|
||||||
'success_criteria:',
|
'success_criteria:',
|
||||||
' - { id: AC-NS-1, text: t, tier: 0 }',
|
' - { id: AC-NS-1, text: t }',
|
||||||
'workstreams:',
|
'workstreams:',
|
||||||
' - { id: A, title: t }',
|
' - { id: A, title: t }',
|
||||||
'goals:',
|
'goals:',
|
||||||
' - { id: A1, title: t, phase: 1, tier: 0, priority: must-have, depends_on: [] }',
|
' - { id: A1, title: t, phase: 1, priority: must-have, depends_on: [] }',
|
||||||
'assumptions:',
|
'assumptions:',
|
||||||
' - { id: ASM-1, vetoable: true, text: t }',
|
' - { id: ASM-1, vetoable: true, text: t }',
|
||||||
'spend:',
|
'spend:',
|
||||||
|
|||||||
@@ -1,177 +0,0 @@
|
|||||||
import { lstat, mkdir, mkdtemp, readFile, rm, symlink, writeFile } from 'node:fs/promises';
|
|
||||||
import { tmpdir } from 'node:os';
|
|
||||||
import { join } from 'node:path';
|
|
||||||
import { afterEach, describe, expect, it } from 'vitest';
|
|
||||||
|
|
||||||
import { placeUnitFile, resolveLeaseBrokerSocketForPreflight } from './fleet.js';
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Unit-placement regression harness for #1292.
|
|
||||||
*
|
|
||||||
* The two measured defects this suite pins:
|
|
||||||
* 1. `systemctl enable <name>` does NOT rewrite an existing by-path
|
|
||||||
* wants-symlink — so placement must remove stale residue explicitly, and
|
|
||||||
* acceptance asserts on the RESULTING SYMLINK TARGET, never on the enable
|
|
||||||
* call's argument (asserting the call cannot see where the link ended up).
|
|
||||||
* 2. Node's copyFile FOLLOWS a by-path symlink at the destination and
|
|
||||||
* overwrites the SEED template. Acceptance asserts on the SEED's bytes
|
|
||||||
* AND mtime — unchanged — which is the only check that can redden for
|
|
||||||
* finding 2. The symlink-target assertion catches finding 1; these are
|
|
||||||
* different defects with different failure modes.
|
|
||||||
*
|
|
||||||
* Fixtures are entirely inside tmpdirs (source template, active systemd dir,
|
|
||||||
* wants dir) — no real host paths are touched by this suite.
|
|
||||||
*/
|
|
||||||
|
|
||||||
describe('placeUnitFile (#1292 unit placement)', () => {
|
|
||||||
const cleanup: string[] = [];
|
|
||||||
afterEach(async () => {
|
|
||||||
while (cleanup.length > 0) {
|
|
||||||
await rm(cleanup.pop()!, { recursive: true, force: true });
|
|
||||||
}
|
|
||||||
});
|
|
||||||
|
|
||||||
async function fixture() {
|
|
||||||
const root = await mkdtemp(join(tmpdir(), 'place-unit-'));
|
|
||||||
cleanup.push(root);
|
|
||||||
const seedDir = join(root, 'seed');
|
|
||||||
const activeDir = join(root, 'active');
|
|
||||||
await mkdir(seedDir, { recursive: true });
|
|
||||||
await mkdir(activeDir, { recursive: true });
|
|
||||||
const seedTemplate = join(seedDir, 'unit-under-test.service');
|
|
||||||
await writeFile(
|
|
||||||
seedTemplate,
|
|
||||||
'[Unit]\nDescription=seed template\n[Service]\nType=oneshot\nExecStart=/bin/true\n[Install]\nWantedBy=default.target\n',
|
|
||||||
);
|
|
||||||
const activeSource = join(root, 'active-source.service');
|
|
||||||
await writeFile(
|
|
||||||
activeSource,
|
|
||||||
'[Unit]\nDescription=active copy v2\n[Service]\nType=oneshot\nExecStart=/bin/true\n[Install]\nWantedBy=default.target\n',
|
|
||||||
);
|
|
||||||
return { root, seedDir, activeDir, seedTemplate, activeSource };
|
|
||||||
}
|
|
||||||
|
|
||||||
it('places a regular file on a clean host (negative control: no residue anywhere)', async () => {
|
|
||||||
const f = await fixture();
|
|
||||||
const result = await placeUnitFile(f.activeSource, f.activeDir, 'unit-under-test.service');
|
|
||||||
expect(result.unlinkedDestinationSymlink).toBe(false);
|
|
||||||
expect(result.removedStaleWantsSymlink).toBe(false);
|
|
||||||
const info = await lstat(join(f.activeDir, 'unit-under-test.service'));
|
|
||||||
expect(info.isSymbolicLink()).toBe(false);
|
|
||||||
expect(await readFile(join(f.activeDir, 'unit-under-test.service'), 'utf8')).toContain(
|
|
||||||
'active copy v2',
|
|
||||||
);
|
|
||||||
// Seed untouched by construction — but assert it, so the clean-host case
|
|
||||||
// cannot silently regress into seed-mutation.
|
|
||||||
expect(await readFile(f.seedTemplate, 'utf8')).toContain('seed template');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('by-path residue: unlinks destination symlink, places the file, seed bytes AND mtime unchanged (finding 2)', async () => {
|
|
||||||
const f = await fixture();
|
|
||||||
const seedBefore = await readFile(f.seedTemplate, 'utf8');
|
|
||||||
const mtimeBefore = (await lstat(f.seedTemplate)).mtimeMs;
|
|
||||||
// The fomo-lin convention: by-path enable left a symlink AT the unit name
|
|
||||||
// pointing at the seed template, plus a wants-symlink doing the same.
|
|
||||||
await symlink(f.seedTemplate, join(f.activeDir, 'unit-under-test.service'));
|
|
||||||
const wantsDir = join(f.activeDir, 'default.target.wants');
|
|
||||||
await mkdir(wantsDir, { recursive: true });
|
|
||||||
await symlink(f.seedTemplate, join(wantsDir, 'unit-under-test.service'));
|
|
||||||
|
|
||||||
const result = await placeUnitFile(f.activeSource, f.activeDir, 'unit-under-test.service');
|
|
||||||
expect(result.unlinkedDestinationSymlink).toBe(true);
|
|
||||||
expect(result.removedStaleWantsSymlink).toBe(true);
|
|
||||||
|
|
||||||
// FINDING 2's check: the seed is byte-identical and its mtime did not move.
|
|
||||||
expect(await readFile(f.seedTemplate, 'utf8')).toBe(seedBefore);
|
|
||||||
expect((await lstat(f.seedTemplate)).mtimeMs).toBe(mtimeBefore);
|
|
||||||
|
|
||||||
// The destination is now a regular file carrying the ACTIVE content.
|
|
||||||
const destInfo = await lstat(join(f.activeDir, 'unit-under-test.service'));
|
|
||||||
expect(destInfo.isSymbolicLink()).toBe(false);
|
|
||||||
expect(await readFile(join(f.activeDir, 'unit-under-test.service'), 'utf8')).toContain(
|
|
||||||
'active copy v2',
|
|
||||||
);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('by-path residue: no wants-symlink remains pointing at the seed (finding 1 residue cleared)', async () => {
|
|
||||||
const f = await fixture();
|
|
||||||
await symlink(f.seedTemplate, join(f.activeDir, 'unit-under-test.service'));
|
|
||||||
const wantsDir = join(f.activeDir, 'default.target.wants');
|
|
||||||
await mkdir(wantsDir, { recursive: true });
|
|
||||||
await symlink(f.seedTemplate, join(wantsDir, 'unit-under-test.service'));
|
|
||||||
|
|
||||||
await placeUnitFile(f.activeSource, f.activeDir, 'unit-under-test.service');
|
|
||||||
|
|
||||||
// After placement the stale wants link is GONE (enable-by-name recreates
|
|
||||||
// it correctly). A link still present must not point at the seed.
|
|
||||||
try {
|
|
||||||
const link = await lstat(join(wantsDir, 'unit-under-test.service'));
|
|
||||||
if (link.isSymbolicLink()) {
|
|
||||||
const target = await readFile(join(wantsDir, 'unit-under-test.service'), 'utf8').catch(
|
|
||||||
async () => '',
|
|
||||||
);
|
|
||||||
expect(target).not.toContain('seed template');
|
|
||||||
}
|
|
||||||
} catch {
|
|
||||||
// absent wants link — the expected post-placement state
|
|
||||||
}
|
|
||||||
});
|
|
||||||
|
|
||||||
it('idempotence: second placement on a reconciled host is a no-op producing the identical final state', async () => {
|
|
||||||
const f = await fixture();
|
|
||||||
// Reconciled starting state: regular file at the name, wants link to the active copy.
|
|
||||||
await writeFile(
|
|
||||||
join(f.activeDir, 'unit-under-test.service'),
|
|
||||||
await readFile(f.activeSource, 'utf8'),
|
|
||||||
);
|
|
||||||
const wantsDir = join(f.activeDir, 'default.target.wants');
|
|
||||||
await mkdir(wantsDir, { recursive: true });
|
|
||||||
await symlink(
|
|
||||||
join(f.activeDir, 'unit-under-test.service'),
|
|
||||||
join(wantsDir, 'unit-under-test.service'),
|
|
||||||
);
|
|
||||||
const before = await readFile(join(f.activeDir, 'unit-under-test.service'), 'utf8');
|
|
||||||
|
|
||||||
const result = await placeUnitFile(f.activeSource, f.activeDir, 'unit-under-test.service');
|
|
||||||
// No destructive step fired: no unlink, no wants removal.
|
|
||||||
expect(result.unlinkedDestinationSymlink).toBe(false);
|
|
||||||
expect(result.removedStaleWantsSymlink).toBe(false);
|
|
||||||
// Identical final state.
|
|
||||||
expect(await readFile(join(f.activeDir, 'unit-under-test.service'), 'utf8')).toBe(before);
|
|
||||||
const link = await lstat(join(wantsDir, 'unit-under-test.service'));
|
|
||||||
expect(link.isSymbolicLink()).toBe(true);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('double install on by-path residue converges to the identical reconciled state', async () => {
|
|
||||||
const f = await fixture();
|
|
||||||
await symlink(f.seedTemplate, join(f.activeDir, 'unit-under-test.service'));
|
|
||||||
const wantsDir = join(f.activeDir, 'default.target.wants');
|
|
||||||
await mkdir(wantsDir, { recursive: true });
|
|
||||||
await symlink(f.seedTemplate, join(wantsDir, 'unit-under-test.service'));
|
|
||||||
|
|
||||||
await placeUnitFile(f.activeSource, f.activeDir, 'unit-under-test.service');
|
|
||||||
const first = await readFile(join(f.activeDir, 'unit-under-test.service'), 'utf8');
|
|
||||||
const secondRun = await placeUnitFile(f.activeSource, f.activeDir, 'unit-under-test.service');
|
|
||||||
const second = await readFile(join(f.activeDir, 'unit-under-test.service'), 'utf8');
|
|
||||||
expect(secondRun.unlinkedDestinationSymlink).toBe(false);
|
|
||||||
expect(second).toBe(first);
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('resolveLeaseBrokerSocketForPreflight (#1292 preflight resolution)', () => {
|
|
||||||
it('explicit MOSAIC_LEASE_BROKER_SOCKET wins', () => {
|
|
||||||
expect(
|
|
||||||
resolveLeaseBrokerSocketForPreflight({ MOSAIC_LEASE_BROKER_SOCKET: '/custom/sock' }, 1000),
|
|
||||||
).toBe('/custom/sock');
|
|
||||||
});
|
|
||||||
it('XDG_RUNTIME_DIR next', () => {
|
|
||||||
expect(resolveLeaseBrokerSocketForPreflight({ XDG_RUNTIME_DIR: '/run/user/1001' }, 1000)).toBe(
|
|
||||||
'/run/user/1001/mosaic-lease/broker.sock',
|
|
||||||
);
|
|
||||||
});
|
|
||||||
it('falls back to /run/user/<uid>', () => {
|
|
||||||
expect(resolveLeaseBrokerSocketForPreflight({}, 1002)).toBe(
|
|
||||||
'/run/user/1002/mosaic-lease/broker.sock',
|
|
||||||
);
|
|
||||||
});
|
|
||||||
});
|
|
||||||
@@ -73,10 +73,6 @@ function program(
|
|||||||
runner,
|
runner,
|
||||||
reconcileDeps: {
|
reconcileDeps: {
|
||||||
homeDirectory: '/home/mosaic',
|
homeDirectory: '/home/mosaic',
|
||||||
// Deterministic broker presence: without a seam the reconciler probes the
|
|
||||||
// REAL host socket (#1297 F3), making every CLI start test answer the
|
|
||||||
// host's broker state instead of its own property.
|
|
||||||
checkBrokerSocket: async () => true,
|
|
||||||
readHolderIdentity: async () => '11111111-1111-4111-8111-111111111111',
|
readHolderIdentity: async () => '11111111-1111-4111-8111-111111111111',
|
||||||
validateRoster: async () => undefined,
|
validateRoster: async () => undefined,
|
||||||
prepareProjections: async () => [{ agentName: 'coder0' }],
|
prepareProjections: async () => [{ agentName: 'coder0' }],
|
||||||
|
|||||||
@@ -1,24 +1,11 @@
|
|||||||
import {
|
import { chmod, lstat, mkdir, mkdtemp, readFile, rm, stat, writeFile } from 'node:fs/promises';
|
||||||
chmod,
|
|
||||||
lstat,
|
|
||||||
mkdir,
|
|
||||||
mkdtemp,
|
|
||||||
readFile,
|
|
||||||
readlink,
|
|
||||||
rm,
|
|
||||||
stat,
|
|
||||||
symlink,
|
|
||||||
writeFile,
|
|
||||||
} from 'node:fs/promises';
|
|
||||||
import { tmpdir } from 'node:os';
|
import { tmpdir } from 'node:os';
|
||||||
import { dirname, join, resolve } from 'node:path';
|
import { dirname, join, resolve } from 'node:path';
|
||||||
import { createServer } from 'node:net';
|
|
||||||
import { Command } from 'commander';
|
import { Command } from 'commander';
|
||||||
import { afterEach, describe, expect, it, vi } from 'vitest';
|
import { afterEach, describe, expect, it, vi } from 'vitest';
|
||||||
import {
|
import {
|
||||||
acquireRestartLock,
|
acquireRestartLock,
|
||||||
addAgentToRoster,
|
addAgentToRoster,
|
||||||
brokerSocketPresent,
|
|
||||||
buildAgentSendCommand,
|
buildAgentSendCommand,
|
||||||
buildAgentWatchAttachCommand,
|
buildAgentWatchAttachCommand,
|
||||||
buildAgentWatchCommand,
|
buildAgentWatchCommand,
|
||||||
@@ -55,7 +42,6 @@ import {
|
|||||||
parseSystemdShow,
|
parseSystemdShow,
|
||||||
parseTmuxListPanes,
|
parseTmuxListPanes,
|
||||||
parseTmuxListSessions,
|
parseTmuxListSessions,
|
||||||
placeUnitFile,
|
|
||||||
registerFleetCommand,
|
registerFleetCommand,
|
||||||
removeAgentFromRoster,
|
removeAgentFromRoster,
|
||||||
resolveFleetPaths,
|
resolveFleetPaths,
|
||||||
@@ -64,7 +50,6 @@ import {
|
|||||||
RESTART_LOCK_STALE_MS,
|
RESTART_LOCK_STALE_MS,
|
||||||
RUNTIME_ACCEPTABLE_COMMANDS,
|
RUNTIME_ACCEPTABLE_COMMANDS,
|
||||||
serializeRosterToYaml,
|
serializeRosterToYaml,
|
||||||
UnitPlacementError,
|
|
||||||
VERIFY_DEFAULT_TIMEOUT_MS,
|
VERIFY_DEFAULT_TIMEOUT_MS,
|
||||||
VERIFY_POLL_INTERVAL_MS,
|
VERIFY_POLL_INTERVAL_MS,
|
||||||
type AgentPsRow,
|
type AgentPsRow,
|
||||||
@@ -851,25 +836,13 @@ describe('fleet command construction', () => {
|
|||||||
};
|
};
|
||||||
const program = new Command();
|
const program = new Command();
|
||||||
program.exitOverride();
|
program.exitOverride();
|
||||||
// #1292: inject a present broker socket so the preflight passes and this
|
registerFleetCommand(program, { runner, mosaicHome: home });
|
||||||
// spec keeps testing its ORIGINAL property (holder-before-agent ordering).
|
|
||||||
// The preflight's own refusal behavior has dedicated specs below.
|
|
||||||
registerFleetCommand(program, {
|
|
||||||
runner,
|
|
||||||
mosaicHome: home,
|
|
||||||
checkBrokerSocket: async () => true,
|
|
||||||
});
|
|
||||||
|
|
||||||
try {
|
try {
|
||||||
await program.parseAsync(['node', 'mosaic', 'fleet', 'start']);
|
await program.parseAsync(['node', 'mosaic', 'fleet', 'start']);
|
||||||
await program.parseAsync(['node', 'mosaic', 'fleet', 'stop']);
|
await program.parseAsync(['node', 'mosaic', 'fleet', 'stop']);
|
||||||
|
|
||||||
expect(calls).toEqual([
|
expect(calls).toEqual([
|
||||||
// #1292: fleet start enables + starts the broker FIRST (enable is
|
|
||||||
// idempotent; the unit exists after install), re-checking the socket
|
|
||||||
// before any holder/agent lifecycle effect.
|
|
||||||
['systemctl', '--user', 'enable', 'mosaic-lease-broker.service'],
|
|
||||||
['systemctl', '--user', 'start', 'mosaic-lease-broker.service'],
|
|
||||||
['systemctl', '--user', 'start', 'mosaic-tmux-holder.service'],
|
['systemctl', '--user', 'start', 'mosaic-tmux-holder.service'],
|
||||||
['systemctl', '--user', 'start', '[email protected]'],
|
['systemctl', '--user', 'start', '[email protected]'],
|
||||||
['systemctl', '--user', 'stop', '[email protected]'],
|
['systemctl', '--user', 'stop', '[email protected]'],
|
||||||
@@ -880,92 +853,6 @@ describe('fleet command construction', () => {
|
|||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
it('fleet start refuses with a named error when the broker socket does not appear (#1292)', async () => {
|
|
||||||
const home = await tempDir();
|
|
||||||
const rosterPath = join(home, 'fleet', 'roster.yaml');
|
|
||||||
await mkdir(join(home, 'fleet'), { recursive: true });
|
|
||||||
await writeFile(
|
|
||||||
rosterPath,
|
|
||||||
['version: 1', 'transport: tmux', 'agents:', ' - name: coder0', ' runtime: codex'].join(
|
|
||||||
'\n',
|
|
||||||
),
|
|
||||||
);
|
|
||||||
const calls: string[][] = [];
|
|
||||||
const runner: CommandRunner = async (command, args) => {
|
|
||||||
calls.push([command, ...args]);
|
|
||||||
return { stdout: '', stderr: '', exitCode: 0 };
|
|
||||||
};
|
|
||||||
const program = new Command();
|
|
||||||
program.exitOverride();
|
|
||||||
const errors: string[] = [];
|
|
||||||
const origError = console.error;
|
|
||||||
console.error = (...args: unknown[]) => {
|
|
||||||
errors.push(args.join(' '));
|
|
||||||
};
|
|
||||||
registerFleetCommand(program, {
|
|
||||||
runner,
|
|
||||||
mosaicHome: home,
|
|
||||||
checkBrokerSocket: async () => false,
|
|
||||||
});
|
|
||||||
try {
|
|
||||||
await program.parseAsync(['node', 'mosaic', 'fleet', 'start']);
|
|
||||||
// Refused: no holder/agent starts were issued after the broker attempt.
|
|
||||||
expect(calls).toEqual([
|
|
||||||
['systemctl', '--user', 'enable', 'mosaic-lease-broker.service'],
|
|
||||||
['systemctl', '--user', 'start', 'mosaic-lease-broker.service'],
|
|
||||||
]);
|
|
||||||
expect(errors.join('\n')).toContain('broker-absent');
|
|
||||||
expect(errors.join('\n')).toContain('mosaic fleet install');
|
|
||||||
} finally {
|
|
||||||
console.error = origError;
|
|
||||||
await rm(home, { recursive: true, force: true });
|
|
||||||
}
|
|
||||||
});
|
|
||||||
|
|
||||||
it('fleet start re-probes the broker on the SECOND invocation — no ActiveState trust (#1292 sticky half)', async () => {
|
|
||||||
const home = await tempDir();
|
|
||||||
const rosterPath = join(home, 'fleet', 'roster.yaml');
|
|
||||||
await mkdir(join(home, 'fleet'), { recursive: true });
|
|
||||||
await writeFile(
|
|
||||||
rosterPath,
|
|
||||||
['version: 1', 'transport: tmux', 'agents:', ' - name: coder0', ' runtime: codex'].join(
|
|
||||||
'\n',
|
|
||||||
),
|
|
||||||
);
|
|
||||||
const calls: string[][] = [];
|
|
||||||
const runner: CommandRunner = async (command, args) => {
|
|
||||||
calls.push([command, ...args]);
|
|
||||||
return { stdout: '', stderr: '', exitCode: 0 };
|
|
||||||
};
|
|
||||||
const program = new Command();
|
|
||||||
program.exitOverride();
|
|
||||||
// Broker socket NEVER appears — the second start must refuse exactly like
|
|
||||||
// the first; RemainAfterExit-style stale unit state changes nothing
|
|
||||||
// because the check is the socket, not systemctl.
|
|
||||||
registerFleetCommand(program, {
|
|
||||||
runner,
|
|
||||||
mosaicHome: home,
|
|
||||||
checkBrokerSocket: async () => false,
|
|
||||||
});
|
|
||||||
const errors: string[] = [];
|
|
||||||
const origError = console.error;
|
|
||||||
console.error = (...args: unknown[]) => {
|
|
||||||
errors.push(args.join(' '));
|
|
||||||
};
|
|
||||||
try {
|
|
||||||
await program.parseAsync(['node', 'mosaic', 'fleet', 'start']);
|
|
||||||
await program.parseAsync(['node', 'mosaic', 'fleet', 'start']);
|
|
||||||
// Two invocations, each refusing after its own broker attempt:
|
|
||||||
expect(
|
|
||||||
calls.filter((c) => c.join(' ') === 'systemctl --user start [email protected]'),
|
|
||||||
).toHaveLength(0);
|
|
||||||
expect(errors.filter((e) => e.includes('broker-absent')).length).toBeGreaterThanOrEqual(2);
|
|
||||||
} finally {
|
|
||||||
console.error = origError;
|
|
||||||
await rm(home, { recursive: true, force: true });
|
|
||||||
}
|
|
||||||
});
|
|
||||||
|
|
||||||
it('waits for an in-flight restart to clear before relaunching (re-entry guard)', async () => {
|
it('waits for an in-flight restart to clear before relaunching (re-entry guard)', async () => {
|
||||||
const home = await tempDir();
|
const home = await tempDir();
|
||||||
const rosterPath = join(home, 'fleet', 'roster.yaml');
|
const rosterPath = join(home, 'fleet', 'roster.yaml');
|
||||||
@@ -2179,19 +2066,8 @@ describe('fleet install — auto-enable units for boot-survival', () => {
|
|||||||
|
|
||||||
await enableFleetUnits(runner, minimalRoster, {});
|
await enableFleetUnits(runner, minimalRoster, {});
|
||||||
|
|
||||||
expect(calls).toContainEqual(['systemctl', '--user', 'enable', 'mosaic-lease-broker.service']);
|
|
||||||
expect(calls).toContainEqual(['systemctl', '--user', 'enable', 'mosaic-tmux-holder.service']);
|
expect(calls).toContainEqual(['systemctl', '--user', 'enable', 'mosaic-tmux-holder.service']);
|
||||||
expect(calls).toContainEqual(['systemctl', '--user', 'enable', '[email protected]']);
|
expect(calls).toContainEqual(['systemctl', '--user', 'enable', '[email protected]']);
|
||||||
// The broker must be enabled BEFORE the holder and agents: a start of any
|
|
||||||
// gated runtime without the broker is exactly the #1292 4-second death.
|
|
||||||
const brokerIndex = calls.findIndex(
|
|
||||||
(c) => c.join(' ') === 'systemctl --user enable mosaic-lease-broker.service',
|
|
||||||
);
|
|
||||||
const holderIndex = calls.findIndex(
|
|
||||||
(c) => c.join(' ') === 'systemctl --user enable mosaic-tmux-holder.service',
|
|
||||||
);
|
|
||||||
expect(brokerIndex).toBeGreaterThanOrEqual(0);
|
|
||||||
expect(brokerIndex).toBeLessThan(holderIndex);
|
|
||||||
});
|
});
|
||||||
|
|
||||||
it('install still succeeds when systemctl enable returns non-zero (non-fatal)', async () => {
|
it('install still succeeds when systemctl enable returns non-zero (non-fatal)', async () => {
|
||||||
@@ -4486,70 +4362,3 @@ describe('fleet ps — heartbeat path resolution', () => {
|
|||||||
);
|
);
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('#1297 review: the real broker probe, exercised without any seam', () => {
|
|
||||||
it('brokerSocketPresent answers a REAL unix socket via stat().isSocket() (access(S_IFSOCK) threw ERR_OUT_OF_RANGE)', async () => {
|
|
||||||
const dir = await tempDir();
|
|
||||||
const sockPath = join(dir, 'broker.sock');
|
|
||||||
const server = createServer();
|
|
||||||
await new Promise<void>((resolve) => {
|
|
||||||
server.listen(sockPath, resolve);
|
|
||||||
});
|
|
||||||
try {
|
|
||||||
// A live unix socket answers true through the REAL probe — no seam.
|
|
||||||
expect(await brokerSocketPresent({}, { MOSAIC_LEASE_BROKER_SOCKET: sockPath })).toBe(true);
|
|
||||||
// Discrimination is by file type: a regular file that EXISTS is not a
|
|
||||||
// socket. The old implementation could not reach either verdict — it
|
|
||||||
// threw ERR_OUT_OF_RANGE (node >= 24) and the catch answered false.
|
|
||||||
const notASocket = join(dir, 'not-a-sock');
|
|
||||||
await writeFile(notASocket, 'x');
|
|
||||||
expect(await brokerSocketPresent({}, { MOSAIC_LEASE_BROKER_SOCKET: notASocket })).toBe(false);
|
|
||||||
// Absent path: false, not a throw.
|
|
||||||
expect(
|
|
||||||
await brokerSocketPresent({}, { MOSAIC_LEASE_BROKER_SOCKET: join(dir, 'gone.sock') }),
|
|
||||||
).toBe(false);
|
|
||||||
} finally {
|
|
||||||
await new Promise<void>((resolve) => {
|
|
||||||
server.close(() => resolve());
|
|
||||||
});
|
|
||||||
}
|
|
||||||
await rm(dir, { recursive: true, force: true });
|
|
||||||
});
|
|
||||||
|
|
||||||
// EACCES-based unlink failure requires a non-root uid: root bypasses
|
|
||||||
// directory mode bits (CAP_DAC_OVERRIDE), so the abort path cannot be
|
|
||||||
// triggered this way under CI's root runner. Skipped there, exercised on
|
|
||||||
// every non-root dev host.
|
|
||||||
const itUnlessRoot =
|
|
||||||
typeof process.getuid === 'function' && process.getuid() === 0 ? it.skip : it;
|
|
||||||
itUnlessRoot(
|
|
||||||
'placeUnitFile aborts with UnitPlacementError when unlink fails — never copies through a live symlink',
|
|
||||||
async () => {
|
|
||||||
const dir = await tempDir();
|
|
||||||
const unitDir = join(dir, 'systemd', 'user');
|
|
||||||
await mkdir(unitDir, { recursive: true });
|
|
||||||
// By-path residue: destination is a symlink pointing somewhere else.
|
|
||||||
const residueTarget = join(dir, 'residue-target');
|
|
||||||
await writeFile(residueTarget, 'RESIDUE-BYTES');
|
|
||||||
const destination = join(unitDir, 'x.service');
|
|
||||||
await symlink(residueTarget, destination);
|
|
||||||
const source = join(dir, 'seed.service');
|
|
||||||
await writeFile(source, 'UNIT-BYTES');
|
|
||||||
// Read-only unit dir: unlink now fails EACCES (test runs as the owner,
|
|
||||||
// not root, so mode bits are enforced).
|
|
||||||
await chmod(unitDir, 0o500);
|
|
||||||
try {
|
|
||||||
await expect(placeUnitFile(source, unitDir, 'x.service')).rejects.toThrow(
|
|
||||||
UnitPlacementError,
|
|
||||||
);
|
|
||||||
} finally {
|
|
||||||
await chmod(unitDir, 0o700);
|
|
||||||
}
|
|
||||||
// The copy-through never happened: residue bytes intact, destination
|
|
||||||
// still the symlink (abort, not overwrite-through).
|
|
||||||
expect(await readFile(residueTarget, 'utf8')).toBe('RESIDUE-BYTES');
|
|
||||||
expect(await readlink(destination)).toBe(residueTarget);
|
|
||||||
await rm(dir, { recursive: true, force: true });
|
|
||||||
},
|
|
||||||
);
|
|
||||||
});
|
|
||||||
|
|||||||
@@ -1,13 +1,11 @@
|
|||||||
import { constants, type Stats } from 'node:fs';
|
import { constants } from 'node:fs';
|
||||||
import {
|
import {
|
||||||
access,
|
access,
|
||||||
chmod,
|
chmod,
|
||||||
copyFile,
|
copyFile,
|
||||||
lstat,
|
|
||||||
mkdir,
|
mkdir,
|
||||||
open,
|
open,
|
||||||
readFile,
|
readFile,
|
||||||
readlink,
|
|
||||||
stat,
|
stat,
|
||||||
unlink,
|
unlink,
|
||||||
writeFile,
|
writeFile,
|
||||||
@@ -92,8 +90,6 @@ export type SleepFn = (ms: number) => Promise<void>;
|
|||||||
|
|
||||||
export interface FleetCommandDeps {
|
export interface FleetCommandDeps {
|
||||||
runner?: CommandRunner;
|
runner?: CommandRunner;
|
||||||
/** Test seam for the #1292 fleet-start broker preflight (socket presence). */
|
|
||||||
checkBrokerSocket?: (path: string) => Promise<boolean> | boolean;
|
|
||||||
/** Injectable interactive runner for commands needing inherited TTY (e.g., `tmux attach`). */
|
/** Injectable interactive runner for commands needing inherited TTY (e.g., `tmux attach`). */
|
||||||
interactiveRunner?: InteractiveRunner;
|
interactiveRunner?: InteractiveRunner;
|
||||||
/**
|
/**
|
||||||
@@ -195,10 +191,6 @@ export interface NorthStarIdText {
|
|||||||
text: string;
|
text: string;
|
||||||
}
|
}
|
||||||
|
|
||||||
export interface NorthStarCriterion extends NorthStarIdText {
|
|
||||||
tier: number;
|
|
||||||
}
|
|
||||||
|
|
||||||
export interface NorthStarWorkstream {
|
export interface NorthStarWorkstream {
|
||||||
id: string;
|
id: string;
|
||||||
title: string;
|
title: string;
|
||||||
@@ -208,8 +200,6 @@ export interface NorthStarGoal {
|
|||||||
id: string;
|
id: string;
|
||||||
title: string;
|
title: string;
|
||||||
phase: number;
|
phase: number;
|
||||||
/** Capability tier this goal serves: 0 dogfood, 1 MVP, 2 full vision. Orthogonal to phase, which is build order. */
|
|
||||||
tier: number;
|
|
||||||
priority: string;
|
priority: string;
|
||||||
depends_on: string[];
|
depends_on: string[];
|
||||||
}
|
}
|
||||||
@@ -230,7 +220,7 @@ export interface NorthStar {
|
|||||||
mission: string;
|
mission: string;
|
||||||
substrate: { note: string };
|
substrate: { note: string };
|
||||||
standing_objectives: NorthStarIdText[];
|
standing_objectives: NorthStarIdText[];
|
||||||
success_criteria: NorthStarCriterion[];
|
success_criteria: NorthStarIdText[];
|
||||||
workstreams: NorthStarWorkstream[];
|
workstreams: NorthStarWorkstream[];
|
||||||
goals: NorthStarGoal[];
|
goals: NorthStarGoal[];
|
||||||
assumptions: NorthStarAssumption[];
|
assumptions: NorthStarAssumption[];
|
||||||
@@ -290,13 +280,9 @@ export function parseNorthStar(rawText: string): NorthStar {
|
|||||||
standing_objectives: requireArray(parsed.standing_objectives, 'standing_objectives').map(
|
standing_objectives: requireArray(parsed.standing_objectives, 'standing_objectives').map(
|
||||||
(row, i) => idText(row, 'standing_objectives', i),
|
(row, i) => idText(row, 'standing_objectives', i),
|
||||||
),
|
),
|
||||||
success_criteria: requireArray(parsed.success_criteria, 'success_criteria').map((row, i) => {
|
success_criteria: requireArray(parsed.success_criteria, 'success_criteria').map((row, i) =>
|
||||||
const tier = (row as Record<string, unknown>)?.tier;
|
idText(row, 'success_criteria', i),
|
||||||
if (typeof tier !== 'number') {
|
),
|
||||||
throw new Error(`NORTH_STAR.yaml: success_criteria[${i}].tier must be a number.`);
|
|
||||||
}
|
|
||||||
return { ...idText(row, 'success_criteria', i), tier };
|
|
||||||
}),
|
|
||||||
workstreams: requireArray(parsed.workstreams, 'workstreams').map((row, i) => {
|
workstreams: requireArray(parsed.workstreams, 'workstreams').map((row, i) => {
|
||||||
const ws = row as Record<string, unknown>;
|
const ws = row as Record<string, unknown>;
|
||||||
return {
|
return {
|
||||||
@@ -314,15 +300,10 @@ export function parseNorthStar(rawText: string): NorthStar {
|
|||||||
if (typeof phase !== 'number') {
|
if (typeof phase !== 'number') {
|
||||||
throw new Error(`NORTH_STAR.yaml: goals[${i}].phase must be a number.`);
|
throw new Error(`NORTH_STAR.yaml: goals[${i}].phase must be a number.`);
|
||||||
}
|
}
|
||||||
const tier = goal?.tier;
|
|
||||||
if (typeof tier !== 'number') {
|
|
||||||
throw new Error(`NORTH_STAR.yaml: goals[${i}].tier must be a number.`);
|
|
||||||
}
|
|
||||||
return {
|
return {
|
||||||
id: requireString(goal?.id, `goals[${i}].id`),
|
id: requireString(goal?.id, `goals[${i}].id`),
|
||||||
title: requireString(goal?.title, `goals[${i}].title`),
|
title: requireString(goal?.title, `goals[${i}].title`),
|
||||||
phase,
|
phase,
|
||||||
tier,
|
|
||||||
priority: requireString(goal?.priority, `goals[${i}].priority`),
|
priority: requireString(goal?.priority, `goals[${i}].priority`),
|
||||||
depends_on: dependsRaw.map((dep, j) => requireString(dep, `goals[${i}].depends_on[${j}]`)),
|
depends_on: dependsRaw.map((dep, j) => requireString(dep, `goals[${i}].depends_on[${j}]`)),
|
||||||
};
|
};
|
||||||
@@ -368,7 +349,7 @@ function renderMarkdownTable(headers: string[], rows: string[][]): string[] {
|
|||||||
* Pure function of its input — same input always yields byte-identical output,
|
* Pure function of its input — same input always yields byte-identical output,
|
||||||
* so the round-trip (YAML → render → write) is stable across runs. No clock, no
|
* so the round-trip (YAML → render → write) is stable across runs. No clock, no
|
||||||
* network, no CLI. Layout follows the repo's existing doctrine-doc convention
|
* network, no CLI. Layout follows the repo's existing doctrine-doc convention
|
||||||
* (heading, blockquote banner, then sections + tables, e.g. FLEET-DOCTRINE.md /
|
* (heading, blockquote banner, then sections + tables, e.g. north-star.md /
|
||||||
* mission-control/BOARD.md).
|
* mission-control/BOARD.md).
|
||||||
*/
|
*/
|
||||||
export function renderNorthStarMarkdown(ns: NorthStar): string {
|
export function renderNorthStarMarkdown(ns: NorthStar): string {
|
||||||
@@ -381,7 +362,7 @@ export function renderNorthStarMarkdown(ns: NorthStar): string {
|
|||||||
'> Projected deterministically from [`NORTH_STAR.yaml`](./NORTH_STAR.yaml) by the pure',
|
'> Projected deterministically from [`NORTH_STAR.yaml`](./NORTH_STAR.yaml) by the pure',
|
||||||
);
|
);
|
||||||
lines.push('> generator in `packages/mosaic/src/commands/fleet.ts` (`renderNorthStarMarkdown`).');
|
lines.push('> generator in `packages/mosaic/src/commands/fleet.ts` (`renderNorthStarMarkdown`).');
|
||||||
lines.push('> Edit the YAML, then regenerate. Self-contained Mosaic.');
|
lines.push('> Edit the YAML, then regenerate. Self-contained Mosaic — no Hermes dependency.');
|
||||||
lines.push('');
|
lines.push('');
|
||||||
|
|
||||||
lines.push('## Mission');
|
lines.push('## Mission');
|
||||||
@@ -404,7 +385,7 @@ export function renderNorthStarMarkdown(ns: NorthStar): string {
|
|||||||
lines.push('## Success criteria');
|
lines.push('## Success criteria');
|
||||||
lines.push('');
|
lines.push('');
|
||||||
for (const ac of ns.success_criteria) {
|
for (const ac of ns.success_criteria) {
|
||||||
lines.push(`- **${ac.id}** (tier ${ac.tier}) — ${ac.text}`);
|
lines.push(`- **${ac.id}** — ${ac.text}`);
|
||||||
}
|
}
|
||||||
lines.push('');
|
lines.push('');
|
||||||
|
|
||||||
@@ -422,11 +403,10 @@ export function renderNorthStarMarkdown(ns: NorthStar): string {
|
|||||||
lines.push('');
|
lines.push('');
|
||||||
lines.push(
|
lines.push(
|
||||||
...renderMarkdownTable(
|
...renderMarkdownTable(
|
||||||
['id', 'title', 'tier', 'phase', 'priority', 'depends_on'],
|
['id', 'title', 'phase', 'priority', 'depends_on'],
|
||||||
ns.goals.map((goal) => [
|
ns.goals.map((goal) => [
|
||||||
goal.id,
|
goal.id,
|
||||||
goal.title,
|
goal.title,
|
||||||
String(goal.tier),
|
|
||||||
String(goal.phase),
|
String(goal.phase),
|
||||||
goal.priority,
|
goal.priority,
|
||||||
goal.depends_on.length > 0 ? goal.depends_on.join(', ') : '—',
|
goal.depends_on.length > 0 ? goal.depends_on.join(', ') : '—',
|
||||||
@@ -819,135 +799,6 @@ export function buildSystemdEnableCommand(unit: string): string[] {
|
|||||||
return ['systemctl', '--user', 'enable', unit];
|
return ['systemctl', '--user', 'enable', unit];
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Place a unit file into the ACTIVE systemd user directory, never through a
|
|
||||||
* symlink (#1292, measured 2026-08-17).
|
|
||||||
*
|
|
||||||
* ⚠ SET-INDEPENDENCE (fomo-lin, 2026-08-17): the set of unit names carrying
|
|
||||||
* by-path residue and the set of unit names this install copies are
|
|
||||||
* INDEPENDENT. Until 0.0.50 they were disjoint only by accident of which
|
|
||||||
* units the install happened to name — fomo-lin survived copy-through solely
|
|
||||||
* because its one by-path symlink (the broker) was the one unit the install
|
|
||||||
* did NOT copy. Adding the broker to the copy set made the intersection
|
|
||||||
* non-empty on the first run. Whoever adds a fifth unit to the placement
|
|
||||||
* list inherits this helper and its unlink step; do not place units with a
|
|
||||||
* bare copyFile.
|
|
||||||
*
|
|
||||||
* A host provisioned by the enable-by-path convention carries a symlink AT
|
|
||||||
* the unit-name path in ~/.config/systemd/user/ pointing at the shipped
|
|
||||||
* template under ~/.config/mosaic/systemd/user/. Node's copyFile FOLLOWS
|
|
||||||
* that link and overwrites the SEED template instead of placing the active
|
|
||||||
* unit (verified with fs.copyFile on a throwaway systemd user instance) —
|
|
||||||
* silent, rc=0, and it mutates the directory every later reseed reads from.
|
|
||||||
* The same measurement showed `systemctl enable <name>` does NOT rewrite an
|
|
||||||
* existing by-path wants-symlink, so reconciliation must be explicit.
|
|
||||||
*
|
|
||||||
* Placement therefore: if the destination is a symlink, unlink it first
|
|
||||||
* (unlink → copy — copy-then-unlink would mutate the seed and then destroy
|
|
||||||
* the evidence that it did); then copy. Also removes a stale
|
|
||||||
* `default.target.wants/<name>` symlink that points outside the active
|
|
||||||
* directory (readlink — NOT readFile, which follows the link and returns the
|
|
||||||
* target's CONTENT), so the subsequent enable-by-name recreates it against
|
|
||||||
* the active copy. Idempotent: on a clean or already-reconciled destination
|
|
||||||
* every step is a no-op (the copy rewrites identical bytes).
|
|
||||||
*
|
|
||||||
* Returns what was done, for assertions and install reporting.
|
|
||||||
*/
|
|
||||||
export interface PlaceUnitResult {
|
|
||||||
readonly unit: string;
|
|
||||||
readonly destination: string;
|
|
||||||
/** A symlink at the unit-name path was unlinked (by-path residue). */
|
|
||||||
readonly unlinkedDestinationSymlink: boolean;
|
|
||||||
/** A stale wants-symlink pointing outside the active dir was removed. */
|
|
||||||
readonly removedStaleWantsSymlink: boolean;
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* placeUnitFile failed. Thrown BEFORE any copy: no destination bytes were
|
|
||||||
* written, so a residue target cannot have been clobbered by a copy-through
|
|
||||||
* (#1297 review F2).
|
|
||||||
*/
|
|
||||||
export class UnitPlacementError extends Error {
|
|
||||||
constructor(
|
|
||||||
readonly unit: string,
|
|
||||||
message: string,
|
|
||||||
) {
|
|
||||||
super(message);
|
|
||||||
this.name = UnitPlacementError.name;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
function isErrnoException(error: unknown): error is NodeJS.ErrnoException {
|
|
||||||
return error instanceof Error && 'code' in error && typeof error.code === 'string';
|
|
||||||
}
|
|
||||||
|
|
||||||
export async function placeUnitFile(
|
|
||||||
source: string,
|
|
||||||
systemdUserDir: string,
|
|
||||||
unit: string,
|
|
||||||
): Promise<PlaceUnitResult> {
|
|
||||||
const destination = join(systemdUserDir, unit);
|
|
||||||
let unlinkedDestinationSymlink = false;
|
|
||||||
// Destination-absent and unlink-FAILED are different outcomes and must not
|
|
||||||
// share a catch (#1297 review F2): a swallowed unlink error used to fall
|
|
||||||
// through to copyFile through the still-live symlink, silently reintroducing
|
|
||||||
// the exact copy-through this helper exists to prevent.
|
|
||||||
let destinationInfo: Stats | undefined;
|
|
||||||
try {
|
|
||||||
destinationInfo = await lstat(destination);
|
|
||||||
} catch (error) {
|
|
||||||
if (!isErrnoException(error) || error.code !== 'ENOENT') {
|
|
||||||
throw new UnitPlacementError(
|
|
||||||
unit,
|
|
||||||
`cannot inspect destination ${destination}: ${error instanceof Error ? error.message : String(error)}`,
|
|
||||||
);
|
|
||||||
}
|
|
||||||
// ENOENT: absent destination — nothing to unlink, copy below is safe.
|
|
||||||
}
|
|
||||||
if (destinationInfo?.isSymbolicLink()) {
|
|
||||||
try {
|
|
||||||
await unlink(destination);
|
|
||||||
} catch (error) {
|
|
||||||
// Abort BEFORE the copy: proceeding would run copyFile through the
|
|
||||||
// still-live symlink and overwrite the residue target's bytes.
|
|
||||||
throw new UnitPlacementError(
|
|
||||||
unit,
|
|
||||||
`cannot unlink destination symlink ${destination}: ${error instanceof Error ? error.message : String(error)}`,
|
|
||||||
);
|
|
||||||
}
|
|
||||||
unlinkedDestinationSymlink = true;
|
|
||||||
}
|
|
||||||
await copyFile(source, destination);
|
|
||||||
|
|
||||||
let removedStaleWantsSymlink = false;
|
|
||||||
const wantsLink = join(systemdUserDir, 'default.target.wants', unit);
|
|
||||||
try {
|
|
||||||
const wantsInfo = await lstat(wantsLink);
|
|
||||||
if (wantsInfo.isSymbolicLink()) {
|
|
||||||
// readlink — NOT readFile: readFile FOLLOWS the link and returns the
|
|
||||||
// target file's CONTENT, which is not the question being asked.
|
|
||||||
let target: string | undefined;
|
|
||||||
try {
|
|
||||||
target = await readlink(wantsLink);
|
|
||||||
} catch {
|
|
||||||
target = undefined;
|
|
||||||
}
|
|
||||||
// Normalize (systemctl writes absolute targets; a relative one resolves
|
|
||||||
// against the wants dir). A wants-symlink pointing anywhere other than
|
|
||||||
// the active copy (the by-path convention points at the seed template)
|
|
||||||
// survives enable-by-name unchanged — remove it so enable recreates it.
|
|
||||||
if (target !== undefined && resolve(dirname(wantsLink), target) !== destination) {
|
|
||||||
await unlink(wantsLink);
|
|
||||||
removedStaleWantsSymlink = true;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
} catch {
|
|
||||||
// absent wants link — nothing to reconcile
|
|
||||||
}
|
|
||||||
|
|
||||||
return { unit, destination, unlinkedDestinationSymlink, removedStaleWantsSymlink };
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Returns the systemctl --user disable command for a given unit.
|
* Returns the systemctl --user disable command for a given unit.
|
||||||
* Used by `fleet remove` so a removed agent's enabled unit cannot resurrect on
|
* Used by `fleet remove` so a removed agent's enabled unit cannot resurrect on
|
||||||
@@ -982,22 +833,6 @@ export async function enableFleetUnits(
|
|||||||
let succeeded = 0;
|
let succeeded = 0;
|
||||||
let failed = 0;
|
let failed = 0;
|
||||||
|
|
||||||
// The lease broker ships with the fleet and every gated runtime needs it
|
|
||||||
// (#1292): seats die at lease registration without it, and no documented
|
|
||||||
// path ever enabled it. Enabled first — alongside the holder — and the
|
|
||||||
// unit must have been placed by installFleet's placeUnitFile step.
|
|
||||||
const brokerResult = await runner(
|
|
||||||
...splitCommand(buildSystemdEnableCommand('mosaic-lease-broker.service')),
|
|
||||||
);
|
|
||||||
if (brokerResult.exitCode === 0) {
|
|
||||||
succeeded++;
|
|
||||||
} else {
|
|
||||||
failed++;
|
|
||||||
process.stderr.write(
|
|
||||||
`Warning: could not enable mosaic-lease-broker.service: ${brokerResult.stderr || brokerResult.stdout || 'non-zero exit'}\n`,
|
|
||||||
);
|
|
||||||
}
|
|
||||||
|
|
||||||
const holderResult = await runner(
|
const holderResult = await runner(
|
||||||
...splitCommand(buildSystemdEnableCommand('mosaic-tmux-holder.service')),
|
...splitCommand(buildSystemdEnableCommand('mosaic-tmux-holder.service')),
|
||||||
);
|
);
|
||||||
@@ -1694,7 +1529,7 @@ export function registerFleetCommand(program: Command, deps: FleetCommandDeps =
|
|||||||
.description('Install local fleet tools and user systemd units')
|
.description('Install local fleet tools and user systemd units')
|
||||||
.option('--no-enable', 'Skip enabling units for boot-survival')
|
.option('--no-enable', 'Skip enabling units for boot-survival')
|
||||||
.action(async (opts: { enable?: boolean }) => {
|
.action(async (opts: { enable?: boolean }) => {
|
||||||
await installFleet(cmd, frameworkRoot, runner);
|
await installFleet(cmd, frameworkRoot);
|
||||||
// Unit enablement needs agent names only, so it reads either version.
|
// Unit enablement needs agent names only, so it reads either version.
|
||||||
const roster = await loadRosterReadModel(cmd);
|
const roster = await loadRosterReadModel(cmd);
|
||||||
await enableFleetUnits(runner, roster, opts);
|
await enableFleetUnits(runner, roster, opts);
|
||||||
@@ -1705,7 +1540,7 @@ export function registerFleetCommand(program: Command, deps: FleetCommandDeps =
|
|||||||
.description('Install local fleet tools and user systemd units')
|
.description('Install local fleet tools and user systemd units')
|
||||||
.option('--no-enable', 'Skip enabling units for boot-survival')
|
.option('--no-enable', 'Skip enabling units for boot-survival')
|
||||||
.action(async (opts: { enable?: boolean }) => {
|
.action(async (opts: { enable?: boolean }) => {
|
||||||
await installFleet(cmd, frameworkRoot, runner);
|
await installFleet(cmd, frameworkRoot);
|
||||||
// Unit enablement needs agent names only, so it reads either version.
|
// Unit enablement needs agent names only, so it reads either version.
|
||||||
const roster = await loadRosterReadModel(cmd);
|
const roster = await loadRosterReadModel(cmd);
|
||||||
await enableFleetUnits(runner, roster, opts);
|
await enableFleetUnits(runner, roster, opts);
|
||||||
@@ -1758,37 +1593,6 @@ export function registerFleetCommand(program: Command, deps: FleetCommandDeps =
|
|||||||
);
|
);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
if (action === 'start') {
|
|
||||||
// Broker preflight (#1292), re-probed on EVERY invocation: a
|
|
||||||
// gated runtime started without a live lease broker dies ~4s in
|
|
||||||
// while the unit reports active (RemainAfterExit) — enabling +
|
|
||||||
// starting here and then RE-CHECKING the socket refuses loudly
|
|
||||||
// instead of reporting rc0 over a doomed start. This is the
|
|
||||||
// second-start check as much as the first: it never trusts unit
|
|
||||||
// ActiveState.
|
|
||||||
await runChecked(runner, [
|
|
||||||
'systemctl',
|
|
||||||
'--user',
|
|
||||||
'enable',
|
|
||||||
'mosaic-lease-broker.service',
|
|
||||||
]);
|
|
||||||
await runChecked(runner, [
|
|
||||||
'systemctl',
|
|
||||||
'--user',
|
|
||||||
'start',
|
|
||||||
'mosaic-lease-broker.service',
|
|
||||||
]);
|
|
||||||
if (!(await brokerSocketPresent(deps))) {
|
|
||||||
console.error(
|
|
||||||
'[fleet] broker-absent: lease broker socket did not appear after enable+start (#1292).',
|
|
||||||
);
|
|
||||||
console.error(
|
|
||||||
'[fleet] remedy: mosaic fleet install (it reconciles either enable convention)',
|
|
||||||
);
|
|
||||||
process.exitCode = 1;
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
if (action === 'restart') {
|
if (action === 'restart') {
|
||||||
// Serialize the holder+agents teardown/relaunch behind the restart lock
|
// Serialize the holder+agents teardown/relaunch behind the restart lock
|
||||||
// so a re-entrant restart waits for clean shutdown before relaunching,
|
// so a re-entrant restart waits for clean shutdown before relaunching,
|
||||||
@@ -2547,11 +2351,7 @@ export function registerFleetAgentCommands(
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
async function installFleet(
|
async function installFleet(cmd: Command, frameworkRoot: string): Promise<void> {
|
||||||
cmd: Command,
|
|
||||||
frameworkRoot: string,
|
|
||||||
runner: CommandRunner,
|
|
||||||
): Promise<void> {
|
|
||||||
const activePaths = resolveFleetPaths(cmd.opts<{ mosaicHome: string }>().mosaicHome);
|
const activePaths = resolveFleetPaths(cmd.opts<{ mosaicHome: string }>().mosaicHome);
|
||||||
assertDefaultMosaicHomeForSystemd(activePaths.mosaicHome);
|
assertDefaultMosaicHomeForSystemd(activePaths.mosaicHome);
|
||||||
// Read model first: every file this function places is roster-independent, and
|
// Read model first: every file this function places is roster-independent, and
|
||||||
@@ -2603,40 +2403,18 @@ async function installFleet(
|
|||||||
for (const toolPath of executableToolPaths) {
|
for (const toolPath of executableToolPaths) {
|
||||||
await chmod(toolPath, 0o755);
|
await chmod(toolPath, 0o755);
|
||||||
}
|
}
|
||||||
// Unit placement (#1292): every unit goes through placeUnitFile — never a
|
await copyFile(
|
||||||
// bare copyFile — so a by-path-enable symlink at the destination is
|
join(frameworkRoot, 'systemd', 'user', 'mosaic-tmux-holder.service'),
|
||||||
// unlinked rather than written through (copy-through would silently
|
join(activePaths.systemdUserDir, 'mosaic-tmux-holder.service'),
|
||||||
// overwrite the SEED template, measured 2026-08-17). The lease broker unit
|
|
||||||
// is placed here too: previously the install named three units and omitted
|
|
||||||
// the broker entirely, which is why no documented path ever enabled it.
|
|
||||||
const placedUnits = await Promise.all(
|
|
||||||
[
|
|
||||||
'mosaic-tmux-holder.service',
|
|
||||||
'[email protected]',
|
|
||||||
'[email protected]',
|
|
||||||
'mosaic-lease-broker.service',
|
|
||||||
].map((unit) =>
|
|
||||||
placeUnitFile(join(frameworkRoot, 'systemd', 'user', unit), activePaths.systemdUserDir, unit),
|
|
||||||
),
|
|
||||||
);
|
);
|
||||||
const reconciled = placedUnits.filter(
|
await copyFile(
|
||||||
(result) => result.unlinkedDestinationSymlink || result.removedStaleWantsSymlink,
|
join(frameworkRoot, 'systemd', 'user', '[email protected]'),
|
||||||
|
join(activePaths.systemdUserDir, '[email protected]'),
|
||||||
|
);
|
||||||
|
await copyFile(
|
||||||
|
join(frameworkRoot, 'systemd', 'user', '[email protected]'),
|
||||||
|
join(activePaths.systemdUserDir, '[email protected]'),
|
||||||
);
|
);
|
||||||
if (reconciled.length > 0) {
|
|
||||||
console.log(
|
|
||||||
`Reconciled ${reconciled.length} unit placement(s) from by-path enable residue: ${reconciled.map((r) => r.unit).join(', ')}`,
|
|
||||||
);
|
|
||||||
}
|
|
||||||
// systemd will not see a replaced unit file without a reload; do it once
|
|
||||||
// after all placements, before any enable call below. runCommand never
|
|
||||||
// rejects (it resolves exitCode 127 on spawn error), so a plain await with
|
|
||||||
// an exitCode check matches the rest of this file's systemctl handling.
|
|
||||||
const reloadResult = await runner(...splitCommand(['systemctl', '--user', 'daemon-reload']));
|
|
||||||
if (reloadResult.exitCode !== 0) {
|
|
||||||
process.stderr.write(
|
|
||||||
`Warning: systemctl --user daemon-reload after unit placement failed (non-systemd host?): ${reloadResult.stderr || reloadResult.stdout || 'non-zero exit'}\n`,
|
|
||||||
);
|
|
||||||
}
|
|
||||||
|
|
||||||
// On roster v2 the reconciler owns the generated env: `apply` writes it and
|
// On roster v2 the reconciler owns the generated env: `apply` writes it and
|
||||||
// `regen` rebuilds it, both from projectRosterV2AgentGeneratedEnv. Writing it
|
// `regen` rebuilds it, both from projectRosterV2AgentGeneratedEnv. Writing it
|
||||||
@@ -2833,45 +2611,6 @@ function splitCommand(command: string[]): [string, string[]] {
|
|||||||
return [bin, args];
|
return [bin, args];
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Lease-broker socket presence for the fleet-start preflight (#1292).
|
|
||||||
* Resolution precedence matches launch.ts's defaultLeaseBrokerSocket and
|
|
||||||
* start-agent-session.sh's broker_socket_path: explicit
|
|
||||||
* MOSAIC_LEASE_BROKER_SOCKET, else $XDG_RUNTIME_DIR/mosaic-lease/broker.sock,
|
|
||||||
* else /run/user/<uid>/mosaic-lease/broker.sock. Pure filesystem check — this
|
|
||||||
* deliberately does NOT consult systemd state: a unit can be active
|
|
||||||
* (RemainAfterExit) with no live socket, and the socket is the thing the
|
|
||||||
* gated runtime connects to. Injectable via deps for tests.
|
|
||||||
*/
|
|
||||||
export function resolveLeaseBrokerSocketForPreflight(
|
|
||||||
env: NodeJS.ProcessEnv = process.env,
|
|
||||||
uid: number = typeof process.getuid === 'function' ? process.getuid() : 0,
|
|
||||||
): string {
|
|
||||||
if (env['MOSAIC_LEASE_BROKER_SOCKET']) return env['MOSAIC_LEASE_BROKER_SOCKET'];
|
|
||||||
const runtimeDir = env['XDG_RUNTIME_DIR'] ?? `/run/user/${uid}`;
|
|
||||||
return join(runtimeDir, 'mosaic-lease', 'broker.sock');
|
|
||||||
}
|
|
||||||
|
|
||||||
export async function brokerSocketPresent(
|
|
||||||
deps: FleetCommandDeps,
|
|
||||||
env: NodeJS.ProcessEnv = process.env,
|
|
||||||
): Promise<boolean> {
|
|
||||||
const check = deps.checkBrokerSocket;
|
|
||||||
const socketPath = resolveLeaseBrokerSocketForPreflight(env);
|
|
||||||
if (check) return check(socketPath);
|
|
||||||
// S_IFSOCK (0xC000) is a file-TYPE constant, not an access() mode (0-7):
|
|
||||||
// access(path, S_IFSOCK) throws ERR_OUT_OF_RANGE on node >= 24 (measured on
|
|
||||||
// v24.18.0, #1297 review F1) and cannot succeed on any node — the old catch
|
|
||||||
// swallowed the throw, so this probe could NEVER return true and every
|
|
||||||
// un-seamed call reported the broker absent. stat() + isSocket() is the real
|
|
||||||
// check and matches the bash side's [ -S ].
|
|
||||||
try {
|
|
||||||
return (await stat(socketPath)).isSocket();
|
|
||||||
} catch {
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
/** All supported fleet profile names. */
|
/** All supported fleet profile names. */
|
||||||
export type FleetProfile =
|
export type FleetProfile =
|
||||||
| 'general'
|
| 'general'
|
||||||
|
|||||||
@@ -205,12 +205,6 @@ export async function runLeaseEnforcementDoctorCheck(
|
|||||||
message:
|
message:
|
||||||
`Lease-enforcement hooks (${matchedMarkers.join(', ')}) are wired in ~/.claude/settings.json, but ${reasons.join(' and ')}. ` +
|
`Lease-enforcement hooks (${matchedMarkers.join(', ')}) are wired in ~/.claude/settings.json, but ${reasons.join(' and ')}. ` +
|
||||||
'Every gated tool call will fail closed and BRICK this agent (see #869). ' +
|
'Every gated tool call will fail closed and BRICK this agent (see #869). ' +
|
||||||
// #1292: one remedy, correct under BOTH enable conventions (by-path on
|
'Remediate by activating the lease-broker supervisor (systemd unit + socket) or by removing the enforcement hooks from ~/.claude/settings.json.',
|
||||||
// the seed template, and copy-then-enable in the active dir). Written
|
|
||||||
// from the 2026-08-17 symlink measurement: `systemctl enable` by name
|
|
||||||
// does NOT rewrite an existing by-path wants-symlink, so teaching a
|
|
||||||
// manual systemctl line here could leave a host with two competing
|
|
||||||
// wants links. fleet install reconciles either shape.
|
|
||||||
'Remedy: run `mosaic fleet install` (it reconciles either enable convention), or remove the enforcement hooks from ~/.claude/settings.json.',
|
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -922,8 +922,8 @@ describe('fleet operator documentation', (): void => {
|
|||||||
);
|
);
|
||||||
expect(
|
expect(
|
||||||
surfaces.filter((surface): boolean => surface.category === 'InlineLiteral'),
|
surfaces.filter((surface): boolean => surface.category === 'InlineLiteral'),
|
||||||
).toHaveLength(882);
|
).toHaveLength(863);
|
||||||
expect(surfaces).toHaveLength(906);
|
expect(surfaces).toHaveLength(887);
|
||||||
|
|
||||||
const rosterSource = await readFile(join(fleetDocs, 'examples', 'roster-v2.yaml'), 'utf8');
|
const rosterSource = await readFile(join(fleetDocs, 'examples', 'roster-v2.yaml'), 'utf8');
|
||||||
const auxiliary: CodeSurface = {
|
const auxiliary: CodeSurface = {
|
||||||
|
|||||||
@@ -169,16 +169,6 @@ function reconcileDeps(host: FakeLifecycleHost): FleetReconcileDeps {
|
|||||||
applyProjection: async () => undefined,
|
applyProjection: async () => undefined,
|
||||||
readRoster: async () => host.roster,
|
readRoster: async () => host.roster,
|
||||||
acquireMutationLock: async () => async () => undefined,
|
acquireMutationLock: async () => async () => undefined,
|
||||||
// Hermetic broker observation (#1297 F3): without this, the plan probes
|
|
||||||
// the REAL host filesystem, so the "stable JSON" fixtures answered true
|
|
||||||
// on any machine with a live lease broker and false elsewhere. Pointing
|
|
||||||
// both paths at fixtures that do not exist pins socketPresent:false and
|
|
||||||
// unitInstalled:false on every host, which is what these fixtures assert.
|
|
||||||
brokerSocketEnv: {
|
|
||||||
MOSAIC_LEASE_BROKER_SOCKET: '/nonexistent/mosaic-lease/broker.sock',
|
|
||||||
XDG_CONFIG_HOME: '/nonexistent/mosaic-config',
|
|
||||||
XDG_RUNTIME_DIR: '/nonexistent/run',
|
|
||||||
},
|
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -469,7 +459,6 @@ describe('FCM-M3-002 reconciler lifecycle acceptance', (): void => {
|
|||||||
plan: {
|
plan: {
|
||||||
generation: 7,
|
generation: 7,
|
||||||
holder: 'owned',
|
holder: 'owned',
|
||||||
broker: { unitInstalled: false, socketPresent: false },
|
|
||||||
agents: [
|
agents: [
|
||||||
{
|
{
|
||||||
name: 'coder0',
|
name: 'coder0',
|
||||||
|
|||||||
@@ -1,7 +1,6 @@
|
|||||||
import { chmod, mkdir, mkdtemp, readFile, rm, symlink, writeFile } from 'node:fs/promises';
|
import { chmod, mkdir, mkdtemp, readFile, rm, symlink, writeFile } from 'node:fs/promises';
|
||||||
import { tmpdir } from 'node:os';
|
import { tmpdir } from 'node:os';
|
||||||
import { join } from 'node:path';
|
import { join } from 'node:path';
|
||||||
import { createServer } from 'node:net';
|
|
||||||
import { afterEach, describe, expect, it } from 'vitest';
|
import { afterEach, describe, expect, it } from 'vitest';
|
||||||
import {
|
import {
|
||||||
acquirePrivateReconcileLock,
|
acquirePrivateReconcileLock,
|
||||||
@@ -93,179 +92,6 @@ async function run(command: FleetReconcileCommand, overrides: Partial<FleetRecon
|
|||||||
}
|
}
|
||||||
|
|
||||||
describe('fleet roster-owned reconciler', (): void => {
|
describe('fleet roster-owned reconciler', (): void => {
|
||||||
// ── #1292: broker as first-class plan member + broker-first start ordering ──
|
|
||||||
|
|
||||||
it('reports broker unit and socket state in the plan (socket is the signal, not unit state)', async (): Promise<void> => {
|
|
||||||
const result = await run('status', {
|
|
||||||
statPath: async () => true,
|
|
||||||
checkBrokerSocket: async () => true,
|
|
||||||
});
|
|
||||||
expect(result.plan.broker).toEqual({ unitInstalled: true, socketPresent: true });
|
|
||||||
});
|
|
||||||
|
|
||||||
it('reports a dead broker as socketPresent=false even when the unit is installed (enabled-but-dead is the #1292 shape)', async (): Promise<void> => {
|
|
||||||
const result = await run('status', {
|
|
||||||
statPath: async () => true,
|
|
||||||
checkBrokerSocket: async () => false,
|
|
||||||
});
|
|
||||||
expect(result.plan.broker).toEqual({ unitInstalled: true, socketPresent: false });
|
|
||||||
});
|
|
||||||
|
|
||||||
it('probes the REAL filesystem when no seam is injected — live socket and unit report healthy, absent paths report absent (#1297 F3)', async (): Promise<void> => {
|
|
||||||
const dir = await mkdtemp(join(tmpdir(), 'mosaic-broker-probe-'));
|
|
||||||
cleanup = dir;
|
|
||||||
const configHome = join(dir, 'config');
|
|
||||||
const unitDir = join(configHome, 'systemd', 'user');
|
|
||||||
await mkdir(unitDir, { recursive: true });
|
|
||||||
await writeFile(join(unitDir, 'mosaic-lease-broker.service'), '[Unit]\n');
|
|
||||||
const sockPath = join(dir, 'broker.sock');
|
|
||||||
const server = createServer();
|
|
||||||
await new Promise<void>((resolve) => {
|
|
||||||
server.listen(sockPath, resolve);
|
|
||||||
});
|
|
||||||
try {
|
|
||||||
const result = await run('status', {
|
|
||||||
brokerSocketEnv: {
|
|
||||||
MOSAIC_LEASE_BROKER_SOCKET: sockPath,
|
|
||||||
XDG_CONFIG_HOME: configHome,
|
|
||||||
XDG_RUNTIME_DIR: dir,
|
|
||||||
},
|
|
||||||
});
|
|
||||||
expect(result.plan.broker).toEqual({ unitInstalled: true, socketPresent: true });
|
|
||||||
// Absent paths through the SAME seam-less path answer false — this is
|
|
||||||
// the half the old default got right; healthy is the half it got wrong.
|
|
||||||
const absent = await run('status', {
|
|
||||||
brokerSocketEnv: {
|
|
||||||
MOSAIC_LEASE_BROKER_SOCKET: join(dir, 'gone.sock'),
|
|
||||||
XDG_CONFIG_HOME: join(dir, 'gone-config'),
|
|
||||||
},
|
|
||||||
});
|
|
||||||
expect(absent.plan.broker).toEqual({ unitInstalled: false, socketPresent: false });
|
|
||||||
} finally {
|
|
||||||
await new Promise<void>((resolve) => {
|
|
||||||
server.close(() => resolve());
|
|
||||||
});
|
|
||||||
}
|
|
||||||
});
|
|
||||||
|
|
||||||
it('command start refuses with a named error when the broker socket does not appear after enable+start (#1297 F3)', async (): Promise<void> => {
|
|
||||||
const calls: string[][] = [];
|
|
||||||
await expect(
|
|
||||||
run('start', {
|
|
||||||
checkBrokerSocket: async () => false,
|
|
||||||
runner: async (command, args) => {
|
|
||||||
calls.push([command, ...args]);
|
|
||||||
if (command === 'tmux' && args.includes('list-sessions')) {
|
|
||||||
return { stdout: '_holder\ncoder0\n', stderr: '', exitCode: 0 };
|
|
||||||
}
|
|
||||||
if (command === 'tmux' && args.includes('show-environment')) {
|
|
||||||
return {
|
|
||||||
stdout:
|
|
||||||
'HOME=/home/mosaic\nMOSAIC_FLEET_OWNER=11111111-1111-4111-8111-111111111111\nMOSAIC_TMUX_HOLDER=_holder\nMOSAIC_TMUX_SOCKET=mosaic-fleet\nPATH=/usr/bin:/bin\nPWD=/home/mosaic\n',
|
|
||||||
stderr: '',
|
|
||||||
exitCode: 0,
|
|
||||||
};
|
|
||||||
}
|
|
||||||
return { stdout: '', stderr: '', exitCode: 0 };
|
|
||||||
},
|
|
||||||
}),
|
|
||||||
).rejects.toThrow(/broker-absent/);
|
|
||||||
// Refused: broker enable+start attempted, no holder/agent unit touched.
|
|
||||||
const agentStarts = calls.filter(
|
|
||||||
(c) => c.join(' ') === 'systemctl --user start [email protected]',
|
|
||||||
);
|
|
||||||
expect(agentStarts).toHaveLength(0);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('command start enables and starts the broker BEFORE the holder and any agent unit', async (): Promise<void> => {
|
|
||||||
const calls: string[][] = [];
|
|
||||||
const result = await run('start', {
|
|
||||||
// Deterministic broker presence: without the seam this test answers the
|
|
||||||
// HOST's broker state (passes on a machine with a live broker, refuses
|
|
||||||
// on CI), not the ordering property it exists for (#1297 follow-up).
|
|
||||||
checkBrokerSocket: async () => true,
|
|
||||||
runner: async (command, args) => {
|
|
||||||
calls.push([command, ...args]);
|
|
||||||
if (command === 'tmux' && args.includes('list-sessions')) {
|
|
||||||
return { stdout: '_holder\ncoder0\n', stderr: '', exitCode: 0 };
|
|
||||||
}
|
|
||||||
if (command === 'tmux' && args.includes('show-environment')) {
|
|
||||||
return {
|
|
||||||
stdout:
|
|
||||||
'HOME=/home/mosaic\nMOSAIC_FLEET_OWNER=11111111-1111-4111-8111-111111111111\nMOSAIC_TMUX_HOLDER=_holder\nMOSAIC_TMUX_SOCKET=mosaic-fleet\nPATH=/usr/bin:/bin\nPWD=/home/mosaic\n',
|
|
||||||
stderr: '',
|
|
||||||
exitCode: 0,
|
|
||||||
};
|
|
||||||
}
|
|
||||||
return { stdout: '', stderr: '', exitCode: 0 };
|
|
||||||
},
|
|
||||||
});
|
|
||||||
expect(result.lifecycle).toBe('complete');
|
|
||||||
const brokerEnable = calls.findIndex(
|
|
||||||
(c) => c.join(' ') === 'systemctl --user enable mosaic-lease-broker.service',
|
|
||||||
);
|
|
||||||
const brokerStart = calls.findIndex(
|
|
||||||
(c) => c.join(' ') === 'systemctl --user start mosaic-lease-broker.service',
|
|
||||||
);
|
|
||||||
const holderStart = calls.findIndex(
|
|
||||||
(c) => c.join(' ') === 'systemctl --user start mosaic-tmux-holder.service',
|
|
||||||
);
|
|
||||||
const agentStart = calls.findIndex(
|
|
||||||
(c) => c.join(' ') === 'systemctl --user start [email protected]',
|
|
||||||
);
|
|
||||||
expect(brokerEnable).toBeGreaterThanOrEqual(0);
|
|
||||||
expect(brokerStart).toBeGreaterThan(brokerEnable);
|
|
||||||
// Holder start may be absent (holder 'owned' in this fixture); if present it must follow the broker.
|
|
||||||
if (holderStart >= 0) expect(holderStart).toBeGreaterThan(brokerStart);
|
|
||||||
expect(agentStart).toBeGreaterThan(brokerStart);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('apply with a running desired agent also enables and starts the broker first', async (): Promise<void> => {
|
|
||||||
const calls: string[][] = [];
|
|
||||||
const runningRoster: FleetRosterV2 = {
|
|
||||||
...roster,
|
|
||||||
agents: roster.agents.map((agent) =>
|
|
||||||
agent.name === 'coder0'
|
|
||||||
? { ...agent, lifecycle: { enabled: true, desiredState: 'running' as const } }
|
|
||||||
: agent,
|
|
||||||
),
|
|
||||||
};
|
|
||||||
const result = await executeFleetReconcile({
|
|
||||||
roster: runningRoster,
|
|
||||||
command: 'apply',
|
|
||||||
expectedGeneration: 7,
|
|
||||||
deps: deps({
|
|
||||||
readRoster: async () => runningRoster,
|
|
||||||
// Deterministic broker presence (see start-ordering test note).
|
|
||||||
checkBrokerSocket: async () => true,
|
|
||||||
runner: async (command, args) => {
|
|
||||||
calls.push([command, ...args]);
|
|
||||||
if (command === 'tmux' && args.includes('list-sessions')) {
|
|
||||||
return { stdout: '_holder\n', stderr: '', exitCode: 0 };
|
|
||||||
}
|
|
||||||
if (command === 'tmux' && args.includes('show-environment')) {
|
|
||||||
return {
|
|
||||||
stdout:
|
|
||||||
'HOME=/home/mosaic\nMOSAIC_FLEET_OWNER=11111111-1111-4111-8111-111111111111\nMOSAIC_TMUX_HOLDER=_holder\nMOSAIC_TMUX_SOCKET=mosaic-fleet\nPATH=/usr/bin:/bin\nPWD=/home/mosaic\n',
|
|
||||||
stderr: '',
|
|
||||||
exitCode: 0,
|
|
||||||
};
|
|
||||||
}
|
|
||||||
return { stdout: '', stderr: '', exitCode: 0 };
|
|
||||||
},
|
|
||||||
}),
|
|
||||||
});
|
|
||||||
expect(result.applied).toBe(true);
|
|
||||||
const brokerStart = calls.findIndex(
|
|
||||||
(c) => c.join(' ') === 'systemctl --user start mosaic-lease-broker.service',
|
|
||||||
);
|
|
||||||
const agentStart = calls.findIndex(
|
|
||||||
(c) => c.join(' ') === 'systemctl --user start [email protected]',
|
|
||||||
);
|
|
||||||
expect(brokerStart).toBeGreaterThanOrEqual(0);
|
|
||||||
expect(agentStart).toBeGreaterThan(brokerStart);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('fails closed on a symlinked fleet ancestor without touching its target', async (): Promise<void> => {
|
it('fails closed on a symlinked fleet ancestor without touching its target', async (): Promise<void> => {
|
||||||
const home = await lockHome();
|
const home = await lockHome();
|
||||||
const fleet = join(home, 'fleet');
|
const fleet = join(home, 'fleet');
|
||||||
@@ -549,8 +375,6 @@ describe('fleet roster-owned reconciler', (): void => {
|
|||||||
expectedGeneration: 7,
|
expectedGeneration: 7,
|
||||||
deps: deps({
|
deps: deps({
|
||||||
readRoster: async () => runningRoster,
|
readRoster: async () => runningRoster,
|
||||||
// Deterministic broker presence (see start-ordering test note).
|
|
||||||
checkBrokerSocket: async () => true,
|
|
||||||
runner: async (command, args) => {
|
runner: async (command, args) => {
|
||||||
calls.push([command, ...args]);
|
calls.push([command, ...args]);
|
||||||
if (command === 'tmux' && args.includes('list-sessions')) {
|
if (command === 'tmux' && args.includes('list-sessions')) {
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
import { constants } from 'node:fs';
|
import { constants } from 'node:fs';
|
||||||
import { lstat, open, readFile, stat, unlink, type FileHandle } from 'node:fs/promises';
|
import { lstat, open, readFile, unlink, type FileHandle } from 'node:fs/promises';
|
||||||
import { randomUUID } from 'node:crypto';
|
import { randomUUID } from 'node:crypto';
|
||||||
import { homedir } from 'node:os';
|
import { homedir } from 'node:os';
|
||||||
import { join } from 'node:path';
|
import { join } from 'node:path';
|
||||||
@@ -44,10 +44,6 @@ export interface FleetReconcileDeps {
|
|||||||
readonly overrideDir?: string;
|
readonly overrideDir?: string;
|
||||||
readonly homeDirectory?: string;
|
readonly homeDirectory?: string;
|
||||||
readonly readHolderIdentity?: () => Promise<string>;
|
readonly readHolderIdentity?: () => Promise<string>;
|
||||||
/** Test/observation seams for the lease-broker plan member (#1292). */
|
|
||||||
readonly statPath?: (path: string) => Promise<boolean> | boolean;
|
|
||||||
readonly checkBrokerSocket?: (path: string) => Promise<boolean> | boolean;
|
|
||||||
readonly brokerSocketEnv?: NodeJS.ProcessEnv;
|
|
||||||
readonly validateRoster?: (roster: FleetRosterV2) => Promise<void>;
|
readonly validateRoster?: (roster: FleetRosterV2) => Promise<void>;
|
||||||
readonly prepareProjections?: (roster: FleetRosterV2) => Promise<readonly unknown[]>;
|
readonly prepareProjections?: (roster: FleetRosterV2) => Promise<readonly unknown[]>;
|
||||||
readonly applyProjection?: (prepared: unknown) => Promise<unknown>;
|
readonly applyProjection?: (prepared: unknown) => Promise<unknown>;
|
||||||
@@ -79,17 +75,6 @@ export interface FleetReconcileObservedAgent {
|
|||||||
export interface FleetReconcilePlan {
|
export interface FleetReconcilePlan {
|
||||||
readonly generation: number;
|
readonly generation: number;
|
||||||
readonly holder: 'owned' | 'missing' | 'ownership-mismatch';
|
readonly holder: 'owned' | 'missing' | 'ownership-mismatch';
|
||||||
/**
|
|
||||||
* Lease broker observation (#1292): every gated runtime registers with the
|
|
||||||
* broker or dies ~4s in — a broker not in the plan cannot be reported as
|
|
||||||
* drifted, which made "broker died an hour ago" and "broker fine"
|
|
||||||
* produce identical output. `unitInstalled` = unit file present in the
|
|
||||||
* active dir; `socketPresent` = live broker at the resolved socket path.
|
|
||||||
*/
|
|
||||||
readonly broker: {
|
|
||||||
readonly unitInstalled: boolean;
|
|
||||||
readonly socketPresent: boolean;
|
|
||||||
};
|
|
||||||
readonly agents: readonly FleetReconcileObservedAgent[];
|
readonly agents: readonly FleetReconcileObservedAgent[];
|
||||||
readonly unmanagedSessions: readonly string[];
|
readonly unmanagedSessions: readonly string[];
|
||||||
}
|
}
|
||||||
@@ -261,17 +246,7 @@ export async function executeFleetReconcile(
|
|||||||
lifecycle: 'complete',
|
lifecycle: 'complete',
|
||||||
plan,
|
plan,
|
||||||
};
|
};
|
||||||
} catch (error: unknown) {
|
} catch {
|
||||||
// A named lifecycle precondition (broker-absent after enable+start,
|
|
||||||
// #1297 F3) must surface as itself — converting it to the generic
|
|
||||||
// recoverable result would hide the diagnosis and report a clean
|
|
||||||
// refusal where a loud one is the point.
|
|
||||||
if (
|
|
||||||
error instanceof FleetReconcileError &&
|
|
||||||
error.code === 'lifecycle-precondition-failed'
|
|
||||||
) {
|
|
||||||
throw error;
|
|
||||||
}
|
|
||||||
result = {
|
result = {
|
||||||
applied: false,
|
applied: false,
|
||||||
authoritativeRoster: 'unchanged',
|
authoritativeRoster: 'unchanged',
|
||||||
@@ -340,63 +315,6 @@ function isObservational(command: FleetReconcileCommand): boolean {
|
|||||||
return command === 'plan' || command === 'status' || command === 'verify' || command === 'doctor';
|
return command === 'plan' || command === 'status' || command === 'verify' || command === 'doctor';
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Observe the lease broker for the plan (#1292). Unit presence via systemctl
|
|
||||||
* is-system-running is NOT the signal — a unit can be enabled-but-dead. The
|
|
||||||
* authoritative signal is the socket the gated runtimes connect to, matching
|
|
||||||
* broker-supervisor.ts's `checkBrokerSupervisorHealth` (healthy ===
|
|
||||||
* socketPresent). Injectable so tests drive every branch without a broker.
|
|
||||||
*/
|
|
||||||
function resolveBrokerSocketPath(env: NodeJS.ProcessEnv): string {
|
|
||||||
const uid = typeof process.getuid === 'function' ? process.getuid() : 0;
|
|
||||||
const runtimeDir = env['XDG_RUNTIME_DIR'] ?? `/run/user/${uid}`;
|
|
||||||
return env['MOSAIC_LEASE_BROKER_SOCKET'] ?? join(runtimeDir, 'mosaic-lease', 'broker.sock');
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Probe the broker socket. Seams take precedence, but with no seam injected
|
|
||||||
* the REAL stat().isSocket() runs (#1297 review F3): production passes no
|
|
||||||
* seams, and defaulting to false made plan/status/doctor report a healthy
|
|
||||||
* broker as absent — a dead broker was indistinguishable from noise.
|
|
||||||
*/
|
|
||||||
async function brokerSocketPresent(
|
|
||||||
deps: FleetReconcileDeps,
|
|
||||||
env: NodeJS.ProcessEnv,
|
|
||||||
): Promise<boolean> {
|
|
||||||
const socketPath = resolveBrokerSocketPath(env);
|
|
||||||
const check = deps.checkBrokerSocket;
|
|
||||||
if (check) return check(socketPath);
|
|
||||||
try {
|
|
||||||
return (await stat(socketPath)).isSocket();
|
|
||||||
} catch {
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
async function observeBroker(deps: FleetReconcileDeps): Promise<FleetReconcilePlan['broker']> {
|
|
||||||
const homeDirectory = deps.homeDirectory ?? homedir();
|
|
||||||
const env = (deps.brokerSocketEnv ?? process.env) as NodeJS.ProcessEnv;
|
|
||||||
const configHome = env['XDG_CONFIG_HOME'] ?? join(homeDirectory, '.config');
|
|
||||||
const unitPath = join(configHome, 'systemd', 'user', 'mosaic-lease-broker.service');
|
|
||||||
const statPath = deps.statPath;
|
|
||||||
let unitInstalled = false;
|
|
||||||
let socketPresent = false;
|
|
||||||
try {
|
|
||||||
// Same principle as the socket probe: no seam → look at the real
|
|
||||||
// filesystem. A unit file placed by installFleet (or a by-path residue
|
|
||||||
// symlink resolving to it) satisfies stat().isFile().
|
|
||||||
unitInstalled = statPath ? await statPath(unitPath) : (await stat(unitPath)).isFile();
|
|
||||||
} catch {
|
|
||||||
unitInstalled = false;
|
|
||||||
}
|
|
||||||
try {
|
|
||||||
socketPresent = await brokerSocketPresent(deps, env);
|
|
||||||
} catch {
|
|
||||||
socketPresent = false;
|
|
||||||
}
|
|
||||||
return { unitInstalled, socketPresent };
|
|
||||||
}
|
|
||||||
|
|
||||||
async function observeFleet(
|
async function observeFleet(
|
||||||
roster: FleetRosterV2,
|
roster: FleetRosterV2,
|
||||||
deps: FleetReconcileDeps,
|
deps: FleetReconcileDeps,
|
||||||
@@ -407,12 +325,10 @@ async function observeFleet(
|
|||||||
'-F',
|
'-F',
|
||||||
'#{session_name}',
|
'#{session_name}',
|
||||||
]);
|
]);
|
||||||
const broker = await observeBroker(deps);
|
|
||||||
if (sessionsResult.exitCode !== 0) {
|
if (sessionsResult.exitCode !== 0) {
|
||||||
return {
|
return {
|
||||||
generation: roster.generation,
|
generation: roster.generation,
|
||||||
holder: 'missing',
|
holder: 'missing',
|
||||||
broker,
|
|
||||||
agents: await observeAgents(roster, deps, new Set<string>()),
|
agents: await observeAgents(roster, deps, new Set<string>()),
|
||||||
unmanagedSessions: [],
|
unmanagedSessions: [],
|
||||||
};
|
};
|
||||||
@@ -435,7 +351,6 @@ async function observeFleet(
|
|||||||
return {
|
return {
|
||||||
generation: roster.generation,
|
generation: roster.generation,
|
||||||
holder,
|
holder,
|
||||||
broker,
|
|
||||||
agents: await observeAgents(roster, deps, sessions),
|
agents: await observeAgents(roster, deps, sessions),
|
||||||
unmanagedSessions: Object.freeze(unmanagedSessions.sort()),
|
unmanagedSessions: Object.freeze(unmanagedSessions.sort()),
|
||||||
};
|
};
|
||||||
@@ -592,17 +507,6 @@ async function executeExplicitLifecycle(
|
|||||||
plan: FleetReconcilePlan,
|
plan: FleetReconcilePlan,
|
||||||
agents: readonly FleetRosterV2Agent[],
|
agents: readonly FleetRosterV2Agent[],
|
||||||
): Promise<FleetReconcileResult> {
|
): Promise<FleetReconcileResult> {
|
||||||
const lifecycleApplyFailed = (): FleetReconcileResult => ({
|
|
||||||
applied: false,
|
|
||||||
authoritativeRoster: 'unchanged',
|
|
||||||
projections: 'not-applied',
|
|
||||||
lifecycle: 'incomplete',
|
|
||||||
plan,
|
|
||||||
recovery: {
|
|
||||||
code: 'lifecycle-apply-failed',
|
|
||||||
action: 'rerun-after-inspecting-owned-resources',
|
|
||||||
},
|
|
||||||
});
|
|
||||||
if (request.command === 'start') {
|
if (request.command === 'start') {
|
||||||
for (const agent of agents) {
|
for (const agent of agents) {
|
||||||
if (!agent.lifecycle.enabled) {
|
if (!agent.lifecycle.enabled) {
|
||||||
@@ -613,40 +517,6 @@ async function executeExplicitLifecycle(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
// Broker FIRST (#1292): a gated runtime started without a running lease
|
|
||||||
// broker dies ~4 seconds in at registration — enable the unit (install
|
|
||||||
// places it) and start it before any holder/agent lifecycle effect.
|
|
||||||
try {
|
|
||||||
if (request.command === 'start') {
|
|
||||||
await runChecked(request.deps, 'systemctl', [
|
|
||||||
'--user',
|
|
||||||
'enable',
|
|
||||||
'mosaic-lease-broker.service',
|
|
||||||
]);
|
|
||||||
await runChecked(request.deps, 'systemctl', [
|
|
||||||
'--user',
|
|
||||||
'start',
|
|
||||||
'mosaic-lease-broker.service',
|
|
||||||
]);
|
|
||||||
}
|
|
||||||
} catch {
|
|
||||||
return lifecycleApplyFailed();
|
|
||||||
}
|
|
||||||
if (request.command === 'start') {
|
|
||||||
// Socket re-check after start, as a NAMED precondition (#1297 review
|
|
||||||
// F3) — the same protection the v1 path in commands/fleet.ts has had all
|
|
||||||
// along: the unit reporting active is not the signal; the socket is.
|
|
||||||
// Deliberately outside the try/catch above: a swallowed FleetReconcileError
|
|
||||||
// here read as a generic recoverable failure, hiding the named refusal.
|
|
||||||
// Runs BEFORE any holder/agent unit is touched so nothing doomed starts.
|
|
||||||
const env = (request.deps.brokerSocketEnv ?? process.env) as NodeJS.ProcessEnv;
|
|
||||||
if (!(await brokerSocketPresent(request.deps, env))) {
|
|
||||||
throw new FleetReconcileError(
|
|
||||||
'lifecycle-precondition-failed',
|
|
||||||
'broker-absent: lease broker socket did not appear after enable+start (#1292; #1297 F3). Remedy: mosaic fleet install.',
|
|
||||||
);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
try {
|
try {
|
||||||
if (request.command === 'start' && plan.holder === 'missing') {
|
if (request.command === 'start' && plan.holder === 'missing') {
|
||||||
await runChecked(request.deps, 'systemctl', [
|
await runChecked(request.deps, 'systemctl', [
|
||||||
@@ -663,7 +533,17 @@ async function executeExplicitLifecycle(
|
|||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
} catch {
|
} catch {
|
||||||
return lifecycleApplyFailed();
|
return {
|
||||||
|
applied: false,
|
||||||
|
authoritativeRoster: 'unchanged',
|
||||||
|
projections: 'not-applied',
|
||||||
|
lifecycle: 'incomplete',
|
||||||
|
plan,
|
||||||
|
recovery: {
|
||||||
|
code: 'lifecycle-apply-failed',
|
||||||
|
action: 'rerun-after-inspecting-owned-resources',
|
||||||
|
},
|
||||||
|
};
|
||||||
}
|
}
|
||||||
return {
|
return {
|
||||||
applied: true,
|
applied: true,
|
||||||
@@ -683,22 +563,6 @@ async function applyDesiredLifecycle(
|
|||||||
(agent: FleetRosterV2Agent): boolean =>
|
(agent: FleetRosterV2Agent): boolean =>
|
||||||
agent.lifecycle.enabled && agent.lifecycle.desiredState === 'running',
|
agent.lifecycle.enabled && agent.lifecycle.desiredState === 'running',
|
||||||
);
|
);
|
||||||
// Broker before any running agent, same ordering and reason as the
|
|
||||||
// command-driven path above (#1292).
|
|
||||||
if (needsRunningAgent) {
|
|
||||||
await runChecked(deps, 'systemctl', ['--user', 'enable', 'mosaic-lease-broker.service']);
|
|
||||||
await runChecked(deps, 'systemctl', ['--user', 'start', 'mosaic-lease-broker.service']);
|
|
||||||
// Same socket re-check as the explicit start path (#1297 F3): apply with
|
|
||||||
// running desired agents starts gated runtimes too, and a broker that
|
|
||||||
// starts but never binds dooms them the same way.
|
|
||||||
const env = (deps.brokerSocketEnv ?? process.env) as NodeJS.ProcessEnv;
|
|
||||||
if (!(await brokerSocketPresent(deps, env))) {
|
|
||||||
throw new FleetReconcileError(
|
|
||||||
'lifecycle-precondition-failed',
|
|
||||||
'broker-absent: lease broker socket did not appear after enable+start (#1292; #1297 F3). Remedy: mosaic fleet install.',
|
|
||||||
);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
if (needsRunningAgent && plan.holder === 'missing') {
|
if (needsRunningAgent && plan.holder === 'missing') {
|
||||||
await runChecked(deps, 'systemctl', ['--user', 'start', 'mosaic-tmux-holder.service']);
|
await runChecked(deps, 'systemctl', ['--user', 'start', 'mosaic-tmux-holder.service']);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,79 +0,0 @@
|
|||||||
#!/usr/bin/env python3
|
|
||||||
"""D29 contracts: no lease is a no-op success; half-provisioned still fails closed."""
|
|
||||||
|
|
||||||
from __future__ import annotations
|
|
||||||
|
|
||||||
import importlib.util
|
|
||||||
import unittest
|
|
||||||
from pathlib import Path
|
|
||||||
|
|
||||||
TOOLS = Path(__file__).parents[2] / "framework/tools/lease-broker"
|
|
||||||
REVOKE_PATH = TOOLS / "revoke-lease.py"
|
|
||||||
|
|
||||||
_spec = importlib.util.spec_from_file_location("revoke_lease", REVOKE_PATH)
|
|
||||||
assert _spec and _spec.loader
|
|
||||||
revoke_lease = importlib.util.module_from_spec(_spec)
|
|
||||||
import sys as _sys
|
|
||||||
|
|
||||||
_sys.path.insert(0, str(TOOLS))
|
|
||||||
_spec.loader.exec_module(revoke_lease)
|
|
||||||
|
|
||||||
ARGV = ["--runtime", "claude", "--reason", "pre-compact"]
|
|
||||||
VALID_SESSION = "a" * 64
|
|
||||||
|
|
||||||
|
|
||||||
def _explode(*_args, **_kwargs):
|
|
||||||
raise AssertionError("broker must not be contacted when no lease is held")
|
|
||||||
|
|
||||||
|
|
||||||
class RevokeWithoutLease(unittest.TestCase):
|
|
||||||
def test_no_lease_variables_is_a_noop_success(self) -> None:
|
|
||||||
"""The D29 case: bare-launched session, nothing to revoke, must not deny."""
|
|
||||||
self.assertEqual(
|
|
||||||
revoke_lease.main(ARGV, environ={}, request=_explode),
|
|
||||||
0,
|
|
||||||
)
|
|
||||||
|
|
||||||
def test_no_lease_does_not_contact_the_broker(self) -> None:
|
|
||||||
"""A no-op must be vacuous: no socket, no generation bump, no transport."""
|
|
||||||
revoke_lease.main(ARGV, environ={"HOME": "/nonexistent"}, request=_explode)
|
|
||||||
|
|
||||||
def test_socket_without_session_still_fails_closed(self) -> None:
|
|
||||||
"""Half-provisioned is misconfiguration, not absence. Fail-closed stands."""
|
|
||||||
self.assertEqual(
|
|
||||||
revoke_lease.main(
|
|
||||||
ARGV,
|
|
||||||
environ={"MOSAIC_LEASE_BROKER_SOCKET": "/tmp/nonexistent.sock"},
|
|
||||||
request=_explode,
|
|
||||||
),
|
|
||||||
2,
|
|
||||||
)
|
|
||||||
|
|
||||||
def test_session_without_socket_still_fails_closed(self) -> None:
|
|
||||||
"""The mirror case, so the guard cannot be satisfied by either half alone."""
|
|
||||||
self.assertEqual(
|
|
||||||
revoke_lease.main(
|
|
||||||
ARGV,
|
|
||||||
environ={"MOSAIC_LEASE_SESSION_ID": VALID_SESSION},
|
|
||||||
request=_explode,
|
|
||||||
),
|
|
||||||
2,
|
|
||||||
)
|
|
||||||
|
|
||||||
def test_empty_string_counts_as_absent(self) -> None:
|
|
||||||
"""An exported-but-empty variable is not a lease."""
|
|
||||||
self.assertEqual(
|
|
||||||
revoke_lease.main(
|
|
||||||
ARGV,
|
|
||||||
environ={
|
|
||||||
"MOSAIC_LEASE_BROKER_SOCKET": "",
|
|
||||||
"MOSAIC_LEASE_SESSION_ID": "",
|
|
||||||
},
|
|
||||||
request=_explode,
|
|
||||||
),
|
|
||||||
0,
|
|
||||||
)
|
|
||||||
|
|
||||||
|
|
||||||
if __name__ == "__main__":
|
|
||||||
unittest.main()
|
|
||||||
@@ -337,13 +337,7 @@ class ExecutableEntrypointTest(unittest.TestCase):
|
|||||||
runpy.run_path(str(TOOLS_DIR / "launch-runtime.py"), run_name="__main__")
|
runpy.run_path(str(TOOLS_DIR / "launch-runtime.py"), run_name="__main__")
|
||||||
self.assertEqual(raised.exception.code, 64)
|
self.assertEqual(raised.exception.code, 64)
|
||||||
|
|
||||||
def test_revoker_entrypoint_noops_when_identity_environment_is_absent(self) -> None:
|
def test_revoker_entrypoint_denies_when_identity_environment_is_absent(self) -> None:
|
||||||
# D29 supersession. This assertion previously pinned rc=2. Absent identity
|
|
||||||
# means no lease was ever held, so there is nothing to revoke and the correct
|
|
||||||
# result is no-op success. The old pin was written in e4d7d45 (WI-3), the same
|
|
||||||
# commit that shipped launch-runtime.py's lease-var provisioning, on the
|
|
||||||
# assumption that an envless revoker was unreachable. D29 falsified that in
|
|
||||||
# production. Behavioural pins live in src/lease-broker/revoke_noop_unittest.py.
|
|
||||||
with patch.object(
|
with patch.object(
|
||||||
sys,
|
sys,
|
||||||
"argv",
|
"argv",
|
||||||
@@ -358,42 +352,9 @@ class ExecutableEntrypointTest(unittest.TestCase):
|
|||||||
io.StringIO()
|
io.StringIO()
|
||||||
), self.assertRaises(SystemExit) as raised:
|
), self.assertRaises(SystemExit) as raised:
|
||||||
runpy.run_path(str(TOOLS_DIR / "revoke-lease.py"), run_name="__main__")
|
runpy.run_path(str(TOOLS_DIR / "revoke-lease.py"), run_name="__main__")
|
||||||
self.assertEqual(raised.exception.code, 0)
|
self.assertEqual(raised.exception.code, 2)
|
||||||
|
|
||||||
def test_revoker_entrypoint_denies_when_identity_environment_is_half_provisioned(
|
|
||||||
self,
|
|
||||||
) -> None:
|
|
||||||
# The no-op above is reachable ONLY when identity is TOTALLY absent. A
|
|
||||||
# half-provisioned environment is a machinery-present failure and must still
|
|
||||||
# fail closed. main() already pins this; the entrypoint did not, and the
|
|
||||||
# entrypoint is what the runtime extension actually spawns.
|
|
||||||
half_provisioned = (
|
|
||||||
{"MOSAIC_LEASE_BROKER_SOCKET": "/run/test/broker.sock"},
|
|
||||||
{"MOSAIC_LEASE_SESSION_ID": "d" * 64},
|
|
||||||
)
|
|
||||||
for environment in half_provisioned:
|
|
||||||
with self.subTest(environment=environment), patch.object(
|
|
||||||
sys,
|
|
||||||
"argv",
|
|
||||||
[
|
|
||||||
str(TOOLS_DIR / "revoke-lease.py"),
|
|
||||||
"--runtime",
|
|
||||||
"claude",
|
|
||||||
"--reason",
|
|
||||||
"pre-compact",
|
|
||||||
],
|
|
||||||
), patch.dict(os.environ, environment, clear=True), redirect_stderr(
|
|
||||||
io.StringIO()
|
|
||||||
), self.assertRaises(SystemExit) as raised:
|
|
||||||
runpy.run_path(str(TOOLS_DIR / "revoke-lease.py"), run_name="__main__")
|
|
||||||
self.assertEqual(raised.exception.code, 2)
|
|
||||||
|
|
||||||
def test_gate_entrypoint_denies_when_identity_environment_is_absent(self) -> None:
|
def test_gate_entrypoint_denies_when_identity_environment_is_absent(self) -> None:
|
||||||
# Deliberately NOT changed alongside its revoker twin above. The asymmetry is
|
|
||||||
# intentional: the gate's deny-on-absent is the authorization path and is
|
|
||||||
# load-bearing, so absent identity must fail closed here. The revoker's rc=2
|
|
||||||
# was inert in the same case (no session id means no broker call is possible),
|
|
||||||
# which is why only the revoker moved under D29. Do not "restore symmetry".
|
|
||||||
class Stdin:
|
class Stdin:
|
||||||
buffer = io.BytesIO(b'{"tool_name":"Bash"}')
|
buffer = io.BytesIO(b'{"tool_name":"Bash"}')
|
||||||
|
|
||||||
@@ -742,11 +703,8 @@ class LeaseRevocationTest(unittest.TestCase):
|
|||||||
"MOSAIC_RUNTIME_GENERATION": "1",
|
"MOSAIC_RUNTIME_GENERATION": "1",
|
||||||
}
|
}
|
||||||
malformed_session = {**good, "MOSAIC_LEASE_SESSION_ID": "not-a-session"}
|
malformed_session = {**good, "MOSAIC_LEASE_SESSION_ID": "not-a-session"}
|
||||||
# D29 exemption: the `({}, ...)` case was removed from this list. An empty
|
|
||||||
# environment is absence-of-lease, not an identity/reply/transport failure, and
|
|
||||||
# its correct result is no-op success (pinned in revoke_noop_unittest.py). The
|
|
||||||
# five cases below are all machinery-present failures and stay fail-closed.
|
|
||||||
cases = [
|
cases = [
|
||||||
|
({}, lambda *_args: {"ok": True, "state": "UNVERIFIED"}),
|
||||||
(malformed_session, lambda *_args: {"ok": True, "state": "UNVERIFIED"}),
|
(malformed_session, lambda *_args: {"ok": True, "state": "UNVERIFIED"}),
|
||||||
(good, lambda *_args: {"ok": False, "state": "UNVERIFIED"}),
|
(good, lambda *_args: {"ok": False, "state": "UNVERIFIED"}),
|
||||||
(good, lambda *_args: {"ok": True, "state": "VERIFIED"}),
|
(good, lambda *_args: {"ok": True, "state": "VERIFIED"}),
|
||||||
|
|||||||
Reference in New Issue
Block a user