Document Review Help the author finish a sound document they can use to carry out the agreed work. Find problems that would change that work's outcome or materially hinder execution, and resolve them within the authority already given. Judge the document by whether it guides correct work, not by how much detail it contains. Serious consequences warrant attention even when the defect is small. An adequate document needs no changes. Reviewer personas supply evidence; the judgment is yours. Check their claims against the whole document, project facts, and settled decisions. Correct proven errors…

\\n'/ }\"\n if [ \"${#evidence}\" -gt 300 ]; then\n evidence=\"${evidence:0:147} ... ${evidence: -147}\"\n fi\n printf '%s' \"$evidence\"\n}\n\n# A transient provider-capacity response (HTTP 529), unlike an account or\n# session quota. Claude's ACP adapter reports one its CLI could not ride out as\n# the prompt's own error with data.errorKind \"overloaded\"; the other adapters\n# retry overloads internally and have no distinct form, so they never match.\nprovider_overloaded() {\n [ \"$PROVIDER_OUTCOME\" = \"overloaded\" ]\n}\n\n# Run one route for a provider; leaves a schema-shaped (pre-normalization) $RAW_OUT on success.\n# Success is the prompt's own end_turn result, not acpx's exit status.\nattempt_route() { # \u003cprovider> \u003croute>\n local provider=\"$1\" route=\"$2\" outcome reason started remaining node_line cancel_reason\n : > \"$PEERLOG\"; : > \"$PEERERR\"; rm -f \"$RAW_OUT\" \"$OUT\" \"$TEXT_OUT\"\n RUN_SUCCEEDED=false\n PROVIDER_OUTCOME=\"failed\"\n build_cmd \"$route\"\n PEER_ROUTE=\"$route\"\n log \"peer run: provider=$provider route=$route model=$(route_model \"$route\") (effort $(route_effort \"$route\")) transport=$(acpx_package_spec) lens=$REVIEWER_NAME read-only least-privilege (idle ${IDLE_SECS}s / hard ${HARD_SECS}s); full document content egresses to this provider via this route\"\n started=\"$(date +%s)\"\n run_peer_cmd\n if acpx_npm_resolution_retryable \"$PEERERR\" \"$PEERLOG\" && [ \"$(acpx_outcome \"$PEERLOG\")\" = not-sent ]; then\n remaining=$(( HARD_SECS - ( $(date +%s) - started ) ))\n if [ \"$remaining\" -gt 0 ]; then\n log \"npm could not resolve a package; retrying the same route once with fresh metadata in the ${remaining}s left (nothing was sent; npm policy unchanged)\"\n ACPX_NPM_REFRESH=true\n HARD_SECS=\"$remaining\"\n build_cmd \"$route\"\n run_peer_cmd\n fi\n fi\n outcome=\"$(acpx_outcome \"$PEERLOG\")\"\n case \"$outcome\" in\n end_turn)\n RUN_SUCCEEDED=true\n PROVIDER_OUTCOME=\"ok\"\n acpx_text \"$PEERLOG\" \"$TEXT_OUT\" && recover_findings_json \"$TEXT_OUT\" \"$RAW_OUT\"\n extract_model_receipt \"$route\"\n ;;\n not-sent)\n # Nothing reached the provider: npm could not fetch acpx (every route\n # fails alike), or the adapter refused before the prompt (this route only).\n reason=\"$(acpx_npm_failure_reason \"$PEERERR\")\"\n if [ -n \"$reason\" ]; then\n PRE_EGRESS=\"transport unavailable (pre-egress, shared): ${reason:0:200}\"\n elif [ \"$SESSION_STALLED\" = true ]; then\n PRE_EGRESS=\"transport unavailable (pre-egress, route): Antigravity is waiting for a Google sign-in; the agy CLI's sign-in does not carry over, so sign the ACP runtime in once: put {\\\"auth\\\":{\\\"type\\\":\\\"oauth-personal\\\"}} in ${GEMINI_HOME:-~/.gemini}/antigravity-acp/settings.json, run npx -y --package '$(acpx_package_spec)' -- acpx antigravity exec ok in a terminal and finish the browser sign-in, then retry\"\n else\n node_line=\"$(acpx_node_startup_line \"$PEERERR\")\"\n if [ -n \"$node_line\" ]; then\n PRE_EGRESS=\"transport unavailable (pre-egress, route): ${node_line:0:200}\"\n else\n reason=\"$(acpx_npm_failure_reason \"$PEERLOG\")\"\n [ -n \"$reason\" ] || reason=\"$(acpx_failure_evidence \"$PEERLOG\")\"\n PRE_EGRESS=\"transport unavailable (pre-egress, route): ${reason:-acpx exited $PEER_EXIT before sending the prompt}\"\n fi\n fi\n ;;\n *)\n if [ \"$outcome\" = error ] && jq -eR 'fromjson? | select(.error.data.errorKind? == \"overloaded\")' \"$PEERLOG\" >/dev/null 2>&1; then\n PROVIDER_OUTCOME=\"overloaded\"\n fi\n cancel_reason=\"$(acpx_permission_cancel_reason \"$PEERLOG\" \"$PEER_EXIT\")\"\n if [ -n \"$cancel_reason\" ]; then\n log \"peer run ended: $cancel_reason (acpx exit $PEER_EXIT)\"\n else\n log \"peer run ended with $outcome (acpx exit $PEER_EXIT)\"\n fi\n ;;\n esac\n}\n\n# Run one host-resolved provider through its fixed route.\nrun_provider() { # \u003cprovider>\n local provider=\"$1\" primary=\"\" fixed=\"${CROSS_MODEL_FIXED_ROUTE:-}\"\n local provider_deadline remaining\n OUT=\"$RUN_DIR/$REVIEWER_NAME-$provider.json\"\n RAW_OUT=\"$SCRATCH/$REVIEWER_NAME-$provider.raw.json\"\n [ -n \"$fixed\" ] || { log \"host must resolve one fixed route before egress; skipping\"; rm -f \"$OUT\"; return 0; }\n [ \"$(route_target \"$fixed\")\" = \"$provider\" ] || { log \"fixed route '$fixed' does not match target '$provider'; skipping\"; rm -f \"$OUT\"; return 0; }\n if [ \"$fixed\" = \"grok-cursor\" ] && ! cursor_egress_ok; then\n log \"fixed route 'grok-cursor' requires Cursor intermediary sanction; skipping\"\n rm -f \"$OUT\"\n return 0\n fi\n primary=\"$fixed\"\n PY_BIN=\"$(resolve_python)\"\n [ -n \"$PY_BIN\" ] || { log \"working Python 3 interpreter required to recover peer findings; skipping\"; rm -f \"$OUT\"; return 0; }\n validate_model_override \"$primary\" || { log \"model override '${CROSS_MODEL_MODEL_OVERRIDE:-}' not compatible with route '$primary'; skipping\"; rm -f \"$OUT\"; return 0; }\n validate_effort_override \"$primary\" || { log \"effort override '${CROSS_MODEL_EFFORT_OVERRIDE:-}' not compatible with route '$primary'; skipping\"; rm -f \"$OUT\"; return 0; }\n case \"$HARD_SECS\" in\n ''|0*|*[!0-9]*) log \"peer hard budget must be a positive integer; skipping\"; rm -f \"$OUT\"; return 0 ;;\n esac\n # Per-peer empty workspace, kept SEPARATE from the shared fold-in dir (RUN_DIR)\n # and from private scratch: it is the peer's working directory, so a peer has\n # no path handle to RUN_DIR or to another lens's published artifact from it.\n PEER_WORKDIR=\"$(mktemp -d \"${TMPDIR:-/tmp}/xmodel-doc-peer-XXXXXX\")\" || { PEER_WORKDIR=\"\"; log \"provider $provider workspace isolation unavailable; skipping\"; rm -f \"$OUT\"; return 0; }\n acpx_preflight \"$primary\" \"$PEER_WORKDIR\" || { log \"transport unavailable (pre-egress, $ACPX_SCOPE): $ACPX_UNAVAILABLE\"; rm -f \"$OUT\"; return 0; }\n if [ \"$primary\" = claude ]; then\n CLAUDE_WRAPPER=\"$(acpx_claude_wrapper \"$SCRATCH\")\" || { log \"transport unavailable (pre-egress, route): cannot prepare the Claude --safe-mode launcher\"; rm -f \"$OUT\"; return 0; }\n fi\n if [ \"$primary\" = antigravity ]; then\n GEMINI_HOME_DIR=\"$(acpx_antigravity_home \"$SCRATCH\")\" || { log \"transport unavailable (pre-egress, route): cannot prepare a private Antigravity home\"; rm -f \"$OUT\"; return 0; }\n fi\n # Track the route that actually produced the fold-in, so the artifact records\n # whether a grok return went out directly (grok-cli -> xAI) or through Cursor\n # (grok-cursor -> Cursor also received the full document). The \u003clens>-\u003cprovider>\n # filename alone cannot encode that intermediary.\n ACTUAL_ROUTE=\"$primary\"\n provider_deadline=$(( $(date +%s) + HARD_SECS ))\n PRE_EGRESS=\"\"\n attempt_route \"$provider\" \"$primary\"\n # A configured model is exact; only the editorial Argon request falls back.\n if [ \"$primary\" = antigravity ] && [ \"$(route_model antigravity)\" = \"$M_ANTIGRAVITY\" ] &&\n [ \"${CROSS_MODEL_MODEL_OVERRIDE_TARGET:-}\" != antigravity ]; then\n case \"$PRE_EGRESS\" in\n *\"did not advertise\"*)\n # The fallback shares the original deadline, as the overload retry does.\n remaining=$(( provider_deadline - $(date +%s) ))\n if [ \"$remaining\" -gt 0 ]; then\n log \"Antigravity does not offer $M_ANTIGRAVITY on this account; falling back to $M_ANTIGRAVITY_FALLBACK\"\n ANTIGRAVITY_MODEL=\"$M_ANTIGRAVITY_FALLBACK\"\n PRE_EGRESS=\"\"\n HARD_SECS=\"$remaining\"\n attempt_route \"$provider\" \"$primary\"\n fi\n ;;\n esac\n fi\n if [ -n \"$PRE_EGRESS\" ]; then\n log \"$PRE_EGRESS\"\n rm -f \"$OUT\"\n return 0\n fi\n if [ ! -s \"$RAW_OUT\" ] && provider_overloaded; then\n remaining=$(( provider_deadline - $(date +%s) ))\n if [ \"$remaining\" -le \"$TRANSIENT_RETRY_DELAY_SECS\" ]; then\n log \"provider overload 529; shared peer budget spent, not retrying\"\n else\n log \"provider overload 529; retrying same route once after ${TRANSIENT_RETRY_DELAY_SECS}s\"\n sleep \"$TRANSIENT_RETRY_DELAY_SECS\"\n remaining=$(( provider_deadline - $(date +%s) ))\n if [ \"$remaining\" -gt 0 ]; then\n HARD_SECS=\"$remaining\"\n attempt_route \"$provider\" \"$primary\"\n [ -z \"$PRE_EGRESS\" ] || log \"$PRE_EGRESS\"\n fi\n fi\n fi\n\n # --- normalize + validate against the synthesis reviewer-return contract ---\n # Force reviewer = \u003creviewer-name>-\u003cprovider>; backfill soft arrays; drop the\n # file if findings is not an array. Peer findings fold in as a corroboration\n # signal only -- synthesis (references/synthesis-and-presentation.md) never\n # auto-applies them and caps the cross-model bonus at one anchor step.\n # Downgrade any peer finding's autofix_class from safe_auto to gated_auto: the\n # no-silent-fixes rule forbids a peer from granting silent-apply authority, and enforcing it here (not\n # only in synthesis prose) means a peer cannot self-authorize a Phase 4 auto-apply\n # regardless of what it returns. gated_auto preserves the peer's proposed fix but\n # routes it through user confirmation.\n # Publish ONLY the normalized OUT into RUN_DIR. RAW_OUT lives in private scratch\n # and is never a fold-in artifact — if this script dies before normalize\n # (orphaned launch), synthesis finds no .json in RUN_DIR.\n rm -f \"$OUT\"\n if [ \"$RUN_SUCCEEDED\" = true ] && [ -s \"$RAW_OUT\" ]; then\n _norm=\"$SCRATCH/normalized.json\"\n case \"$ACTUAL_ROUTE:$MODEL_ACTUAL\" in\n cursor:*) _target_family=\"unknown\" ;;\n composer:unverified|grok-cursor:unverified) _target_family=\"unknown\" ;;\n *) _target_family=\"$(target_serving_family \"$provider\")\" ;;\n esac\n _independent=false\n [ \"$HOST_PROVIDER\" != \"unknown\" ] && [ \"$_target_family\" != \"unknown\" ] && [ \"$HOST_PROVIDER\" != \"$_target_family\" ] && _independent=true\n if jq --arg r \"$REVIEWER_NAME-$provider\" --arg route \"$ACTUAL_ROUTE\" \\\n --arg target \"$provider\" --arg harness \"$(route_harness \"$ACTUAL_ROUTE\")\" \\\n --arg family \"$_target_family\" --argjson independent \"$_independent\" \\\n --arg mreq \"$(route_model \"$ACTUAL_ROUTE\")\" --arg mact \"$MODEL_ACTUAL\" \\\n --arg ereq \"$(route_effort \"$ACTUAL_ROUTE\")\" \\\n 'if (.findings|type)==\"array\"\n then { reviewer: $r,\n cross_model_route: $route,\n cross_model_target: $target,\n cross_model_harness: $harness,\n serving_family: $family,\n independence_verified: $independent,\n model_requested: $mreq,\n model_actual: $mact,\n effort_requested: $ereq,\n findings: [ .findings[] | if (.autofix_class? == \"safe_auto\") then .autofix_class = \"gated_auto\" else . end ],\n residual_risks: (.residual_risks // []),\n deferred_questions: (.deferred_questions // []) }\n else empty end' \\\n \"$RAW_OUT\" > \"$_norm\" 2>/dev/null; then\n mv \"$_norm\" \"$OUT\"\n else\n rm -f \"$_norm\"\n fi\n rm -f \"$RAW_OUT\"\n fi\n if [ -s \"$OUT\" ] && jq -e '(.reviewer|type==\"string\") and (.findings|type==\"array\") and (.residual_risks|type==\"array\") and (.deferred_questions|type==\"array\")' \"$OUT\" >/dev/null 2>&1; then\n n=\"$(jq '.findings | length' \"$OUT\" 2>/dev/null || echo '?')\"\n log \"wrote $n finding(s) to $OUT (reviewer $REVIEWER_NAME-$provider)\"\n else\n log \"provider $provider produced no usable schema-shaped output; skipping fold-in\"\n # Surface bounded peer output so the orchestrator can\n # reason about WHY it was skipped (quota/usage-limit exhaustion vs an ordinary\n # empty review) and, in a repeated-pass session, deprioritize an exhausted\n # route. Harness-agnostic: the agent classifies from the text; this only makes\n # the evidence visible in out.log. Provider errors arrive as ACP error\n # messages or as reply text on stdout; npm and adapter diagnostics on stderr.\n _pt=\"$(acpx_permission_cancel_reason \"$PEERLOG\" \"$PEER_EXIT\")\"\n [ -n \"$_pt\" ] || _pt=\"$(acpx_failure_evidence \"$PEERLOG\")\"\n [ -n \"$_pt\" ] && log \" peer skip evidence: $_pt\"\n _pr=\"$(acpx_reply_evidence \"$PEERLOG\")\"\n [ -n \"$_pr\" ] && log \" peer skip evidence (reply): $_pr\"\n _pm=\"$(acpx_advertised_models \"$PEERLOG\")\"\n [ -n \"$_pm\" ] && log \" peer skip evidence (advertised models): $_pm\"\n if [ -s \"$PEERERR\" ]; then\n _pe=\"$(bounded_failure_evidence \"$PEERERR\")\"\n log \" peer skip evidence (stderr): $_pe\"\n fi\n rm -f \"$OUT\" \"$RAW_OUT\"\n fi\n # Tear down the per-peer workspace (never RUN_DIR, which holds the published OUT).\n [ -n \"$PEER_WORKDIR\" ] && rm -rf \"$PEER_WORKDIR\"\n PEER_WORKDIR=\"\"\n}\n\n# --- run the host-sanctioned fixed target -----------------------------------\n# Discovery preserves caller order and MAX_PEERS, but live egress is already\n# frozen to one route. Dispatch that route's target directly so a later eligible\n# candidate is not discarded by the discovery-order cap. run_provider never\n# changes recipients after dispatch.\nFIXED_TARGET=\"$(route_target \"${CROSS_MODEL_FIXED_ROUTE:-}\")\"\nif [ -n \"$FIXED_TARGET\" ]; then\n case \" $SELECTED \" in\n *\" $FIXED_TARGET \"*) run_provider \"$FIXED_TARGET\" ;;\n *) log \"fixed route '${CROSS_MODEL_FIXED_ROUTE:-}' target '$FIXED_TARGET' is not an eligible reachable candidate; skipping\" ;;\n esac\nelse\n log \"host must resolve one fixed route before egress; skipping\"\nfi\nexit 0\n","content_type":"application/x-sh; charset=utf-8","language":"bash","size":66549,"content_sha256":"8e2e6c9c81914953680ee6fdb98893b4d5a8d539fa7f746ad291340c6a819e59"},{"filename":"scripts/packs-resolve.py","content":"#!/usr/bin/env python3\n\"\"\"Resolve the Compound Packs declared in this repo's CE config into pack roots.\n\nReads the `packs:` list from `\u003crepo-root>/.compound-engineering/config.yaml`\nand `config.local.yaml` (both layers concatenate; local adds, never replaces),\nvalidates each entry, resolves path and git sources, enumerates the packs each\nsource publishes, applies selection, and prints one JSON object to stdout:\n\n {\"roots\": [{\"id\": \"...\", \"dir\": \"/abs/path\", \"nested_rule_shaped\": 0}],\n \"warnings\": [...], \"errors\": [...], \"entries\": \u003cnumber of parsed packs entries>}\n\n`nested_rule_shaped` counts the rule-shaped `.md` files one level below the\npack's top level. Discovery never reads them (subdirectories are storage), so\nthe count is informational and is reported by the health check, not warned.\n\nExit 0 whenever resolution ran (per-entry failures are data in `errors` /\n`warnings`); non-zero only when the resolver itself cannot run. Consumers treat\n`errors` as loud per-entry configuration problems and `warnings` as degraded\navailability (e.g. an unreachable git source skipped per the warn-and-continue\ncontract).\n\n`--declared-only` answers the config-only question without touching git or the\ncache: it parses both config layers, shape-checks each entry, and prints\n\n {\"declared\": \u003cbool|null>, \"entries\": \u003cn>, \"errors\": [...]}\n\nwhere `declared` is true when any entry parsed or the `packs:` block is\nmalformed (a broken declaration is still a declaration the consumer should\nsurface), false when neither config layer names packs, and null when no\nrepository -- and so no config -- could be located.\n\nEntry shape (documented subset -- anything else under `packs:` is a loud error):\n\n packs:\n - source: packs/local-rules # repo-relative path\n - source: ~/packs/kk-style # ~ or absolute path\n - source: https://github.com/o/r # git URL: ref required\n ref: v1.2.0 # tag, sha, or branch\n path: packs # optional subfolder (git only)\n pack: [rails, inertia] # one id, a list, or omit = all\n id: rails-core # rename (single-pack entries)\n - source: https://github.com/o/r/tree/main/packs # tree-URL sugar\n\nGit sources cache under `\u003cscratch-root>/ce-packs/\u003csha256(url\\\\nref)>` with an\natomic temp-clone-then-rename, so a keyed path's existence proves a complete\nclone. All git subprocesses run non-interactively (GIT_TERMINAL_PROMPT=0, ssh\nBatchMode, bounded timeout): missing credentials degrade to a warning, never a\nhang. A missing `git` binary degrades git sources only (each warns and is\nskipped); path sources still resolve, with the repository located by walking up\nfrom the working directory to the nearest `.git` entry. Environment overrides:\nCE_PACKS_CACHE_ROOT (cache base for tests), CE_PACKS_GIT_TIMEOUT (seconds,\ndefault 60).\n\nA published pack is a directory a consumer lists and reads itself, so nothing\nin it may link outside its source: a pack whose tree holds such a link is not\npublished (a loud per-entry error), never trimmed a file at a time.\n\"\"\"\n\nfrom __future__ import annotations\n\nimport argparse\nimport functools\nimport hashlib\nimport json\nimport os\nimport re\nimport shutil\nimport stat\nimport subprocess\nimport sys\nimport tempfile\n\nIS_WINDOWS = os.name == \"nt\"\n_uid_getter = getattr(os, \"geteuid\", None) or getattr(os, \"getuid\", None)\n_EFFECTIVE_UID = _uid_getter() if _uid_getter is not None else None\nGIT_TIMEOUT = float(os.environ.get(\"CE_PACKS_GIT_TIMEOUT\") or 60)\n\nCONFIG_FILES = (\"config.yaml\", \"config.local.yaml\")\nKNOWN_KEYS = {\"source\", \"ref\", \"path\", \"pack\", \"id\"}\nSCALAR_KEYS = (\"source\", \"ref\", \"path\", \"id\") # every known key but the list-valued `pack`\n_TREE_URL_RE = re.compile(\n r\"^(?P\u003cbase>https?://github\\.com/[^/\\s]+/[^/\\s]+?)(?:\\.git)?/tree/(?P\u003cref>[^/\\s]+)(?:/(?P\u003cpath>[^\\s]*))?/?$\"\n)\n\n\ndef _is_git_url(source: str) -> bool:\n return bool(\n re.match(r\"^(https?|ssh|git|file)://\", source) or re.match(r\"^[\\w.-]+@[\\w.-]+:\", source)\n )\n\n\ndef _within(path: str, parent: str) -> bool:\n \"\"\"True when `path` is `parent` or lies below it (both already realpath'd).\"\"\"\n return path == parent or path.startswith(parent + os.sep)\n\n\n# --- scratch root (peer-job-runner shape: probe /tmp, fall back to TMPDIR) ---\n\ndef _owned_dir(path: str) -> bool:\n \"\"\"Directory, not a symlink, owned by the effective uid (POSIX).\"\"\"\n try:\n st = os.lstat(path)\n except OSError:\n return False\n if not stat.S_ISDIR(st.st_mode):\n return False\n if _EFFECTIVE_UID is not None and st.st_uid != _EFFECTIVE_UID:\n return False\n return True\n\n\ndef _private_root_usable(path: str) -> bool:\n \"\"\"Create or repair `path` as a private (0700) root owned by this user.\"\"\"\n try:\n os.mkdir(path, 0o700)\n except FileExistsError:\n pass\n except OSError:\n return False\n if not IS_WINDOWS:\n if not _owned_dir(path):\n return False\n try:\n os.chmod(path, 0o700)\n except OSError:\n return False\n return os.path.isdir(path) and os.access(path, os.W_OK)\n\n\ndef _trusted_checkout(path: str) -> bool:\n \"\"\"A real directory (never a symlink) that this user owns; ownership is POSIX-only.\"\"\"\n if os.path.islink(path) or not os.path.isdir(path):\n return False\n return IS_WINDOWS or _owned_dir(path)\n\n\[email protected]_cache(maxsize=None)\ndef cache_base() -> str | None:\n configured = os.environ.get(\"CE_PACKS_CACHE_ROOT\")\n if configured:\n root = os.path.abspath(configured)\n os.makedirs(root, exist_ok=True)\n return root\n if IS_WINDOWS:\n base = os.environ.get(\"LOCALAPPDATA\") or tempfile.gettempdir()\n root = os.path.join(base, \"compound-engineering-packs\")\n return root if _private_root_usable(root) else None\n if _EFFECTIVE_UID is None:\n return None\n for base in (\"/tmp\", os.environ.get(\"TMPDIR\") or \"/tmp\"):\n root = os.path.join(base, f\"compound-engineering-{_EFFECTIVE_UID}\")\n if _private_root_usable(root):\n packs = os.path.join(root, \"ce-packs\")\n if _private_root_usable(packs):\n return packs\n return None\n\n\n# --- minimal YAML reader for the documented packs: subset --------------------\n\ndef _strip_comment(line: str) -> str:\n \"\"\"Drop a trailing comment (a # preceded by whitespace, outside quotes).\n\n A quote toggles quoted state only when it opens a value (start of line or\n after `: `/`- `/`[`/`,`) or closes one it opened -- a mid-word apostrophe\n (``it's``) is ordinary content and must not absorb a later comment.\n \"\"\"\n out, quote = [], \"\"\n for i, ch in enumerate(line):\n prev = line[i - 1] if i else \" \"\n if quote:\n if ch == quote:\n quote = \"\"\n elif ch in \"'\\\"\" and prev in \" \\t[,:\":\n quote = ch\n elif ch == \"#\" and prev in \" \\t\":\n break\n out.append(ch)\n return \"\".join(out).rstrip()\n\n\ndef _scalar(raw: str):\n raw = raw.strip()\n if len(raw) >= 2 and raw[0] == raw[-1] and raw[0] in \"'\\\"\":\n return raw[1:-1]\n if raw.lower() in (\"true\", \"false\"):\n return raw.lower() == \"true\"\n return raw\n\n\ndef _parse_value(raw: str):\n raw = raw.strip()\n if raw.startswith(\"[\") and raw.endswith(\"]\"):\n inner = raw[1:-1].strip()\n return [] if not inner else [_scalar(part) for part in inner.split(\",\")]\n return _scalar(raw)\n\n\ndef parse_packs_block(path: str, errors: list) -> list:\n \"\"\"Return the entry dicts under this file's top-level `packs:` key.\"\"\"\n if not os.path.isfile(path):\n return []\n with open(path, encoding=\"utf-8-sig\", errors=\"replace\") as fh:\n lines = fh.read().splitlines()\n entries, in_packs, current, pending_list_key = [], False, None, None\n for lineno, raw in enumerate(lines, 1):\n line = _strip_comment(raw)\n if not line.strip():\n continue\n indent = len(line) - len(line.lstrip())\n if indent == 0 and not (in_packs and line.lstrip().startswith(\"-\")):\n # A new top-level key ends the packs block; a zero-indent list item\n # (`- source: ...`) is still part of it -- YAML allows both styles.\n key, sep, rest = line.partition(\":\")\n in_packs = bool(sep) and key.strip() == \"packs\" and rest.strip() in (\"\", \"[]\")\n if sep and key.strip() == \"packs\" and not in_packs:\n # `packs: \u003cinline value>` is malformed, not absent: say so rather\n # than letting a flow list or a bare path declare nothing.\n errors.append(\n f\"{os.path.basename(path)}:{lineno}: `packs:` must be a block list of\"\n f\" `- source: ...` entries (got `{line.strip()}`)\"\n )\n current, pending_list_key = None, None\n continue\n if not in_packs:\n continue\n stripped = line.strip()\n loc = f\"{os.path.basename(path)}:{lineno}\"\n if stripped.startswith(\"- \") or stripped == \"-\":\n body = stripped[1:].strip()\n if pending_list_key and current is not None and \":\" not in body:\n current[pending_list_key].append(_scalar(body))\n continue\n current, pending_list_key = {\"_origin\": os.path.basename(path), \"_line\": lineno}, None\n entries.append(current)\n if body:\n if \":\" not in body:\n errors.append(f\"{loc}: unrecognized packs entry `{stripped}` -- expected `key: value`\")\n continue\n key, _, val = body.partition(\":\")\n _set_key(current, key.strip(), val, loc, errors)\n continue\n if current is None:\n errors.append(f\"{loc}: unrecognized line under packs: `{stripped}` -- expected a `- source: ...` entry\")\n continue\n if \":\" not in stripped:\n errors.append(f\"{loc}: unrecognized line under packs: `{stripped}` -- expected `key: value`\")\n continue\n key, _, val = stripped.partition(\":\")\n key = key.strip()\n if val.strip() == \"\" and key == \"pack\":\n current[key] = []\n pending_list_key = key\n continue\n pending_list_key = None\n _set_key(current, key, val, loc, errors)\n return entries\n\n\ndef _set_key(entry: dict, key: str, raw_val: str, loc: str, errors: list) -> None:\n if key not in KNOWN_KEYS:\n errors.append(f\"{loc}: unknown packs entry key `{key}:` -- accepted keys: {', '.join(sorted(KNOWN_KEYS))}\")\n return\n entry[key] = _parse_value(raw_val)\n\n\n# --- git ---------------------------------------------------------------------\n\[email protected]_cache(maxsize=None)\ndef _git_env() -> dict:\n env = dict(os.environ)\n env[\"GIT_TERMINAL_PROMPT\"] = \"0\"\n env[\"GIT_ASKPASS\"] = env.get(\"GIT_ASKPASS\") or \"true\"\n ssh = env.get(\"GIT_SSH_COMMAND\") or \"ssh\"\n if \"BatchMode\" not in ssh:\n env[\"GIT_SSH_COMMAND\"] = ssh + \" -o BatchMode=yes\"\n return env\n\n\ndef _run_git(args: list, cwd: str | None = None):\n return subprocess.run(\n [\"git\", *args], cwd=cwd, env=_git_env(), timeout=GIT_TIMEOUT,\n capture_output=True, text=True,\n )\n\n\ndef resolve_git_source(url: str, ref: str, warnings: list, label: str) -> str | None:\n \"\"\"Return the cached checkout dir for url@ref, cloning on miss. None = warn+skip.\"\"\"\n if shutil.which(\"git\") is None:\n warnings.append(f\"{label}: git binary not found; source skipped\")\n return None\n base = cache_base()\n if base is None:\n warnings.append(f\"{label}: no writable cache root for git sources; source skipped\")\n return None\n key = hashlib.sha256(f\"{url}\\n{ref}\".encode()).hexdigest()\n dest = os.path.join(base, key)\n if os.path.lexists(dest):\n if _trusted_checkout(dest):\n return dest\n warnings.append(f\"{label}: cached checkout {dest} is a symlink or not owned by this user; refetching\")\n # rmtree refuses to follow a symlink (and ignores a plain file); unlink\n # those explicitly so a planted link is removed, never its target.\n shutil.rmtree(dest, ignore_errors=True)\n if os.path.islink(dest) or os.path.isfile(dest):\n try:\n os.unlink(dest)\n except OSError:\n pass\n if os.path.lexists(dest):\n warnings.append(f\"{label}: cannot replace untrusted cached checkout {dest}; source skipped\")\n return None\n tmp = tempfile.mkdtemp(prefix=f\"{key}.part-\", dir=base)\n try:\n try:\n proc = _run_git([\"clone\", \"--quiet\", \"--depth\", \"1\", \"--no-recurse-submodules\",\n \"--branch\", ref, \"--end-of-options\", url, tmp])\n except subprocess.TimeoutExpired:\n warnings.append(f\"{label}: git clone timed out after {int(GIT_TIMEOUT)}s; source skipped\")\n return None\n if proc.returncode != 0:\n # tag/branch clone failed -- retry treating ref as a commit sha\n try:\n fetched = (\n _run_git([\"init\", \"--quiet\", tmp]).returncode == 0\n and _run_git([\"fetch\", \"--quiet\", \"--depth\", \"1\", \"--end-of-options\", url, ref], cwd=tmp).returncode == 0\n and _run_git([\"checkout\", \"--quiet\", \"FETCH_HEAD\"], cwd=tmp).returncode == 0\n )\n if not fetched:\n warnings.append(f\"{label}: cannot fetch `{ref}` from {url}; source skipped\")\n return None\n except subprocess.TimeoutExpired:\n warnings.append(f\"{label}: git fetch timed out after {int(GIT_TIMEOUT)}s; source skipped\")\n return None\n if not os.path.lexists(dest):\n try:\n os.replace(tmp, dest)\n except OSError:\n # Expected when another resolver published the same key\n # concurrently; anything else leaves no checkout to return.\n if not os.path.lexists(dest):\n warnings.append(f\"{label}: could not publish the clone to {dest}; source skipped\")\n return None\n if _trusted_checkout(dest):\n return dest\n warnings.append(\n f\"{label}: cached checkout {dest} appeared during the clone and is a symlink or not owned by this user; source skipped\"\n )\n return None\n finally:\n if os.path.isdir(tmp):\n shutil.rmtree(tmp, ignore_errors=True)\n\n\n# --- pack enumeration --------------------------------------------------------\n\n_FRONTMATTER_KEYS = (\"title:\", \"applies_when:\")\n_FRONTMATTER_CAP = 64 * 1024 # frontmatter must close within this many characters\n# A pack's README is its description, never a rule, whatever frontmatter it\n# carries: it is not published, not reported as skipped, and not counted.\n_README = \"readme.md\"\n\n\[email protected]_cache(maxsize=None)\ndef _is_knowledge_file(path: str) -> bool:\n \"\"\"True when the file opens with a `---` frontmatter block, closed by a `---`\n line within the cap, that carries `title:` and `applies_when:`. A UTF-8 BOM\n is not part of the content.\"\"\"\n try:\n with open(path, encoding=\"utf-8-sig\", errors=\"replace\") as fh:\n lines = fh.read(_FRONTMATTER_CAP).splitlines()\n except OSError:\n return False\n if not lines or lines[0].strip() != \"---\":\n return False\n for end, line in enumerate(lines[1:], 1):\n if line.strip() == \"---\":\n fm = \"\\n\".join(lines[1:end])\n break\n else:\n return False\n return all(re.search(rf\"^\\s*{re.escape(k)}\", fm, re.MULTILINE) for k in _FRONTMATTER_KEYS)\n\n\ndef _contained_md_files(directory: str, boundary: str, escaped: list) -> list:\n \"\"\"Paths of the `.md` entries directly under `directory`, minus its README,\n whose real path stays within `boundary` (a realpath). An entry that links\n outside it is appended to `escaped` and never opened, so a pack cannot read\n files off the user's machine.\"\"\"\n try:\n names = sorted(os.listdir(directory))\n except OSError:\n return []\n files = []\n for name in names:\n if not name.endswith(\".md\") or name.lower() == _README:\n continue\n child = os.path.join(directory, name)\n if _within(os.path.realpath(child), boundary):\n files.append(child)\n else:\n escaped.append(child)\n return files\n\n\ndef _contained_child_dirs(directory: str, boundary: str, escaped: list) -> list:\n \"\"\"`(name, path)` of the non-hidden directories directly under `directory`\n whose real path stays within `boundary`; one that links outside it is\n appended to `escaped` and never entered.\"\"\"\n try:\n names = sorted(os.listdir(directory))\n except OSError:\n return []\n dirs = []\n for name in names:\n child = os.path.join(directory, name)\n if name.startswith(\".\") or not os.path.isdir(child):\n continue\n if _within(os.path.realpath(child), boundary):\n dirs.append((name, child))\n else:\n escaped.append(child)\n return dirs\n\n\ndef _has_knowledge_files(directory: str, boundary: str, escaped: list) -> bool:\n return any(_is_knowledge_file(f) for f in _contained_md_files(directory, boundary, escaped))\n\n\ndef _escaping_links(root: str, boundary: str) -> list:\n \"\"\"Symlinks anywhere under `root` (walked without following links) whose real\n path leaves `boundary`, sorted. Only a link can leave: every other entry sits\n under `root`, which is already inside the boundary.\"\"\"\n leaks = []\n for dirpath, dirnames, filenames in os.walk(root, followlinks=False):\n for name in dirnames + filenames:\n child = os.path.join(dirpath, name)\n if os.path.islink(child) and not _within(os.path.realpath(child), boundary):\n leaks.append(child)\n return sorted(leaks)\n\n\ndef enumerate_packs(source_root: str, boundary: str, escaped: list, self_name: str | None = None) -> dict:\n \"\"\"Map published pack id -> dir. Immediate children only; self = single pack.\n\n `boundary` is the realpath every child directory and rule file must stay\n within (the git checkout, the repository, or the source root itself); a\n child that links outside it is recorded in `escaped` and skipped. Symlinks\n that stay inside the boundary are ordinary content.\n \"\"\"\n if _has_knowledge_files(source_root, boundary, escaped):\n name = self_name or os.path.basename(os.path.abspath(source_root))\n return {name: source_root}\n return {\n name: child\n for name, child in _contained_child_dirs(source_root, boundary, escaped)\n if _has_knowledge_files(child, boundary, escaped)\n }\n\n\ndef nested_rule_shaped(pack_dir: str, boundary: str) -> tuple:\n \"\"\"`(total, subdirs)`: how many rule-shaped files sit in the immediate\n subdirectories of `pack_dir`, and which subdirectories hold them. Scans one\n level down only, inside `boundary`, so the pack's data is never walked.\n Subdirectories are storage, so these files are counted, never published.\"\"\"\n hits, total = [], 0\n for name, child in _contained_child_dirs(pack_dir, boundary, []):\n count = sum(1 for f in _contained_md_files(child, boundary, []) if _is_knowledge_file(f))\n if count:\n hits.append(name)\n total += count\n return total, hits\n\n\ndef nested_rules_warning(pack_id: str, pack_dir: str, boundary: str) -> str | None:\n \"\"\"The warning for a pack directory with no rule at its top level whose\n subdirectories hold rule-shaped files: it registers, yet nothing can ever be\n discovered. None when the subdirectories hold none. The caller establishes\n that the top level is empty of rules; a pack with top-level rules keeps its\n nested files as storage and gets `nested_rule_shaped` instead.\"\"\"\n total, hits = nested_rule_shaped(pack_dir, boundary)\n if not hits:\n return None\n where = \", \".join(f\"`{name}/`\" for name in hits)\n return (\n f\"pack `{pack_id}` has {total} rule-shaped file(s) under {where} that discovery\"\n \" never reads -- move rules to the pack's top level (see https://everyinc.github.io/compound-engineering-plugin/guides/packs/, Pack layout)\"\n )\n\n\n# --- entry resolution --------------------------------------------------------\n\ndef _entry_label(entry: dict) -> str:\n return f\"{entry.get('_origin', 'config')}:{entry.get('_line', '?')}\"\n\n\ndef _entry_shape_ok(entry: dict, label: str, errors: list) -> bool:\n \"\"\"I/O-free checks on one parsed entry: `source:` present, and the scalar\n keys actually scalar. False after appending the error.\"\"\"\n for key in SCALAR_KEYS:\n val = entry.get(key)\n if val is not None and not isinstance(val, str):\n errors.append(f\"{label}: `{key}:` must be a single string (got {val!r})\")\n return False\n if not entry.get(\"source\"):\n errors.append(f\"{label}: entry has no `source:`\")\n return False\n return True\n\n\ndef resolve_entry(entry: dict, repo_root: str, roots: list, warnings: list, errors: list) -> None:\n label = _entry_label(entry)\n if not _entry_shape_ok(entry, label, errors):\n return\n source, ref, sub_path = entry[\"source\"], entry.get(\"ref\"), entry.get(\"path\")\n\n tree = _TREE_URL_RE.match(source)\n if tree:\n t_ref, t_path = tree.group(\"ref\"), tree.group(\"path\") or \"\"\n if isinstance(ref, str) and ref != t_ref:\n errors.append(f\"{label}: tree URL pins ref `{t_ref}` but entry says `ref: {ref}` -- remove one\")\n return\n if isinstance(sub_path, str) and sub_path.strip(\"/\") != t_path.strip(\"/\"):\n errors.append(f\"{label}: tree URL path `{t_path}` conflicts with `path: {sub_path}` -- remove one\")\n return\n source, ref, sub_path = tree.group(\"base\"), t_ref, t_path or None\n\n if _is_git_url(source):\n if not isinstance(ref, str) or not ref:\n errors.append(f\"{label}: git source `{source}` requires `ref:` (tag, sha, or branch)\")\n return\n if ref.startswith(\"-\") or source.startswith(\"-\"):\n errors.append(f\"{label}: git source/ref may not begin with `-`\")\n return\n checkout = resolve_git_source(source, ref, warnings, label)\n if checkout is None:\n if tree:\n warnings.append(\n f\"{label}: if the branch name contains `/`, tree-URL parsing splits it wrong -- use explicit `ref:` and `path:` fields\"\n )\n return\n git_meta = {\"url\": source, \"ref\": ref}\n source_root = os.path.join(checkout, sub_path) if sub_path else checkout\n real_root, real_checkout = os.path.realpath(source_root), os.path.realpath(checkout)\n if not _within(real_root, real_checkout):\n errors.append(f\"{label}: path `{sub_path}` escapes the source checkout\")\n return\n source_root, boundary = real_root, real_checkout\n if not os.path.isdir(source_root):\n errors.append(f\"{label}: path `{sub_path}` does not exist in {source}@{ref}\")\n return\n else:\n git_meta = None\n if ref is not None:\n errors.append(f\"{label}: `ref:` is only valid on git sources; path sources are read live\")\n return\n if sub_path is not None:\n errors.append(f\"{label}: `path:` is only valid on git sources; point `source:` at the directory instead\")\n return\n expanded = os.path.expanduser(source)\n if os.path.isabs(expanded):\n source_root = os.path.realpath(expanded)\n boundary = source_root\n else:\n source_root = os.path.realpath(os.path.join(repo_root, expanded))\n repo_real = os.path.realpath(repo_root)\n if not _within(source_root, repo_real) or _within(source_root, os.path.join(repo_real, \".git\")):\n errors.append(f\"{label}: repo-relative source `{source}` resolves outside the repository\")\n return\n boundary = repo_real\n if not os.path.isdir(source_root):\n errors.append(f\"{label}: source directory `{source}` does not exist\")\n return\n\n if git_meta:\n # Display name for a single-pack git source: the path: subfolder's\n # basename, else the URL's last path segment (never the cache key).\n tail = (sub_path or source).rstrip(\"/\").rsplit(\"/\", 1)[-1]\n self_name = re.sub(r\"\\.git$\", \"\", tail.split(\":\")[-1]) or None\n else:\n self_name = None\n escaped = []\n published = enumerate_packs(source_root, boundary, escaped, self_name)\n for link in escaped:\n warnings.append(\n f\"{label}: skipped `{os.path.relpath(link, source_root)}` in `{source}` -- it links outside the source\"\n )\n if not published:\n warnings.append(f\"{label}: source `{source}` publishes no packs (no directories with valid knowledge files)\")\n # Each child directory is a would-be pack with no top-level rule (one\n # with a rule would have published); say when its rules sit one level\n # too deep, so the author learns why nothing published.\n for name, child in _contained_child_dirs(source_root, boundary, []):\n nested = nested_rules_warning(name, child, boundary)\n if nested:\n warnings.append(f\"{label}: {nested}\")\n return\n\n selection = entry.get(\"pack\")\n if selection is None:\n selected = dict(published)\n else:\n wanted = selection if isinstance(selection, list) else [selection]\n if not wanted:\n warnings.append(f\"{label}: `pack:` lists no ids; nothing installed from `{source}`\")\n return\n missing = [w for w in wanted if w not in published]\n if missing:\n errors.append(\n f\"{label}: pack id(s) {', '.join(map(str, missing))} not published by `{source}`\"\n f\" -- available: {', '.join(sorted(published)) or 'none'}\"\n )\n return\n selected = {w: published[w] for w in wanted}\n\n override = entry.get(\"id\")\n if override is not None:\n if len(selected) != 1:\n errors.append(f\"{label}: `id:` override requires the entry to install exactly one pack\")\n return\n selected = {str(override): next(iter(selected.values()))}\n\n for pack_id, pack_dir in selected.items():\n # Consumers list the pack directory themselves, so a link that leaves\n # the source anywhere inside it would let pack content read files off\n # the user's machine: refuse the whole pack. (Top-level escapes were\n # already named as skipped during enumeration.)\n leaks = _escaping_links(pack_dir, boundary)\n if leaks:\n names = \", \".join(f\"`{os.path.relpath(p, pack_dir)}`\" for p in leaks)\n errors.append(f\"{label}: pack `{pack_id}` not published -- {names} link(s) outside the source\")\n continue\n for child in _contained_md_files(pack_dir, boundary, []):\n name = os.path.basename(child)\n if os.path.isfile(child) and not _is_knowledge_file(child):\n warnings.append(\n f\"{label}: skipped pack file `{pack_id}/{name}` (missing `title`/`applies_when` frontmatter)\"\n )\n # A published pack has a rule at its top level, so rule-shaped files in\n # its subdirectories are storage: counted for the health report, not warned.\n nested_total, _ = nested_rule_shaped(pack_dir, boundary)\n root = {\"id\": pack_id, \"dir\": pack_dir, \"nested_rule_shaped\": nested_total, \"_label\": label}\n if git_meta:\n root.update(git_meta)\n roots.append(root)\n\n\ndef _emit(declared_only: bool, entries: list, roots: list, warnings: list, errors: list,\n declared: bool | None = None) -> int:\n if declared_only:\n print(json.dumps({\"declared\": declared, \"entries\": len(entries), \"errors\": errors}))\n else:\n print(json.dumps({\n \"roots\": [{k: v for k, v in r.items() if not k.startswith(\"_\")} for r in roots],\n \"warnings\": warnings,\n \"errors\": errors,\n \"entries\": len(entries),\n }))\n return 0\n\n\ndef _repo_root() -> str | None:\n \"\"\"The enclosing checkout's top level: git's answer when git is on PATH,\n otherwise the nearest ancestor of the working directory holding a `.git`\n entry (a directory, or the file a worktree or submodule leaves). None when\n the working directory is not inside a checkout.\"\"\"\n if shutil.which(\"git\") is not None:\n proc = subprocess.run([\"git\", \"rev-parse\", \"--show-toplevel\"],\n stdout=subprocess.PIPE, stderr=subprocess.DEVNULL, text=True)\n return proc.stdout.strip() if proc.returncode == 0 else None\n current = os.getcwd()\n while True:\n if os.path.lexists(os.path.join(current, \".git\")):\n return current\n parent = os.path.dirname(current)\n if parent == current:\n return None\n current = parent\n\n\ndef _main(argv: list) -> int:\n parser = argparse.ArgumentParser(description=\"Resolve the Compound Packs declared in CE config.\")\n parser.add_argument(\n \"--declared-only\", action=\"store_true\",\n help=\"parse and shape-check the packs: entries only; no git or cache work\",\n )\n args = parser.parse_args(argv)\n warnings, errors, roots, entries = [], [], [], []\n\n repo_root = _repo_root()\n if repo_root is None:\n warnings.append(\"not inside a git repository; no CE config to read\")\n return _emit(args.declared_only, entries, roots, warnings, errors)\n cfg_dir = os.path.join(repo_root, \".compound-engineering\")\n for name in CONFIG_FILES:\n entries.extend(parse_packs_block(os.path.join(cfg_dir, name), errors))\n\n if args.declared_only:\n for entry in entries:\n _entry_shape_ok(entry, _entry_label(entry), errors)\n return _emit(True, entries, roots, warnings, errors, declared=bool(entries or errors))\n\n for entry in entries:\n try:\n resolve_entry(entry, repo_root, roots, warnings, errors)\n except Exception as exc: # one entry's surprise is that entry's error, not everyone's\n errors.append(f\"{_entry_label(entry)}: unexpected error resolving entry: {exc}\")\n\n # First declaration wins: config.yaml entries precede config.local.yaml in\n # CONFIG_FILES, so a personal pack can never displace the team's.\n by_id = {}\n final = []\n for root in roots:\n prev = by_id.get(root[\"id\"])\n if prev is not None:\n errors.append(\n f\"duplicate pack id `{root['id']}`: {root['_label']} ignored, {prev['_label']} kept\"\n \" -- rename one with `id:`\"\n )\n continue\n by_id[root[\"id\"]] = root\n final.append(root)\n return _emit(False, entries, final, warnings, errors)\n\n\ndef main() -> int:\n try:\n return _main(sys.argv[1:])\n except Exception as exc: # never a traceback: consumers need valid JSON\n print(json.dumps({\n \"roots\": [], \"warnings\": [], \"errors\": [f\"packs resolver failed unexpectedly: {exc}\"], \"entries\": 0,\n }))\n return 0\n\n\nif __name__ == \"__main__\":\n sys.exit(main())\n","content_type":"text/x-python; charset=utf-8","language":"python","size":31561,"content_sha256":"39c36946cd1a59ca564e60e4e049934831eb4bfb5cfce9de32ab6b4b522ef228"},{"filename":"scripts/peer-job-runner.py","content":"#!/usr/bin/env python3\n\"\"\"Detached peer-job runner: start/status/wait/result/reap for delegated work.\n\nSome harnesses enforce a hard ceiling on a single shell tool call and kill the\nsupervising shell mid-run, so no tool call may span a peer worker's runtime.\nThis runner splits the lifecycle so every call is short and all durable state\nlives on disk:\n\n start claim a job dir, preflight the worker, detach it into its own\n session (double fork with os.setsid between the forks), print ONLY\n the job id, return fast. The detached process supervises the worker\n and writes ONE atomic terminal record. Also sweeps sibling run roots\n older than 24 hours (best-effort, owner-checked).\n status print each job's state word without blocking.\n wait bounded poll (~1s cadence, never longer than --max-secs) that\n returns early once every watched job has settled. With\n --deadline-secs it also ends at the aggregate deadline and closes\n with one line naming what to do next.\n result ownership-checked bounded read of a done job's published artifact.\n reap ask each job's detached supervisor to terminate it now; returns fast.\n If the supervisor itself is gone, reap kills the worker tree and\n writes the terminal record itself. Reaping a terminal job is a\n safe no-op.\n\nJob directory (durable state, the source of truth):\n \u003croot>/\u003cskill>/\u003crun-id>/jobs/\u003cjob-id>/\n meta.json identity: skill, run id, label, input digest, start time,\n worker argv, result path (written at start, before detach)\n pid supervisor pid + worker pid (written by the supervisor before\n start returns; its presence marks \"detached\"). Platform-\n conditional fields — consumers must use .get(): POSIX adds\n supervisor_pgid; Windows adds job_name (its job object) and\n supervisor_identity / worker_identity (GetProcessTimes guards\n so a recycled pid is not treated as the original process).\n out.log worker's combined stdout+stderr (byte growth = liveness)\n reason terminal detail, written before the status rename so the\n status file is always the LAST record to land\n status exactly one word, published atomically (tmp + os.replace):\n done | failed | timeout | died-without-result\n\nStates reported by status/wait:\n running detached, no terminal record yet\n done worker exited 0 (and, when --result-path was declared,\n the result file exists non-empty)\n failed nonzero exit, byte-cap kill, or exit 0 without the\n declared result\n timeout supervisor idle/hard window fired, or a requested reap\n died-without-result worker killed by an external signal with no result\n evidence (or vanished together with its supervisor)\n never-started meta exists but nothing was ever detached (preflight\n failure)\n unreadable an ownership or sanity check failed; content withheld\n\nSupervision (runs inside the detached session, never in a tool call): poll\n~2s; liveness is out.log byte growth; idle window with no growth reaps the\nworker tree; a hard cap reaps it regardless; byte caps on out.log and the\npublished result classify as failed with a recorded reason. Reaping is TERM\nto the worker's own process group (the worker is started as a session/group\nleader), a grace period, then KILL — with a deepest-first tree walk as the\nfallback when the group kill is unavailable. The supervisor classifies the\noutcome exactly once; when both the worker's internal cap and the\nsupervisor's window fire, the supervisor's record wins.\n\nEnvironment overrides (defaults in parentheses):\n CE_PEER_JOBS_ROOT base dir (/tmp/compound-engineering-\u003ceffective-uid>,\n or $TMPDIR/compound-engineering-\u003ceffective-uid> when\n /tmp cannot host a writable private root, e.g. under\n a sandbox that only allowlists $TMPDIR)\n CE_WORK_RUNS_ROOT parent CE Work dir containing all \u003crun-id>/ dirs\n CE_PEER_IDLE_SECS idle window, no out.log growth (240)\n CE_PEER_HARD_SECS hard cap on worker wall clock\n (default: max(1230, CROSS_MODEL_HARD_SECS+30);\n an explicit value always wins)\n CROSS_MODEL_HARD_SECS when CE_PEER_HARD_SECS is unset, widens the\n supervisor hard window (see above)\n CE_PEER_LOG_MAX_BYTES out.log byte cap (10485760)\n CE_PEER_RESULT_MAX_BYTES result byte cap, supervise + read (5242880)\n CE_PEER_POLL_SECS supervisor poll interval (2)\n CE_PEER_GRACE_SECS TERM-to-KILL grace during reap (5)\n CE_PEER_BASH Windows: absolute bash.exe for peer workers\n (preferred over PATH / WSL System32 bash)\n CLAUDE_CODE_GIT_BASH_PATH Claude Code Git Bash path; used on Windows when\n CE_PEER_BASH is unset (#1268)\n\nSecurity posture: the job root is a predictable, owner-private directory under\nworld-shared /tmp. Every read of job state opens the file first (no-follow) and\nverifies the descriptor's owner (os.fstat st_uid == os.geteuid, guarded where\ngeteuid is unavailable) before any content is emitted; a mismatch reports\n\"unreadable\", never content. Reads are bounded by size caps — out.log is never\nslurped. Directory/file creation uses 0700/0600 modes, exclusive no-follow\ncreation, owner/type verification on path components, exact 0700 verification\non the top-level root, and atomic rename for every publish. The worker argv is\nexec'd directly (argv list, never a shell); job\nids are minted internally; --skill/--run-id/--label are restricted to\n[A-Za-z0-9._-]. Nothing here ever prompts: headless/CI-safe by design.\n\nPlatform (#1243): the mechanisms above describe POSIX. Native Windows Python\nhas no fork/setsid, uid, mode bits, or process groups, so the same contract is\nmet by win32 equivalents, all behind `sys.platform == \"win32\"` branches so the\nPOSIX path is behaviorally unchanged:\n detach re-invoke this script as a DETACHED_PROCESS | CREATE_NEW_PROCESS_GROUP\n child (CREATE_BREAKAWAY_FROM_JOB where the job allows) running the\n internal `__supervise` entrypoint; the pid file is the ack.\n reap cmd_reap drops a `.reap` marker the supervisor polls for (no directed\n signal to a detached, console-less process).\n teardown the worker tree lives in a named Job Object -- the real killpg\n analog, since it reaches descendants of an already-exited leader,\n which taskkill /T cannot (it walks parent->child from a LIVE\n parent). Windows releases a named object's name once the last\n handle closes, so a cmd_reap running after the supervisor died\n falls back to a Toolhelp32 snapshot walk; that works because\n Windows never reparents orphans, so a dead pid still appears as\n th32ParentProcessID on its live children. A recycled pid that is\n now this process (or whose GetProcessTimes identity does not\n match the pid file) is not the original leader: sweep stale-PPID\n descendants, do not TerminateProcess the live reused process.\n ownership st_uid == geteuid becomes: the object's owner SID is one this token\n creates objects as (user or default-owner SID), checked on the opened\n handle (GetSecurityInfo) exactly like the POSIX fstat-by-fd check.\n privacy 0700/0600 modes become a hardened ACL (icacls: break inheritance,\n grant only the user + SYSTEM + Administrators — the root-equivalents).\n jobs root defaults under %LOCALAPPDATA%\\\\compound-engineering-jobs (then the\n user temp dir), owner-private, since there is no shared /tmp.\n\nPure stdlib. No third-party dependencies.\n\"\"\"\nimport argparse\nimport calendar\nimport glob\nimport json\nimport math\nimport os\nimport re\nimport shutil\nimport signal\nimport stat\nimport subprocess\nimport sys\nimport tempfile\nimport time\n\n# Identifier charset for --skill/--run-id/--label and bare job refs. The dot is\n# allowed (model/date tokens use it) but an all-dot value (\".\", \"..\") would be a\n# path component that escapes the jobs root, so it is rejected separately below.\nSAFE_RE = re.compile(r\"^[A-Za-z0-9._-]+$\")\n\n\ndef _is_safe_token(value: str) -> bool:\n return bool(SAFE_RE.match(value)) and value.strip(\".\") != \"\"\n\n\nTERMINAL_STATES = (\"done\", \"failed\", \"timeout\", \"died-without-result\")\nIS_WINDOWS = sys.platform == \"win32\"\n_uid_getter = getattr(os, \"geteuid\", None) or getattr(os, \"getuid\", None)\n_EFFECTIVE_UID = _uid_getter() if _uid_getter is not None else None\nif IS_WINDOWS:\n # No geteuid on Windows; the current-user SID is the ownership identity\n # (see the Windows security section below), and the per-user jobs root lives\n # under LOCALAPPDATA (falling back to the user temp dir) with a hardened ACL\n # so the jobs root has a working default rather than a required override.\n _WIN_ROOT_BASE = os.environ.get(\"LOCALAPPDATA\") or tempfile.gettempdir()\n DEFAULT_ROOT = os.path.join(_WIN_ROOT_BASE, \"compound-engineering-jobs\")\nelif _EFFECTIVE_UID is not None:\n DEFAULT_ROOT = os.path.join(\"/tmp\", f\"compound-engineering-{_EFFECTIVE_UID}\")\nelse:\n DEFAULT_ROOT = None\nO_NOFOLLOW = getattr(os, \"O_NOFOLLOW\", 0)\n# Windows CPython opens os.open() descriptors in CRT *text* mode by default:\n# writes expand \\n -> \\r\\n and reads stop at the first 0x1A (Ctrl-Z EOF), which\n# would silently corrupt and truncate a peer's result artifact and desync the\n# out.log byte caps from st_size. O_BINARY is 0 on POSIX, so this is a no-op\n# there and every os.open below stays byte-exact on both platforms.\nO_BINARY = getattr(os, \"O_BINARY\", 0)\nSWEEP_AGE_SECS = 24 * 3600\nCLAIM_ATTEMPTS = 16\nSTATUS_READ_CAP = 256\nREASON_READ_CAP = 1024\nMETA_READ_CAP = 64 * 1024\nSTARTED_AT_FORMAT = \"%Y-%m-%dT%H:%M:%SZ\"\nREAP_SETTLE_SECS = 15\n\nEXIT_CODES_DOC = \"\"\"\\\nexit codes:\n 0 the command itself succeeded. For status/wait this means the query ran;\n it says nothing about job outcomes — parse stdout (or --json) for states.\n For `result` it means a done job's artifact (or a --path file) was emitted;\n for reap it includes the safe no-op on an already-terminal job.\n 1 runtime error (preflight failure, unknown job, detach failure); for a\n multi-job reap, also the code when its jobs failed in different ways\n 2 usage error; for `result`: the job is still running\n 3 for `result`: job settled but not done (failed / timeout /\n died-without-result / never-started), or the result file is missing\n 4 the read was refused, so content is never emitted: the ownership check\n failed (job state or result not owned by the current user), or the path\n is there but unreadable (a symlink rejected by O_NOFOLLOW, a non-regular\n file, a byte-cap overrun). Only a genuinely absent file is 3.\n\nenvironment overrides: CE_PEER_JOBS_ROOT, CE_WORK_RUNS_ROOT, CE_PEER_IDLE_SECS,\nCE_PEER_HARD_SECS, CROSS_MODEL_HARD_SECS, CE_PEER_LOG_MAX_BYTES,\nCE_PEER_RESULT_MAX_BYTES, CE_PEER_POLL_SECS, CE_PEER_GRACE_SECS (defaults in\nthe module docstring).\n\"\"\"\n\n\nclass RunnerError(Exception):\n \"\"\"Actionable operational error: message to stderr, exit 1.\"\"\"\n\n\nclass Unreadable(Exception):\n \"\"\"Job state failed an ownership or sanity check; content withheld.\"\"\"\n\n\n# --- configuration -----------------------------------------------------------\n\n# Supervisor hard-window floor: clears the highest cross-model worker default\n# (review skills use CROSS_MODEL_HARD_SECS:-1200) so an unset knob still nests\n# worker \u003c deadline \u003c runner without orchestrator arithmetic. Grace matches the\n# historical prose +30s so a raised knob widens the runner the same way.\n_RUNNER_HARD_FLOOR = 1230.0\n_RUNNER_HARD_GRACE = 30.0\n\n\ndef _private_root_usable(path: str) -> bool:\n \"\"\"True when `path` is (or can now be) a directory we own and can write into.\n\n Creation is the probe: a sandbox that denies writes under /tmp refuses the\n mkdir, and one that lets a pre-existing root stand still fails the access\n check, so both land on the fallback instead of failing at the first job.\n \"\"\"\n try:\n os.mkdir(path, 0o700)\n except FileExistsError:\n pass\n except OSError:\n return False\n try:\n _check_owned_dir(path)\n except (OSError, RunnerError):\n return False\n return os.access(path, os.W_OK)\n\n\ndef _fallback_root() -> str:\n return os.path.join(os.environ.get(\"TMPDIR\") or \"/tmp\", f\"compound-engineering-{_EFFECTIVE_UID}\")\n\n\ndef jobs_root_base() -> str:\n configured = os.environ.get(\"CE_PEER_JOBS_ROOT\")\n if configured:\n return os.path.abspath(configured)\n if DEFAULT_ROOT is None:\n raise RunnerError(\"effective user ID is unavailable; cannot derive the jobs root\")\n if IS_WINDOWS or _private_root_usable(DEFAULT_ROOT):\n return os.path.abspath(DEFAULT_ROOT)\n # Same order and candidates as the skills' shell preamble, so a job started\n # there is found here.\n return os.path.abspath(_fallback_root())\n\n\ndef candidate_jobs_root_bases() -> list:\n \"\"\"Every root an existing job may live under: the configured root alone, or\n both the /tmp root and the $TMPDIR fallback (deduplicated, primary first).\n\n Creation uses jobs_root_base(); lookup of an already-started job must not\n depend on which root *this* invocation would create under, because a\n sandboxed session and a later unsandboxed one resolve different roots.\n \"\"\"\n configured = os.environ.get(\"CE_PEER_JOBS_ROOT\")\n if configured:\n return [os.path.abspath(configured)]\n if DEFAULT_ROOT is None:\n raise RunnerError(\"effective user ID is unavailable; cannot derive the jobs root\")\n bases = [os.path.abspath(DEFAULT_ROOT)]\n if not IS_WINDOWS:\n fallback = os.path.abspath(_fallback_root())\n if fallback not in bases:\n bases.append(fallback)\n return bases\n\n\ndef skill_runs_root(skill: str) -> str:\n if skill == \"ce-work\" and os.environ.get(\"CE_WORK_RUNS_ROOT\"):\n return os.path.abspath(os.environ[\"CE_WORK_RUNS_ROOT\"])\n return os.path.join(jobs_root_base(), skill)\n\n\ndef candidate_skill_runs_roots(skill: str) -> list:\n if skill == \"ce-work\" and os.environ.get(\"CE_WORK_RUNS_ROOT\"):\n return [os.path.abspath(os.environ[\"CE_WORK_RUNS_ROOT\"])]\n return [os.path.join(base, skill) for base in candidate_jobs_root_bases()]\n\n\ndef _env_num(name: str, default: float, conv, *, allow_zero: bool = False):\n raw = os.environ.get(name)\n if not raw:\n return default\n try:\n val = conv(raw)\n except ValueError:\n return default\n if allow_zero and val == 0:\n return None\n return val if val > 0 else default\n\n\ndef _derived_hard_default() -> float:\n \"\"\"Outermost supervisor hard window when CE_PEER_HARD_SECS is unset.\n\n Reads ambient CROSS_MODEL_HARD_SECS (the runner already forwards os.environ\n to the worker, so a user-set knob is present here). Explicit CE_PEER_HARD_SECS\n still wins via cfg() — ce-work and elevation paths keep their own windows.\n \"\"\"\n cross = _env_num(\"CROSS_MODEL_HARD_SECS\", 0.0, float)\n return max(_RUNNER_HARD_FLOOR, cross + _RUNNER_HARD_GRACE)\n\n\ndef cfg(skill=None) -> dict:\n return {\n \"idle\": _env_num(\"CE_PEER_IDLE_SECS\", 240.0, float, allow_zero=skill == \"ce-work\"),\n \"hard\": _env_num(\"CE_PEER_HARD_SECS\", _derived_hard_default(), float),\n \"log_max\": int(_env_num(\"CE_PEER_LOG_MAX_BYTES\", 10 * 1024 * 1024, int)),\n \"result_max\": int(_env_num(\"CE_PEER_RESULT_MAX_BYTES\", 5 * 1024 * 1024, int)),\n \"poll\": _env_num(\"CE_PEER_POLL_SECS\", 2.0, float),\n \"grace\": _env_num(\"CE_PEER_GRACE_SECS\", 5.0, float),\n }\n\n\n# --- Windows security + process primitives ------------------------------------\n#\n# POSIX ownership is `fstat().st_uid == geteuid()` plus mode 0700/0600. Windows\n# has neither uids nor mode bits, so the equivalent identity is the current\n# user's SID: a job dir/file is \"ours\" when its owner SID is one this process's\n# token creates objects as (the user SID or the token's default owner SID -- an\n# elevated process defaults new objects to Administrators). A foreign user's\n# planted dir carries neither SID and is rejected, exactly as a uid mismatch is\n# on POSIX. The DACL is hardened to user+SYSTEM+Administrators (root-equivalents,\n# mirroring how root still reaches a 0700 dir) with inheritance broken, so no\n# world/Users grant survives. Pure stdlib via ctypes -- no pywin32.\n\nif IS_WINDOWS:\n import ctypes\n import msvcrt\n from ctypes import wintypes\n\n _advapi32 = ctypes.WinDLL(\"advapi32\", use_last_error=True)\n _kernel32 = ctypes.WinDLL(\"kernel32\", use_last_error=True)\n\n _SE_FILE_OBJECT = 1\n _OWNER_SECURITY_INFORMATION = 0x00000001\n _TOKEN_QUERY = 0x0008\n _TOKEN_USER_CLASS = 1\n _TOKEN_OWNER_CLASS = 4\n _STILL_ACTIVE = 259\n _WAIT_TIMEOUT = 0x00000102\n _PROCESS_QUERY_LIMITED_INFORMATION = 0x1000\n _SYNCHRONIZE = 0x00100000\n # A detached, console-less parent still gives its children a NEW console\n # unless this is set, so every job would flash a window on the user's\n # desktop. Applied to the worker and to every helper tool we shell out to.\n _WIN_NO_WINDOW = getattr(subprocess, \"CREATE_NO_WINDOW\", 0x08000000)\n\n def _win_tool(name: str) -> str:\n \"\"\"Absolute path to a System32 tool. CreateProcess searches the\n application and current directories before System32, so invoking\n `icacls`/`taskkill` by bare name from an untrusted CWD is a binary-\n hijack surface. Falls back to the bare name only if System32 is\n unresolvable, which is strictly better than never running.\"\"\"\n root = os.environ.get(\"SystemRoot\") or r\"C:\\Windows\"\n candidate = os.path.join(root, \"System32\", name + \".exe\")\n return candidate if os.path.isfile(candidate) else name\n\n _advapi32.ConvertSidToStringSidW.argtypes = [\n ctypes.c_void_p, ctypes.POINTER(ctypes.c_wchar_p)]\n _advapi32.ConvertSidToStringSidW.restype = wintypes.BOOL\n _advapi32.OpenProcessToken.argtypes = [\n wintypes.HANDLE, wintypes.DWORD, ctypes.POINTER(wintypes.HANDLE)]\n _advapi32.OpenProcessToken.restype = wintypes.BOOL\n _advapi32.GetTokenInformation.argtypes = [\n wintypes.HANDLE, ctypes.c_int, ctypes.c_void_p, wintypes.DWORD,\n ctypes.POINTER(wintypes.DWORD)]\n _advapi32.GetTokenInformation.restype = wintypes.BOOL\n _advapi32.GetSecurityInfo.argtypes = [\n wintypes.HANDLE, ctypes.c_int, wintypes.DWORD,\n ctypes.POINTER(ctypes.c_void_p), ctypes.c_void_p, ctypes.c_void_p,\n ctypes.c_void_p, ctypes.POINTER(ctypes.c_void_p)]\n _advapi32.GetSecurityInfo.restype = wintypes.DWORD\n _advapi32.GetNamedSecurityInfoW.argtypes = [\n wintypes.LPCWSTR, ctypes.c_int, wintypes.DWORD,\n ctypes.POINTER(ctypes.c_void_p), ctypes.c_void_p, ctypes.c_void_p,\n ctypes.c_void_p, ctypes.POINTER(ctypes.c_void_p)]\n _advapi32.GetNamedSecurityInfoW.restype = wintypes.DWORD\n _kernel32.GetCurrentProcess.restype = wintypes.HANDLE\n _kernel32.OpenProcess.argtypes = [\n wintypes.DWORD, wintypes.BOOL, wintypes.DWORD]\n _kernel32.OpenProcess.restype = wintypes.HANDLE\n _kernel32.GetProcessTimes.argtypes = [\n wintypes.HANDLE,\n ctypes.POINTER(wintypes.FILETIME),\n ctypes.POINTER(wintypes.FILETIME),\n ctypes.POINTER(wintypes.FILETIME),\n ctypes.POINTER(wintypes.FILETIME),\n ]\n _kernel32.GetProcessTimes.restype = wintypes.BOOL\n _kernel32.QueryFullProcessImageNameW.argtypes = [\n wintypes.HANDLE, wintypes.DWORD, wintypes.LPWSTR,\n ctypes.POINTER(wintypes.DWORD)]\n _kernel32.QueryFullProcessImageNameW.restype = wintypes.BOOL\n _kernel32.WaitForSingleObject.argtypes = [wintypes.HANDLE, wintypes.DWORD]\n _kernel32.WaitForSingleObject.restype = wintypes.DWORD\n _kernel32.CloseHandle.argtypes = [wintypes.HANDLE]\n _kernel32.CloseHandle.restype = wintypes.BOOL\n _kernel32.LocalFree.argtypes = [wintypes.HGLOBAL]\n _kernel32.LocalFree.restype = wintypes.HGLOBAL\n _kernel32.CreateJobObjectW.argtypes = [ctypes.c_void_p, wintypes.LPCWSTR]\n _kernel32.CreateJobObjectW.restype = wintypes.HANDLE\n _kernel32.OpenJobObjectW.argtypes = [\n wintypes.DWORD, wintypes.BOOL, wintypes.LPCWSTR]\n _kernel32.OpenJobObjectW.restype = wintypes.HANDLE\n _kernel32.AssignProcessToJobObject.argtypes = [\n wintypes.HANDLE, wintypes.HANDLE]\n _kernel32.AssignProcessToJobObject.restype = wintypes.BOOL\n _kernel32.TerminateJobObject.argtypes = [wintypes.HANDLE, wintypes.UINT]\n _kernel32.TerminateJobObject.restype = wintypes.BOOL\n\n _JOB_OBJECT_TERMINATE = 0x0008\n _PROCESS_TERMINATE = 0x0001\n _PROCESS_SET_QUOTA = 0x0100\n _TH32CS_SNAPPROCESS = 0x00000002\n _TH32CS_SNAPTHREAD = 0x00000004\n _THREAD_SUSPEND_RESUME = 0x0002\n # CreateProcess CREATE_SUSPENDED: primary thread starts frozen so we can\n # AssignProcessToJobObject before any user code (or child spawn) runs.\n _CREATE_SUSPENDED = 0x00000004\n\n class _PROCESSENTRY32W(ctypes.Structure):\n _fields_ = [\n (\"dwSize\", wintypes.DWORD),\n (\"cntUsage\", wintypes.DWORD),\n (\"th32ProcessID\", wintypes.DWORD),\n (\"th32DefaultHeapID\", ctypes.POINTER(ctypes.c_ulong)),\n (\"th32ModuleID\", wintypes.DWORD),\n (\"cntThreads\", wintypes.DWORD),\n (\"th32ParentProcessID\", wintypes.DWORD),\n (\"pcPriClassBase\", ctypes.c_long),\n (\"dwFlags\", wintypes.DWORD),\n (\"szExeFile\", ctypes.c_wchar * 260),\n ]\n\n class _THREADENTRY32(ctypes.Structure):\n _fields_ = [\n (\"dwSize\", wintypes.DWORD),\n (\"cntUsage\", wintypes.DWORD),\n (\"th32ThreadID\", wintypes.DWORD),\n (\"th32OwnerProcessID\", wintypes.DWORD),\n (\"tpBasePri\", ctypes.c_long),\n (\"tpDeltaPri\", ctypes.c_long),\n (\"dwFlags\", wintypes.DWORD),\n ]\n\n _kernel32.CreateToolhelp32Snapshot.argtypes = [wintypes.DWORD, wintypes.DWORD]\n _kernel32.CreateToolhelp32Snapshot.restype = ctypes.c_void_p\n _kernel32.Process32FirstW.argtypes = [\n ctypes.c_void_p, ctypes.POINTER(_PROCESSENTRY32W)]\n _kernel32.Process32FirstW.restype = wintypes.BOOL\n _kernel32.Process32NextW.argtypes = [\n ctypes.c_void_p, ctypes.POINTER(_PROCESSENTRY32W)]\n _kernel32.Process32NextW.restype = wintypes.BOOL\n _kernel32.Thread32First.argtypes = [\n ctypes.c_void_p, ctypes.POINTER(_THREADENTRY32)]\n _kernel32.Thread32First.restype = wintypes.BOOL\n _kernel32.Thread32Next.argtypes = [\n ctypes.c_void_p, ctypes.POINTER(_THREADENTRY32)]\n _kernel32.Thread32Next.restype = wintypes.BOOL\n _kernel32.OpenThread.argtypes = [\n wintypes.DWORD, wintypes.BOOL, wintypes.DWORD]\n _kernel32.OpenThread.restype = wintypes.HANDLE\n _kernel32.ResumeThread.argtypes = [wintypes.HANDLE]\n _kernel32.ResumeThread.restype = wintypes.DWORD\n _kernel32.TerminateProcess.argtypes = [wintypes.HANDLE, wintypes.UINT]\n _kernel32.TerminateProcess.restype = wintypes.BOOL\n\n def _win_process_children_map():\n \"\"\"th32ParentProcessID -> [child pids] from one Toolhelp snapshot.\"\"\"\n snap = _kernel32.CreateToolhelp32Snapshot(_TH32CS_SNAPPROCESS, 0)\n if not snap or snap == ctypes.c_void_p(-1).value:\n return {}\n children = {}\n try:\n entry = _PROCESSENTRY32W()\n entry.dwSize = ctypes.sizeof(_PROCESSENTRY32W)\n more = _kernel32.Process32FirstW(snap, ctypes.byref(entry))\n while more:\n children.setdefault(entry.th32ParentProcessID, []).append(\n entry.th32ProcessID)\n more = _kernel32.Process32NextW(snap, ctypes.byref(entry))\n finally:\n _kernel32.CloseHandle(ctypes.c_void_p(snap))\n return children\n\n def _win_descendants_deepest_first(root_pid: int, children=None):\n \"\"\"Children before parents, via a process snapshot. This is the direct\n analog of the POSIX `ps`-based walk and carries the same pid-reuse\n exposure. It works on an EXITED leader because Windows never reparents\n orphans: a dead pid still appears as th32ParentProcessID on its live\n children (unlike POSIX, where orphans are reparented to init).\"\"\"\n if children is None:\n children = _win_process_children_map()\n order, queue = [], [root_pid]\n while queue:\n for child in children.get(queue.pop(0), []):\n order.append(child)\n queue.append(child)\n return list(reversed(order))\n\n def _win_process_identity(pid: int):\n \"\"\"Creation time plus image path — analog of `ps -o lstart= -o command=`.\n\n Creation time is the PID-reuse guard: Windows recycles PIDs aggressively,\n and a recycled pid always carries a later FILETIME than the worker we\n recorded. None means unproven (gone or unopenable).\"\"\"\n handle = _kernel32.OpenProcess(\n _PROCESS_QUERY_LIMITED_INFORMATION, False, pid)\n if not handle:\n return None\n try:\n created, exited, kernel, user = (wintypes.FILETIME() for _ in range(4))\n if not _kernel32.GetProcessTimes(\n handle, ctypes.byref(created), ctypes.byref(exited),\n ctypes.byref(kernel), ctypes.byref(user)):\n return None\n started = (created.dwHighDateTime \u003c\u003c 32) | created.dwLowDateTime\n if not started:\n return None\n size = wintypes.DWORD(32768)\n buf = ctypes.create_unicode_buffer(size.value)\n image = (buf.value if _kernel32.QueryFullProcessImageNameW(\n handle, 0, buf, ctypes.byref(size)) else \"\")\n finally:\n _kernel32.CloseHandle(handle)\n return \"{} {}\".format(started, image)\n\n def _win_process_start_time(pid: int):\n handle = _kernel32.OpenProcess(\n _PROCESS_QUERY_LIMITED_INFORMATION, False, pid)\n if not handle:\n return None\n try:\n created, exited, kernel, user = (wintypes.FILETIME() for _ in range(4))\n if not _kernel32.GetProcessTimes(\n handle, ctypes.byref(created), ctypes.byref(exited),\n ctypes.byref(kernel), ctypes.byref(user)):\n return None\n started = (created.dwHighDateTime \u003c\u003c 32) | created.dwLowDateTime\n return started or None\n finally:\n _kernel32.CloseHandle(handle)\n\n def _win_process_identity_matches(pid: int, recorded) -> bool:\n if not recorded:\n return True\n current = _win_process_identity(pid)\n return current is not None and current == recorded\n\n def _win_terminate_pid(pid: int) -> bool:\n if pid \u003c= 0 or pid == os.getpid():\n return False\n handle = _kernel32.OpenProcess(_PROCESS_TERMINATE, False, pid)\n if not handle:\n return False\n try:\n return bool(_kernel32.TerminateProcess(handle, 1))\n finally:\n _kernel32.CloseHandle(handle)\n\n def _win_job_name(job_dir: str) -> str:\n \"\"\"A per-job named kernel object. Naming it is what makes this a real\n pgid analog: a DIFFERENT process (cmd_reap, after the supervisor is\n gone) can reopen it by name and terminate the whole tree.\"\"\"\n return \"Local\\\\ce-peer-job-\" + os.path.basename(job_dir.rstrip(\"\\\\/\"))\n\n def _win_create_job(name: str):\n \"\"\"Create the job the worker tree will live in. Deliberately WITHOUT\n JOB_OBJECT_LIMIT_KILL_ON_JOB_CLOSE: like a POSIX process group, the job\n must outlive the supervisor so a dead-supervisor reap still finds a\n live tree to classify and sweep (matching the POSIX lifecycle tests).\"\"\"\n handle = _kernel32.CreateJobObjectW(None, name)\n return handle or None\n\n def _win_assign_to_job(job_handle, pid: int) -> bool:\n proc = _kernel32.OpenProcess(\n _PROCESS_SET_QUOTA | _PROCESS_TERMINATE, False, pid)\n if not proc:\n return False\n try:\n return bool(_kernel32.AssignProcessToJobObject(job_handle, proc))\n finally:\n _kernel32.CloseHandle(proc)\n\n def _win_resume_process(pid: int) -> bool:\n \"\"\"Resume every thread of a CREATE_SUSPENDED process. subprocess.Popen\n does not expose hThread from PROCESS_INFORMATION, so walk the thread\n snapshot. CREATE_SUSPENDED only freezes the primary thread; resuming\n all owned threads is still correct and idempotent for running ones.\"\"\"\n snap = _kernel32.CreateToolhelp32Snapshot(_TH32CS_SNAPTHREAD, 0)\n if not snap or snap == ctypes.c_void_p(-1).value:\n return False\n resumed = False\n try:\n entry = _THREADENTRY32()\n entry.dwSize = ctypes.sizeof(_THREADENTRY32)\n more = _kernel32.Thread32First(snap, ctypes.byref(entry))\n while more:\n if entry.th32OwnerProcessID == pid:\n handle = _kernel32.OpenThread(\n _THREAD_SUSPEND_RESUME, False, entry.th32ThreadID)\n if handle:\n try:\n # (DWORD)-1 == failure; 0xFFFFFFFF as unsigned.\n if _kernel32.ResumeThread(handle) != 0xFFFFFFFF:\n resumed = True\n finally:\n _kernel32.CloseHandle(handle)\n more = _kernel32.Thread32Next(snap, ctypes.byref(entry))\n finally:\n _kernel32.CloseHandle(ctypes.c_void_p(snap))\n return resumed\n\n def _win_terminate_job(name: str) -> bool:\n \"\"\"Terminate every process in the named job, whatever the tree shape.\n This is the piece taskkill /T cannot do: it reaches descendants whose\n parent has already exited, because job membership is inherited and\n does not depend on a live parent to walk from.\"\"\"\n handle = _kernel32.OpenJobObjectW(_JOB_OBJECT_TERMINATE, False, name)\n if not handle:\n return False\n try:\n return bool(_kernel32.TerminateJobObject(handle, 1))\n finally:\n _kernel32.CloseHandle(handle)\n\n _WIN_IDENTITY_SIDS = None\n\n def _win_sid_to_string(psid) -> str:\n strp = ctypes.c_wchar_p()\n if not _advapi32.ConvertSidToStringSidW(psid, ctypes.byref(strp)):\n raise OSError(f\"ConvertSidToStringSid failed: {ctypes.get_last_error()}\")\n try:\n return strp.value\n finally:\n _kernel32.LocalFree(ctypes.cast(strp, wintypes.HGLOBAL))\n\n def _win_token_sid(token, info_class) -> str:\n size = wintypes.DWORD(0)\n _advapi32.GetTokenInformation(token, info_class, None, 0, ctypes.byref(size))\n buf = (ctypes.c_byte * size.value)()\n if not _advapi32.GetTokenInformation(\n token, info_class, buf, size, ctypes.byref(size)\n ):\n raise OSError(f\"GetTokenInformation failed: {ctypes.get_last_error()}\")\n # TOKEN_USER / TOKEN_OWNER both begin with a PSID at offset 0.\n sid_ptr = ctypes.cast(buf, ctypes.POINTER(ctypes.c_void_p))[0]\n return _win_sid_to_string(ctypes.c_void_p(sid_ptr))\n\n def _win_identity_sids() -> frozenset:\n \"\"\"The SID strings this process's token creates objects as: the user SID\n and the default-owner SID (they differ for an elevated process). Cached;\n an object owned by any of these is treated as ours.\"\"\"\n global _WIN_IDENTITY_SIDS\n if _WIN_IDENTITY_SIDS is not None:\n return _WIN_IDENTITY_SIDS\n token = wintypes.HANDLE()\n if not _advapi32.OpenProcessToken(\n _kernel32.GetCurrentProcess(), _TOKEN_QUERY, ctypes.byref(token)\n ):\n raise OSError(f\"OpenProcessToken failed: {ctypes.get_last_error()}\")\n try:\n sids = {\n _win_token_sid(token, _TOKEN_USER_CLASS),\n _win_token_sid(token, _TOKEN_OWNER_CLASS),\n }\n finally:\n _kernel32.CloseHandle(token)\n _WIN_IDENTITY_SIDS = frozenset(s for s in sids if s)\n return _WIN_IDENTITY_SIDS\n\n def _win_owner_sid(api, target) -> str:\n \"\"\"Shared GetSecurityInfo / GetNamedSecurityInfoW shape: read the OWNER\n SID into a freshly allocated security descriptor and stringify it. The\n SID points INSIDE that descriptor, so freeing the descriptor is the only\n (and required) cleanup -- never free the SID separately.\"\"\"\n psid = ctypes.c_void_p()\n psd = ctypes.c_void_p()\n err = api(target, _SE_FILE_OBJECT, _OWNER_SECURITY_INFORMATION,\n ctypes.byref(psid), None, None, None, ctypes.byref(psd))\n if err != 0:\n raise OSError(f\"{api.__name__} failed: {err}\")\n try:\n return _win_sid_to_string(psid)\n finally:\n _kernel32.LocalFree(ctypes.cast(psd, wintypes.HGLOBAL))\n\n def _win_owner_sid_from_handle(handle: int) -> str:\n return _win_owner_sid(_advapi32.GetSecurityInfo, wintypes.HANDLE(handle))\n\n def _win_owner_sid_from_path(path: str) -> str:\n return _win_owner_sid(_advapi32.GetNamedSecurityInfoW, path)\n\n def _win_owns_path(path: str) -> bool:\n return _win_owner_sid_from_path(path) in _win_identity_sids()\n\n def _win_owns_handle(handle: int) -> bool:\n return _win_owner_sid_from_handle(handle) in _win_identity_sids()\n\n def _win_run_quiet(cmd) -> bool:\n \"\"\"Fire-and-forget a Windows tool (icacls/taskkill): output suppressed,\n exit status returned but never raised. check=False suppresses a NONZERO\n exit, NOT a missing executable -- Popen still raises FileNotFoundError\n when the tool is absent from PATH, which would otherwise escape the\n supervisor's teardown and turn an already-classified `done` job into\n `failed`. Returns True only when the tool ran and exited 0.\"\"\"\n try:\n return subprocess.run(\n cmd, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL,\n check=False, creationflags=_WIN_NO_WINDOW).returncode == 0\n except OSError:\n return False\n\n def _win_harden_acl(path: str) -> None:\n \"\"\"Break inheritance and grant only the current user plus the\n root-equivalents (SYSTEM, Administrators), so no world/Users grant\n survives -- the 0700 analog. Best-effort: the owner check and the\n O_EXCL/O_CREAT claim remain the hard gates if icacls is unavailable.\"\"\"\n sids = _win_identity_sids()\n if not sids:\n return False\n inherit = \"(OI)(CI)\" if os.path.isdir(path) else \"\"\n # Grant EVERY identity SID, not an arbitrary one from the set: an\n # elevated token carries two (user + default owner), and picking one\n # nondeterministically could grant the wrong principal.\n grants = []\n for sid in sorted(sids) + [\"S-1-5-18\", \"S-1-5-32-544\"]:\n grants += [\"/grant:r\", f\"*{sid}:{inherit}F\"]\n return _win_run_quiet([_win_tool(\"icacls\"), path, \"/inheritance:r\"] + grants)\n\n def _win_pid_alive(pid: int) -> bool:\n handle = _kernel32.OpenProcess(\n _PROCESS_QUERY_LIMITED_INFORMATION | _SYNCHRONIZE, False, pid)\n if not handle:\n return False\n try:\n return _kernel32.WaitForSingleObject(handle, 0) == _WAIT_TIMEOUT\n finally:\n _kernel32.CloseHandle(handle)\n\n def _win_kill_tree(\n root_pid: int, grace: float, job_name=None, expected_identity=None,\n ) -> bool:\n \"\"\"Terminate the worker tree. Returns whether the LEADER was\n alive when the kill began -- the reap classification signal -- which is\n independent of how much of the tree we then sweep.\n\n The job object is the primary mechanism and the true killpg analog: it\n reaches descendants even after the leader has exited. taskkill /T can\n NOT -- it walks parent->child from a live parent, so against an exited\n pid it returns \"process not found\" and silently leaves grandchildren\n running forever. That is why the sweep is attempted whenever a job name\n exists, regardless of leader liveness.\n\n No graceful phase: a console-less worker cannot receive taskkill's\n WM_CLOSE (it reports \"can only be terminated forcefully\"), so the grace\n window was pure latency that also widened the cmd_reap race.\n\n The job is only reachable by the process that created it: Windows\n releases a named object's NAME once the last handle closes, even while\n member processes keep the object alive (verified: OpenJobObject then\n fails with ERROR_FILE_NOT_FOUND). So a cmd_reap running after the\n supervisor died cannot use it, and falls back to the snapshot walk --\n which is exactly the dead-leader case, hence deepest-first descendants\n BEFORE the leader, and never gated on leader liveness.\n\n A live pid at root_pid is not automatically the original leader:\n Windows recycles PIDs, and cmd_reap is often the next python.exe after\n the worker exits (the orphan-grandchild smoke). Never terminate this\n process, and never terminate a live pid whose GetProcessTimes identity\n does not match the recorded worker. Stale-PPID orphans still show the\n dead leader as parent. When the pid was reused, the start-time cutoff\n applies only to *direct* children of that pid (the new process's own\n children vs stale-PPID orphans). A pre-reuse child's full subtree is\n still original-tree work, including descendants spawned after reuse.\"\"\"\n self_pid = os.getpid()\n is_self = root_pid == self_pid\n alive = (not is_self) and _win_pid_alive(root_pid)\n recorded_leader = alive and _win_process_identity_matches(\n root_pid, expected_identity)\n if job_name:\n _win_terminate_job(job_name)\n # Always Toolhelp-sweep after (or without) the job terminate: children\n # that raced outside the job before AssignProcessToJobObject completed\n # are not members, and TerminateJobObject alone would leave them.\n # CREATE_SUSPENDED closes that spawn race; this remains the belt.\n children_map = _win_process_children_map()\n reuse_cutoff = None\n if not recorded_leader and (is_self or alive):\n reuse_cutoff = _win_process_start_time(root_pid)\n if reuse_cutoff is not None:\n def _predates_reuse(pid):\n started = _win_process_start_time(pid)\n return started is None or started \u003c reuse_cutoff\n kill_set = _pre_reuse_descendant_pids(\n root_pid, children_map, _predates_reuse, self_pid)\n else:\n kill_set = None\n for pid in _win_descendants_deepest_first(root_pid, children_map):\n if pid == self_pid:\n continue\n if kill_set is not None and pid not in kill_set:\n continue\n _win_terminate_pid(pid)\n if recorded_leader:\n _win_terminate_pid(root_pid)\n return recorded_leader\n\n\n# --- hardened I/O primitives --------------------------------------------------\n\ndef _euid():\n return _EFFECTIVE_UID\n\n\ndef _check_owned_dir(path: str, require_private: bool = False) -> None:\n st = os.lstat(path)\n if not stat.S_ISDIR(st.st_mode):\n raise RunnerError(f\"{path}: not a real directory (symlink or file planted?)\")\n if IS_WINDOWS:\n # SID ownership stands in for st_uid; the hardened ACL (not a mode bit)\n # provides privacy, so there is no separate require_private gate.\n if not _win_owns_path(path):\n raise RunnerError(f\"{path}: not owned by the current user\")\n return\n euid = _euid()\n if euid is not None and st.st_uid != euid:\n raise RunnerError(f\"{path}: not owned by the current user\")\n if require_private:\n mode = stat.S_IMODE(st.st_mode)\n if mode != 0o700:\n raise RunnerError(f\"{path}: must have mode 0700, found {mode:04o}\")\n\n\ndef ensure_owned_dirs(base: str, path: str) -> None:\n \"\"\"mkdir -p `path` (mode 0700) verifying owner and type on every component\n from `base` down — a planted symlink or foreign dir aborts, never traversed.\"\"\"\n rel = os.path.relpath(path, base)\n comps = [] if rel == \".\" else rel.split(os.sep)\n cur = base\n created_base = True\n try:\n os.mkdir(cur, 0o700)\n except FileExistsError:\n created_base = False\n _check_owned_dir(cur)\n if IS_WINDOWS:\n # `icacls /inheritance:r` is destructive and irreversible in a way\n # POSIX's chmod 0700 is not: it permanently drops inherited ACEs. So\n # only re-ACL a root this runner owns -- one we just created, or the\n # managed default (repairing a default left non-private, which is what\n # the POSIX unconditional chmod is for). A pre-existing user-supplied\n # CE_PEER_JOBS_ROOT keeps its ACLs and rests on the owner check.\n default_root = os.path.abspath(DEFAULT_ROOT) if DEFAULT_ROOT else None\n ours = created_base or (\n default_root is not None\n and os.path.normcase(cur) == os.path.normcase(default_root))\n if ours and not _win_harden_acl(cur):\n # Never proceed as if hardened: an unverified root is the one case\n # where the privacy half of the model would silently be missing.\n raise RunnerError(\n f\"{cur}: could not harden the jobs-root ACL (icacls failed or \"\n \"is unavailable); refusing to use a root whose privacy is \"\n \"unverified\"\n )\n else:\n os.chmod(cur, 0o700)\n _check_owned_dir(cur, require_private=True)\n for comp in comps:\n cur = os.path.join(cur, comp)\n created = False\n try:\n os.mkdir(cur, 0o700)\n created = True\n except FileExistsError:\n pass\n if created:\n if IS_WINDOWS:\n _win_harden_acl(cur)\n else:\n os.chmod(cur, 0o700)\n _check_owned_dir(cur)\n\n\ndef read_owned(path: str, cap: int) -> bytes:\n \"\"\"Open no-follow, verify the OPENED descriptor's owner via fstat, enforce\n the size cap, and return content. Raises Unreadable on any trust failure.\"\"\"\n fd = os.open(path, os.O_RDONLY | O_NOFOLLOW | O_BINARY)\n try:\n st = os.fstat(fd)\n if IS_WINDOWS:\n # Verify the OPENED handle's owner SID (TOCTOU-safe, like the POSIX\n # fstat-by-fd check) before emitting a byte.\n if not _win_owns_handle(msvcrt.get_osfhandle(fd)):\n raise Unreadable(f\"{path}: not owned by the current user; refusing to read\")\n else:\n euid = _euid()\n if euid is not None and st.st_uid != euid:\n raise Unreadable(f\"{path}: not owned by the current user; refusing to read\")\n if not stat.S_ISREG(st.st_mode):\n raise Unreadable(f\"{path}: not a regular file\")\n if st.st_size > cap:\n raise Unreadable(f\"{path}: {st.st_size} bytes exceeds the {cap}-byte read cap\")\n chunks = []\n got = 0\n while got \u003c= cap:\n chunk = os.read(fd, 65536)\n if not chunk:\n break\n chunks.append(chunk)\n got += len(chunk)\n if got > cap:\n raise Unreadable(f\"{path}: grew past the {cap}-byte read cap during read\")\n return b\"\".join(chunks)\n finally:\n os.close(fd)\n\n\ndef create_exclusive(path: str, data: bytes = b\"\", mode: int = 0o600) -> None:\n fd = os.open(path, os.O_WRONLY | os.O_CREAT | os.O_EXCL | O_NOFOLLOW | O_BINARY, mode)\n try:\n if data:\n os.write(fd, data)\n finally:\n os.close(fd)\n\n\ndef write_atomic(path: str, data: bytes) -> None:\n fd, tmp = tempfile.mkstemp(dir=os.path.dirname(path), prefix=\".tmp-\")\n try:\n with os.fdopen(fd, \"wb\") as f:\n f.write(data)\n os.replace(tmp, path)\n except BaseException:\n try:\n os.unlink(tmp)\n except OSError:\n pass\n raise\n\n\ndef write_terminal(job_dir: str, state: str, reason: str, overwrite: bool = True) -> None:\n \"\"\"Publish the single terminal record. The reason detail lands FIRST so the\n atomic status rename is always the last record; a reason write failure never\n blocks the status.\"\"\"\n status_path = os.path.join(job_dir, \"status\")\n if not overwrite and os.path.lexists(status_path):\n return\n try:\n write_atomic(os.path.join(job_dir, \"reason\"), (reason.rstrip(\"\\n\") + \"\\n\").encode())\n except OSError:\n pass\n write_atomic(status_path, (state + \"\\n\").encode())\n\n\n# --- job identity and resolution ----------------------------------------------\n\ndef mint_job_id() -> str:\n return f\"{time.strftime('%Y%m%dT%H%M%SZ', time.gmtime())}-{os.urandom(4).hex()}\"\n\n\ndef claim_job_dir(jobs_root: str):\n \"\"\"Atomically claim a fresh job dir: os.mkdir (no -p) fails on collision,\n so the id is regenerated rather than a dir ever being shared.\"\"\"\n for _ in range(CLAIM_ATTEMPTS):\n job_id = mint_job_id()\n job_dir = os.path.join(jobs_root, job_id)\n try:\n os.mkdir(job_dir, 0o700)\n return job_id, job_dir\n except FileExistsError:\n continue\n raise RunnerError(f\"could not claim a unique job dir after {CLAIM_ATTEMPTS} attempts\")\n\n\ndef resolve_job_dir(ref: str, skill=None) -> str:\n if os.sep in ref:\n p = os.path.abspath(ref)\n if os.path.isdir(p):\n return p\n raise RunnerError(f\"no such job dir: {ref}\")\n if not _is_safe_token(ref):\n raise RunnerError(f\"invalid job ref: {ref!r}\")\n if skill is not None:\n if not _is_safe_token(skill):\n raise RunnerError(f\"invalid skill: {skill!r}\")\n search_roots = candidate_skill_runs_roots(skill)\n patterns = [os.path.join(root, \"*\", \"jobs\", ref) for root in search_roots]\n else:\n search_roots = candidate_jobs_root_bases()\n patterns = [os.path.join(root, \"*\", \"*\", \"jobs\", ref) for root in search_roots]\n matches = sorted({match for pattern in patterns for match in glob.glob(pattern)})\n if not matches:\n raise RunnerError(f\"job not found under {', '.join(search_roots)}: {ref}\")\n if len(matches) > 1:\n raise RunnerError(f\"ambiguous job id {ref}: {len(matches)} matches; pass the job dir path\")\n return matches[0]\n\n\ndef job_state(job_dir: str) -> str:\n try:\n _check_owned_dir(job_dir)\n except (RunnerError, OSError):\n return \"unreadable\"\n try:\n word = read_owned(os.path.join(job_dir, \"status\"), STATUS_READ_CAP)\n word = word.decode(\"utf-8\", \"replace\").strip()\n return word if word in TERMINAL_STATES else \"unreadable\"\n except FileNotFoundError:\n pass\n except (Unreadable, OSError):\n return \"unreadable\"\n if os.path.lexists(os.path.join(job_dir, \"pid\")):\n return \"running\"\n return \"never-started\"\n\n\ndef job_reason(job_dir: str) -> str:\n \"\"\"The terminal record's detail line, or \"\" when unavailable. Decorative\n context for a message; never load-bearing, so every failure reads as \"\".\"\"\"\n try:\n raw = read_owned(os.path.join(job_dir, \"reason\"), REASON_READ_CAP)\n except (Unreadable, OSError):\n return \"\"\n return raw.decode(\"utf-8\", \"replace\").strip()\n\n\n# --- process-tree control -----------------------------------------------------\n\ndef _pid_alive(pid: int) -> bool:\n if IS_WINDOWS:\n return _win_pid_alive(pid)\n try:\n os.kill(pid, 0)\n return True\n except ProcessLookupError:\n return False\n except OSError:\n return True\n\n\ndef _pid_running(pid: int) -> bool:\n \"\"\"True only for a live process, NOT a \u003cdefunct> zombie. os.kill(pid, 0)\n succeeds for a zombie (the process exited but has not been reaped), which\n must not count as a live worker when classifying a reap: a zombie leader\n means the worker is gone (died-without-result), not still running (timeout).\n Falls back to the kill -0 result when process state is unavailable.\"\"\"\n if IS_WINDOWS:\n # Windows has no \u003cdefunct> zombie state -- a terminated process's handle\n # is signaled and OpenProcess-based liveness already reports it dead.\n return _win_pid_alive(pid)\n if not _pid_alive(pid):\n return False\n try:\n out = subprocess.run(\n [\"ps\", \"-o\", \"state=\", \"-p\", str(pid)],\n capture_output=True, text=True, check=False,\n ).stdout.strip()\n except OSError:\n return True\n if not out:\n return False\n return not out.startswith(\"Z\")\n\n\ndef _kill_quiet(pid: int, sig: int) -> bool:\n try:\n os.kill(pid, sig)\n return True\n except OSError:\n return False\n\n\ndef _killpg_quiet(pgid: int, sig: int) -> bool:\n try:\n os.killpg(pgid, sig)\n return True\n except OSError:\n return False\n\n\ndef _descendants_deepest_first(root_pid: int):\n \"\"\"Fallback tree enumeration via ps when a process-group kill is not\n available: children die before their parents can respawn or orphan them.\"\"\"\n try:\n out = subprocess.run(\n [\"ps\", \"-eo\", \"pid=,ppid=\"], capture_output=True, text=True, check=False\n ).stdout\n except OSError:\n return []\n children = {}\n for line in out.splitlines():\n parts = line.split()\n if len(parts) != 2:\n continue\n try:\n pid, ppid = int(parts[0]), int(parts[1])\n except ValueError:\n continue\n children.setdefault(ppid, []).append(pid)\n order, queue = [], [root_pid]\n while queue:\n for child in children.get(queue.pop(0), []):\n order.append(child)\n queue.append(child)\n return list(reversed(order))\n\n\ndef _signal_group_or_tree(pid: int, sig: int) -> None:\n \"\"\"Signal the pid's process group, falling back to a deepest-first tree\n walk when the group kill is unavailable.\"\"\"\n if not _killpg_quiet(pid, sig):\n for descendant in _descendants_deepest_first(pid):\n _kill_quiet(descendant, sig)\n _kill_quiet(pid, sig)\n\n\ndef _pre_reuse_descendant_pids(root_pid, children, predates_reuse, skip_pid=None):\n \"\"\"Direct children that predate a recycled leader pid, plus each of those\n children's full subtree.\n\n Toolhelp still lists the original tree under a dead pid as parent, mixed\n with the new process's own children. The start-time cutoff applies only to\n direct children. A pre-reuse child's later descendants stay original-tree\n work even if they started after the reuse.\n \"\"\"\n keep = set()\n queue = []\n for child in children.get(root_pid, []):\n if skip_pid is not None and child == skip_pid:\n continue\n if not predates_reuse(child):\n continue\n queue.append(child)\n while queue:\n pid = queue.pop(0)\n if skip_pid is not None and pid == skip_pid:\n continue\n if pid in keep:\n continue\n keep.add(pid)\n queue.extend(children.get(pid, []))\n return keep\n\n\ndef kill_tree(root_pid: int, grace: float, job_name=None, expected_identity=None) -> bool:\n \"\"\"TERM the pid's process group (workers are started as group leaders),\n falling back to a deepest-first tree walk; grace, then KILL survivors.\n\n `job_name` is Windows-only (the worker's job object, the pgid analog) and\n is ignored on POSIX, where the pgid is derived from the pid itself.\n `expected_identity` is Windows-only (GetProcessTimes identity recorded at\n start) and is ignored on POSIX.\"\"\"\n if IS_WINDOWS:\n return _win_kill_tree(root_pid, grace, job_name, expected_identity)\n # Do NOT early-return just because the leader pid is dead: killpg targets\n # the pgid, which persists while any group member lives even after the\n # leader exits, so a dead leader can still front a live group we must sweep.\n # Use _pid_running (zombie-aware), not _pid_alive: a just-exited leader is\n # briefly a \u003cdefunct> zombie for which kill -0 still succeeds, and counting\n # that as alive would misclassify the reap as timeout instead of\n # died-without-result (and make the dead-leader sweep test timing-dependent).\n leader_alive = _pid_running(root_pid)\n # Snapshot the descendant set BEFORE any KILL: once the group leader is\n # reaped its children reparent to init and drop out of the tree, so a set\n # enumerated after the kill would miss them and leak orphans.\n survivors = _descendants_deepest_first(root_pid)\n _signal_group_or_tree(root_pid, signal.SIGTERM)\n deadline = time.monotonic() + grace\n while time.monotonic() \u003c deadline:\n if leader_alive and not _pid_alive(root_pid):\n break\n time.sleep(0.1)\n _killpg_quiet(root_pid, signal.SIGKILL)\n for pid in survivors:\n _kill_quiet(pid, signal.SIGKILL)\n _kill_quiet(root_pid, signal.SIGKILL)\n return leader_alive\n\n\n# --- the supervisor (runs inside the detached session) -------------------------\n\ndef classify_exit(rc: int, result_path, conf: dict):\n result_size = None\n if result_path:\n try:\n st = os.lstat(result_path)\n if stat.S_ISREG(st.st_mode) and st.st_size > 0:\n result_size = st.st_size\n except OSError:\n pass\n if result_size is not None and result_size > conf[\"result_max\"]:\n return \"failed\", (\n f\"result exceeded byte cap ({result_size} > {conf['result_max']} bytes)\"\n )\n if rc == 0:\n if result_path is None or result_size is not None:\n return \"done\", \"worker exited 0\"\n return \"failed\", \"worker exited 0 without publishing a non-empty result\"\n if rc \u003c 0:\n if result_size is not None:\n return \"done\", f\"worker killed by signal {-rc} after publishing its result\"\n return \"died-without-result\", (\n f\"worker killed by signal {-rc} with no result evidence\"\n )\n return \"failed\", f\"worker exited {rc}\"\n\n\ndef classify_exit_with_pending_reap(rc: int, result_path, conf: dict, reap_pending: bool):\n \"\"\"Classify a worker that already exited, optionally under a pending reap.\n\n When reap is pending (Windows `.reap` or POSIX SIGTERM flag) and the worker\n was killed by the fallback path, classify_exit would record \"failed\" for a\n non-zero kill exit — prefer timeout. When the worker already completed\n successfully (done + result), keep that: a late reap must not rewrite a\n finished peer run.\n \"\"\"\n state, reason = classify_exit(rc, result_path, conf)\n if reap_pending and state != \"done\":\n return \"timeout\", \"reaped on request before completion\"\n return state, reason\n\n\ndef _reap_worker(proc, conf: dict, job_name=None) -> None:\n # Deliberately parallel to kill_tree but driven by proc.poll(): an unreaped\n # Popen child is a zombie that os.kill(pid, 0) still reports alive, so the\n # pid-based liveness check would burn the whole grace window.\n if proc.poll() is not None:\n return\n if IS_WINDOWS:\n _win_kill_tree(proc.pid, conf[\"grace\"], job_name)\n try:\n proc.wait(timeout=5)\n except Exception:\n pass\n return\n _signal_group_or_tree(proc.pid, signal.SIGTERM)\n deadline = time.monotonic() + conf[\"grace\"]\n while time.monotonic() \u003c deadline:\n if proc.poll() is not None:\n break\n time.sleep(0.1)\n if proc.poll() is None:\n _killpg_quiet(proc.pid, signal.SIGKILL)\n for pid in _descendants_deepest_first(proc.pid):\n _kill_quiet(pid, signal.SIGKILL)\n try:\n proc.wait(timeout=5)\n except Exception:\n pass\n\n\ndef _reap_requested(flag: dict, job_dir: str) -> bool:\n \"\"\"POSIX delivers the reap as SIGTERM (sets flag). Windows has no reliable\n directed-signal path to a detached, console-less supervisor, so cmd_reap\n drops a `.reap` marker in the job dir and the loop polls for it.\"\"\"\n if flag[\"reap\"]:\n return True\n if IS_WINDOWS and os.path.lexists(os.path.join(job_dir, \".reap\")):\n return True\n return False\n\n\ndef _interruptible_sleep(secs: float, flag: dict, job_dir: str) -> None:\n end = time.monotonic() + secs\n while time.monotonic() \u003c end:\n # On POSIX _reap_requested reduces to flag[\"reap\"] (the IS_WINDOWS\n # branch never fires), so this is the original signal-driven behavior.\n if _reap_requested(flag, job_dir):\n return\n time.sleep(min(0.1, max(0.01, end - time.monotonic())))\n\n\ndef _is_system32_wsl_bash(path: str) -> bool:\n \"\"\"True for Windows System32 WSL launchers, including Sysnative aliases.\"\"\"\n if not path:\n return False\n base = os.path.basename(path).lower()\n if base not in (\"bash\", \"bash.exe\", \"sh\", \"sh.exe\"):\n return False\n system_root = os.environ.get(\"SystemRoot\") or r\"C:\\Windows\"\n windows_root = os.path.abspath(system_root)\n blocked_parents = {\n os.path.normcase(os.path.join(windows_root, name))\n for name in (\"System32\", \"Sysnative\")\n }\n parent = os.path.normcase(os.path.dirname(os.path.abspath(path)))\n return parent in blocked_parents\n\n\ndef _git_bash_well_known_paths():\n \"\"\"Standard Git for Windows bash.exe locations.\"\"\"\n pf64 = os.environ.get(\"ProgramW6432\") or \"\"\n pf = os.environ.get(\"ProgramFiles\") or r\"C:\\Program Files\"\n pf86 = os.environ.get(\"ProgramFiles(x86)\") or r\"C:\\Program Files (x86)\"\n local = os.environ.get(\"LOCALAPPDATA\") or \"\"\n roots = []\n seen = set()\n for root in (pf64, pf, pf86):\n if not root:\n continue\n key = os.path.normcase(os.path.abspath(root))\n if key in seen:\n continue\n seen.add(key)\n roots.append(root)\n paths = []\n for root in roots:\n paths.extend([\n os.path.join(root, \"Git\", \"bin\", \"bash.exe\"),\n os.path.join(root, \"Git\", \"usr\", \"bin\", \"bash.exe\"),\n ])\n if local:\n paths.extend([\n os.path.join(local, \"Programs\", \"Git\", \"bin\", \"bash.exe\"),\n os.path.join(local, \"Programs\", \"Git\", \"usr\", \"bin\", \"bash.exe\"),\n ])\n return paths\n\n\ndef _windows_path_shell_candidates():\n \"\"\"Every bash/sh on PATH in PATH order (not only shutil.which's first hit).\"\"\"\n path_env = os.environ.get(\"PATH\") or \"\"\n names = (\"bash.exe\", \"bash\", \"sh.exe\", \"sh\")\n found = []\n seen = set()\n for directory in path_env.split(os.pathsep):\n if not directory:\n continue\n for name in names:\n candidate = os.path.join(directory, name)\n try:\n if not os.path.isfile(candidate):\n continue\n except OSError:\n continue\n key = os.path.normcase(os.path.abspath(candidate))\n if key in seen:\n continue\n seen.add(key)\n found.append(candidate)\n return found\n\n\ndef _env_assignment_token(token: str, allow_option_like: bool = False) -> bool:\n \"\"\"True for env(1) NAME=value operands (not options or the command).\"\"\"\n if (\n not token\n or (token.startswith(\"-\") and not allow_option_like)\n or \"=\" not in token\n ):\n return False\n return True\n\n\ndef _env_option_advance(tok: str) -> int:\n \"\"\"How many argv slots an env(1) option occupies (incl. the option itself).\n\n GNU env options that take a separate operand: -u/--unset, -C/--chdir.\n Attached `--name=value` forms are a single slot.\n Short options may be clustered. No-operand flags (-i/-v and their exact\n long aliases and signal-handling options) advance one slot; -u/-C consume\n the rest of the token as an\n attached operand or the next argv slot. Unsupported options fail closed\n before worker detach.\n (#1292 Codex P2)\n \"\"\"\n if tok in (\"-u\", \"--unset\", \"-C\", \"--chdir\"):\n return 2\n if tok.startswith((\"--unset=\", \"--chdir=\")):\n return 1\n if tok in (\"--ignore-environment\", \"--debug\"):\n return 1\n if tok == \"--list-signal-handling\" or tok in (\n \"--block-signal\",\n \"--default-signal\",\n \"--ignore-signal\",\n ) or tok.startswith((\n \"--block-signal=\",\n \"--default-signal=\",\n \"--ignore-signal=\",\n )):\n return 1\n if tok == \"--null\":\n raise RunnerError(\n \"env -0/--null cannot be used with a command by native Windows \"\n \"peer workers; remove the null-output option\"\n )\n if not tok.startswith(\"-\"):\n return 1\n if tok.startswith(\"--\"):\n raise RunnerError(\n f\"unsupported env long option {tok!r} for native Windows peer \"\n \"workers; use an exact supported option or pass -- before \"\n \"option-like assignments\"\n )\n\n cluster = tok[1:]\n for index, option in enumerate(cluster):\n if option in \"iv\":\n continue\n if option == \"0\":\n raise RunnerError(\n \"env -0/--null cannot be used with a command by native \"\n \"Windows peer workers; remove the null-output option\"\n )\n if option == \"S\":\n raise RunnerError(\n \"env -S/--split-string is unsupported for native Windows \"\n \"peer workers; pass env assignments and the command as \"\n \"separate arguments\"\n )\n if option in \"uC\":\n return 1 if index + 1 \u003c len(cluster) else 2\n raise RunnerError(\n f\"unsupported env short-option cluster {tok!r} for native \"\n \"Windows peer workers; pass env options separately\"\n )\n return 1\n\n\ndef _env_bash_index(argv):\n \"\"\"Locate the env(1)-launched bash/sh command, for #1268/#1292 rewriting.\n\n Matches the production cross-model shape `env VAR=… bash script.sh …`\n (#1268). Operand-taking options (-u/-C and long forms) consume their\n arguments before the command token is sought (#1292). Split-string forms\n fail closed because Python shlex does not match Git env.exe semantics.\n\n Returns (argv_index, None), or (-1, None) when no bash/sh command is\n present.\n \"\"\"\n if not argv:\n return -1, None\n if os.path.basename(argv[0]).lower() not in (\"env\", \"env.exe\"):\n return -1, None\n i = 1\n options_done = False\n while i \u003c len(argv):\n tok = argv[i]\n if tok in (\"-\", \"--\") and not options_done:\n options_done = True\n i += 1\n continue\n if _env_assignment_token(tok, allow_option_like=options_done):\n options_done = True\n i += 1\n continue\n if not options_done and (\n tok in (\"-S\", \"--split-string\") or tok.startswith(\n (\"-S\", \"--split-string=\")\n )\n ):\n raise RunnerError(\n \"env -S/--split-string is unsupported for native Windows \"\n \"peer workers; pass env assignments and the command as \"\n \"separate arguments\"\n )\n if not options_done and tok.startswith(\"-\"):\n span = _env_option_advance(tok)\n if span > 1 and i + 1 >= len(argv):\n return -1, None\n i += span\n continue\n base = os.path.basename(tok).lower()\n if base in (\"bash\", \"bash.exe\", \"sh\", \"sh.exe\"):\n return i, None\n return -1, None\n return -1, None\n\n\ndef _windows_path_is_absolute(path: str) -> bool:\n \"\"\"True for Windows absolute paths (drive letter or path separator).\"\"\"\n return os.sep in path or (len(path) >= 2 and path[1] == \":\")\n\n\ndef _prefer_windows_posix_shell(token: str) -> str:\n \"\"\"Absolute non-WSL bash/sh kept; bare names and System32 go through resolve.\n\n Explicit absolute paths (portable Git, custom installs) must not be\n substituted by the preferred resolver (#1292 Codex P2). Bare `bash`/`sh`\n and System32 WSL launchers still use `_resolve_windows_posix_shell()`.\n \"\"\"\n if _windows_path_is_absolute(token):\n path = os.path.abspath(token)\n if not os.path.isfile(path):\n raise RunnerError(\n f\"peer worker shell does not exist or is not a regular file: {token}\"\n )\n if _is_system32_wsl_bash(path):\n return _resolve_windows_posix_shell()\n return path\n return _resolve_windows_posix_shell()\n\n\ndef _rewrite_windows_env_bash_argv(argv):\n \"\"\"Rewrite bare bash/sh inside an env-prefixed argv.\n\n Returns (argv, resolved_shell_or_None). Raises RunnerError when a bash/sh\n token is present but no usable non-WSL shell can be resolved. Absolute\n non-WSL bash tokens are kept unchanged (#1292 P2). Split-string options\n are rejected before detach because their parser semantics are not safely\n reproduced here (#1292).\n \"\"\"\n idx, split_prefix = _env_bash_index(argv)\n if idx \u003c 0:\n return list(argv), None\n out = list(argv)\n assert split_prefix is None\n shell = _prefer_windows_posix_shell(out[idx])\n if os.path.normcase(os.path.abspath(out[idx])) != os.path.normcase(shell):\n out[idx] = shell\n return out, shell\n\n\ndef _resolve_windows_posix_shell() -> str:\n \"\"\"Absolute path to a non-WSL POSIX shell for native Windows peer workers.\n\n Order: CE_PEER_BASH, CLAUDE_CODE_GIT_BASH_PATH, well-known Git Bash\n installs, then every PATH bash/sh excluding System32 WSL. Fail closed when\n nothing usable remains — never select System32\\\\bash.exe (#1268).\n \"\"\"\n candidates = []\n for key in (\"CE_PEER_BASH\", \"CLAUDE_CODE_GIT_BASH_PATH\"):\n val = (os.environ.get(key) or \"\").strip()\n if val:\n candidates.append(val)\n candidates.extend(_git_bash_well_known_paths())\n candidates.extend(_windows_path_shell_candidates())\n\n seen = set()\n for raw in candidates:\n path = os.path.abspath(raw)\n key = os.path.normcase(path)\n if key in seen:\n continue\n seen.add(key)\n if not os.path.isfile(path):\n continue\n if _is_system32_wsl_bash(path):\n continue\n return path\n\n raise RunnerError(\n \"no usable Git Bash (or other non-WSL POSIX shell) for native Windows \"\n \"peer workers; install Git for Windows or set CE_PEER_BASH / \"\n \"CLAUDE_CODE_GIT_BASH_PATH to an absolute bash.exe path \"\n \"(System32\\\\bash.exe / WSL is not used)\"\n )\n\n\ndef _popen_argv(argv):\n \"\"\"Argv for subprocess.Popen.\n\n On Windows, CreateProcess does not honor shebang, so a bare *.sh / *.bash\n worker must be launched through bash/sh. Prefer Git Bash over System32\n WSL bash (#1268). Bare `bash`/`sh` prefixes (review skills) and bare\n `bash`/`sh` tokens after `env VAR=…` (cross-model) are rewritten to that\n absolute path. Explicit absolute non-WSL bash/sh paths are kept (#1292 P2).\n meta.json still records the caller argv for authorize-dispatch contracts\n that forbid a shell prefix on ce-work.\n \"\"\"\n if not IS_WINDOWS or not argv:\n return list(argv)\n head = argv[0]\n base = os.path.basename(head).lower()\n if base in (\"env\", \"env.exe\"):\n rewritten, _shell = _rewrite_windows_env_bash_argv(argv)\n return rewritten\n if base in (\"bash\", \"bash.exe\", \"sh\", \"sh.exe\"):\n shell = _prefer_windows_posix_shell(head)\n if os.path.normcase(os.path.abspath(head)) == os.path.normcase(shell):\n return list(argv)\n return [shell] + list(argv[1:])\n lower = head.lower()\n if not (lower.endswith(\".sh\") or lower.endswith(\".bash\")):\n return list(argv)\n shell = _resolve_windows_posix_shell()\n return [shell, head] + list(argv[1:])\n\n\ndef supervise(job_dir: str, argv, result_path, conf: dict, ack_fd: int) -> None:\n \"\"\"The watchdog around the worker child. Owns liveness (out.log growth),\n the idle/hard windows, byte caps, reap-on-request, and the single terminal\n classification.\"\"\"\n flag = {\"reap\": False}\n\n def on_term(signum, frame):\n flag[\"reap\"] = True\n\n if not IS_WINDOWS:\n signal.signal(signal.SIGTERM, on_term)\n signal.signal(signal.SIGHUP, signal.SIG_IGN)\n # On Windows there is no SIGHUP and no reliable directed SIGTERM to a\n # detached supervisor; reap arrives via the `.reap` marker polled below.\n\n acked = False\n\n def ack():\n nonlocal acked\n if acked:\n return\n acked = True\n # Windows detach has no ack pipe (ack_fd is None): the pid file written\n # just above IS the ack, and the parent polls for it.\n if ack_fd is None:\n return\n try:\n os.write(ack_fd, b\"ok\")\n os.close(ack_fd)\n except OSError:\n pass\n\n log_fd = None\n job_name = None\n job_handle = None\n try:\n log_fd = os.open(\n os.path.join(job_dir, \"out.log\"),\n os.O_WRONLY | os.O_APPEND | O_NOFOLLOW | O_BINARY)\n if IS_WINDOWS:\n # Created BEFORE the worker so the tree can never start outside it.\n # The handle is held for the supervisor's lifetime; the job is what\n # makes teardown reach descendants of an exited leader.\n job_name = _win_job_name(job_dir)\n job_handle = _win_create_job(job_name)\n devnull = os.open(os.devnull, os.O_RDONLY)\n try:\n # Export the interpreter running this supervisor so Windows workers\n # (and any adapter that honors it) do not re-resolve to the Store\n # python3 stub — see resolve-python convention / #1247.\n worker_env = {\n **os.environ,\n \"CE_PEER_JOB_ID\": os.path.basename(job_dir),\n \"CE_PEER_PYTHON\": sys.executable,\n }\n popen_kwargs = dict(\n stdin=devnull,\n stdout=log_fd,\n stderr=log_fd,\n env=worker_env,\n close_fds=True,\n )\n if IS_WINDOWS:\n # New process group so the worker's own tree is isolated; reap =\n # job terminate + Toolhelp walk (there is no killpg on Windows).\n # CREATE_NO_WINDOW because the supervisor is console-less, so\n # without it Windows allocates a NEW console per worker and the\n # user sees a window flash for every job. (It is mutually\n # exclusive with DETACHED_PROCESS, which is why the supervisor\n # itself uses DETACHED_PROCESS and only the worker uses this.)\n # CREATE_SUSPENDED: assign to the Job Object before any worker\n # code runs, so early child spawns inherit membership.\n popen_kwargs[\"creationflags\"] = (\n subprocess.CREATE_NEW_PROCESS_GROUP\n | _WIN_NO_WINDOW\n | _CREATE_SUSPENDED)\n else:\n popen_kwargs[\"start_new_session\"] = True # worker leads its own group\n # Wrap bare *.sh on Windows at spawn time only — meta still has the\n # caller argv (see _popen_argv).\n proc = subprocess.Popen(_popen_argv(argv), **popen_kwargs)\n finally:\n os.close(devnull)\n pid_doc = {\n \"supervisor_pid\": os.getpid(),\n \"worker_pid\": proc.pid,\n }\n if IS_WINDOWS:\n sup_ident = _win_process_identity(os.getpid())\n worker_ident = _win_process_identity(proc.pid)\n if sup_ident:\n pid_doc[\"supervisor_identity\"] = sup_ident\n if worker_ident:\n pid_doc[\"worker_identity\"] = worker_ident\n # Assign while still suspended, then resume. Record the job only\n # once the worker is actually a member, so a later reap never trusts\n # a name that owns nothing. If assignment fails (job creation denied,\n # or a nested-job restriction), leave it unset and let teardown fall\n # back to the Toolhelp walk.\n if job_handle is not None and _win_assign_to_job(job_handle, proc.pid):\n pid_doc[\"job_name\"] = job_name\n else:\n job_name = None\n if not _win_resume_process(proc.pid):\n raise RunnerError(\n f\"could not resume suspended Windows worker pid {proc.pid}\"\n )\n else:\n # pgid drives POSIX group kills; Windows reaps by job object.\n pid_doc[\"supervisor_pgid\"] = os.getpgid(0)\n # The pid file lands before the parent is acked, so a returned `start`\n # guarantees the detach marker exists (status never mis-reads a fresh\n # job as never-started).\n write_atomic(os.path.join(job_dir, \"pid\"), (json.dumps(pid_doc) + \"\\n\").encode())\n except Exception as exc:\n write_terminal(job_dir, \"failed\", f\"could not launch worker: {exc}\")\n ack()\n return\n ack()\n\n start_t = time.monotonic()\n last_growth = start_t\n last_size = 0\n while True:\n rc = proc.poll()\n if rc is not None:\n state, reason = classify_exit_with_pending_reap(\n rc, result_path, conf, _reap_requested(flag, job_dir),\n )\n break\n if _reap_requested(flag, job_dir):\n # Classification is fixed BEFORE the kill: even if the worker\n # publishes and exits 0 during the grace window, the supervisor's\n # record wins.\n _reap_worker(proc, conf, job_name)\n state, reason = \"timeout\", \"reaped on request before completion\"\n break\n try:\n size = os.fstat(log_fd).st_size\n except OSError:\n size = last_size\n now = time.monotonic()\n if size > last_size:\n last_size, last_growth = size, now\n if size > conf[\"log_max\"]:\n _reap_worker(proc, conf, job_name)\n state, reason = \"failed\", (\n f\"out.log exceeded byte cap ({size} > {conf['log_max']} bytes)\"\n )\n break\n if conf[\"idle\"] is not None and now - last_growth >= conf[\"idle\"]:\n _reap_worker(proc, conf, job_name)\n state, reason = \"timeout\", f\"no output for {conf['idle']:g}s (idle window)\"\n break\n if now - start_t >= conf[\"hard\"]:\n _reap_worker(proc, conf, job_name)\n state, reason = \"timeout\", f\"hard cap {conf['hard']:g}s exceeded\"\n break\n _interruptible_sleep(conf[\"poll\"], flag, job_dir)\n # An externally killed worker can leave group members behind (its shell's\n # children); sweep the group before publishing so no orphan outlives the\n # terminal record. A pgid cannot be recycled while members remain.\n if IS_WINDOWS:\n # Job-object sweep: unlike taskkill this still reaches descendants when\n # the worker leader has already exited, which is the orphan case the\n # POSIX killpg pair below covers.\n _win_kill_tree(proc.pid, min(conf[\"grace\"], 1.0), job_name)\n else:\n _killpg_quiet(proc.pid, signal.SIGTERM)\n _killpg_quiet(proc.pid, signal.SIGKILL)\n write_terminal(job_dir, state, reason)\n\n\ndef detach_supervisor(job_dir: str, argv, result_path, conf: dict) -> bool:\n \"\"\"setsid double-fork. The grandchild (new session, stdio on /dev/null,\n reparented to init) runs the supervisor; the parent returns once the\n supervisor acks that the pid file exists.\"\"\"\n if IS_WINDOWS:\n return detach_supervisor_windows(job_dir, argv, result_path, conf)\n sys.stdout.flush()\n sys.stderr.flush()\n read_fd, write_fd = os.pipe()\n pid1 = os.fork()\n if pid1 == 0:\n os.close(read_fd)\n os.setsid()\n if os.fork() > 0:\n os._exit(0)\n rc = 0\n try:\n devnull = os.open(os.devnull, os.O_RDWR)\n os.dup2(devnull, 0)\n os.dup2(devnull, 1)\n os.dup2(devnull, 2)\n if devnull > 2:\n os.close(devnull)\n supervise(job_dir, argv, result_path, conf, write_fd)\n except BaseException:\n rc = 1\n try:\n write_terminal(\n job_dir, \"failed\", \"supervisor crashed before classification\",\n overwrite=False,\n )\n except BaseException:\n pass\n os._exit(rc)\n os.close(write_fd)\n os.waitpid(pid1, 0)\n ack = b\"\"\n try:\n while len(ack) \u003c 2:\n chunk = os.read(read_fd, 2 - len(ack))\n if not chunk:\n break\n ack += chunk\n finally:\n os.close(read_fd)\n return ack == b\"ok\"\n\n\ndef detach_supervisor_windows(job_dir: str, argv, result_path, conf: dict) -> bool:\n \"\"\"Windows detach: there is no fork/setsid, so re-invoke this script as a\n fresh DETACHED_PROCESS running the internal `__supervise` entrypoint. The\n spawn spec travels through an owner-private file in the job dir; the parent\n returns once the supervisor has left its ack marker (the pid file, or a\n terminal status if the worker could not launch). CREATE_BREAKAWAY_FROM_JOB\n is the analog of setsid's reparent-to-init: it lets the supervisor outlive a\n launching harness that runs inside a kill-on-close Job Object, falling back\n when the job forbids breakaway.\"\"\"\n spec = {\"argv\": list(argv), \"result_path\": result_path, \"conf\": conf}\n create_exclusive(\n os.path.join(job_dir, \".spawn.json\"),\n (json.dumps(spec) + \"\\n\").encode(),\n )\n cmd = [sys.executable, os.path.abspath(__file__), \"__supervise\", job_dir]\n base_flags = subprocess.DETACHED_PROCESS | subprocess.CREATE_NEW_PROCESS_GROUP\n devnull = os.open(os.devnull, os.O_RDWR)\n proc = None\n try:\n for flags in (base_flags | subprocess.CREATE_BREAKAWAY_FROM_JOB, base_flags):\n try:\n proc = subprocess.Popen(\n cmd,\n stdin=devnull, stdout=devnull, stderr=devnull,\n close_fds=True, creationflags=flags,\n )\n break\n except OSError:\n proc = None\n if proc is None:\n return False\n finally:\n os.close(devnull)\n\n pid_path = os.path.join(job_dir, \"pid\")\n status_path = os.path.join(job_dir, \"status\")\n deadline = time.monotonic() + 15.0\n while time.monotonic() \u003c deadline:\n if os.path.lexists(pid_path) or os.path.lexists(status_path):\n return True\n if proc.poll() is not None:\n # Supervisor process exited without leaving a marker: detach failed.\n return os.path.lexists(pid_path) or os.path.lexists(status_path)\n time.sleep(0.05)\n # Deadline with the supervisor still running. Do not abandon it: reporting\n # a detach failure while leaving a live, unreachable supervisor/worker pair\n # behind is exactly the orphan this runner exists to prevent. The snapshot\n # walk reaches the worker as the supervisor's child.\n try:\n if proc.poll() is None:\n _win_kill_tree(proc.pid, 0.0)\n except Exception:\n pass\n return False\n\n\ndef _win_supervise_from_spec(job_dir: str) -> int:\n \"\"\"Internal `__supervise` entrypoint: the detached Windows supervisor. Drops\n its console-less std handles onto NUL, reads the owner-checked spawn spec,\n and runs the shared supervisor loop with a file-based (not fd) ack.\"\"\"\n rc = 0\n # Redundant with detach_supervisor_windows, which already binds this\n # process's stdio to NUL via Popen -- kept deliberately so the entrypoint is\n # self-contained: a supervisor is long-lived, and any future/manual\n # invocation that inherited a real pipe could block forever once it filled.\n try:\n devnull = os.open(os.devnull, os.O_RDWR)\n for fd in (0, 1, 2):\n try:\n os.dup2(devnull, fd)\n except OSError:\n pass\n if devnull > 2:\n os.close(devnull)\n except OSError:\n pass\n try:\n _check_owned_dir(job_dir)\n spec = json.loads(read_owned(os.path.join(job_dir, \".spawn.json\"), META_READ_CAP))\n argv = spec[\"argv\"]\n result_path = spec.get(\"result_path\")\n conf = spec[\"conf\"]\n try:\n os.unlink(os.path.join(job_dir, \".spawn.json\"))\n except OSError:\n pass\n supervise(job_dir, argv, result_path, conf, None)\n except BaseException:\n rc = 1\n try:\n write_terminal(\n job_dir, \"failed\", \"supervisor crashed before classification\",\n overwrite=False,\n )\n except BaseException:\n pass\n return rc\n\n\n# --- subcommands ---------------------------------------------------------------\n\ndef sweep_stale_runs(skill_dir: str, keep: str) -> None:\n \"\"\"Best-effort retention: remove sibling run roots older than 24h.\n Owner-checked via lstat; never raises, never touches the current run.\"\"\"\n try:\n entries = list(os.scandir(skill_dir))\n except OSError:\n return\n now = time.time()\n euid = _euid()\n keep_abs = os.path.abspath(keep)\n for entry in entries:\n if os.path.abspath(entry.path) == keep_abs:\n continue\n try:\n st = entry.stat(follow_symlinks=False)\n except OSError:\n continue\n if not stat.S_ISDIR(st.st_mode):\n continue\n if IS_WINDOWS:\n try:\n if not _win_owns_path(entry.path):\n continue\n except OSError:\n continue\n elif euid is not None and st.st_uid != euid:\n continue\n if now - st.st_mtime \u003c= SWEEP_AGE_SECS:\n continue\n shutil.rmtree(entry.path, ignore_errors=True)\n\n\ndef _require_detach_support() -> None:\n \"\"\"Detached peer jobs need a supported detach path: os.fork/os.setsid on\n POSIX, or the native Windows DETACHED_PROCESS path (#1243). Checked first,\n before jobs_root_base()/geteuid, so an unsupported host fails with this clear\n message instead of jobs_root_base()'s unrelated \"effective user ID is\n unavailable\" error or an AttributeError mid-detach. Native Windows is now\n supported; only a non-win32 Python missing fork/setsid (some embedded\n builds) is rejected here.\"\"\"\n if IS_WINDOWS:\n return\n if not hasattr(os, \"fork\") or not hasattr(os, \"setsid\"):\n raise RunnerError(\n \"detached peer jobs require os.fork/os.setsid on this platform; no \"\n \"job was started. Run under a POSIX Python, or on native Windows use \"\n \"a Windows Python 3 build (see \"\n \"EveryInc/compound-engineering-plugin#1243).\"\n )\n\n\ndef cmd_start(args, worker_argv) -> int:\n _require_detach_support()\n for flag, value in ((\"--skill\", args.skill), (\"--run-id\", args.run_id)):\n if not _is_safe_token(value):\n raise RunnerError(f\"{flag} must match [A-Za-z0-9._-]+ and not be all dots (got {value!r})\")\n if args.label is not None and not _is_safe_token(args.label):\n raise RunnerError(f\"--label must match [A-Za-z0-9._-]+ and not be all dots (got {args.label!r})\")\n if not worker_argv:\n raise RunnerError(\"no worker argv; place it after `--`\")\n\n base = jobs_root_base()\n skill_dir = skill_runs_root(args.skill)\n run_dir = os.path.join(skill_dir, args.run_id)\n jobs_root = os.path.join(run_dir, \"jobs\")\n ensure_owned_dirs(skill_dir if skill_dir != os.path.join(base, args.skill) else base, jobs_root)\n if not args.no_sweep:\n sweep_stale_runs(skill_dir, keep=run_dir)\n\n job_id, job_dir = claim_job_dir(jobs_root)\n result_path = os.path.abspath(args.result_path) if args.result_path else None\n\n argv0 = worker_argv[0]\n problem = None\n windows_posix_shell = None\n base0 = os.path.basename(argv0).lower()\n if IS_WINDOWS and base0 in (\"bash\", \"bash.exe\", \"sh\", \"sh.exe\"):\n # Prefer Git Bash over PATH/System32 WSL before meta + detach (#1268).\n # Keep an explicit absolute non-WSL bash (portable Git) (#1292 P2).\n try:\n resolved = _prefer_windows_posix_shell(argv0)\n windows_posix_shell = resolved\n except RunnerError as exc:\n problem = str(exc)\n resolved = argv0\n elif IS_WINDOWS and base0 in (\"env\", \"env.exe\"):\n # Production cross-model: env VAR=… bash script.sh — rewrite bash\n # before detach so env cannot PATH-resolve System32 WSL (#1268).\n if os.sep in argv0 or (len(argv0) >= 2 and argv0[1] == \":\"):\n resolved = os.path.abspath(argv0)\n if not os.path.isfile(resolved):\n problem = \"does not exist or is not a regular file\"\n else:\n resolved = shutil.which(argv0)\n if resolved is None:\n problem = \"was not found on PATH\"\n resolved = argv0\n try:\n rewritten, shell = _rewrite_windows_env_bash_argv(list(worker_argv))\n if shell is not None:\n windows_posix_shell = shell\n worker_argv = rewritten\n except RunnerError as exc:\n problem = str(exc) if problem is None else f\"{problem}; {exc}\"\n elif os.sep in argv0 or (IS_WINDOWS and len(argv0) >= 2 and argv0[1] == \":\"):\n resolved = os.path.abspath(argv0)\n if not os.path.isfile(resolved):\n problem = \"does not exist or is not a regular file\"\n elif IS_WINDOWS and resolved.lower().endswith((\".sh\", \".bash\")):\n # CreateProcess cannot run shebang scripts; _popen_argv wraps with\n # Git Bash. Require that shell now so start fails closed, not after\n # detach. Skip the X_OK check — Windows often marks .sh non-exec.\n try:\n windows_posix_shell = _resolve_windows_posix_shell()\n except RunnerError as exc:\n problem = str(exc)\n elif not os.access(resolved, os.X_OK):\n problem = \"is not executable\"\n else:\n resolved = shutil.which(argv0)\n if resolved is None:\n problem = \"was not found on PATH\"\n resolved = argv0\n elif IS_WINDOWS and resolved.lower().endswith((\".sh\", \".bash\")):\n try:\n windows_posix_shell = _resolve_windows_posix_shell()\n except RunnerError as exc:\n problem = str(exc)\n elif IS_WINDOWS and os.path.basename(resolved).lower() in (\n \"bash\", \"bash.exe\", \"sh\", \"sh.exe\",\n ):\n # which() may have returned System32 WSL — rewrite now.\n try:\n resolved = _resolve_windows_posix_shell()\n windows_posix_shell = resolved\n except RunnerError as exc:\n problem = str(exc)\n argv = [resolved] + list(worker_argv[1:])\n\n conf = cfg(args.skill)\n meta = {\n \"job_id\": job_id,\n \"skill\": args.skill,\n \"run_id\": args.run_id,\n \"label\": args.label,\n \"input_digest\": args.input_digest,\n \"started_at\": time.strftime(STARTED_AT_FORMAT, time.gmtime()),\n \"worker_argv\": argv,\n \"result_path\": result_path,\n \"sweep_enabled\": not args.no_sweep,\n \"supervision\": conf,\n }\n if windows_posix_shell:\n meta[\"windows_posix_shell\"] = windows_posix_shell\n try:\n create_exclusive(\n os.path.join(job_dir, \"meta.json\"),\n (json.dumps(meta, indent=2) + \"\\n\").encode(),\n )\n except OSError as exc:\n raise RunnerError(f\"cannot write job metadata for {job_id}: {exc}\")\n\n if problem is not None:\n raise RunnerError(\n f\"preflight failed for job {job_id}: worker {argv0!r} {problem}; \"\n f\"nothing was detached (job left never-started at {job_dir})\"\n )\n try:\n create_exclusive(os.path.join(job_dir, \"out.log\"))\n except OSError as exc:\n raise RunnerError(\n f\"preflight failed for job {job_id}: job dir not writable ({exc}); \"\n \"nothing was detached\"\n )\n\n if not detach_supervisor(job_dir, argv, result_path, conf):\n raise RunnerError(\n f\"detach failed for job {job_id}: supervisor did not acknowledge; \"\n f\"inspect {job_dir}\"\n )\n print(job_id)\n return 0\n\n\ndef _emit_states(rows, as_json: bool) -> None:\n if as_json:\n print(json.dumps(\n [{\"ref\": r, \"job_dir\": d, \"state\": s} for r, d, s in rows]\n ))\n elif len(rows) == 1:\n print(rows[0][2])\n else:\n for ref, _, state in rows:\n print(f\"{ref}\\t{state}\")\n\n\ndef cmd_status(args) -> int:\n rows = []\n for ref in args.jobs:\n job_dir = resolve_job_dir(ref, args.skill)\n rows.append((ref, job_dir, job_state(job_dir)))\n _emit_states(rows, args.json)\n return 0\n\n\ndef _finite_secs(raw: str) -> float:\n try:\n value = float(raw)\n except ValueError:\n raise argparse.ArgumentTypeError(f\"not a number of seconds: {raw!r}\")\n if not math.isfinite(value) or value \u003c 0:\n raise argparse.ArgumentTypeError(f\"must be a finite, nonnegative number of seconds: {raw!r}\")\n return value\n\n\ndef _job_meta(job_dir: str) -> dict:\n try:\n meta = json.loads(read_owned(os.path.join(job_dir, \"meta.json\"), META_READ_CAP))\n except (Unreadable, OSError, ValueError):\n return {}\n return meta if isinstance(meta, dict) else {}\n\n\ndef _started_epoch(job_dir: str, meta: dict):\n try:\n return calendar.timegm(time.strptime(meta[\"started_at\"], STARTED_AT_FORMAT))\n except (KeyError, TypeError, ValueError):\n pass\n try:\n return os.lstat(os.path.join(job_dir, \"meta.json\")).st_mtime\n except OSError:\n return None\n\n\ndef _summary_reason(job_dir: str, state: str) -> str:\n if state == \"unreadable\":\n # Never read `reason` from a directory that failed its ownership check.\n return \"state unreadable (ownership or corruption); no result\"\n if state == \"never-started\":\n return job_reason(job_dir) or \"never started; no result\"\n if state in TERMINAL_STATES and state != \"done\":\n return job_reason(job_dir)\n return \"\"\n\n\ndef _emit_deadline_summary(rows, metas, deadline_at, as_json: bool) -> None:\n now = time.time()\n jobs = []\n for ref, d, state in rows:\n meta = metas[ref]\n started = _started_epoch(d, meta)\n jobs.append({\n \"ref\": ref,\n \"job_dir\": d,\n \"label\": meta.get(\"label\") or \"-\",\n \"state\": state,\n \"elapsed_secs\": int(now - started) if started is not None else None,\n \"reason\": _summary_reason(d, state),\n })\n running = [j[\"ref\"] for j in jobs if j[\"state\"] == \"running\"]\n secs_left = max(0, math.ceil(deadline_at - now))\n deadline_passed = bool(running) and now >= deadline_at\n if as_json:\n print(json.dumps({\n \"jobs\": jobs,\n \"all_settled\": not running,\n \"deadline_passed\": deadline_passed,\n \"secs_left\": secs_left,\n }))\n return\n for j in jobs:\n elapsed = \"?\" if j[\"elapsed_secs\"] is None else f\"{j['elapsed_secs']}s\"\n print(f\"{j['label']}\\t{j['ref']}\\t{j['state']}\\t{elapsed}\" + (f\"\\t{j['reason']}\" if j[\"reason\"] else \"\"))\n if not running:\n print(\"all settled\")\n elif deadline_passed:\n print(\n f\"deadline passed: reap {' '.join(running)}, then wait once more; \"\n \"a job still running after that counts as timed out\"\n )\n else:\n print(f\"waiting: {len(running)} of {len(jobs)} still running; {secs_left}s left before the deadline\")\n\n\ndef cmd_wait(args) -> int:\n dirs = [(ref, resolve_job_dir(ref, args.skill)) for ref in args.jobs]\n deadline = time.monotonic() + max(0.0, args.max_secs)\n metas = {}\n deadline_at = None\n if args.deadline_secs is not None:\n metas = {ref: _job_meta(d) for ref, d in dirs}\n starts = [s for s in (_started_epoch(d, metas[ref]) for ref, d in dirs) if s is not None]\n # No start time anywhere means no way to measure the window: let it pass\n # now rather than restart it on every call.\n window = args.deadline_secs + args.deadline_grace_secs\n last_start = max(starts) if starts else time.time() - window\n deadline_at = last_start + window\n until_deadline = deadline_at - time.time()\n if until_deadline > 0:\n deadline = min(deadline, time.monotonic() + until_deadline)\n else:\n # Past the deadline, poll briefly so jobs just reaped can record\n # their terminal state before the summary prints.\n deadline = min(deadline, time.monotonic() + REAP_SETTLE_SECS)\n rows = [(ref, d, \"running\") for ref, d in dirs]\n while True:\n # Settled states are final; only still-running jobs get re-read.\n rows = [\n (ref, d, state if state != \"running\" else job_state(d))\n for ref, d, state in rows\n ]\n if all(state != \"running\" for _, _, state in rows):\n break\n remaining = deadline - time.monotonic()\n if remaining \u003c= 0:\n break\n time.sleep(min(1.0, remaining))\n if args.deadline_secs is not None:\n _emit_deadline_summary(rows, metas, deadline_at, args.json)\n else:\n _emit_states(rows, args.json)\n return 0\n\n\ndef _emit_bytes(data: bytes) -> None:\n buffer = getattr(sys.stdout, \"buffer\", None)\n if buffer is not None:\n buffer.write(data)\n buffer.flush()\n else:\n sys.stdout.write(data.decode(\"utf-8\", \"replace\"))\n\n\ndef _report_absent_artifact(target: str, args) -> int:\n \"\"\"An absent --path artifact is an outcome, not a read error: a peer that\n skipped its gate exits 0 and writes nothing, so the file is legitimately\n missing on the most common fold-in path. Name that outcome, and when the\n caller also passed the job id, name the job's state -- otherwise \"still\n running\" and \"ran, produced nothing\" arrive as one errno the caller cannot\n act on. Each outcome keeps the exit code the job-result contract already\n assigns it, so a trust or lookup failure never reads as the routine skip:\n 2 running, 4 ownership, 1 unknown job, 3 settled with no artifact.\"\"\"\n sys.stderr.write(f\"peer-job-runner: no artifact at {target}\\n\")\n if not args.job:\n return 3\n try:\n job_dir = resolve_job_dir(args.job, args.skill)\n except RunnerError as exc:\n sys.stderr.write(f\"peer-job-runner: {exc}\\n\")\n return 1\n state = job_state(job_dir)\n if state == \"unreadable\":\n # Do not read `reason` here: job_dir already failed its owner check, and\n # O_NOFOLLOW guards only the final component, so a swapped directory\n # could redirect that read.\n sys.stderr.write(\n f\"peer-job-runner: job state unreadable (ownership or corruption): {job_dir}\\n\"\n )\n return 4\n if state == \"running\":\n sys.stderr.write(f\"peer-job-runner: job {args.job} is still running\\n\")\n return 2\n reason = job_reason(job_dir)\n sys.stderr.write(\n f\"peer-job-runner: job {args.job}: {state}\" + (f\" ({reason})\" if reason else \"\") + \"\\n\"\n )\n return 3\n\n\ndef cmd_result(args) -> int:\n if not getattr(args, \"path\", None) and not args.job:\n sys.stderr.write(\"peer-job-runner: result needs a job id or --path FILE\\n\")\n return 2\n if getattr(args, \"path\", None):\n # Verified read of an arbitrary artifact: same fd-ownership check and\n # bounded read as job results. Exists because fold-in filenames can embed\n # values unknown at start time (so no --result-path was declared), yet the\n # consumer must never read a predictable /tmp path unchecked.\n target = os.path.abspath(args.path)\n try:\n data = read_owned(target, cfg()[\"result_max\"])\n except Unreadable as exc:\n sys.stderr.write(f\"peer-job-runner: unreadable: {exc}\\n\")\n return 4\n except FileNotFoundError:\n return _report_absent_artifact(target, args)\n except OSError as exc:\n # Only a genuine ENOENT is \"the peer produced nothing\". Every other\n # read failure means the path is there but was refused -- a planted\n # symlink rejected by O_NOFOLLOW is the case this guard exists for --\n # so it takes the trust-failure code, never the routine one.\n sys.stderr.write(f\"peer-job-runner: refused to read {target}: {exc}\\n\")\n return 4\n _emit_bytes(data)\n return 0\n job_dir = resolve_job_dir(args.job, args.skill)\n state = job_state(job_dir)\n if state == \"unreadable\":\n sys.stderr.write(\n f\"peer-job-runner: job state unreadable (ownership or corruption): {job_dir}\\n\"\n )\n return 4\n if state == \"running\":\n sys.stderr.write(\"peer-job-runner: running\\n\")\n return 2\n if state != \"done\":\n sys.stderr.write(f\"peer-job-runner: {state}\\n\")\n return 3\n conf = cfg()\n try:\n meta = json.loads(read_owned(os.path.join(job_dir, \"meta.json\"), META_READ_CAP))\n except Unreadable as exc:\n sys.stderr.write(f\"peer-job-runner: unreadable: {exc}\\n\")\n return 4\n except (OSError, ValueError) as exc:\n sys.stderr.write(f\"peer-job-runner: cannot read job metadata: {exc}\\n\")\n return 4\n result_path = meta.get(\"result_path\") if isinstance(meta, dict) else None\n if not result_path:\n sys.stderr.write(\"peer-job-runner: job declared no result path; nothing to emit\\n\")\n return 0\n try:\n data = read_owned(result_path, conf[\"result_max\"])\n except Unreadable as exc:\n sys.stderr.write(f\"peer-job-runner: unreadable: {exc}\\n\")\n return 4\n except OSError as exc:\n sys.stderr.write(f\"peer-job-runner: result missing or unreadable: {exc}\\n\")\n return 3\n _emit_bytes(data)\n return 0\n\n\ndef cmd_reap(args) -> int:\n codes = set()\n for ref in args.jobs:\n try:\n rc = _reap_one(ref, args.skill)\n except RunnerError as exc:\n sys.stderr.write(f\"peer-job-runner: {exc}\\n\")\n rc = 1\n if rc:\n sys.stderr.write(f\"peer-job-runner: reap {ref}: exit {rc}\\n\")\n codes.add(rc)\n # Exit codes are categories, so a mixed set reports the generic failure and\n # leaves the per-job detail on stderr.\n return codes.pop() if len(codes) == 1 else (1 if codes else 0)\n\n\ndef _reap_one(ref: str, skill) -> int:\n job_dir = resolve_job_dir(ref, skill)\n state = job_state(job_dir)\n if state in TERMINAL_STATES or state == \"never-started\":\n return 0\n if state == \"unreadable\":\n sys.stderr.write(\n f\"peer-job-runner: job state unreadable (ownership or corruption): {job_dir}\\n\"\n )\n return 4\n conf = cfg()\n pid_doc = None\n try:\n pid_doc = json.loads(read_owned(os.path.join(job_dir, \"pid\"), META_READ_CAP))\n except (Unreadable, OSError, ValueError):\n pid_doc = None\n if not isinstance(pid_doc, dict):\n pid_doc = {}\n sup_pid = pid_doc.get(\"supervisor_pid\")\n sup_pgid = pid_doc.get(\"supervisor_pgid\")\n worker_pid = pid_doc.get(\"worker_pid\")\n # Windows-only: the worker tree's job object. Named precisely so this\n # process -- which never held the supervisor's handle -- can reopen and\n # terminate the tree even after the worker leader has exited.\n job_name = pid_doc.get(\"job_name\")\n worker_identity = pid_doc.get(\"worker_identity\")\n supervisor_identity = pid_doc.get(\"supervisor_identity\")\n\n supervisor_ours = (\n isinstance(sup_pid, int)\n and not (IS_WINDOWS and sup_pid == os.getpid())\n and _pid_alive(sup_pid)\n and (\n not IS_WINDOWS\n or _win_process_identity_matches(sup_pid, supervisor_identity)\n )\n )\n if supervisor_ours:\n # The supervisor owns TERM-grace-KILL and the terminal classification.\n # POSIX signals it (SIGTERM to the group or pid); Windows drops the\n # `.reap` marker the supervisor's loop polls for.\n if IS_WINDOWS:\n try:\n with open(os.path.join(job_dir, \".reap\"), \"w\") as f:\n f.write(\"reap\\n\")\n signaled = True\n except OSError:\n signaled = False\n else:\n signaled = (isinstance(sup_pgid, int) and _killpg_quiet(sup_pgid, signal.SIGTERM)) \\\n or _kill_quiet(sup_pid, signal.SIGTERM)\n if signaled:\n # kill -0 is true for a zombie, so confirm the classification landed\n # rather than trusting the signal; fall through to self-cleanup if not.\n # Windows: the supervisor only notices `.reap` on its next poll tick\n # (default 2s), so min(grace, 1.0) alone is shorter than one poll and\n # races into the fallback self-classify path.\n wait_budget = min(conf[\"grace\"], 1.0)\n if IS_WINDOWS:\n wait_budget = max(wait_budget, conf[\"poll\"] + 0.25)\n deadline = time.monotonic() + wait_budget\n while time.monotonic() \u003c deadline:\n if job_state(job_dir) in TERMINAL_STATES:\n return 0\n time.sleep(0.05)\n\n # Supervisor gone: perform the tree kill and classification ourselves,\n # with a short grace so reap still returns quickly. Sweep whenever we have a\n # worker pid, NOT only when its leader is still alive: a child can survive in\n # the worker's process group after the leader exits, and kill_tree targets\n # the pgid precisely so that orphan is swept instead of leaked. Guarding this\n # on _pid_alive would re-defeat kill_tree's dead-leader-safe path. kill_tree\n # returns whether the leader was alive, which is the reap classification.\n worker_leader_alive = False\n if isinstance(worker_pid, int):\n worker_leader_alive = kill_tree(\n worker_pid, min(conf[\"grace\"], 1.0), job_name, worker_identity)\n # A worker can publish its declared result and exit before this fallback runs\n # (e.g. the supervisor died mid-run, then the worker completed cleanly). Honor\n # that result instead of discarding it as died-without-result: read the\n # declared result_path and classify from the artifact, mirroring\n # classify_exit. Only with no usable result do we fall back to timeout (leader\n # was alive) / died-without-result (leader gone).\n result_path = _job_meta(job_dir).get(\"result_path\")\n result_size = None\n if result_path:\n try:\n st = os.lstat(result_path)\n if stat.S_ISREG(st.st_mode) and st.st_size > 0:\n result_size = st.st_size\n except OSError:\n pass\n if result_size is not None and result_size > conf[\"result_max\"]:\n word, reason = \"failed\", (\n f\"result exceeded byte cap ({result_size} > {conf['result_max']} bytes)\"\n )\n elif result_size is not None:\n word, reason = \"done\", \"worker published its result before reap (supervisor was gone)\"\n elif worker_leader_alive:\n word, reason = \"timeout\", (\n \"reaped by request; supervisor was gone, worker tree killed by reap\"\n )\n else:\n word, reason = \"died-without-result\", (\n \"supervisor and worker both gone without a terminal record\"\n )\n write_terminal(job_dir, word, reason, overwrite=False)\n return 0\n\n\n# --- CLI -----------------------------------------------------------------------\n\ndef build_parser() -> argparse.ArgumentParser:\n parser = argparse.ArgumentParser(\n prog=\"peer-job-runner.py\",\n description=(\n \"Detached, supervised job lifecycle for delegated peer work: \"\n \"no call here ever spans the worker's runtime.\"\n ),\n epilog=EXIT_CODES_DOC,\n formatter_class=argparse.RawDescriptionHelpFormatter,\n )\n sub = parser.add_subparsers(dest=\"cmd\", required=True)\n\n p_start = sub.add_parser(\n \"start\",\n help=\"claim a job, preflight, detach the worker, print the job id\",\n )\n p_start.add_argument(\"--skill\", required=True)\n p_start.add_argument(\"--run-id\", required=True, dest=\"run_id\")\n p_start.add_argument(\"--label\", default=None)\n p_start.add_argument(\"--input-digest\", default=None, dest=\"input_digest\")\n p_start.add_argument(\n \"--result-path\", default=None, dest=\"result_path\",\n help=\"worker's expected result file; done then requires it non-empty\",\n )\n p_start.add_argument(\n \"--no-sweep\", action=\"store_true\",\n help=\"retain old sibling run roots (ce-work durable recovery)\",\n )\n\n p_status = sub.add_parser(\"status\", help=\"print each job's state word\")\n p_status.add_argument(\"--skill\", default=None, help=\"limit job-id lookup to this skill\")\n p_status.add_argument(\"--json\", action=\"store_true\")\n p_status.add_argument(\"jobs\", nargs=\"+\", help=\"job ids or job dir paths\")\n\n p_wait = sub.add_parser(\n \"wait\", help=\"bounded poll until all watched jobs settle (or the cap)\"\n )\n p_wait.add_argument(\"--skill\", default=None, help=\"limit job-id lookup to this skill\")\n p_wait.add_argument(\"--max-secs\", type=float, default=30.0, dest=\"max_secs\")\n p_wait.add_argument(\n \"--deadline-secs\", type=_finite_secs, default=None, dest=\"deadline_secs\",\n help=(\n \"aggregate deadline, counted from the last job's start; prints each \"\n \"job's label, state and elapsed time, then one line: 'all settled', \"\n \"'waiting: ...', or 'deadline passed: reap ...'\"\n ),\n )\n p_wait.add_argument(\n \"--deadline-grace-secs\", type=_finite_secs, default=0.0, dest=\"deadline_grace_secs\",\n help=\"added to --deadline-secs, so a caller can pass a worker cap unchanged\",\n )\n p_wait.add_argument(\"--json\", action=\"store_true\")\n p_wait.add_argument(\"jobs\", nargs=\"+\", help=\"job ids or job dir paths\")\n\n p_result = sub.add_parser(\n \"result\",\n help=\"emit a done job's artifact (exit: 0 done, 2 running, 3 other, 4 unreadable)\",\n )\n p_result.add_argument(\"--skill\", default=None, help=\"limit job-id lookup to this skill\")\n p_result.add_argument(\"job\", nargs=\"?\", default=None)\n p_result.add_argument(\n \"--path\",\n default=None,\n help=(\n \"ownership-checked bounded read of this file instead of a job's \"\n \"declared result; pass the job id too so an absent file reports \"\n \"that job's state\"\n ),\n )\n\n p_reap = sub.add_parser(\n \"reap\", help=\"terminate running jobs now; no-op for any already terminal\"\n )\n p_reap.add_argument(\"--skill\", default=None, help=\"limit job-id lookup to this skill\")\n p_reap.add_argument(\"jobs\", nargs=\"+\", help=\"job ids or job dir paths\")\n return parser\n\n\ndef main(argv) -> int:\n # Internal Windows detach re-invocation (not a user-facing subcommand): the\n # detached supervisor process runs `__supervise \u003cjob_dir>`. Gated on\n # IS_WINDOWS so POSIX keeps its previous behavior exactly (argparse usage\n # error), and so a non-win32 Python without geteuid -- where the ownership\n # checks degrade -- can never be steered into exec'ing argv from a planted\n # .spawn.json. Only the Windows detach path ever emits this argv.\n if IS_WINDOWS and argv and argv[0] == \"__supervise\":\n if len(argv) \u003c 2:\n return 2\n return _win_supervise_from_spec(argv[1])\n worker_argv = []\n if \"--\" in argv:\n split = argv.index(\"--\")\n argv, worker_argv = argv[:split], argv[split + 1:]\n args = build_parser().parse_args(argv)\n try:\n if args.cmd == \"start\":\n return cmd_start(args, worker_argv)\n if args.cmd == \"status\":\n return cmd_status(args)\n if args.cmd == \"wait\":\n return cmd_wait(args)\n if args.cmd == \"result\":\n return cmd_result(args)\n if args.cmd == \"reap\":\n return cmd_reap(args)\n return 2\n except RunnerError as exc:\n sys.stderr.write(f\"peer-job-runner: {exc}\\n\")\n return 1\n except Unreadable as exc:\n sys.stderr.write(f\"peer-job-runner: unreadable: {exc}\\n\")\n return 4\n\n\nif __name__ == \"__main__\":\n sys.exit(main(sys.argv[1:]))\n","content_type":"text/x-python; charset=utf-8","language":"python","size":109274,"content_sha256":"bf177d5142120d024b59d8285bbfea45505b6a226c333ddb4afc069228032412"}],"content_json":{"type":"doc","content":[{"type":"heading","attrs":{"level":1},"content":[{"text":"Document Review","type":"text"}]},{"type":"paragraph","content":[{"text":"Help the author finish a sound document they can use to carry out the agreed work. Find problems that would change that work's outcome or materially hinder execution, and resolve them within the authority already given. Judge the document by whether it guides correct work, not by how much detail it contains. Serious consequences warrant attention even when the defect is small. An adequate document needs no changes.","type":"text"}]},{"type":"paragraph","content":[{"text":"Reviewer personas supply evidence; the judgment is yours. Check their claims against the whole document, project facts, and settled decisions. Correct proven errors that prevent an existing decision from being carried out, within the edit authority and reviewer requirements the synthesis reference states. Return only worthwhile improvements still needing permission, consequential choices or essential information only the user can supply, and useful observations.","type":"text"}]},{"type":"paragraph","content":[{"text":"Done when:","type":"text","marks":[{"type":"strong"}]},{"text":" every selected reviewer has returned or is named as failed in Coverage, retained findings have a verified consequence for the agreed work, and every authorized correction assigned to Apply has been made and checked. Report that final state through the interactive approval or decision process, or return it as structured text in non-interactive mode.","type":"text"}]},{"type":"heading","attrs":{"level":2},"content":[{"text":"Interactive mode rules","type":"text"}]},{"type":"paragraph","content":[{"text":"Read ","type":"text","marks":[{"type":"strong"}]},{"text":"references/modes.md","type":"text","marks":[{"type":"code_inline"},{"type":"strong"}]},{"text":" before anything else.","type":"text","marks":[{"type":"strong"}]},{"text":" It defines how the mode is detected, the non-interactive argument contract, and the question-tool rules: match the host's blocking question tool already in the current tool list (never call a user-facing question tool to discover it), pre-load it at the top of the interactive flow if it is listed but unloaded, and fall back to a numbered list only when the harness genuinely lacks one.","type":"text"}]},{"type":"paragraph","content":[{"text":"Either way, a question that calls for a user decision calls the tool or falls back loudly. Narrating it as plain text is a bug.","type":"text"}]},{"type":"heading","attrs":{"level":2},"content":[{"text":"Artifact Root","type":"text"}]},{"type":"paragraph","content":[{"text":"Resolve ","type":"text"},{"text":"\u003croot>","type":"text","marks":[{"type":"code_inline"}]},{"text":" ","type":"text"},{"text":"only","type":"text","marks":[{"type":"strong"}]},{"text":" in the no-path interactive branch, which discovers the most recent plan under ","type":"text"},{"text":"\u003croot>/plans/","type":"text","marks":[{"type":"code_inline"}]},{"text":". Every other run reads the document at the path it was handed. So an absolute-path or non-interactive review — ","type":"text"},{"text":"/tmp/plan.md","type":"text","marks":[{"type":"code_inline"}]},{"text":", possibly outside any repo — never depends on a repo root or a CE config it does not need.","type":"text"}]},{"type":"paragraph","content":[{"text":"\u003c!-- ce-docs-root:start --> ","type":"text"},{"text":"Resolve the CE artifact root ","type":"text","marks":[{"type":"strong"}]},{"text":"\u003croot>","type":"text","marks":[{"type":"code_inline"},{"type":"strong"}]},{"text":" before composing any artifact path.","type":"text","marks":[{"type":"strong"}]}]},{"type":"bullet_list","content":[{"type":"list_item","content":[{"type":"paragraph","content":[{"text":"Read","type":"text","marks":[{"type":"strong"}]},{"text":" ","type":"text"},{"text":"docs_root","type":"text","marks":[{"type":"code_inline"}]},{"text":" from ","type":"text"},{"text":"\u003crepo-root>/.compound-engineering/config.yaml","type":"text","marks":[{"type":"code_inline"}]},{"text":" only (","type":"text"},{"text":"\u003crepo-root>","type":"text","marks":[{"type":"code_inline"}]},{"text":" = ","type":"text"},{"text":"git rev-parse --show-toplevel","type":"text","marks":[{"type":"code_inline"}]},{"text":"). Do not read it from ","type":"text"},{"text":"config.local.yaml","type":"text","marks":[{"type":"code_inline"}]},{"text":". Unset -> ","type":"text"},{"text":"\u003croot>","type":"text","marks":[{"type":"code_inline"}]},{"text":" is ","type":"text"},{"text":"docs","type":"text","marks":[{"type":"code_inline"}]},{"text":", exactly as before.","type":"text"}]}]},{"type":"list_item","content":[{"type":"paragraph","content":[{"text":"Validate","type":"text","marks":[{"type":"strong"}]},{"text":" a set value: a repo-relative directory whose real, symlink-resolved path stays inside the repo and is neither the repo root nor under ","type":"text"},{"text":".git/","type":"text","marks":[{"type":"code_inline"}]},{"text":". Otherwise stop with an error naming ","type":"text"},{"text":"docs_root","type":"text","marks":[{"type":"code_inline"}]},{"text":" and the value -- never fall back to ","type":"text"},{"text":"docs","type":"text","marks":[{"type":"code_inline"}]},{"text":".","type":"text"}]}]},{"type":"list_item","content":[{"type":"paragraph","content":[{"text":"Use","type":"text","marks":[{"type":"strong"}]},{"text":" ","type":"text"},{"text":"\u003croot>","type":"text","marks":[{"type":"code_inline"}]},{"text":" as the sole artifact location: create it if absent, compose each path as ","type":"text"},{"text":"\u003croot>/\u003csubdir>","type":"text","marks":[{"type":"code_inline"}]},{"text":" with this skill's own subdirectory, and never also read ","type":"text"},{"text":"docs","type":"text","marks":[{"type":"code_inline"}]},{"text":". \u003c!-- ce-docs-root:end -->","type":"text"}]}]}]},{"type":"heading","attrs":{"level":2},"content":[{"text":"Phase 1: Get and Analyze Document","type":"text"}]},{"type":"paragraph","content":[{"text":"Read ","type":"text","marks":[{"type":"strong"}]},{"text":"references/document-intake.md","type":"text","marks":[{"type":"code_inline"},{"type":"strong"}]},{"text":" now.","type":"text","marks":[{"type":"strong"}]},{"text":" It covers how the document is obtained in each mode, what to do and say when no document is found, and the classification signals.","type":"text"}]},{"type":"paragraph","content":[{"text":"Two of its rules apply to every later step.","type":"text"}]},{"type":"paragraph","content":[{"text":"Verify before any dispatch.","type":"text","marks":[{"type":"strong"}]},{"text":" Every resolved path must be readable on disk. If one is not, dispatch ","type":"text"},{"text":"no","type":"text","marks":[{"type":"strong"}]},{"text":" personas: reviewers read from the filesystem, so they cannot reach a path that exists only on an unchecked-out branch (issue #925).","type":"text"}]},{"type":"paragraph","content":[{"text":"Classify by content, not readiness labels or file path.","type":"text","marks":[{"type":"strong"}]},{"text":" A unified artifact with only a Product Contract is ","type":"text"},{"text":"unified-requirements","type":"text","marks":[{"type":"code_inline"},{"type":"strong"}]},{"text":"; missing implementation sections are expected. Any implementation planning makes it ","type":"text"},{"text":"unified-plan","type":"text","marks":[{"type":"code_inline"},{"type":"strong"}]},{"text":", including incomplete or blocked planning that needs review. Other artifacts use the legacy ","type":"text"},{"text":"requirements","type":"text","marks":[{"type":"code_inline"}]},{"text":" / ","type":"text"},{"text":"plan","type":"text","marks":[{"type":"code_inline"}]},{"text":" split.","type":"text"}]},{"type":"paragraph","content":[{"text":"HTML unified artifacts take the same routes. Every fix lands in the document's native format; never insert markdown into HTML. That reference covers ID-bearing items. Pass the classification to each persona in the ","type":"text"},{"text":"{document_type}","type":"text","marks":[{"type":"code_inline"}]},{"text":" slot.","type":"text"}]},{"type":"heading","attrs":{"level":2},"content":[{"text":"Phase 2: Announce and Dispatch Personas","type":"text"}]},{"type":"paragraph","content":[{"text":"Skip dispatch only when the completed-review reuse condition in ","type":"text"},{"text":"references/document-intake.md","type":"text","marks":[{"type":"code_inline"}]},{"text":" passes; continue with that evidence at Phase 3.","type":"text"}]},{"type":"paragraph","content":[{"text":"Read ","type":"text","marks":[{"type":"strong"}]},{"text":"references/persona-selection.md","type":"text","marks":[{"type":"code_inline"},{"type":"strong"}]},{"text":" for each conditional persona's activation signals and the announcement format. Two of those signals over-activate on plausible evidence: the sensitive-data bound on ","type":"text"},{"text":"security-lens-reviewer","type":"text","marks":[{"type":"code_inline"}]},{"text":", and the challenge-surface bar on ","type":"text"},{"text":"adversarial-document-reviewer","type":"text","marks":[{"type":"code_inline"}]},{"text":". Then read ","type":"text"},{"text":"references/dispatch.md","type":"text","marks":[{"type":"code_inline"},{"type":"strong"}]},{"text":" for payload variables, slicing, model tiering, and reviewer-failure handling.","type":"text"}]},{"type":"paragraph","content":[{"text":"The team is ","type":"text"},{"text":"coherence-reviewer","type":"text","marks":[{"type":"code_inline"}]},{"text":" and ","type":"text"},{"text":"feasibility-reviewer","type":"text","marks":[{"type":"code_inline"}]},{"text":" always, plus each activated conditional persona. Announce the team briefly before any dispatch, and stay quiet while reviewers run.","type":"text"}]},{"type":"paragraph","content":[{"text":"Dispatch generic subagents with ","type":"text"},{"text":"bounded parallelism","type":"text","marks":[{"type":"strong"}]},{"text":" through the platform's subagent primitive. Give each one its ","type":"text"},{"text":"references/personas/\u003creviewer-name>.md","type":"text","marks":[{"type":"code_inline"}]},{"text":" as ","type":"text"},{"text":"references/dispatch.md","type":"text","marks":[{"type":"code_inline"}]},{"text":" describes. Never dispatch a standalone agent by type or name.","type":"text"}]},{"type":"paragraph","content":[{"text":"A capacity rejection is backpressure, not reviewer failure: wait and retry. If capacity cannot recover and selected reviewers remain undispatched, collect and clean up any started cross-model jobs as ","type":"text"},{"text":"references/cross-model-review.md","type":"text","marks":[{"type":"code_inline"}]},{"text":" describes, then stop as incomplete without synthesis, fixes, or a success handoff. Preserve collected outcomes and report which reviewers completed, failed, or could not run, and why.","type":"text"}]},{"type":"heading","attrs":{"level":3},"content":[{"text":"Cross-Model Judgment Pass","type":"text"}]},{"type":"paragraph","content":[{"text":"Run this pass if any of the ","type":"text"},{"text":"conditional judgment trio","type":"text","marks":[{"type":"strong"}]},{"text":" was activated: ","type":"text"},{"text":"adversarial-document-reviewer","type":"text","marks":[{"type":"code_inline"}]},{"text":", ","type":"text"},{"text":"product-lens-reviewer","type":"text","marks":[{"type":"code_inline"}]},{"text":", ","type":"text"},{"text":"security-lens-reviewer","type":"text","marks":[{"type":"code_inline"}]},{"text":". Follow ","type":"text"},{"text":"references/cross-model-review.md","type":"text","marks":[{"type":"code_inline"}]},{"text":", which defines the whole pass: confirming the host, the one target and route used for the whole document, the disclosure before anything leaves the machine, and how peers are launched, collected, and folded in.","type":"text"}]},{"type":"paragraph","content":[{"text":"The pass is additive and non-blocking: a failure or timeout stops nothing and is named in Coverage. The checkout's ","type":"text"},{"text":"cross_model_review_mode","type":"text","marks":[{"type":"code_inline"}]},{"text":" setting is checked first and can skip the pass with a named reason. Filter recipients only when ","type":"text"},{"text":"CROSS_MODEL_PEERS","type":"text","marks":[{"type":"code_inline"}]},{"text":" is set — unset means unfiltered, not unsanctioned. Never silently change an explicit model or recipient.","type":"text"}]},{"type":"heading","attrs":{"level":2},"content":[{"text":"Phases 3-5: Synthesis, Presentation, and Next Action","type":"text"}]},{"type":"paragraph","content":[{"text":"Wait until every dispatched agent has returned, including any cross-model ","type":"text"},{"text":"\u003creviewer-name>-\u003cprovider>.json","type":"text","marks":[{"type":"code_inline"}]},{"text":" returns. Then read ","type":"text"},{"text":"references/synthesis-and-presentation.md","type":"text","marks":[{"type":"code_inline"}]},{"text":". It defines synthesis, how each finding is routed by confidence and fix class, fix application, the non-interactive result format, and the handoff to the routing question. When promoting agreement, only an artifact with ","type":"text"},{"text":"independence_verified: true","type":"text","marks":[{"type":"code_inline"}]},{"text":" counts as an independent reviewer.","type":"text"}]},{"type":"paragraph","content":[{"text":"Interactive mode only.","type":"text","marks":[{"type":"strong"}]},{"text":" Read ","type":"text"},{"text":"references/walkthrough.md","type":"text","marks":[{"type":"code_inline"}]},{"text":" for the routing question, the per-finding walk-through, and the completion report. Read ","type":"text"},{"text":"references/bulk-preview.md","type":"text","marks":[{"type":"code_inline"}]},{"text":" for the preview behind the walk-through's auto-resolve. Load neither before review evidence is complete, whether newly collected or validly reused, and a non-interactive run never loads them at all — it stops at the structured review result.","type":"text"}]},{"type":"hr","attrs":{"markup":"---"}},{"type":"paragraph","content":[{"text":"Read or pass only the persona prompts the current review selected, each to its own reviewer. The template and schema the dispatch payload fills:","type":"text"}]},{"type":"paragraph","content":[{"text":"@./references/subagent-template.md","type":"text"}]},{"type":"paragraph","content":[{"text":"@./references/findings-schema.json","type":"text"}]}]},"metadata":{"date":"2026-11-02","name":"ce-doc-review","author":"@skillopedia","source":{"stars":25863,"repo_name":"compound-engineering-plugin","origin_url":"https://github.com/EveryInc/compound-engineering-plugin/blob/HEAD/skills/ce-doc-review/SKILL.md","repo_owner":"EveryInc","body_sha256":"d3ceb28dd26441f9cc2b40458a555b4b2d0831cd5be883c1d5ef6299719f3621","cluster_key":"7d4a85e7d39cc930801457c14e1ac5ece7b436bc52f75854057120d000cbc26c","clean_bundle":{"format":"clean-skill-bundle-v1","source":"EveryInc/compound-engineering-plugin/skills/ce-doc-review/SKILL.md","text_bytes":482495,"source_commit":"77cc431daaebea7ee9f33ac530a66eec5404572b","skills_sh_rank":5554,"attachment_count":25,"text_attachments":25,"attachment_storage":"skill_attachments","binary_attachments":0},"cluster_size":1,"skill_md_path":"skills/ce-doc-review/SKILL.md","import_metadata":{"date":"2026-11-02","author":"@skillopedia","version":"v1","category":"business-sales-marketing","category_label":"Business"},"exact_dupes_collapsed_into_this":0},"version":"v1","category":"business-sales-marketing","import_tag":"skills-sh-top20k-2026-11","description":"Review requirements, plans, or specs with role-specific lenses. Use when the user wants to improve an existing planning document.","argument-hint":"[mode:non-interactive] [path/to/document.{md,html}]"}},"renderedAt":1793686552380}

Document Review Help the author finish a sound document they can use to carry out the agreed work. Find problems that would change that work's outcome or materially hinder execution, and resolve them within the authority already given. Judge the document by whether it guides correct work, not by how much detail it contains. Serious consequences warrant attention even when the defect is small. An adequate document needs no changes. Reviewer personas supply evidence; the judgment is yours. Check their claims against the whole document, project facts, and settled decisions. Correct proven errors…