clementine-agent 1.18.143 → 1.18.144

package/dist/agent/self-improve.js

@@ -17,6 +17,7 @@ import pino from 'pino';
 import { BASE_DIR, SELF_IMPROVE_DIR, SOUL_FILE, CRON_FILE, WORKFLOWS_DIR, VAULT_DIR, MEMORY_DB_PATH, AGENTS_DIR, CRON_REFLECTIONS_DIR, GOALS_DIR, } from '../config.js';
 import { listAllGoals } from '../tools/shared.js';
 import { MemoryStore } from '../memory/store.js';
+import { ANTHROPIC_SKILL_NAME_PATTERN } from './skill-store.js';
 const logger = pino({ name: 'clementine.self-improve' });
 // ── Defaults ─────────────────────────────────────────────────────────
 const DEFAULT_CONFIG = {
@@ -2196,7 +2197,7 @@ export function validateProposal(area, target, proposedChange) {
         // present + non-empty, no XML tags in description, no Anthropic-
         // reserved words in name. Reuses the centralized validator that
         // dashboard + MCP + auto-extract all share.
-        if (!/^[a-z0-9][a-z0-9-]{0,63}$/.test(target)) {
+        if (!ANTHROPIC_SKILL_NAME_PATTERN.test(target)) {
             return { valid: false, error: `skill target must be a valid slug (got "${target}")` };
         }
         let parsed;
@@ -2210,7 +2211,7 @@ export function validateProposal(area, target, proposedChange) {
         const name = typeof fm.name === 'string' ? fm.name : '';
         const description = typeof fm.description === 'string' ? fm.description : '';
         const body = parsed.content || '';
-        if (!name || !/^[a-z0-9][a-z0-9-]{0,63}$/.test(name)) {
+        if (!name || !ANTHROPIC_SKILL_NAME_PATTERN.test(name)) {
             return { valid: false, error: 'skill frontmatter "name" missing or invalid slug' };
         }
         if (name !== target) {

package/dist/agent/skill-store.d.ts

@@ -22,6 +22,12 @@
  * crons → folder-form skills.
  */
 import type { Skill, SkillScope, SkillValidationWarning, CronJobDefinition } from '../types.js';
+/**
+ * Anthropic skill slug regex. Exported (1.18.144) so other modules
+ * (self-improve, migration tooling) don't drift their own copies.
+ * Lowercase letters/digits/dashes, must start with [a-z0-9], ≤64 chars.
+ */
+export declare const ANTHROPIC_SKILL_NAME_PATTERN: RegExp;
 /** Run Anthropic-spec validations on a parsed skill. Errors are spec
  *  violations (skill would be rejected by the Anthropic API); warnings
  *  are best-practice hints (still loadable). Findings render in the

package/dist/agent/skill-store.js

@@ -37,7 +37,13 @@ function projectSkillsDir(workDir) {
     return existsSync(dir) ? dir : null;
 }
 // ── Anthropic spec validations ────────────────────────────────────────
-const NAME_PATTERN = /^[a-z0-9][a-z0-9-]{0,63}$/;
+/**
+ * Anthropic skill slug regex. Exported (1.18.144) so other modules
+ * (self-improve, migration tooling) don't drift their own copies.
+ * Lowercase letters/digits/dashes, must start with [a-z0-9], ≤64 chars.
+ */
+export const ANTHROPIC_SKILL_NAME_PATTERN = /^[a-z0-9][a-z0-9-]{0,63}$/;
+const NAME_PATTERN = ANTHROPIC_SKILL_NAME_PATTERN;
 const RESERVED_NAMES = new Set(['anthropic', 'claude']);
 const NAME_MAX_LEN = 64;
 const DESCRIPTION_MAX_LEN = 1024;

package/dist/cli/routes/workflows.js

@@ -116,7 +116,8 @@ export function workflowsRouter(deps) {
             const { wf, agentSlug } = candidates[0];
             // Slugify the workflow name to the Anthropic regex
             const slug = name.toLowerCase().replace(/[^a-z0-9-]+/g, '-').replace(/^-+|-+$/g, '').slice(0, 64);
-            if (!/^[a-z0-9][a-z0-9-]{0,63}$/.test(slug)) {
+            const { ANTHROPIC_SKILL_NAME_PATTERN } = await import('../../agent/skill-store.js');
+            if (!ANTHROPIC_SKILL_NAME_PATTERN.test(slug)) {
                 res.status(400).json({ ok: false, error: `Workflow name "${name}" cannot be slugified to Anthropic regex` });
                 return;
             }

package/dist/gateway/scheduler-state.d.ts

@@ -33,6 +33,22 @@
  * missing fields, drop invalid values).
  */
 export declare function loadStateFile<T>(filePath: string, defaultValue: T, validator?: (raw: unknown) => T): T;
+/**
+ * 1.18.144 — Generic JSON file reader for non-scheduler call sites.
+ *
+ * Same try/parse/fallback shape as loadStateFile but with a `silent`
+ * option for callers who expect missing files to be common (e.g. a
+ * tool reading an optional config) and don't want every miss to log.
+ *
+ * Six+ files were inlining `try { JSON.parse(readFileSync(p)) } catch
+ * { return default }` before this. They now share one well-tested
+ * helper, which means the next file that needs to read a JSON sidecar
+ * doesn't add a seventh variant.
+ */
+export declare function loadJsonFile<T>(filePath: string, defaultValue: T, opts?: {
+    silent?: boolean;
+    validator?: (raw: unknown) => T;
+}): T;
 export interface SaveStateOptions {
     /**
      * If true, write to `<file>.tmp` then rename — guarantees the on-disk

package/dist/gateway/scheduler-state.js

@@ -48,6 +48,32 @@ export function loadStateFile(filePath, defaultValue, validator) {
         return defaultValue;
     }
 }
+/**
+ * 1.18.144 — Generic JSON file reader for non-scheduler call sites.
+ *
+ * Same try/parse/fallback shape as loadStateFile but with a `silent`
+ * option for callers who expect missing files to be common (e.g. a
+ * tool reading an optional config) and don't want every miss to log.
+ *
+ * Six+ files were inlining `try { JSON.parse(readFileSync(p)) } catch
+ * { return default }` before this. They now share one well-tested
+ * helper, which means the next file that needs to read a JSON sidecar
+ * doesn't add a seventh variant.
+ */
+export function loadJsonFile(filePath, defaultValue, opts = {}) {
+    try {
+        if (!existsSync(filePath))
+            return defaultValue;
+        const raw = JSON.parse(readFileSync(filePath, 'utf-8'));
+        return opts.validator ? opts.validator(raw) : raw;
+    }
+    catch (err) {
+        if (!opts.silent) {
+            logger.warn({ err, filePath }, 'Failed to load JSON file — using default');
+        }
+        return defaultValue;
+    }
+}
 /**
  * Write a JSON state file. Creates the parent directory if missing.
  * Returns true on success, false on failure (always logs a warning

package/dist/tools/shared.d.ts

@@ -676,6 +676,15 @@ export declare function agentTasksFile(slug: string | null): string;
 export declare function agentWorkingMemoryFile(slug: string | null): string;
 export declare function agentGoalsDir(slug: string | null): string;
 export declare function agentDailyNotesDir(slug: string | null): string;
+/**
+ * 1.18.144 — Single source of truth for "all currently-hired agent
+ * slugs." Five+ files used to inline the same readdirSync + filter
+ * pattern (skipping leading-underscore directories like _archive).
+ *
+ * Returns slugs sorted alphabetically. Returns [] when AGENTS_DIR
+ * doesn't exist or can't be read.
+ */
+export declare function listAgentSlugs(): string[];
 export type GoalRecord = {
     id: string;
     title: string;

package/dist/tools/shared.js

@@ -80,6 +80,27 @@ export function agentDailyNotesDir(slug) {
         return DAILY_NOTES_DIR;
     return path.join(AGENTS_DIR, slug, 'daily-notes');
 }
+/**
+ * 1.18.144 — Single source of truth for "all currently-hired agent
+ * slugs." Five+ files used to inline the same readdirSync + filter
+ * pattern (skipping leading-underscore directories like _archive).
+ *
+ * Returns slugs sorted alphabetically. Returns [] when AGENTS_DIR
+ * doesn't exist or can't be read.
+ */
+export function listAgentSlugs() {
+    if (!existsSync(AGENTS_DIR))
+        return [];
+    try {
+        return readdirSync(AGENTS_DIR, { withFileTypes: true })
+            .filter((d) => d.isDirectory() && !d.name.startsWith('_'))
+            .map((d) => d.name)
+            .sort();
+    }
+    catch {
+        return [];
+    }
+}
 /** Return the directory where a goal owned by `owner` should live. */
 export function goalDirForOwner(owner) {
     if (!owner || owner === 'clementine')

package/dist/tools/skill-tools.js

@@ -25,7 +25,10 @@ import { VAULT_DIR, textResult, logger } from './shared.js';
 // 1.18.124 — name regex is the only validator skill-tools still uses
 // directly (for update_skill's pre-flight slug check). All other
 // validations + the file write live in skill-store.writeSkill.
-const NAME_PATTERN = /^[a-z0-9][a-z0-9-]{0,63}$/;
+// 1.18.144 — pulled the regex from skill-store's exported canonical
+// constant so all skill-name validation now traces to one source.
+import { ANTHROPIC_SKILL_NAME_PATTERN } from '../agent/skill-store.js';
+const NAME_PATTERN = ANTHROPIC_SKILL_NAME_PATTERN;
 const DESCRIPTION_MAX_LEN = 1024;
 function globalSkillsDir() {
     return path.join(VAULT_DIR, '00-System', 'skills');

package/dist/tools/team-tools.js

@@ -7,13 +7,14 @@ import path from 'node:path';
 import { z } from 'zod';
 import { ACTIVE_AGENT_SLUG, AGENTS_DIR, BASE_DIR, DELEGATIONS_BASE, TEAM_COMMS_LOG, env, logger, parseTasks, textResult, } from './shared.js';
 import { todayISO } from '../gateway/cron-scheduler.js';
+import { listAgentSlugs } from './shared.js';
 async function loadTeamAgents() {
     const matterMod = await import('gray-matter');
     const agents = [];
     const seen = new Set();
     if (existsSync(AGENTS_DIR)) {
         try {
-            for (const slug of readdirSync(AGENTS_DIR, { withFileTypes: true }).filter(d => d.isDirectory() && !d.name.startsWith('_')).map(d => d.name)) {
+            for (const slug of listAgentSlugs()) {
                 const agentFile = path.join(AGENTS_DIR, slug, 'agent.md');
                 if (!existsSync(agentFile))
                     continue;
@@ -349,10 +350,7 @@ export function registerTeamTools(server) {
         const agentsBase = AGENTS_DIR;
         if (!existsSync(agentsBase))
             return textResult('No agents found.');
-        const agentSlugs = readdirSync(agentsBase, { withFileTypes: true })
-            .filter(d => d.isDirectory())
-            .map(d => d.name)
-            .filter(n => !agent || n === agent);
+        const agentSlugs = listAgentSlugs().filter(n => !agent || n === agent);
         if (!agentSlugs.length)
             return textResult('No agents found.');
         const matterMod = await import('gray-matter');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clementine-agent",
-  "version": "1.18.143",
+  "version": "1.18.144",
   "description": "Clementine — Personal AI Assistant (TypeScript)",
   "type": "module",
   "main": "dist/index.js",
@@ -28,7 +28,7 @@
     "postinstall": "node scripts/postinstall.js 2>/dev/null || true"
   },
   "dependencies": {
-    "@anthropic-ai/claude-agent-sdk": "^0.2.137",
+    "@anthropic-ai/claude-agent-sdk": "^0.2.138",
     "@anthropic-ai/sdk": "^0.91.0",
     "@composio/claude-agent-sdk": "^0.8.1",
     "@composio/core": "^0.8.1",