From b96f5d94dba6176ca04dd01f405e241e8d7cf432 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 27 Mar 2026 02:26:30 +0000 Subject: [PATCH] chore(telegram): downgrade default network logs --- extensions/telegram/src/fetch.test.ts | 28 +++++++++++++++++++++++++++ extensions/telegram/src/fetch.ts | 4 ++-- 2 files changed, 30 insertions(+), 2 deletions(-) diff --git a/extensions/telegram/src/fetch.test.ts b/extensions/telegram/src/fetch.test.ts index 55f6696f907..098868aa9b7 100644 --- a/extensions/telegram/src/fetch.test.ts +++ b/extensions/telegram/src/fetch.test.ts @@ -2,6 +2,8 @@ import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vite const setDefaultResultOrder = vi.hoisted(() => vi.fn()); const setDefaultAutoSelectFamily = vi.hoisted(() => vi.fn()); +const loggerInfo = vi.hoisted(() => vi.fn()); +const loggerDebug = vi.hoisted(() => vi.fn()); const undiciFetch = vi.hoisted(() => vi.fn()); const setGlobalDispatcher = vi.hoisted(() => vi.fn()); @@ -54,6 +56,21 @@ vi.mock("undici", () => ({ setGlobalDispatcher, })); +vi.mock("openclaw/plugin-sdk/runtime-env", () => ({ + createSubsystemLogger: () => ({ + info: loggerInfo, + debug: loggerDebug, + warn: vi.fn(), + error: vi.fn(), + child: () => ({ + info: loggerInfo, + debug: loggerDebug, + warn: vi.fn(), + error: vi.fn(), + }), + }), +})); + let resolveFetch: typeof import("../../../src/infra/fetch.js").resolveFetch; let resolveTelegramFetch: typeof import("./fetch.js").resolveTelegramFetch; let resolveTelegramTransport: typeof import("./fetch.js").resolveTelegramTransport; @@ -66,6 +83,8 @@ beforeAll(async () => { beforeEach(() => { vi.unstubAllEnvs(); + loggerInfo.mockReset(); + loggerDebug.mockReset(); }); function resolveTelegramFetchOrThrow( @@ -272,6 +291,15 @@ describe("resolveTelegramFetch", () => { expect(typeof dispatcher?.options?.connect?.lookup).toBe("function"); }); + it("emits default transport decisions at debug level", () => { + resolveTelegramFetchOrThrow(); + + expect(loggerInfo).not.toHaveBeenCalledWith("autoSelectFamily=true (default-node22)"); + expect(loggerInfo).not.toHaveBeenCalledWith("dnsResultOrder=ipv4first (default-node22)"); + expect(loggerDebug).toHaveBeenCalledWith("autoSelectFamily=true (default-node22)"); + expect(loggerDebug).toHaveBeenCalledWith("dnsResultOrder=ipv4first (default-node22)"); + }); + it("uses EnvHttpProxyAgent dispatcher when proxy env is configured", async () => { vi.stubEnv("HTTPS_PROXY", "http://127.0.0.1:7890"); undiciFetch.mockResolvedValue({ ok: true } as Response); diff --git a/extensions/telegram/src/fetch.ts b/extensions/telegram/src/fetch.ts index 370d01d2c3c..6aa47c4f057 100644 --- a/extensions/telegram/src/fetch.ts +++ b/extensions/telegram/src/fetch.ts @@ -348,11 +348,11 @@ function logResolverNetworkDecisions(params: { const sourceLabel = params.autoSelectDecision.source ? ` (${params.autoSelectDecision.source})` : ""; - log.info(`autoSelectFamily=${params.autoSelectDecision.value}${sourceLabel}`); + log.debug(`autoSelectFamily=${params.autoSelectDecision.value}${sourceLabel}`); } if (params.dnsDecision.value !== null) { const sourceLabel = params.dnsDecision.source ? ` (${params.dnsDecision.source})` : ""; - log.info(`dnsResultOrder=${params.dnsDecision.value}${sourceLabel}`); + log.debug(`dnsResultOrder=${params.dnsDecision.value}${sourceLabel}`); } }