Compare commits
4 Commits
feat/869-c
...
main
| Author | SHA1 | Date | |
|---|---|---|---|
| 529c177830 | |||
| a32ce4c8f9 | |||
| d351caad36 | |||
| 76b86a246e |
@@ -18,12 +18,33 @@ set -Eeuo pipefail
|
|||||||
# MOSAIC_INSTALL_MODE — prompt|keep|overwrite (default: prompt)
|
# MOSAIC_INSTALL_MODE — prompt|keep|overwrite (default: prompt)
|
||||||
# MOSAIC_ALLOW_MISSING_SEQUENTIAL_THINKING — 1 to bypass MCP check
|
# MOSAIC_ALLOW_MISSING_SEQUENTIAL_THINKING — 1 to bypass MCP check
|
||||||
# MOSAIC_SKIP_SKILLS_SYNC — 1 to skip skill sync
|
# MOSAIC_SKIP_SKILLS_SYNC — 1 to skip skill sync
|
||||||
|
#
|
||||||
|
# Flags (CLI args, NOT environment variables — see #869 Point-1 C2):
|
||||||
|
# --allow-inactive-enforcement Explicit, per-invocation opt-out that lets the
|
||||||
|
# lease-enforcement hooks (mutator-gate.py,
|
||||||
|
# receipt-observer-client.py) be wired into
|
||||||
|
# ~/.claude/settings.json even when this host
|
||||||
|
# cannot confirm it can ACTIVATE them. Loud on
|
||||||
|
# use (see mosaic-link-runtime-assets). Default
|
||||||
|
# (flag absent) is fail-loud: the enforcement
|
||||||
|
# hooks are NOT wired and the framework's
|
||||||
|
# runtime-asset-link step reports a failure.
|
||||||
# ──────────────────────────────────────────────────────────────────────────────
|
# ──────────────────────────────────────────────────────────────────────────────
|
||||||
|
|
||||||
SOURCE_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
|
SOURCE_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
|
||||||
TARGET_DIR="${MOSAIC_HOME:-$HOME/.config/mosaic}"
|
TARGET_DIR="${MOSAIC_HOME:-$HOME/.config/mosaic}"
|
||||||
INSTALL_MODE="${MOSAIC_INSTALL_MODE:-prompt}"
|
INSTALL_MODE="${MOSAIC_INSTALL_MODE:-prompt}"
|
||||||
|
|
||||||
|
# Deliberately parsed from "$@" (a real, explicit, per-invocation argument) —
|
||||||
|
# never an environment variable — so this opt-out can never sit silently
|
||||||
|
# inherited in a shell profile. See #869 Point-1 C2.
|
||||||
|
ALLOW_INACTIVE_ENFORCEMENT=0
|
||||||
|
for _arg in "$@"; do
|
||||||
|
case "$_arg" in
|
||||||
|
--allow-inactive-enforcement) ALLOW_INACTIVE_ENFORCEMENT=1 ;;
|
||||||
|
esac
|
||||||
|
done
|
||||||
|
|
||||||
# Shared framework path-ownership manifest reader (#791). Parity with
|
# Shared framework path-ownership manifest reader (#791). Parity with
|
||||||
# packages/mosaic/src/framework/manifest.ts — both consume framework-manifest.txt.
|
# packages/mosaic/src/framework/manifest.ts — both consume framework-manifest.txt.
|
||||||
# Sourcing does not run its CLI dispatch (guarded by BASH_SOURCE==$0).
|
# Sourcing does not run its CLI dispatch (guarded by BASH_SOURCE==$0).
|
||||||
@@ -670,10 +691,15 @@ step "Post-install tasks"
|
|||||||
SCRIPTS="$TARGET_DIR/tools/_scripts"
|
SCRIPTS="$TARGET_DIR/tools/_scripts"
|
||||||
|
|
||||||
if [[ -x "$SCRIPTS/mosaic-link-runtime-assets" ]]; then
|
if [[ -x "$SCRIPTS/mosaic-link-runtime-assets" ]]; then
|
||||||
if "$SCRIPTS/mosaic-link-runtime-assets" >/dev/null 2>&1; then
|
link_args=()
|
||||||
|
[[ "$ALLOW_INACTIVE_ENFORCEMENT" == "1" ]] && link_args+=(--allow-inactive-enforcement)
|
||||||
|
# stdout is suppressed as before, but stderr is left connected: the
|
||||||
|
# install-ordering guard's FAIL LOUD message (#869 Point-1 C2) must reach
|
||||||
|
# the operator, not be swallowed silently.
|
||||||
|
if "$SCRIPTS/mosaic-link-runtime-assets" "${link_args[@]}" >/dev/null; then
|
||||||
ok "Runtime assets linked"
|
ok "Runtime assets linked"
|
||||||
else
|
else
|
||||||
warn "Runtime asset linking failed (non-fatal)"
|
warn "Runtime asset linking failed (non-fatal) — see message above for details."
|
||||||
fi
|
fi
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
|||||||
@@ -4,6 +4,22 @@ set -euo pipefail
|
|||||||
MOSAIC_HOME="${MOSAIC_HOME:-$HOME/.config/mosaic}"
|
MOSAIC_HOME="${MOSAIC_HOME:-$HOME/.config/mosaic}"
|
||||||
backup_stamp="$(date +%Y%m%d%H%M%S)"
|
backup_stamp="$(date +%Y%m%d%H%M%S)"
|
||||||
|
|
||||||
|
# ─── Install-ordering guard opt-out (#869 Point-1 C2) ───────────────────────
|
||||||
|
# Explicit, per-invocation CLI flag ONLY — deliberately NOT read from an
|
||||||
|
# environment variable, so it can never sit as a silently-inherited default in
|
||||||
|
# a shell profile or CI env. Absent (the default) => hard fail-loud path.
|
||||||
|
allow_inactive_enforcement=0
|
||||||
|
for arg in "$@"; do
|
||||||
|
case "$arg" in
|
||||||
|
--allow-inactive-enforcement) allow_inactive_enforcement=1 ;;
|
||||||
|
esac
|
||||||
|
done
|
||||||
|
|
||||||
|
# Tracks whether the Claude settings install-ordering guard (below) reported a
|
||||||
|
# degraded (enforcement-not-wired) outcome, so this script's own exit status
|
||||||
|
# reflects it even though the rest of the runtime-asset sync must still run.
|
||||||
|
guard_degraded=0
|
||||||
|
|
||||||
copy_file_managed() {
|
copy_file_managed() {
|
||||||
local src="$1"
|
local src="$1"
|
||||||
local dst="$2"
|
local dst="$2"
|
||||||
@@ -24,6 +40,103 @@ copy_file_managed() {
|
|||||||
cp "$src" "$dst"
|
cp "$src" "$dst"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
# ─── Install-ordering guard for settings.json (#869 Point-1 C2) ─────────────
|
||||||
|
#
|
||||||
|
# settings.json is where #828's enforcement hooks (PreToolUse mutator-gate.py,
|
||||||
|
# Stop receipt-observer-client.py) get wired unconditionally. Before copying
|
||||||
|
# it, delegate to `mosaic __link-claude-settings` (packages/mosaic/src/commands/
|
||||||
|
# install-ordering-guard.ts) so the wiring decision is made by importing the
|
||||||
|
# C1 activation probe (`leaseEnforcementActivatable()`) directly, rather than
|
||||||
|
# re-implementing the capability/supervisor checks in shell. That subcommand:
|
||||||
|
# - activatable -> writes settings.json with hooks intact, exits 0
|
||||||
|
# - NOT activatable -> writes settings.json with hooks STRIPPED,
|
||||||
|
# prints an actionable message, exits 1
|
||||||
|
# - NOT activatable + opt-out -> writes settings.json with hooks intact,
|
||||||
|
# prints a loud warning, exits 0
|
||||||
|
# The `mosaic` CLI is expected on PATH at this point ("No executables are
|
||||||
|
# placed on PATH — the mosaic npm CLI is the only binary", per install.sh).
|
||||||
|
# If it is not resolvable at all, that is itself strong evidence the
|
||||||
|
# activation half is absent, so the same fail-loud default applies via a
|
||||||
|
# minimal python3 fallback (this repo already depends on python3 for the
|
||||||
|
# lease broker itself).
|
||||||
|
copy_claude_settings_guarded() {
|
||||||
|
local src="$1"
|
||||||
|
local dst="$2"
|
||||||
|
|
||||||
|
local guard_args=(__link-claude-settings "$src" "$dst")
|
||||||
|
if [[ "$allow_inactive_enforcement" == "1" ]]; then
|
||||||
|
guard_args+=(--allow-inactive-enforcement)
|
||||||
|
fi
|
||||||
|
|
||||||
|
if command -v mosaic >/dev/null 2>&1; then
|
||||||
|
if mosaic "${guard_args[@]}"; then
|
||||||
|
return 0
|
||||||
|
fi
|
||||||
|
echo "[mosaic-link] Enforcement hooks were NOT wired into $dst (see message above)." >&2
|
||||||
|
guard_degraded=1
|
||||||
|
return 0
|
||||||
|
fi
|
||||||
|
|
||||||
|
echo "[mosaic-link] ERROR: 'mosaic' CLI not found on PATH — cannot confirm lease-enforcement" >&2
|
||||||
|
echo "[mosaic-link] activation capability. enforcement requested but activation half absent —" >&2
|
||||||
|
echo "[mosaic-link] needs a published CLI carrying launch-runtime activation + a broker" >&2
|
||||||
|
echo "[mosaic-link] supervisor; refusing to wire a dead gate (see #869)." >&2
|
||||||
|
|
||||||
|
if [[ "$allow_inactive_enforcement" == "1" ]]; then
|
||||||
|
echo "[mosaic-link] WARNING: --allow-inactive-enforcement set — wiring $dst AS-IS (with" >&2
|
||||||
|
echo "[mosaic-link] enforcement hooks) despite being unable to confirm activation." >&2
|
||||||
|
copy_file_managed "$src" "$dst"
|
||||||
|
return 0
|
||||||
|
fi
|
||||||
|
|
||||||
|
mkdir -p "$(dirname "$dst")"
|
||||||
|
if command -v python3 >/dev/null 2>&1; then
|
||||||
|
python3 - "$src" "$dst" <<'PYEOF'
|
||||||
|
import json, sys
|
||||||
|
|
||||||
|
src, dest = sys.argv[1], sys.argv[2]
|
||||||
|
with open(src) as f:
|
||||||
|
data = json.load(f)
|
||||||
|
|
||||||
|
hooks = data.get("hooks", {})
|
||||||
|
|
||||||
|
pre = hooks.get("PreToolUse", [])
|
||||||
|
hooks["PreToolUse"] = [
|
||||||
|
t for t in pre
|
||||||
|
if not any("mutator-gate.py" in h.get("command", "") for h in t.get("hooks", []))
|
||||||
|
]
|
||||||
|
if not hooks["PreToolUse"]:
|
||||||
|
del hooks["PreToolUse"]
|
||||||
|
|
||||||
|
stop = hooks.get("Stop", [])
|
||||||
|
new_stop = []
|
||||||
|
for t in stop:
|
||||||
|
kept = [h for h in t.get("hooks", []) if "receipt-observer-client.py" not in h.get("command", "")]
|
||||||
|
if kept:
|
||||||
|
t = dict(t)
|
||||||
|
t["hooks"] = kept
|
||||||
|
new_stop.append(t)
|
||||||
|
if new_stop:
|
||||||
|
hooks["Stop"] = new_stop
|
||||||
|
elif "Stop" in hooks:
|
||||||
|
del hooks["Stop"]
|
||||||
|
|
||||||
|
if hooks:
|
||||||
|
data["hooks"] = hooks
|
||||||
|
else:
|
||||||
|
data.pop("hooks", None)
|
||||||
|
|
||||||
|
with open(dest, "w") as f:
|
||||||
|
json.dump(data, f, indent=2)
|
||||||
|
f.write("\n")
|
||||||
|
PYEOF
|
||||||
|
else
|
||||||
|
cp "$src" "$dst"
|
||||||
|
fi
|
||||||
|
guard_degraded=1
|
||||||
|
return 0
|
||||||
|
}
|
||||||
|
|
||||||
remove_legacy_path() {
|
remove_legacy_path() {
|
||||||
local p="$1"
|
local p="$1"
|
||||||
|
|
||||||
@@ -110,6 +223,13 @@ for runtime_file in \
|
|||||||
fi
|
fi
|
||||||
src="$MOSAIC_HOME/runtime/claude/$runtime_file"
|
src="$MOSAIC_HOME/runtime/claude/$runtime_file"
|
||||||
[[ -f "$src" ]] || continue
|
[[ -f "$src" ]] || continue
|
||||||
|
if [[ "$runtime_file" == "settings.json" ]]; then
|
||||||
|
# Install-ordering guard (#869 Point-1 C2): gate enforcement-hook wiring
|
||||||
|
# on confirmed activation instead of the plain copy_file_managed used for
|
||||||
|
# every other runtime file. See copy_claude_settings_guarded() above.
|
||||||
|
copy_claude_settings_guarded "$src" "$HOME/.claude/$runtime_file"
|
||||||
|
continue
|
||||||
|
fi
|
||||||
copy_file_managed "$src" "$HOME/.claude/$runtime_file"
|
copy_file_managed "$src" "$HOME/.claude/$runtime_file"
|
||||||
done
|
done
|
||||||
|
|
||||||
@@ -167,3 +287,12 @@ fi
|
|||||||
|
|
||||||
echo "[mosaic-link] Runtime assets synced (non-symlink mode)"
|
echo "[mosaic-link] Runtime assets synced (non-symlink mode)"
|
||||||
echo "[mosaic-link] Canonical source: $MOSAIC_HOME"
|
echo "[mosaic-link] Canonical source: $MOSAIC_HOME"
|
||||||
|
|
||||||
|
# Propagate the install-ordering guard's outcome (#869 Point-1 C2): every
|
||||||
|
# other runtime asset above is best-effort/non-fatal, but a degraded
|
||||||
|
# (enforcement-not-wired) settings.json must make THIS script's own exit
|
||||||
|
# status non-zero so callers (framework/install.sh, finalize.ts) can surface
|
||||||
|
# it — never silently.
|
||||||
|
if [[ "$guard_degraded" == "1" ]]; then
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
|||||||
@@ -0,0 +1,180 @@
|
|||||||
|
#!/usr/bin/env bash
|
||||||
|
# Regression harness for issue #869 Point-1 C2 — the install-ordering guard
|
||||||
|
# wired into mosaic-link-runtime-assets.
|
||||||
|
#
|
||||||
|
# Root cause under test: mosaic-link-runtime-assets copies
|
||||||
|
# runtime/claude/settings.json (which embeds the PreToolUse mutator-gate.py
|
||||||
|
# hook and the Stop receipt-observer-client.py hook) straight into
|
||||||
|
# ~/.claude/settings.json, unconditionally. If the lease-broker activation
|
||||||
|
# half cannot be confirmed on this host, wiring those hooks bricks it with a
|
||||||
|
# fail-closed gate that can never be satisfied.
|
||||||
|
#
|
||||||
|
# This harness never invokes a real `mosaic` CLI build — it stubs the
|
||||||
|
# `__link-claude-settings` contract with a fake `mosaic` on PATH so the shell
|
||||||
|
# WIRING (does mosaic-link-runtime-assets call out correctly? does it
|
||||||
|
# propagate a degraded outcome? does it still copy every other runtime file?
|
||||||
|
# does --allow-inactive-enforcement forward through?) is exercised
|
||||||
|
# independently of the TS guard's own logic (already covered by
|
||||||
|
# install-ordering-guard.spec.ts). It also exercises the no-mosaic-on-PATH
|
||||||
|
# python3 fallback directly.
|
||||||
|
#
|
||||||
|
# Scenarios:
|
||||||
|
# 1. probe=true (fake mosaic exits 0) -> settings.json copied, script exits 0.
|
||||||
|
# 2. probe=false (fake mosaic exits 1) -> script exits 1 (guard_degraded
|
||||||
|
# propagated), but every OTHER runtime file is still copied.
|
||||||
|
# 3. probe=false + --allow-inactive-enforcement -> the flag is forwarded to
|
||||||
|
# the fake mosaic stub.
|
||||||
|
# 4. No `mosaic` on PATH at all (activation unconfirmable) -> the python3
|
||||||
|
# fallback strips the enforcement hooks itself and the script exits 1.
|
||||||
|
# 5. No `mosaic` on PATH + --allow-inactive-enforcement -> the python3
|
||||||
|
# fallback wires the hooks AS-IS and the script exits 0.
|
||||||
|
|
||||||
|
set -uo pipefail
|
||||||
|
|
||||||
|
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
|
||||||
|
LINK_SCRIPT="$SCRIPT_DIR/mosaic-link-runtime-assets"
|
||||||
|
|
||||||
|
TMP_ROOT=$(mktemp -d)
|
||||||
|
trap 'rm -rf "$TMP_ROOT"' EXIT
|
||||||
|
|
||||||
|
fail=0
|
||||||
|
fail_msg() {
|
||||||
|
echo "FAIL: $*" >&2
|
||||||
|
fail=1
|
||||||
|
}
|
||||||
|
|
||||||
|
FIXTURE_SETTINGS='{
|
||||||
|
"model": "opus",
|
||||||
|
"hooks": {
|
||||||
|
"PreToolUse": [
|
||||||
|
{ "matcher": ".*", "hooks": [ { "type": "command", "command": "python3 ~/.config/mosaic/tools/lease-broker/mutator-gate.py --runtime claude" } ] },
|
||||||
|
{ "matcher": "Write|Edit|MultiEdit", "hooks": [ { "type": "command", "command": "~/.config/mosaic/tools/qa/prevent-memory-write.sh" } ] }
|
||||||
|
],
|
||||||
|
"Stop": [
|
||||||
|
{ "hooks": [
|
||||||
|
{ "type": "command", "command": "python3 ~/.config/mosaic/tools/lease-broker/receipt-observer-client.py --runtime claude" },
|
||||||
|
{ "type": "command", "command": "~/.config/mosaic/tools/qa/reflect-stop-hook.sh" }
|
||||||
|
] }
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}'
|
||||||
|
|
||||||
|
# Sets up a fresh $MOSAIC_HOME/runtime/claude/{settings.json,CLAUDE.md,
|
||||||
|
# hooks-config.json,context7-integration.md} + fresh $HOME, echoes both paths
|
||||||
|
# space-separated for the caller to `read`.
|
||||||
|
new_scenario_dirs() {
|
||||||
|
local scenario="$1"
|
||||||
|
local base="$TMP_ROOT/$scenario"
|
||||||
|
local mosaic_home="$base/mosaic-home"
|
||||||
|
local home="$base/home"
|
||||||
|
mkdir -p "$mosaic_home/runtime/claude" "$home"
|
||||||
|
printf '%s' "$FIXTURE_SETTINGS" > "$mosaic_home/runtime/claude/settings.json"
|
||||||
|
echo "claude.md fixture" > "$mosaic_home/runtime/claude/CLAUDE.md"
|
||||||
|
echo '{"hooks":{}}' > "$mosaic_home/runtime/claude/hooks-config.json"
|
||||||
|
echo "context7 fixture" > "$mosaic_home/runtime/claude/context7-integration.md"
|
||||||
|
echo "$mosaic_home" "$home"
|
||||||
|
}
|
||||||
|
|
||||||
|
settings_has_marker() {
|
||||||
|
local file="$1" marker="$2"
|
||||||
|
[[ -f "$file" ]] && grep -q "$marker" "$file"
|
||||||
|
}
|
||||||
|
|
||||||
|
# A fake `mosaic` binary implementing only the __link-claude-settings contract
|
||||||
|
# this harness needs: writes dest verbatim (fixture is unmodified either way —
|
||||||
|
# this stub only exercises the CALL CONTRACT, not the TS strip logic, which
|
||||||
|
# has its own vitest coverage) and exits with the code the scenario wants.
|
||||||
|
# Records the args it was called with so the harness can assert forwarding.
|
||||||
|
make_fake_mosaic() {
|
||||||
|
local bin_dir="$1" exit_code="$2"
|
||||||
|
mkdir -p "$bin_dir"
|
||||||
|
cat > "$bin_dir/mosaic" <<EOF
|
||||||
|
#!/usr/bin/env bash
|
||||||
|
set -euo pipefail
|
||||||
|
echo "\$@" > "$bin_dir/mosaic.args"
|
||||||
|
if [[ "\$1" == "__link-claude-settings" ]]; then
|
||||||
|
cp "\$2" "\$3"
|
||||||
|
exit $exit_code
|
||||||
|
fi
|
||||||
|
exit 0
|
||||||
|
EOF
|
||||||
|
chmod +x "$bin_dir/mosaic"
|
||||||
|
}
|
||||||
|
|
||||||
|
# --- Scenario 1: probe=true (fake mosaic exits 0) ---------------------------
|
||||||
|
read -r MOSAIC_HOME_1 HOME_1 < <(new_scenario_dirs scenario1)
|
||||||
|
BIN_1="$TMP_ROOT/scenario1/bin"
|
||||||
|
make_fake_mosaic "$BIN_1" 0
|
||||||
|
|
||||||
|
OUTPUT=$(MOSAIC_HOME="$MOSAIC_HOME_1" HOME="$HOME_1" PATH="$BIN_1:$PATH" "$LINK_SCRIPT" 2>&1)
|
||||||
|
STATUS=$?
|
||||||
|
[[ "$STATUS" -eq 0 ]] || fail_msg "scenario1 (probe=true): expected exit 0, got $STATUS. Output: $OUTPUT"
|
||||||
|
[[ -f "$HOME_1/.claude/settings.json" ]] || fail_msg "scenario1: settings.json was not copied"
|
||||||
|
|
||||||
|
# --- Scenario 2: probe=false (fake mosaic exits 1) --------------------------
|
||||||
|
read -r MOSAIC_HOME_2 HOME_2 < <(new_scenario_dirs scenario2)
|
||||||
|
BIN_2="$TMP_ROOT/scenario2/bin"
|
||||||
|
make_fake_mosaic "$BIN_2" 1
|
||||||
|
|
||||||
|
OUTPUT=$(MOSAIC_HOME="$MOSAIC_HOME_2" HOME="$HOME_2" PATH="$BIN_2:$PATH" "$LINK_SCRIPT" 2>&1)
|
||||||
|
STATUS=$?
|
||||||
|
[[ "$STATUS" -ne 0 ]] || fail_msg "scenario2 (probe=false, default): expected non-zero exit, got 0. Output: $OUTPUT"
|
||||||
|
[[ -f "$HOME_2/.claude/CLAUDE.md" ]] || fail_msg "scenario2: CLAUDE.md was NOT copied even though it is independent of the settings.json guard"
|
||||||
|
[[ -f "$HOME_2/.claude/hooks-config.json" ]] || fail_msg "scenario2: hooks-config.json was NOT copied"
|
||||||
|
[[ -f "$HOME_2/.claude/context7-integration.md" ]] || fail_msg "scenario2: context7-integration.md was NOT copied"
|
||||||
|
case "$OUTPUT" in
|
||||||
|
*"NOT be wired"*|*"NOT wired"*) ;;
|
||||||
|
*) fail_msg "scenario2: expected an actionable degraded-wiring message in output, got: $OUTPUT" ;;
|
||||||
|
esac
|
||||||
|
|
||||||
|
# --- Scenario 3: probe=false + --allow-inactive-enforcement forwards the flag
|
||||||
|
read -r MOSAIC_HOME_3 HOME_3 < <(new_scenario_dirs scenario3)
|
||||||
|
BIN_3="$TMP_ROOT/scenario3/bin"
|
||||||
|
make_fake_mosaic "$BIN_3" 0
|
||||||
|
|
||||||
|
MOSAIC_HOME="$MOSAIC_HOME_3" HOME="$HOME_3" PATH="$BIN_3:$PATH" "$LINK_SCRIPT" --allow-inactive-enforcement >/dev/null 2>&1
|
||||||
|
RECORDED_ARGS="$(cat "$BIN_3/mosaic.args" 2>/dev/null || true)"
|
||||||
|
case "$RECORDED_ARGS" in
|
||||||
|
*"--allow-inactive-enforcement"*) ;;
|
||||||
|
*) fail_msg "scenario3: --allow-inactive-enforcement was not forwarded to the mosaic CLI invocation (got: '$RECORDED_ARGS')" ;;
|
||||||
|
esac
|
||||||
|
|
||||||
|
# --- Scenario 4: no `mosaic` on PATH at all -> python3 fallback strips hooks
|
||||||
|
read -r MOSAIC_HOME_4 HOME_4 < <(new_scenario_dirs scenario4)
|
||||||
|
EMPTY_BIN="$TMP_ROOT/scenario4/empty-bin"
|
||||||
|
mkdir -p "$EMPTY_BIN"
|
||||||
|
# A PATH containing only python3 (for the fallback) + core utils, no mosaic.
|
||||||
|
FALLBACK_PATH="$EMPTY_BIN:/usr/bin:/bin"
|
||||||
|
|
||||||
|
OUTPUT=$(MOSAIC_HOME="$MOSAIC_HOME_4" HOME="$HOME_4" PATH="$FALLBACK_PATH" "$LINK_SCRIPT" 2>&1)
|
||||||
|
STATUS=$?
|
||||||
|
[[ "$STATUS" -ne 0 ]] || fail_msg "scenario4 (no mosaic on PATH, default): expected non-zero exit, got 0. Output: $OUTPUT"
|
||||||
|
if settings_has_marker "$HOME_4/.claude/settings.json" "mutator-gate.py"; then
|
||||||
|
fail_msg "scenario4: mutator-gate.py hook was wired even though mosaic could not be resolved (activation unconfirmable)"
|
||||||
|
fi
|
||||||
|
if settings_has_marker "$HOME_4/.claude/settings.json" "receipt-observer-client.py"; then
|
||||||
|
fail_msg "scenario4: receipt-observer-client.py hook was wired even though mosaic could not be resolved"
|
||||||
|
fi
|
||||||
|
if ! settings_has_marker "$HOME_4/.claude/settings.json" "prevent-memory-write.sh"; then
|
||||||
|
fail_msg "scenario4: the unrelated prevent-memory-write.sh hook was incorrectly dropped too"
|
||||||
|
fi
|
||||||
|
|
||||||
|
# --- Scenario 5: no `mosaic` on PATH + --allow-inactive-enforcement --------
|
||||||
|
read -r MOSAIC_HOME_5 HOME_5 < <(new_scenario_dirs scenario5)
|
||||||
|
|
||||||
|
OUTPUT=$(MOSAIC_HOME="$MOSAIC_HOME_5" HOME="$HOME_5" PATH="$FALLBACK_PATH" "$LINK_SCRIPT" --allow-inactive-enforcement 2>&1)
|
||||||
|
STATUS=$?
|
||||||
|
[[ "$STATUS" -eq 0 ]] || fail_msg "scenario5 (no mosaic, opt-out): expected exit 0, got $STATUS. Output: $OUTPUT"
|
||||||
|
if ! settings_has_marker "$HOME_5/.claude/settings.json" "mutator-gate.py"; then
|
||||||
|
fail_msg "scenario5: mutator-gate.py hook should have been wired (explicit opt-out set)"
|
||||||
|
fi
|
||||||
|
case "$OUTPUT" in
|
||||||
|
*"WARNING"*"--allow-inactive-enforcement"*) ;;
|
||||||
|
*) fail_msg "scenario5: expected a loud WARNING mentioning --allow-inactive-enforcement, got: $OUTPUT" ;;
|
||||||
|
esac
|
||||||
|
|
||||||
|
if [[ "$fail" -eq 0 ]]; then
|
||||||
|
echo "install-ordering-guard regression passed (5/5 scenarios)"
|
||||||
|
fi
|
||||||
|
|
||||||
|
exit "$fail"
|
||||||
@@ -0,0 +1,180 @@
|
|||||||
|
#!/usr/bin/env python3
|
||||||
|
"""Enforcement-side version-coupling gate (issue #869, Point-1 card C4).
|
||||||
|
|
||||||
|
Root cause this exists to guard against (#828 version skew, restated from
|
||||||
|
the C1 activation probe in ``lease-activation-probe.ts``): the lease
|
||||||
|
broker's ENFORCEMENT half (this toolkit — ``launch-runtime.py``,
|
||||||
|
``mutator-gate.py``, ``revoke-lease.py``) and its ACTIVATION half
|
||||||
|
(``execLeaseGatedRuntime()`` in ``launch.ts``, which chains the gated
|
||||||
|
runtime through ``launch-runtime.py`` and injects ``MOSAIC_LEASE_*``) ship
|
||||||
|
on different channels — an npm package and a framework/CLI reseed. C1 gave
|
||||||
|
the activation half a versioned, machine-checkable identity
|
||||||
|
(``LEASE_ACTIVATION_CAPABILITY``, printed by the CLI's hidden
|
||||||
|
``mosaic __lease-capability`` subcommand). That identity is inert on its
|
||||||
|
own: nothing yet asserted that ENFORCEMENT actually requires the version
|
||||||
|
ACTIVATION advertises. This module is that assertion, owned by the
|
||||||
|
enforcement side.
|
||||||
|
|
||||||
|
``EXPECTED_ACTIVATION_CAPABILITY`` below is this toolkit's own contract
|
||||||
|
declaration — bump it only when this toolkit's launch/gate seam starts
|
||||||
|
requiring a different activation contract (new env vars it depends on,
|
||||||
|
changed chaining behavior, etc.), independent of any package semver, for
|
||||||
|
the same reason C1's constant is: #828 happened precisely because a
|
||||||
|
version number that should have moved did not.
|
||||||
|
|
||||||
|
This module never talks to a real broker or a real installed CLI in its
|
||||||
|
own tests — both the probe's command resolution and its ``run`` transport
|
||||||
|
are injectable so tests can drive every branch with fakes/stubs (see
|
||||||
|
``version_coupling_unittest.py``).
|
||||||
|
"""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import json
|
||||||
|
import os
|
||||||
|
import shlex
|
||||||
|
import shutil
|
||||||
|
import subprocess
|
||||||
|
from collections.abc import Callable, Mapping
|
||||||
|
from typing import Final, TypedDict
|
||||||
|
|
||||||
|
|
||||||
|
class ActivationCapability(TypedDict):
|
||||||
|
name: str
|
||||||
|
version: int
|
||||||
|
|
||||||
|
|
||||||
|
# ENFORCEMENT-side expected activation contract. OWNED by this toolkit (the
|
||||||
|
# enforcement half). Mirrors — but is deliberately a SEPARATE constant from
|
||||||
|
# — `LEASE_ACTIVATION_CAPABILITY` in
|
||||||
|
# `packages/mosaic/src/commands/lease-activation-probe.ts` (the activation
|
||||||
|
# half's own declaration of what it implements). The two are compared at
|
||||||
|
# runtime by `assert_activation_capability_matches()`; drift between them is
|
||||||
|
# exactly the version-skew failure mode #828/#869 exist to catch, and must
|
||||||
|
# FAIL LOUD, never a silent pass and never a dead (always-true) gate.
|
||||||
|
EXPECTED_ACTIVATION_CAPABILITY: Final[ActivationCapability] = {
|
||||||
|
"name": "lease-runtime-activation",
|
||||||
|
"version": 1,
|
||||||
|
}
|
||||||
|
|
||||||
|
# Matches `LEASE_CAPABILITY_PROBE_COMMAND` in lease-activation-probe.ts —
|
||||||
|
# the hidden CLI subcommand that prints the activation half's advertised
|
||||||
|
# capability as compact JSON.
|
||||||
|
LEASE_CAPABILITY_PROBE_COMMAND: Final = "__lease-capability"
|
||||||
|
|
||||||
|
PROBE_TIMEOUT_SECONDS: Final = 2.0
|
||||||
|
|
||||||
|
# Override hook: a full shell-style command line (parsed with `shlex.split`)
|
||||||
|
# to run INSTEAD of resolving `mosaic` on PATH and appending the probe
|
||||||
|
# subcommand. Real deployments should never need this — `mosaic` is on PATH
|
||||||
|
# whenever a runtime was launched via `mosaic <cmd>` in the first place, the
|
||||||
|
# only real caller of this seam. It exists for integration tests that spawn
|
||||||
|
# `launch-runtime.py` directly (never through the real CLI) to supply a
|
||||||
|
# fake/stub CLI probe, matching the existing convention of those tests
|
||||||
|
# supplying a fake broker and a fake runtime binary rather than depending on
|
||||||
|
# host state.
|
||||||
|
MOSAIC_COMMAND_OVERRIDE_VAR: Final = "MOSAIC_LEASE_VERSION_PROBE_COMMAND"
|
||||||
|
|
||||||
|
|
||||||
|
class VersionCouplingError(Exception):
|
||||||
|
"""Raised when the activation capability is absent, unreadable, or does
|
||||||
|
not match what enforcement expects. Callers MUST fail loud on this
|
||||||
|
(non-zero exit, clear actionable stderr) — never swallow it into a
|
||||||
|
silent pass, and never let its absence be treated as compatible."""
|
||||||
|
|
||||||
|
|
||||||
|
def _resolve_probe_command(environ: Mapping[str, str]) -> list[str] | None:
|
||||||
|
override = environ.get(MOSAIC_COMMAND_OVERRIDE_VAR)
|
||||||
|
if override:
|
||||||
|
parsed = shlex.split(override)
|
||||||
|
return parsed or None
|
||||||
|
resolved = shutil.which("mosaic")
|
||||||
|
if resolved is None:
|
||||||
|
return None
|
||||||
|
return [resolved, LEASE_CAPABILITY_PROBE_COMMAND]
|
||||||
|
|
||||||
|
|
||||||
|
def default_probe_activation_capability(
|
||||||
|
environ: Mapping[str, str] | None = None,
|
||||||
|
*,
|
||||||
|
run: Callable[..., subprocess.CompletedProcess[str]] = subprocess.run,
|
||||||
|
) -> ActivationCapability | None:
|
||||||
|
"""Real capability lookup: resolves and executes the CLI's hidden
|
||||||
|
``__lease-capability`` probe subcommand out-of-process (the same
|
||||||
|
mechanism `defaultCapabilityProbe()` in lease-activation-probe.ts uses
|
||||||
|
from the activation side) and parses its JSON stdout. Any failure to
|
||||||
|
resolve a command, spawn it, have it exit zero, or produce a well-shaped
|
||||||
|
``{name, version}`` JSON object is treated as NO capability (``None``)
|
||||||
|
— fail-closed, never a fabricated/guessed capability.
|
||||||
|
"""
|
||||||
|
source_environment = os.environ if environ is None else environ
|
||||||
|
command = _resolve_probe_command(source_environment)
|
||||||
|
if command is None:
|
||||||
|
return None
|
||||||
|
try:
|
||||||
|
completed = run(
|
||||||
|
command,
|
||||||
|
capture_output=True,
|
||||||
|
text=True,
|
||||||
|
timeout=PROBE_TIMEOUT_SECONDS,
|
||||||
|
check=False,
|
||||||
|
)
|
||||||
|
except (OSError, subprocess.TimeoutExpired, ValueError):
|
||||||
|
return None
|
||||||
|
if completed.returncode != 0:
|
||||||
|
return None
|
||||||
|
try:
|
||||||
|
parsed = json.loads(completed.stdout)
|
||||||
|
except json.JSONDecodeError:
|
||||||
|
return None
|
||||||
|
if (
|
||||||
|
not isinstance(parsed, dict)
|
||||||
|
or not isinstance(parsed.get("name"), str)
|
||||||
|
or not isinstance(parsed.get("version"), int)
|
||||||
|
or isinstance(parsed.get("version"), bool)
|
||||||
|
):
|
||||||
|
return None
|
||||||
|
return {"name": parsed["name"], "version": parsed["version"]}
|
||||||
|
|
||||||
|
|
||||||
|
def format_mismatch_message(
|
||||||
|
activation: ActivationCapability | None,
|
||||||
|
expected: ActivationCapability,
|
||||||
|
) -> str:
|
||||||
|
"""Actionable, non-silent remediation message for either failure shape:
|
||||||
|
absent/unreadable capability, or a present-but-incompatible one."""
|
||||||
|
if activation is None:
|
||||||
|
return (
|
||||||
|
"Mosaic lease activation capability unreadable: enforcement "
|
||||||
|
f"expects '{expected['name']}' v{expected['version']} but the "
|
||||||
|
f"CLI's `mosaic {LEASE_CAPABILITY_PROBE_COMMAND}` probe produced "
|
||||||
|
"no usable result (mosaic not on PATH, non-zero exit, or "
|
||||||
|
"malformed output) — framework/CLI version skew; upgrade both "
|
||||||
|
"as one unit; see #869."
|
||||||
|
)
|
||||||
|
if activation["name"] != expected["name"]:
|
||||||
|
return (
|
||||||
|
f"activation capability name '{activation['name']}' != "
|
||||||
|
f"enforcement expects '{expected['name']}' — framework/CLI "
|
||||||
|
"version skew; upgrade both as one unit; see #869"
|
||||||
|
)
|
||||||
|
return (
|
||||||
|
f"activation capability v{activation['version']} != enforcement "
|
||||||
|
f"expects v{expected['version']} — framework/CLI version skew; "
|
||||||
|
"upgrade both as one unit; see #869"
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def assert_activation_capability_matches(
|
||||||
|
activation: ActivationCapability | None,
|
||||||
|
expected: ActivationCapability = EXPECTED_ACTIVATION_CAPABILITY,
|
||||||
|
) -> None:
|
||||||
|
"""Raise `VersionCouplingError` unless `activation` is present AND its
|
||||||
|
`name`/`version` exactly match `expected`. Absence is treated the same
|
||||||
|
as a mismatch — never a silent pass."""
|
||||||
|
if (
|
||||||
|
activation is None
|
||||||
|
or activation.get("name") != expected["name"]
|
||||||
|
or activation.get("version") != expected["version"]
|
||||||
|
):
|
||||||
|
raise VersionCouplingError(format_mismatch_message(activation, expected))
|
||||||
@@ -12,11 +12,24 @@ from collections.abc import Callable, Mapping, Sequence
|
|||||||
from pathlib import Path
|
from pathlib import Path
|
||||||
from typing import Final
|
from typing import Final
|
||||||
|
|
||||||
|
from activation_version_gate import (
|
||||||
|
EXPECTED_ACTIVATION_CAPABILITY,
|
||||||
|
ActivationCapability,
|
||||||
|
VersionCouplingError,
|
||||||
|
assert_activation_capability_matches,
|
||||||
|
default_probe_activation_capability,
|
||||||
|
)
|
||||||
from lease_generation import initialize_runtime_generation
|
from lease_generation import initialize_runtime_generation
|
||||||
|
|
||||||
MAX_FRAME: Final = 64 * 1024
|
MAX_FRAME: Final = 64 * 1024
|
||||||
BROKER_TIMEOUT_SECONDS: Final = 1.5
|
BROKER_TIMEOUT_SECONDS: Final = 1.5
|
||||||
CLAUDE_DANGEROUS_FLAG: Final = "--dangerously-skip-permissions"
|
CLAUDE_DANGEROUS_FLAG: Final = "--dangerously-skip-permissions"
|
||||||
|
# Distinct, non-overlapping exit code for the C4 version-coupling gate (see
|
||||||
|
# `activation_version_gate.py`) — deliberately different from the `1`
|
||||||
|
# (broker registration failed closed) and `64` (usage error) codes already
|
||||||
|
# owned by this script, so a version-skew denial is unambiguous in caller
|
||||||
|
# logs/tests and is never confused with a broker-availability failure.
|
||||||
|
EXIT_VERSION_SKEW: Final = 65
|
||||||
|
|
||||||
|
|
||||||
def broker_request(socket_path: Path, request: dict[str, object]) -> dict[str, object]:
|
def broker_request(socket_path: Path, request: dict[str, object]) -> dict[str, object]:
|
||||||
@@ -47,6 +60,10 @@ def main(
|
|||||||
request: Callable[[Path, dict[str, object]], dict[str, object]] = broker_request,
|
request: Callable[[Path, dict[str, object]], dict[str, object]] = broker_request,
|
||||||
execute: Callable[[str, list[str], dict[str, str]], object] = os.execvpe,
|
execute: Callable[[str, list[str], dict[str, str]], object] = os.execvpe,
|
||||||
initialize_generation: Callable[[Path, int], None] = initialize_runtime_generation,
|
initialize_generation: Callable[[Path, int], None] = initialize_runtime_generation,
|
||||||
|
probe_activation_capability: Callable[
|
||||||
|
[Mapping[str, str]], ActivationCapability | None
|
||||||
|
] = default_probe_activation_capability,
|
||||||
|
expected_activation_capability: ActivationCapability = EXPECTED_ACTIVATION_CAPABILITY,
|
||||||
) -> int:
|
) -> int:
|
||||||
parser = argparse.ArgumentParser()
|
parser = argparse.ArgumentParser()
|
||||||
parser.add_argument("--runtime", required=True, choices=("claude", "pi"))
|
parser.add_argument("--runtime", required=True, choices=("claude", "pi"))
|
||||||
@@ -66,6 +83,25 @@ def main(
|
|||||||
command = [command[0], CLAUDE_DANGEROUS_FLAG, *command[1:]]
|
command = [command[0], CLAUDE_DANGEROUS_FLAG, *command[1:]]
|
||||||
|
|
||||||
source_environment = os.environ if environ is None else environ
|
source_environment = os.environ if environ is None else environ
|
||||||
|
|
||||||
|
# C4 version-coupling gate (#869 Point-1): before this ENFORCEMENT half
|
||||||
|
# chains into anything, assert that the ACTIVATION contract it is about
|
||||||
|
# to rely on (MOSAIC_LEASE_* injection, broker chaining) matches what
|
||||||
|
# this enforcement build expects. This is a build/deploy-defect check,
|
||||||
|
# not a broker-availability question, so it runs before — and
|
||||||
|
# independently of — broker registration below, and it FAILS LOUD: a
|
||||||
|
# clear stderr message plus a dedicated non-zero exit code, never a
|
||||||
|
# silent pass and never folded into the generic registration-failure
|
||||||
|
# branch.
|
||||||
|
try:
|
||||||
|
assert_activation_capability_matches(
|
||||||
|
probe_activation_capability(source_environment),
|
||||||
|
expected_activation_capability,
|
||||||
|
)
|
||||||
|
except VersionCouplingError as version_error:
|
||||||
|
print(str(version_error), file=sys.stderr)
|
||||||
|
return EXIT_VERSION_SKEW
|
||||||
|
|
||||||
try:
|
try:
|
||||||
socket_path = Path(source_environment["MOSAIC_LEASE_BROKER_SOCKET"])
|
socket_path = Path(source_environment["MOSAIC_LEASE_BROKER_SOCKET"])
|
||||||
generation = int(source_environment.get("MOSAIC_RUNTIME_GENERATION", "1"))
|
generation = int(source_environment.get("MOSAIC_RUNTIME_GENERATION", "1"))
|
||||||
|
|||||||
@@ -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": "python3 src/lease-broker/daemon_deadline_unittest.py && python3 src/lease-broker/normative_fragments_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/framework_skill_portability_unittest.py && python3 src/mutator-gate/runtime_tools_unittest.py && python3 src/mutator-gate/runtime_launch_guard_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-review-gitea-comment.sh && bash framework/tools/git/test-ci-queue-wait-branch-absent.sh && bash framework/tools/git/test-git-credential-mosaic.sh && bash framework/tools/git/test-gitea-token-identity.sh"
|
"test:framework-shell": "python3 src/lease-broker/daemon_deadline_unittest.py && python3 src/lease-broker/normative_fragments_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/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-review-gitea-comment.sh && bash framework/tools/git/test-ci-queue-wait-branch-absent.sh && bash framework/tools/git/test-git-credential-mosaic.sh && bash framework/tools/git/test-gitea-token-identity.sh && bash framework/tools/_scripts/test-install-ordering-guard.sh"
|
||||||
},
|
},
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"@mosaicstack/brain": "workspace:*",
|
"@mosaicstack/brain": "workspace:*",
|
||||||
|
|||||||
@@ -22,6 +22,7 @@ import { registerSkillCommand } from './commands/skill.js';
|
|||||||
// prdy is registered via launch.ts
|
// prdy is registered via launch.ts
|
||||||
import { registerLaunchCommands } from './commands/launch.js';
|
import { registerLaunchCommands } from './commands/launch.js';
|
||||||
import { registerLeaseCapabilityProbe } from './commands/lease-activation-probe.js';
|
import { registerLeaseCapabilityProbe } from './commands/lease-activation-probe.js';
|
||||||
|
import { registerInstallOrderingGuardCommand } from './commands/install-ordering-guard.js';
|
||||||
import { registerAuthCommand } from './commands/auth.js';
|
import { registerAuthCommand } from './commands/auth.js';
|
||||||
import { registerFederationCommand } from './commands/federation.js';
|
import { registerFederationCommand } from './commands/federation.js';
|
||||||
import { registerGatewayCommand } from './commands/gateway.js';
|
import { registerGatewayCommand } from './commands/gateway.js';
|
||||||
@@ -31,10 +32,7 @@ import {
|
|||||||
formatAllPackagesTable,
|
formatAllPackagesTable,
|
||||||
getInstallAllCommand,
|
getInstallAllCommand,
|
||||||
repairFleetCommsTools,
|
repairFleetCommsTools,
|
||||||
runFrameworkReseed,
|
runUpdateReseedFlow,
|
||||||
refreshActiveFleetUnits,
|
|
||||||
readRosterAgentNames,
|
|
||||||
buildRelaunchCommands,
|
|
||||||
checkFrameworkDrift,
|
checkFrameworkDrift,
|
||||||
FRAMEWORK_RESEED_PACKAGE,
|
FRAMEWORK_RESEED_PACKAGE,
|
||||||
} from './runtime/update-checker.js';
|
} from './runtime/update-checker.js';
|
||||||
@@ -83,6 +81,10 @@ registerLaunchCommands(program);
|
|||||||
|
|
||||||
registerLeaseCapabilityProbe(program);
|
registerLeaseCapabilityProbe(program);
|
||||||
|
|
||||||
|
// ─── install-ordering guard (hidden; #869 Point-1 C2) ───────────────────
|
||||||
|
|
||||||
|
registerInstallOrderingGuardCommand(program);
|
||||||
|
|
||||||
// ─── login ──────────────────────────────────────────────────────────────
|
// ─── login ──────────────────────────────────────────────────────────────
|
||||||
|
|
||||||
program
|
program
|
||||||
@@ -440,12 +442,18 @@ program
|
|||||||
'--repair-tools',
|
'--repair-tools',
|
||||||
'Restore the supported current-version TOOLS contract and executable fleet helper',
|
'Restore the supported current-version TOOLS contract and executable fleet helper',
|
||||||
)
|
)
|
||||||
|
.option(
|
||||||
|
'--allow-inactive-enforcement',
|
||||||
|
'Wire lease-enforcement hooks into settings.json even when activation cannot be confirmed ' +
|
||||||
|
'(explicit, loud, non-default opt-out for the post-reseed install-ordering guard — see #869/#882)',
|
||||||
|
)
|
||||||
.action(
|
.action(
|
||||||
async (opts: {
|
async (opts: {
|
||||||
check?: boolean;
|
check?: boolean;
|
||||||
reseed?: boolean;
|
reseed?: boolean;
|
||||||
relaunch?: boolean;
|
relaunch?: boolean;
|
||||||
repairTools?: boolean;
|
repairTools?: boolean;
|
||||||
|
allowInactiveEnforcement?: boolean;
|
||||||
}) => {
|
}) => {
|
||||||
if (opts.repairTools) {
|
if (opts.repairTools) {
|
||||||
const repair = repairFleetCommsTools();
|
const repair = repairFleetCommsTools();
|
||||||
@@ -466,57 +474,24 @@ program
|
|||||||
// checkForAllUpdates imported statically above
|
// checkForAllUpdates imported statically above
|
||||||
const { execSync } = await import('node:child_process');
|
const { execSync } = await import('node:child_process');
|
||||||
|
|
||||||
// Re-seed the framework from the freshly-installed package, propagate shipped
|
// Re-seed the framework from the freshly-installed package, re-apply the
|
||||||
// systemd unit fixes to the active units, and (opt-in) relaunch durable
|
// install-ordering guard to settings.json (#882 (b) — closes the
|
||||||
// agents. Shared by the "packages updated" and the "framework drift" paths.
|
// `--sync-only` bypass so `mosaic update` never leaves enforcement-hook
|
||||||
|
// wiring stale/unguarded), propagate shipped systemd unit fixes to the
|
||||||
|
// active units, and (opt-in) relaunch durable agents. Shared by the
|
||||||
|
// "packages updated" and the "framework drift" paths. Extracted to
|
||||||
|
// update-checker.ts (`runUpdateReseedFlow`) for direct unit testability.
|
||||||
const reseedFramework = (reason: string): void => {
|
const reseedFramework = (reason: string): void => {
|
||||||
console.log(reason);
|
const flow = runUpdateReseedFlow(reason, {
|
||||||
const reseed = runFrameworkReseed();
|
reseed: opts.reseed,
|
||||||
if (!reseed.ok) {
|
relaunch: opts.relaunch,
|
||||||
console.error(
|
allowInactiveEnforcement: opts.allowInactiveEnforcement === true,
|
||||||
`\n⚠ Framework re-seed skipped: ${reseed.reason ?? 'unknown'}.\n` +
|
});
|
||||||
' Activate manually: bash "$(npm root -g)/@mosaicstack/mosaic/framework/install.sh" ' +
|
if (flow.settingsGuard?.ran && flow.settingsGuard.result?.exitCode === 1) {
|
||||||
'(MOSAIC_SYNC_ONLY=1 MOSAIC_INSTALL_MODE=keep)',
|
// Fail-loud: enforcement hooks were refused/stripped. Surface this
|
||||||
);
|
// in the command's own exit status without aborting the rest of
|
||||||
return;
|
// the update (mirrors mosaic-link-runtime-assets' guard_degraded).
|
||||||
}
|
process.exitCode = 1;
|
||||||
console.log('✔ Framework re-seeded.');
|
|
||||||
if (reseed.skillSyncError) {
|
|
||||||
console.error(` ⚠ Claude skill reconciliation skipped: ${reseed.skillSyncError}`);
|
|
||||||
}
|
|
||||||
const skillConflicts = reseed.skillSync?.conflicts ?? [];
|
|
||||||
const skillChanges =
|
|
||||||
(reseed.skillSync?.registered.length ?? 0) + (reseed.skillSync?.repaired.length ?? 0);
|
|
||||||
if (skillChanges > 0) {
|
|
||||||
console.log(`✔ Registered ${skillChanges.toString()} Mosaic skill(s) with Claude Code.`);
|
|
||||||
}
|
|
||||||
for (const conflict of skillConflicts) {
|
|
||||||
console.error(` ⚠ Skill registration skipped for ${conflict.name}: ${conflict.reason}`);
|
|
||||||
}
|
|
||||||
// Propagate shipped systemd unit fixes to the ACTIVE units (re-seed only
|
|
||||||
// touches ~/.config/mosaic/systemd/user; systemd runs ~/.config/systemd/user).
|
|
||||||
const units = refreshActiveFleetUnits();
|
|
||||||
if (units.refreshed.length > 0) {
|
|
||||||
console.log(`✔ Refreshed ${units.refreshed.length} active systemd unit(s).`);
|
|
||||||
}
|
|
||||||
const agents = readRosterAgentNames();
|
|
||||||
if (agents.length === 0) return;
|
|
||||||
if (opts.relaunch) {
|
|
||||||
console.log(`\nRelaunching ${agents.length} fleet agent(s) to pick up the new runtime…`);
|
|
||||||
for (const restart of buildRelaunchCommands(agents)) {
|
|
||||||
try {
|
|
||||||
execSync(restart.join(' '), { stdio: 'inherit', timeout: 30_000 });
|
|
||||||
} catch {
|
|
||||||
console.error(` ⚠ failed to restart agent — run: ${restart.join(' ')}`);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
console.log('✔ Agents relaunched.');
|
|
||||||
} else {
|
|
||||||
console.log(
|
|
||||||
`\nℹ ${agents.length} fleet agent(s) are still running the previous runtime. ` +
|
|
||||||
'Restart them to activate the update:\n mosaic update --relaunch ' +
|
|
||||||
'(or: mosaic fleet restart <agent>)',
|
|
||||||
);
|
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
@@ -539,7 +514,7 @@ program
|
|||||||
// package is reported outdated. Detect that via the framework version and
|
// package is reported outdated. Detect that via the framework version and
|
||||||
// re-seed so shipped launcher/runtime fixes still activate.
|
// re-seed so shipped launcher/runtime fixes still activate.
|
||||||
const drift = checkFrameworkDrift();
|
const drift = checkFrameworkDrift();
|
||||||
if (drift.drifted && opts.reseed !== false) {
|
if (drift.drifted) {
|
||||||
reseedFramework(
|
reseedFramework(
|
||||||
`\nFramework drift detected (on-disk v${drift.installed} < bundled v${drift.bundled}) — ` +
|
`\nFramework drift detected (on-disk v${drift.installed} < bundled v${drift.bundled}) — ` +
|
||||||
'the CLI was updated outside `mosaic update`. Re-seeding framework files into ' +
|
'the CLI was updated outside `mosaic update`. Re-seeding framework files into ' +
|
||||||
@@ -577,7 +552,7 @@ program
|
|||||||
(r: { package: string }) => r.package === FRAMEWORK_RESEED_PACKAGE,
|
(r: { package: string }) => r.package === FRAMEWORK_RESEED_PACKAGE,
|
||||||
);
|
);
|
||||||
const drift = checkFrameworkDrift();
|
const drift = checkFrameworkDrift();
|
||||||
if ((mosaicUpdated || drift.drifted) && opts.reseed !== false) {
|
if (mosaicUpdated || drift.drifted) {
|
||||||
reseedFramework(
|
reseedFramework(
|
||||||
'\nRe-seeding framework files into ~/.config/mosaic (data-safe; keeps your edits)…',
|
'\nRe-seeding framework files into ~/.config/mosaic (data-safe; keeps your edits)…',
|
||||||
);
|
);
|
||||||
|
|||||||
301
packages/mosaic/src/commands/install-ordering-guard.spec.ts
Normal file
301
packages/mosaic/src/commands/install-ordering-guard.spec.ts
Normal file
@@ -0,0 +1,301 @@
|
|||||||
|
import { describe, it, expect, afterEach } from 'vitest';
|
||||||
|
import { mkdtempSync, rmSync, writeFileSync, readFileSync, existsSync } from 'node:fs';
|
||||||
|
import { tmpdir } from 'node:os';
|
||||||
|
import { join } from 'node:path';
|
||||||
|
import { leaseEnforcementActivatable } from './lease-activation-probe.js';
|
||||||
|
import {
|
||||||
|
ENFORCEMENT_HOOK_MARKERS,
|
||||||
|
FAIL_LOUD_MESSAGE,
|
||||||
|
guardClaudeSettingsWiring,
|
||||||
|
loudOptOutMessage,
|
||||||
|
runInstallOrderingGuard,
|
||||||
|
settingsHasEnforcementHooks,
|
||||||
|
stripEnforcementHooks,
|
||||||
|
} from './install-ordering-guard.js';
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Red-first tests for issue #869 Point-1 C2 — the install-ordering guard.
|
||||||
|
*
|
||||||
|
* Root cause under test: `mosaic-link-runtime-assets` copies
|
||||||
|
* `runtime/claude/settings.json` (which embeds the PreToolUse
|
||||||
|
* `mutator-gate.py` hook and the Stop `receipt-observer-client.py` hook)
|
||||||
|
* straight into `~/.claude/settings.json`, unconditionally. If the
|
||||||
|
* activation half (C1: `leaseEnforcementActivatable()`) cannot be confirmed,
|
||||||
|
* wiring those hooks bricks the host with a fail-closed gate that can never
|
||||||
|
* be satisfied. This guard must refuse to wire in that case by default, and
|
||||||
|
* only wire anyway on an explicit, loud opt-out.
|
||||||
|
*
|
||||||
|
* All fixtures use temp directories — this suite never reads or writes the
|
||||||
|
* real `~/.claude/settings.json`.
|
||||||
|
*/
|
||||||
|
|
||||||
|
const FIXTURE_SETTINGS = {
|
||||||
|
model: 'opus',
|
||||||
|
hooks: {
|
||||||
|
PreCompact: [
|
||||||
|
{
|
||||||
|
matcher: '.*',
|
||||||
|
hooks: [{ type: 'command', command: 'python3 revoke-lease.py --reason pre-compact' }],
|
||||||
|
},
|
||||||
|
],
|
||||||
|
PreToolUse: [
|
||||||
|
{
|
||||||
|
matcher: '.*',
|
||||||
|
hooks: [
|
||||||
|
{
|
||||||
|
type: 'command',
|
||||||
|
command: 'python3 ~/.config/mosaic/tools/lease-broker/mutator-gate.py --runtime claude',
|
||||||
|
timeout: 3,
|
||||||
|
},
|
||||||
|
],
|
||||||
|
},
|
||||||
|
{
|
||||||
|
matcher: 'Write|Edit|MultiEdit',
|
||||||
|
hooks: [{ type: 'command', command: '~/.config/mosaic/tools/qa/prevent-memory-write.sh' }],
|
||||||
|
},
|
||||||
|
],
|
||||||
|
PostToolUse: [
|
||||||
|
{
|
||||||
|
matcher: 'Edit|MultiEdit|Write',
|
||||||
|
hooks: [{ type: 'command', command: '~/.config/mosaic/tools/qa/qa-hook-stdin.sh' }],
|
||||||
|
},
|
||||||
|
],
|
||||||
|
Stop: [
|
||||||
|
{
|
||||||
|
hooks: [
|
||||||
|
{
|
||||||
|
type: 'command',
|
||||||
|
command:
|
||||||
|
'python3 ~/.config/mosaic/tools/lease-broker/receipt-observer-client.py --runtime claude',
|
||||||
|
timeout: 3,
|
||||||
|
},
|
||||||
|
{ type: 'command', command: '~/.config/mosaic/tools/qa/reflect-stop-hook.sh' },
|
||||||
|
],
|
||||||
|
},
|
||||||
|
],
|
||||||
|
},
|
||||||
|
enabledPlugins: { 'feature-dev@claude-plugins-official': true },
|
||||||
|
};
|
||||||
|
|
||||||
|
function fixtureJson(): string {
|
||||||
|
return JSON.stringify(FIXTURE_SETTINGS, null, 2) + '\n';
|
||||||
|
}
|
||||||
|
|
||||||
|
describe('stripEnforcementHooks', () => {
|
||||||
|
it('removes the PreToolUse mutator-gate trigger entirely', () => {
|
||||||
|
const { settings } = stripEnforcementHooks(FIXTURE_SETTINGS);
|
||||||
|
const hooks = settings['hooks'] as Record<string, unknown[]>;
|
||||||
|
const preToolUse = hooks['PreToolUse'] as Array<{ hooks: Array<{ command: string }> }>;
|
||||||
|
expect(preToolUse.some((t) => t.hooks.some((h) => h.command.includes('mutator-gate.py')))).toBe(
|
||||||
|
false,
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('preserves the sibling prevent-memory-write.sh PreToolUse trigger', () => {
|
||||||
|
const { settings } = stripEnforcementHooks(FIXTURE_SETTINGS);
|
||||||
|
const hooks = settings['hooks'] as Record<string, unknown[]>;
|
||||||
|
const preToolUse = hooks['PreToolUse'] as Array<{ hooks: Array<{ command: string }> }>;
|
||||||
|
expect(
|
||||||
|
preToolUse.some((t) => t.hooks.some((h) => h.command.includes('prevent-memory-write.sh'))),
|
||||||
|
).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('removes only the receipt-observer-client.py hook from Stop, keeping reflect-stop-hook.sh', () => {
|
||||||
|
const { settings } = stripEnforcementHooks(FIXTURE_SETTINGS);
|
||||||
|
const hooks = settings['hooks'] as Record<string, unknown[]>;
|
||||||
|
const stop = hooks['Stop'] as Array<{ hooks: Array<{ command: string }> }>;
|
||||||
|
const commands = stop.flatMap((t) => t.hooks.map((h) => h.command));
|
||||||
|
expect(commands.some((c) => c.includes('receipt-observer-client.py'))).toBe(false);
|
||||||
|
expect(commands.some((c) => c.includes('reflect-stop-hook.sh'))).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('leaves PreCompact/PostToolUse hooks byte-identical', () => {
|
||||||
|
const { settings } = stripEnforcementHooks(FIXTURE_SETTINGS);
|
||||||
|
const hooks = settings['hooks'] as Record<string, unknown>;
|
||||||
|
expect(hooks['PreCompact']).toEqual(FIXTURE_SETTINGS.hooks.PreCompact);
|
||||||
|
expect(hooks['PostToolUse']).toEqual(FIXTURE_SETTINGS.hooks.PostToolUse);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('reports what it removed', () => {
|
||||||
|
const { removed } = stripEnforcementHooks(FIXTURE_SETTINGS);
|
||||||
|
expect(removed).toContain(`PreToolUse:${ENFORCEMENT_HOOK_MARKERS.preToolUse}`);
|
||||||
|
expect(removed).toContain(`Stop:${ENFORCEMENT_HOOK_MARKERS.stop}`);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('settingsHasEnforcementHooks', () => {
|
||||||
|
it('is true for the unmodified fixture', () => {
|
||||||
|
expect(settingsHasEnforcementHooks(FIXTURE_SETTINGS)).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('is false after stripping', () => {
|
||||||
|
const { settings } = stripEnforcementHooks(FIXTURE_SETTINGS);
|
||||||
|
expect(settingsHasEnforcementHooks(settings)).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('is false for settings with no hooks key at all', () => {
|
||||||
|
expect(settingsHasEnforcementHooks({ model: 'opus' })).toBe(false);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('guardClaudeSettingsWiring', () => {
|
||||||
|
it('probe=false (default, no opt-out): strips enforcement hooks and reports non-zero with a loud, actionable message', () => {
|
||||||
|
const outcome = guardClaudeSettingsWiring(fixtureJson(), {}, { activatable: () => false });
|
||||||
|
|
||||||
|
expect(outcome.exitCode).toBe(1);
|
||||||
|
expect(outcome.wired).toBe(false);
|
||||||
|
expect(settingsHasEnforcementHooks(JSON.parse(outcome.json) as Record<string, unknown>)).toBe(
|
||||||
|
false,
|
||||||
|
);
|
||||||
|
expect(outcome.logs).toHaveLength(1);
|
||||||
|
expect(outcome.logs[0]?.level).toBe('error');
|
||||||
|
expect(outcome.logs[0]?.message).toBe(FAIL_LOUD_MESSAGE);
|
||||||
|
expect(outcome.logs[0]?.message).toMatch(/refusing to wire a dead gate/i);
|
||||||
|
expect(outcome.logs[0]?.message).toMatch(/#869/);
|
||||||
|
expect(outcome.logs[0]?.message).toMatch(/--allow-inactive-enforcement/);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('probe=false + explicit opt-out flag: wires hooks as-is and emits a loud warning', () => {
|
||||||
|
const outcome = guardClaudeSettingsWiring(
|
||||||
|
fixtureJson(),
|
||||||
|
{ allowInactiveEnforcement: true },
|
||||||
|
{ activatable: () => false },
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(outcome.exitCode).toBe(0);
|
||||||
|
expect(outcome.wired).toBe(true);
|
||||||
|
expect(settingsHasEnforcementHooks(JSON.parse(outcome.json) as Record<string, unknown>)).toBe(
|
||||||
|
true,
|
||||||
|
);
|
||||||
|
expect(outcome.logs).toHaveLength(1);
|
||||||
|
expect(outcome.logs[0]?.level).toBe('warn');
|
||||||
|
expect(outcome.logs[0]?.message).toBe(loudOptOutMessage());
|
||||||
|
expect(outcome.logs[0]?.message).toMatch(/WITHOUT confirmed activation/);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('probe=true: wires hooks normally with no logs, regardless of opt-out', () => {
|
||||||
|
const outcome = guardClaudeSettingsWiring(fixtureJson(), {}, { activatable: () => true });
|
||||||
|
|
||||||
|
expect(outcome.exitCode).toBe(0);
|
||||||
|
expect(outcome.wired).toBe(true);
|
||||||
|
expect(outcome.logs).toHaveLength(0);
|
||||||
|
expect(JSON.parse(outcome.json)).toEqual(FIXTURE_SETTINGS);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('probe=true + opt-out flag set anyway: still wires normally, no spurious warning', () => {
|
||||||
|
const outcome = guardClaudeSettingsWiring(
|
||||||
|
fixtureJson(),
|
||||||
|
{ allowInactiveEnforcement: true },
|
||||||
|
{ activatable: () => true },
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(outcome.exitCode).toBe(0);
|
||||||
|
expect(outcome.wired).toBe(true);
|
||||||
|
expect(outcome.logs).toHaveLength(0);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('defaults to the real leaseEnforcementActivatable() when no activatable dep is injected', () => {
|
||||||
|
// Deliberately does not assume a fixed true/false value for the real
|
||||||
|
// probe (whether dist/cli.js happens to be built varies by environment —
|
||||||
|
// asserting a hardcoded expectation here would make the test flaky, not
|
||||||
|
// red-first). Instead it proves the wiring is genuinely delegated: the
|
||||||
|
// no-deps call must agree with an explicit call to the same real
|
||||||
|
// predicate, not some other hardcoded value.
|
||||||
|
const reallyActivatable = leaseEnforcementActivatable();
|
||||||
|
const outcome = guardClaudeSettingsWiring(fixtureJson());
|
||||||
|
|
||||||
|
if (reallyActivatable) {
|
||||||
|
expect(outcome.exitCode).toBe(0);
|
||||||
|
expect(outcome.wired).toBe(true);
|
||||||
|
} else {
|
||||||
|
expect(outcome.exitCode).toBe(1);
|
||||||
|
expect(outcome.wired).toBe(false);
|
||||||
|
}
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('runInstallOrderingGuard (file-level, temp dirs only)', () => {
|
||||||
|
let dir: string;
|
||||||
|
|
||||||
|
afterEach(() => {
|
||||||
|
if (dir) rmSync(dir, { recursive: true, force: true });
|
||||||
|
});
|
||||||
|
|
||||||
|
function makeSrc(): string {
|
||||||
|
dir = mkdtempSync(join(tmpdir(), 'mosaic-install-ordering-guard-'));
|
||||||
|
const src = join(dir, 'settings.json');
|
||||||
|
writeFileSync(src, fixtureJson());
|
||||||
|
return src;
|
||||||
|
}
|
||||||
|
|
||||||
|
it('probe=false: writes a dest settings.json with hooks stripped and returns exitCode 1', () => {
|
||||||
|
const src = makeSrc();
|
||||||
|
const dest = join(dir, 'claude-settings.json');
|
||||||
|
|
||||||
|
const result = runInstallOrderingGuard(src, dest, {}, { activatable: () => false });
|
||||||
|
|
||||||
|
expect(result.exitCode).toBe(1);
|
||||||
|
expect(result.destWritten).toBe(true);
|
||||||
|
expect(existsSync(dest)).toBe(true);
|
||||||
|
const written = JSON.parse(readFileSync(dest, 'utf-8')) as Record<string, unknown>;
|
||||||
|
expect(settingsHasEnforcementHooks(written)).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('probe=false + opt-out: writes dest with hooks intact and returns exitCode 0', () => {
|
||||||
|
const src = makeSrc();
|
||||||
|
const dest = join(dir, 'claude-settings.json');
|
||||||
|
|
||||||
|
const result = runInstallOrderingGuard(
|
||||||
|
src,
|
||||||
|
dest,
|
||||||
|
{ allowInactiveEnforcement: true },
|
||||||
|
{ activatable: () => false },
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(result.exitCode).toBe(0);
|
||||||
|
const written = JSON.parse(readFileSync(dest, 'utf-8')) as Record<string, unknown>;
|
||||||
|
expect(settingsHasEnforcementHooks(written)).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('probe=true: writes dest with hooks intact and returns exitCode 0', () => {
|
||||||
|
const src = makeSrc();
|
||||||
|
const dest = join(dir, 'claude-settings.json');
|
||||||
|
|
||||||
|
const result = runInstallOrderingGuard(src, dest, {}, { activatable: () => true });
|
||||||
|
|
||||||
|
expect(result.exitCode).toBe(0);
|
||||||
|
const written = JSON.parse(readFileSync(dest, 'utf-8')) as Record<string, unknown>;
|
||||||
|
expect(settingsHasEnforcementHooks(written)).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('backs up a pre-existing divergent dest before overwriting (copy_file_managed parity)', () => {
|
||||||
|
const src = makeSrc();
|
||||||
|
const dest = join(dir, 'claude-settings.json');
|
||||||
|
writeFileSync(dest, JSON.stringify({ preexisting: true }));
|
||||||
|
|
||||||
|
const result = runInstallOrderingGuard(src, dest, {}, { activatable: () => true });
|
||||||
|
|
||||||
|
expect(result.destWritten).toBe(true);
|
||||||
|
expect(result.backupPath).toBeDefined();
|
||||||
|
expect(existsSync(result.backupPath!)).toBe(true);
|
||||||
|
expect(JSON.parse(readFileSync(result.backupPath!, 'utf-8'))).toEqual({ preexisting: true });
|
||||||
|
});
|
||||||
|
|
||||||
|
it('is a no-op write when dest already matches the guarded content (idempotent)', () => {
|
||||||
|
const src = makeSrc();
|
||||||
|
const dest = join(dir, 'claude-settings.json');
|
||||||
|
|
||||||
|
const first = runInstallOrderingGuard(src, dest, {}, { activatable: () => true });
|
||||||
|
expect(first.destWritten).toBe(true);
|
||||||
|
|
||||||
|
const second = runInstallOrderingGuard(src, dest, {}, { activatable: () => true });
|
||||||
|
expect(second.destWritten).toBe(false);
|
||||||
|
expect(second.backupPath).toBeUndefined();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('never touches the real home directory settings path used by this test file', () => {
|
||||||
|
// Sanity guard for the suite itself: every dest path used above lives
|
||||||
|
// under the mkdtemp() scratch dir, never under homedir()/.claude.
|
||||||
|
expect(dir).toContain('mosaic-install-ordering-guard-');
|
||||||
|
});
|
||||||
|
});
|
||||||
327
packages/mosaic/src/commands/install-ordering-guard.ts
Normal file
327
packages/mosaic/src/commands/install-ordering-guard.ts
Normal file
@@ -0,0 +1,327 @@
|
|||||||
|
/**
|
||||||
|
* Install-ordering guard (issue #869, Point-1 card C2).
|
||||||
|
*
|
||||||
|
* Root cause this exists to guard against (#828 version skew, restated): the
|
||||||
|
* framework reseed / install path (`framework/install.sh` →
|
||||||
|
* `mosaic-link-runtime-assets` → copies `runtime/claude/settings.json` to
|
||||||
|
* `~/.claude/settings.json`) wires the ENFORCEMENT half of the lease broker —
|
||||||
|
* the `PreToolUse` `mutator-gate.py` hook and the `Stop`
|
||||||
|
* `receipt-observer-client.py` hook — unconditionally. If the ACTIVATION half
|
||||||
|
* (a CLI build advertising launch-runtime activation + a running broker
|
||||||
|
* supervisor — see `lease-activation-probe.ts`, C1) is absent, the fail-closed
|
||||||
|
* gate then denies every tool call with GATE_UNAVAILABLE: a bricked host.
|
||||||
|
*
|
||||||
|
* This module is the WIRING gate, not the enforcement gate: it decides
|
||||||
|
* whether the enforcement hook entries are written into the settings.json
|
||||||
|
* that ships to `~/.claude/`. It never touches `mutator-gate.py`'s own
|
||||||
|
* fail-closed-on-absent-identity runtime behavior (test-locked in
|
||||||
|
* `runtime_tools_unittest.py` / `fail-closed-regression.spec.ts`).
|
||||||
|
*
|
||||||
|
* Default (no opt-out): NOT activatable → strip the enforcement hook entries
|
||||||
|
* from the written settings.json and report a non-zero outcome with a loud,
|
||||||
|
* actionable message (see FAIL_LOUD_MESSAGE below).
|
||||||
|
*
|
||||||
|
* Opt-out: `--allow-inactive-enforcement` (an explicit, per-invocation CLI
|
||||||
|
* flag — deliberately NOT an environment variable, so it can never sit as a
|
||||||
|
* silently-inherited default in a shell profile). When set on a NOT
|
||||||
|
* activatable host, the hooks ARE wired but a loud warning is emitted saying
|
||||||
|
* so, and the outcome is reported ok (this is a conscious, informed choice).
|
||||||
|
*/
|
||||||
|
|
||||||
|
import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
|
||||||
|
import { dirname } from 'node:path';
|
||||||
|
import type { Command } from 'commander';
|
||||||
|
import { leaseEnforcementActivatable } from './lease-activation-probe.js';
|
||||||
|
|
||||||
|
// ─── Enforcement hook identification ────────────────────────────────────────
|
||||||
|
|
||||||
|
/** Substrings that identify the two enforcement hook commands #828 wired
|
||||||
|
* unconditionally. Matches the marker strings documented in
|
||||||
|
* `lease-activation-probe.ts`. */
|
||||||
|
export const ENFORCEMENT_HOOK_MARKERS = {
|
||||||
|
preToolUse: 'mutator-gate.py',
|
||||||
|
stop: 'receipt-observer-client.py',
|
||||||
|
} as const;
|
||||||
|
|
||||||
|
interface HookEntry {
|
||||||
|
command?: string;
|
||||||
|
[key: string]: unknown;
|
||||||
|
}
|
||||||
|
|
||||||
|
interface HookTrigger {
|
||||||
|
matcher?: string;
|
||||||
|
hooks?: HookEntry[];
|
||||||
|
[key: string]: unknown;
|
||||||
|
}
|
||||||
|
|
||||||
|
type HooksMap = Record<string, HookTrigger[]>;
|
||||||
|
|
||||||
|
function cloneJson<T>(value: T): T {
|
||||||
|
return JSON.parse(JSON.stringify(value)) as T;
|
||||||
|
}
|
||||||
|
|
||||||
|
function commandIncludes(hook: HookEntry, marker: string): boolean {
|
||||||
|
return String(hook.command ?? '').includes(marker);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Return a deep clone of `settings` with the enforcement hook entries removed:
|
||||||
|
* - Any `PreToolUse` trigger group containing a `mutator-gate.py` command is
|
||||||
|
* dropped in full (that trigger exists solely to run the gate).
|
||||||
|
* - Within `Stop` trigger groups, only the individual `receipt-observer-client.py`
|
||||||
|
* hook entry is dropped; sibling hooks in the same trigger (e.g.
|
||||||
|
* `reflect-stop-hook.sh`) are preserved.
|
||||||
|
* Every other hook (PreCompact/SessionStart revoke-lease, the
|
||||||
|
* `prevent-memory-write.sh` PreToolUse trigger, PostToolUse qa/typecheck
|
||||||
|
* hooks) is left byte-identical — this function only ever removes the two
|
||||||
|
* markers above.
|
||||||
|
*/
|
||||||
|
export function stripEnforcementHooks(settings: Record<string, unknown>): {
|
||||||
|
settings: Record<string, unknown>;
|
||||||
|
removed: string[];
|
||||||
|
} {
|
||||||
|
const cloned = cloneJson(settings);
|
||||||
|
const removed: string[] = [];
|
||||||
|
const hooks = cloned['hooks'] as HooksMap | undefined;
|
||||||
|
if (!hooks || typeof hooks !== 'object') {
|
||||||
|
return { settings: cloned, removed };
|
||||||
|
}
|
||||||
|
|
||||||
|
const preToolUse = hooks['PreToolUse'];
|
||||||
|
if (Array.isArray(preToolUse)) {
|
||||||
|
const kept = preToolUse.filter((trigger) => {
|
||||||
|
const hasGate = (trigger.hooks ?? []).some((h) =>
|
||||||
|
commandIncludes(h, ENFORCEMENT_HOOK_MARKERS.preToolUse),
|
||||||
|
);
|
||||||
|
if (hasGate) removed.push('PreToolUse:mutator-gate.py');
|
||||||
|
return !hasGate;
|
||||||
|
});
|
||||||
|
if (kept.length > 0) hooks['PreToolUse'] = kept;
|
||||||
|
else delete hooks['PreToolUse'];
|
||||||
|
}
|
||||||
|
|
||||||
|
const stop = hooks['Stop'];
|
||||||
|
if (Array.isArray(stop)) {
|
||||||
|
const rebuilt: HookTrigger[] = [];
|
||||||
|
for (const trigger of stop) {
|
||||||
|
const innerHooks = trigger.hooks ?? [];
|
||||||
|
const keptHooks = innerHooks.filter((h) => {
|
||||||
|
const isReceiptObserver = commandIncludes(h, ENFORCEMENT_HOOK_MARKERS.stop);
|
||||||
|
if (isReceiptObserver) removed.push('Stop:receipt-observer-client.py');
|
||||||
|
return !isReceiptObserver;
|
||||||
|
});
|
||||||
|
if (keptHooks.length > 0) {
|
||||||
|
rebuilt.push({ ...trigger, hooks: keptHooks });
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if (rebuilt.length > 0) hooks['Stop'] = rebuilt;
|
||||||
|
else delete hooks['Stop'];
|
||||||
|
}
|
||||||
|
|
||||||
|
if (Object.keys(hooks).length === 0) {
|
||||||
|
delete cloned['hooks'];
|
||||||
|
} else {
|
||||||
|
cloned['hooks'] = hooks;
|
||||||
|
}
|
||||||
|
|
||||||
|
return { settings: cloned, removed };
|
||||||
|
}
|
||||||
|
|
||||||
|
/** True iff `settings` currently wires either enforcement hook. */
|
||||||
|
export function settingsHasEnforcementHooks(settings: Record<string, unknown>): boolean {
|
||||||
|
const hooks = settings['hooks'] as HooksMap | undefined;
|
||||||
|
if (!hooks || typeof hooks !== 'object') return false;
|
||||||
|
|
||||||
|
const preToolUse = hooks['PreToolUse'] ?? [];
|
||||||
|
const preHit = preToolUse.some((trigger) =>
|
||||||
|
(trigger.hooks ?? []).some((h) => commandIncludes(h, ENFORCEMENT_HOOK_MARKERS.preToolUse)),
|
||||||
|
);
|
||||||
|
if (preHit) return true;
|
||||||
|
|
||||||
|
const stop = hooks['Stop'] ?? [];
|
||||||
|
return stop.some((trigger) =>
|
||||||
|
(trigger.hooks ?? []).some((h) => commandIncludes(h, ENFORCEMENT_HOOK_MARKERS.stop)),
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
// ─── Guard predicate ────────────────────────────────────────────────────────
|
||||||
|
|
||||||
|
export const FAIL_LOUD_MESSAGE =
|
||||||
|
'[mosaic] ERROR: enforcement requested but activation half absent — needs a published CLI ' +
|
||||||
|
'carrying launch-runtime activation + a broker supervisor; refusing to wire a dead gate (see #869). ' +
|
||||||
|
'The PreToolUse mutator-gate.py hook and Stop receipt-observer-client.py hook were NOT written to ' +
|
||||||
|
'settings.json. Fix by installing/updating the CLI and broker, then re-run the framework reseed. ' +
|
||||||
|
'To wire anyway (NOT recommended — the fail-closed gate will deny every tool call with ' +
|
||||||
|
'GATE_UNAVAILABLE until activation is restored), re-run with --allow-inactive-enforcement.';
|
||||||
|
|
||||||
|
export function loudOptOutMessage(): string {
|
||||||
|
return (
|
||||||
|
'[mosaic] WARNING: wiring lease-enforcement hooks (mutator-gate.py / receipt-observer-client.py) ' +
|
||||||
|
'WITHOUT confirmed activation — --allow-inactive-enforcement was set explicitly. The fail-closed ' +
|
||||||
|
'gate will deny every tool call (GATE_UNAVAILABLE) until the activation half (launch-runtime ' +
|
||||||
|
'activation capability + a running broker supervisor) is present on this host (see #869).'
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
export type GuardLogLevel = 'error' | 'warn';
|
||||||
|
|
||||||
|
export interface GuardLogLine {
|
||||||
|
level: GuardLogLevel;
|
||||||
|
message: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface InstallOrderingGuardOptions {
|
||||||
|
/** Explicit, per-invocation opt-out. Never source this from an environment
|
||||||
|
* variable — see module doc. */
|
||||||
|
allowInactiveEnforcement?: boolean;
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface InstallOrderingGuardDeps {
|
||||||
|
/** Defaults to {@link leaseEnforcementActivatable}. Injectable for tests. */
|
||||||
|
activatable?: () => boolean;
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface InstallOrderingGuardOutcome {
|
||||||
|
/** The settings.json content to write (pretty-printed, trailing newline). */
|
||||||
|
json: string;
|
||||||
|
/** Whether the enforcement hooks are present in `json`. */
|
||||||
|
wired: boolean;
|
||||||
|
/** 0 = proceed normally; 1 = enforcement was refused (fail-loud default path). */
|
||||||
|
exitCode: 0 | 1;
|
||||||
|
logs: GuardLogLine[];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The install-ordering guard: decide whether the enforcement hooks embedded
|
||||||
|
* in the Claude settings.json template may be wired into the settings.json
|
||||||
|
* actually shipped to `~/.claude/`.
|
||||||
|
*
|
||||||
|
* - activatable → wire as-is. exitCode 0, no logs.
|
||||||
|
* - NOT activatable, no opt-out → strip enforcement hooks. exitCode 1,
|
||||||
|
* one 'error' log with the actionable FAIL_LOUD_MESSAGE.
|
||||||
|
* - NOT activatable, opt-out set → wire as-is anyway. exitCode 0, one
|
||||||
|
* 'warn' log making the risk explicit and loud.
|
||||||
|
*
|
||||||
|
* Pure function: takes the raw settings.json text, returns the text to write
|
||||||
|
* plus metadata. No filesystem access — callers (the hidden CLI subcommand
|
||||||
|
* below, or a test) own reading/writing so this stays trivially testable with
|
||||||
|
* fakes/temp files and never risks touching a real `~/.claude/settings.json`.
|
||||||
|
*/
|
||||||
|
export function guardClaudeSettingsWiring(
|
||||||
|
rawSettingsJson: string,
|
||||||
|
options: InstallOrderingGuardOptions = {},
|
||||||
|
deps: InstallOrderingGuardDeps = {},
|
||||||
|
): InstallOrderingGuardOutcome {
|
||||||
|
const parsed = JSON.parse(rawSettingsJson) as Record<string, unknown>;
|
||||||
|
const activatable = deps.activatable ?? leaseEnforcementActivatable;
|
||||||
|
const isActivatable = activatable();
|
||||||
|
|
||||||
|
const serialize = (settings: Record<string, unknown>): string =>
|
||||||
|
JSON.stringify(settings, null, 2) + '\n';
|
||||||
|
|
||||||
|
if (isActivatable) {
|
||||||
|
return {
|
||||||
|
json: serialize(parsed),
|
||||||
|
wired: settingsHasEnforcementHooks(parsed),
|
||||||
|
exitCode: 0,
|
||||||
|
logs: [],
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
if (options.allowInactiveEnforcement === true) {
|
||||||
|
return {
|
||||||
|
json: serialize(parsed),
|
||||||
|
wired: settingsHasEnforcementHooks(parsed),
|
||||||
|
exitCode: 0,
|
||||||
|
logs: [{ level: 'warn', message: loudOptOutMessage() }],
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
const { settings: stripped } = stripEnforcementHooks(parsed);
|
||||||
|
return {
|
||||||
|
json: serialize(stripped),
|
||||||
|
wired: settingsHasEnforcementHooks(stripped),
|
||||||
|
exitCode: 1,
|
||||||
|
logs: [{ level: 'error', message: FAIL_LOUD_MESSAGE }],
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
// ─── File-level runner (shared by the CLI action + tests) ──────────────────
|
||||||
|
|
||||||
|
export interface RunInstallOrderingGuardResult extends InstallOrderingGuardOutcome {
|
||||||
|
destWritten: boolean;
|
||||||
|
backupPath?: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Read `src`, guard it, and write the result to `dest` — mirroring
|
||||||
|
* `copy_file_managed`'s backup-on-change semantics from
|
||||||
|
* `mosaic-link-runtime-assets` (skip the write if content is unchanged;
|
||||||
|
* back up an existing divergent file once, timestamped). Exported standalone
|
||||||
|
* (not only reachable via the CLI action closure) so tests can exercise real
|
||||||
|
* file I/O against temp directories without ever touching `~/.claude/`.
|
||||||
|
*/
|
||||||
|
export function runInstallOrderingGuard(
|
||||||
|
src: string,
|
||||||
|
dest: string,
|
||||||
|
options: InstallOrderingGuardOptions = {},
|
||||||
|
deps: InstallOrderingGuardDeps = {},
|
||||||
|
): RunInstallOrderingGuardResult {
|
||||||
|
const raw = readFileSync(src, 'utf-8');
|
||||||
|
const outcome = guardClaudeSettingsWiring(raw, options, deps);
|
||||||
|
|
||||||
|
mkdirSync(dirname(dest), { recursive: true });
|
||||||
|
|
||||||
|
const existing = existsSync(dest) ? readFileSync(dest, 'utf-8') : null;
|
||||||
|
let destWritten = false;
|
||||||
|
let backupPath: string | undefined;
|
||||||
|
|
||||||
|
if (existing !== outcome.json) {
|
||||||
|
if (existing !== null) {
|
||||||
|
const stamp = new Date()
|
||||||
|
.toISOString()
|
||||||
|
.replace(/[-:]/g, '')
|
||||||
|
.replace(/\..+$/, '')
|
||||||
|
.replace('T', '');
|
||||||
|
backupPath = `${dest}.mosaic-bak-${stamp}`;
|
||||||
|
writeFileSync(backupPath, existing);
|
||||||
|
}
|
||||||
|
writeFileSync(dest, outcome.json);
|
||||||
|
destWritten = true;
|
||||||
|
}
|
||||||
|
|
||||||
|
return { ...outcome, destWritten, backupPath };
|
||||||
|
}
|
||||||
|
|
||||||
|
// ─── Hidden CLI bridge (bash → TS) ──────────────────────────────────────────
|
||||||
|
|
||||||
|
/** Hidden CLI subcommand name. `mosaic-link-runtime-assets` (bash) invokes
|
||||||
|
* this instead of its generic `copy_file_managed` for the settings.json
|
||||||
|
* runtime file specifically, so the guard's decision is made by importing
|
||||||
|
* `leaseEnforcementActivatable()` directly rather than re-implementing the
|
||||||
|
* capability/supervisor probes in shell. Deliberately undocumented (hidden
|
||||||
|
* from `--help`) — internal wiring, not a user-facing command. */
|
||||||
|
export const INSTALL_ORDERING_GUARD_COMMAND = '__link-claude-settings';
|
||||||
|
|
||||||
|
export function registerInstallOrderingGuardCommand(program: Command): void {
|
||||||
|
program
|
||||||
|
.command(`${INSTALL_ORDERING_GUARD_COMMAND} <src> <dest>`, { hidden: true })
|
||||||
|
.description(
|
||||||
|
'Internal: copy the Claude settings.json template, gating enforcement-hook ' +
|
||||||
|
'wiring on lease-activation capability (#869 Point-1 C2)',
|
||||||
|
)
|
||||||
|
.option(
|
||||||
|
'--allow-inactive-enforcement',
|
||||||
|
'Wire enforcement hooks even when activation cannot be confirmed on this host ' +
|
||||||
|
'(explicit, loud, non-default opt-out — see #869)',
|
||||||
|
)
|
||||||
|
.action((src: string, dest: string, opts: { allowInactiveEnforcement?: boolean }) => {
|
||||||
|
const result = runInstallOrderingGuard(src, dest, {
|
||||||
|
allowInactiveEnforcement: opts.allowInactiveEnforcement === true,
|
||||||
|
});
|
||||||
|
for (const line of result.logs) {
|
||||||
|
(line.level === 'error' ? console.error : console.warn)(line.message);
|
||||||
|
}
|
||||||
|
process.exit(result.exitCode);
|
||||||
|
});
|
||||||
|
}
|
||||||
@@ -47,6 +47,22 @@ const piLifecyclePath = join(frameworkRoot, 'runtime/pi/lease-lifecycle.ts');
|
|||||||
const prdyInitPath = join(frameworkRoot, 'tools/prdy/prdy-init.sh');
|
const prdyInitPath = join(frameworkRoot, 'tools/prdy/prdy-init.sh');
|
||||||
const prdyUpdatePath = join(frameworkRoot, 'tools/prdy/prdy-update.sh');
|
const prdyUpdatePath = join(frameworkRoot, 'tools/prdy/prdy-update.sh');
|
||||||
const remediationHandlerPath = join(frameworkRoot, 'tools/qa/remediation-hook-handler.sh');
|
const remediationHandlerPath = join(frameworkRoot, 'tools/qa/remediation-hook-handler.sh');
|
||||||
|
|
||||||
|
// C4 (#869 Point-1): launch-runtime.py now asserts, before anything else,
|
||||||
|
// that the CLI's advertised lease-activation capability (normally read via
|
||||||
|
// the hidden `mosaic __lease-capability` subcommand) matches what
|
||||||
|
// enforcement expects — see framework/tools/lease-broker/
|
||||||
|
// activation_version_gate.py. This suite drives launch-runtime.py directly
|
||||||
|
// as a subprocess (never through the real `mosaic` CLI), so — exactly like
|
||||||
|
// the fake broker (daemon.py) and fake `claude` binaries already used
|
||||||
|
// below — it must supply a fake activation-capability probe rather than
|
||||||
|
// depend on a real `mosaic` binary being on PATH. `MOSAIC_LEASE_VERSION_PROBE_COMMAND`
|
||||||
|
// is launch-runtime.py's injection point for that fake; this literal
|
||||||
|
// {name, version} pair must be kept in sync with
|
||||||
|
// `EXPECTED_ACTIVATION_CAPABILITY` (activation_version_gate.py) and
|
||||||
|
// `LEASE_ACTIVATION_CAPABILITY` (lease-activation-probe.ts) — all three
|
||||||
|
// currently agree on v1.
|
||||||
|
const leaseCapabilityProbeStub = `python3 -c "import json; print(json.dumps({'name': 'lease-runtime-activation', 'version': 1}))"`;
|
||||||
const children: ChildProcess[] = [];
|
const children: ChildProcess[] = [];
|
||||||
const temporaryRoots: string[] = [];
|
const temporaryRoots: string[] = [];
|
||||||
|
|
||||||
@@ -184,6 +200,7 @@ raise SystemExit(0 if len(session_id) == 64 and denied else 1)
|
|||||||
MOSAIC_PRDY_RUNTIME: 'claude',
|
MOSAIC_PRDY_RUNTIME: 'claude',
|
||||||
MOSAIC_LEASE_BROKER_SOCKET: socket,
|
MOSAIC_LEASE_BROKER_SOCKET: socket,
|
||||||
MOSAIC_RUNTIME_GENERATION: '1',
|
MOSAIC_RUNTIME_GENERATION: '1',
|
||||||
|
MOSAIC_LEASE_VERSION_PROBE_COMMAND: leaseCapabilityProbeStub,
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
@@ -743,6 +760,7 @@ describe('whole mutator-class lease gate', () => {
|
|||||||
...process.env,
|
...process.env,
|
||||||
MOSAIC_LEASE_BROKER_SOCKET: socket,
|
MOSAIC_LEASE_BROKER_SOCKET: socket,
|
||||||
MOSAIC_RUNTIME_GENERATION: '1',
|
MOSAIC_RUNTIME_GENERATION: '1',
|
||||||
|
MOSAIC_LEASE_VERSION_PROBE_COMMAND: leaseCapabilityProbeStub,
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
);
|
);
|
||||||
@@ -761,6 +779,7 @@ describe('whole mutator-class lease gate', () => {
|
|||||||
...process.env,
|
...process.env,
|
||||||
MOSAIC_LEASE_BROKER_SOCKET: join(tmpdir(), 'missing-mosaic-broker.sock'),
|
MOSAIC_LEASE_BROKER_SOCKET: join(tmpdir(), 'missing-mosaic-broker.sock'),
|
||||||
MOSAIC_RUNTIME_GENERATION: '1',
|
MOSAIC_RUNTIME_GENERATION: '1',
|
||||||
|
MOSAIC_LEASE_VERSION_PROBE_COMMAND: leaseCapabilityProbeStub,
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
);
|
);
|
||||||
@@ -878,6 +897,7 @@ raise SystemExit(0 if len(session_id) == 64 and hook_present and observers_prese
|
|||||||
PATH: `${binDir}:${process.env.PATH ?? ''}`,
|
PATH: `${binDir}:${process.env.PATH ?? ''}`,
|
||||||
MOSAIC_LEASE_BROKER_SOCKET: socket,
|
MOSAIC_LEASE_BROKER_SOCKET: socket,
|
||||||
MOSAIC_RUNTIME_GENERATION: '1',
|
MOSAIC_RUNTIME_GENERATION: '1',
|
||||||
|
MOSAIC_LEASE_VERSION_PROBE_COMMAND: leaseCapabilityProbeStub,
|
||||||
},
|
},
|
||||||
proxyGate: () =>
|
proxyGate: () =>
|
||||||
Promise.resolve({
|
Promise.resolve({
|
||||||
|
|||||||
@@ -39,6 +39,18 @@ LAUNCHER = load_tool("lease_runtime_launcher", "launch-runtime.py")
|
|||||||
GATE = load_tool("lease_mutator_gate", "mutator-gate.py")
|
GATE = load_tool("lease_mutator_gate", "mutator-gate.py")
|
||||||
|
|
||||||
|
|
||||||
|
def matching_activation_probe(*_args: object, **_kwargs: object) -> dict[str, object]:
|
||||||
|
"""Fake activation-capability probe matching what enforcement expects
|
||||||
|
(C4, #869 Point-1). Injected into `LAUNCHER.main()` calls below that are
|
||||||
|
exercising OTHER branches (registration, exec, generation init, ...) so
|
||||||
|
the new version-coupling gate — which runs before those — never blocks
|
||||||
|
on host state (no real `mosaic` CLI on PATH in a test sandbox). The
|
||||||
|
version-coupling gate's OWN behavior (match/mismatch/absent) is covered
|
||||||
|
by its dedicated red-first tests in `version_coupling_unittest.py`."""
|
||||||
|
|
||||||
|
return dict(LAUNCHER.EXPECTED_ACTIVATION_CAPABILITY)
|
||||||
|
|
||||||
|
|
||||||
class FakeSocket:
|
class FakeSocket:
|
||||||
def __init__(self, *chunks: bytes):
|
def __init__(self, *chunks: bytes):
|
||||||
self.chunks = list(chunks)
|
self.chunks = list(chunks)
|
||||||
@@ -95,6 +107,7 @@ class LaunchRuntimeTest(unittest.TestCase):
|
|||||||
request=request,
|
request=request,
|
||||||
execute=execute,
|
execute=execute,
|
||||||
initialize_generation=initialize_generation,
|
initialize_generation=initialize_generation,
|
||||||
|
probe_activation_capability=matching_activation_probe,
|
||||||
)
|
)
|
||||||
|
|
||||||
self.assertEqual(result, 0)
|
self.assertEqual(result, 0)
|
||||||
@@ -127,6 +140,7 @@ class LaunchRuntimeTest(unittest.TestCase):
|
|||||||
request=lambda *_args: {"ok": True, "session_id": "e" * 64},
|
request=lambda *_args: {"ok": True, "session_id": "e" * 64},
|
||||||
execute=lambda *args: executed.append(args),
|
execute=lambda *args: executed.append(args),
|
||||||
initialize_generation=lambda *_args: None,
|
initialize_generation=lambda *_args: None,
|
||||||
|
probe_activation_capability=matching_activation_probe,
|
||||||
)
|
)
|
||||||
self.assertEqual(result, 0)
|
self.assertEqual(result, 0)
|
||||||
self.assertEqual(
|
self.assertEqual(
|
||||||
@@ -153,6 +167,7 @@ class LaunchRuntimeTest(unittest.TestCase):
|
|||||||
request=lambda *_args: {"ok": True, "session_id": "f" * 64},
|
request=lambda *_args: {"ok": True, "session_id": "f" * 64},
|
||||||
execute=lambda *args: executed.append(args),
|
execute=lambda *args: executed.append(args),
|
||||||
initialize_generation=lambda *_args: None,
|
initialize_generation=lambda *_args: None,
|
||||||
|
probe_activation_capability=matching_activation_probe,
|
||||||
)
|
)
|
||||||
self.assertEqual(result, 0)
|
self.assertEqual(result, 0)
|
||||||
self.assertEqual(executed[0][0:2], ("pi", ["pi", "--print", "hello"]))
|
self.assertEqual(executed[0][0:2], ("pi", ["pi", "--print", "hello"]))
|
||||||
@@ -188,6 +203,7 @@ class LaunchRuntimeTest(unittest.TestCase):
|
|||||||
environ=environment,
|
environ=environment,
|
||||||
request=lambda *_args, value=reply: value,
|
request=lambda *_args, value=reply: value,
|
||||||
execute=lambda *args: executed.append(args),
|
execute=lambda *args: executed.append(args),
|
||||||
|
probe_activation_capability=matching_activation_probe,
|
||||||
)
|
)
|
||||||
self.assertEqual(result, 1)
|
self.assertEqual(result, 1)
|
||||||
self.assertEqual(executed, [])
|
self.assertEqual(executed, [])
|
||||||
@@ -203,6 +219,7 @@ class LaunchRuntimeTest(unittest.TestCase):
|
|||||||
initialize_generation=lambda *_args: (_ for _ in ()).throw(
|
initialize_generation=lambda *_args: (_ for _ in ()).throw(
|
||||||
OSError("unsafe state")
|
OSError("unsafe state")
|
||||||
),
|
),
|
||||||
|
probe_activation_capability=matching_activation_probe,
|
||||||
),
|
),
|
||||||
1,
|
1,
|
||||||
)
|
)
|
||||||
@@ -220,6 +237,7 @@ class LaunchRuntimeTest(unittest.TestCase):
|
|||||||
environ={"MOSAIC_LEASE_BROKER_SOCKET": "/x"},
|
environ={"MOSAIC_LEASE_BROKER_SOCKET": "/x"},
|
||||||
request=request,
|
request=request,
|
||||||
execute=lambda *_args: self.fail("must not execute"),
|
execute=lambda *_args: self.fail("must not execute"),
|
||||||
|
probe_activation_capability=matching_activation_probe,
|
||||||
),
|
),
|
||||||
1,
|
1,
|
||||||
)
|
)
|
||||||
@@ -233,6 +251,7 @@ class LaunchRuntimeTest(unittest.TestCase):
|
|||||||
request=lambda *_args: {"ok": True, "session_id": "c" * 64},
|
request=lambda *_args: {"ok": True, "session_id": "c" * 64},
|
||||||
execute=lambda *_args: (_ for _ in ()).throw(OSError("missing")),
|
execute=lambda *_args: (_ for _ in ()).throw(OSError("missing")),
|
||||||
initialize_generation=lambda *_args: None,
|
initialize_generation=lambda *_args: None,
|
||||||
|
probe_activation_capability=matching_activation_probe,
|
||||||
),
|
),
|
||||||
1,
|
1,
|
||||||
)
|
)
|
||||||
|
|||||||
287
packages/mosaic/src/mutator-gate/version_coupling_unittest.py
Normal file
287
packages/mosaic/src/mutator-gate/version_coupling_unittest.py
Normal file
@@ -0,0 +1,287 @@
|
|||||||
|
#!/usr/bin/env python3
|
||||||
|
"""Red-first tests for issue #869 Point-1 C4 — the enforcement/activation
|
||||||
|
version-coupling assertion at the `launch-runtime.py` seam.
|
||||||
|
|
||||||
|
Root cause under test (#828 restated): the lease broker's ENFORCEMENT half
|
||||||
|
(this toolkit) and its ACTIVATION half (`execLeaseGatedRuntime()` in
|
||||||
|
`launch.ts`, chained through `launch-runtime.py`) shipped on different
|
||||||
|
channels and drifted. C1 (`lease-activation-probe.ts`) gave the activation
|
||||||
|
half a versioned, machine-checkable identity
|
||||||
|
(`LEASE_ACTIVATION_CAPABILITY`, printed via the hidden CLI subcommand
|
||||||
|
`mosaic __lease-capability`). C4 (this module + `activation_version_gate.py`)
|
||||||
|
is the assertion that actually USES that identity: enforcement must refuse
|
||||||
|
to proceed — loudly, with an actionable remediation message, never a
|
||||||
|
silent pass — unless the activation capability it observes exactly matches
|
||||||
|
what enforcement expects.
|
||||||
|
|
||||||
|
Every case here drives the seam with injected fakes/stubs (a fake
|
||||||
|
`probe_activation_capability` callable at the `launch-runtime.py` level, or
|
||||||
|
a fake `run` transport at the `activation_version_gate` level) — never a
|
||||||
|
real broker, a real installed CLI, or a real `mosaic` binary on PATH.
|
||||||
|
"""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import importlib.util
|
||||||
|
import io
|
||||||
|
import subprocess
|
||||||
|
import sys
|
||||||
|
import unittest
|
||||||
|
from contextlib import redirect_stderr
|
||||||
|
from pathlib import Path
|
||||||
|
|
||||||
|
|
||||||
|
TOOLS_DIR = Path(__file__).parents[2] / "framework/tools/lease-broker"
|
||||||
|
if str(TOOLS_DIR) not in sys.path:
|
||||||
|
sys.path.insert(0, str(TOOLS_DIR))
|
||||||
|
|
||||||
|
|
||||||
|
def load_tool(module_name: str, filename: str):
|
||||||
|
spec = importlib.util.spec_from_file_location(module_name, TOOLS_DIR / filename)
|
||||||
|
if spec is None or spec.loader is None:
|
||||||
|
raise RuntimeError(f"unable to load {filename}")
|
||||||
|
module = importlib.util.module_from_spec(spec)
|
||||||
|
spec.loader.exec_module(module)
|
||||||
|
return module
|
||||||
|
|
||||||
|
|
||||||
|
# Loaded under distinct module names from runtime_tools_unittest.py's own
|
||||||
|
# LAUNCHER/GATE loads — importlib.util.module_from_spec() gives each load a
|
||||||
|
# fresh module object regardless of name collisions, but distinct names keep
|
||||||
|
# tracebacks/debugging unambiguous when both files run in the same process.
|
||||||
|
LAUNCHER = load_tool("lease_runtime_launcher_version_coupling", "launch-runtime.py")
|
||||||
|
VERSION_GATE = load_tool("lease_activation_version_gate_test", "activation_version_gate.py")
|
||||||
|
|
||||||
|
|
||||||
|
def matching_capability() -> dict[str, object]:
|
||||||
|
return dict(VERSION_GATE.EXPECTED_ACTIVATION_CAPABILITY)
|
||||||
|
|
||||||
|
|
||||||
|
class AssertActivationCapabilityMatchesTest(unittest.TestCase):
|
||||||
|
"""Unit-level coverage of `activation_version_gate.py`'s own assertion,
|
||||||
|
isolated from the launch-runtime.py seam it is wired into below."""
|
||||||
|
|
||||||
|
def test_matching_capability_passes_silently(self) -> None:
|
||||||
|
VERSION_GATE.assert_activation_capability_matches(matching_capability())
|
||||||
|
# No exception is the assertion; nothing further to check.
|
||||||
|
|
||||||
|
def test_absent_capability_fails_closed_not_silent_pass(self) -> None:
|
||||||
|
with self.assertRaises(VERSION_GATE.VersionCouplingError) as raised:
|
||||||
|
VERSION_GATE.assert_activation_capability_matches(None)
|
||||||
|
message = str(raised.exception)
|
||||||
|
self.assertIn("#869", message)
|
||||||
|
self.assertIn("upgrade", message.lower())
|
||||||
|
|
||||||
|
def test_version_mismatch_message_is_actionable(self) -> None:
|
||||||
|
expected = {"name": "lease-runtime-activation", "version": 1}
|
||||||
|
mismatched = {"name": "lease-runtime-activation", "version": 2}
|
||||||
|
with self.assertRaises(VERSION_GATE.VersionCouplingError) as raised:
|
||||||
|
VERSION_GATE.assert_activation_capability_matches(mismatched, expected)
|
||||||
|
message = str(raised.exception)
|
||||||
|
self.assertIn("v2", message)
|
||||||
|
self.assertIn("v1", message)
|
||||||
|
self.assertIn("#869", message)
|
||||||
|
self.assertIn("upgrade", message.lower())
|
||||||
|
self.assertIn("version skew", message.lower())
|
||||||
|
|
||||||
|
def test_name_mismatch_fails_loud(self) -> None:
|
||||||
|
expected = {"name": "lease-runtime-activation", "version": 1}
|
||||||
|
mismatched = {"name": "some-other-capability", "version": 1}
|
||||||
|
with self.assertRaises(VERSION_GATE.VersionCouplingError) as raised:
|
||||||
|
VERSION_GATE.assert_activation_capability_matches(mismatched, expected)
|
||||||
|
message = str(raised.exception)
|
||||||
|
self.assertIn("some-other-capability", message)
|
||||||
|
self.assertIn("lease-runtime-activation", message)
|
||||||
|
self.assertIn("#869", message)
|
||||||
|
|
||||||
|
def test_reversed_drift_newer_activation_than_enforcement_expects_also_fails(self) -> None:
|
||||||
|
# A build/deploy where ACTIVATION shipped ahead of ENFORCEMENT is
|
||||||
|
# exactly as much version skew as the reverse (#828's actual shape
|
||||||
|
# was enforcement ahead of activation) — the assertion must not special
|
||||||
|
# case direction.
|
||||||
|
expected = {"name": "lease-runtime-activation", "version": 1}
|
||||||
|
newer_activation = {"name": "lease-runtime-activation", "version": 2}
|
||||||
|
with self.assertRaises(VERSION_GATE.VersionCouplingError):
|
||||||
|
VERSION_GATE.assert_activation_capability_matches(newer_activation, expected)
|
||||||
|
|
||||||
|
|
||||||
|
class ProbeActivationCapabilityTest(unittest.TestCase):
|
||||||
|
"""Coverage of the probe's command resolution and fail-closed transport
|
||||||
|
handling — never spawns a real `mosaic` process."""
|
||||||
|
|
||||||
|
def test_returns_none_when_mosaic_is_not_resolvable_on_path(self) -> None:
|
||||||
|
result = VERSION_GATE.default_probe_activation_capability(
|
||||||
|
{"PATH": "/nonexistent-bin-dir-for-869-c4-test"}
|
||||||
|
)
|
||||||
|
self.assertIsNone(result)
|
||||||
|
|
||||||
|
def test_override_command_is_parsed_and_the_probe_subcommand_is_not_double_appended(
|
||||||
|
self,
|
||||||
|
) -> None:
|
||||||
|
captured: list[list[str]] = []
|
||||||
|
|
||||||
|
class FakeCompleted:
|
||||||
|
returncode = 0
|
||||||
|
stdout = '{"name": "lease-runtime-activation", "version": 1}'
|
||||||
|
|
||||||
|
def fake_run(argv: list[str], **_kwargs: object) -> FakeCompleted:
|
||||||
|
captured.append(argv)
|
||||||
|
return FakeCompleted()
|
||||||
|
|
||||||
|
result = VERSION_GATE.default_probe_activation_capability(
|
||||||
|
{VERSION_GATE.MOSAIC_COMMAND_OVERRIDE_VAR: "/fake/mosaic __lease-capability"},
|
||||||
|
run=fake_run,
|
||||||
|
)
|
||||||
|
self.assertEqual(result, {"name": "lease-runtime-activation", "version": 1})
|
||||||
|
self.assertEqual(captured, [["/fake/mosaic", "__lease-capability"]])
|
||||||
|
|
||||||
|
def test_fails_closed_on_nonzero_exit_malformed_json_and_missing_fields(self) -> None:
|
||||||
|
class NonZeroExit:
|
||||||
|
returncode = 1
|
||||||
|
stdout = '{"name": "lease-runtime-activation", "version": 1}'
|
||||||
|
|
||||||
|
class MalformedOutput:
|
||||||
|
returncode = 0
|
||||||
|
stdout = "not-json"
|
||||||
|
|
||||||
|
class MissingVersion:
|
||||||
|
returncode = 0
|
||||||
|
stdout = '{"name": "lease-runtime-activation"}'
|
||||||
|
|
||||||
|
class WrongShapeVersion:
|
||||||
|
returncode = 0
|
||||||
|
stdout = '{"name": "lease-runtime-activation", "version": "1"}'
|
||||||
|
|
||||||
|
class BooleanVersion:
|
||||||
|
# bool is a subclass of int in Python; must not be accepted as
|
||||||
|
# a version number.
|
||||||
|
returncode = 0
|
||||||
|
stdout = '{"name": "lease-runtime-activation", "version": true}'
|
||||||
|
|
||||||
|
for fake in (
|
||||||
|
NonZeroExit(),
|
||||||
|
MalformedOutput(),
|
||||||
|
MissingVersion(),
|
||||||
|
WrongShapeVersion(),
|
||||||
|
BooleanVersion(),
|
||||||
|
):
|
||||||
|
with self.subTest(stdout=fake.stdout, returncode=fake.returncode):
|
||||||
|
result = VERSION_GATE.default_probe_activation_capability(
|
||||||
|
{VERSION_GATE.MOSAIC_COMMAND_OVERRIDE_VAR: "/fake/mosaic"},
|
||||||
|
run=lambda *_a, fake=fake, **_kw: fake,
|
||||||
|
)
|
||||||
|
self.assertIsNone(result)
|
||||||
|
|
||||||
|
def test_fails_closed_on_timeout_and_transport_error(self) -> None:
|
||||||
|
def timeout_run(*_args: object, **_kwargs: object) -> None:
|
||||||
|
raise subprocess.TimeoutExpired(cmd="mosaic", timeout=2.0)
|
||||||
|
|
||||||
|
def oserror_run(*_args: object, **_kwargs: object) -> None:
|
||||||
|
raise OSError("no such file or directory")
|
||||||
|
|
||||||
|
for run_fake in (timeout_run, oserror_run):
|
||||||
|
with self.subTest(run=run_fake.__name__):
|
||||||
|
result = VERSION_GATE.default_probe_activation_capability(
|
||||||
|
{VERSION_GATE.MOSAIC_COMMAND_OVERRIDE_VAR: "/fake/mosaic"},
|
||||||
|
run=run_fake,
|
||||||
|
)
|
||||||
|
self.assertIsNone(result)
|
||||||
|
|
||||||
|
|
||||||
|
class LaunchRuntimeVersionCouplingSeamTest(unittest.TestCase):
|
||||||
|
"""End-to-end (still fully faked) coverage of the seam as wired into
|
||||||
|
`launch-runtime.py`'s `main()` — the strongest natural enforcement point
|
||||||
|
per the C4 card, run before any broker registration."""
|
||||||
|
|
||||||
|
def _run(self, *, probe):
|
||||||
|
calls: dict[str, object] = {}
|
||||||
|
|
||||||
|
def request(_path: Path, payload: dict[str, object]) -> dict[str, object]:
|
||||||
|
calls["registered"] = True
|
||||||
|
calls["request"] = payload
|
||||||
|
return {"ok": True, "session_id": "a" * 64}
|
||||||
|
|
||||||
|
def execute(command: str, argv: list[str], environment: dict[str, str]) -> None:
|
||||||
|
calls["executed"] = (command, argv, environment)
|
||||||
|
|
||||||
|
def initialize_generation(_path: Path, _generation: int) -> None:
|
||||||
|
calls["generation_initialized"] = True
|
||||||
|
|
||||||
|
stderr = io.StringIO()
|
||||||
|
with redirect_stderr(stderr):
|
||||||
|
result = LAUNCHER.main(
|
||||||
|
["--runtime", "claude", "--", "claude", "--print", "hello"],
|
||||||
|
environ={"MOSAIC_LEASE_BROKER_SOCKET": "/run/test/broker.sock"},
|
||||||
|
request=request,
|
||||||
|
execute=execute,
|
||||||
|
initialize_generation=initialize_generation,
|
||||||
|
probe_activation_capability=probe,
|
||||||
|
)
|
||||||
|
return result, stderr.getvalue(), calls
|
||||||
|
|
||||||
|
def test_matching_activation_version_passes_and_the_gate_proceeds(self) -> None:
|
||||||
|
result, stderr_text, calls = self._run(probe=lambda *_a, **_kw: matching_capability())
|
||||||
|
self.assertEqual(result, 0)
|
||||||
|
self.assertEqual(stderr_text, "")
|
||||||
|
self.assertTrue(calls.get("registered"))
|
||||||
|
self.assertIn("executed", calls)
|
||||||
|
|
||||||
|
def test_version_mismatch_fails_loud_denies_and_never_registers_or_execs(self) -> None:
|
||||||
|
expected = LAUNCHER.EXPECTED_ACTIVATION_CAPABILITY
|
||||||
|
mismatched = {"name": expected["name"], "version": expected["version"] + 1}
|
||||||
|
result, stderr_text, calls = self._run(probe=lambda *_a, **_kw: mismatched)
|
||||||
|
|
||||||
|
self.assertEqual(result, LAUNCHER.EXIT_VERSION_SKEW)
|
||||||
|
self.assertNotEqual(result, 0)
|
||||||
|
self.assertIn("#869", stderr_text)
|
||||||
|
self.assertIn(f"v{mismatched['version']}", stderr_text)
|
||||||
|
self.assertIn(f"v{expected['version']}", stderr_text)
|
||||||
|
self.assertIn("upgrade", stderr_text.lower())
|
||||||
|
# Never reaches broker registration or exec — the version gate is a
|
||||||
|
# hard stop, not advisory.
|
||||||
|
self.assertNotIn("registered", calls)
|
||||||
|
self.assertNotIn("executed", calls)
|
||||||
|
|
||||||
|
def test_name_mismatch_fails_loud(self) -> None:
|
||||||
|
expected = LAUNCHER.EXPECTED_ACTIVATION_CAPABILITY
|
||||||
|
mismatched = {"name": "some-other-capability", "version": expected["version"]}
|
||||||
|
result, stderr_text, calls = self._run(probe=lambda *_a, **_kw: mismatched)
|
||||||
|
|
||||||
|
self.assertEqual(result, LAUNCHER.EXIT_VERSION_SKEW)
|
||||||
|
self.assertIn("#869", stderr_text)
|
||||||
|
self.assertIn("some-other-capability", stderr_text)
|
||||||
|
self.assertNotIn("registered", calls)
|
||||||
|
self.assertNotIn("executed", calls)
|
||||||
|
|
||||||
|
def test_absent_activation_capability_fails_closed_not_a_silent_pass(self) -> None:
|
||||||
|
result, stderr_text, calls = self._run(probe=lambda *_a, **_kw: None)
|
||||||
|
|
||||||
|
self.assertEqual(result, LAUNCHER.EXIT_VERSION_SKEW)
|
||||||
|
self.assertNotEqual(result, 0)
|
||||||
|
self.assertIn("#869", stderr_text)
|
||||||
|
self.assertNotIn("registered", calls)
|
||||||
|
self.assertNotIn("executed", calls)
|
||||||
|
|
||||||
|
def test_version_gate_runs_before_and_independently_of_broker_registration(self) -> None:
|
||||||
|
def request_must_not_be_called(*_args: object, **_kwargs: object) -> dict[str, object]:
|
||||||
|
self.fail("broker must not be contacted when activation version is mismatched")
|
||||||
|
|
||||||
|
stderr = io.StringIO()
|
||||||
|
with redirect_stderr(stderr):
|
||||||
|
result = LAUNCHER.main(
|
||||||
|
["--runtime", "claude", "--", "claude"],
|
||||||
|
environ={"MOSAIC_LEASE_BROKER_SOCKET": "/run/test/broker.sock"},
|
||||||
|
request=request_must_not_be_called,
|
||||||
|
probe_activation_capability=lambda *_a, **_kw: None,
|
||||||
|
)
|
||||||
|
self.assertEqual(result, LAUNCHER.EXIT_VERSION_SKEW)
|
||||||
|
|
||||||
|
def test_dedicated_exit_code_never_collides_with_usage_or_registration_codes(self) -> None:
|
||||||
|
# Distinctness guard: a version-skew denial must never be mistaken
|
||||||
|
# for the pre-existing usage error (64) or registration/exec
|
||||||
|
# fail-closed code (1) this script already owns.
|
||||||
|
self.assertNotIn(LAUNCHER.EXIT_VERSION_SKEW, (0, 1, 64))
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
unittest.main()
|
||||||
@@ -0,0 +1,424 @@
|
|||||||
|
import { describe, it, expect, afterEach, vi } from 'vitest';
|
||||||
|
import { existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
|
||||||
|
import { tmpdir } from 'node:os';
|
||||||
|
import { join } from 'node:path';
|
||||||
|
import {
|
||||||
|
ENFORCEMENT_HOOK_MARKERS,
|
||||||
|
FAIL_LOUD_MESSAGE,
|
||||||
|
settingsHasEnforcementHooks,
|
||||||
|
} from '../commands/install-ordering-guard.js';
|
||||||
|
import {
|
||||||
|
runUpdatePathSettingsGuard,
|
||||||
|
runUpdateReseedFlow,
|
||||||
|
type FrameworkReseedResult,
|
||||||
|
} from './update-checker.js';
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Red-first tests for issue #882 (b) — the `mosaic update --sync-only`
|
||||||
|
* install-ordering-guard bypass (Mos-ruled "Option C").
|
||||||
|
*
|
||||||
|
* Root cause under test: `runFrameworkReseed()` runs the package's
|
||||||
|
* install.sh with MOSAIC_SYNC_ONLY=1, which exits after the file-system
|
||||||
|
* phase, BEFORE the "Post-install tasks" step that would otherwise run
|
||||||
|
* `mosaic-link-runtime-assets` — the only place the #869 Point-1 C2
|
||||||
|
* install-ordering guard evaluated whether the lease-enforcement hooks
|
||||||
|
* (PreToolUse mutator-gate.py / Stop receipt-observer-client.py) may be
|
||||||
|
* wired into `~/.claude/settings.json`. A plain `mosaic update` therefore
|
||||||
|
* never re-evaluated that decision. These tests prove the post-reseed step
|
||||||
|
* added to close that gap (`runUpdatePathSettingsGuard`, wired into the
|
||||||
|
* `mosaic update` reseed flow via `runUpdateReseedFlow`) reuses the EXACT
|
||||||
|
* C2 guard — no forked logic — and is skipped only when `--no-reseed` means
|
||||||
|
* there was nothing to re-seed/re-link in the first place.
|
||||||
|
*
|
||||||
|
* All fixtures use temp directories — this suite never reads or writes the
|
||||||
|
* real `~/.claude/settings.json` or `~/.config/mosaic`.
|
||||||
|
*/
|
||||||
|
|
||||||
|
const FIXTURE_SETTINGS = {
|
||||||
|
model: 'opus',
|
||||||
|
hooks: {
|
||||||
|
PreToolUse: [
|
||||||
|
{
|
||||||
|
matcher: '.*',
|
||||||
|
hooks: [
|
||||||
|
{
|
||||||
|
type: 'command',
|
||||||
|
command: 'python3 ~/.config/mosaic/tools/lease-broker/mutator-gate.py --runtime claude',
|
||||||
|
timeout: 3,
|
||||||
|
},
|
||||||
|
],
|
||||||
|
},
|
||||||
|
],
|
||||||
|
Stop: [
|
||||||
|
{
|
||||||
|
hooks: [
|
||||||
|
{
|
||||||
|
type: 'command',
|
||||||
|
command:
|
||||||
|
'python3 ~/.config/mosaic/tools/lease-broker/receipt-observer-client.py --runtime claude',
|
||||||
|
timeout: 3,
|
||||||
|
},
|
||||||
|
],
|
||||||
|
},
|
||||||
|
],
|
||||||
|
},
|
||||||
|
};
|
||||||
|
|
||||||
|
function fixtureJson(): string {
|
||||||
|
return JSON.stringify(FIXTURE_SETTINGS, null, 2) + '\n';
|
||||||
|
}
|
||||||
|
|
||||||
|
describe('runUpdatePathSettingsGuard', () => {
|
||||||
|
let root: string;
|
||||||
|
let mosaicHome: string;
|
||||||
|
let claudeHome: string;
|
||||||
|
|
||||||
|
afterEach(() => {
|
||||||
|
if (root) rmSync(root, { recursive: true, force: true });
|
||||||
|
});
|
||||||
|
|
||||||
|
function makeTemplate(): void {
|
||||||
|
root = mkdtempSync(join(tmpdir(), 'mosaic-update-settings-guard-'));
|
||||||
|
mosaicHome = join(root, 'mosaic-home');
|
||||||
|
claudeHome = join(root, 'claude-home');
|
||||||
|
mkdirSync(join(mosaicHome, 'runtime', 'claude'), { recursive: true });
|
||||||
|
writeFileSync(join(mosaicHome, 'runtime', 'claude', 'settings.json'), fixtureJson());
|
||||||
|
}
|
||||||
|
|
||||||
|
it('does not run when there is no settings.json template to re-link', () => {
|
||||||
|
root = mkdtempSync(join(tmpdir(), 'mosaic-update-settings-guard-'));
|
||||||
|
mosaicHome = join(root, 'mosaic-home');
|
||||||
|
claudeHome = join(root, 'claude-home');
|
||||||
|
// Deliberately no runtime/claude/settings.json under mosaicHome.
|
||||||
|
|
||||||
|
const outcome = runUpdatePathSettingsGuard(mosaicHome, claudeHome);
|
||||||
|
|
||||||
|
expect(outcome.ran).toBe(false);
|
||||||
|
expect(outcome.result).toBeUndefined();
|
||||||
|
expect(existsSync(join(claudeHome, 'settings.json'))).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('activatable=false (default, no opt-out): strips enforcement hooks and fails loud, exactly as install-time', () => {
|
||||||
|
makeTemplate();
|
||||||
|
|
||||||
|
const outcome = runUpdatePathSettingsGuard(
|
||||||
|
mosaicHome,
|
||||||
|
claudeHome,
|
||||||
|
{},
|
||||||
|
{ activatable: () => false },
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(outcome.ran).toBe(true);
|
||||||
|
expect(outcome.result?.exitCode).toBe(1);
|
||||||
|
expect(outcome.result?.wired).toBe(false);
|
||||||
|
expect(outcome.result?.logs).toHaveLength(1);
|
||||||
|
expect(outcome.result?.logs[0]?.level).toBe('error');
|
||||||
|
expect(outcome.result?.logs[0]?.message).toBe(FAIL_LOUD_MESSAGE);
|
||||||
|
|
||||||
|
const written = JSON.parse(readFileSync(join(claudeHome, 'settings.json'), 'utf-8')) as Record<
|
||||||
|
string,
|
||||||
|
unknown
|
||||||
|
>;
|
||||||
|
expect(settingsHasEnforcementHooks(written)).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('activatable=true: wires hooks normally, no strip, no logs', () => {
|
||||||
|
makeTemplate();
|
||||||
|
|
||||||
|
const outcome = runUpdatePathSettingsGuard(
|
||||||
|
mosaicHome,
|
||||||
|
claudeHome,
|
||||||
|
{},
|
||||||
|
{ activatable: () => true },
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(outcome.ran).toBe(true);
|
||||||
|
expect(outcome.result?.exitCode).toBe(0);
|
||||||
|
expect(outcome.result?.wired).toBe(true);
|
||||||
|
expect(outcome.result?.logs).toHaveLength(0);
|
||||||
|
|
||||||
|
const written = JSON.parse(readFileSync(join(claudeHome, 'settings.json'), 'utf-8')) as Record<
|
||||||
|
string,
|
||||||
|
unknown
|
||||||
|
>;
|
||||||
|
expect(settingsHasEnforcementHooks(written)).toBe(true);
|
||||||
|
expect(written).toEqual(FIXTURE_SETTINGS);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('activatable=false + --allow-inactive-enforcement: wires hooks anyway with a loud warning', () => {
|
||||||
|
makeTemplate();
|
||||||
|
|
||||||
|
const outcome = runUpdatePathSettingsGuard(
|
||||||
|
mosaicHome,
|
||||||
|
claudeHome,
|
||||||
|
{ allowInactiveEnforcement: true },
|
||||||
|
{ activatable: () => false },
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(outcome.ran).toBe(true);
|
||||||
|
expect(outcome.result?.exitCode).toBe(0);
|
||||||
|
expect(outcome.result?.wired).toBe(true);
|
||||||
|
expect(outcome.result?.logs).toHaveLength(1);
|
||||||
|
expect(outcome.result?.logs[0]?.level).toBe('warn');
|
||||||
|
expect(outcome.result?.logs[0]?.message).toMatch(/WITHOUT confirmed activation/);
|
||||||
|
|
||||||
|
const written = JSON.parse(readFileSync(join(claudeHome, 'settings.json'), 'utf-8')) as Record<
|
||||||
|
string,
|
||||||
|
unknown
|
||||||
|
>;
|
||||||
|
expect(settingsHasEnforcementHooks(written)).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('never touches the real home directory settings path used by this test file', () => {
|
||||||
|
// Sanity guard for the suite itself.
|
||||||
|
makeTemplate();
|
||||||
|
expect(mosaicHome).toContain('mosaic-update-settings-guard-');
|
||||||
|
expect(claudeHome).toContain('mosaic-update-settings-guard-');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('runUpdateReseedFlow (the `mosaic update` post-reseed guard wiring, #882 (b))', () => {
|
||||||
|
const okReseed: FrameworkReseedResult = { ok: true };
|
||||||
|
|
||||||
|
it('--no-reseed: the reseed is never attempted and the settings guard is never invoked', () => {
|
||||||
|
const doReseed = vi.fn(() => okReseed);
|
||||||
|
const doGuard = vi.fn(() => ({ ran: true }));
|
||||||
|
const doRefresh = vi.fn(() => ({ refreshed: [], ok: true }));
|
||||||
|
const doReadRoster = vi.fn(() => []);
|
||||||
|
const log = vi.fn();
|
||||||
|
const warnLog = vi.fn();
|
||||||
|
const errorLog = vi.fn();
|
||||||
|
|
||||||
|
const result = runUpdateReseedFlow(
|
||||||
|
'should never be printed',
|
||||||
|
{ reseed: false },
|
||||||
|
{
|
||||||
|
runFrameworkReseed: doReseed,
|
||||||
|
runUpdatePathSettingsGuard: doGuard,
|
||||||
|
refreshActiveFleetUnits: doRefresh,
|
||||||
|
readRosterAgentNames: doReadRoster,
|
||||||
|
log,
|
||||||
|
warnLog,
|
||||||
|
errorLog,
|
||||||
|
},
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(result.attempted).toBe(false);
|
||||||
|
expect(doReseed).not.toHaveBeenCalled();
|
||||||
|
expect(doGuard).not.toHaveBeenCalled();
|
||||||
|
expect(log).not.toHaveBeenCalled();
|
||||||
|
expect(errorLog).not.toHaveBeenCalled();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('reseed ran + activatable=false: the guard fires (hooks stripped) and the fail-loud message is surfaced, not swallowed', () => {
|
||||||
|
const doReseed = vi.fn(() => okReseed);
|
||||||
|
const doGuard = vi.fn(() => ({
|
||||||
|
ran: true,
|
||||||
|
result: {
|
||||||
|
json: '{}',
|
||||||
|
wired: false,
|
||||||
|
exitCode: 1 as const,
|
||||||
|
logs: [{ level: 'error' as const, message: FAIL_LOUD_MESSAGE }],
|
||||||
|
destWritten: true,
|
||||||
|
},
|
||||||
|
}));
|
||||||
|
const doRefresh = vi.fn(() => ({ refreshed: [], ok: true }));
|
||||||
|
const doReadRoster = vi.fn(() => []);
|
||||||
|
const log = vi.fn();
|
||||||
|
const warnLog = vi.fn();
|
||||||
|
const errorLog = vi.fn();
|
||||||
|
|
||||||
|
const result = runUpdateReseedFlow(
|
||||||
|
'Re-seeding…',
|
||||||
|
{ reseed: true },
|
||||||
|
{
|
||||||
|
runFrameworkReseed: doReseed,
|
||||||
|
runUpdatePathSettingsGuard: doGuard,
|
||||||
|
refreshActiveFleetUnits: doRefresh,
|
||||||
|
readRosterAgentNames: doReadRoster,
|
||||||
|
log,
|
||||||
|
warnLog,
|
||||||
|
errorLog,
|
||||||
|
},
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(result.attempted).toBe(true);
|
||||||
|
expect(doReseed).toHaveBeenCalledTimes(1);
|
||||||
|
expect(doGuard).toHaveBeenCalledTimes(1);
|
||||||
|
expect(result.settingsGuard?.result?.exitCode).toBe(1);
|
||||||
|
// The guard's fail-loud message must reach the operator (stderr), never swallowed.
|
||||||
|
expect(errorLog).toHaveBeenCalledWith(FAIL_LOUD_MESSAGE);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('reseed ran + activatable=true: the guard wires hooks with no error output', () => {
|
||||||
|
const doReseed = vi.fn(() => okReseed);
|
||||||
|
const doGuard = vi.fn(() => ({
|
||||||
|
ran: true,
|
||||||
|
result: {
|
||||||
|
json: '{}',
|
||||||
|
wired: true,
|
||||||
|
exitCode: 0 as const,
|
||||||
|
logs: [],
|
||||||
|
destWritten: true,
|
||||||
|
},
|
||||||
|
}));
|
||||||
|
const doRefresh = vi.fn(() => ({ refreshed: [], ok: true }));
|
||||||
|
const doReadRoster = vi.fn(() => []);
|
||||||
|
const log = vi.fn();
|
||||||
|
const warnLog = vi.fn();
|
||||||
|
const errorLog = vi.fn();
|
||||||
|
|
||||||
|
const result = runUpdateReseedFlow(
|
||||||
|
'Re-seeding…',
|
||||||
|
{ reseed: true },
|
||||||
|
{
|
||||||
|
runFrameworkReseed: doReseed,
|
||||||
|
runUpdatePathSettingsGuard: doGuard,
|
||||||
|
refreshActiveFleetUnits: doRefresh,
|
||||||
|
readRosterAgentNames: doReadRoster,
|
||||||
|
log,
|
||||||
|
warnLog,
|
||||||
|
errorLog,
|
||||||
|
},
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(result.attempted).toBe(true);
|
||||||
|
expect(result.settingsGuard?.result?.exitCode).toBe(0);
|
||||||
|
expect(errorLog).not.toHaveBeenCalled();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('threads --allow-inactive-enforcement through to the settings guard', () => {
|
||||||
|
const doReseed = vi.fn(() => okReseed);
|
||||||
|
const doGuard = vi.fn(() => ({
|
||||||
|
ran: true,
|
||||||
|
result: {
|
||||||
|
json: '{}',
|
||||||
|
wired: true,
|
||||||
|
exitCode: 0 as const,
|
||||||
|
logs: [{ level: 'warn' as const, message: 'opt-out warning' }],
|
||||||
|
destWritten: true,
|
||||||
|
},
|
||||||
|
}));
|
||||||
|
const doRefresh = vi.fn(() => ({ refreshed: [], ok: true }));
|
||||||
|
const doReadRoster = vi.fn(() => []);
|
||||||
|
const warnLog = vi.fn();
|
||||||
|
|
||||||
|
runUpdateReseedFlow(
|
||||||
|
'Re-seeding…',
|
||||||
|
{ reseed: true, allowInactiveEnforcement: true },
|
||||||
|
{
|
||||||
|
runFrameworkReseed: doReseed,
|
||||||
|
runUpdatePathSettingsGuard: doGuard,
|
||||||
|
refreshActiveFleetUnits: doRefresh,
|
||||||
|
readRosterAgentNames: doReadRoster,
|
||||||
|
log: vi.fn(),
|
||||||
|
warnLog,
|
||||||
|
errorLog: vi.fn(),
|
||||||
|
},
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(doGuard).toHaveBeenCalledWith(undefined, undefined, {
|
||||||
|
allowInactiveEnforcement: true,
|
||||||
|
});
|
||||||
|
expect(warnLog).toHaveBeenCalledWith('opt-out warning');
|
||||||
|
});
|
||||||
|
|
||||||
|
it('reseed failure: the settings guard is not invoked (nothing was re-seeded to re-link)', () => {
|
||||||
|
const doReseed = vi.fn(
|
||||||
|
() => ({ ok: false, reason: 'installer not found' }) as FrameworkReseedResult,
|
||||||
|
);
|
||||||
|
const doGuard = vi.fn(() => ({ ran: true }));
|
||||||
|
const doRefresh = vi.fn(() => ({ refreshed: [], ok: true }));
|
||||||
|
const doReadRoster = vi.fn(() => []);
|
||||||
|
const errorLog = vi.fn();
|
||||||
|
|
||||||
|
const result = runUpdateReseedFlow(
|
||||||
|
'Re-seeding…',
|
||||||
|
{ reseed: true },
|
||||||
|
{
|
||||||
|
runFrameworkReseed: doReseed,
|
||||||
|
runUpdatePathSettingsGuard: doGuard,
|
||||||
|
refreshActiveFleetUnits: doRefresh,
|
||||||
|
readRosterAgentNames: doReadRoster,
|
||||||
|
log: vi.fn(),
|
||||||
|
warnLog: vi.fn(),
|
||||||
|
errorLog,
|
||||||
|
},
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(result.attempted).toBe(true);
|
||||||
|
expect(result.settingsGuard).toBeUndefined();
|
||||||
|
expect(doGuard).not.toHaveBeenCalled();
|
||||||
|
expect(errorLog).toHaveBeenCalledWith(expect.stringContaining('Framework re-seed skipped'));
|
||||||
|
});
|
||||||
|
|
||||||
|
it('end-to-end (real runUpdatePathSettingsGuard, real temp files): reseed ok + activatable=false strips hooks in the live settings.json path', () => {
|
||||||
|
const root = mkdtempSync(join(tmpdir(), 'mosaic-update-reseed-flow-e2e-'));
|
||||||
|
try {
|
||||||
|
const mosaicHome = join(root, 'mosaic-home');
|
||||||
|
const claudeHome = join(root, 'claude-home');
|
||||||
|
mkdirSync(join(mosaicHome, 'runtime', 'claude'), { recursive: true });
|
||||||
|
writeFileSync(join(mosaicHome, 'runtime', 'claude', 'settings.json'), fixtureJson());
|
||||||
|
// Pre-existing (stale, install-time) settings.json still carrying the
|
||||||
|
// enforcement hooks — this is the exact state #882 (b) left behind.
|
||||||
|
mkdirSync(claudeHome, { recursive: true });
|
||||||
|
writeFileSync(join(claudeHome, 'settings.json'), fixtureJson());
|
||||||
|
|
||||||
|
const errorLog = vi.fn();
|
||||||
|
const result = runUpdateReseedFlow(
|
||||||
|
'Re-seeding…',
|
||||||
|
{ reseed: true },
|
||||||
|
{
|
||||||
|
runFrameworkReseed: () => okReseed,
|
||||||
|
runUpdatePathSettingsGuard: (mh, ch, options, deps) =>
|
||||||
|
// Exercise the REAL function (imported above), pointed at temp dirs,
|
||||||
|
// with the activation probe faked to prove this is not a live-host test.
|
||||||
|
runUpdatePathSettingsGuardWithFakeActivation(
|
||||||
|
mh ?? mosaicHome,
|
||||||
|
ch ?? claudeHome,
|
||||||
|
options,
|
||||||
|
deps,
|
||||||
|
),
|
||||||
|
refreshActiveFleetUnits: () => ({ refreshed: [], ok: true }),
|
||||||
|
readRosterAgentNames: () => [],
|
||||||
|
log: vi.fn(),
|
||||||
|
warnLog: vi.fn(),
|
||||||
|
errorLog,
|
||||||
|
},
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(result.settingsGuard?.result?.exitCode).toBe(1);
|
||||||
|
const written = JSON.parse(
|
||||||
|
readFileSync(join(claudeHome, 'settings.json'), 'utf-8'),
|
||||||
|
) as Record<string, unknown>;
|
||||||
|
expect(settingsHasEnforcementHooks(written)).toBe(false);
|
||||||
|
expect(errorLog).toHaveBeenCalledWith(FAIL_LOUD_MESSAGE);
|
||||||
|
} finally {
|
||||||
|
rmSync(root, { recursive: true, force: true });
|
||||||
|
}
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
function runUpdatePathSettingsGuardWithFakeActivation(
|
||||||
|
mosaicHome: string,
|
||||||
|
claudeHome: string,
|
||||||
|
options: Parameters<typeof runUpdatePathSettingsGuard>[2],
|
||||||
|
_deps: Parameters<typeof runUpdatePathSettingsGuard>[3],
|
||||||
|
): ReturnType<typeof runUpdatePathSettingsGuard> {
|
||||||
|
return runUpdatePathSettingsGuard(mosaicHome, claudeHome, options, { activatable: () => false });
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Sanity check: the enforcement markers this suite exercises must match the
|
||||||
|
* ones the C2 guard (`install-ordering-guard.ts`) actually looks for, so a
|
||||||
|
* drift in either module's marker strings would fail this suite loudly
|
||||||
|
* rather than silently passing on the wrong hooks.
|
||||||
|
*/
|
||||||
|
describe('marker parity with the C2 guard', () => {
|
||||||
|
it('the fixture uses the same marker commands the guard matches on', () => {
|
||||||
|
const preToolUse = FIXTURE_SETTINGS.hooks.PreToolUse[0]?.hooks[0]?.command ?? '';
|
||||||
|
const stop = FIXTURE_SETTINGS.hooks.Stop[0]?.hooks[0]?.command ?? '';
|
||||||
|
expect(preToolUse).toContain(ENFORCEMENT_HOOK_MARKERS.preToolUse);
|
||||||
|
expect(stop).toContain(ENFORCEMENT_HOOK_MARKERS.stop);
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -44,6 +44,12 @@ import {
|
|||||||
readRegularFileSecure,
|
readRegularFileSecure,
|
||||||
} from '../fleet/secure-file.js';
|
} from '../fleet/secure-file.js';
|
||||||
import { getDefaultSkillPaths, syncClaudeSkills, type SkillSyncResult } from '../commands/skill.js';
|
import { getDefaultSkillPaths, syncClaudeSkills, type SkillSyncResult } from '../commands/skill.js';
|
||||||
|
import {
|
||||||
|
runInstallOrderingGuard,
|
||||||
|
type InstallOrderingGuardDeps,
|
||||||
|
type InstallOrderingGuardOptions,
|
||||||
|
type RunInstallOrderingGuardResult,
|
||||||
|
} from '../commands/install-ordering-guard.js';
|
||||||
|
|
||||||
// ─── Types ──────────────────────────────────────────────────────────────────
|
// ─── Types ──────────────────────────────────────────────────────────────────
|
||||||
|
|
||||||
@@ -908,6 +914,175 @@ export function runFrameworkReseed(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// ─── Post-reseed install-ordering guard (#882, Point-2 precondition) ────────
|
||||||
|
//
|
||||||
|
// Root cause (restated): `runFrameworkReseed` above runs the package's
|
||||||
|
// install.sh with MOSAIC_SYNC_ONLY=1, which — by design (see install.sh) —
|
||||||
|
// exits after the file-system phase, BEFORE the "Post-install tasks" step
|
||||||
|
// that runs `mosaic-link-runtime-assets`. That script is where the #869
|
||||||
|
// Point-1 C2 install-ordering guard (`runInstallOrderingGuard`,
|
||||||
|
// `packages/mosaic/src/commands/install-ordering-guard.ts`) decides whether
|
||||||
|
// the lease-enforcement hooks (PreToolUse mutator-gate.py / Stop
|
||||||
|
// receipt-observer-client.py) get wired into `~/.claude/settings.json`. A
|
||||||
|
// plain `mosaic update` reseed therefore never re-evaluated that wiring
|
||||||
|
// decision against current activation state — the bypass this closes.
|
||||||
|
//
|
||||||
|
// `runUpdatePathSettingsGuard` re-applies the EXACT SAME guard (no forked
|
||||||
|
// logic) against the MANAGED settings.json template the reseed just
|
||||||
|
// refreshed (`<mosaicHome>/runtime/claude/settings.json`) and the live
|
||||||
|
// `<claudeHome>/settings.json` — mirroring `copy_claude_settings_guarded`'s
|
||||||
|
// src/dest pair in `mosaic-link-runtime-assets`.
|
||||||
|
|
||||||
|
export interface UpdatePathSettingsGuardResult {
|
||||||
|
/** False when there is no settings.json template on disk to re-link (e.g. a
|
||||||
|
* framework layout that predates runtime/claude/settings.json) — nothing to
|
||||||
|
* guard, so the guard did not run. */
|
||||||
|
ran: boolean;
|
||||||
|
result?: RunInstallOrderingGuardResult;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function runUpdatePathSettingsGuard(
|
||||||
|
mosaicHome = join(homedir(), '.config', 'mosaic'),
|
||||||
|
claudeHome = process.env['CLAUDE_HOME'] ?? join(homedir(), '.claude'),
|
||||||
|
options: InstallOrderingGuardOptions = {},
|
||||||
|
deps: InstallOrderingGuardDeps = {},
|
||||||
|
): UpdatePathSettingsGuardResult {
|
||||||
|
const src = join(mosaicHome, 'runtime', 'claude', 'settings.json');
|
||||||
|
if (!existsSync(src)) {
|
||||||
|
return { ran: false };
|
||||||
|
}
|
||||||
|
const dest = join(claudeHome, 'settings.json');
|
||||||
|
return { ran: true, result: runInstallOrderingGuard(src, dest, options, deps) };
|
||||||
|
}
|
||||||
|
|
||||||
|
// ─── update-reseed flow (extracted for testability; called from cli.ts) ────
|
||||||
|
//
|
||||||
|
// Everything `mosaic update`'s `.action()` does once it has decided a reseed
|
||||||
|
// should happen (both call sites already gate on `opts.reseed !== false`
|
||||||
|
// before invoking this). Extracted out of cli.ts so the post-reseed guard
|
||||||
|
// wiring (#882 (b)) — and the `--no-reseed` short-circuit — are directly unit
|
||||||
|
// testable with injected fakes, matching the existing update-checker
|
||||||
|
// conventions (see update-checker.reseed.spec.ts).
|
||||||
|
|
||||||
|
export interface UpdateReseedFlowOptions {
|
||||||
|
/** Mirrors the CLI's `--no-reseed` flag (commander sets `reseed: false`
|
||||||
|
* when passed). `false` is a pure no-op: nothing is reseeded and the
|
||||||
|
* post-reseed settings guard is not invoked either — there is nothing to
|
||||||
|
* re-link. */
|
||||||
|
reseed?: boolean;
|
||||||
|
relaunch?: boolean;
|
||||||
|
/** Threads `--allow-inactive-enforcement` to the post-reseed settings
|
||||||
|
* guard, identically to the install path (see install-ordering-guard.ts).
|
||||||
|
* Never sourced from an environment variable — explicit per-invocation
|
||||||
|
* opt-out only. */
|
||||||
|
allowInactiveEnforcement?: boolean;
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface UpdateReseedFlowDeps {
|
||||||
|
runFrameworkReseed?: typeof runFrameworkReseed;
|
||||||
|
runUpdatePathSettingsGuard?: typeof runUpdatePathSettingsGuard;
|
||||||
|
refreshActiveFleetUnits?: typeof refreshActiveFleetUnits;
|
||||||
|
readRosterAgentNames?: typeof readRosterAgentNames;
|
||||||
|
execSync?: typeof execSync;
|
||||||
|
log?: (message: string) => void;
|
||||||
|
warnLog?: (message: string) => void;
|
||||||
|
errorLog?: (message: string) => void;
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface UpdateReseedFlowResult {
|
||||||
|
/** Whether a reseed was actually attempted (false only for `--no-reseed`). */
|
||||||
|
attempted: boolean;
|
||||||
|
reseed?: FrameworkReseedResult;
|
||||||
|
settingsGuard?: UpdatePathSettingsGuardResult;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function runUpdateReseedFlow(
|
||||||
|
reason: string,
|
||||||
|
options: UpdateReseedFlowOptions = {},
|
||||||
|
deps: UpdateReseedFlowDeps = {},
|
||||||
|
): UpdateReseedFlowResult {
|
||||||
|
if (options.reseed === false) {
|
||||||
|
// Nothing to re-seed, and therefore nothing to re-link/guard either.
|
||||||
|
return { attempted: false };
|
||||||
|
}
|
||||||
|
|
||||||
|
const log = deps.log ?? console.log;
|
||||||
|
const warnLog = deps.warnLog ?? console.warn;
|
||||||
|
const errorLog = deps.errorLog ?? console.error;
|
||||||
|
const doReseed = deps.runFrameworkReseed ?? runFrameworkReseed;
|
||||||
|
const doGuard = deps.runUpdatePathSettingsGuard ?? runUpdatePathSettingsGuard;
|
||||||
|
const doRefresh = deps.refreshActiveFleetUnits ?? refreshActiveFleetUnits;
|
||||||
|
const doReadRoster = deps.readRosterAgentNames ?? readRosterAgentNames;
|
||||||
|
const exec = deps.execSync ?? execSync;
|
||||||
|
|
||||||
|
log(reason);
|
||||||
|
const reseed = doReseed();
|
||||||
|
if (!reseed.ok) {
|
||||||
|
errorLog(
|
||||||
|
`\n⚠ Framework re-seed skipped: ${reseed.reason ?? 'unknown'}.\n` +
|
||||||
|
' Activate manually: bash "$(npm root -g)/@mosaicstack/mosaic/framework/install.sh" ' +
|
||||||
|
'(MOSAIC_SYNC_ONLY=1 MOSAIC_INSTALL_MODE=keep)',
|
||||||
|
);
|
||||||
|
return { attempted: true, reseed };
|
||||||
|
}
|
||||||
|
log('✔ Framework re-seeded.');
|
||||||
|
if (reseed.skillSyncError) {
|
||||||
|
errorLog(` ⚠ Claude skill reconciliation skipped: ${reseed.skillSyncError}`);
|
||||||
|
}
|
||||||
|
const skillConflicts = reseed.skillSync?.conflicts ?? [];
|
||||||
|
const skillChanges =
|
||||||
|
(reseed.skillSync?.registered.length ?? 0) + (reseed.skillSync?.repaired.length ?? 0);
|
||||||
|
if (skillChanges > 0) {
|
||||||
|
log(`✔ Registered ${skillChanges.toString()} Mosaic skill(s) with Claude Code.`);
|
||||||
|
}
|
||||||
|
for (const conflict of skillConflicts) {
|
||||||
|
errorLog(` ⚠ Skill registration skipped for ${conflict.name}: ${conflict.reason}`);
|
||||||
|
}
|
||||||
|
|
||||||
|
// #882 (b): re-apply the install-ordering guard (C2) to the MANAGED
|
||||||
|
// settings.json the reseed just refreshed. install.sh's sync-only mode
|
||||||
|
// never reaches the post-install step that would otherwise do this, so
|
||||||
|
// `mosaic update` must do it itself — closing the bypass for every update
|
||||||
|
// path. Never swallow the guard's fail-loud/opt-out output on this path.
|
||||||
|
const settingsGuard = doGuard(undefined, undefined, {
|
||||||
|
allowInactiveEnforcement: options.allowInactiveEnforcement === true,
|
||||||
|
});
|
||||||
|
if (settingsGuard.ran && settingsGuard.result) {
|
||||||
|
for (const line of settingsGuard.result.logs) {
|
||||||
|
(line.level === 'error' ? errorLog : warnLog)(line.message);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Propagate shipped systemd unit fixes to the ACTIVE units (re-seed only
|
||||||
|
// touches ~/.config/mosaic/systemd/user; systemd runs ~/.config/systemd/user).
|
||||||
|
const units = doRefresh();
|
||||||
|
if (units.refreshed.length > 0) {
|
||||||
|
log(`✔ Refreshed ${units.refreshed.length} active systemd unit(s).`);
|
||||||
|
}
|
||||||
|
const agents = doReadRoster();
|
||||||
|
if (agents.length > 0) {
|
||||||
|
if (options.relaunch) {
|
||||||
|
log(`\nRelaunching ${agents.length} fleet agent(s) to pick up the new runtime…`);
|
||||||
|
for (const restart of buildRelaunchCommands(agents)) {
|
||||||
|
try {
|
||||||
|
exec(restart.join(' '), { stdio: 'inherit', timeout: 30_000 });
|
||||||
|
} catch {
|
||||||
|
errorLog(` ⚠ failed to restart agent — run: ${restart.join(' ')}`);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
log('✔ Agents relaunched.');
|
||||||
|
} else {
|
||||||
|
log(
|
||||||
|
`\nℹ ${agents.length} fleet agent(s) are still running the previous runtime. ` +
|
||||||
|
'Restart them to activate the update:\n mosaic update --relaunch ' +
|
||||||
|
'(or: mosaic fleet restart <agent>)',
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return { attempted: true, reseed, settingsGuard };
|
||||||
|
}
|
||||||
|
|
||||||
// ─── Framework drift detection (#642) ────────────────────────────────────────
|
// ─── Framework drift detection (#642) ────────────────────────────────────────
|
||||||
//
|
//
|
||||||
// `mosaic update` only re-seeds the framework when the @mosaicstack/mosaic
|
// `mosaic update` only re-seeds the framework when the @mosaicstack/mosaic
|
||||||
|
|||||||
@@ -13,22 +13,37 @@ import {
|
|||||||
type SkillSyncResult as ClaudeSkillSyncResult,
|
type SkillSyncResult as ClaudeSkillSyncResult,
|
||||||
} from '../commands/skill.js';
|
} from '../commands/skill.js';
|
||||||
|
|
||||||
function linkRuntimeAssets(mosaicHome: string, skipClaudeHooks: boolean): void {
|
/**
|
||||||
|
* Link runtime assets. Returns a warning string when the install-ordering
|
||||||
|
* guard (#869 Point-1 C2) reported a degraded outcome — i.e. the
|
||||||
|
* lease-enforcement hooks were NOT wired into ~/.claude/settings.json because
|
||||||
|
* this host could not confirm it can activate them — so the caller can
|
||||||
|
* surface it via `p.warn(...)` instead of it being swallowed by `stdio:
|
||||||
|
* 'pipe'`. Non-fatal either way: the wizard always continues.
|
||||||
|
*/
|
||||||
|
function linkRuntimeAssets(mosaicHome: string, skipClaudeHooks: boolean): string | undefined {
|
||||||
const script = join(mosaicHome, 'bin', 'mosaic-link-runtime-assets');
|
const script = join(mosaicHome, 'bin', 'mosaic-link-runtime-assets');
|
||||||
if (existsSync(script)) {
|
if (!existsSync(script)) return undefined;
|
||||||
try {
|
try {
|
||||||
spawnSync('bash', [script], {
|
const result = spawnSync('bash', [script], {
|
||||||
timeout: 30000,
|
timeout: 30000,
|
||||||
stdio: 'pipe',
|
stdio: 'pipe',
|
||||||
|
encoding: 'utf-8',
|
||||||
env: {
|
env: {
|
||||||
...process.env,
|
...process.env,
|
||||||
...(skipClaudeHooks ? { MOSAIC_SKIP_CLAUDE_HOOKS: '1' } : {}),
|
...(skipClaudeHooks ? { MOSAIC_SKIP_CLAUDE_HOOKS: '1' } : {}),
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
|
if (result.status !== 0) {
|
||||||
|
const stderr = (result.stderr ?? '').trim();
|
||||||
|
return (
|
||||||
|
stderr || 'Runtime asset linking reported a non-zero exit (see mosaic doctor for details).'
|
||||||
|
);
|
||||||
|
}
|
||||||
} catch {
|
} catch {
|
||||||
// Non-fatal: wizard continues
|
// Non-fatal: wizard continues
|
||||||
}
|
}
|
||||||
}
|
return undefined;
|
||||||
}
|
}
|
||||||
|
|
||||||
interface SyncSkillsResult {
|
interface SyncSkillsResult {
|
||||||
@@ -201,7 +216,7 @@ export async function finalizeStage(
|
|||||||
// copied into ~/.claude/ while still linking the other runtime files.
|
// copied into ~/.claude/ while still linking the other runtime files.
|
||||||
spin.update('Linking runtime assets...');
|
spin.update('Linking runtime assets...');
|
||||||
const skipClaudeHooks = state.hooks?.accepted === false;
|
const skipClaudeHooks = state.hooks?.accepted === false;
|
||||||
linkRuntimeAssets(state.mosaicHome, skipClaudeHooks);
|
const linkWarning = linkRuntimeAssets(state.mosaicHome, skipClaudeHooks);
|
||||||
|
|
||||||
// 4. Sync skills (only installs the user-selected subset)
|
// 4. Sync skills (only installs the user-selected subset)
|
||||||
let skillsResult: SyncSkillsResult = { success: true, installedCount: 0 };
|
let skillsResult: SyncSkillsResult = { success: true, installedCount: 0 };
|
||||||
@@ -236,6 +251,10 @@ export async function finalizeStage(
|
|||||||
|
|
||||||
spin.stop('Installation complete');
|
spin.stop('Installation complete');
|
||||||
|
|
||||||
|
// Surface the install-ordering guard's outcome (#869 Point-1 C2) — never
|
||||||
|
// silent, even though the wizard continues either way.
|
||||||
|
if (linkWarning) p.warn(linkWarning);
|
||||||
|
|
||||||
// Report skill install failure clearly (non-fatal but user should know)
|
// Report skill install failure clearly (non-fatal but user should know)
|
||||||
if (!skillsResult.success && skillsResult.failureReason) {
|
if (!skillsResult.success && skillsResult.failureReason) {
|
||||||
p.warn(skillsResult.failureReason);
|
p.warn(skillsResult.failureReason);
|
||||||
|
|||||||
Reference in New Issue
Block a user