@integrity-labs/agt-cli 0.28.507 → 0.28.508

package/dist/{chunk-4PRR22KH.js → chunk-LNPJI2AE.js}

@@ -10371,6 +10371,54 @@ var FLAG_REGISTRY = [
     // Turning this on lets agents originate proposals that create customer
     // infrastructure once approved — same posture as augmented-support-writes.
     sensitive: true
+  },
+  {
+    key: "github-broker-credentials",
+    description: "Deliver GitHub App installation tokens to `git` and the `gh` CLI through the broker credential endpoint at USE time, instead of materializing them into the agent process environment at spawn (ENG-8344). A GitHub App installation token has a ~1h TTL, so the spawn-frozen GITHUB_ACCESS_TOKEN / GITHUB_TOKEN pair rotates hourly; the wrapper sources `.env.integrations` exactly once, so the only way a running agent sees the new value is a full session respawn (measured: 6 agents, ~144 kills/day). ENG-8343 stopped those kills landing mid-turn; this removes them. ON: `POST /host/agent-integrations` stops minting the installation token into the payload for auth_type=github_app rows (so computeIntegrationsHash stops seeing a rotating value) and marks the row credential_delivery=broker; the claudecode adapter then writes a git credential helper + a `gh` shim that fetch a fresh token per use from POST /host/agent-integrations/github/credential (ENG-8264 name addressing). OFF (default): byte-for-byte today's behaviour - the token is minted into the payload and materialized as GITHUB_ACCESS_TOKEN / GITHUB_TOKEN, no helper, no shim. NOTE the scope of the security claim: this removes the token from the agent's ENVIRONMENT, which is what stops the respawns. It does NOT make the token unreachable by a determined agent - the agent can invoke the helper itself. Strictly better than today, not a sandbox. Evaluated API-side only, per agent, in POST /host/agent-integrations; no host-side envVar. The host does not read this flag - it keys off the resulting credential_delivery marker on the row - so there is deliberately no host override that could put the token back on its own. Roll back from the flag.",
+    flagType: "boolean",
+    // Declared safe value is `false` = today's env delivery. Fail-safe direction:
+    // a flag-DB read error must never strip `git`/`gh` credentials off a live
+    // agent, which is what the ON path does when its host has no helper writer.
+    defaultValue: false,
+    // Flipping ON changes how every GitHub App agent on the scope authenticates
+    // `git push` / `gh`. A bad flip takes those away, so it is a deliberate,
+    // confirmed action (ADR-0022 §4).
+    sensitive: true,
+    // Set even though the flag is evaluated centrally, because `since` is a
+    // REACH threshold ("a host this old has no consumer code for the flag") and
+    // that is exactly the hazard here: the host does not read the flag, but it
+    // does need the claudecode adapter's helper/shim writer and the wrapper PATH
+    // prepend. On a host too old for those, the API would stop shipping the
+    // token and NOTHING would replace it. Without `since` the flip-reach modal
+    // reports FULL reach and hides that.
+    //
+    // DELIBERATE UNREACHABLE PLACEHOLDER - NOT A REAL THRESHOLD YET.
+    // The sibling entries in this registry (`slack-hot-thread-guard` 0.28.339,
+    // `slack-scheduled-channel-guard` 0.28.421) are BISECTED from published npm
+    // artifacts, which is only possible AFTER this merges and publishes. A
+    // pre-merge guess cannot be right, and the two directions are not
+    // symmetric:
+    //   - too HIGH  -> hosts classify `stale`, the modal over-warns. Annoying.
+    //   - too LOW   -> hosts classify `honors` while carrying NO helper writer,
+    //                  the modal promises full reach, and a flip strips `git`/
+    //                  `gh` credentials with nothing replacing them. Outage.
+    // So this is pinned ABOVE every published release on purpose: it classifies
+    // EVERY host as `stale`, so the modal reports no reach rather than reach it
+    // has no basis for. That is the intended pre-bisect state.
+    //
+    // BUT `since` IS ADVISORY, NOT A GUARD. It is read in exactly one place -
+    // `computeFlagReach` behind GET /admin/reach, which renders the flip-reach
+    // modal. `evaluate.ts` never reads it and neither does the override write
+    // path. The placeholder therefore changes what an operator is TOLD; it does
+    // not stop a flip on an under-`since` host. What actually holds the line
+    // here is `defaultValue: false` plus `sensitive: true` plus the runbook.
+    //
+    // BEFORE THIS FLAG IS FLIPPED ANYWHERE: bisect the published dist for the
+    // first agt-cli whose claudecode adapter writes agt-bin/ and whose wrapper
+    // prepends it, then replace this with that exact version. The rollout
+    // runbook gates on it. Leaving the placeholder in place is safe (nothing
+    // flips); shipping a guessed number is not.
+    since: "0.29.0"
   }
 ];
 var REGISTRY_BY_KEY = new Map(FLAG_REGISTRY.map((definition) => [definition.key, definition]));
@@ -13705,6 +13753,10 @@ function writePersistentClaudeWrapper(args) {
       "set +a"
     );
   }
+  const brokerBinDir = join6(projectDir, ".claude", "agt-bin");
+  if (existsSync6(brokerBinDir)) {
+    wrapperLines.push(`export PATH=${JSON.stringify(brokerBinDir)}:"$PATH"`);
+  }
   const initPromptArg = initPrompt ? `${JSON.stringify(initPrompt)} ` : "";
   wrapperLines.push(
     `exec ${JSON.stringify(claudeBin)} ${initPromptArg}${claudeArgsJoined}`
@@ -14856,4 +14908,4 @@ export {
   stopAllSessionsAndWait,
   getProjectDir
 };
-//# sourceMappingURL=chunk-4PRR22KH.js.map
+//# sourceMappingURL=chunk-LNPJI2AE.js.map