@syengup/friday-channel-next 0.1.16 → 0.1.18

package/dist/src/http/handlers/messages.js

@@ -23,12 +23,15 @@ import { fridayAttachmentLookupKey, fridayFilesPublicUrl, readFile, resolveMedia
 import { runFridayDispatch } from "../../agent/dispatch-bridge.js";
 import { saveInboundMediaBuffer } from "../../agent/media-bridge.js";
 import { contextTokensFromUsageRecord, getRunMetadata, getRunRoute, hasRunFinalDelivered, markRunFinalDelivered, registerRunRoute, setRunMetadata, } from "../../run-metadata.js";
-import { createFridayNextLogger } from "../../logging.js";
+import { createFridayNextLogger, setFridayNextLogLevel } from "../../logging.js";
 const logger = createFridayNextLogger("messages");
-const log = (action, deviceId, runId, detail) => {
+// Routine per-message / per-stream lifecycle events log at "debug" so they stay out of
+// the default ("info") OpenClaw log; only genuine problems (rejections, run errors) surface.
+// Raise the friday-next channel logLevel to "debug" to see the full per-message trace.
+const log = (action, deviceId, runId, detail, level = "debug") => {
     const runPart = runId ? ` runId=${runId}` : "";
     const detailPart = detail ? ` detail=${detail}` : "";
-    logger.info(`[${action}] deviceId=${deviceId}${runPart}${detailPart}`);
+    logger[level](`[${action}] deviceId=${deviceId}${runPart}${detailPart}`);
 };
 function collectReplyPayloadMediaUrls(pl) {
     const fromArr = Array.isArray(pl.mediaUrls)
@@ -285,7 +288,7 @@ export async function handleMessages(req, res) {
     }
     const token = extractBearerToken(req);
     if (!token) {
-        log("AUTH_FAILED", "(unknown)", undefined, "missing or invalid token");
+        log("AUTH_FAILED", "(unknown)", undefined, "missing or invalid token", "warn");
         res.statusCode = 401;
         res.setHeader("Content-Type", "application/json");
         res.end(JSON.stringify({ error: "Unauthorized: bearer token mismatch" }));
@@ -293,7 +296,7 @@ export async function handleMessages(req, res) {
     }
     const payload = (await readJsonBody(req));
     if (!payload) {
-        log("BAD_REQUEST", "(unknown)", undefined, "invalid JSON body");
+        log("BAD_REQUEST", "(unknown)", undefined, "invalid JSON body", "warn");
         res.statusCode = 400;
         res.setHeader("Content-Type", "application/json");
         res.end(JSON.stringify({ error: "Invalid JSON body" }));
@@ -302,7 +305,7 @@ export async function handleMessages(req, res) {
     const { deviceId, text, attachments = [], sessionKey: rawSessionKey } = payload;
     const normalizedDeviceId = deviceId?.trim().toUpperCase();
     if (typeof rawSessionKey !== "string" || !rawSessionKey.length) {
-        log("BAD_REQUEST", "(unknown)", undefined, "missing sessionKey");
+        log("BAD_REQUEST", "(unknown)", undefined, "missing sessionKey", "warn");
         res.statusCode = 400;
         res.setHeader("Content-Type", "application/json");
         res.end(JSON.stringify({ error: "Missing required field: sessionKey" }));
@@ -311,14 +314,14 @@ export async function handleMessages(req, res) {
     const appSessionKey = rawSessionKey.trim();
     const baseSessionKey = toSessionStoreKey(appSessionKey);
     if (!normalizedDeviceId) {
-        log("BAD_REQUEST", "(unknown)", undefined, "missing deviceId");
+        log("BAD_REQUEST", "(unknown)", undefined, "missing deviceId", "warn");
         res.statusCode = 400;
         res.setHeader("Content-Type", "application/json");
         res.end(JSON.stringify({ error: "Missing required field: deviceId" }));
         return true;
     }
     if (!text || !text.trim()) {
-        log("BAD_REQUEST", normalizedDeviceId, undefined, "missing text");
+        log("BAD_REQUEST", normalizedDeviceId, undefined, "missing text", "warn");
         res.statusCode = 400;
         res.setHeader("Content-Type", "application/json");
         res.end(JSON.stringify({ error: "Missing required field: text" }));
@@ -334,6 +337,7 @@ export async function handleMessages(req, res) {
     res.end(JSON.stringify({ accepted: true, deviceId: normalizedDeviceId, runId }));
     log("MESSAGE_RECEIVED", normalizedDeviceId, runId, `textLen=${trimmedText.length} attachments=${attachments.length} sessionKey=${baseSessionKey}`);
     const cfg = resolveFridayNextConfig(getHostOpenClawConfigSnapshot(runtime.config));
+    setFridayNextLogLevel(cfg.logLevel);
     // Resolve defaults from the OpenClaw agent config so settings are never left empty. Prefers the
     // target agent's own model/thinking over the global defaults (see resolveAgentDefaults).
     const { model: defaultModel, thinking: defaultThinking } = resolveAgentDefaults(baseSessionKey);
@@ -415,7 +419,7 @@ export async function handleMessages(req, res) {
                         }
                     },
                     onError: (err) => {
-                        log("RUN_ERROR", normalizedDeviceId, runId, String(err));
+                        log("RUN_ERROR", normalizedDeviceId, runId, String(err), "error");
                         sseEmitter.broadcastToRun(runId, {
                             type: "outbound",
                             data: {
@@ -455,7 +459,7 @@ export async function handleMessages(req, res) {
             log("RUN_COMPLETE", normalizedDeviceId, runId);
         }
         catch (err) {
-            log("RUN_ERROR", normalizedDeviceId, runId, String(err));
+            log("RUN_ERROR", normalizedDeviceId, runId, String(err), "error");
             sseEmitter.broadcastToRun(runId, {
                 type: "outbound",
                 data: {
@@ -473,7 +477,7 @@ export async function handleMessages(req, res) {
         }
     };
     runAgent().catch((err) => {
-        log("RUN_ERROR", normalizedDeviceId, runId, String(err));
+        log("RUN_ERROR", normalizedDeviceId, runId, String(err), "error");
         sseEmitter.untrackRun(runId);
     });
     return true;

package/dist/src/logging.d.ts

@@ -1,5 +1,6 @@
 import type { FridayNextLogLevel } from "./config.js";
-export declare function createFridayNextLogger(scope: string, level?: FridayNextLogLevel): {
+export declare function setFridayNextLogLevel(level: FridayNextLogLevel): void;
+export declare function createFridayNextLogger(scope: string, _level?: FridayNextLogLevel): {
     debug: (message: string) => void;
     info: (message: string) => void;
     warn: (message: string) => void;

package/dist/src/logging.js

@@ -4,9 +4,18 @@ const levelOrder = {
     warn: 30,
     error: 40,
 };
-export function createFridayNextLogger(scope, level = "info") {
+// Process-wide active level, honored by every logger at call time. Handlers call
+// setFridayNextLogLevel() once the friday-next channel config resolves, so
+// `channels.friday-next.logLevel: "debug"` turns the full per-message trace back on.
+let activeLevel = "info";
+export function setFridayNextLogLevel(level) {
+    activeLevel = level;
+}
+// The optional `level` arg is accepted for call-site compatibility but the effective
+// threshold is the process-wide active level (single knob driven by config.logLevel).
+export function createFridayNextLogger(scope, _level) {
     const base = `[friday-next:${scope}]`;
-    const enabled = (current) => levelOrder[current] >= levelOrder[level];
+    const enabled = (current) => levelOrder[current] >= levelOrder[activeLevel];
     return {
         debug: (message) => {
             if (enabled("debug"))

package/install.js

@@ -385,18 +385,24 @@ if (ipType === "tailscale") {
   log("This URL appears to be publicly accessible (" + ip + ").");
 }
 
-// On a cloud server the advertised IP is the internal/NAT address, which a phone
-// over the internet can't reach. Best-effort: detect the public IP so the user
-// has the address to actually connect with (and a reminder to open the port).
+// The advertised IP is the local/NAT address, which a phone over the internet
+// can't reach. Best-effort: detect the network's public-facing IP. NOTE: an echo
+// service only reports the egress address — on a cloud VPS that's a routable 1:1
+// NAT, but on a home/carrier-grade NAT (CGNAT) it's a shared egress IP that is
+// NOT reachable inbound. So we present it as a hint, not a guarantee.
 if (ipType === "private" || ipType === "loopback") {
   const publicIp = await detectPublicIp();
   if (publicIp && publicIp !== ip) {
     log("");
-    log(BOLD_YELLOW("Looks like a cloud server. For remote access use the PUBLIC address:"));
-    log(BOLD_YELLOW("检测到云服务器，远程连接请改用公网地址："));
+    log(BOLD_YELLOW("Network public-facing IP detected — for remote access try:"));
+    log(BOLD_YELLOW("检测到网络出口公网 IP —— 远程连接可尝试："));
     log("Public URL:   " + BOLD_YELLOW(`http://${publicIp}:${gatewayPort}`));
-    log("(Open inbound TCP port " + gatewayPort + " in your firewall / security group first.)");
-    log("(需先在防火墙/安全组放行入站 TCP 端口 " + gatewayPort + "。)");
+    log("First open inbound TCP port " + gatewayPort + " in your firewall / cloud security group.");
+    log("请先在防火墙 / 云安全组放行入站 TCP 端口 " + gatewayPort + "。");
+    log("If this is a home/carrier network (CGNAT), this IP may NOT be reachable from outside —");
+    log("use a tunnel like Tailscale instead.");
+    log("若为家庭宽带 / 运营商大内网(CGNAT)，此地址可能无法从外部入站访问，");
+    log("请改用 Tailscale 等内网穿透方案。");
   }
 }
 log("--------------------------------------------------");

package/package.json

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

package/src/http/handlers/messages.ts

@@ -57,14 +57,23 @@ import {
   registerRunRoute,
   setRunMetadata,
 } from "../../run-metadata.js";
-import { createFridayNextLogger } from "../../logging.js";
+import { createFridayNextLogger, setFridayNextLogLevel } from "../../logging.js";
 
 const logger = createFridayNextLogger("messages");
 
-const log = (action: string, deviceId: string, runId?: string, detail?: string) => {
+// Routine per-message / per-stream lifecycle events log at "debug" so they stay out of
+// the default ("info") OpenClaw log; only genuine problems (rejections, run errors) surface.
+// Raise the friday-next channel logLevel to "debug" to see the full per-message trace.
+const log = (
+  action: string,
+  deviceId: string,
+  runId?: string,
+  detail?: string,
+  level: "debug" | "info" | "warn" | "error" = "debug",
+) => {
   const runPart = runId ? ` runId=${runId}` : "";
   const detailPart = detail ? ` detail=${detail}` : "";
-  logger.info(`[${action}] deviceId=${deviceId}${runPart}${detailPart}`);
+  logger[level](`[${action}] deviceId=${deviceId}${runPart}${detailPart}`);
 };
 
 function collectReplyPayloadMediaUrls(pl: { mediaUrls?: string[]; mediaUrl?: string | null }): string[] {
@@ -381,7 +390,7 @@ export async function handleMessages(req: IncomingMessage, res: ServerResponse):
 
   const token = extractBearerToken(req);
   if (!token) {
-    log("AUTH_FAILED", "(unknown)", undefined, "missing or invalid token");
+    log("AUTH_FAILED", "(unknown)", undefined, "missing or invalid token", "warn");
     res.statusCode = 401;
     res.setHeader("Content-Type", "application/json");
     res.end(JSON.stringify({ error: "Unauthorized: bearer token mismatch" }));
@@ -390,7 +399,7 @@ export async function handleMessages(req: IncomingMessage, res: ServerResponse):
 
   const payload = (await readJsonBody(req)) as FridayMessagePayload | null;
   if (!payload) {
-    log("BAD_REQUEST", "(unknown)", undefined, "invalid JSON body");
+    log("BAD_REQUEST", "(unknown)", undefined, "invalid JSON body", "warn");
     res.statusCode = 400;
     res.setHeader("Content-Type", "application/json");
     res.end(JSON.stringify({ error: "Invalid JSON body" }));
@@ -401,7 +410,7 @@ export async function handleMessages(req: IncomingMessage, res: ServerResponse):
   const normalizedDeviceId = deviceId?.trim().toUpperCase();
 
   if (typeof rawSessionKey !== "string" || !rawSessionKey.length) {
-    log("BAD_REQUEST", "(unknown)", undefined, "missing sessionKey");
+    log("BAD_REQUEST", "(unknown)", undefined, "missing sessionKey", "warn");
     res.statusCode = 400;
     res.setHeader("Content-Type", "application/json");
     res.end(JSON.stringify({ error: "Missing required field: sessionKey" }));
@@ -412,7 +421,7 @@ export async function handleMessages(req: IncomingMessage, res: ServerResponse):
   const baseSessionKey = toSessionStoreKey(appSessionKey);
 
   if (!normalizedDeviceId) {
-    log("BAD_REQUEST", "(unknown)", undefined, "missing deviceId");
+    log("BAD_REQUEST", "(unknown)", undefined, "missing deviceId", "warn");
     res.statusCode = 400;
     res.setHeader("Content-Type", "application/json");
     res.end(JSON.stringify({ error: "Missing required field: deviceId" }));
@@ -420,7 +429,7 @@ export async function handleMessages(req: IncomingMessage, res: ServerResponse):
   }
 
   if (!text || !text.trim()) {
-    log("BAD_REQUEST", normalizedDeviceId, undefined, "missing text");
+    log("BAD_REQUEST", normalizedDeviceId, undefined, "missing text", "warn");
     res.statusCode = 400;
     res.setHeader("Content-Type", "application/json");
     res.end(JSON.stringify({ error: "Missing required field: text" }));
@@ -447,6 +456,7 @@ export async function handleMessages(req: IncomingMessage, res: ServerResponse):
   );
 
   const cfg = resolveFridayNextConfig(getHostOpenClawConfigSnapshot(runtime.config));
+  setFridayNextLogLevel(cfg.logLevel);
 
   // Resolve defaults from the OpenClaw agent config so settings are never left empty. Prefers the
   // target agent's own model/thinking over the global defaults (see resolveAgentDefaults).
@@ -544,7 +554,7 @@ export async function handleMessages(req: IncomingMessage, res: ServerResponse):
             }
           },
           onError: (err: unknown) => {
-            log("RUN_ERROR", normalizedDeviceId, runId, String(err));
+            log("RUN_ERROR", normalizedDeviceId, runId, String(err), "error");
             sseEmitter.broadcastToRun(
               runId,
               {
@@ -588,7 +598,7 @@ export async function handleMessages(req: IncomingMessage, res: ServerResponse):
       });
       log("RUN_COMPLETE", normalizedDeviceId, runId);
     } catch (err) {
-      log("RUN_ERROR", normalizedDeviceId, runId, String(err));
+      log("RUN_ERROR", normalizedDeviceId, runId, String(err), "error");
       sseEmitter.broadcastToRun(
         runId,
         {
@@ -610,7 +620,7 @@ export async function handleMessages(req: IncomingMessage, res: ServerResponse):
   };
 
   runAgent().catch((err) => {
-    log("RUN_ERROR", normalizedDeviceId, runId, String(err));
+    log("RUN_ERROR", normalizedDeviceId, runId, String(err), "error");
     sseEmitter.untrackRun(runId);
   });
 

package/src/logging.ts

@@ -7,9 +7,20 @@ const levelOrder: Record<FridayNextLogLevel, number> = {
   error: 40,
 };
 
-export function createFridayNextLogger(scope: string, level: FridayNextLogLevel = "info") {
+// Process-wide active level, honored by every logger at call time. Handlers call
+// setFridayNextLogLevel() once the friday-next channel config resolves, so
+// `channels.friday-next.logLevel: "debug"` turns the full per-message trace back on.
+let activeLevel: FridayNextLogLevel = "info";
+
+export function setFridayNextLogLevel(level: FridayNextLogLevel): void {
+  activeLevel = level;
+}
+
+// The optional `level` arg is accepted for call-site compatibility but the effective
+// threshold is the process-wide active level (single knob driven by config.logLevel).
+export function createFridayNextLogger(scope: string, _level?: FridayNextLogLevel) {
   const base = `[friday-next:${scope}]`;
-  const enabled = (current: FridayNextLogLevel) => levelOrder[current] >= levelOrder[level];
+  const enabled = (current: FridayNextLogLevel) => levelOrder[current] >= levelOrder[activeLevel];
   return {
     debug: (message: string) => {
       if (enabled("debug")) console.debug(`${base} ${message}`);