@syengup/friday-channel-next 0.1.20 → 0.1.22

package/dist/index.js

@@ -10,6 +10,8 @@ import { forwardAgentEventRaw, getLastRegisteredFridayDeviceId, resolveFridayDev
 import { setFridayAgentForwardRuntime } from "./src/agent-forward-runtime.js";
 import { getOpenClawAgentRunContext } from "./src/agent-run-context-bridge.js";
 import { accumulateRunUsage } from "./src/agent/run-usage-accumulator.js";
+import { createFridayNextLogger } from "./src/logging.js";
+const hookLogger = createFridayNextLogger("hook");
 export { fridayNextChannelPlugin } from "./src/channel.js";
 export { setFridayNextRuntime } from "./src/runtime.js";
 /** `api.on` returns void — register tool hooks at most once per process. */
@@ -136,8 +138,7 @@ export default defineChannelPluginEntry({
             const deviceId = deviceIdFromToolContext(ctx);
             const runId = ctx.runId ?? "(unknown)";
             const logLine = (detail) => {
-                const ts = new Date().toISOString();
-                console.error(`[Friday-HOOK] [${ts}] [TOOL_CALL] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`);
+                hookLogger.debug(`[TOOL_CALL] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`);
             };
             if (!deviceId) {
                 logLine("SKIP_no_deviceId");
@@ -163,8 +164,7 @@ export default defineChannelPluginEntry({
             const deviceId = deviceIdFromToolContext(ctx);
             const runId = ctx.runId ?? "(unknown)";
             const logLine = (detail) => {
-                const ts = new Date().toISOString();
-                console.error(`[Friday-HOOK] [${ts}] [TOOL_DONE] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`);
+                hookLogger.debug(`[TOOL_DONE] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`);
             };
             if (!deviceId) {
                 logLine("SKIP_no_deviceId");

package/dist/src/agent-forward-runtime.d.ts

@@ -14,6 +14,8 @@ export type FridayAgentForwardRuntime = {
         sessionKey: string;
         update: (entry: Record<string, unknown>) => Record<string, unknown> | null | Promise<Record<string, unknown> | null>;
     }) => Promise<Record<string, unknown> | null>;
+    /** Resolves an agent's workspace dir — used to read IDENTITY.md for the name fallback. */
+    resolveAgentWorkspaceDir?: (cfg: unknown, agentId: string) => string;
     getConfig: () => unknown;
 };
 /** Called from `registerFull` so terminal lifecycle forwards can read `sessions.json` after persist. */

package/dist/src/agent-forward-runtime.js

@@ -6,6 +6,8 @@ export function setFridayAgentForwardRuntime(api) {
         loadSessionStore: api.runtime.agent.session.loadSessionStore,
         updateSessionStoreEntry: api.runtime.agent.session
             .updateSessionStoreEntry,
+        resolveAgentWorkspaceDir: api.runtime.agent
+            .resolveAgentWorkspaceDir,
         getConfig: () => api.runtime.config.current(),
     };
 }

package/dist/src/history/normalize-message.js

@@ -180,16 +180,25 @@ export function normalizeHistoryMessage(raw, index) {
     }
     if (role === "toolResult") {
         const split = splitMediaLines(parsed.text);
+        const toolName = readString(record.toolName);
+        // Canvas snapshots come back as base64 image blocks on the `canvas` tool result so the *agent*
+        // can "see" the rendered page — they must never surface as chat attachments on history rebuild.
+        // The streaming deliver path already drops the temp-file form (see isCanvasSnapshotMediaPath in
+        // http/handlers/messages.ts); this is the transcript-rebuild counterpart. The canvas tool has no
+        // other image-returning action, so all images on a canvas result are snapshots.
+        const isCanvasResult = toolName === "canvas";
+        const images = isCanvasResult ? [] : parsed.images;
+        const mediaPaths = isCanvasResult ? [] : split.paths;
         const toolResult = {
             ...(readString(record.toolCallId) ? { toolCallId: readString(record.toolCallId) } : {}),
-            ...(readString(record.toolName) ? { toolName: readString(record.toolName) } : {}),
+            ...(toolName ? { toolName } : {}),
             ...(record.isError === true ? { isError: true } : {}),
             ...(split.text ? { text: split.text } : {}),
-            ...(parsed.images.length ? { images: parsed.images } : {}),
+            ...(images.length ? { images } : {}),
         };
         message.toolResult = toolResult;
-        if (split.paths.length)
-            message.mediaPaths = split.paths;
+        if (mediaPaths.length)
+            message.mediaPaths = mediaPaths;
         return message;
     }
     const split = splitMediaLines(parsed.text);

package/dist/src/http/handlers/agents-list.d.ts

@@ -10,4 +10,12 @@ export interface FridayAgentEntry {
     emoji?: string;
     avatar?: string;
 }
+/**
+ * Extract the `Name` field from an agent's IDENTITY.md, mirroring OpenClaw's
+ * `parseIdentityMarkdown` (src/agents/identity-file.ts) for the name label only:
+ * drop the leading "- ", split on the first ":", strip markdown emphasis, and
+ * skip the unfilled template placeholder. Returns the raw value verbatim (e.g.
+ * "星期五 (Friday)") so it matches what ControlUI shows under "身份名称".
+ */
+export declare function parseIdentityNameFromMarkdown(content: string): string | undefined;
 export declare function handleAgentsList(req: IncomingMessage, res: ServerResponse): Promise<boolean>;

package/dist/src/http/handlers/agents-list.js

@@ -1,4 +1,6 @@
-import { getFridayAgentForwardRuntime } from "../../agent-forward-runtime.js";
+import fs from "node:fs";
+import path from "node:path";
+import { getFridayAgentForwardRuntime, } from "../../agent-forward-runtime.js";
 import { extractBearerToken } from "../middleware/auth.js";
 const DEFAULT_AGENT_ID = "main";
 /** Agent ids already in path/shell-safe form skip the slug rewrite below. */
@@ -31,6 +33,60 @@ function resolvePrimaryModel(model) {
 function readString(value) {
     return typeof value === "string" && value.trim() ? value.trim() : undefined;
 }
+/** Unfilled IDENTITY.md template prompts that must not surface as a real name. */
+const IDENTITY_NAME_PLACEHOLDERS = new Set(["pick something you like"]);
+/**
+ * Extract the `Name` field from an agent's IDENTITY.md, mirroring OpenClaw's
+ * `parseIdentityMarkdown` (src/agents/identity-file.ts) for the name label only:
+ * drop the leading "- ", split on the first ":", strip markdown emphasis, and
+ * skip the unfilled template placeholder. Returns the raw value verbatim (e.g.
+ * "星期五 (Friday)") so it matches what ControlUI shows under "身份名称".
+ */
+export function parseIdentityNameFromMarkdown(content) {
+    for (const rawLine of content.split(/\r?\n/)) {
+        const cleaned = rawLine.trim().replace(/^\s*-\s*/, "");
+        const colonIndex = cleaned.indexOf(":");
+        if (colonIndex === -1)
+            continue;
+        const label = cleaned.slice(0, colonIndex).replace(/[*_`]/g, "").trim().toLowerCase();
+        if (label !== "name")
+            continue;
+        const value = cleaned
+            .slice(colonIndex + 1)
+            .replace(/^[*_`\s]+|[*_`\s]+$/g, "")
+            .trim();
+        if (!value)
+            continue;
+        let normalized = value.replace(/[–—]/g, "-");
+        if (normalized.startsWith("(") && normalized.endsWith(")")) {
+            normalized = normalized.slice(1, -1).trim();
+        }
+        if (IDENTITY_NAME_PLACEHOLDERS.has(normalized.toLowerCase()))
+            continue;
+        return value;
+    }
+    return undefined;
+}
+/**
+ * Name fallback for agents with no `name`/`identity.name` in config (e.g. the
+ * implicit `main`): resolve the agent's workspace and parse its IDENTITY.md, the
+ * same source ControlUI reads. Best-effort — any failure yields undefined.
+ */
+function readWorkspaceIdentityName(rt, cfg, agentId) {
+    const resolveWorkspace = rt.resolveAgentWorkspaceDir;
+    if (!resolveWorkspace)
+        return undefined;
+    try {
+        const workspace = resolveWorkspace(cfg, agentId);
+        if (!workspace)
+            return undefined;
+        const content = fs.readFileSync(path.join(workspace, "IDENTITY.md"), "utf-8");
+        return parseIdentityNameFromMarkdown(content);
+    }
+    catch {
+        return undefined;
+    }
+}
 /**
  * Reads the configured agents directly from the runtime config (same approach as
  * models-list.ts). When no agents are configured OpenClaw runs an implicit "main"
@@ -64,7 +120,9 @@ function resolveConfiguredAgents() {
         const identity = agent.identity;
         entries.push({
             id,
-            name: readString(agent.name) ?? readString(identity?.name),
+            name: readString(agent.name) ??
+                readString(identity?.name) ??
+                readWorkspaceIdentityName(rt, cfg, id),
             description: readString(agent.description),
             model: resolvePrimaryModel(agent.model),
             thinkingDefault: readString(agent.thinkingDefault),

package/index.ts

@@ -17,6 +17,9 @@ import {
 import { setFridayAgentForwardRuntime } from "./src/agent-forward-runtime.js";
 import { getOpenClawAgentRunContext } from "./src/agent-run-context-bridge.js";
 import { accumulateRunUsage } from "./src/agent/run-usage-accumulator.js";
+import { createFridayNextLogger } from "./src/logging.js";
+
+const hookLogger = createFridayNextLogger("hook");
 
 export { fridayNextChannelPlugin } from "./src/channel.js";
 export { setFridayNextRuntime } from "./src/runtime.js";
@@ -148,9 +151,8 @@ export default defineChannelPluginEntry({
       const runId = ctx.runId ?? "(unknown)";
 
       const logLine = (detail: string) => {
-        const ts = new Date().toISOString();
-        console.error(
-          `[Friday-HOOK] [${ts}] [TOOL_CALL] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`,
+        hookLogger.debug(
+          `[TOOL_CALL] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`,
         );
       };
 
@@ -180,9 +182,8 @@ export default defineChannelPluginEntry({
       const runId = ctx.runId ?? "(unknown)";
 
       const logLine = (detail: string) => {
-        const ts = new Date().toISOString();
-        console.error(
-          `[Friday-HOOK] [${ts}] [TOOL_DONE] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`,
+        hookLogger.debug(
+          `[TOOL_DONE] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`,
         );
       };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@syengup/friday-channel-next",
-  "version": "0.1.20",
+  "version": "0.1.22",
   "description": "OpenClaw Friday Next Apple channel plugin",
   "license": "MIT",
   "type": "module",

package/src/agent-forward-runtime.ts

@@ -14,6 +14,8 @@ export type FridayAgentForwardRuntime = {
       entry: Record<string, unknown>,
     ) => Record<string, unknown> | null | Promise<Record<string, unknown> | null>;
   }) => Promise<Record<string, unknown> | null>;
+  /** Resolves an agent's workspace dir — used to read IDENTITY.md for the name fallback. */
+  resolveAgentWorkspaceDir?: (cfg: unknown, agentId: string) => string;
   getConfig: () => unknown;
 };
 
@@ -26,6 +28,8 @@ export function setFridayAgentForwardRuntime(api: OpenClawPluginApi): void {
     loadSessionStore: api.runtime.agent.session.loadSessionStore,
     updateSessionStoreEntry: (api.runtime.agent.session as Record<string, unknown>)
       .updateSessionStoreEntry as FridayAgentForwardRuntime["updateSessionStoreEntry"],
+    resolveAgentWorkspaceDir: (api.runtime.agent as Record<string, unknown>)
+      .resolveAgentWorkspaceDir as FridayAgentForwardRuntime["resolveAgentWorkspaceDir"],
     getConfig: () => api.runtime.config.current(),
   };
 }

package/src/history/normalize-message.test.ts

@@ -108,6 +108,36 @@ describe("normalizeHistoryMessage", () => {
     });
   });
 
+  it("drops base64 image blocks from a canvas snapshot toolResult (agent-only, never a chat attachment)", () => {
+    const out = normalizeHistoryMessage(
+      {
+        role: "toolResult",
+        toolCallId: "tc-canvas",
+        toolName: "canvas",
+        content: [{ type: "image", mimeType: "image/jpeg", data: "BASE64SNAPSHOT" }],
+        ...meta("entry-canvas", 5),
+      },
+      0,
+    );
+    expect(out?.role).toBe("toolResult");
+    expect(out?.toolResult).toEqual({ toolCallId: "tc-canvas", toolName: "canvas" });
+    expect(out?.toolResult?.images).toBeUndefined();
+  });
+
+  it("keeps image blocks on non-canvas toolResults", () => {
+    const out = normalizeHistoryMessage(
+      {
+        role: "toolResult",
+        toolCallId: "tc-img",
+        toolName: "image_generation",
+        content: [{ type: "image", mimeType: "image/png", data: "REALIMG" }],
+        ...meta("entry-img", 6),
+      },
+      0,
+    );
+    expect(out?.toolResult?.images).toEqual([{ mimeType: "image/png", data: "REALIMG" }]);
+  });
+
   it("strips MEDIA: lines from text into mediaPaths", () => {
     const out = normalizeHistoryMessage(
       {

package/src/history/normalize-message.ts

@@ -253,15 +253,24 @@ export function normalizeHistoryMessage(
 
   if (role === "toolResult") {
     const split = splitMediaLines(parsed.text);
+    const toolName = readString(record.toolName);
+    // Canvas snapshots come back as base64 image blocks on the `canvas` tool result so the *agent*
+    // can "see" the rendered page — they must never surface as chat attachments on history rebuild.
+    // The streaming deliver path already drops the temp-file form (see isCanvasSnapshotMediaPath in
+    // http/handlers/messages.ts); this is the transcript-rebuild counterpart. The canvas tool has no
+    // other image-returning action, so all images on a canvas result are snapshots.
+    const isCanvasResult = toolName === "canvas";
+    const images = isCanvasResult ? [] : parsed.images;
+    const mediaPaths = isCanvasResult ? [] : split.paths;
     const toolResult: FridayHistoryToolResult = {
       ...(readString(record.toolCallId) ? { toolCallId: readString(record.toolCallId) } : {}),
-      ...(readString(record.toolName) ? { toolName: readString(record.toolName) } : {}),
+      ...(toolName ? { toolName } : {}),
       ...(record.isError === true ? { isError: true } : {}),
       ...(split.text ? { text: split.text } : {}),
-      ...(parsed.images.length ? { images: parsed.images } : {}),
+      ...(images.length ? { images } : {}),
     };
     message.toolResult = toolResult;
-    if (split.paths.length) message.mediaPaths = split.paths;
+    if (mediaPaths.length) message.mediaPaths = mediaPaths;
     return message;
   }
 

package/src/http/handlers/agents-list.test.ts

@@ -1,6 +1,9 @@
 import { describe, it, expect, beforeEach, afterEach } from "vitest";
 import { EventEmitter } from "node:events";
-import { handleAgentsList } from "./agents-list.js";
+import fs from "node:fs";
+import os from "node:os";
+import path from "node:path";
+import { handleAgentsList, parseIdentityNameFromMarkdown } from "./agents-list.js";
 import { setMockRuntime } from "../../test-support/mock-runtime.js";
 import {
   setFridayAgentForwardRuntime,
@@ -108,6 +111,33 @@ describe("handleAgentsList", () => {
     ]);
   });
 
+  it("falls back to the IDENTITY.md name when config has none", async () => {
+    const workspace = fs.mkdtempSync(path.join(os.tmpdir(), "friday-identity-"));
+    fs.writeFileSync(
+      path.join(workspace, "IDENTITY.md"),
+      "# IDENTITY.md\n\n- **Name:** 星期五 (Friday)\n- **Emoji:** 🌿\n",
+    );
+    try {
+      setFridayAgentForwardRuntime({
+        runtime: {
+          agent: {
+            session: { resolveStorePath: () => "", loadSessionStore: () => ({}) },
+            resolveAgentWorkspaceDir: () => workspace,
+          },
+          config: { current: () => ({ agents: { list: [{ id: "main" }] } }) },
+        },
+      } as any);
+
+      const res = new MockRes();
+      await handleAgentsList(makeReq(AUTH), res as any);
+
+      const body = JSON.parse(res.body);
+      expect(body.agents).toEqual([{ id: "main", name: "星期五 (Friday)", isDefault: true }]);
+    } finally {
+      fs.rmSync(workspace, { recursive: true, force: true });
+    }
+  });
+
   it("defaults to the first entry when none is marked default and dedups ids", async () => {
     setConfig({
       agents: {
@@ -127,3 +157,24 @@ describe("handleAgentsList", () => {
     expect(body.agents[0].isDefault).toBe(true);
   });
 });
+
+describe("parseIdentityNameFromMarkdown", () => {
+  it("extracts the Name value from the OpenClaw template format", () => {
+    const md = "# IDENTITY.md\n\n- **Name:** 星期五 (Friday)\n- **Emoji:** 🌿\n";
+    expect(parseIdentityNameFromMarkdown(md)).toBe("星期五 (Friday)");
+  });
+
+  it("handles a plain unstyled `Name:` line", () => {
+    expect(parseIdentityNameFromMarkdown("Name: Jarvis")).toBe("Jarvis");
+  });
+
+  it("returns undefined when there is no Name field", () => {
+    expect(parseIdentityNameFromMarkdown("- **Emoji:** 🌿\n- **Vibe:** calm")).toBeUndefined();
+  });
+
+  it("skips the unfilled template placeholder", () => {
+    expect(
+      parseIdentityNameFromMarkdown("- **Name:** _(pick something you like)_"),
+    ).toBeUndefined();
+  });
+});

package/src/http/handlers/agents-list.ts

@@ -1,5 +1,10 @@
 import type { IncomingMessage, ServerResponse } from "node:http";
-import { getFridayAgentForwardRuntime } from "../../agent-forward-runtime.js";
+import fs from "node:fs";
+import path from "node:path";
+import {
+  getFridayAgentForwardRuntime,
+  type FridayAgentForwardRuntime,
+} from "../../agent-forward-runtime.js";
 import { extractBearerToken } from "../middleware/auth.js";
 
 const DEFAULT_AGENT_ID = "main";
@@ -54,6 +59,60 @@ function readString(value: unknown): string | undefined {
   return typeof value === "string" && value.trim() ? value.trim() : undefined;
 }
 
+/** Unfilled IDENTITY.md template prompts that must not surface as a real name. */
+const IDENTITY_NAME_PLACEHOLDERS = new Set(["pick something you like"]);
+
+/**
+ * Extract the `Name` field from an agent's IDENTITY.md, mirroring OpenClaw's
+ * `parseIdentityMarkdown` (src/agents/identity-file.ts) for the name label only:
+ * drop the leading "- ", split on the first ":", strip markdown emphasis, and
+ * skip the unfilled template placeholder. Returns the raw value verbatim (e.g.
+ * "星期五 (Friday)") so it matches what ControlUI shows under "身份名称".
+ */
+export function parseIdentityNameFromMarkdown(content: string): string | undefined {
+  for (const rawLine of content.split(/\r?\n/)) {
+    const cleaned = rawLine.trim().replace(/^\s*-\s*/, "");
+    const colonIndex = cleaned.indexOf(":");
+    if (colonIndex === -1) continue;
+    const label = cleaned.slice(0, colonIndex).replace(/[*_`]/g, "").trim().toLowerCase();
+    if (label !== "name") continue;
+    const value = cleaned
+      .slice(colonIndex + 1)
+      .replace(/^[*_`\s]+|[*_`\s]+$/g, "")
+      .trim();
+    if (!value) continue;
+    let normalized = value.replace(/[–—]/g, "-");
+    if (normalized.startsWith("(") && normalized.endsWith(")")) {
+      normalized = normalized.slice(1, -1).trim();
+    }
+    if (IDENTITY_NAME_PLACEHOLDERS.has(normalized.toLowerCase())) continue;
+    return value;
+  }
+  return undefined;
+}
+
+/**
+ * Name fallback for agents with no `name`/`identity.name` in config (e.g. the
+ * implicit `main`): resolve the agent's workspace and parse its IDENTITY.md, the
+ * same source ControlUI reads. Best-effort — any failure yields undefined.
+ */
+function readWorkspaceIdentityName(
+  rt: FridayAgentForwardRuntime,
+  cfg: unknown,
+  agentId: string,
+): string | undefined {
+  const resolveWorkspace = rt.resolveAgentWorkspaceDir;
+  if (!resolveWorkspace) return undefined;
+  try {
+    const workspace = resolveWorkspace(cfg, agentId);
+    if (!workspace) return undefined;
+    const content = fs.readFileSync(path.join(workspace, "IDENTITY.md"), "utf-8");
+    return parseIdentityNameFromMarkdown(content);
+  } catch {
+    return undefined;
+  }
+}
+
 /**
  * Reads the configured agents directly from the runtime config (same approach as
  * models-list.ts). When no agents are configured OpenClaw runs an implicit "main"
@@ -89,7 +148,10 @@ function resolveConfiguredAgents(): ResolvedAgents {
     const identity = agent.identity as Record<string, unknown> | undefined;
     entries.push({
       id,
-      name: readString(agent.name) ?? readString(identity?.name),
+      name:
+        readString(agent.name) ??
+        readString(identity?.name) ??
+        readWorkspaceIdentityName(rt, cfg, id),
       description: readString(agent.description),
       model: resolvePrimaryModel(agent.model),
       thinkingDefault: readString(agent.thinkingDefault),