@syengup/friday-channel-next 0.1.19 → 0.1.21

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/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/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/install.js

@@ -96,7 +96,6 @@ if (!hasOpenclaw()) {
 // --------------- install plugin package ---------------
 
 log("Installing Friday Next channel plugin...");
-let installed = false;
 
 try {
   const out = execSync(
@@ -104,7 +103,6 @@ try {
     { encoding: "utf8", stdio: "pipe", timeout: 120000 }
   );
   if (out.trim()) console.log(out.trim());
-  installed = true;
   log("Plugin registered with install record — auto-upgrade enabled.");
 
   // Remove old manual install to avoid "duplicate plugin id" warning.
@@ -115,25 +113,9 @@ try {
   }
 } catch (e) {
   const msg = (e.stderr || e.stdout || e.message || "").toString();
-  warn("openclaw plugins install failed: " + msg.trim().split("\n").pop());
-  warn("Falling back to manual install...");
-}
-
-// --------------- fallback: manual install ---------------
-
-if (!installed) {
-  const PKG = has("npm") ? "npm" : has("pnpm") ? "pnpm" : null;
-  if (!PKG) {
-    err("npm is required for manual install. Install Node.js first.");
-    process.exit(1);
-  }
-  warn("Manual install complete, but auto-upgrade is NOT available.");
-  warn("To enable auto-upgrade later, run: openclaw plugins install @syengup/friday-channel-next --force");
-  // Clean up legacy dir even in fallback to avoid duplicate warnings
-  if (existsSync(join(USER_HOME, ".openclaw", "extensions", "friday-channel-next"))) {
-    warn("Legacy install detected. Remove it to avoid duplicate warnings:");
-    warn("  rm -rf ~/.openclaw/extensions/friday-channel-next");
-  }
+  err("Plugin install failed: " + (msg.trim().split("\n").pop() || "unknown error"));
+  err("Fix the error above and re-run: npx -y @syengup/friday-channel-next");
+  process.exit(1);
 }
 
 // --------------- configure OpenClaw ---------------
@@ -326,17 +308,24 @@ async function verifyGateway(url, token, retries = 30) {
 log("Verifying gateway...");
 const verified = await verifyGateway(verifyUrl, gatewayToken);
 
-// --------------- show connection info ---------------
-
 const BOLD_YELLOW = (s) => `\x1b[1;33m${s}\x1b[0m`;
 
-log("--------------------------------------------------");
-if (verified) {
-  log("Installation complete! Friday Next channel is now active.");
-} else {
-  warn("Installation complete, but gateway verification failed.");
-  warn("Check 'openclaw gateway status' and restart if needed.");
+// Hard gate: if the gateway didn't verify, the install did NOT succeed — stop here
+// with a non-zero exit and do NOT print the QR / URL / "complete" block, so a failure
+// can never look like a success.
+if (!verified) {
+  log("--------------------------------------------------");
+  err("Installation FAILED: the Friday Next gateway did not come up.");
+  err("Diagnose with:  openclaw gateway status");
+  err("Then restart:   openclaw gateway restart");
+  err("And re-run:     npx -y @syengup/friday-channel-next");
+  process.exit(1);
 }
+
+// --------------- show connection info ---------------
+
+log("--------------------------------------------------");
+log("Installation complete! Friday Next channel is now active.");
 log("");
 
 // --------------- QR code ---------------

package/package.json

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

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;
   }