@integrity-labs/agt-cli 0.28.869 → 0.28.871

package/dist/assets/review/SKILL.md

@@ -90,8 +90,16 @@ gh api repos/OWNER/NAME/contents/CLAUDE.md?ref=HEAD_SHA --jq .content
 gh api repos/OWNER/NAME/contents/docs/reference/review-rules.md?ref=HEAD_SHA --jq .content
 ```
 
-Both are base64; decode them. Both may 404, which is fine and normal — most
-repos have neither.
+Both are base64; decode them. **Issue both requests before you begin step 4 —
+this is not optional.** Either may 404, which is fine and normal: most repos
+have neither.
+
+Record which happened, because "absent" and "never asked" are different facts
+and the artefact cannot tell them apart on its own. `rulesFile` is the path when
+the fetch returned content (at either ref), and `null` **only after you have
+actually observed a 404 at both**. If you did not issue the request, the review is `incomplete` with that as
+the reason — a review conducted without rules it never looked for must not be
+filed as though the rules were absent.
 
 Where `review-rules.md` exists it **outranks your priors**: a rule saying "this
 is deliberate here, do not flag it" exists because a reviewer already raised
@@ -99,9 +107,25 @@ that exact thing and a human said no. Where it does not exist, review anyway and
 say so in your summary — a repo without one gets a generic review, which is
 worth having and worth labelling as such.
 
-Pin both reads to `HEAD_SHA`. Reading the default branch's rules against a
+Pin both reads to `HEAD_SHA` first. Reading the default branch's rules against a
 branch's code compares the change to something it was never written against.
 
+**If `review-rules.md` 404s at `HEAD_SHA`, retry it at the default branch, and
+record which ref supplied it.** A branch cut before the rules file landed — or
+simply never merged forward — has no copy of it at its own head, and that is the
+common case, not the rare one: measured 2026-09-08, **8 of the 12 newest open
+PRs** 404 on it. Pinning alone therefore withholds the corpus from most reviews
+precisely while it is newest. Rules describe the repo's current standards rather
+than the branch's snapshot, so the fallback is sound; what would not be sound is
+applying them without saying so. Set `rulesRef` to `"head"` or `"default"`
+alongside `rulesFile`.
+
+Do NOT pipe or redirect these commands. Use `--jq` on the `gh` call itself and
+read the output directly. A reviewer runs unattended under a tool allowlist
+narrow enough that it cannot post to GitHub, and `gh … | jq` or `gh … > file`
+is a compound command that such an allowlist rejects — so composing a shell
+pipeline costs you turns and buys nothing.
+
 ## 4. Review
 
 Prioritise, in this order:
@@ -157,6 +181,7 @@ directory (create it if absent):
   "head": "abc123def456",
   "reviewedAt": "2026-09-07T00:00:00Z",
   "rulesFile": "docs/reference/review-rules.md",
+  "rulesRef": "head",
   "status": "reviewed",
   "findings": [
     {

package/dist/bin/agt.js

@@ -40,12 +40,12 @@ import {
   success,
   table,
   warn
-} from "../chunk-SJW4JNJQ.js";
+} from "../chunk-74PZNNGN.js";
 import {
   getProjectDir,
   isSessionResumeDisabled,
   readDirectChatSessionState
-} from "../chunk-EK7QNCEO.js";
+} from "../chunk-QYI3M3WC.js";
 import {
   AnchorSessionClient,
   CHANNEL_REGISTRY,
@@ -79,7 +79,7 @@ import {
   serializeManifestForSlackCli,
   sessionFileExists,
   sessionTranscriptDir
-} from "../chunk-EOEAZDKE.js";
+} from "../chunk-XK54CUEP.js";
 import "../chunk-XWVM4KPK.js";
 
 // src/bin/agt.ts
@@ -5467,7 +5467,7 @@ import { execFileSync, execSync } from "child_process";
 import { existsSync as existsSync11, realpathSync as realpathSync2 } from "fs";
 import chalk18 from "chalk";
 import ora16 from "ora";
-var cliVersion = true ? "0.28.869" : "dev";
+var cliVersion = true ? "0.28.871" : "dev";
 async function fetchLatestVersion() {
   const host2 = getHost();
   if (!host2) return null;
@@ -6658,7 +6658,7 @@ function handleError(err) {
 }
 
 // src/bin/agt.ts
-var cliVersion2 = true ? "0.28.869" : "dev";
+var cliVersion2 = true ? "0.28.871" : "dev";
 var program = new Command();
 program.name("agt").description("Augmented CLI \u2014 agent provisioning and management").version(cliVersion2).option("--json", "Emit machine-readable JSON output (suppress spinners and colors)").option("--skip-update-check", "Skip the automatic update check on startup");
 program.hook("preAction", async (thisCommand, actionCommand) => {

package/dist/{chunk-SJW4JNJQ.js → chunk-74PZNNGN.js}

@@ -16,7 +16,7 @@ import {
   parseEnvIntegrations,
   shellQuote,
   summarizeUnanswerablePane
-} from "./chunk-EK7QNCEO.js";
+} from "./chunk-QYI3M3WC.js";
 import {
   BIND_FAILURE_QUARANTINE_THRESHOLD,
   INTEGRATIONS_SECTION_END,
@@ -76,7 +76,7 @@ import {
   sessionTranscriptDir,
   worseConnectivityOutcome,
   wrapScheduledTaskPrompt
-} from "./chunk-EOEAZDKE.js";
+} from "./chunk-XK54CUEP.js";
 import {
   parsePsRows
 } from "./chunk-XWVM4KPK.js";
@@ -3341,7 +3341,50 @@ function provisionIsolationHook(codeName, agentId) {
   const hookScriptPath = join4(claudeDir, "agt-isolation-hook.sh");
   const hookScript = [
     "#!/bin/bash",
-    "# Auto-generated by Augmented \u2014 prevents cross-agent file access.",
+    "# Auto-generated by Augmented.",
+    "#",
+    "# WHAT THIS IS: a guard against ACCIDENTS. It catches an agent that reaches",
+    "# for another agent's directory by mistake, and it says so in isolation.log.",
+    "#",
+    "# WHAT THIS IS NOT: a confidentiality boundary. ENG-10266 settled this",
+    "# explicitly, and the reasons are structural rather than fixable here:",
+    "#",
+    "#   1. THE SUBJECT CAN REWRITE THIS FILE. Every agent runs as uid=0, and this",
+    "#      script and the settings.local.json that registers it both live inside",
+    "#      the agent directory this hook constrains. An ordinary file write",
+    "#      disables it. Moving the file elsewhere does not help: a root process",
+    "#      on a shared filesystem reaches wherever it is put.",
+    "#",
+    "#   2. IT MATCHES RAW PRE-EXPANSION TEXT, and that fails in BOTH directions",
+    "#      depending on WHERE the expansion sits. The two cases are different and",
+    "#      worth keeping apart:",
+    "#",
+    "#      HIDDEN BASE -> under-blocks (the hole). If the expansion hides the base",
+    "#      path itself, the literal text this greps for never appears, the branch",
+    "#      exits 0, and the shell expands afterwards. Verified:",
+    "#          ~/.augmented/<other>/x       does not fire, proceeds",
+    "#          $HOME/.augmented/<other>/x   does not fire, proceeds",
+    "#      The platform's own CLAUDE.md documents working paths in tilde form, so",
+    "#      the spelling agents are taught is the spelling this cannot see.",
+    "#",
+    "#      EXPANSION AFTER THE BASE -> over-blocks (a false positive). The literal",
+    "#      base IS present, so the branch fires, takes the unexpanded token as the",
+    "#      agent segment, finds it in no allowlist, and refuses. Verified:",
+    "#          /root/.augmented/$CN/x       fires, AGENT_DIR=[$CN]    -> BLOCK",
+    "#          /root/.augmented/${VAR}/x    fires, AGENT_DIR=[${VAR}] -> BLOCK",
+    "#      This refuses commands that were going nowhere near another agent.",
+    "#",
+    "#   3. IT OVER-BLOCKS ON PUNCTUATION TOO. [^/]+ does not stop at a shell",
+    "#      metacharacter, so `ls /root/.augmented/<self> && echo hi` yields an",
+    "#      AGENT_DIR of `<self> && echo hi`, which matches no allowlist entry and",
+    "#      is refused. Agents have been blocked from their OWN directories this",
+    "#      way, repeatedly. Check isolation.log before reading an entry as an",
+    "#      attempted breach: most of them are not.",
+    "#",
+    "# Real isolation between co-located agents needs OS enforcement - a uid per",
+    "# agent, a mount namespace, or a container per agent (ENG-4745, ENG-6633).",
+    "# Do not treat a passing hook as evidence that separation exists.",
+    "#",
     "# Exit 0 = allow, Exit 2 = block (with stderr message shown to agent)",
     "set -euo pipefail",
     "INPUT=$(cat)",
@@ -6728,7 +6771,7 @@ function exchangeFailureKind(err) {
 }
 
 // src/lib/api-client.ts
-var agtCliVersion = true ? "0.28.869" : "dev";
+var agtCliVersion = true ? "0.28.871" : "dev";
 var lastConfigHash = null;
 function setConfigHash(hash) {
   lastConfigHash = hash && hash.length > 0 ? hash : null;
@@ -11338,4 +11381,4 @@ export {
   managerInstallSystemUnitCommand,
   managerUninstallSystemUnitCommand
 };
-//# sourceMappingURL=chunk-SJW4JNJQ.js.map
+//# sourceMappingURL=chunk-74PZNNGN.js.map