agents-inc 0.168.0 → 0.170.0

package/dist/{chunk-C2UISKIM.js → chunk-MQ6CGZNJ.js}

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 import {
   BUILT_IN_MATRIX,
-  allSkills,
   getSkillById,
   initializeMatrix,
   matrix,
@@ -9,7 +8,7 @@ import {
   typedFromEntries,
   typedKeys,
   typedValues
-} from "./chunk-NU2K5G54.js";
+} from "./chunk-52HDHJ7I.js";
 import {
   AGENT_NAMES,
   AUTHOR_HANDLE_PATTERN,
@@ -65,7 +64,6 @@ import {
   bytewise,
   cacheRoot,
   categoryDomain,
-  createSelectionSemantics,
   diagnostics,
   editorConfigUrl,
   effectivelyExcludedSkillIds,
@@ -85,12 +83,12 @@ import {
   seedAgentScope,
   sourceDirName,
   validateSelection
-} from "./chunk-AG3ZVM3B.js";
+} from "./chunk-PJWGTNOX.js";
 import {
   defaultCategories,
   defaultRules,
   defaultStacks
-} from "./chunk-OI2H6M5I.js";
+} from "./chunk-NQW25DCF.js";
 
 // src/cli/lib/loading/source-loader.ts
 import os14 from "os";
@@ -871,6 +869,7 @@ async function grantOwnerWrite(target) {
 }
 
 // src/cli/utils/logger.ts
+import { AsyncLocalStorage as AsyncLocalStorage2 } from "async_hooks";
 var verboseMode = false;
 function setVerbose(enabled) {
   verboseMode = enabled;
@@ -885,6 +884,8 @@ function log(msg) {
 }
 var bufferMode = false;
 var messageBuffer = [];
+var alreadySaid = /* @__PURE__ */ new Set();
+var eachWarningOnce = new AsyncLocalStorage2();
 function enableBuffering() {
   bufferMode = true;
   messageBuffer = [];
@@ -899,17 +900,23 @@ function disableBuffering() {
   messageBuffer = [];
 }
 function warn(msg) {
+  if (isARepeatToHoldBack(msg)) return;
   if (bufferMode) {
     messageBuffer.push({ level: "warn", text: msg });
     return;
   }
+  alreadySaid.add(msg);
   console.warn(`  Warning: ${msg}`);
 }
-var alreadySaid = /* @__PURE__ */ new Set();
+function isARepeatToHoldBack(msg) {
+  if (eachWarningOnce.getStore() !== true) return false;
+  return alreadySaid.has(msg) || messageBuffer.some((message) => message.text === msg);
+}
+function sayingEachWarningOnce(body) {
+  return eachWarningOnce.run(true, body);
+}
 function warnOnce(msg) {
-  if (alreadySaid.has(msg)) return;
-  alreadySaid.add(msg);
-  warn(msg);
+  sayingEachWarningOnce(() => warn(msg));
 }
 
 // src/cli/lib/validate-kebab-name.ts
@@ -940,7 +947,7 @@ var ERROR_MESSAGES = {
    * What `compile` refuses with when every pass discovered zero skills. It is reached only
    * after an installation was detected, so the state it describes is a configuration with
    * nothing installed under it — the same state `doctor` reports as `config-empty` and
-   * names `init` for, because `init` on a config that declares nothing opens the wizard
+   * names `init` for, because `init` on a config that declares nothing opens the editor
    * rather than the dashboard. `edit` is the wrong half of the pair here: it modifies the
    * currently installed skills, and there are none to modify.
    */
@@ -1050,7 +1057,7 @@ function scopeBlockedStackAssignment(agentNames, skillId) {
 }
 function globalScopedAgentsHint(count) {
   const [subject, object] = count === 1 ? ["agent is", "it"] : ["agents are", "them"];
-  return `${count} ${subject} global-scoped \u2014 run '${CLI_INVOKE_COMMAND} compile' from your home directory, or edit from this project, to recompile ${object}.`;
+  return `${count} ${subject} global-scoped \u2014 run '${CLI_INVOKE_COMMAND} compile' from your home directory to recompile ${object}.`;
 }
 function marketplacesRefreshed(count) {
   return `Refreshed ${plural(count, "marketplace listing")}. Installed plugins were not changed.`;
@@ -1157,6 +1164,13 @@ function savedSkillMetadataUnusableError(entries) {
     METADATA_UNUSABLE_WAY_OUT
   ].join("\n");
 }
+function editorSkillMetadataUnusableError(entries) {
+  return [
+    metadataUnusableOpening(entries),
+    `The editor opens on what these files describe, so no link is made until each one describes its skill.`,
+    METADATA_UNUSABLE_WAY_OUT
+  ].join("\n");
+}
 var SHARED_CONFIG_GREENFIELD_HINT = "installing a shared configuration is a fresh setup, not a merge";
 function sharedConfigDestinations(id) {
   return [
@@ -1222,27 +1236,14 @@ function sharedConfigNeedsTerminal(id) {
 function nothingInstalledToApplyTo(id) {
   return `No installation found. Run '${CLI_INVOKE_COMMAND} init --from ${id}' to install configuration '${id}' here.`;
 }
-var RUNS_WITHOUT_A_TERMINAL = {
-  init: `'${CLI_INVOKE_COMMAND} init --from <id>' installs a configuration built in the editor without one`,
-  edit: `'${CLI_INVOKE_COMMAND} edit --ui' opens this installation in the editor without one`
-};
-function wizardNeedsTerminal(command) {
-  return [
-    `'${CLI_INVOKE_COMMAND} ${command}' opens a wizard, and there is no terminal here to open it in.`,
-    `Run it from a terminal \u2014 ${RUNS_WITHOUT_A_TERMINAL[command]}.`
-  ].join("\n");
-}
 function uninstallNeedsTerminal() {
   return [
     "Uninstalling removes files, so it has to be confirmed \u2014 and there is no terminal here to confirm it at.",
     `Run '${CLI_INVOKE_COMMAND} uninstall' from a terminal, or '${CLI_INVOKE_COMMAND} uninstall --yes' to remove it without being asked.`
   ].join("\n");
 }
-function editorLoadsItsOwnMarketplace(named) {
-  return `--marketplace '${named}' cannot be handed to the editor: the editor loads marketplaces itself. Run '${CLI_INVOKE_COMMAND} init --ui' and load it there.`;
-}
-function marketplaceFixedAtInstall(stored, named) {
-  return `This folder's installation was made from the marketplace '${stored}', and an installation's marketplace is chosen once, when it is made \u2014 so --marketplace '${named}' was not used. Run '${CLI_INVOKE_COMMAND} init' without it to open this installation, or '${CLI_INVOKE_COMMAND} uninstall' first to set this folder up from another.`;
+function addMarketplaceInTheEditor(ref, isFolder) {
+  return isFolder ? `${ref} is a folder on this machine, which the editor cannot read \u2014 push it to GitHub, then add it from the editor's Marketplace button.` : `Add ${ref} from the editor's Marketplace button.`;
 }
 var SHARED_CONFIG_ARRIVALS = {
   PROJECT_HEADING: "Into this project:",
@@ -1258,11 +1259,18 @@ function keptAsInstalledGlobally(skillIds, agentNames) {
     `Change them with '${CLI_INVOKE_COMMAND} edit' from your home directory.`
   ].join("\n");
 }
+function addedToGlobalSubAgents(joined) {
+  return [
+    "Added to sub-agents in the global install \u2014 each keeps everything it already has, since a project run only adds to the global install:",
+    ...joined.map(
+      ({ agentName, skillIds }) => `  sub-agent ${agentName}: + ${skillIds.join(", ")}`
+    )
+  ].join("\n");
+}
 function authoredHereKept(skillIds) {
   return [
     "Kept \u2014 written here rather than installed, so a shared configuration never carried them:",
-    ...skillIds.map((id) => `  skill ${id}`),
-    `Remove them with '${CLI_INVOKE_COMMAND} edit'.`
+    ...skillIds.map((id) => `  skill ${id}`)
   ].join("\n");
 }
 function unplaceableKept(skillIds) {
@@ -1275,35 +1283,30 @@ function unplaceableKept(skillIds) {
 function keptUnassigned(skillId, agentName, category) {
   return `${skillId}: kept, no longer assigned to ${agentName}'s ${category}`;
 }
-function otherWaysPastUnreadableConfigs(count) {
-  return [
-    `Or build one at ${EDITOR_URL} and install it with '${CLI_INVOKE_COMMAND} init --from <id>'.`,
-    `'${CLI_INVOKE_COMMAND} doctor' reports the same ${count === 1 ? "file" : "files"}, alongside whatever else is wrong here.`
-  ];
+function doctorReportsTheSame(count) {
+  return `'${CLI_INVOKE_COMMAND} doctor' reports the same ${count === 1 ? "file" : "files"}, alongside whatever else is wrong here.`;
 }
 function installationConfigsUnreadable(failures) {
-  return [
-    ...failures.flatMap(whoseConfigAndItsWayOut),
-    ...otherWaysPastUnreadableConfigs(failures.length)
-  ].join("\n");
+  return [...failures.flatMap(whoseConfigAndItsWayOut), doctorReportsTheSame(failures.length)].join(
+    "\n"
+  );
 }
 function whoseConfigAndItsWayOut(failure) {
   return [
     `${scopeLabel(failure.scope)}'s config at '${failure.configPath}' could not be loaded: ${failure.reason}`,
-    `There is no automatic repair for this \u2014 recreate the configuration by running ${recreateConfigFrom(failure.scopeRoot)}.`
+    `There is no automatic repair for this \u2014 fix the file, or recreate the configuration by running ${recreateConfigFrom(failure.scopeRoot)}.`
   ];
 }
 function uninstallConfigUnreadable(failure) {
   return `Could not read the ${failure.scope} config \u2014 plugins and compiled agents it lists may be left behind: ${failure.message}`;
 }
 function recreateConfigFrom(scopeRoot) {
-  return `'${CLI_INVOKE_COMMAND} uninstall', which still works on a config it cannot read, then '${CLI_INVOKE_COMMAND} init', from '${scopeRoot}'`;
+  return `'${CLI_INVOKE_COMMAND} uninstall' from '${scopeRoot}', which still works on a config it cannot read, then '${CLI_INVOKE_COMMAND} init --from <id>' with an id from ${EDITOR_URL}`;
 }
 function configUnreadableError(configLoadFailure2) {
   return [
     configLoadFailure2,
-    `There is no automatic repair for this \u2014 recreate the configuration: '${CLI_INVOKE_COMMAND} uninstall' still works on a config it cannot read, then '${CLI_INVOKE_COMMAND} init'.`,
-    ...otherWaysPastUnreadableConfigs(1)
+    "This is the marketplace's own config rather than an installation's, so it is fixed in the marketplace repository it comes from."
   ].join("\n");
 }
 function registeredProjectsUpdateFailed(reason) {
@@ -1927,17 +1930,6 @@ var CATEGORIES = [
   "web-ui-kit",
   "web-utilities"
 ];
-var DOMAINS = [
-  "ai",
-  "api",
-  "cli",
-  "desktop",
-  "infra",
-  "meta",
-  "mobile",
-  "shared",
-  "web"
-];
 var AGENT_NAMES2 = [
   "agent-summoner",
   "ai-developer",
@@ -1963,9 +1955,6 @@ var AGENT_NAMES2 = [
 function isRecord(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-function isDomain(value) {
-  return DOMAINS.includes(value);
-}
 function isAgentName(value) {
   return AGENT_NAMES2.includes(value);
 }
@@ -2944,12 +2933,6 @@ function isLocalSource(source) {
   }
   return true;
 }
-function isSameMarketplace(a, b, fromDir) {
-  if (isLocalSource(a) && isLocalSource(b)) {
-    return path5.resolve(fromDir, a) === path5.resolve(fromDir, b);
-  }
-  return a === b;
-}
 
 // src/cli/lib/compile-seat.ts
 seatDiagnostics({ warn, verbose });
@@ -3915,11 +3898,6 @@ var KNOWN_AGENTS = new Set(AGENT_NAMES);
 function sourceForSkill(skill) {
   return skill.availableSources?.find((s) => s.primary)?.name ?? DEFAULT_PUBLIC_SOURCE_NAME;
 }
-function addToDomainSelections(selections, domain, category, skillId) {
-  const categorySelections = selections[domain] ??= {};
-  const skillList = categorySelections[category] ??= [];
-  if (!skillList.includes(skillId)) skillList.push(skillId);
-}
 function addToAgentStack(stack, agent, category, assignment) {
   const agentStack = stack[agent] ??= {};
   const assignments = agentStack[category] ??= [];
@@ -3968,8 +3946,8 @@ function unwritableAssignmentsError(assignments) {
     "Re-share it with each sub-agent above pinned to the project, or each skill above at global scope."
   ].join("\n");
 }
-function seedToWizardResult(payload, catalog) {
-  const domainSelections = {};
+function seedToInstallSelection(payload, catalog) {
+  const domains = /* @__PURE__ */ new Set();
   const assignedStack = {};
   const skills = [];
   const skippedSkillIds = [];
@@ -3986,7 +3964,7 @@ function seedToWizardResult(payload, catalog) {
     }
     const id = skillId;
     const category = skill.category;
-    addToDomainSelections(domainSelections, domain, category, id);
+    domains.add(domain);
     skills.push({
       id,
       scope: entry.scope,
@@ -4034,11 +4012,7 @@ function seedToWizardResult(payload, catalog) {
       ...payload.description !== void 0 && {
         description: payload.description
       },
-      domainSelections,
-      selectedDomains: orderDomains(typedKeys2(domainSelections)),
-      // The skipped ids came off the wire and are reported directly; no saved entry was removed.
-      unresolvableSkillIds: [],
-      cancelled: false,
+      selectedDomains: orderDomains([...domains]),
       // Revalidated: the payload was consistent against the authoring app's catalog, not this one.
       validation: validateSelection(
         skills.map((skill) => skill.id),
@@ -4458,7 +4432,7 @@ var PROJECT_CONFIG_INTERFACE_AFTER = `export interface ProjectConfig {
   /** Agents source path or URL (when agents come from a different source than skills) */
   agentsSource?: string
 
-  /** Selected domains from the wizard */
+  /** Selected domains */
   selectedDomains?: Domain[]
 
   /** Branding overrides for white-labeling the CLI */
@@ -6128,7 +6102,7 @@ function wereScopeCompatible(priorSkillScope, priorAgentScope) {
   return priorSkillScope !== void 0 && priorAgentScope !== void 0 && isScopePairCompatible(priorSkillScope, priorAgentScope);
 }
 function resolveStackProperty(generated, assigned) {
-  return generated && assigned ? assigned : generated;
+  return generated === void 0 ? void 0 : assigned;
 }
 function resolveDescription(loadedStack, shared) {
   return loadedStack === null ? shared : loadedStack.description;
@@ -6143,22 +6117,22 @@ async function loadCuratedStack(projectDir, projectStack) {
 async function loadGlobalStack() {
   return (await loadInstalledConfig(os10.homedir()))?.config.stack;
 }
-async function buildInstallConfig(wizardResult, sourceResult, projectDir) {
-  const skillIds = unique4(wizardResult.skills.map((s) => s.id));
+async function buildInstallConfig(selection, sourceResult, projectDir) {
+  const skillIds = unique4(selection.skills.map((s) => s.id));
   verbose(
-    `buildInstallConfig: selectedStackId='${wizardResult.selectedStackId}', skills=[${skillIds.join(", ")}], selectedAgents=[${wizardResult.selectedAgents.join(", ")}]`
+    `buildInstallConfig: selectedStackId='${selection.selectedStackId}', skills=[${skillIds.join(", ")}], selectedAgents=[${selection.selectedAgents.join(", ")}]`
   );
-  const loadedStack = await loadSelectedStack(wizardResult.selectedStackId, sourceResult);
+  const loadedStack = await loadSelectedStack(selection.selectedStackId, sourceResult);
   const existing = await loadProjectConfig(projectDir);
   const existingStack = await loadCuratedStack(projectDir, existing?.config.stack);
   const options = generatorOptionsFor(
-    wizardResult,
+    selection,
     existing?.config,
     stackToPreserve(loadedStack, existingStack)
   );
   const generated = generateProjectConfigFromSkills(configNameFor(projectDir), skillIds, options);
-  const stack = resolveStackProperty(generated.stack, wizardResult.assignedStack);
-  const description = resolveDescription(loadedStack, wizardResult.description);
+  const stack = resolveStackProperty(generated.stack, selection.assignedStack);
+  const description = resolveDescription(loadedStack, selection.description);
   const localConfig = {
     ...generated,
     ...stack && { stack },
@@ -6186,30 +6160,30 @@ function stackToPreserve(loadedStack, curatedStack) {
   if (loadedStack === null) return curatedStack;
   return { ...buildStackProperty(loadedStack), ...curatedStack };
 }
-function generatorOptionsFor(wizardResult, persisted, existingStack) {
+function generatorOptionsFor(selection, persisted, existingStack) {
   return {
-    skillConfigs: wizardResult.skills,
-    agentConfigs: wizardResult.agentConfigs,
+    skillConfigs: selection.skills,
+    agentConfigs: selection.agentConfigs,
     existingStack,
-    newlyAddedSkillIds: computeNewlyAddedSkillIds(wizardResult.skills, persisted?.skills),
+    newlyAddedSkillIds: computeNewlyAddedSkillIds(selection.skills, persisted?.skills),
     scopeEligibilityGained: computeScopeEligibilityGained(
-      wizardResult.skills,
-      wizardResult.agentConfigs,
+      selection.skills,
+      selection.agentConfigs,
       persisted?.skills,
       persisted?.agents
     ),
-    ...wizardResult.selectedAgents.length > 0 && {
-      selectedAgents: wizardResult.selectedAgents
+    ...selection.selectedAgents.length > 0 && {
+      selectedAgents: selection.selectedAgents
     }
   };
 }
 function stackSizeForLog(stack) {
   return stack ? `${Object.keys(stack).length} agents` : "UNDEFINED";
 }
-function setConfigMetadata(config, wizardResult, sourceResult, sourceFlag) {
+function setConfigMetadata(config, selection, sourceResult, sourceFlag) {
   const result = { ...config };
-  if (wizardResult.selectedDomains.length > 0) {
-    result.selectedDomains = wizardResult.selectedDomains;
+  if (selection.selectedDomains.length > 0) {
+    result.selectedDomains = selection.selectedDomains;
   }
   if (sourceFlag) {
     result.marketplace = sourceFlag;
@@ -6221,10 +6195,10 @@ function setConfigMetadata(config, wizardResult, sourceResult, sourceFlag) {
   }
   return result;
 }
-async function buildAndMergeConfig(wizardResult, sourceResult, projectDir, sourceFlag, authoritativeScope) {
-  const { config } = await buildInstallConfig(wizardResult, sourceResult, projectDir);
+async function buildAndMergeConfig(selection, sourceResult, projectDir, sourceFlag, authoritativeScope) {
+  const { config } = await buildInstallConfig(selection, sourceResult, projectDir);
   verbose(`buildAndMergeConfig: before merge \u2014 stack=${stackSizeForLog(config.stack)}`);
-  const configWithMetadata = setConfigMetadata(config, wizardResult, sourceResult, sourceFlag);
+  const configWithMetadata = setConfigMetadata(config, selection, sourceResult, sourceFlag);
   const result = await mergeWithExistingConfig(configWithMetadata, {
     projectDir,
     ...authoritativeScope !== void 0 && { authoritativeScope }
@@ -6958,7 +6932,7 @@ function hostAt(root) {
 
 // src/cli/lib/installation/mode-migrator.ts
 function ejectCopyFailureError(failedCount) {
-  return `Failed to copy ${failedCount} skill(s) for eject. Eject intent could not be honored. Check that the destination skills directory is writable, then run '${CLI_INVOKE_COMMAND} edit' again \u2014 or switch the affected skills back to plugin mode.`;
+  return `Failed to copy ${failedCount} skill(s) for eject. Eject intent could not be honored. Check that the destination skills directory is writable, then apply the same id again with '${CLI_INVOKE_COMMAND} edit --from <id>' \u2014 or switch the affected skills back to plugin mode.`;
 }
 function detectMigrations(oldSkills, newSkills) {
   const oldById = new Map(oldSkills.map((s) => [s.id, s]));
@@ -7466,13 +7440,13 @@ async function loadSkillsOfPlugin({
   }
 }
 function mergeInPluginOrder(perPluginSkills) {
-  const allSkills2 = {};
+  const allSkills = {};
   for (const pluginSkills of perPluginSkills) {
     for (const [id, skill] of typedEntries(pluginSkills)) {
-      if (skill) allSkills2[id] = skill;
+      if (skill) allSkills[id] = skill;
     }
   }
-  return allSkills2;
+  return allSkills;
 }
 async function listPluginNames(projectDir) {
   try {
@@ -7703,66 +7677,25 @@ async function extractLocalSkill(localSkillsPath, skillDirName) {
 
 // src/cli/lib/skills/unresolved-skill-entries.ts
 import path26 from "path";
-async function findUnusableSavedSkillMetadata(unresolvedIds, savedSkills, projectDir) {
-  const classified = await classifyUnresolvedSkills(unresolvedIds, savedSkills, projectDir);
-  return classified.flatMap(({ fate }) => fate.kind === "unusable-metadata" ? [fate.detail] : []);
-}
-async function unresolvedSkillRemovalReasons(unresolvedIds, savedSkills, projectDir, sourceLabel) {
-  const classified = await classifyUnresolvedSkills(unresolvedIds, savedSkills, projectDir);
-  return new Map(
-    classified.flatMap(({ id, fate }) => {
-      const reason = removalReason(id, fate, sourceLabel);
-      return reason === null ? [] : [[id, reason]];
-    })
-  );
-}
-async function classifyUnresolvedSkills(unresolvedIds, savedSkills, projectDir) {
-  return Promise.all(
-    unresolvedIds.map(async (id) => {
-      const saved = savedSkills.find((entry) => entry.id === id);
-      return { id, fate: await classifySavedSkill(id, saved, projectDir) };
-    })
+async function findUnusableSavedSkillMetadata(savedSkills, projectDir) {
+  const verdicts = await Promise.all(
+    savedSkills.map((saved) => unusableMetadataOf(saved, projectDir))
   );
+  return verdicts.filter((verdict) => verdict !== null);
 }
-async function classifySavedSkill(id, saved, projectDir) {
-  if (saved?.origin !== EJECT_SOURCE) return { kind: "dropped-by-source" };
-  const skillDir = path26.join(resolveInstallPaths(projectDir, saved.scope).skillsDir, id);
-  if (!await directoryExists2(skillDir)) return { kind: "files-gone", skillDir };
-  const metadataPath = path26.join(skillDir, STANDARD_FILES.METADATA_YAML);
-  if (!await fileExists(metadataPath)) return { kind: "not-installed-there", skillDir };
+async function unusableMetadataOf(saved, projectDir) {
+  if (!claimsLocalCopy(saved)) return null;
+  const metadataPath = installedMetadataPath(saved, projectDir);
+  if (!await fileExists(metadataPath)) return null;
   const read = await readSkillMetadata(metadataPath);
-  if (!read.usable) {
-    return {
-      kind: "unusable-metadata",
-      detail: { skillDirName: id, metadataPath, reason: read.reason }
-    };
-  }
-  if (!await skillMdNames(skillDir, id)) return { kind: "not-installed-there", skillDir };
-  return { kind: "unplaceable-category", skillDir, category: read.metadata.category };
-}
-async function skillMdNames(skillDir, id) {
-  const skillMdPath = path26.join(skillDir, STANDARD_FILES.SKILL_MD);
-  if (!await fileExists(skillMdPath)) return false;
-  const frontmatter = parseFrontmatter(await readFile(skillMdPath), skillMdPath);
-  return frontmatter?.name === id;
-}
-function removalReason(id, fate, sourceLabel) {
-  switch (fate.kind) {
-    case "dropped-by-source":
-      return `not present in ${sourceLabel}`;
-    case "files-gone":
-      return `skill files no longer exist at ${fate.skillDir}`;
-    case "not-installed-there":
-      return `no skill named '${id}' is installed at ${fate.skillDir}`;
-    case "unplaceable-category":
-      return `installed at ${fate.skillDir}, but its category '${fate.category}' is not one this source knows`;
-    case "unusable-metadata":
-      return null;
-    default: {
-      const exhaustive = fate;
-      return exhaustive;
-    }
-  }
+  return read.usable ? null : { skillDirName: saved.id, metadataPath, reason: read.reason };
+}
+function claimsLocalCopy(saved) {
+  return !saved.excluded && saved.origin === EJECT_SOURCE;
+}
+function installedMetadataPath(saved, projectDir) {
+  const { skillsDir: skillsDir2 } = resolveInstallPaths(projectDir, saved.scope);
+  return path26.join(skillsDir2, saved.id, STANDARD_FILES.METADATA_YAML);
 }
 
 // src/cli/lib/skills/local-skill-mover.ts
@@ -7821,10 +7754,6 @@ async function migrateLocalSkillScope(skillId, fromScope, projectDir) {
   }
   verbose(`Migrated skill '${skillId}' from ${fromScope} to ${toScope}`);
 }
-async function foldLocalSkillIntoGlobal(projectDir, skillId) {
-  await deleteLocalSkill(projectDir, skillId, "global");
-  await migrateLocalSkillScope(skillId, "project", projectDir);
-}
 
 // src/cli/lib/matrix/matrix-loader.ts
 import { parse as parseYaml5 } from "yaml";
@@ -8123,233 +8052,6 @@ function buildResolvedSkill(skill, _categories, relationships, slugMap) {
   };
 }
 
-// src/cli/lib/matrix/matrix-resolver.ts
-import { groupBy as groupBy4 } from "remeda";
-function getLabel(skill) {
-  return stripTerminalControls(skill.displayName);
-}
-function joinWithConjunction(items, conjunction) {
-  if (items.length <= 1) return items[0] ?? "";
-  if (items.length === 2) return `${items[0]} ${conjunction} ${items[1]}`;
-  return `${items.slice(0, -1).join(", ")} ${conjunction} ${items[items.length - 1]}`;
-}
-function initializeSelectionContext(currentSelections) {
-  const resolvedSelections = currentSelections.map((s) => getSkillById(s).id);
-  const selectedSet = new Set(resolvedSelections);
-  return { resolvedSelections, selectedSet };
-}
-function toSelectionFacts(source) {
-  return {
-    skills: typedValues(source.skills).map((skill) => ({
-      id: skill.id,
-      categoryId: skill.category,
-      conflictsWith: skill.conflictsWith.map((relation) => relation.skillId),
-      discourages: skill.discourages.map((relation) => ({
-        skillId: relation.skillId,
-        // The one author string that leaves this module without being re-rendered: the shared
-        // semantics carry it through and `getCellState` hands the verdict back whole, so a
-        // discouraged cell prints exactly what was put in here. Made inert on the way in, which
-        // is the only point on that route this package owns.
-        reason: stripTerminalControls(relation.reason)
-      })),
-      requires: skill.requires.map((requirement) => ({
-        skillIds: requirement.skillIds,
-        needsAny: requirement.needsAny
-      }))
-    })),
-    exclusiveCategoryIds: new Set(
-      typedValues(source.categories).filter((category) => category.exclusive !== false).map((category) => category.id)
-    )
-  };
-}
-var boundSemantics;
-var boundTo;
-function judgeSelections(resolvedSelections) {
-  if (!boundSemantics || boundTo !== matrix) {
-    boundSemantics = createSelectionSemantics(toSelectionFacts(matrix));
-    boundTo = matrix;
-  }
-  return boundSemantics(resolvedSelections);
-}
-function renderIncompatibility(cause) {
-  switch (cause.kind) {
-    case "conflict":
-      return `conflicts with ${labelOf(cause.skillId)}`;
-    case "unreachableRequirement": {
-      if (cause.requirement.needsAny) {
-        const labels = cause.requirement.skillIds.map(labelOf);
-        return `requires ${joinWithConjunction(labels, "or")} (all conflict with current selection)`;
-      }
-      const lostLabels = cause.lostIds.map(labelOf);
-      return `requires ${joinWithConjunction(lostLabels, "and")} which conflicts with current selection`;
-    }
-  }
-}
-function labelOf(skillId) {
-  return getLabel(getSkillById(skillId));
-}
-function isRequirementMet(req, selectedSet) {
-  return req.needsAny ? req.skillIds.some((id) => selectedSet.has(id)) : req.skillIds.every((id) => selectedSet.has(id));
-}
-function missingRequirementIds(req, selectedSet) {
-  if (isRequirementMet(req, selectedSet)) return [];
-  return req.needsAny ? [...req.skillIds] : req.skillIds.filter((id) => !selectedSet.has(id));
-}
-function getUnmetRequiredBy(skillId, currentSelections) {
-  getSkillById(skillId);
-  const { resolvedSelections, selectedSet } = initializeSelectionContext(currentSelections);
-  if (selectedSet.has(skillId)) return void 0;
-  for (const selectedId of resolvedSelections) {
-    const selectedSkill = matrix.skills[selectedId];
-    if (!selectedSkill) continue;
-    for (const req of selectedSkill.requires) {
-      if (!req.skillIds.includes(skillId)) continue;
-      if (req.needsAny) {
-        if (!req.skillIds.some((id) => selectedSet.has(id))) {
-          return getLabel(selectedSkill);
-        }
-      } else {
-        return getLabel(selectedSkill);
-      }
-    }
-  }
-  return void 0;
-}
-function hasUnmetRequirements(skillId, currentSelections) {
-  return getUnmetRequirementsReason(skillId, currentSelections) !== void 0;
-}
-function getCellState(skillId, currentSelections) {
-  const skill = getSkillById(skillId);
-  const { resolvedSelections } = initializeSelectionContext(currentSelections);
-  const verdict = judgeSelections(resolvedSelections).verdictOf(skill.id);
-  return verdict.status === "incompatible" ? { status: "incompatible", reason: renderIncompatibility(verdict.cause) } : verdict;
-}
-function getUnmetRequirementsReason(skillId, currentSelections) {
-  const skill = getSkillById(skillId);
-  const { selectedSet } = initializeSelectionContext(currentSelections);
-  for (const requirement of skill.requires) {
-    const missing = missingRequirementIds(requirement, selectedSet);
-    if (missing.length === 0) continue;
-    const names = missing.map((id) => {
-      const s = matrix.skills[id];
-      return s ? getLabel(s) : stripTerminalControls(id);
-    });
-    return withAuthorsReason(namesTheMissing(names, requirement.needsAny), requirement.reason);
-  }
-  return void 0;
-}
-function namesTheMissing(names, needsAny) {
-  return `requires ${joinWithConjunction(names, needsAny ? "or" : "and")}`;
-}
-var REASON_BUDGET = 60;
-function withAuthorsReason(synthesised, reason) {
-  const stated = reason.trim();
-  if (stated === "") return synthesised;
-  return `${synthesised} \u2014 ${truncateText(stated, REASON_BUDGET)}`;
-}
-function unmetRequirementsReasonFor(isSelected, skillId, currentSelections) {
-  if (!isSelected) return {};
-  const reason = getUnmetRequirementsReason(skillId, currentSelections);
-  return reason === void 0 ? {} : { unmetRequirementsReason: reason };
-}
-function validateConflicts(resolvedSelections) {
-  const errors = [];
-  for (const [index, skillAId] of resolvedSelections.entries()) {
-    const skillA = matrix.skills[skillAId];
-    if (!skillA) continue;
-    for (const skillBId of resolvedSelections.slice(index + 1)) {
-      const conflict = skillA.conflictsWith.find((c) => c.skillId === skillBId);
-      if (conflict) {
-        errors.push({
-          type: "conflict",
-          // The author's own sentence, made inert but not shortened: this message is reported on
-          // its own line rather than into a fixed-height cell, so it needs no budget the way
-          // `withAuthorsReason` does — only the guarantee that a terminal will print it.
-          message: `${getLabel(skillA)} conflicts with ${getLabel(getSkillById(skillBId))}: ${stripTerminalControls(conflict.reason)}`,
-          skills: [skillA.id, skillBId]
-        });
-      }
-    }
-  }
-  return errors;
-}
-function validateRequirements(resolvedSelections, selectedSet) {
-  const errors = [];
-  for (const skillId of resolvedSelections) {
-    const skill = matrix.skills[skillId];
-    if (!skill) continue;
-    for (const requirement of skill.requires) {
-      if (requirement.needsAny) {
-        const hasAny = requirement.skillIds.some((reqId) => selectedSet.has(reqId));
-        if (!hasAny) {
-          errors.push({
-            type: "missingRequirement",
-            message: `${getLabel(skill)} requires one of: ${requirement.skillIds.map((id) => getLabel(getSkillById(id))).join(", ")}`,
-            skills: [skillId, ...requirement.skillIds]
-          });
-        }
-      } else {
-        const missingIds = requirement.skillIds.filter((reqId) => !selectedSet.has(reqId));
-        if (missingIds.length > 0) {
-          errors.push({
-            type: "missingRequirement",
-            message: `${getLabel(skill)} requires: ${missingIds.map((id) => getLabel(getSkillById(id))).join(", ")}`,
-            skills: [skillId, ...missingIds]
-          });
-        }
-      }
-    }
-  }
-  return errors;
-}
-function validateExclusivity(resolvedSelections) {
-  const errors = [];
-  const validSkills = resolvedSelections.map((skillId) => ({ skillId, skill: matrix.skills[skillId] })).filter((entry) => entry.skill != null);
-  const categorySelections = groupBy4(validSkills, (entry) => entry.skill.category);
-  for (const [categoryId, entries] of typedEntries(categorySelections)) {
-    if (entries.length > 1) {
-      const skillIds = entries.map((e) => e.skillId);
-      if (categoryId === LOCAL_PSEUDO_CATEGORY) continue;
-      const category = matrix.categories[categoryId];
-      if (category?.exclusive) {
-        errors.push({
-          type: "categoryExclusive",
-          message: `Category "${stripTerminalControls(category.displayName)}" only allows one selection, but multiple selected: ${skillIds.map((id) => getLabel(getSkillById(id))).join(", ")}`,
-          skills: skillIds
-        });
-      }
-    }
-  }
-  return errors;
-}
-function validateSelection2(selections) {
-  const { resolvedSelections, selectedSet } = initializeSelectionContext(selections);
-  const errors = [
-    ...validateConflicts(resolvedSelections),
-    ...validateRequirements(resolvedSelections, selectedSet),
-    ...validateExclusivity(resolvedSelections)
-  ];
-  return {
-    valid: errors.length === 0,
-    errors
-  };
-}
-function getAvailableSkills(categoryId, currentSelections) {
-  const { selectedSet } = initializeSelectionContext(currentSelections);
-  return getSkillsByCategory(categoryId).map((skill) => {
-    const isSelected = selectedSet.has(skill.id);
-    return {
-      id: skill.id,
-      selected: isSelected,
-      hasUnmetRequirements: isSelected && hasUnmetRequirements(skill.id, currentSelections),
-      ...unmetRequirementsReasonFor(isSelected, skill.id, currentSelections)
-    };
-  });
-}
-function getSkillsByCategory(categoryId) {
-  return allSkills().filter((skill) => skill.category === categoryId);
-}
-
 // src/cli/lib/configuration/skill-audit.ts
 var skillAudit = {
   // ── B1 — web core: frameworks, meta-frameworks, routing, server state (20) ──
@@ -9892,7 +9594,7 @@ var skillAudit = {
     batch: "shared-meta",
     classification: { class: "B", frameworks: ["react"] },
     sources: [
-      'skill body self-scope (description "Use when reviewing React components"; When-to-use ".tsx/.jsx with JSX"; "React PR review" auto-detection) plus a React-keyed majority (4 of 5 MUSTs, 6 of 7 patterns)'
+      'skill body self-scope (description "Load when reviewing a diff that adds or changes React components or hooks"; Applies-to "Diffs that add or change React components or custom hooks"; "React PR review" auto-detection) plus a React-keyed majority (4 of 5 critical requirements, 6 of 7 patterns)'
     ]
   },
   "meta-reviewing-api-reviewing": {
@@ -9923,8 +9625,8 @@ var skillAudit = {
 var auditVerdictsPendingApply = {};
 
 // src/cli/lib/matrix/matrix-health-check.ts
-function checkMatrixHealth(matrix2) {
-  const issues = [
+function matrixHealthIssues(matrix2) {
+  return [
     ...checkCategoryDomains(matrix2),
     ...checkSkillCategories(matrix2),
     ...checkSkillRelationRefs(matrix2),
@@ -9932,6 +9634,9 @@ function checkMatrixHealth(matrix2) {
     ...checkAuditVerdictContradictions(matrix2),
     ...checkUnauditedSkills(matrix2)
   ];
+}
+function checkMatrixHealth(matrix2) {
+  const issues = matrixHealthIssues(matrix2);
   for (const issue of issues) {
     warn(`[matrix] ${issue.details}`);
   }
@@ -9941,7 +9646,7 @@ function checkCategoryDomains(matrix2) {
   return typedEntries(matrix2.categories).filter(([, cat]) => cat !== void 0 && !cat.domain).map(([catId]) => ({
     severity: "warning",
     finding: "category-missing-domain",
-    details: `Category '${catId}' has no domain \u2014 it won't appear in any wizard domain view`
+    details: `Category '${catId}' has no domain \u2014 it won't appear under any domain`
   }));
 }
 function checkSkillCategories(matrix2) {
@@ -10026,7 +9731,10 @@ function checkUnauditedSkills(matrix2) {
 }
 
 // src/cli/lib/loading/source-loader.ts
-async function loadSkillsMatrixFromSource(options = {}) {
+function loadSkillsMatrixFromSource(options = {}) {
+  return sayingEachWarningOnce(() => loadCatalogue(options));
+}
+async function loadCatalogue(options) {
   const {
     purpose = "fetch",
     caller = "stored",
@@ -10269,12 +9977,6 @@ async function loadAndMergeFromBasePath(basePath, source) {
   initializeMatrix(mergedMatrix);
   const stacks = await resolveOfferedStacks(basePath, stacksRelFile, source);
   mergedMatrix.suggestedStacks = stacks.map((stack) => convertStackToResolvedStack(stack));
-  const agentDefinedDomains = domainsDeclaredBy(await loadAllAgents(basePath));
-  const domainCount = typedKeys(agentDefinedDomains).length;
-  if (domainCount > 0) {
-    mergedMatrix.agentDefinedDomains = agentDefinedDomains;
-    verbose(`Loaded ${domainCount} agent domain definition(s)`);
-  }
   return mergedMatrix;
 }
 async function loadSourceTaxonomy(basePath) {
@@ -10300,13 +10002,6 @@ async function loadSourceTaxonomy(basePath) {
   }
   return { categories, sourceRules };
 }
-function domainsDeclaredBy(agents) {
-  return typedFromEntries(
-    typedEntries(agents).flatMap(
-      ([agentId, agentDef]) => agentDef.domain ? [[agentId, agentDef.domain]] : []
-    )
-  );
-}
 var CATALOGUE_SKILL_IDS = new Set(typedKeys(BUILT_IN_MATRIX.skills));
 var MAX_REPORTED_COLLISIONS = 10;
 async function refuseCatalogueCollisions(basePath, source, skills) {
@@ -10460,7 +10155,6 @@ export {
   stripTerminalControls,
   truncateText,
   plural,
-  toTitleCase,
   getErrorMessage,
   setVerbose,
   verbose,
@@ -10469,6 +10163,7 @@ export {
   drainBuffer,
   disableBuffering,
   warn,
+  sayingEachWarningOnce,
   renderAgentRoleToml,
   agentsLeftOutOfCodexMessage,
   unexpressibleOnCodexMessage,
@@ -10527,6 +10222,7 @@ export {
   skillMetadataUnusableDetail,
   skillMetadataUnusableError,
   savedSkillMetadataUnusableError,
+  editorSkillMetadataUnusableError,
   sharedConfigDestinations,
   applySharedConfigHere,
   shareRefusesEjectedSkills,
@@ -10541,12 +10237,11 @@ export {
   projectCopyRemoved,
   sharedConfigNeedsTerminal,
   nothingInstalledToApplyTo,
-  wizardNeedsTerminal,
   uninstallNeedsTerminal,
-  editorLoadsItsOwnMarketplace,
-  marketplaceFixedAtInstall,
+  addMarketplaceInTheEditor,
   SHARED_CONFIG_ARRIVALS,
   keptAsInstalledGlobally,
+  addedToGlobalSubAgents,
   authoredHereKept,
   unplaceableKept,
   keptUnassigned,
@@ -10580,7 +10275,6 @@ export {
   copy,
   formatZodErrors,
   isRecord,
-  isDomain,
   isAgentName,
   skillCategoriesFileSchema,
   skillRulesFileSchema,
@@ -10606,7 +10300,6 @@ export {
   resolveBranding,
   resolvePrimarySourceEntry,
   isLocalSource,
-  isSameMarketplace,
   normalizeStackRecord,
   defaultUsageGuidance,
   resolveAgentConfigToSkills,
@@ -10633,7 +10326,7 @@ export {
   typedEntries2 as typedEntries,
   isScopePairCompatible,
   splitConfigByScope,
-  seedToWizardResult,
+  seedToInstallSelection,
   mergeConfigs,
   generateConfigTypesSource,
   deriveCategories,
@@ -10689,16 +10382,10 @@ export {
   printCompilationSummary,
   discoverLocalSkills,
   findUnusableSavedSkillMetadata,
-  unresolvedSkillRemovalReasons,
   deleteLocalSkill,
   migrateLocalSkillScope,
-  foldLocalSkillIntoGlobal,
   claimSlug,
-  getUnmetRequiredBy,
-  getCellState,
-  validateSelection2 as validateSelection,
-  getAvailableSkills,
-  checkMatrixHealth,
+  matrixHealthIssues,
   generateMarketplace,
   validateMarketplaceName,
   validateSkillIdNamespace,
@@ -10724,4 +10411,4 @@ export {
   detectMigrations,
   executeMigration
 };
-//# sourceMappingURL=chunk-C2UISKIM.js.map
+//# sourceMappingURL=chunk-MQ6CGZNJ.js.map