@evomap/evolver 1.29.8 → 1.30.2

package/src/gep/skillPublisher.js

@@ -3,54 +3,90 @@
 var { getHubUrl, buildHubHeaders, getNodeId } = require('./a2aProtocol');
 
 /**
- * Convert a Gene object into SKILL.md format (Claude/Anthropic style).
- *
- * @param {object} gene - Gene asset
- * @returns {string} SKILL.md content
+ * Sanitize a raw gene id into a human-readable kebab-case skill name.
+ * Returns null if the name is unsalvageable (pure numbers, tool name, etc.).
  */
 function sanitizeSkillName(rawName) {
   var name = rawName.replace(/[\r\n]+/g, '-').replace(/^gene_distilled_/, '').replace(/^gene_/, '').replace(/_/g, '-');
+  // Strip ALL embedded timestamps (10+ digit sequences) anywhere in the name
+  name = name.replace(/-?\d{10,}-?/g, '-').replace(/-+/g, '-').replace(/^-|-$/g, '');
   if (/^\d{8,}/.test(name) || /^(cursor|vscode|vim|emacs|windsurf|copilot|cline|codex)[-]?\d*$/i.test(name)) {
     return null;
   }
-  name = name.replace(/-?\d{10,}$/g, '').replace(/-+$/, '');
   if (name.replace(/[-]/g, '').length < 6) return null;
   return name;
 }
 
-function geneToSkillMd(gene) {
-  var rawName = gene.id || 'unnamed-skill';
-  var name = sanitizeSkillName(rawName);
-  if (!name) {
-    var fallbackWords = [];
-    if (Array.isArray(gene.signals_match)) {
-      gene.signals_match.slice(0, 3).forEach(function (s) {
-        String(s).toLowerCase().replace(/[^a-z0-9]+/g, ' ').trim().split(/\s+/).forEach(function (w) {
-          if (w.length >= 3 && fallbackWords.length < 5) fallbackWords.push(w);
-        });
-      });
-    }
-    if (fallbackWords.length < 2 && gene.summary) {
-      String(gene.summary).toLowerCase().replace(/[^a-z0-9]+/g, ' ').trim().split(/\s+/).forEach(function (w) {
-        if (w.length >= 3 && fallbackWords.length < 5) fallbackWords.push(w);
+/**
+ * Derive a Title Case display name from a kebab-case skill name.
+ * "retry-with-backoff" -> "Retry With Backoff"
+ */
+function toTitleCase(kebabName) {
+  return kebabName.split('-').map(function (w) {
+    if (!w) return '';
+    return w.charAt(0).toUpperCase() + w.slice(1);
+  }).join(' ');
+}
+
+/**
+ * Derive fallback name words from gene signals/summary when id is not usable.
+ */
+function deriveFallbackName(gene) {
+  var fallbackWords = [];
+  var STOP = new Set(['the', 'and', 'for', 'with', 'from', 'that', 'this', 'into', 'when', 'are', 'was', 'has', 'had', 'not', 'but', 'its']);
+  if (Array.isArray(gene.signals_match)) {
+    gene.signals_match.slice(0, 3).forEach(function (s) {
+      String(s).toLowerCase().replace(/[^a-z0-9]+/g, ' ').trim().split(/\s+/).forEach(function (w) {
+        if (w.length >= 3 && !STOP.has(w) && fallbackWords.length < 5) fallbackWords.push(w);
       });
-    }
-    var seen = {};
-    fallbackWords = fallbackWords.filter(function (w) { if (seen[w]) return false; seen[w] = true; return true; });
-    name = fallbackWords.length >= 2 ? fallbackWords.join('-') : 'auto-distilled-skill';
+    });
+  }
+  if (fallbackWords.length < 2 && gene.summary) {
+    String(gene.summary).toLowerCase().replace(/[^a-z0-9]+/g, ' ').trim().split(/\s+/).forEach(function (w) {
+      if (w.length >= 3 && !STOP.has(w) && fallbackWords.length < 5) fallbackWords.push(w);
+    });
   }
-  var desc = (gene.summary || 'AI agent skill distilled from evolution experience.').replace(/[\r\n]+/g, ' ').trim();
+  var seen = {};
+  fallbackWords = fallbackWords.filter(function (w) { if (seen[w]) return false; seen[w] = true; return true; });
+  return fallbackWords.length >= 2 ? fallbackWords.join('-') : 'auto-distilled-skill';
+}
+
+/**
+ * Convert a Gene object into SKILL.md format -- marketplace-quality content.
+ *
+ * @param {object} gene - Gene asset
+ * @returns {string} SKILL.md content
+ */
+function geneToSkillMd(gene) {
+  var rawName = gene.id || 'unnamed-skill';
+  var name = sanitizeSkillName(rawName) || deriveFallbackName(gene);
+  var displayName = toTitleCase(name);
+  var desc = (gene.summary || '').replace(/[\r\n]+/g, ' ').replace(/\s*\d{10,}\s*$/g, '').trim();
+  if (!desc || desc.length < 10) desc = 'AI agent skill distilled from evolution experience.';
 
   var lines = [
     '---',
-    'name: ' + name,
+    'name: ' + displayName,
     'description: ' + desc,
     '---',
     '',
-    '# ' + name,
+    '# ' + displayName,
+    '',
+    desc,
     '',
   ];
 
+  // -- When to Use (derived from signals; preconditions go in their own section) --
+  if (gene.signals_match && gene.signals_match.length > 0) {
+    lines.push('## When to Use');
+    lines.push('');
+    lines.push('- When your project encounters: ' + gene.signals_match.slice(0, 4).map(function (s) {
+      return '`' + s + '`';
+    }).join(', '));
+    lines.push('');
+  }
+
+  // -- Trigger Signals --
   if (gene.signals_match && gene.signals_match.length > 0) {
     lines.push('## Trigger Signals');
     lines.push('');
@@ -60,6 +96,7 @@ function geneToSkillMd(gene) {
     lines.push('');
   }
 
+  // -- Preconditions --
   if (gene.preconditions && gene.preconditions.length > 0) {
     lines.push('## Preconditions');
     lines.push('');
@@ -69,27 +106,36 @@ function geneToSkillMd(gene) {
     lines.push('');
   }
 
+  // -- Strategy --
   if (gene.strategy && gene.strategy.length > 0) {
     lines.push('## Strategy');
     lines.push('');
     gene.strategy.forEach(function (step, i) {
-      lines.push((i + 1) + '. ' + step);
+      var text = String(step);
+      var verb = extractStepVerb(text);
+      if (verb) {
+        lines.push((i + 1) + '. **' + verb + '** -- ' + stripLeadingVerb(text));
+      } else {
+        lines.push((i + 1) + '. ' + text);
+      }
     });
     lines.push('');
   }
 
+  // -- Constraints --
   if (gene.constraints) {
     lines.push('## Constraints');
     lines.push('');
     if (gene.constraints.max_files) {
-      lines.push('- Max files: ' + gene.constraints.max_files);
+      lines.push('- Max files per invocation: ' + gene.constraints.max_files);
     }
     if (gene.constraints.forbidden_paths && gene.constraints.forbidden_paths.length > 0) {
-      lines.push('- Forbidden paths: ' + gene.constraints.forbidden_paths.join(', '));
+      lines.push('- Forbidden paths: ' + gene.constraints.forbidden_paths.map(function (p) { return '`' + p + '`'; }).join(', '));
     }
     lines.push('');
   }
 
+  // -- Validation --
   if (gene.validation && gene.validation.length > 0) {
     lines.push('## Validation');
     lines.push('');
@@ -101,6 +147,16 @@ function geneToSkillMd(gene) {
     });
   }
 
+  // -- Metadata --
+  lines.push('## Metadata');
+  lines.push('');
+  lines.push('- Category: `' + (gene.category || 'innovate') + '`');
+  lines.push('- Schema version: `' + (gene.schema_version || '1.6.0') + '`');
+  if (gene._distilled_meta && gene._distilled_meta.source_capsule_count) {
+    lines.push('- Distilled from: ' + gene._distilled_meta.source_capsule_count + ' successful capsules');
+  }
+  lines.push('');
+
   lines.push('---');
   lines.push('');
   lines.push('*This Skill was generated by [Evolver](https://github.com/autogame-17/evolver) and is distributed under the [EvoMap Skill License (ESL-1.0)](https://evomap.ai/terms). Unauthorized redistribution, bulk scraping, or republishing is prohibited. See LICENSE file for full terms.*');
@@ -109,6 +165,31 @@ function geneToSkillMd(gene) {
   return lines.join('\n');
 }
 
+/**
+ * Extract the leading verb from a strategy step for bolding.
+ * Only extracts a single verb to avoid splitting compound phrases.
+ * e.g. "Verify Cursor CLI installation" -> "Verify"
+ *      "Run `npm test` to check" -> "Run"
+ *      "Configure non-interactive mode" -> "Configure"
+ */
+function extractStepVerb(step) {
+  // Only match a capitalized verb at the very start (no leading backtick/special chars)
+  var match = step.match(/^([A-Z][a-z]+)/);
+  return match ? match[1] : '';
+}
+
+/**
+ * Remove the leading verb from a step (already shown in bold).
+ */
+function stripLeadingVerb(step) {
+  var verb = extractStepVerb(step);
+  if (verb && step.startsWith(verb)) {
+    var rest = step.slice(verb.length).replace(/^[\s:.\-]+/, '');
+    return rest || step;
+  }
+  return step;
+}
+
 /**
  * Publish a Gene as a Skill to the Hub skill store.
  *
@@ -121,18 +202,37 @@ function publishSkillToHub(gene, opts) {
   var hubUrl = getHubUrl();
   if (!hubUrl) return Promise.resolve({ ok: false, error: 'no_hub_url' });
 
-  var content = geneToSkillMd(gene);
+  // Shallow-copy gene to avoid mutating the caller's object
+  var geneCopy = {};
+  Object.keys(gene).forEach(function (k) { geneCopy[k] = gene[k]; });
+  if (Array.isArray(geneCopy.signals_match)) {
+    try {
+      var distiller = require('./skillDistiller');
+      geneCopy.signals_match = distiller.sanitizeSignalsMatch(geneCopy.signals_match);
+    } catch (e) { /* distiller not available, skip */ }
+  }
+
+  var content = geneToSkillMd(geneCopy);
   var nodeId = getNodeId();
   var fmName = content.match(/^name:\s*(.+)$/m);
   var derivedName = fmName ? fmName[1].trim().toLowerCase().replace(/[^a-z0-9]+/g, '_') : (gene.id || 'unnamed').replace(/^gene_/, '');
+  // Strip ALL embedded timestamps from skillId
+  derivedName = derivedName.replace(/_?\d{10,}_?/g, '_').replace(/_+/g, '_').replace(/^_|_$/g, '');
   var skillId = 'skill_' + derivedName;
 
+  // Clean tags: use already-sanitized signals from geneCopy
+  var tags = opts.tags || geneCopy.signals_match || [];
+  tags = tags.filter(function (t) {
+    var s = String(t || '').trim();
+    return s.length >= 3 && !/^\d+$/.test(s) && !/\d{10,}/.test(s);
+  });
+
   var body = {
     sender_id: nodeId,
     skill_id: skillId,
     content: content,
-    category: opts.category || gene.category || null,
-    tags: opts.tags || gene.signals_match || [],
+    category: opts.category || geneCopy.category || null,
+    tags: tags,
   };
 
   var endpoint = hubUrl.replace(/\/+$/, '') + '/a2a/skill/store/publish';
@@ -165,12 +265,18 @@ function updateSkillOnHub(nodeId, skillId, content, opts, gene) {
   var hubUrl = getHubUrl();
   if (!hubUrl) return Promise.resolve({ ok: false, error: 'no_hub_url' });
 
+  var tags = opts.tags || gene.signals_match || [];
+  tags = tags.filter(function (t) {
+    var s = String(t || '').trim();
+    return s.length >= 3 && !/^\d+$/.test(s) && !/\d{10,}/.test(s);
+  });
+
   var body = {
     sender_id: nodeId,
     skill_id: skillId,
     content: content,
     category: opts.category || gene.category || null,
-    tags: opts.tags || gene.signals_match || [],
+    tags: tags,
     changelog: 'Iterative evolution update',
   };
 
@@ -196,4 +302,6 @@ module.exports = {
   geneToSkillMd: geneToSkillMd,
   publishSkillToHub: publishSkillToHub,
   updateSkillOnHub: updateSkillOnHub,
+  sanitizeSkillName: sanitizeSkillName,
+  toTitleCase: toTitleCase,
 };

package/src/gep/solidify.js

@@ -20,6 +20,7 @@ const { buildValidationReport } = require('./validationReport');
 const { logAssetCall } = require('./assetCallLog');
 const { recordNarrative } = require('./narrativeMemory');
 const { isLlmReviewEnabled, runLlmReview } = require('./llmReview');
+const { buildExecutionTrace } = require('./executionTrace');
 
 function nowIso() {
   return new Date().toISOString();
@@ -1225,6 +1226,22 @@ function solidify({ intent, summary, dryRun = false, rollbackOnFailure = true }
       memory_graph: memoryGraphPath(),
     },
   };
+  // Build desensitized execution trace for cross-agent experience sharing
+  const executionTrace = buildExecutionTrace({
+    gene: geneUsed,
+    mutation,
+    signals,
+    blast,
+    constraintCheck,
+    validation,
+    canary,
+    outcomeStatus,
+    startedAt: validation.startedAt,
+  });
+  if (executionTrace) {
+    event.execution_trace = executionTrace;
+  }
+
   event.asset_id = computeAssetId(event);
 
   let capsule = null;
@@ -1350,7 +1367,10 @@ function solidify({ intent, summary, dryRun = false, rollbackOnFailure = true }
   state.last_solidify = {
     run_id: runId, at: ts, event_id: event.id, capsule_id: capsuleId, outcome: event.outcome,
   };
-  if (!dryRun) writeStateForSolidify(state);
+  if (!dryRun) {
+    state.solidify_count = (state.solidify_count || 0) + 1;
+    writeStateForSolidify(state);
+  }
 
   if (!dryRun) {
     try {