lazyclaw 6.5.0 → 6.6.0

package/agents.mjs

@@ -90,6 +90,16 @@ function defaultShape(name) {
     tools: [...DEFAULT_TOOLS],
     tags: [],
     iconEmoji: '',
+    // Explicit Team Live sprite choice: an integer 1..20 picks one of the 20
+    // built-in pixel-art avatars (web/avatars/NN.png); null lets the dashboard
+    // keep inferring one from the agent's name/role/tags. (dashboard.js
+    // avatarIndexFor already honours rec.avatar — this is the registry side.)
+    avatar: null,
+    // Optional custom character image: a ready-to-use <img src> — either a
+    // remote http(s) URL or a daemon-served '/agent-avatars/<file>' path for a
+    // photo the user supplied (copied under <configDir>/agent-avatars/). null =
+    // none. Takes precedence over `avatar` and the keyword inference.
+    avatarImage: null,
     // Optional parent agent (hierarchy). '' = top-level. A team's org tree is
     // derived from members' manager links (see teams.teamTree). Validated to
     // reference a registered agent and to never form a cycle.
@@ -124,6 +134,18 @@ function writeAtomic(filePath, obj) {
 const VALID_MEMORY_WRITE = ['auto', 'manual', 'off'];
 const VALID_SKILL_WRITE = ['auto', 'manual', 'off'];
 
+// Normalise an explicit avatar choice. null/''/undefined → null (keep the
+// dashboard's keyword inference). A value that parses to an integer 1..20 picks
+// that built-in sprite. Anything else (0, 21, fractional, non-numeric) throws.
+function validateAvatar(v) {
+  if (v === undefined || v === null || v === '') return null;
+  const n = typeof v === 'number' ? v : Number(String(v).trim());
+  if (!Number.isInteger(n) || n < 1 || n > 20) {
+    throw new AgentError('avatar must be an integer 1..20 (or null to clear)', 'AGENT_BAD_AVATAR');
+  }
+  return n;
+}
+
 // Validate a proposed `manager` (parent agent) for `name`: it must reference a
 // registered agent, may not be the agent itself, and may not close a cycle
 // (i.e. `name` must not already be an ancestor of the proposed manager).
@@ -151,7 +173,7 @@ function validateManager(name, manager, configDir) {
   return mgr;
 }
 
-export function registerAgent({ name, displayName, role = '', provider = 'claude-cli', model = '', tools, tags = [], iconEmoji = '', memoryWrite, memoryMaxChars, skillWrite, manager } = {}, configDir = defaultConfigDir()) {
+export function registerAgent({ name, displayName, role = '', provider = 'claude-cli', model = '', tools, tags = [], iconEmoji = '', avatar, memoryWrite, memoryMaxChars, skillWrite, manager } = {}, configDir = defaultConfigDir()) {
   ensureValidName(name);
   const p = agentPath(name, configDir);
   if (fs.existsSync(p)) {
@@ -175,6 +197,7 @@ export function registerAgent({ name, displayName, role = '', provider = 'claude
     tools: toolsClean,
     tags: Array.isArray(tags) ? tags : [],
     iconEmoji: String(iconEmoji || ''),
+    avatar: validateAvatar(avatar),
     memoryWrite: mw,
     memoryMaxChars: Number.isFinite(+memoryMaxChars) && +memoryMaxChars > 0 ? +memoryMaxChars : 12 * 1024,
     skillWrite: sw,
@@ -220,6 +243,12 @@ export function patchAgent(name, patch, configDir = defaultConfigDir()) {
   if (patch.skillWrite !== undefined && !VALID_SKILL_WRITE.includes(patch.skillWrite)) {
     throw new AgentError(`skillWrite must be one of ${VALID_SKILL_WRITE.join(', ')}`, 'AGENT_BAD_SKILL_WRITE');
   }
+  if (patch.avatar !== undefined) {
+    next.avatar = validateAvatar(patch.avatar);
+  }
+  if (patch.avatarImage !== undefined) {
+    next.avatarImage = (patch.avatarImage == null || patch.avatarImage === '') ? null : String(patch.avatarImage);
+  }
   if (patch.manager !== undefined) {
     next.manager = validateManager(name, patch.manager, configDir);
   }
@@ -227,6 +256,44 @@ export function patchAgent(name, patch, configDir = defaultConfigDir()) {
   return next;
 }
 
+// Supported custom-avatar image types → their served content-type. svg is
+// deliberately excluded (inline-script XSS risk in an <img>/object context).
+export const AVATAR_IMAGE_TYPES = {
+  '.png': 'image/png', '.jpg': 'image/jpeg', '.jpeg': 'image/jpeg',
+  '.gif': 'image/gif', '.webp': 'image/webp',
+};
+
+// Point an agent at a custom character image. `src` is either a remote http(s)
+// URL (stored verbatim) or a path to a local image file (copied into
+// <configDir>/agent-avatars/<name><ext> and stored as a daemon-served
+// '/agent-avatars/<name><ext>' path). Returns the patched record. The image
+// takes precedence over the numeric sprite + keyword inference in the dashboard.
+export function setAgentAvatarImage(name, src, configDir = defaultConfigDir()) {
+  if (!getAgent(name, configDir)) throw new AgentError(`no agent "${name}"`, 'AGENT_NO_AGENT');
+  const s = String(src ?? '').trim();
+  if (!s) throw new AgentError('avatar image source required (a file path or http(s) URL)', 'AGENT_BAD_AVATAR_IMAGE');
+  if (/^https?:\/\//i.test(s)) {
+    return patchAgent(name, { avatarImage: s }, configDir);
+  }
+  const ext = path.extname(s).toLowerCase();
+  if (!AVATAR_IMAGE_TYPES[ext]) {
+    throw new AgentError(`unsupported image type "${ext || '(none)'}" — use png/jpg/jpeg/gif/webp or an http(s) URL`, 'AGENT_BAD_AVATAR_IMAGE');
+  }
+  if (!fs.existsSync(s) || !fs.statSync(s).isFile()) {
+    throw new AgentError(`no such image file: ${s}`, 'AGENT_BAD_AVATAR_IMAGE');
+  }
+  const destDir = path.join(configDir, 'agent-avatars');
+  fs.mkdirSync(destDir, { recursive: true });
+  // Drop any previously stored image for this agent (it may have a different
+  // extension) so a re-point doesn't leave a stale file shadowing the new one.
+  for (const e of Object.keys(AVATAR_IMAGE_TYPES)) {
+    const old = path.join(destDir, `${name}${e}`);
+    if (fs.existsSync(old)) fs.unlinkSync(old);
+  }
+  fs.copyFileSync(s, path.join(destDir, `${name}${ext}`));
+  return patchAgent(name, { avatarImage: `/agent-avatars/${name}${ext}` }, configDir);
+}
+
 export function removeAgent(name, configDir = defaultConfigDir()) {
   const p = agentPath(name, configDir);
   if (!fs.existsSync(p)) {

package/commands/agents_registry.mjs

@@ -35,6 +35,7 @@ export async function cmdAgentRegistry(sub, positional, flags = {}) {
           model: flags.model || '',
           tools: tools === null ? undefined : tools,
           tags: agentsMod.parseToolsFlag(flags.tags) || [],
+          avatar: flags.avatar,
           skillWrite: flags['skill-write'],
           manager: flags.manager,
         }, cfgDir);
@@ -64,6 +65,7 @@ export async function cmdAgentRegistry(sub, positional, flags = {}) {
       if (flags.tags !== undefined)         patch.tags = agentsMod.parseToolsFlag(flags.tags);
       if (flags['skill-write'] !== undefined)  patch.skillWrite = String(flags['skill-write']);
       if (flags['memory-write'] !== undefined) patch.memoryWrite = String(flags['memory-write']);
+      if (flags.avatar !== undefined)          patch.avatar = /^(none|clear|off)$/i.test(String(flags.avatar)) ? null : flags.avatar;
       if (Object.keys(patch).length === 0) {
         console.error('agent edit: no fields to update');
         process.exit(2);
@@ -72,6 +74,26 @@ export async function cmdAgentRegistry(sub, positional, flags = {}) {
       catch (err) { console.error(`agent edit: ${err?.message || err}`); process.exit(2); }
       return;
     }
+    case 'set-avatar': {
+      const val = positional[1];
+      if (!name || val === undefined) {
+        console.error('Usage: lazyclaw agent set-avatar <name> <1-20 | none | <image-file> | http(s)://url>\n  1-20      pick a built-in pixel-art sprite\n  none      keep the role-inferred default\n  file/url  use a custom character photo');
+        process.exit(2);
+      }
+      const v = String(val).trim();
+      try {
+        if (/^(none|clear|off)$/i.test(v)) {
+          emitJson(agentsMod.patchAgent(name, { avatar: null, avatarImage: null }, cfgDir));
+        } else if (/^\d+$/.test(v)) {
+          // a built-in sprite index — clear any custom image so it shows through
+          emitJson(agentsMod.patchAgent(name, { avatar: v, avatarImage: null }, cfgDir));
+        } else {
+          // a local image file or a remote URL
+          emitJson(agentsMod.setAgentAvatarImage(name, val, cfgDir));
+        }
+      } catch (err) { console.error(`agent set-avatar: ${err?.message || err}`); process.exit(2); }
+      return;
+    }
     case 'remove':
     case 'rm':
     case 'delete': {

package/daemon/route_table.mjs

@@ -28,6 +28,7 @@ export const ROUTES = [
   { m: (c) => c.route === 'GET /dashboard.css', h: meta.dashboardCss },
   { m: (c) => c.route === 'GET /dashboard.js', h: meta.dashboardJs },
   { m: (c) => c.req.method === 'GET' && /^\/avatars\/\d{2}\.png$/.test(c.path || ''), h: meta.avatar },
+  { m: (c) => c.req.method === 'GET' && /^\/agent-avatars\/[A-Za-z0-9_-]+\.[a-z]+$/.test(c.path || ''), h: meta.agentAvatar },
   { m: (c) => c.route === 'GET /version', h: meta.version },
   { m: (c) => c.route === 'POST /exec/request', h: conversation.execRequest },
   { m: (c) => c.route === 'GET /health' || c.route === 'GET /healthz', h: meta.health },

package/daemon/routes/meta.mjs

@@ -65,6 +65,35 @@ export async function dashboardJs(c) {
   return serveWebFile(c, 'dashboard.js', 'text/javascript; charset=utf-8');
 }
 
+// Serve a custom agent character image (web app: <img src="/agent-avatars/NN">).
+// These are user-supplied photos copied under <configDir>/agent-avatars/ by
+// `lazyclaw agent set-avatar`. Served from the config dir (NOT the package web/
+// dir) since they're per-install user data. The filename is constrained to
+// `<word>.<ext>` so no path traversal escapes the agent-avatars directory.
+const _AGENT_AVATAR_CT = {
+  '.png': 'image/png', '.jpg': 'image/jpeg', '.jpeg': 'image/jpeg',
+  '.gif': 'image/gif', '.webp': 'image/webp',
+};
+export async function agentAvatar(c) {
+  const { res } = c;
+  const m = /^\/agent-avatars\/([A-Za-z0-9_-]+\.[a-z]+)$/.exec(c.path || '');
+  const file = m && m[1];
+  const ct = file && _AGENT_AVATAR_CT[nodePath.extname(file).toLowerCase()];
+  const base = c.gwConfigDir;
+  if (!file || file.includes('..') || !ct || !base) {
+    res.writeHead(404, { 'content-type': 'text/plain; charset=utf-8' });
+    return res.end('not found\n');
+  }
+  try {
+    const body = _readAssetCached(nodePath.join(base, 'agent-avatars', file));
+    res.writeHead(200, { 'content-type': ct, 'cache-control': 'no-cache' });
+    return res.end(body);
+  } catch (e) {
+    res.writeHead(404, { 'content-type': 'text/plain; charset=utf-8' });
+    return res.end(`not found: ${file} (${e?.message || e})\n`);
+  }
+}
+
 export async function dashboard(c) {
   const { res } = c;
           // Serve the lazyclaw-only web dashboard (a single static

package/lib/args.mjs

@@ -277,6 +277,6 @@ _lazyclaw "$@"
 // AGENT_REG_SUBS to decide whether a bare `agent <sub>` routes to the agent
 // registry vs a one-shot agent run. TEAM_SUBS/TASK_SUBS mirror their command's
 // subcommands for the same disambiguation pattern.
-export const AGENT_REG_SUBS = new Set(['add', 'list', 'show', 'edit', 'remove', 'rm', 'delete', 'memory', 'reflect', 'skill-synth']);
+export const AGENT_REG_SUBS = new Set(['add', 'list', 'show', 'edit', 'remove', 'rm', 'delete', 'memory', 'reflect', 'skill-synth', 'set-avatar']);
 export const TEAM_SUBS = new Set(['add', 'list', 'show', 'edit', 'remove', 'rm', 'delete']);
 export const TASK_SUBS = new Set(['start', 'tick', 'list', 'show', 'abandon', 'done', 'transcript', 'remove', 'rm', 'delete']);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lazyclaw",
-  "version": "6.5.0",
+  "version": "6.6.0",
   "description": "Lazy, elegant terminal CLI for chatting with Claude / OpenAI / Gemini / Ollama, orchestrating multi-step LLM workflows, and running multi-agent Slack teams with cross-task memory. Banner-on-launch, slash-command ghost autocomplete, persistent sessions, local HTTP gateway.",
   "keywords": [
     "claude",

package/web/dashboard.js

@@ -1409,7 +1409,13 @@
       }
       return 1; // generic PM look
     }
-    function avatarSrc(rec) { return `/avatars/${String(avatarIndexFor(rec)).padStart(2, '0')}.png`; }
+    // A user-supplied custom image (set via `lazyclaw agent set-avatar`) wins
+    // over the picked/inferred built-in sprite. rec.avatarImage is already a
+    // ready-to-use src (a remote URL or a daemon-served /agent-avatars/ path).
+    function avatarSrc(rec) {
+      if (rec && rec.avatarImage) return rec.avatarImage;
+      return `/avatars/${String(avatarIndexFor(rec)).padStart(2, '0')}.png`;
+    }
     // Build the { name, children[] } tree rooted at the lead (mirrors teamTree).
     function buildTeamTree(team, byId) {
       const lead = team.lead;