@evomap/evolver 1.29.4 → 1.29.9

package/SKILL.md

@@ -2,6 +2,108 @@
 name: capability-evolver
 description: A self-evolution engine for AI agents. Analyzes runtime history to identify improvements and applies protocol-constrained evolution.
 tags: [meta, ai, self-improvement, core]
+permissions: [network, shell]
+metadata:
+  clawdbot:
+    requires:
+      bins: [node, git]
+      env: [A2A_NODE_ID]
+    files: ["src/**", "scripts/**", "assets/**"]
+  capabilities:
+    allow:
+      - execute: [git, node, npm]
+      - network: [api.github.com, evomap.ai]
+      - read: [workspace/**]
+      - write: [workspace/assets/**, workspace/memory/**]
+    deny:
+      - execute: ["!git", "!node", "!npm", "!ps", "!pgrep", "!df"]
+      - network: ["!api.github.com", "!*.evomap.ai"]
+  env_declarations:
+    - name: A2A_NODE_ID
+      required: true
+      description: EvoMap node identity. Set after node registration.
+    - name: A2A_HUB_URL
+      required: false
+      default: https://evomap.ai
+      description: EvoMap Hub API base URL.
+    - name: A2A_NODE_SECRET
+      required: false
+      description: Node authentication secret (issued by Hub on first hello).
+    - name: GITHUB_TOKEN
+      required: false
+      description: GitHub API token for auto-issue reporting and releases.
+    - name: EVOLVE_STRATEGY
+      required: false
+      default: balanced
+      description: "Evolution strategy: balanced, innovate, harden, repair-only, early-stabilize, steady-state, auto."
+    - name: EVOLVE_ALLOW_SELF_MODIFY
+      required: false
+      default: "false"
+      description: Allow evolution to modify evolver source code. NOT recommended.
+    - name: EVOLVE_LOAD_MAX
+      required: false
+      default: "2.0"
+      description: Max 1-min load average before evolver backs off.
+    - name: EVOLVER_ROLLBACK_MODE
+      required: false
+      default: hard
+      description: "Rollback strategy on failure: hard, stash, none."
+    - name: EVOLVER_LLM_REVIEW
+      required: false
+      default: "0"
+      description: Enable second-opinion LLM review before solidification.
+    - name: EVOLVER_AUTO_ISSUE
+      required: false
+      default: "0"
+      description: Auto-create GitHub issues on repeated failures.
+    - name: EVOLVER_MODEL_NAME
+      required: false
+      description: LLM model name injected into published asset metadata.
+    - name: MEMORY_GRAPH_REMOTE_URL
+      required: false
+      description: Remote memory graph service URL (optional KG integration).
+    - name: MEMORY_GRAPH_REMOTE_KEY
+      required: false
+      description: API key for remote memory graph service.
+  network_endpoints:
+    - host: api.github.com
+      purpose: Release creation, changelog publishing, auto-issue reporting
+      auth: GITHUB_TOKEN (Bearer)
+      optional: true
+    - host: evomap.ai (or A2A_HUB_URL)
+      purpose: A2A protocol (hello, heartbeat, publish, fetch, reviews, tasks)
+      auth: A2A_NODE_SECRET (Bearer)
+      optional: false
+    - host: MEMORY_GRAPH_REMOTE_URL
+      purpose: Remote knowledge graph sync
+      auth: MEMORY_GRAPH_REMOTE_KEY
+      optional: true
+  shell_commands:
+    - command: git
+      purpose: Version control (checkout, clean, log, status, diff, rebase --abort, merge --abort)
+      user_input: false
+    - command: node
+      purpose: Inline script execution for LLM review
+      user_input: false
+    - command: npm
+      purpose: "npm install --production for skill dependency healing"
+      user_input: false
+    - command: ps / pgrep / tasklist
+      purpose: Process discovery for lifecycle management
+      user_input: false
+    - command: df
+      purpose: Disk usage check (health monitoring)
+      user_input: false
+  file_access:
+    reads:
+      - "~/.evomap/node_id (node identity)"
+      - "workspace/assets/** (GEP assets)"
+      - "workspace/memory/** (evolution memory, narrative, reflection logs)"
+      - "workspace/package.json (version info)"
+    writes:
+      - "workspace/assets/gep/** (genes, capsules, events)"
+      - "workspace/memory/** (memory graph, narrative, reflection)"
+      - "workspace/src/** (evolved code, only when changes are solidified)"
 ---
 
 # 🧬 Capability Evolver
@@ -59,13 +161,69 @@ Do not hardcode the node ID in scripts. `getNodeId()` in `src/gep/a2aProtocol.js
 
 ## Configuration
 
-| Environment Variable | Default | Description |
+### Required Environment Variables
+
+| Variable | Default | Description |
 |---|---|---|
-| `A2A_NODE_ID` | (required) | Your EvoMap node identity. Set this after node registration -- never hardcode it in scripts. Read automatically by `getNodeId()` in `a2aProtocol.js`. |
-| `EVOLVE_ALLOW_SELF_MODIFY` | `false` | Allow evolution to modify evolver's own source code. **NOT recommended for production.** Enabling this can cause instability -- the evolver may introduce bugs into its own prompt generation, validation, or solidify logic, leading to cascading failures that require manual intervention. Only enable for controlled experiments. |
-| `EVOLVE_LOAD_MAX` | `2.0` | Maximum 1-minute load average before evolver backs off. |
+| `A2A_NODE_ID` | (required) | Your EvoMap node identity. Set after node registration -- never hardcode in scripts. |
+
+### Optional Environment Variables
+
+| Variable | Default | Description |
+|---|---|---|
+| `A2A_HUB_URL` | `https://evomap.ai` | EvoMap Hub API base URL. |
+| `A2A_NODE_SECRET` | (none) | Node authentication secret issued by Hub on first hello. Stored locally after registration. |
 | `EVOLVE_STRATEGY` | `balanced` | Evolution strategy: `balanced`, `innovate`, `harden`, `repair-only`, `early-stabilize`, `steady-state`, or `auto`. |
-| `EVOLVER_ROLLBACK_MODE` | `hard` | Rollback strategy when evolution fails. `hard`: use `git reset --hard` (destructive, original behavior). `stash`: use `git stash` to preserve changes for recovery. `none`: skip rollback entirely. Use `stash` for safer operation in active workspaces. |
+| `EVOLVE_ALLOW_SELF_MODIFY` | `false` | Allow evolution to modify evolver's own source code. **NOT recommended for production.** |
+| `EVOLVE_LOAD_MAX` | `2.0` | Maximum 1-minute load average before evolver backs off. |
+| `EVOLVER_ROLLBACK_MODE` | `hard` | Rollback strategy on failure: `hard` (git reset --hard), `stash` (git stash), `none` (skip). Use `stash` for safer operation. |
+| `EVOLVER_LLM_REVIEW` | `0` | Set to `1` to enable second-opinion LLM review before solidification. |
+| `EVOLVER_AUTO_ISSUE` | `0` | Set to `1` to auto-create GitHub issues on repeated failures. Requires `GITHUB_TOKEN`. |
+| `EVOLVER_ISSUE_REPO` | (none) | GitHub repo for auto-issue reporting (e.g. `EvoMap/evolver`). |
+| `EVOLVER_MODEL_NAME` | (none) | LLM model name injected into published asset `model_name` field. |
+| `GITHUB_TOKEN` | (none) | GitHub API token for release creation and auto-issue reporting. Also accepts `GH_TOKEN` or `GITHUB_PAT`. |
+| `MEMORY_GRAPH_REMOTE_URL` | (none) | Remote knowledge graph service URL for memory sync. |
+| `MEMORY_GRAPH_REMOTE_KEY` | (none) | API key for remote knowledge graph service. |
+| `EVOLVE_REPORT_TOOL` | (auto) | Override report tool (e.g. `feishu-card`). |
+| `RANDOM_DRIFT` | `0` | Enable random drift in evolution strategy selection. |
+
+### Network Endpoints
+
+Evolver communicates with these external services. All are authenticated and documented.
+
+| Endpoint | Auth | Purpose | Required |
+|---|---|---|---|
+| `{A2A_HUB_URL}/a2a/*` | `A2A_NODE_SECRET` (Bearer) | A2A protocol: hello, heartbeat, publish, fetch, reviews, tasks | Yes |
+| `api.github.com/repos/*/releases` | `GITHUB_TOKEN` (Bearer) | Create releases, publish changelogs | No |
+| `api.github.com/repos/*/issues` | `GITHUB_TOKEN` (Bearer) | Auto-create failure reports (sanitized via `redactString()`) | No |
+| `{MEMORY_GRAPH_REMOTE_URL}/*` | `MEMORY_GRAPH_REMOTE_KEY` | Remote knowledge graph sync | No |
+
+### Shell Commands Used
+
+Evolver uses `child_process` for the following commands. No user-controlled input is passed to shell.
+
+| Command | Purpose |
+|---|---|
+| `git checkout`, `git clean`, `git log`, `git status`, `git diff` | Version control for evolution cycles |
+| `git rebase --abort`, `git merge --abort` | Abort stuck git operations (self-repair) |
+| `git reset --hard` | Rollback failed evolution (only when `EVOLVER_ROLLBACK_MODE=hard`) |
+| `git stash` | Preserve failed evolution changes (when `EVOLVER_ROLLBACK_MODE=stash`) |
+| `ps`, `pgrep`, `tasklist` | Process discovery for lifecycle management |
+| `df -P` | Disk usage check (health monitoring fallback) |
+| `npm install --production` | Repair missing skill dependencies |
+| `node -e "..."` | Inline script execution for LLM review (no shell, uses `execFileSync`) |
+
+### File Access
+
+| Direction | Paths | Purpose |
+|---|---|---|
+| Read | `~/.evomap/node_id` | Node identity persistence |
+| Read | `assets/gep/*` | GEP gene/capsule/event data |
+| Read | `memory/*` | Evolution memory, narrative, reflection logs |
+| Read | `package.json` | Version information |
+| Write | `assets/gep/*` | Updated genes, capsules, evolution events |
+| Write | `memory/*` | Memory graph, narrative log, reflection log |
+| Write | `src/**` | Evolved code (only during solidify, with git tracking) |
 
 ## GEP Protocol (Auditable Evolution)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@evomap/evolver",
-  "version": "1.29.4",
+  "version": "1.29.9",
   "description": "A GEP-powered self-evolution engine for AI agents. Features automated log analysis and Genome Evolution Protocol (GEP) for auditable, reusable evolution assets.",
   "main": "index.js",
   "bin": {

package/src/gep/a2aProtocol.js

@@ -469,12 +469,14 @@ function sendHelloToHub() {
 }
 
 function getHubNodeSecret() {
+  if (process.env.A2A_NODE_SECRET) return process.env.A2A_NODE_SECRET;
   if (_cachedHubNodeSecret) return _cachedHubNodeSecret;
   var persisted = _loadPersistedNodeSecret();
   if (persisted) {
     _cachedHubNodeSecret = persisted;
     return persisted;
   }
+  if (process.env.A2A_HUB_TOKEN) return process.env.A2A_HUB_TOKEN;
   return null;
 }
 

package/src/gep/prompt.js

@@ -134,10 +134,14 @@ ENSURE VALID JSON SYNTAX (escape quotes in strings).
 
 3. Gene (The Knowledge)
    - Reuse/update existing ID if possible. Create new only if novel pattern.
+   - ID MUST be descriptive: gene_<descriptive_name> (e.g., gene_retry_on_timeout)
+   - NEVER use timestamps, random numbers, or tool names (cursor, vscode, etc.) in IDs
+   - summary MUST be a clear human-readable sentence describing what the Gene does
    {
      "type": "Gene",
      "schema_version": "1.5.0",
-     "id": "gene_<name>",
+     "id": "gene_<descriptive_name>",
+     "summary": "<clear description of what this gene does>",
      "category": "repair|optimize|innovate",
      "signals_match": ["<pattern>"],
      "preconditions": ["<condition>"],
@@ -439,24 +443,33 @@ When creating a new skill in skills/<name>/:
    |- assets/           (optional: templates, data files)
    Creating an empty directory or a directory missing index.js = FAILED.
    Do NOT create unnecessary files (README.md, CHANGELOG.md, INSTALLATION_GUIDE.md, etc.).
-2. SKILL.MD FRONTMATTER: Every SKILL.md MUST start with YAML frontmatter:
+2. SKILL NAMING (CRITICAL):
+   a) <name> MUST be descriptive kebab-case (e.g., "log-rotation", "retry-handler", "cache-manager")
+   b) NEVER use timestamps, random numbers, tool names (cursor, vscode), or UUIDs as names
+   c) Names like "cursor-1773331925711", "skill-12345", "fix-1" = FAILED
+   d) Name must be 2-6 descriptive words separated by hyphens, conveying what the skill does
+   e) Good: "http-retry-with-backoff", "log-file-rotation", "config-validator"
+   f) Bad: "cursor-auto-1234", "new-skill", "test-skill", "my-skill"
+3. SKILL.MD FRONTMATTER: Every SKILL.md MUST start with YAML frontmatter:
    ---
    name: <skill-name>
    description: <what it does and when to use it>
    ---
+   The name MUST follow the naming rules above.
    The description is the triggering mechanism -- include WHAT the skill does and WHEN to use it.
-3. CONCISENESS: SKILL.md body should be under 500 lines. Keep instructions lean.
+   Description must be a clear, complete sentence (min 20 chars). Generic descriptions = FAILED.
+4. CONCISENESS: SKILL.md body should be under 500 lines. Keep instructions lean.
    Only include information the agent does not already know. Move detailed reference
    material to references/ files, not into SKILL.md itself.
-4. EXPORT VERIFICATION: Every exported function must be importable.
+5. EXPORT VERIFICATION: Every exported function must be importable.
    Run: node -e "const s = require('./skills/<name>'); console.log(Object.keys(s))"
    If this fails, the skill is broken. Fix before solidify.
-5. NO HARDCODED SECRETS: Never embed API keys, tokens, or secrets in code.
+6. NO HARDCODED SECRETS: Never embed API keys, tokens, or secrets in code.
    Use process.env or .env references. Hardcoded App ID, App Secret, Bearer tokens = FAILED.
-6. TEST BEFORE SOLIDIFY: Actually run the skill's core function to verify it works:
+7. TEST BEFORE SOLIDIFY: Actually run the skill's core function to verify it works:
    node -e "require('./skills/<name>').main ? require('./skills/<name>').main() : console.log('ok')"
    Scripts in scripts/ must also be tested by executing them.
-7. ATOMIC CREATION: Create ALL files for a skill in a single cycle.
+8. ATOMIC CREATION: Create ALL files for a skill in a single cycle.
    Do not create a directory in one cycle and fill it in the next.
    Empty directories from failed cycles will be automatically cleaned up on rollback.
 

package/src/gep/skillDistiller.js

@@ -223,18 +223,78 @@ function buildDistillationPrompt(analysis, existingGenes, sampleCapsules) {
   });
 
   return [
-    'You are a Gene synthesis engine for the GEP (Gene Expression Protocol).',
+    'You are a Gene synthesis engine for the GEP (Genome Evolution Protocol).',
+    'Your job is to distill successful evolution capsules into a high-quality, reusable Gene',
+    'that other AI agents can discover, fetch, and execute.',
     '',
-    'Analyze the following successful evolution capsules and extract a reusable Gene.',
+    '## OUTPUT FORMAT',
+    '',
+    'Output ONLY a single valid JSON object (no markdown fences, no explanation).',
+    '',
+    '## GENE ID RULES (CRITICAL)',
+    '',
+    '- The id MUST start with "' + DISTILLED_ID_PREFIX + '" followed by a descriptive kebab-case name.',
+    '- The suffix MUST describe the core capability in 3-6 hyphen-separated words.',
+    '- NEVER include timestamps, numeric IDs, random numbers, tool names (cursor, vscode, etc.), or UUIDs.',
+    '- Good: "gene_distilled_retry-with-exponential-backoff", "gene_distilled_database-migration-rollback"',
+    '- Bad: "gene_distilled_cursor-1773331925711", "gene_distilled_1234567890", "gene_distilled_fix-1"',
+    '',
+    '## SUMMARY RULES',
+    '',
+    '- The "summary" MUST be a clear, human-readable sentence (30-200 chars) describing',
+    '  WHAT capability this Gene provides and WHY it is useful.',
+    '- Write as if for a marketplace listing -- the summary is the first thing other agents see.',
+    '- Good: "Retry failed HTTP requests with exponential backoff, jitter, and circuit breaker to prevent cascade failures"',
+    '- Bad: "Distilled from capsules", "AI agent skill", "cursor automation", "1773331925711"',
+    '- NEVER include timestamps, build numbers, or tool names in the summary.',
+    '',
+    '## SIGNALS_MATCH RULES',
+    '',
+    '- Each signal MUST be a generic, reusable keyword that describes WHEN to trigger this Gene.',
+    '- Use lowercase_snake_case. Signals should be domain terms, not implementation artifacts.',
+    '- NEVER include timestamps, build numbers, tool names, session IDs, or random suffixes.',
+    '- Include 3-7 signals covering both the problem domain and the solution approach.',
+    '- Good: ["http_retry", "request_timeout", "exponential_backoff", "circuit_breaker", "resilience"]',
+    '- Bad: ["cursor_auto_1773331925711", "cli_headless_1773331925711", "bypass_123"]',
+    '',
+    '## STRATEGY RULES',
+    '',
+    '- Strategy steps MUST be actionable, concrete instructions an AI agent can execute.',
+    '- Each step should be a clear imperative sentence starting with a verb.',
+    '- Include 5-10 steps. Each step should be self-contained and specific.',
+    '- Do NOT describe what happened; describe what TO DO.',
+    '- Include rationale or context in parentheses when non-obvious.',
+    '- Where applicable, include inline code examples using backtick notation.',
+    '- Good: "Wrap the HTTP call in a retry loop with `maxRetries=3` and initial delay of 500ms"',
+    '- Bad: "Handle retries", "Fix the issue", "Improve reliability"',
+    '',
+    '## PRECONDITIONS RULES',
+    '',
+    '- List concrete, verifiable conditions that must be true before applying this Gene.',
+    '- Each precondition should be a testable statement, not a vague requirement.',
+    '- Good: "Project uses Node.js >= 18 with ES module support"',
+    '- Bad: "need to fix something"',
+    '',
+    '## CONSTRAINTS',
     '',
-    'RULES:',
-    '- Strategy steps MUST be actionable operations, NOT summaries',
-    '- Each step must be a concrete instruction an AI agent can execute',
-    '- Do NOT describe what happened; describe what TO DO next time',
-    '- The Gene MUST have a unique id starting with "' + DISTILLED_ID_PREFIX + '"',
     '- constraints.max_files MUST be <= ' + DISTILLED_MAX_FILES,
     '- constraints.forbidden_paths MUST include at least [".git", "node_modules"]',
-    '- Output valid Gene JSON only (no markdown, no explanation)',
+    '',
+    '## VALIDATION',
+    '',
+    '- Validation commands MUST start with "node ", "npm ", or "npx " (security constraint).',
+    '- Include commands that actually verify the Gene was applied correctly.',
+    '- Good: "npx tsc --noEmit", "npm test"',
+    '- Bad: "node -v" (proves nothing about the Gene)',
+    '',
+    '## QUALITY BAR',
+    '',
+    'Imagine this Gene will be published on a marketplace for thousands of AI agents.',
+    'It should be as professional and useful as a well-written library README.',
+    'Ask yourself: "Would another agent find this Gene by searching for the signals?',
+    'Would the summary make them want to fetch it? Would the strategy be enough to execute?"',
+    '',
+    '---',
     '',
     'SUCCESSFUL CAPSULES (grouped by pattern):',
     JSON.stringify(samples, null, 2),
@@ -246,7 +306,7 @@ function buildDistillationPrompt(analysis, existingGenes, sampleCapsules) {
     JSON.stringify(analysis, null, 2),
     '',
     'Output a single Gene JSON object with these fields:',
-    '{ "type": "Gene", "id": "gene_distilled_...", "category": "...", "signals_match": [...], "preconditions": [...], "strategy": [...], "constraints": { "max_files": N, "forbidden_paths": [...] }, "validation": [...] }',
+    '{ "type": "Gene", "id": "gene_distilled_<descriptive-kebab-name>", "summary": "<clear marketplace-quality description>", "category": "repair|optimize|innovate", "signals_match": ["generic_signal_1", ...], "preconditions": ["Concrete condition 1", ...], "strategy": ["Step 1: verb ...", "Step 2: verb ...", ...], "constraints": { "max_files": N, "forbidden_paths": [".git", "node_modules", ...] }, "validation": ["npx tsc --noEmit", ...], "schema_version": "1.6.0" }',
   ].join('\n');
 }
 
@@ -254,6 +314,64 @@ function distillRequestPath() {
   return path.join(paths.getMemoryDir(), 'distill_request.json');
 }
 
+// ---------------------------------------------------------------------------
+// Derive a descriptive ID from gene content when the LLM gives a bad name
+// ---------------------------------------------------------------------------
+function deriveDescriptiveId(gene) {
+  var words = [];
+  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 && words.length < 6) words.push(w);
+      });
+    });
+  }
+  if (words.length < 3 && gene.summary) {
+    var STOP = new Set(['the', 'and', 'for', 'with', 'from', 'that', 'this', 'into', 'when', 'are', 'was', 'has', 'had']);
+    String(gene.summary).toLowerCase().replace(/[^a-z0-9]+/g, ' ').trim().split(/\s+/).forEach(function (w) {
+      if (w.length >= 3 && !STOP.has(w) && words.length < 6) words.push(w);
+    });
+  }
+  if (words.length < 3 && Array.isArray(gene.strategy) && gene.strategy.length > 0) {
+    String(gene.strategy[0]).toLowerCase().replace(/[^a-z0-9]+/g, ' ').trim().split(/\s+/).forEach(function (w) {
+      if (w.length >= 3 && words.length < 6) words.push(w);
+    });
+  }
+  if (words.length < 2) words = ['auto', 'distilled', 'strategy'];
+  var unique = [];
+  var seen = new Set();
+  words.forEach(function (w) { if (!seen.has(w)) { seen.add(w); unique.push(w); } });
+  return DISTILLED_ID_PREFIX + unique.slice(0, 5).join('-');
+}
+
+// ---------------------------------------------------------------------------
+// Step 4: sanitizeSignalsMatch -- strip timestamps, random suffixes, tool names
+// ---------------------------------------------------------------------------
+function sanitizeSignalsMatch(signals) {
+  if (!Array.isArray(signals)) return [];
+  var cleaned = [];
+  signals.forEach(function (s) {
+    var sig = String(s || '').trim().toLowerCase();
+    if (!sig) return;
+    // Strip trailing timestamps (10+ digits) and random suffixes
+    sig = sig.replace(/[_-]\d{10,}$/g, '');
+    // Strip leading/trailing underscores/hyphens left over
+    sig = sig.replace(/^[_-]+|[_-]+$/g, '');
+    // Reject signals that are purely numeric
+    if (/^\d+$/.test(sig)) return;
+    // Reject signals that are just a tool name with optional number
+    if (/^(cursor|vscode|vim|emacs|windsurf|copilot|cline|codex|bypass|distill)[_-]?\d*$/i.test(sig)) return;
+    // Reject signals shorter than 3 chars after cleaning
+    if (sig.length < 3) return;
+    // Reject signals that still contain long numeric sequences (session IDs, etc.)
+    if (/\d{8,}/.test(sig)) return;
+    cleaned.push(sig);
+  });
+  // Deduplicate
+  var seen = {};
+  return cleaned.filter(function (s) { if (seen[s]) return false; seen[s] = true; return true; });
+}
+
 // ---------------------------------------------------------------------------
 // Step 4: validateSynthesizedGene
 // ---------------------------------------------------------------------------
@@ -267,10 +385,56 @@ function validateSynthesizedGene(gene, existingGenes) {
   if (!Array.isArray(gene.signals_match) || gene.signals_match.length === 0) errors.push('missing or empty signals_match');
   if (!Array.isArray(gene.strategy) || gene.strategy.length === 0) errors.push('missing or empty strategy');
 
+  // --- Signals sanitization (BEFORE id derivation so deriveDescriptiveId uses clean signals) ---
+  if (Array.isArray(gene.signals_match)) {
+    gene.signals_match = sanitizeSignalsMatch(gene.signals_match);
+    if (gene.signals_match.length === 0) {
+      errors.push('signals_match is empty after sanitization (all signals were invalid)');
+    }
+  }
+
+  // --- Summary sanitization (BEFORE id derivation so deriveDescriptiveId uses clean summary) ---
+  if (gene.summary) {
+    gene.summary = gene.summary.replace(/\s*\d{10,}\s*$/g, '').replace(/\.\s*\d{10,}/g, '.').trim();
+  }
+
+  // --- ID sanitization ---
   if (gene.id && !String(gene.id).startsWith(DISTILLED_ID_PREFIX)) {
     gene.id = DISTILLED_ID_PREFIX + String(gene.id).replace(/^gene_/, '');
   }
 
+  if (gene.id) {
+    var suffix = String(gene.id).replace(DISTILLED_ID_PREFIX, '');
+    // Strip ALL embedded timestamps (10+ digit sequences) anywhere in the id
+    suffix = suffix.replace(/[-_]?\d{10,}[-_]?/g, '-').replace(/[-_]+/g, '-').replace(/^[-_]+|[-_]+$/g, '');
+    var needsRename = /^\d+$/.test(suffix) || /^\d{10,}/.test(suffix)
+      || /^(cursor|vscode|vim|emacs|windsurf|copilot|cline|codex)[-_]?\d*$/i.test(suffix);
+    if (needsRename) {
+      gene.id = deriveDescriptiveId(gene);
+    } else {
+      gene.id = DISTILLED_ID_PREFIX + suffix;
+    }
+    var cleanSuffix = String(gene.id).replace(DISTILLED_ID_PREFIX, '');
+    if (cleanSuffix.replace(/[-_]/g, '').length < 6) {
+      gene.id = deriveDescriptiveId(gene);
+    }
+  }
+
+  // --- Summary fallback (summary was already sanitized above, this handles missing/short) ---
+  if (!gene.summary || typeof gene.summary !== 'string' || gene.summary.length < 10) {
+    if (Array.isArray(gene.strategy) && gene.strategy.length > 0) {
+      gene.summary = String(gene.strategy[0]).slice(0, 200);
+    } else if (Array.isArray(gene.signals_match) && gene.signals_match.length > 0) {
+      gene.summary = 'Strategy for: ' + gene.signals_match.slice(0, 3).join(', ');
+    }
+  }
+
+  // --- Strategy quality: require minimum 3 steps ---
+  if (Array.isArray(gene.strategy) && gene.strategy.length < 3) {
+    errors.push('strategy must have at least 3 steps for a quality skill');
+  }
+
+  // --- Constraints ---
   if (!gene.constraints || typeof gene.constraints !== 'object') gene.constraints = {};
   if (!Array.isArray(gene.constraints.forbidden_paths) || gene.constraints.forbidden_paths.length === 0) {
     gene.constraints.forbidden_paths = ['.git', 'node_modules'];
@@ -282,6 +446,7 @@ function validateSynthesizedGene(gene, existingGenes) {
     gene.constraints.max_files = DISTILLED_MAX_FILES;
   }
 
+  // --- Validation command sanitization ---
   var ALLOWED_PREFIXES = ['node ', 'npm ', 'npx '];
   if (Array.isArray(gene.validation)) {
     gene.validation = gene.validation.filter(function (cmd) {
@@ -294,11 +459,16 @@ function validateSynthesizedGene(gene, existingGenes) {
     });
   }
 
+  // --- Schema version ---
+  if (!gene.schema_version) gene.schema_version = '1.6.0';
+
+  // --- Duplicate ID check ---
   var existingIds = new Set((existingGenes || []).map(function (g) { return g.id; }));
   if (gene.id && existingIds.has(gene.id)) {
     gene.id = gene.id + '_' + Date.now().toString(36);
   }
 
+  // --- Signal overlap check ---
   if (gene.signals_match && existingGenes && existingGenes.length > 0) {
     var newSet = new Set(gene.signals_match.map(function (s) { return String(s).toLowerCase(); }));
     for (var i = 0; i < existingGenes.length; i++) {
@@ -501,6 +671,7 @@ module.exports = {
   prepareDistillation: prepareDistillation,
   completeDistillation: completeDistillation,
   validateSynthesizedGene: validateSynthesizedGene,
+  sanitizeSignalsMatch: sanitizeSignalsMatch,
   shouldDistill: shouldDistill,
   buildDistillationPrompt: buildDistillationPrompt,
   extractJsonFromLlmResponse: extractJsonFromLlmResponse,

package/src/gep/skillPublisher.js

@@ -3,25 +3,90 @@
 var { getHubUrl, buildHubHeaders, getNodeId } = require('./a2aProtocol');
 
 /**
- * Convert a Gene object into SKILL.md format (Claude/Anthropic style).
+ * 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;
+  }
+  if (name.replace(/[-]/g, '').length < 6) return null;
+  return name;
+}
+
+/**
+ * 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);
+      });
+    });
+  }
+  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 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 name = (gene.id || 'unnamed-skill').replace(/^gene_distilled_/, '').replace(/_/g, '-');
-  var desc = gene.summary || 'AI agent skill distilled from evolution experience.';
+  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('');
@@ -31,6 +96,7 @@ function geneToSkillMd(gene) {
     lines.push('');
   }
 
+  // -- Preconditions --
   if (gene.preconditions && gene.preconditions.length > 0) {
     lines.push('## Preconditions');
     lines.push('');
@@ -40,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('');
@@ -72,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.*');
@@ -80,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.
  *
@@ -92,16 +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 skillId = 'skill_' + (gene.id || 'unnamed').replace(/^gene_/, '');
+  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';
@@ -134,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',
   };
 
@@ -151,8 +288,13 @@ function updateSkillOnHub(nodeId, skillId, content, opts, gene) {
     body: JSON.stringify(body),
     signal: AbortSignal.timeout(15000),
   })
-    .then(function (res) { return res.json(); })
-    .then(function (data) { return { ok: true, result: data }; })
+    .then(function (res) { return res.json().then(function (data) { return { status: res.status, data: data }; }); })
+    .then(function (result) {
+      if (result.status >= 200 && result.status < 300) {
+        return { ok: true, result: result.data };
+      }
+      return { ok: false, error: result.data?.error || 'update_failed', status: result.status };
+    })
     .catch(function (err) { return { ok: false, error: err.message }; });
 }
 
@@ -160,4 +302,6 @@ module.exports = {
   geneToSkillMd: geneToSkillMd,
   publishSkillToHub: publishSkillToHub,
   updateSkillOnHub: updateSkillOnHub,
+  sanitizeSkillName: sanitizeSkillName,
+  toTitleCase: toTitleCase,
 };

package/src/gep/solidify.js

@@ -136,6 +136,7 @@ function readOpenclawConstraintPolicy() {
       includeExtensions: Array.isArray(pol.includeExtensions) ? pol.includeExtensions.map(String) : defaults.includeExtensions,
     };
   } catch (_) {
+    console.warn('[evolver] readOpenclawConstraintPolicy failed:', _ && _.message || _);
     return defaults;
   }
 }
@@ -159,7 +160,9 @@ function matchAnyRegex(rel, regexList) {
   for (const raw of Array.isArray(regexList) ? regexList : []) {
     try {
       if (new RegExp(String(raw), 'i').test(rel)) return true;
-    } catch (_) {}
+    } catch (_) {
+      console.warn('[evolver] matchAnyRegex invalid pattern:', raw, _ && _.message || _);
+    }
   }
   return false;
 }
@@ -339,7 +342,9 @@ function checkConstraints({ gene, blast, blastRadiusEstimate, repoRoot }) {
         if (entries.length < 2) {
           warnings.push('incomplete_skill: skills/' + skillName + '/ has only ' + entries.length + ' file(s). New skills should have at least index.js + SKILL.md.');
         }
-      } catch (e) { /* dir might not exist yet */ }
+      } catch (e) {
+        console.warn('[evolver] checkConstraints skill dir read failed:', skillName, e && e.message || e);
+      }
     });
   }
 
@@ -381,7 +386,9 @@ function writeStateForSolidify(state) {
   const statePath = path.join(getEvolutionDir(), 'evolution_solidify_state.json');
   try {
     if (!fs.existsSync(memoryDir)) fs.mkdirSync(memoryDir, { recursive: true });
-  } catch {}
+  } catch (e) {
+    console.warn('[evolver] writeStateForSolidify mkdir failed:', memoryDir, e && e.message || e);
+  }
   const tmp = `${statePath}.tmp`;
   fs.writeFileSync(tmp, JSON.stringify(state, null, 2) + '\n', 'utf8');
   fs.renameSync(tmp, statePath);
@@ -559,7 +566,9 @@ function detectDestructiveChanges({ repoRoot, changedFiles, baselineUntracked })
           if (stat.isFile() && stat.size === 0) {
             violations.push(`CRITICAL_FILE_EMPTIED: ${norm}`);
           }
-        } catch (e) {}
+        } catch (e) {
+          console.warn('[evolver] detectDestructiveChanges stat failed:', norm, e && e.message || e);
+        }
       }
     }
   }
@@ -716,7 +725,9 @@ function rollbackNewUntrackedFiles({ repoRoot, baselineUntracked }) {
         fs.unlinkSync(normAbs);
         deleted.push(safeRel);
       }
-    } catch (e) {}
+    } catch (e) {
+      console.warn('[evolver] rollbackNewUntrackedFiles unlink failed:', safeRel, e && e.message || e);
+    }
   }
   if (skipped.length > 0) {
     console.log(`[Rollback] Skipped ${skipped.length} critical protected file(s): ${skipped.slice(0, 5).join(', ')}`);
@@ -749,7 +760,9 @@ function rollbackNewUntrackedFiles({ repoRoot, baselineUntracked }) {
         fs.rmdirSync(dirAbs);
         removedDirs.push(sortedDirs[si]);
       }
-    } catch (e) { /* ignore -- dir may already be gone */ }
+    } catch (e) {
+      console.warn('[evolver] rollbackNewUntrackedFiles rmdir failed:', sortedDirs[si], e && e.message || e);
+    }
   }
   if (removedDirs.length > 0) {
     console.log('[Rollback] Removed ' + removedDirs.length + ' empty director' + (removedDirs.length === 1 ? 'y' : 'ies') + ': ' + removedDirs.slice(0, 5).join(', '));
@@ -1226,7 +1239,9 @@ function solidify({ intent, summary, dryRun = false, rollbackOnFailure = true }
         const list = require('./assetStore').loadCapsules();
         prevCapsule = Array.isArray(list) ? list.find(c => c && c.type === 'Capsule' && String(c.id) === selectedCapsuleId) : null;
       }
-    } catch (e) {}
+    } catch (e) {
+      console.warn('[evolver] solidify loadCapsules failed:', e && e.message || e);
+    }
     const successReason = buildSuccessReason({ gene: geneUsed, signals, blast, mutation, score });
     const capsuleDiff = captureDiffSnapshot(repoRoot);
     const capsuleContent = buildCapsuleContent({ intent, gene: geneUsed, signals, blast, mutation, score });
@@ -1302,7 +1317,7 @@ function solidify({ intent, summary, dryRun = false, rollbackOnFailure = true }
       applyEpigeneticMarks(geneUsed, envFp, outcomeStatus);
       upsertGene(geneUsed);
     } catch (e) {
-      // Non-blocking: epigenetic mark failure must not break solidify
+      console.warn('[evolver] applyEpigeneticMarks failed (non-blocking):', e && e.message || e);
     }
   }
 
@@ -1326,7 +1341,9 @@ function solidify({ intent, summary, dryRun = false, rollbackOnFailure = true }
       if (personalityState) {
         updatePersonalityStats({ personalityState, outcome: outcomeStatus, score, notes: `event:${event.id}` });
       }
-    } catch (e) {}
+    } catch (e) {
+      console.warn('[evolver] updatePersonalityStats failed:', e && e.message || e);
+    }
   }
 
   const runId = lastRun && lastRun.run_id ? String(lastRun.run_id) : stableHash(`${parentEventId || 'root'}|${geneId || 'none'}|${signalKey}`);