agents-inc 0.170.1 → 0.170.2

package/dist/{chunk-PJWGTNOX.js → chunk-7C3PFRBU.js}

@@ -1,5 +1,16 @@
 #!/usr/bin/env node
 
+// ../compile/src/string.ts
+function bytewise(a, b) {
+  return a < b ? -1 : a > b ? 1 : 0;
+}
+function inAnEnglishList(items) {
+  const last = items.at(-1);
+  if (last === void 0) return "";
+  if (items.length === 1) return last;
+  return `${items.slice(0, -1).join(", ")} and ${last}`;
+}
+
 // ../compile/src/paths.ts
 var CLAUDE_DIR = ".claude";
 var SOURCE_ROOT_DIR = ".agents-inc";
@@ -7,7 +18,7 @@ var SOURCE_SRC_DIR = "src";
 var SKILLS_DIR_PATH = `${SOURCE_SRC_DIR}/skills`;
 var LOCAL_SKILLS_PATH = ".claude/skills";
 var DEFAULT_PLUGIN_NAME = "agents-inc";
-var DEFAULT_PUBLIC_SOURCE_NAME = DEFAULT_PLUGIN_NAME;
+var DEFAULT_PUBLIC_SOURCE_NAME = "agents-inc";
 var GITHUB_PREFIX = "github:";
 var DEFAULT_SOURCE = `${GITHUB_PREFIX}agents-inc/skills`;
 var CLI_INVOKE_COMMAND = "npx agents-inc";
@@ -50,7 +61,7 @@ var STANDARD_DIRS = {
   SCRIPTS: "scripts",
   SKILLS: "skills",
   AGENTS: "agents",
-  /** Legacy per-project template override directory (`.claude/templates`). */
+  /** Legacy per-project template override directory. */
   TEMPLATES: "templates"
 };
 
@@ -87,11 +98,6 @@ function effectivelyExcludedSkillIds(skills) {
   );
 }
 
-// src/cli/consts.ts
-import os from "os";
-import path from "path";
-import { fileURLToPath } from "url";
-
 // ../compile/src/catalog.ts
 function isExclusiveCategory(catalog, category) {
   return catalog.categories[category]?.exclusive === true;
@@ -501,8 +507,7 @@ var groupBy = (items, keyOf) => {
   return groups;
 };
 var indexById = (items) => (
-  // Boundary cast: `Object.fromEntries` types every key as `string`; each of
-  // these is the item's own id, which is the key type being claimed.
+  // Cast: `Object.fromEntries` widens keys to `string`; each is the item's own id.
   Object.fromEntries(items.map((item) => [item.id, item]))
 );
 
@@ -9820,8 +9825,7 @@ var categoryDefinitionSchema = z.object({
   id: categorySchema,
   displayName: z.string(),
   description: z.string(),
-  // Optional in the CLI's type. Present on all 89 built-in categories today; a category without
-  // one cannot be placed in the UI, so the read model drops it rather than failing the whole boot.
+  // A category without one is dropped by the read model, not rejected.
   domain: domainIdSchema.exactOptional(),
   exclusive: z.boolean(),
   order: z.number()
@@ -9840,12 +9844,8 @@ var resolvedSkillSchema = z.object({
   slug: z.string(),
   displayName: z.string(),
   description: z.string(),
-  // The sentence a compiled sub-agent reads as its cue to load the skill. Optional
-  // on the CLI's `SkillCore`, so a catalogue entry without one falls back to its
-  // category — but a schema that omits the field STRIPS it, which is the same value
-  // arriving by a route nothing can tell apart from the skill stating nothing.
+  // Declared so parsing does not strip it.
   usageGuidance: z.string().exactOptional(),
-  // The SKILL.md description compile renders — see `SkillCore.activationDescription`.
   activationDescription: z.string().exactOptional(),
   category: categorySchema,
   conflictsWith: z.array(skillRelationSchema),
@@ -9856,20 +9856,16 @@ var resolvedStackSchema = z.object({
   id: z.string(),
   name: z.string(),
   description: z.string(),
-  // agent id → category id → skill ids. Partial because a stack names the few
-  // agents it staffs, not the whole roster; the category keys are the one level
-  // nothing here reads, so they stay unvalidated strings.
+  // agent id → category id → skill ids; a stack names only the agents it staffs.
   skills: z.partialRecord(
     agentNameSchema,
     z.record(z.string(), z.array(skillIdSchema))
   ),
   allSkillIds: z.array(skillIdSchema),
   philosophy: z.string(),
-  // Never populated by the CLI today; the stack rail groups by it when it appears.
   group: z.string().exactOptional()
 });
 var builtInMatrixSchema = z.object({
-  // Stamped into every seed payload so a consumer can explain skipped ids.
   version: z.string(),
   categories: z.record(z.string(), categoryDefinitionSchema),
   skills: z.record(z.string(), resolvedSkillSchema),
@@ -9923,7 +9919,8 @@ var agentDomainOf = (agentId) => {
 var DOMAIN_POSITION = new Map(
   DOMAIN_ORDER.map((domain, index) => [domain, index])
 );
-var compareDomains = (a, b) => (DOMAIN_POSITION.get(a) ?? DOMAIN_ORDER.length) - (DOMAIN_POSITION.get(b) ?? DOMAIN_ORDER.length);
+var positionOf = (domain) => DOMAIN_POSITION.get(domain) ?? DOMAIN_ORDER.length;
+var compareDomains = (a, b) => positionOf(a) - positionOf(b);
 
 // ../matrix/src/read-model/catalog.ts
 var toCatalogSkill = (skill, domainId) => ({
@@ -10175,9 +10172,7 @@ var agentDefinitionSchema = z2.object({
   description: z2.string(),
   model: modelNameSchema.exactOptional(),
   tools: z2.array(z2.string()),
-  // Not narrowed here: which roles are sayable is authored in the read model's
-  // preload table — so the check lives where the list does, in `flavorOf`,
-  // rather than the list being imported back into the boundary.
+  // Narrowed by `flavorOf`, where the role list lives.
   flavor: z2.string()
 });
 var agentDefinitionsSchema = z2.record(
@@ -10195,7 +10190,6 @@ var ROLE_FLAVORS = [
   "meta"
 ];
 var PRELOAD_DEFAULTS = {
-  // Web — frameworks, the concerns built on them, and the tests
   "web-framework-angular-standalone": ["developer", "planning", "researcher", "reviewer", "tester"],
   "web-framework-react": ["developer", "planning", "researcher", "reviewer", "tester"],
   "web-framework-solidjs": ["developer", "planning", "researcher", "reviewer", "tester"],
@@ -10255,7 +10249,6 @@ var PRELOAD_DEFAULTS = {
   "web-testing-vue-test-utils": ["tester"],
   "web-utilities-native-js": ["developer"],
   "web-utilities-vueuse": ["developer"],
-  // API — frameworks, data, auth, and the safety around them
   "api-framework-express": ["developer", "planning", "researcher", "reviewer", "tester"],
   "api-framework-fastify": ["developer", "planning", "researcher", "reviewer", "tester"],
   "api-framework-hono": ["developer", "planning", "researcher", "reviewer", "tester"],
@@ -10292,7 +10285,6 @@ var PRELOAD_DEFAULTS = {
   "api-performance-api-performance": ["developer", "researcher"],
   "api-specs-openapi": ["developer"],
   "api-observability-axiom-pino-sentry": ["developer"],
-  // Mobile
   "mobile-framework-react-native": ["developer", "planning", "researcher", "reviewer", "tester"],
   "mobile-framework-expo": ["developer", "planning", "researcher", "reviewer", "tester"],
   "mobile-navigation-expo-router": ["developer"],
@@ -10306,7 +10298,6 @@ var PRELOAD_DEFAULTS = {
   "mobile-performance-react-native": ["developer", "researcher"],
   "mobile-testing-detox": ["tester"],
   "mobile-testing-maestro": ["tester"],
-  // Desktop
   "desktop-framework-electron": ["developer", "planning", "researcher", "reviewer", "tester"],
   "desktop-framework-tauri": ["developer", "planning", "researcher", "reviewer", "tester"],
   "desktop-backend-tauri": ["developer", "researcher"],
@@ -10315,8 +10306,6 @@ var PRELOAD_DEFAULTS = {
   "desktop-plugins-tauri": ["developer"],
   "desktop-testing-electron": ["tester"],
   "desktop-storage-electron": ["developer", "researcher"],
-  // AI — the provider and the orchestration around it are what the project is
-  // built on, so they carry the planner too; evals and tracing test
   "ai-infrastructure-huggingface-inference": ["developer", "researcher"],
   "ai-infrastructure-ollama": ["developer", "researcher"],
   "ai-infrastructure-replicate": ["developer", "researcher"],
@@ -10332,19 +10321,14 @@ var PRELOAD_DEFAULTS = {
   "ai-provider-google-gemini-sdk": ["developer", "planning", "researcher"],
   "ai-provider-mistral-sdk": ["developer", "planning", "researcher"],
   "ai-provider-openai-sdk": ["developer", "planning", "researcher"],
-  // CLI — the framework and its prompts are the whole surface
   "cli-framework-cli-commander": ["developer", "planning", "researcher", "reviewer", "tester"],
   "cli-framework-oclif-ink": ["developer", "planning", "researcher", "reviewer", "tester"],
   "cli-prompts-clack": ["developer", "researcher", "tester"],
-  // Infra — deployment is occasional, so almost nothing preloads
   "infra-platform-cloudflare-workers": ["developer"],
-  // Meta — the review PROCESS is every review session's material; the domain
-  // checklists are one diff's, so they carry no row and arrive per diff
   "meta-design-composable-components": ["developer"],
   "meta-design-expressive-typescript": ["developer", "meta"],
   "meta-methodology-research-methodology": ["researcher", "meta"],
   "meta-reviewing-reviewing": ["reviewer"],
-  // Shared — carried across every implementation domain
   "shared-security-auth-security": ["developer", "researcher"]
 };
 var KNOWN_SKILL_IDS = new Set(SKILL_IDS);
@@ -10398,9 +10382,6 @@ var toSubAgent = (definition) => {
     description: definition.description,
     ...definition.model !== void 0 && { model: definition.model },
     domainId,
-    // The same string the definition carries, read back through the preload
-    // table's list of sayable roles — a role no entry could name is an error
-    // there, so it is one here rather than a flavor nothing routes on.
     flavor: flavorOf(definition.id)
   };
 };
@@ -10427,9 +10408,8 @@ var subAgentById = (agentId) => isAgentName(agentId) ? SUB_AGENTS_BY_ID[agentId]
 var ROSTER = SUB_AGENT_GROUPS.flatMap(
   (group) => group.agents
 );
-var isCrossDomainRoleAgent = (agent) => agent.domainId === "meta" && agent.flavor !== "meta";
 var implementationDomainReach = (domainId) => ROSTER.filter(
-  (agent) => agent.domainId === domainId || isCrossDomainRoleAgent(agent)
+  (agent) => agent.domainId === domainId || isDomainlessRoleAgent(agent.id)
 );
 var CRAFT_CATEGORIES_BY_FLAVOR = {
   meta: ["meta-design", "meta-methodology"],
@@ -10497,17 +10477,18 @@ var toAgentsBySkill = (stack) => {
 var toLoadByAgent = (skillId) => new Map(
   resolveAssignment(skillId).map(({ agentId, load }) => [agentId, load])
 );
+var toAssignmentTargets = (skillId, agentIds) => {
+  const loadByAgent = toLoadByAgent(skillId);
+  return agentIds.map((agentId) => ({
+    agentId,
+    load: loadByAgent.get(agentId) ?? "lazy"
+  }));
+};
 var toAssignmentsBySkill = (stack) => Object.fromEntries(
-  Object.entries(toAgentsBySkill(stack)).map(([skillId, agentIds]) => {
-    const loadByAgent = toLoadByAgent(skillId);
-    return [
-      skillId,
-      agentIds.map((agentId) => ({
-        agentId,
-        load: loadByAgent.get(agentId) ?? "lazy"
-      }))
-    ];
-  })
+  Object.entries(toAgentsBySkill(stack)).map(([skillId, agentIds]) => [
+    skillId,
+    toAssignmentTargets(skillId, agentIds)
+  ])
 );
 var createStackExpander = (matrix) => (stackId) => {
   const stack = matrix.suggestedStacks.find(
@@ -10530,6 +10511,7 @@ var DEFAULT_SELECTION_OPTIONS = {
 
 // ../matrix/src/read-model/selection-semantics.ts
 var isAmbiguous = (requirement) => requirement.needsAny && requirement.skillIds.length > 1;
+var reasonDiscouraging = (declarer, otherId) => declarer?.discourages.find((relation) => relation.skillId === otherId)?.reason;
 var createSelectionSemantics = (catalog) => {
   const skillsById = new Map(catalog.skills.map((skill) => [skill.id, skill]));
   const conflictIdsBySkill = /* @__PURE__ */ new Map();
@@ -10610,12 +10592,8 @@ var createSelectionSemantics = (catalog) => {
   const discourageAgainst = (skill, sets) => {
     for (const reachedId of sets.reached) {
       if (reachedId === skill.id) continue;
-      const declared = skillsById.get(reachedId)?.discourages.find((relation) => relation.skillId === skill.id);
-      if (declared) return declared.reason;
-      const declaredBack = skill.discourages.find(
-        (relation) => relation.skillId === reachedId
-      );
-      if (declaredBack) return declaredBack.reason;
+      const reason = reasonDiscouraging(skillsById.get(reachedId), skill.id) ?? reasonDiscouraging(skill, reachedId);
+      if (reason !== void 0) return reason;
     }
     return void 0;
   };
@@ -10700,20 +10678,16 @@ var seedScopeSchema = z3.enum(["project", "global"]);
 var seedSkillSchema = z3.object({
   install: z3.enum(["plugin", "eject"]),
   scope: seedScopeSchema,
-  // Sub-agent id → load state; presence is assignment. Per (agent, skill),
-  // matching the granularity of the CLI's stack config.
+  // Sub-agent id → load state; presence is assignment.
   assignments: z3.record(z3.string(), seedLoadStateSchema)
 });
 var seedAgentSchema = z3.object({
   on: z3.boolean().exactOptional(),
   model: seedModelSchema.exactOptional(),
-  // A string rather than the CLI's enum: Codex retires ids, and an enum would
-  // turn each retirement into a version bump. Each consumer drops an id it does
-  // not offer.
+  // Not an enum: Codex retires ids; consumers drop ones they do not offer.
   codexModel: z3.string().min(1).exactOptional(),
   effort: seedEffortSchema.exactOptional(),
-  // Absent means `DEFAULT_SELECTION_OPTIONS.scope`, so the resting choice never
-  // travels.
+  // Absent means `DEFAULT_SELECTION_OPTIONS.scope`.
   scope: seedScopeSchema.exactOptional()
 });
 var MAX_EXTERNAL_SKILL_BYTES = 262144;
@@ -10733,45 +10707,45 @@ var seedSkillTreeSchema = z3.record(z3.string().min(1), z3.string()).refine((fil
 var seedExternalSkillSchema = z3.object({
   displayName: z3.string().min(1),
   description: z3.string(),
-  // The placement the user confirmed at add time, never derived.
+  // Confirmed by the user at add time, never derived.
   categoryId: z3.string().min(1),
-  // GitHub's own `owner/name`.
+  // `owner/name`.
   repo: z3.string().min(1),
-  // The skill's DIRECTORY within it — `skills/docx`, never the SKILL.md.
+  // The skill's directory (`skills/docx`), not its SKILL.md.
   path: z3.string().min(1),
   files: seedSkillTreeSchema
 });
 var seedPayloadSchema = z3.object({
   v: z3.literal(SEED_VERSION),
-  // Diagnostics only. A mismatch with the consumer's matrix must not fail the
-  // decode — it explains why some ids were skipped.
+  // Diagnostics only; a mismatch must not fail the decode.
   matrixVersion: z3.string(),
   stackId: z3.string().nullable(),
-  // The sentence the sharer's config records about itself. It travels instead
-  // of a stack id, which would bring back the stack's whole roster on install.
+  // Travels instead of a stack id, which would reinstall the stack's whole roster.
   description: z3.string().exactOptional(),
-  // Where the skills are fetched from, as `--marketplace` takes it. Absent is
-  // the default public catalogue.
+  // As `--marketplace` takes it; absent is the public catalogue.
   marketplace: z3.string().exactOptional(),
-  // Sparse: presence is selection.
+  // Presence is selection.
   skills: z3.record(z3.string(), seedSkillSchema),
-  // Skill id → its whole directory, for ids no catalogue can resolve.
+  // For ids no catalogue can resolve.
   external: z3.record(z3.string(), seedExternalSkillSchema).exactOptional(),
-  // Sparse: an agent with nothing to say has no entry. Presence is not an
-  // install — assignments and `on: true` decide that.
+  // Presence is not an install; assignments and `on: true` decide that.
   agents: z3.record(z3.string(), seedAgentSchema)
 });
 var isSeedScopePairWritable = (skillScope, agentScope) => !(skillScope === "project" && agentScope === "global");
 var seedAgentScope = (entry) => entry?.scope ?? DEFAULT_SELECTION_OPTIONS.scope;
-var isSwitchedOff = (entry) => entry?.on === false;
 var isUnwritablePair = (agents, skillScope, agent) => {
   const entry = agents[agent];
-  if (isSwitchedOff(entry)) return false;
+  if (entry?.on === false) return false;
   return !isSeedScopePairWritable(skillScope, seedAgentScope(entry));
 };
 var unwritableSeedAssignments = (payload) => Object.entries(payload.skills).flatMap(
   ([skillId, skill]) => Object.keys(skill.assignments).filter((agent) => isUnwritablePair(payload.agents, skill.scope, agent)).map((agent) => ({ skillId, agent }))
 );
+var PATH_SEPARATOR = /[/\\]/;
+var isControlCode = (code) => code <= 31 || code >= 127 && code <= 159;
+var holdsAControl = (id) => Array.from(id, (character) => character.charCodeAt(0)).some(isControlCode);
+var namesOneDirectory = (id) => id !== "" && id !== "." && id !== ".." && !PATH_SEPARATOR.test(id) && !holdsAControl(id);
+var externalIdsNamingNoDirectory = (payload) => Object.keys(payload.external ?? {}).filter((id) => !namesOneDirectory(id));
 var installableSeedPayloadSchema = seedPayloadSchema.superRefine(
   (payload, ctx) => {
     for (const { skillId, agent } of unwritableSeedAssignments(payload)) {
@@ -10781,6 +10755,13 @@ var installableSeedPayloadSchema = seedPayloadSchema.superRefine(
         message: `a project-scoped skill has nowhere to be written on '${agent}', which rests at global scope`
       });
     }
+    for (const id of externalIdsNamingNoDirectory(payload)) {
+      ctx.addIssue({
+        code: "custom",
+        path: ["external", id],
+        message: "a carried skill's id is the directory it is written into, so it must name exactly one directory and hold no control character"
+      });
+    }
   }
 );
 
@@ -10791,8 +10772,7 @@ var matrixCategorySchema = z4.object({
   id: idSchema,
   displayName: z4.string(),
   description: z4.string(),
-  // Optional in the CLI's type. A category without one cannot be placed in the
-  // UI, so a consumer drops it rather than failing the whole catalogue.
+  // A category without one is dropped by consumers, not rejected.
   domain: z4.string().exactOptional(),
   exclusive: z4.boolean(),
   order: z4.number()
@@ -10811,14 +10791,9 @@ var matrixSkillSchema = z4.object({
   slug: z4.string(),
   displayName: z4.string(),
   description: z4.string(),
-  // The sentence a compiled sub-agent reads as its cue to load the skill, which
-  // the editor's preview renders verbatim. Optional because a marketplace's own
-  // skills may state none; omitting it here would strip it from every fetched
-  // `catalog.json` and make every marketplace look like it states none.
+  // A compiled sub-agent's cue to load the skill.
   usageGuidance: z4.string().exactOptional(),
-  // The SKILL.md description a compiled sub-agent's activation table shows —
-  // `description` is the short label. The preview renders this one so
-  // it shows what the install writes (CLI-898); absent on older catalogues.
+  // The SKILL.md description the activation table shows; `description` is the short label.
   activationDescription: z4.string().exactOptional(),
   category: idSchema,
   conflictsWith: z4.array(skillRelationSchema2),
@@ -10829,13 +10804,10 @@ var matrixStackSchema = z4.object({
   id: idSchema,
   name: z4.string(),
   description: z4.string(),
-  // agent id → category id → skill ids. A stack names the few agents it staffs
-  // rather than the whole roster, and the agent roster is as marketplace-owned
-  // as the skills are, so neither level is narrowed.
+  // agent id → category id → skill ids.
   skills: z4.record(z4.string(), z4.record(z4.string(), z4.array(idSchema))),
   allSkillIds: z4.array(idSchema),
   philosophy: z4.string(),
-  // Never populated by the CLI today; a stack rail groups by it when it appears.
   group: z4.string().exactOptional()
 });
 var matrixSchema = z4.object({
@@ -10848,37 +10820,20 @@ var matrixSchema = z4.object({
 // ../matrix/src/skill-index.ts
 import { z as z5 } from "zod";
 var skillIndexEntrySchema = z5.object({
-  // The skill's own name — its frontmatter `name` where there is one, its
-  // directory name where there is not. Never empty: a skill with neither is
-  // not addressable and does not make it into the index.
+  // Frontmatter `name`, else the directory name.
   name: z5.string().min(1),
-  // Frontmatter `description`, or the SKILL.md's first heading. Empty when the
-  // file offers neither, which is a thin result rather than an invalid one.
+  // Frontmatter `description`, else the first heading; may be empty.
   description: z5.string(),
-  // GitHub's own `owner/name`, so a result can be attributed and fetched
-  // without a second lookup.
+  // `owner/name`.
   repo: z5.string().min(1),
-  // The skill DIRECTORY within that repo — `skills/docx`, not
-  // `skills/docx/SKILL.md`. A skill is a directory in every consumer of this
-  // index, and the file is only how the directory proves itself.
+  // The skill's directory (`skills/docx`), not its SKILL.md.
   path: z5.string().min(1),
-  // The repository's star count when the index was built. A property of the
-  // repo rather than the skill, repeated per entry because the dialog ranks
-  // and badges individual results.
+  // The repo's stars, repeated per entry for ranking.
   stars: z5.number().int().nonnegative(),
-  // How much the skill's whole DIRECTORY weighs — the sum over its blobs of the
-  // sizes GitHub reported for them in the tree listing the crawl already made,
-  // never a download. Repeated per entry because the cap it is read against is
-  // per skill: `MAX_EXTERNAL_SKILL_BYTES` decides whether one directory can
-  // ride a shared link, and without this number that decision cannot be made
-  // until the consumer has listed the repository itself. Zero is a real answer,
-  // for a directory GitHub reported no sizes for at all.
+  // Directory size from GitHub's tree listing, checked against `MAX_EXTERNAL_SKILL_BYTES`.
   bytes: z5.number().int().nonnegative()
 });
 var skillIndexSchema = z5.object({
-  // When this index was assembled, as an ISO 8601 instant. The worker's own
-  // freshness check reads it, and it is served so a caller can say how old the
-  // list is rather than having to guess.
   builtAt: z5.iso.datetime(),
   skills: z5.array(skillIndexEntrySchema)
 });
@@ -10915,17 +10870,6 @@ function orderDomains(domains) {
   ];
 }
 
-// ../compile/src/string.ts
-function bytewise(a, b) {
-  return a < b ? -1 : a > b ? 1 : 0;
-}
-function inAnEnglishList(items) {
-  const last = items.at(-1);
-  if (last === void 0) return "";
-  if (items.length === 1) return last;
-  return `${items.slice(0, -1).join(", ")} and ${last}`;
-}
-
 // ../compile/src/selection.ts
 import { groupBy as groupBy2 } from "remeda";
 function skillOrThrow(catalog, id) {
@@ -10938,75 +10882,87 @@ function validateConflicts(catalog, selections) {
   return selections.flatMap((skillAId, index) => {
     const skillA = catalog.skills[skillAId];
     if (!skillA) return [];
-    return selections.slice(index + 1).flatMap((skillBId) => {
-      const conflict = (skillA.conflictsWith ?? []).find(
-        (c) => c.skillId === skillBId
-      );
-      if (!conflict) return [];
-      return [
-        {
-          type: "conflict",
-          message: `${skillA.displayName} conflicts with ${labelOf2(catalog, skillBId)}: ${conflict.reason}`,
-          // Boundary cast: the id came out of the catalogue's own relationship table.
-          skills: [skillA.id, skillBId]
-        }
-      ];
-    });
+    return selections.slice(index + 1).flatMap((skillBId) => declaredConflict(catalog, skillA, skillBId));
   });
 }
+function declaredConflict(catalog, skillA, skillBId) {
+  const conflict = (skillA.conflictsWith ?? []).find(
+    (c) => c.skillId === skillBId
+  );
+  if (!conflict) return [];
+  return [
+    {
+      type: "conflict",
+      message: `${skillA.displayName} conflicts with ${labelOf2(catalog, skillBId)}: ${conflict.reason}`,
+      // Boundary cast: the id came out of the catalogue's own relationship table.
+      skills: [skillA.id, skillBId]
+    }
+  ];
+}
 function validateRequirements(catalog, selections, selectedSet) {
   return selections.flatMap((skillId) => {
     const skill = catalog.skills[skillId];
     if (!skill) return [];
-    return (skill.requires ?? []).flatMap((requirement) => {
-      const requiredIds = requirement.skillIds;
-      if (requirement.needsAny) {
-        if (requiredIds.some((reqId) => selectedSet.has(reqId))) return [];
-        return [
-          {
-            type: "missingRequirement",
-            message: `${skill.displayName} requires one of: ${requiredIds.map((id) => labelOf2(catalog, id)).join(", ")}`,
-            skills: [skillId, ...requiredIds]
-          }
-        ];
-      }
-      const missingIds = requiredIds.filter((reqId) => !selectedSet.has(reqId));
-      if (missingIds.length === 0) return [];
-      return [
-        {
-          type: "missingRequirement",
-          message: `${skill.displayName} requires: ${missingIds.map((id) => labelOf2(catalog, id)).join(", ")}`,
-          skills: [skillId, ...missingIds]
-        }
-      ];
-    });
+    return (skill.requires ?? []).flatMap(
+      (requirement) => unmetRequirement(catalog, skillId, skill, requirement, selectedSet)
+    );
   });
 }
+function unmetRequirement(catalog, skillId, skill, requirement, selectedSet) {
+  const requiredIds = requirement.skillIds;
+  if (requirement.needsAny) {
+    if (requiredIds.some((reqId) => selectedSet.has(reqId))) return [];
+    return [
+      {
+        type: "missingRequirement",
+        message: `${skill.displayName} requires one of: ${labelsOf(catalog, requiredIds)}`,
+        skills: [skillId, ...requiredIds]
+      }
+    ];
+  }
+  const missingIds = requiredIds.filter((reqId) => !selectedSet.has(reqId));
+  if (missingIds.length === 0) return [];
+  return [
+    {
+      type: "missingRequirement",
+      message: `${skill.displayName} requires: ${labelsOf(catalog, missingIds)}`,
+      skills: [skillId, ...missingIds]
+    }
+  ];
+}
+function labelsOf(catalog, ids) {
+  return ids.map((id) => labelOf2(catalog, id)).join(", ");
+}
 function validateExclusivity(catalog, selections) {
   const resolved = selections.map((skillId) => ({ skillId, skill: catalog.skills[skillId] })).filter(
     (entry) => entry.skill != null
   );
   return Object.entries(
     groupBy2(resolved, (entry) => entry.skill.category)
-  ).flatMap(([categoryId, entries]) => {
-    if (entries.length <= 1) return [];
-    if (categoryId === LOCAL_PSEUDO_CATEGORY) return [];
-    const category = catalog.categories[categoryId];
-    if (category?.exclusive !== true) return [];
-    const skillIds = entries.map((e) => e.skillId);
-    return [
-      {
-        type: "categoryExclusive",
-        message: `Category "${category.displayName}" only allows one selection, but multiple selected: ${skillIds.map((id) => labelOf2(catalog, id)).join(", ")}`,
-        skills: skillIds
-      }
-    ];
-  });
+  ).flatMap(
+    ([categoryId, entries]) => exclusivityBreach(
+      catalog,
+      categoryId,
+      entries.map((e) => e.skillId)
+    )
+  );
+}
+function exclusivityBreach(catalog, categoryId, skillIds) {
+  if (skillIds.length <= 1) return [];
+  if (categoryId === LOCAL_PSEUDO_CATEGORY) return [];
+  const category = catalog.categories[categoryId];
+  if (category?.exclusive !== true) return [];
+  return [
+    {
+      type: "categoryExclusive",
+      message: `Category "${category.displayName}" only allows one selection, but multiple selected: ${labelsOf(catalog, skillIds)}`,
+      skills: skillIds
+    }
+  ];
 }
 function validateSelection(selections, catalog) {
   const resolvedSelections = selections.map(
-    // Boundary cast: the id came back off the catalogue entry `id` just found, and a catalogue
-    // skill's `id` is typed `string` on the narrow shape so the wire `Matrix` satisfies it too.
+    // Boundary cast: a catalogue id, typed `string` so the wire `Matrix` fits the shape.
     (id) => skillOrThrow(catalog, id).id
   );
   const selectedSet = new Set(resolvedSelections);
@@ -11025,11 +10981,13 @@ function sourceDirName(provider) {
 }
 
 // src/cli/consts.ts
+import os from "os";
+import path from "path";
+import { fileURLToPath } from "url";
 var __filename2 = fileURLToPath(import.meta.url);
 var __dirname2 = path.dirname(__filename2);
 var isInDist = __dirname2.includes("/dist");
-var CLI_ROOT = isInDist ? path.resolve(__dirname2, "..") : path.resolve(__dirname2, "../..");
-var PROJECT_ROOT = CLI_ROOT;
+var PROJECT_ROOT = isInDist ? path.resolve(__dirname2, "..") : path.resolve(__dirname2, "../..");
 var PLUGINS_SUBDIR = "plugins";
 var PLUGIN_MANIFEST_DIR = ".claude-plugin";
 var MARKETPLACE_JSON = "marketplace.json";
@@ -11045,7 +11003,8 @@ function globalInstallRoot() {
 function cacheRoot() {
   return path.join(os.homedir(), ".cache", DEFAULT_PLUGIN_NAME);
 }
-var EDITOR_URL = "https://agentsinc.sh/editor";
+var DEFAULT_EDITOR_URL = "https://agentsinc.sh/editor";
+var EDITOR_URL = process.env.AGENTS_INC_EDITOR_URL ?? DEFAULT_EDITOR_URL;
 function editorConfigUrl(id) {
   return `${EDITOR_URL}/?fromId=${encodeURIComponent(id)}`;
 }
@@ -11066,7 +11025,7 @@ var SCHEMA_PATHS = {
 var YAML_FORMATTING = {
   INDENT: 2,
   LINE_WIDTH: 120,
-  /** lineWidth: 0 disables wrapping — used for metadata files */
+  /** 0 disables wrapping. */
   LINE_WIDTH_NONE: 0
 };
 var CHECK_GLYPH = "\u2713";
@@ -11087,9 +11046,7 @@ var UI_SYMBOLS = {
   SCROLL_DOWN: "\u25BC",
   CHECK: CHECK_GLYPH,
   CROSS: "\u2717",
-  /** Removed/pending-removal diff marker (ASCII hyphen). */
   REMOVED: "-",
-  /** Added/newly-selected diff marker (ASCII plus). */
   ADDED: "+"
 };
 var GITHUB_SOURCE = {
@@ -11109,7 +11066,6 @@ var MAX_CONFIG_FILE_SIZE = ONE_MB;
 var MAX_JSON_NESTING_DEPTH = 10;
 var MAX_MARKETPLACE_PLUGINS = 1e4;
 var SCROLL_VIEWPORT = {
-  /** Height of the "N more above" scroll indicator */
   SCROLL_INDICATOR_HEIGHT: 1
 };
 var MIN_TERMINAL_SIZE = {
@@ -11167,6 +11123,7 @@ export {
   seedPayloadSchema,
   isSeedScopePairWritable,
   seedAgentScope,
+  externalIdsNamingNoDirectory,
   installableSeedPayloadSchema,
   seatCatalog,
   seatedCatalog,
@@ -11182,6 +11139,7 @@ export {
   LOCAL_SKILLS_PATH,
   DEFAULT_PLUGIN_NAME,
   DEFAULT_PUBLIC_SOURCE_NAME,
+  GITHUB_PREFIX,
   DEFAULT_SOURCE,
   CLI_INVOKE_COMMAND,
   EJECT_SOURCE,
@@ -11241,4 +11199,4 @@ export {
   CLI_COLORS,
   ASCII_LOGO
 };
-//# sourceMappingURL=chunk-PJWGTNOX.js.map
+//# sourceMappingURL=chunk-7C3PFRBU.js.map