agents-inc 0.161.0 → 0.162.0

package/dist/{chunk-YSQ76LWG.js → chunk-UNASGGW2.js}

@@ -10,7 +10,7 @@ import {
   typedFromEntries,
   typedKeys,
   typedValues
-} from "./chunk-BLGETWYI.js";
+} from "./chunk-3QBOVV6A.js";
 import {
   AGENT_NAMES,
   AUTHOR_HANDLE_PATTERN,
@@ -80,7 +80,7 @@ import {
   seatedCatalog,
   seedAgentScope,
   validateSelection
-} from "./chunk-AMZO4YJK.js";
+} from "./chunk-IV42Z63R.js";
 import {
   defaultCategories,
   defaultRules,
@@ -119,9 +119,31 @@ var LOCAL_DEFAULTS = {
 var SKILL_CONTENT_FILES = [STANDARD_FILES.SKILL_MD, STANDARD_FILES.REFERENCE_MD];
 var SKILL_CONTENT_DIRS = [STANDARD_DIRS.EXAMPLES, STANDARD_DIRS.SCRIPTS];
 
+// src/cli/utils/string.ts
+var OSC = /\u001B\][\s\S]*?(?:\u0007|\u001B\\|$)/;
+var CSI = /\u001B\[[0-?]*[ -/]*[@-~]?/;
+var OTHER_ESCAPE = /\u001B.?/;
+var C1 = /[\u0080-\u009F]/;
+var C0 = /[\u0000-\u0008\u000B-\u001F\u007F]/;
+function anyOf(patterns) {
+  return new RegExp(patterns.map((pattern) => pattern.source).join("|"), "g");
+}
+var TERMINAL_CONTROLS = anyOf([OSC, CSI, OTHER_ESCAPE, C1, C0]);
+function stripTerminalControls(text) {
+  return text.replace(TERMINAL_CONTROLS, "");
+}
+function truncateText(text, maxLength) {
+  const inert = stripTerminalControls(text);
+  if (inert.length <= maxLength) return inert;
+  return inert.slice(0, maxLength - 1) + "\u2026";
+}
+function toTitleCase(kebabCase) {
+  return kebabCase.split("-").map((word) => word.charAt(0).toUpperCase() + word.slice(1)).join(" ");
+}
+
 // src/cli/utils/errors.ts
 function getErrorMessage(error) {
-  return error instanceof Error ? error.message : String(error);
+  return stripTerminalControls(error instanceof Error ? error.message : String(error));
 }
 
 // src/cli/utils/fs.ts
@@ -1245,11 +1267,11 @@ import { partition } from "remeda";
 // src/cli/lib/schema-validator.ts
 import "zod";
 function formatZodIssue(issue) {
-  const path25 = issue.path.join(".");
+  const path25 = stripTerminalControls(issue.path.join("."));
   if (issue.code === "unrecognized_keys") {
-    return `Unrecognized key: "${issue.keys.join('", "')}"`;
+    return `Unrecognized key: "${stripTerminalControls(issue.keys.join('", "'))}"`;
   }
-  return path25 ? `${path25}: ${issue.message}` : issue.message;
+  return path25 ? `${path25}: ${stripTerminalControls(issue.message)}` : stripTerminalControls(issue.message);
 }
 function formatZodErrors(error) {
   return error.issues.map(formatZodIssue);
@@ -6076,17 +6098,6 @@ async function extractAllSkills(skillsDir) {
 
 // src/cli/lib/matrix/skill-resolution.ts
 import { unique as unique5, uniqueBy as uniqueBy3 } from "remeda";
-
-// src/cli/utils/string.ts
-function truncateText(text, maxLength) {
-  if (text.length <= maxLength) return text;
-  return text.slice(0, maxLength - 1) + "\u2026";
-}
-function toTitleCase(kebabCase) {
-  return kebabCase.split("-").map((word) => word.charAt(0).toUpperCase() + word.slice(1)).join(" ");
-}
-
-// src/cli/lib/matrix/skill-resolution.ts
 var AUTO_SYNTH_ORDER = 999;
 function warnOnDomainDisagreement(skill, decidedBy) {
   if (skill.domain === decidedBy.domain) return;
@@ -6290,7 +6301,7 @@ function buildResolvedSkill(skill, _categories, relationships, slugMap) {
 // src/cli/lib/matrix/matrix-resolver.ts
 import { groupBy as groupBy4 } from "remeda";
 function getLabel(skill) {
-  return skill.displayName;
+  return stripTerminalControls(skill.displayName);
 }
 function joinWithConjunction(items, conjunction) {
   if (items.length <= 1) return items[0] ?? "";
@@ -6310,7 +6321,11 @@ function toSelectionFacts(source) {
       conflictsWith: skill.conflictsWith.map((relation) => relation.skillId),
       discourages: skill.discourages.map((relation) => ({
         skillId: relation.skillId,
-        reason: relation.reason
+        // 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,
@@ -6366,10 +6381,10 @@ function getUnmetRequiredBy(skillId, currentSelections) {
       if (!req.skillIds.includes(skillId)) continue;
       if (req.needsAny) {
         if (!req.skillIds.some((id) => selectedSet.has(id))) {
-          return selectedSkill.displayName;
+          return getLabel(selectedSkill);
         }
       } else {
-        return selectedSkill.displayName;
+        return getLabel(selectedSkill);
       }
     }
   }
@@ -6392,7 +6407,7 @@ function getUnmetRequirementsReason(skillId, currentSelections) {
     if (missing.length === 0) continue;
     const names = missing.map((id) => {
       const s = matrix.skills[id];
-      return s ? getLabel(s) : id;
+      return s ? getLabel(s) : stripTerminalControls(id);
     });
     return withAuthorsReason(namesTheMissing(names, requirement.needsAny), requirement.reason);
   }
@@ -6422,7 +6437,10 @@ function validateConflicts(resolvedSelections) {
       if (conflict) {
         errors.push({
           type: "conflict",
-          message: `${getLabel(skillA)} conflicts with ${getLabel(getSkillById(skillBId))}: ${conflict.reason}`,
+          // 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]
         });
       }
@@ -6471,7 +6489,7 @@ function validateExclusivity(resolvedSelections) {
       if (category?.exclusive) {
         errors.push({
           type: "categoryExclusive",
-          message: `Category "${category.displayName}" only allows one selection, but multiple selected: ${skillIds.map((id) => getLabel(getSkillById(id))).join(", ")}`,
+          message: `Category "${stripTerminalControls(category.displayName)}" only allows one selection, but multiple selected: ${skillIds.map((id) => getLabel(getSkillById(id))).join(", ")}`,
           skills: skillIds
         });
       }
@@ -8519,6 +8537,9 @@ function mergeLocalSkillsIntoMatrix(matrix2, localResult) {
 }
 
 export {
+  stripTerminalControls,
+  truncateText,
+  toTitleCase,
   getErrorMessage,
   setVerbose,
   verbose,
@@ -8699,8 +8720,6 @@ export {
   unresolvedSkillRemovalReasons,
   deleteLocalSkill,
   migrateLocalSkillScope,
-  truncateText,
-  toTitleCase,
   claimSlug,
   getUnmetRequiredBy,
   getCellState,
@@ -8731,4 +8750,4 @@ export {
   detectMigrations,
   executeMigration
 };
-//# sourceMappingURL=chunk-YSQ76LWG.js.map
+//# sourceMappingURL=chunk-UNASGGW2.js.map