agents-inc 0.167.0 → 0.168.0

package/dist/commands/edit.js

@@ -3,13 +3,13 @@ import {
   publishSeedConfig,
   seedPayloadForInstallation,
   skillsAuthoredHere
-} from "../chunk-Y47EV74W.js";
+} from "../chunk-4DRFEYCE.js";
 import {
   RemovalPlanConfirm
-} from "../chunk-TODCQFRB.js";
+} from "../chunk-W6GO2CYX.js";
 import {
   unofferablePlacementsFound
-} from "../chunk-PXWG7KFO.js";
+} from "../chunk-KTLKR4YY.js";
 import {
   arrivalLines,
   arrivalsByScope,
@@ -21,46 +21,46 @@ import {
   sameAssignments,
   skillLabel,
   withUnplaceableAssignees
-} from "../chunk-V3I7VQ7N.js";
+} from "../chunk-EH2PGPLP.js";
 import {
   runWizardSession
-} from "../chunk-HHIYT3PI.js";
-import "../chunk-WWIFNPJG.js";
-import "../chunk-RAHSOJQ2.js";
-import "../chunk-DAIEQJIF.js";
-import "../chunk-SCOQ34YM.js";
-import "../chunk-4NDBUJHK.js";
-import "../chunk-2VUYFTUB.js";
-import "../chunk-T6UWJAB3.js";
-import "../chunk-6L75SQZU.js";
-import "../chunk-LFWLEDUG.js";
-import "../chunk-TZ2YJJKK.js";
-import "../chunk-SAYZWCH6.js";
-import "../chunk-PNB3524H.js";
-import "../chunk-XZEVWPX6.js";
-import "../chunk-QHDPLLK6.js";
-import "../chunk-MPTNC32T.js";
+} from "../chunk-X6ZZEYO3.js";
+import "../chunk-PCFUXQJA.js";
+import "../chunk-KQ3PDARZ.js";
+import "../chunk-PLX2FLEQ.js";
+import "../chunk-HL3GYRMR.js";
+import "../chunk-JVUN6AOO.js";
+import "../chunk-QWFG7Y3R.js";
+import "../chunk-6WGXTB2L.js";
+import "../chunk-L3W2JWBA.js";
+import "../chunk-HI6YB7XP.js";
+import "../chunk-23DNVNFP.js";
+import "../chunk-7PYXS2QU.js";
+import "../chunk-GRL73YPF.js";
+import "../chunk-DJEO3F2H.js";
+import "../chunk-YN3SWF53.js";
+import "../chunk-4HWOYYY7.js";
 import "../chunk-TSIU7DME.js";
 import "../chunk-JOEA52NR.js";
 import {
   awaitUnderSpinner
 } from "../chunk-VRXX4D6K.js";
-import "../chunk-SXVHSOTH.js";
+import "../chunk-3DIDV6H3.js";
 import "../chunk-XQHZPINL.js";
-import "../chunk-JQRJQ4SS.js";
-import "../chunk-TTY6DPJN.js";
+import "../chunk-JPMV4E4E.js";
+import "../chunk-LFUJCRCT.js";
 import "../chunk-WTJ3AY33.js";
-import "../chunk-WVUDLKIO.js";
+import "../chunk-SLBL3WIP.js";
 import {
   fetchSeedConfig
-} from "../chunk-25MPBIK3.js";
+} from "../chunk-ZUGH4MQG.js";
 import {
   promptConfirm
 } from "../chunk-4AAOOAIN.js";
 import "../chunk-OZXAJNLA.js";
 import {
   hostCompileNotices
-} from "../chunk-ZXKBERQU.js";
+} from "../chunk-27AVGFXH.js";
 import {
   applyMigratedGlobalSources,
   compileAgentsAllScopes,
@@ -71,37 +71,37 @@ import {
   mutateGlobal,
   sayCapturedWarnings,
   writeProjectConfig
-} from "../chunk-G6UREQ4V.js";
+} from "../chunk-5JUQKGCP.js";
 import {
   copyLocalSkills,
   discoverInstalledSkills,
   removeCompiledAgents,
   uninstallPluginSkills
-} from "../chunk-HWEKHVLE.js";
+} from "../chunk-TW35AEB4.js";
 import {
   loadAgentDefs
-} from "../chunk-PMJWD27C.js";
-import "../chunk-ZR4YPKXQ.js";
-import "../chunk-GE3WKJ6Q.js";
-import "../chunk-MAWNVT2C.js";
+} from "../chunk-GE7NUPMI.js";
+import "../chunk-JIBCVXXL.js";
+import "../chunk-3LZNY46H.js";
+import "../chunk-4UKMQUAU.js";
 import "../chunk-JFLVEFFP.js";
 import {
   formatScopeTag
-} from "../chunk-7JNJ52CD.js";
+} from "../chunk-WNLF5EDZ.js";
 import {
   providerFlag,
   providerNamedBy,
   refuseAnAmbiguousInstallation
-} from "../chunk-NBXXFIIW.js";
+} from "../chunk-HR7TK2XW.js";
 import {
   BaseCommand,
   EXIT_CODES,
   carriedSkillsThatDiffer,
   registerExternalSkills,
   writeExternalSkills
-} from "../chunk-Y3VIRVVE.js";
-import "../chunk-VAPTVR3J.js";
-import "../chunk-RZDXDMT6.js";
+} from "../chunk-H4ZKGGTE.js";
+import "../chunk-VVRI35SK.js";
+import "../chunk-CMJBKQUH.js";
 import {
   ERROR_MESSAGES,
   INCOMPLETE_WORK_RECOVERY,
@@ -142,13 +142,13 @@ import {
   unplaceableKept,
   unresolvedSkillRemovalReasons,
   wizardNeedsTerminal
-} from "../chunk-A4KDXNOT.js";
+} from "../chunk-C2UISKIM.js";
 import {
   getSkillById,
   getSkillDisplayName,
   matrix,
   typedKeys
-} from "../chunk-V5WXIM2F.js";
+} from "../chunk-NU2K5G54.js";
 import {
   CLI_COLORS,
   CLI_INVOKE_COMMAND,
@@ -161,7 +161,7 @@ import {
   formatSourceDisplayName,
   isActiveAt,
   isGlobalTombstone
-} from "../chunk-ENGYJCTF.js";
+} from "../chunk-AG3ZVM3B.js";
 import "../chunk-OI2H6M5I.js";
 
 // src/cli/commands/edit.tsx
@@ -340,14 +340,7 @@ var Edit = class _Edit extends BaseCommand {
     }),
     provider: providerFlag()
   };
-  /**
-   * The command in two statements: do the work, then answer for it.
-   *
-   * `applyEdit` owns every path through the edit and returns however it likes; the exit code is
-   * decided once, here, after the last of them. Deciding it inside `applyEdit` would mean
-   * deciding it at each of that method's three endings, and an ending added later would inherit
-   * the exit-0 default silently — which is the defect this pair exists to close.
-   */
+  /** The exit code is decided once, after every path through `applyEdit`. */
   async run() {
     await this.applyEdit();
     this.exitIfWorkIncomplete();
@@ -448,7 +441,6 @@ var Edit = class _Edit extends BaseCommand {
     await this.reportWhatThisHostCannotCarry(editRoot);
     this.logCompletionSummary(changes);
   }
-  /** The interactive producer: open the wizard on what is installed, return what was chosen. */
   async selectionFromWizard(context, editRoot) {
     if (!process.stdin.isTTY) {
       this.error(wizardNeedsTerminal("edit"), { exit: EXIT_CODES.ERROR });
@@ -458,22 +450,10 @@ var Edit = class _Edit extends BaseCommand {
     return { producer: "wizard", result };
   }
   /**
-   * The `--from <id>` producer: fetch, seat what the configuration carries, decode, and put back
-   * what this run has no authority to remove.
-   *
-   * Everything a shared configuration says is a statement about the whole roster, so the apply
-   * that follows is destructive — the project is made to MATCH it. Two exceptions come back into
-   * the result: a skill written here, which the producer drops on the way out because
-   * `forkedFrom` says the CLI never wrote it, and an id the configuration NAMES that this
-   * catalogue cannot place, because a destructive apply removes on intent and never on its own
-   * inability. Both are disclosed in the confirm rather than silently excused.
-   *
-   * From a project the global install comes back too, whole: it is one installation every project
-   * on the machine reads, so a project run only ADDS to it (owner ruling 2026-10-02). Every global
-   * entry the configuration leaves out is put back as installed, every one it states otherwise is
-   * held as installed and named in the confirm, and only the global entries the global install
-   * lacks arrive — a skill with its rows, on a global sub-agent already there too. At the home
-   * directory the run IS the global install, so nothing is held there.
+   * A shared configuration applies destructively, except for a skill written here (`forkedFrom`)
+   * and an id this catalogue cannot place — both disclosed in the confirm. From a project the
+   * global install is only added to (owner ruling 2026-10-02): entries the configuration omits or
+   * states otherwise are held as installed, and only what it lacks arrives.
    */
   async selectionFromSharedConfig(payload, context, editRoot) {
     const { matrix: sourceMatrix } = context.sourceResult;
@@ -517,15 +497,7 @@ var Edit = class _Edit extends BaseCommand {
       carried: arrivingCarried
     };
   }
-  /**
-   * Which installed skills the round trip does not own, asked of the disk exactly as the
-   * producing half asks it.
-   *
-   * Best-effort by nature and never fatal: the question is only ever asked to PROTECT a skill,
-   * so a directory that cannot be read protects nothing and must not fail an apply. The
-   * consequence of an unanswered question is that the entry is treated as the CLI's own, which
-   * is what every other command already assumes about it.
-   */
+  /** Best-effort: an unreadable directory protects nothing and must not fail an apply. */
   async readAuthoredHere(projectConfig, editRoot) {
     if (!projectConfig) return /* @__PURE__ */ new Set();
     try {
@@ -535,16 +507,7 @@ var Edit = class _Edit extends BaseCommand {
       return /* @__PURE__ */ new Set();
     }
   }
-  /**
-   * The id, fetched — or the run refused before it is.
-   *
-   * Two questions are answered first, because nothing in the payload can change either answer:
-   * an apply with nothing to apply to, or that cannot be confirmed, is over before the store is
-   * asked, and a refusal that had already spent a round trip would be describing work it never
-   * intended to do. Whether anything is installed comes before the terminal: the terminal
-   * refusal's way on is this same command from a terminal, which in a folder with nothing
-   * installed would only refuse again.
-   */
+  /** Refuses an empty directory or a missing terminal before spending a round trip. */
   async fetchSharedConfigOrFail(id) {
     if (!await detectProject()) {
       this.error(nothingInstalledToApplyTo(id), { exit: EXIT_CODES.ERROR });
@@ -559,11 +522,6 @@ var Edit = class _Edit extends BaseCommand {
     }
     return fetched.payload;
   }
-  /**
-   * The decode refuses a payload the config model has nowhere to write (see `seedToWizardResult`).
-   * That is a failure of this command, reported with this command's exit code rather than left to
-   * surface as an unhandled throw.
-   */
   decodeSeedOrFail(payload, sourceMatrix) {
     try {
       return seedToWizardResult(payload, sourceMatrix);
@@ -571,13 +529,6 @@ var Edit = class _Edit extends BaseCommand {
       this.handleError(error);
     }
   }
-  /**
-   * Seats the catalogue entries the payload carries with it, or refuses the run. The refusals are
-   * `registerExternalSkills`' own — a carried skill asked for as a plugin, which no marketplace
-   * serves, and a carried skill claiming an id the loaded catalogue already owns, whose bytes would
-   * be written over that catalogue's own copy — reported with this command's exit code for the same
-   * reason the decode's is.
-   */
   registerExternalSkillsOrFail(payload, sourceMatrix, projectDir) {
     try {
       return registerExternalSkills(payload, sourceMatrix, projectDir);
@@ -585,15 +536,10 @@ var Edit = class _Edit extends BaseCommand {
       this.handleError(error);
     }
   }
-  /**
-   * Confirms a shared configuration, then writes the skills it brought with it — answering with
-   * those whose files now read differently.
-   */
   async confirmAndWriteCarried(selection, planned, result) {
     await this.confirmSharedConfigOrCancel(planned, selection.kept, applyArrivals(planned, result));
     return this.writeCarriedSkills(selection.carried);
   }
-  /** Writes the skills the configuration brought with it, and says which they were. */
   async writeCarriedSkills(carried) {
     if (carried.length === 0) return [];
     const rewritten = await writeExternalSkills(carried);
@@ -601,17 +547,8 @@ var Edit = class _Edit extends BaseCommand {
     return rewritten;
   }
   /**
-   * Shows what applying this configuration takes away, and stops the run unless a person says
-   * yes.
-   *
-   * The plan is built from the SAME `ConfigChanges` the apply below acts on, so what is approved
-   * and what is removed are one value read twice. Its kept half is the other side of the same
-   * honesty: nothing is refused over an entry this run cannot remove, and nothing is silent
-   * about one either.
-   *
-   * Above it go the two lists every `--from` install prints — what the apply adds or changes in
-   * this project, and in the global install — so the question is asked about the whole of what a
-   * yes does, not only about what it takes away.
+   * The plan and the apply read the same `ConfigChanges`, so what is approved is what is removed.
+   * The arrivals print above it, so the question covers the whole of what a yes does.
    */
   async confirmSharedConfigOrCancel(changes, kept, arrivals) {
     const plan = sharedConfigPlan(changes, kept);
@@ -633,29 +570,14 @@ var Edit = class _Edit extends BaseCommand {
     this.log("\nEdit cancelled");
     this.error("Cancelled", { exit: EXIT_CODES.CANCELLED });
   }
-  /**
-   * Refuses a configuration whose own host cannot place what it asks for, before the wizard runs.
-   *
-   * `compile` asks the same question of the same roster and `update` asks it too. `edit` is the
-   * command that REWRITES the file, so letting it through would mean a user curating a
-   * configuration no command will act on — and then saving it back, with the unofferable row
-   * carried forward because nothing on the edit path had any reason to drop it.
-   */
+  /** `edit` rewrites the file, so it must not let a user curate a roster no command acts on. */
   async refuseUnofferablePlacements(cwd) {
     const [finding] = await unofferablePlacementsFound(cwd);
     if (finding !== void 0) this.error(finding, { exit: EXIT_CODES.ERROR });
   }
   /**
-   * The outbound half of the editor round trip: this installation, minted as a configuration
-   * the editor opens, and handed to a browser.
-   *
-   * It is the same mint `share` performs — same reader, same mapping, same refusals, through
-   * `seedPayloadForInstallation` — because an id is the whole of what either command produces
-   * and two spellings of "the installation in this directory" would mint two different ids for
-   * one project. What differs is only the ending: `share` reports the id, this opens it.
-   *
-   * Nothing on disk is touched. A configuration is read, not rewritten, so a run that changed
-   * anything here would be editing the project on the way to offering to edit it.
+   * Mints through `seedPayloadForInstallation`, as `share` does, so one project has one id. Touches
+   * nothing on disk.
    */
   async openInEditor(projectDir) {
     const prepared = await seedPayloadForInstallation(projectDir);
@@ -669,17 +591,7 @@ var Edit = class _Edit extends BaseCommand {
     }
     await this.handToBrowser(published.id);
   }
-  /**
-   * An id somebody shared, opened rather than applied.
-   *
-   * The counterpart of `openInEditor` above and deliberately not a variant of it: that one MINTS,
-   * because an installation is not yet a configuration the store holds. An id already is one, so
-   * there is nothing to publish, and no way for this to fail except the browser refusing to launch
-   * — which is a warning beside a link that still works.
-   *
-   * `--from` without `--ui` remains the destructive apply. Looking and applying are the two
-   * things a recipient can do with an id, and this is the one that changes nothing.
-   */
+  /** Opens an id rather than applying it; `--from` without `--ui` is the destructive apply. */
   async openSharedInEditor(id, cwd) {
     const url = editorConfigUrl(id);
     this.log(`Open it at ${url}`);
@@ -688,18 +600,7 @@ var Edit = class _Edit extends BaseCommand {
     const opened = await openUrl(url);
     if (!opened.ok) this.warn(opened.error);
   }
-  /**
-   * The link, printed first and opened second.
-   *
-   * Printed first because it is the only part of this that works everywhere: over a pipe, in CI
-   * and on a machine with no desktop session there is no browser to be anybody's, and a link
-   * nobody can copy is the whole loss. Opening one is the convenience on top, so a failure to
-   * open is a warning beside a link that still works rather than a failed command.
-   *
-   * `process.stdin.isTTY` is the same question `init`'s dashboard asks before it offers a
-   * prompt — is there a person here — because that is exactly what "whose browser is this"
-   * means.
-   */
+  /** The link prints first: it works over a pipe, in CI and with no desktop. Opening is extra. */
   async handToBrowser(id) {
     this.logSuccess(`Shared as ${id}`);
     for (const destination of sharedConfigDestinations(id)) {
@@ -709,20 +610,12 @@ var Edit = class _Edit extends BaseCommand {
     const opened = await openUrl(editorConfigUrl(id));
     if (!opened.ok) this.warn(opened.error);
   }
-  /**
-   * The load below, behind a spinner that comes down whichever way the await ends — all
-   * three of `loadContext`'s refusals are raised while it is mounted.
-   */
   async loadContextUnderSpinner(setupMarketplace) {
     return awaitUnderSpinner(
       STATUS_MESSAGES.LOADING_SKILLS,
       () => this.loadContext(setupMarketplace)
     );
   }
-  /**
-   * @param setupMarketplace - What `init --marketplace` named for the project setup this run
-   *   continues. Absent, the catalogue is the one the installation stored.
-   */
   async loadContext(setupMarketplace) {
     const detected = await detectProject();
     if (!detected) {
@@ -807,43 +700,11 @@ var Edit = class _Edit extends BaseCommand {
     });
   }
   /**
-   * Records, in the GLOBAL config, the install-mode migrations this run already performed
-   * under `$HOME`.
-   *
-   * A project-context edit otherwise never writes global config (commit 403df46, "never
-   * modify global config from project-level operations"), and `authoritativeScope: "owned"`
-   * enforces that for the roster — an inherited global-active entry is read-only, so
-   * `mergeGlobalConfigs` preserves it verbatim. That protection is correct for selection and
-   * scope, but `executeMigration` resolves each skill's paths from ITS OWN scope: switching a
-   * global-scoped skill's source has already copied it under `$HOME` (or deleted it) and
-   * added/removed its user-scope plugin registration by the time the config is written.
-   * Leaving the global config alone there protects nothing — it only makes the recorded
-   * source contradict the filesystem and the plugin registry.
-   *
-   * So authority follows the work actually performed, and no further: only ids
-   * `executeMigration` acted on this run are rewritten, and only their `origin` field. Global
-   * entries this session merely displayed, re-scoped or deselected are untouched, as are
-   * `marketplace`, `stack`, `agents` and every other registered project's view of them.
-   *
-   * Driving a global-scope migration from a project directory is NOT a supported flow, and this
-   * method is no longer the thing that permits it. The wizard now refuses it at source: the two
-   * bulk install-mode keys are withdrawn, and `setInstallMode` ignores a project-context call
-   * against a global slot the hydration snapshot owns — the same authority the Sources grid
-   * already enforced by rendering that row inert. What still reaches here is the residue that
-   * authority leaves legitimate: a mode change committed on the project half of a `[P][G]` pair
-   * and then carried to global scope by a P->G collapse (`s`) in the same session. The entry is
-   * the project's own to configure at the moment it is configured, and global at the moment it
-   * is written — so the migration is real, and the global config must record it.
-   *
-   * Runs BEFORE `writeConfigAndCompile`, so the global config written here is the one
-   * the wizard write reloads, keeps (existing wins in `mergeGlobalConfigs`) and inlines
-   * into the project config — both files then tell the same story. That second write
-   * classifies as a no-op against what this one left on disk, so the fan-out below
-   * happens once.
-   *
-   * The migrated `origin` decides the reference form a compiled agent emits, so every
-   * OTHER registered project's config and agents are stale until this write fans out —
-   * which is why it goes through the gate rather than straight to the file.
+   * Records in the GLOBAL config the install-mode migrations this run already performed under
+   * `$HOME`, and only their `origin`: `executeMigration` resolves paths from each skill's own
+   * scope, so leaving the record alone would contradict the disk and the plugin registry. The
+   * reachable case is a mode change on a `[P][G]` pair's project half, carried to global by `s`.
+   * Runs before `writeConfigAndCompile`, through the gate so every other project is refreshed.
    */
   async recordGlobalSourceMigrations(migratedSkillIds, newSkills, editRoot, context) {
     if (editRoot.isGlobal) return;
@@ -855,9 +716,7 @@ var Edit = class _Edit extends BaseCommand {
       const report = await mutateGlobal(
         { kind: "migrate-skill-sources", sources: migratedSources },
         {
-          // The deps contract in config-gate/deps.ts declares
-          // `() => Promise<MergedSkillsMatrix>`; this caller happens to have its
-          // matrix in hand already.
+          // config-gate/deps.ts declares `loadMatrix` async; the matrix is already in hand.
           // eslint-disable-next-line @typescript-eslint/require-await -- deps contract
           loadMatrix: async () => context.sourceResult.matrix,
           loadAgents: async () => (await loadAgentDefs()).agents
@@ -981,18 +840,8 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
     return migratedSkillIds;
   }
   /**
-   * What the plugin→eject half copied, what it could not, and the refusal to continue past
-   * a failure.
-   *
-   * The shape of `BaseCommand.reportPluginInstalls`, deliberately: the two directions of one
-   * mode switch owe the user the same account, and the eject half having no structured
-   * failure at all is what let a config record `origin: "eject"` for a skill whose copy was
-   * refused. It stays private because `edit` is the only command that migrates a mode —
-   * `init` installs, and has no old state to move.
-   *
-   * The count line is the same one `copyNewLocalSkills` prints, and names no destination:
-   * the migration splits its copies between the project and $HOME by each skill's own scope,
-   * so one directory would misname the other half.
+   * Mirrors `BaseCommand.reportPluginInstalls` for the plugin→eject half. The count line names no
+   * destination: copies split between the project and $HOME by each skill's scope.
    */
   reportEjectCopies(result) {
     for (const item of result.failed) {
@@ -1003,14 +852,12 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
     }
     this.log(chalk.hex(CLI_COLORS.NEUTRAL)(localSkillsCopied(result.copied.length)));
   }
-  /** Names what this run is switching, in the words `init` describes an install mode with. */
   logModeSwitch(count, modeDescription) {
     this.log(chalk.hex(CLI_COLORS.NEUTRAL)(`Switching ${count} skill(s) to ${modeDescription}`));
   }
   /**
-   * The scope moves this edit made, on disk. A collapsed `[P][G]` pair is not one of them: its
-   * project half is dropped or folded ({@link applyCollapsedPairs}), and moving every collapse is
-   * what wrote a dropped copy over the global install.
+   * A collapsed `[P][G]` pair is not a move: its project half is dropped or folded
+   * ({@link applyCollapsedPairs}).
    */
   async applyScopeChanges(changes, filteredResult, context, editRoot) {
     const scopeChanges = scopeMovesOf(changes);
@@ -1044,16 +891,9 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
     }
   }
   /**
-   * The project's own half of each `[P][G]` pair this edit collapsed. A Local copy `s` folded into
-   * a Local global install moves there, edits included, replacing the global copy every project
-   * reads. Every other half is removed at project scope and nowhere else: its ejected copy deleted,
-   * or its project-scope plugin registration uninstalled.
-   *
-   * The global install's MODE is {@link applyMigrations}' to change, and it changes only when the
-   * collapse carried a new install mode with it — a fold into a plugin installs the plugin there.
-   *
-   * The uninstall is diagnostic, as every plugin uninstall here is: the config no longer names the
-   * registration, so one left behind is untidy rather than wrong.
+   * A Local half `s` folded into a Local global install moves there, replacing the global copy.
+   * Every other half is removed at project scope only. The global install's mode is
+   * {@link applyMigrations}' to change. The uninstall is diagnostic, as every one here is.
    */
   async applyCollapsedPairs(changes, filteredResult, context, editRoot) {
     const [foldedCopies, removedPairs] = partition(
@@ -1076,7 +916,6 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
       editRoot
     );
   }
-  /** The project-scope registrations of collapsed pairs' plugin halves, uninstalled and reported. */
   async uninstallProjectPluginHalves(pluginHalves, marketplace, editRoot) {
     if (pluginHalves.length === 0) return;
     if (marketplace === void 0) {
@@ -1187,10 +1026,6 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
         sourceResult: context.sourceResult,
         projectDir: editRoot.dir,
         agentDefs: agentDefsResult,
-        // A full `cc edit` pass is authoritative over the roster it owns, so deselected entries
-        // are removed rather than union-preserved. `applyAuthority` decides
-        // how much that is from the producer and the directory, and hands the same word to the
-        // merger and to the global config the project branch of the gate commits.
         authoritativeScope: authority
       });
     } catch (error) {
@@ -1250,16 +1085,9 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
     }
   }
   /**
-   * What this host could not carry, once per run and after the last write: the lines `init` and
-   * `compile` print, from {@link hostCompileNotices}, so `edit` cannot say something different
-   * about the tree it just compiled.
-   *
-   * After {@link cleanupStaleAgentFiles}, because the question is about the tree as it now stands:
-   * an edit that removed the project's last gated sub-agent must not count the file it deleted.
-   *
-   * Not on a run that owes work, which ends on that account instead: such a run may have left
-   * nothing here to read, and a notice that threw over it would turn the run's
-   * `COMPLETED_WITH_FAILURES` exit into an `ERROR`.
+   * The host notices `init` and `compile` print ({@link hostCompileNotices}), after
+   * {@link cleanupStaleAgentFiles} so a deleted file is not counted. Skipped on a run that owes
+   * work, whose tree may not be readable.
    */
   async reportWhatThisHostCannotCarry(editRoot) {
     if (this.hasIncompleteWork) return;
@@ -1338,6 +1166,7 @@ var RESTING_TUNING = "defaults";
 function describeTuning(agent) {
   const settings = [
     agent.model === void 0 ? void 0 : `model ${agent.model}`,
+    agent.codexModel === void 0 ? void 0 : `codex model ${agent.codexModel}`,
     agent.effort === void 0 ? void 0 : `effort ${agent.effort}`
   ].filter((setting) => setting !== void 0);
   return settings.length === 0 ? RESTING_TUNING : settings.join(", ");