agents-inc 0.167.0 → 0.170.0

package/dist/commands/edit.js

@@ -1,18 +1,22 @@
 #!/usr/bin/env node
+import {
+  awaitUnderSpinner
+} from "../chunk-NEZ4OAOL.js";
 import {
   publishSeedConfig,
   seedPayloadForInstallation,
   skillsAuthoredHere
-} from "../chunk-Y47EV74W.js";
+} from "../chunk-SZYQISA2.js";
 import {
   RemovalPlanConfirm
-} from "../chunk-TODCQFRB.js";
+} from "../chunk-USVBYQSQ.js";
 import {
   unofferablePlacementsFound
-} from "../chunk-PXWG7KFO.js";
+} from "../chunk-WTWMGH2C.js";
 import {
   arrivalLines,
   arrivalsByScope,
+  joinedStatement,
   notInstalledGlobally,
   openUrl,
   readInstalledGlobal,
@@ -21,87 +25,54 @@ import {
   sameAssignments,
   skillLabel,
   withUnplaceableAssignees
-} from "../chunk-V3I7VQ7N.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";
-import "../chunk-TSIU7DME.js";
-import "../chunk-JOEA52NR.js";
-import {
-  awaitUnderSpinner
-} from "../chunk-VRXX4D6K.js";
-import "../chunk-SXVHSOTH.js";
-import "../chunk-XQHZPINL.js";
-import "../chunk-JQRJQ4SS.js";
-import "../chunk-TTY6DPJN.js";
-import "../chunk-WTJ3AY33.js";
-import "../chunk-WVUDLKIO.js";
+} from "../chunk-6HNXYYVF.js";
 import {
   fetchSeedConfig
-} from "../chunk-25MPBIK3.js";
+} from "../chunk-OYJQUA3D.js";
 import {
   promptConfirm
 } from "../chunk-4AAOOAIN.js";
 import "../chunk-OZXAJNLA.js";
 import {
   hostCompileNotices
-} from "../chunk-ZXKBERQU.js";
+} from "../chunk-F5EBEATU.js";
 import {
-  applyMigratedGlobalSources,
   compileAgentsAllScopes,
   compileProjectScope,
   detectProject,
   loadSource,
   movedGlobalAgentInputs,
-  mutateGlobal,
   sayCapturedWarnings,
   writeProjectConfig
-} from "../chunk-G6UREQ4V.js";
+} from "../chunk-7RDMTR64.js";
 import {
   copyLocalSkills,
   discoverInstalledSkills,
   removeCompiledAgents,
   uninstallPluginSkills
-} from "../chunk-HWEKHVLE.js";
+} from "../chunk-F2BHT5C4.js";
 import {
   loadAgentDefs
-} from "../chunk-PMJWD27C.js";
-import "../chunk-ZR4YPKXQ.js";
-import "../chunk-GE3WKJ6Q.js";
-import "../chunk-MAWNVT2C.js";
-import "../chunk-JFLVEFFP.js";
+} from "../chunk-HED2DAXZ.js";
+import "../chunk-3G7X2WKI.js";
+import "../chunk-BMRUNMPV.js";
 import {
   formatScopeTag
-} from "../chunk-7JNJ52CD.js";
+} from "../chunk-NIHUOWE4.js";
+import "../chunk-JFLVEFFP.js";
 import {
   providerFlag,
   providerNamedBy,
   refuseAnAmbiguousInstallation
-} from "../chunk-NBXXFIIW.js";
+} from "../chunk-RIPSGNLU.js";
 import {
   BaseCommand,
   EXIT_CODES,
   carriedSkillsThatDiffer,
   registerExternalSkills,
   writeExternalSkills
-} from "../chunk-Y3VIRVVE.js";
-import "../chunk-VAPTVR3J.js";
-import "../chunk-RZDXDMT6.js";
+} from "../chunk-V3KTSP3Q.js";
+import "../chunk-CLPUWOSA.js";
 import {
   ERROR_MESSAGES,
   INCOMPLETE_WORK_RECOVERY,
@@ -116,53 +87,45 @@ import {
   carriedSkillsWritten,
   deleteLocalSkill,
   detectMigrations,
-  discoverAllPluginSkills,
+  editorSkillMetadataUnusableError,
   ejectCopyFailureError,
   executeMigration,
-  foldLocalSkillIntoGlobal,
   getErrorMessage,
   holdsItsOwnInstallation,
   hostAt,
   isHomeDirectory,
   keptAsInstalledGlobally,
   keptUnassigned,
+  loadProjectConfig,
   localSkillsCopied,
   migrateLocalSkillScope,
   nothingInstalledToApplyTo,
-  plural,
   projectCopyRemoved,
   providerInUse,
   recompileSummary,
   resolveInstallPaths,
-  seedToWizardResult,
+  seedToInstallSelection,
   sharedConfigDestinations,
   sharedConfigNeedsTerminal,
   skippedUnknownAgents,
   skippedUnknownSkills,
-  unplaceableKept,
-  unresolvedSkillRemovalReasons,
-  wizardNeedsTerminal
-} from "../chunk-A4KDXNOT.js";
+  unplaceableKept
+} from "../chunk-MQ6CGZNJ.js";
 import {
   getSkillById,
-  getSkillDisplayName,
-  matrix,
-  typedKeys
-} from "../chunk-V5WXIM2F.js";
+  getSkillDisplayName
+} from "../chunk-52HDHJ7I.js";
 import {
   CLI_COLORS,
   CLI_INVOKE_COMMAND,
-  EDIT_PROJECT_SETUP_FLAG,
-  EDIT_PROJECT_SETUP_MARKETPLACE_FLAG,
   EJECT_SOURCE,
-  activeAgentNames,
   activeAgentScopeMap,
   editorConfigUrl,
   formatSourceDisplayName,
   isActiveAt,
   isGlobalTombstone
-} from "../chunk-ENGYJCTF.js";
-import "../chunk-OI2H6M5I.js";
+} from "../chunk-PJWGTNOX.js";
+import "../chunk-NQW25DCF.js";
 
 // src/cli/commands/edit.tsx
 import path from "path";
@@ -200,13 +163,9 @@ function scopeArrow(change) {
 function formatValueChangeLine(subject, change, kind) {
   return chalk.hex(CLI_COLORS.WARNING)(`  ~ ${subject}`) + chalk.hex(CLI_COLORS.NEUTRAL)(` (${kind}: ${change.from} \u2192 ${change.to})`);
 }
-function loadedSourceLabel(sourceResult) {
-  return sourceResult.marketplace ?? sourceResult.sourceConfig.source;
-}
-function resolveEditRoot(installation, cwd, setupRequested) {
-  const dir = setupRequested ? cwd : installation.projectDir;
-  const isGlobal = isHomeDirectory(dir);
-  return { dir, isGlobal, isProjectSetup: setupRequested && !isGlobal };
+function resolveEditRoot(installation) {
+  const dir = installation.projectDir;
+  return { dir, isGlobal: isHomeDirectory(dir) };
 }
 function sharedConfigPlan(changes, kept) {
   const dropped = droppedProjectHalves(changes);
@@ -228,12 +187,11 @@ function removalSections(skills, agents) {
   ].filter((section) => section.items.length > 0);
 }
 function droppedProjectHalves(changes) {
-  const droppedSkills = changes.collapsedPairs.filter((pair) => !pair.folded);
   const droppedAgents = [...changes.agentScopeChanges].filter(
     ([name, change]) => change.to === "global" && changes.dualScopeAgentTransitions.has(name)
   );
   return {
-    skillIds: droppedSkills.map(({ projectHalf }) => projectHalf.id),
+    skillIds: changes.collapsedPairs.map(({ projectHalf }) => projectHalf.id),
     agentNames: droppedAgents.map(([name]) => name)
   };
 }
@@ -241,13 +199,11 @@ function planHeading(sections) {
   if (sections.length === 0) return SHARED_CONFIG_APPLY.NOTHING_REMOVED;
   return SHARED_CONFIG_APPLY.PREVIEW_HEADING;
 }
-function compilesGlobalScope(producer, editRoot, written) {
-  if (editRoot.isGlobal || producer === "wizard") return true;
-  return movedGlobalAgentInputs(written);
+function compilesGlobalScope(editRoot, written) {
+  return editRoot.isGlobal || movedGlobalAgentInputs(written);
 }
 function withReassignments(changes, result, installedStack, editRoot) {
   const { assignedStack } = result;
-  if (assignedStack === void 0) return { ...changes, reassignedAgents: [] };
   const ownedHere = (agent) => editRoot.isGlobal || isActiveAt(agent, "project");
   const countedAsAdded = (agent) => changes.addedAgents.includes(agent.name);
   const rowsMoved = (agent) => !sameAssignments(assignedStack[agent.name], installedStack?.[agent.name]);
@@ -303,51 +259,34 @@ function applyAuthority(editRoot) {
   return editRoot.isGlobal ? "all" : "owned";
 }
 var Edit = class _Edit extends BaseCommand {
-  static summary = "Edit skills in the plugin";
-  static description = "Modify the currently installed skills via interactive wizard";
+  static summary = "Open this installation in the editor, or apply a configuration with --from";
+  static description = "Open this installation in agentsinc.sh/editor, or apply a configuration built there with --from";
   static examples = [
     {
-      description: "Open the edit wizard",
+      description: "Open this installation in the editor",
       command: "<%= config.bin %> <%= command.id %>"
     },
-    {
-      description: "Open this installation in the editor instead of the wizard",
-      command: "<%= config.bin %> <%= command.id %> --ui"
-    },
     {
       description: "Apply a configuration built in the editor, by its id",
       command: "<%= config.bin %> <%= command.id %> --from <id>"
+    },
+    {
+      description: "Look at a shared configuration without applying it",
+      command: "<%= config.bin %> <%= command.id %> --ui --from <id>"
     }
   ];
   static flags = {
     ui: Flags.boolean({
-      description: "Open agentsinc.sh/editor instead of the wizard \u2014 the id --from names, or this installation",
+      description: "Open agentsinc.sh/editor on this installation, or on the id --from names without applying it \u2014 what a bare edit does",
       default: false
     }),
     from: Flags.string({
       description: "Apply a configuration shared from agentsinc.sh by its id, removing whatever it leaves out \u2014 from a project, only the project's own entries",
       helpValue: "<id>"
     }),
-    [EDIT_PROJECT_SETUP_FLAG]: Flags.boolean({
-      description: "Internal: this run continues an `init` project setup",
-      default: false,
-      hidden: true
-    }),
-    [EDIT_PROJECT_SETUP_MARKETPLACE_FLAG]: Flags.string({
-      description: "Internal: the marketplace `init --marketplace` named for that project setup",
-      hidden: true,
-      dependsOn: [EDIT_PROJECT_SETUP_FLAG]
-    }),
     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();
@@ -364,17 +303,16 @@ var Edit = class _Edit extends BaseCommand {
     );
     await this.ensureConfigReadable(cwd);
     await this.refuseUnofferablePlacements(cwd);
-    if (flags.ui) return this.openInEditor(cwd);
-    const payload = flags.from === void 0 ? null : await this.fetchSharedConfigOrFail(flags.from);
-    const context = await this.loadContextUnderSpinner(flags[EDIT_PROJECT_SETUP_MARKETPLACE_FLAG]);
-    const editRoot = resolveEditRoot(context.installation, cwd, flags[EDIT_PROJECT_SETUP_FLAG]);
+    if (flags.from === void 0) return this.openInEditor(cwd);
+    const payload = await this.fetchSharedConfigOrFail(flags.from);
+    const context = await this.loadContextUnderSpinner();
+    const editRoot = resolveEditRoot(context.installation);
     await this.ensureSavedSkillsReadable(
       context.projectConfig?.skills ?? [],
       context.sourceResult.matrix,
-      context.projectDir
+      editRoot.dir
     );
-    const selection = payload ? await this.selectionFromSharedConfig(payload, context, editRoot) : await this.selectionFromWizard(context, editRoot);
-    if (!selection) this.error("Cancelled", { exit: EXIT_CODES.CANCELLED });
+    const selection = await this.selectionFromSharedConfig(payload, context, editRoot);
     const { result } = selection;
     this.reportValidationErrors(result.validation);
     const activeNewSkills = result.skills.filter((s) => !s.excluded);
@@ -387,18 +325,21 @@ var Edit = class _Edit extends BaseCommand {
       agentConfigs: activeNewAgents
     };
     const filteredOldConfig = context.projectConfig ? { ...context.projectConfig, skills: activeOldSkills, agents: activeOldAgents } : null;
-    const planned = withReassignments(
-      detectConfigChanges(filteredOldConfig, filteredResult, {
-        newSkills: result.skills,
-        oldSkills: context.projectConfig?.skills ?? [],
-        newAgents: result.agentConfigs,
-        oldAgents: context.projectConfig?.agents ?? []
-      }),
-      filteredResult,
-      context.projectConfig?.stack,
-      editRoot
-    );
-    const rewritten = selection.producer === "shared" ? await this.confirmAndWriteCarried(selection, planned, filteredResult) : [];
+    const planned = {
+      ...withReassignments(
+        detectConfigChanges(filteredOldConfig, filteredResult, {
+          newSkills: result.skills,
+          oldSkills: context.projectConfig?.skills ?? [],
+          newAgents: result.agentConfigs,
+          oldAgents: context.projectConfig?.agents ?? []
+        }),
+        filteredResult,
+        context.projectConfig?.stack,
+        editRoot
+      ),
+      joinedGlobalAgents: selection.joinedGlobalAgents
+    };
+    const rewritten = await this.confirmAndWriteCarried(selection, planned, filteredResult);
     const changes = {
       ...planned,
       rewrittenSkills: difference(rewritten, planned.addedSkills)
@@ -406,24 +347,9 @@ var Edit = class _Edit extends BaseCommand {
     const authority = applyAuthority(editRoot);
     if (!hasAnyChanges(changes)) {
       this.log(chalk.hex(CLI_COLORS.NEUTRAL)("No changes made."));
-      if (!editRoot.isProjectSetup) return;
-      await this.writeConfigAndCompile(result, context, editRoot, authority, selection.producer);
-      await this.reportWhatThisHostCannotCarry(editRoot);
-      this.logCompletionSummary(changes);
       return;
     }
-    const removalReasons = await unresolvedSkillRemovalReasons(
-      result.unresolvableSkillIds,
-      activeOldSkills,
-      context.projectDir,
-      loadedSourceLabel(context.sourceResult)
-    );
-    this.logChangeSummary(
-      changes,
-      filteredResult.skills,
-      filteredOldConfig?.skills ?? [],
-      removalReasons
-    );
+    this.logChangeSummary(changes, filteredResult.skills, filteredOldConfig?.skills ?? []);
     const migratedSkillIds = await this.applyMigrations(
       changes,
       filteredResult,
@@ -431,49 +357,22 @@ var Edit = class _Edit extends BaseCommand {
       context,
       editRoot
     );
-    await this.recordGlobalSourceMigrations(
-      migratedSkillIds,
-      filteredResult.skills,
-      editRoot,
-      context
-    );
     await this.applyScopeChanges(changes, filteredResult, context, editRoot);
-    await this.applyCollapsedPairs(changes, filteredResult, context, editRoot);
+    await this.applyCollapsedPairs(changes, context, editRoot);
     await this.applySourceChanges(changes, activeOldSkills, editRoot, migratedSkillIds);
     await this.applyPluginChanges(changes, filteredResult, activeOldSkills, context, editRoot);
     await this.copyNewLocalSkills(changes, filteredResult, context, editRoot);
     await this.removeDeletedLocalSkills(changes, activeOldSkills, editRoot);
-    await this.writeConfigAndCompile(result, context, editRoot, authority, selection.producer);
+    await this.writeConfigAndCompile(result, context, editRoot, authority);
     await this.cleanupStaleAgentFiles(changes, activeOldAgents, editRoot);
     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 });
-    }
-    const result = await this.runEditWizard(context, editRoot);
-    if (!result) return null;
-    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;
@@ -507,25 +406,18 @@ var Edit = class _Edit extends BaseCommand {
     if (skippedSkillIds.length > 0) this.warn(skippedUnknownSkills(skippedSkillIds));
     if (skippedAgentNames.length > 0) this.warn(skippedUnknownAgents(skippedAgentNames));
     return {
-      producer: "shared",
       result: held.result,
       kept: [
         ...keptStatements(reconciled.kept),
         ...unassignedStatement(reconciled.unassigned),
+        ...joinedStatement(held),
         ...keptAsInstalledStatement(held, revisedGlobally)
       ],
-      carried: arrivingCarried
+      carried: arrivingCarried,
+      joinedGlobalAgents: held.joined.map(({ agentName }) => agentName)
     };
   }
-  /**
-   * 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 +427,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,25 +442,13 @@ 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);
+      return seedToInstallSelection(payload, sourceMatrix);
     } catch (error) {
       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 +456,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 +467,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,31 +490,17 @@ 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) {
+    await this.ensureInstalledSkillsReadable(projectDir);
     const prepared = await seedPayloadForInstallation(projectDir);
     if (!prepared.ok) {
       this.error(prepared.error, { exit: EXIT_CODES.ERROR });
@@ -670,16 +513,20 @@ 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.
+   * The payload reads every ejected skill's metadata.yaml for its provenance, and no catalogue is
+   * loaded here to resolve a skill around it, so every installed entry is asked about — not only
+   * those a catalogue lacks, as `edit --from` asks. The config read is the one the payload reads.
    */
+  async ensureInstalledSkillsReadable(projectDir) {
+    const loaded = await loadProjectConfig(projectDir);
+    if (!loaded) return;
+    await this.refuseUnusableSavedSkillMetadata(
+      loaded.config.skills,
+      projectDir,
+      editorSkillMetadataUnusableError
+    );
+  }
+  /** 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 +535,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,21 +545,10 @@ 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)
-    );
+  async loadContextUnderSpinner() {
+    return awaitUnderSpinner(STATUS_MESSAGES.LOADING_SKILLS, () => this.loadContext());
   }
-  /**
-   * @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) {
+  async loadContext() {
     const detected = await detectProject();
     if (!detected) {
       this.error(ERROR_MESSAGES.NO_INSTALLATION, {
@@ -731,147 +556,17 @@ var Edit = class _Edit extends BaseCommand {
       });
     }
     const { installation, config: projectConfig } = detected;
-    const projectDir = installation.projectDir;
-    let sourceResult;
-    let startupMessages = [];
     try {
-      const loaded = await loadSource({
-        projectDir,
-        ...setupMarketplace !== void 0 && { sourceFlag: setupMarketplace },
+      const { sourceResult, startupMessages } = await loadSource({
+        projectDir: installation.projectDir,
         captureStartupMessages: true
       });
-      sourceResult = loaded.sourceResult;
-      startupMessages = loaded.startupMessages;
-      const sourceInfo = sourceResult.isLocal ? "local" : sourceResult.sourceConfig.sourceOrigin;
-      startupMessages.push({
-        level: "info",
-        text: `Loaded ${plural(Object.keys(matrix.skills).length, "skill")} (${sourceInfo})`
-      });
-    } catch (error) {
-      this.handleError(error);
-    }
-    let currentSkillIds;
-    try {
-      const discoveredSkills = await discoverAllPluginSkills(projectDir);
-      const pluginSkillIds = typedKeys(discoveredSkills);
-      const excludedConfigIds = new Set(
-        projectConfig?.skills.filter((s) => s.excluded).map((s) => s.id) ?? []
-      );
-      const configSkillIds = projectConfig?.skills.filter((s) => !s.excluded).map((s) => s.id) ?? [];
-      const filteredPluginSkillIds = pluginSkillIds.filter((id) => !excludedConfigIds.has(id));
-      const mergedIds = /* @__PURE__ */ new Set([...filteredPluginSkillIds, ...configSkillIds]);
-      currentSkillIds = [...mergedIds];
-      startupMessages.push({
-        level: "info",
-        text: `Found ${plural(currentSkillIds.length, "installed skill")}`
-      });
+      return { installation, projectConfig, sourceResult, startupMessages };
     } catch (error) {
       this.handleError(error);
     }
-    return {
-      installation,
-      projectConfig,
-      projectDir,
-      sourceResult,
-      startupMessages,
-      currentSkillIds,
-      ...setupMarketplace !== void 0 && sourceResult.marketplace !== void 0 && { setupMarketplace: sourceResult.marketplace }
-    };
-  }
-  async runEditWizard(context, editRoot) {
-    const { projectConfig, currentSkillIds } = context;
-    const selectedAgents = projectConfig?.agents && activeAgentNames(projectConfig.agents);
-    return runWizardSession({
-      hydrate: {
-        initialStep: "build",
-        ...projectConfig?.selectedDomains !== void 0 && {
-          initialDomains: projectConfig.selectedDomains
-        },
-        ...selectedAgents !== void 0 && { initialAgents: selectedAgents },
-        installedSkillIds: currentSkillIds,
-        ...projectConfig?.skills !== void 0 && { installedSkillConfigs: projectConfig.skills },
-        ...projectConfig?.agents !== void 0 && { installedAgentConfigs: projectConfig.agents },
-        isEditingFromGlobalScope: editRoot.isGlobal,
-        ...context.setupMarketplace !== void 0 && {
-          setupMarketplace: context.setupMarketplace
-        }
-      },
-      props: {
-        version: this.config.version,
-        initialAgents: selectedAgents,
-        installedSkillIds: currentSkillIds,
-        startupMessages: context.startupMessages
-      },
-      onCancel: () => this.log("\nEdit cancelled"),
-      clearTerminal: () => this.clearTerminal()
-    });
-  }
-  /**
-   * 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.
-   */
-  async recordGlobalSourceMigrations(migratedSkillIds, newSkills, editRoot, context) {
-    if (editRoot.isGlobal) return;
-    const migratedSources = new Map(
-      newSkills.filter((s) => migratedSkillIds.has(s.id) && isActiveAt(s, "global")).map((s) => [s.id, s.origin])
-    );
-    if (migratedSources.size === 0) return;
-    try {
-      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.
-          // eslint-disable-next-line @typescript-eslint/require-await -- deps contract
-          loadMatrix: async () => context.sourceResult.matrix,
-          loadAgents: async () => (await loadAgentDefs()).agents
-        }
-      );
-      this.reportFanOut(report);
-    } catch (error) {
-      this.reportIncompleteWork(
-        `Could not record global origin change: ${getErrorMessage(error)}`,
-        INCOMPLETE_WORK_RECOVERY.INSPECT_INSTALLATION
-      );
-    }
   }
-  logChangeSummary(changes, newSkills, oldSkills, removalReasons) {
+  logChangeSummary(changes, newSkills, oldSkills) {
     const {
       addedSkills,
       removedSkills,
@@ -884,6 +579,7 @@ var Edit = class _Edit extends BaseCommand {
       dualScopeAgentTransitions,
       tuningChanges,
       reassignedAgents,
+      joinedGlobalAgents,
       rewrittenSkills
     } = changes;
     this.log(`
@@ -898,10 +594,7 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
     for (const skillId of removedSkills) {
       const scope = oldSkills.find((s) => s.id === skillId)?.scope;
       const scopeLabel = scope ? ` ${formatScopeTag(scope)}` : "";
-      const reason = removalReasons.get(skillId);
-      this.log(
-        chalk.hex(CLI_COLORS.ERROR)(`  - ${getSkillDisplayName(skillId)}${scopeLabel}`) + (reason ? chalk.hex(CLI_COLORS.NEUTRAL)(` (${reason})`) : "")
-      );
+      this.log(chalk.hex(CLI_COLORS.ERROR)(`  - ${getSkillDisplayName(skillId)}${scopeLabel}`));
     }
     for (const agentName of addedAgents) {
       this.log(
@@ -935,6 +628,11 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
         chalk.hex(CLI_COLORS.WARNING)(`  ~ ${agentName}`) + chalk.hex(CLI_COLORS.NEUTRAL)(" (agent: skills reassigned)")
       );
     }
+    for (const agentName of joinedGlobalAgents) {
+      this.log(
+        chalk.hex(CLI_COLORS.WARNING)(`  ~ ${agentName}`) + chalk.hex(CLI_COLORS.NEUTRAL)(` ${formatScopeTag("global")} (agent: skills added)`)
+      );
+    }
     for (const skillId of rewrittenSkills) {
       this.log(
         chalk.hex(CLI_COLORS.WARNING)(`  ~ ${getSkillDisplayName(skillId)}`) + chalk.hex(CLI_COLORS.NEUTRAL)(" (skill: files rewritten)")
@@ -981,18 +679,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 +691,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
+   * ({@link applyCollapsedPairs}).
    */
   async applyScopeChanges(changes, filteredResult, context, editRoot) {
     const scopeChanges = scopeMovesOf(changes);
@@ -1044,29 +730,14 @@ ${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.
+   * Each project 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(
-      changes.collapsedPairs,
-      (pair) => foldsItsCopyIntoGlobal(pair, filteredResult.skills)
-    );
+  async applyCollapsedPairs(changes, context, editRoot) {
     const [ejectedHalves, pluginHalves] = partition(
-      removedPairs.map(({ projectHalf }) => projectHalf),
+      changes.collapsedPairs.map(({ projectHalf }) => projectHalf),
       (half) => half.origin === EJECT_SOURCE
     );
-    for (const { projectHalf } of foldedCopies) {
-      await foldLocalSkillIntoGlobal(editRoot.dir, projectHalf.id);
-    }
     for (const half of ejectedHalves) {
       await deleteLocalSkill(editRoot.dir, half.id, half.scope);
     }
@@ -1076,7 +747,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) {
@@ -1173,7 +843,7 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
       await deleteLocalSkill(editRoot.dir, skillId, oldSkill.scope);
     }
   }
-  async writeConfigAndCompile(result, context, editRoot, authority, producer) {
+  async writeConfigAndCompile(result, context, editRoot, authority) {
     let agentDefsResult;
     try {
       agentDefsResult = await loadAgentDefs();
@@ -1183,14 +853,10 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
     let configResult;
     try {
       configResult = await writeProjectConfig({
-        wizardResult: result,
+        selection: result,
         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) {
@@ -1202,7 +868,7 @@ ${chalk.hex(CLI_COLORS.WHITE).bold("Changes:")}`);
     try {
       const agentScopeMap = activeAgentScopeMap(result.agentConfigs);
       const { allSkills } = await discoverInstalledSkills(editRoot.dir);
-      const compile = compilesGlobalScope(producer, editRoot, configResult.propagation) ? compileAgentsAllScopes : compileProjectScope;
+      const compile = compilesGlobalScope(editRoot, configResult.propagation) ? compileAgentsAllScopes : compileProjectScope;
       const compilationResult = await compile({
         projectDir: editRoot.dir,
         sourcePath: agentDefsResult.sourcePath,
@@ -1250,16 +916,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;
@@ -1274,21 +933,21 @@ ${chalk.hex(CLI_COLORS.SUCCESS)("\u2713 Done")}
 `);
   }
 };
-function detectConfigChanges(oldConfig, wizardResult, fullEntries) {
+function detectConfigChanges(oldConfig, selection, fullEntries) {
   const oldSkillIds = oldConfig?.skills.map((s) => s.id) ?? [];
-  const newSkillIds = wizardResult.skills.map((s) => s.id);
+  const newSkillIds = selection.skills.map((s) => s.id);
   const oldAgentNames = oldConfig?.agents.map((a) => a.name) ?? [];
-  const newAgentNames = wizardResult.agentConfigs.map((a) => a.name);
+  const newAgentNames = selection.agentConfigs.map((a) => a.name);
   const oldSkillsById = indexBy(oldConfig?.skills ?? [], (s) => s.id);
   const oldAgentsByName = indexBy(oldConfig?.agents ?? [], (a) => a.name);
   const scopeChanges = detectPropertyChanges(
-    wizardResult.skills,
+    selection.skills,
     oldSkillsById,
     (s) => s.id,
     (s) => s.scope
   );
   const agentScopeChanges = detectPropertyChanges(
-    wizardResult.agentConfigs,
+    selection.agentConfigs,
     oldAgentsByName,
     (a) => a.name,
     (a) => a.scope
@@ -1302,8 +961,7 @@ function detectConfigChanges(oldConfig, wizardResult, fullEntries) {
   const collapsedPairs = collapsedPairsOf(
     scopeChanges,
     dualScopeSkillTransitions,
-    fullEntries?.oldSkills ?? [],
-    wizardResult.foldedSkillIds ?? []
+    fullEntries?.oldSkills ?? []
   );
   return {
     addedSkills: difference(newSkillIds, oldSkillIds),
@@ -1311,7 +969,7 @@ function detectConfigChanges(oldConfig, wizardResult, fullEntries) {
     addedAgents: difference(newAgentNames, oldAgentNames),
     removedAgents: difference(oldAgentNames, newAgentNames),
     sourceChanges: detectPropertyChanges(
-      wizardResult.skills,
+      selection.skills,
       indexBy(withMaskedGlobalsRestored(oldConfig?.skills ?? [], collapsedPairs), (s) => s.id),
       (s) => s.id,
       (s) => s.origin
@@ -1327,7 +985,7 @@ function detectConfigChanges(oldConfig, wizardResult, fullEntries) {
       (a) => a.name
     ),
     tuningChanges: detectPropertyChanges(
-      wizardResult.agentConfigs,
+      selection.agentConfigs,
       oldAgentsByName,
       (a) => a.name,
       describeTuning
@@ -1338,6 +996,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(", ");
@@ -1365,20 +1024,12 @@ function detectDualScopeTransitions(scopeChanges, fullNew, fullOld, getKey) {
   }
   return result;
 }
-function collapsedPairsOf(scopeChanges, dualScopeSkillTransitions, fullOldSkills, foldedSkillIds) {
+function collapsedPairsOf(scopeChanges, dualScopeSkillTransitions, fullOldSkills) {
   return [...scopeChanges].filter(([id, change]) => change.to === "global" && dualScopeSkillTransitions.has(id)).map(([id]) => ({
     projectHalf: oldHalfOf(fullOldSkills, id, (skill) => isActiveAt(skill, "project")),
-    maskedGlobal: oldHalfOf(fullOldSkills, id, isGlobalTombstone),
-    folded: foldedSkillIds.includes(id)
+    maskedGlobal: oldHalfOf(fullOldSkills, id, isGlobalTombstone)
   }));
 }
-function foldsItsCopyIntoGlobal(pair, newSkills) {
-  const { projectHalf, folded } = pair;
-  const globalEntry = newSkills.find(
-    (skill) => skill.id === projectHalf.id && isActiveAt(skill, "global")
-  );
-  return folded && projectHalf.origin === EJECT_SOURCE && globalEntry?.origin === EJECT_SOURCE;
-}
 function oldHalfOf(fullOldSkills, id, isHalf) {
   const half = fullOldSkills.find((skill) => skill.id === id && isHalf(skill));
   if (half === void 0) {
@@ -1424,7 +1075,7 @@ function planStaleAgentRemovals(changes, oldAgents, editRoot) {
   return STALE_AGENT_SCOPES.map(removalAtScope).filter((removal) => removal.agents.length > 0);
 }
 function hasAnyChanges(changes) {
-  return changes.reassignedAgents.length > 0 || changes.rewrittenSkills.length > 0 || changes.addedSkills.length > 0 || changes.removedSkills.length > 0 || changes.addedAgents.length > 0 || changes.removedAgents.length > 0 || changes.sourceChanges.size > 0 || changes.scopeChanges.size > 0 || changes.agentScopeChanges.size > 0 || changes.tuningChanges.size > 0;
+  return changes.reassignedAgents.length > 0 || changes.joinedGlobalAgents.length > 0 || changes.rewrittenSkills.length > 0 || changes.addedSkills.length > 0 || changes.removedSkills.length > 0 || changes.addedAgents.length > 0 || changes.removedAgents.length > 0 || changes.sourceChanges.size > 0 || changes.scopeChanges.size > 0 || changes.agentScopeChanges.size > 0 || changes.tuningChanges.size > 0;
 }
 async function migratePluginSkillScopes(scopeChanges, skills, marketplace, projectDir) {
   const host = hostAt(projectDir);
@@ -1449,7 +1100,6 @@ async function migratePluginSkillScopes(scopeChanges, skills, marketplace, proje
   return { migrated, failed };
 }
 export {
-  applyMigratedGlobalSources,
   Edit as default,
   detectConfigChanges,
   migratePluginSkillScopes