惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

美团技术团队
人人都是产品经理
人人都是产品经理
月光博客
月光博客
V
V2EX
WordPress大学
WordPress大学
酷 壳 – CoolShell
酷 壳 – CoolShell
Last Week in AI
Last Week in AI
博客园 - 三生石上(FineUI控件)
小众软件
小众软件
Hugging Face - Blog
Hugging Face - Blog
V
Visual Studio Blog
宝玉的分享
宝玉的分享
雷峰网
雷峰网
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
博客园 - Franky
博客园 - 聂微东
博客园 - 司徒正美
博客园 - 【当耐特】
爱范儿
爱范儿
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
大猫的无限游戏
大猫的无限游戏
博客园 - 叶小钗
阮一峰的网络日志
阮一峰的网络日志

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(tools): release bounded web response readers · opencl...
vincentkoc · 2026-06-19 · via Recent Commits to openclaw:main
1-

// Shared web helper tests cover timeout normalization and process-local cache

2-

// expiry guards.

1+

// Shared web helper tests cover timeout normalization, process-local cache

2+

// expiry guards, and bounded response body cleanup.

33

import {

44

MAX_TIMER_TIMEOUT_MS,

55

MAX_TIMER_TIMEOUT_SECONDS,

66

} from "@openclaw/normalization-core/number-coercion";

77

import { afterEach, describe, expect, it, vi } from "vitest";

88

import {

99

readCache,

10+

readResponseText,

1011

resolvePositiveTimeoutSeconds,

1112

resolveTimeoutSeconds,

1213

withTimeout,

@@ -18,6 +19,28 @@ afterEach(() => {

1819

vi.restoreAllMocks();

1920

});

202122+

function responseFromReader(params: {

23+

chunks: string[];

24+

cancel: () => Promise<void>;

25+

releaseLock: () => void;

26+

}): Response {

27+

const chunks: Array<ReadableStreamReadResult<Uint8Array>> = params.chunks.map((chunk) => ({

28+

done: false,

29+

value: new TextEncoder().encode(chunk),

30+

}));

31+

chunks.push({ done: true, value: undefined });

32+

const reader = {

33+

read: async () => chunks.shift() ?? { done: true, value: undefined },

34+

cancel: params.cancel,

35+

releaseLock: params.releaseLock,

36+

} as ReadableStreamDefaultReader<Uint8Array>;

37+38+

return {

39+

body: { getReader: () => reader },

40+

headers: new Headers({ "content-type": "text/plain; charset=utf-8" }),

41+

} as Response;

42+

}

43+2144

describe("web shared timeout seconds", () => {

2245

it("caps timeoutSeconds at the shared timer-safe ceiling", () => {

2346

expect(resolveTimeoutSeconds(Number.MAX_SAFE_INTEGER, 30)).toBe(MAX_TIMER_TIMEOUT_SECONDS);

@@ -86,3 +109,41 @@ describe("web shared withTimeout", () => {

86109

expect(setTimeoutSpy).toHaveBeenCalledWith(expect.any(Function), MAX_TIMER_TIMEOUT_MS);

87110

});

88111

});

112+113+

describe("readResponseText", () => {

114+

it("releases bounded response readers after complete reads", async () => {

115+

const cancel = vi.fn(async () => undefined);

116+

const releaseLock = vi.fn();

117+

const response = responseFromReader({

118+

chunks: ["hello", " world"],

119+

cancel,

120+

releaseLock,

121+

});

122+123+

await expect(readResponseText(response, { maxBytes: 64 })).resolves.toEqual({

124+

text: "hello world",

125+

truncated: false,

126+

bytesRead: 11,

127+

});

128+

expect(cancel).not.toHaveBeenCalled();

129+

expect(releaseLock).toHaveBeenCalledTimes(1);

130+

});

131+132+

it("cancels and releases bounded response readers after truncation", async () => {

133+

const cancel = vi.fn(async () => undefined);

134+

const releaseLock = vi.fn();

135+

const response = responseFromReader({

136+

chunks: ["hello world"],

137+

cancel,

138+

releaseLock,

139+

});

140+141+

await expect(readResponseText(response, { maxBytes: 5 })).resolves.toEqual({

142+

text: "hello",

143+

truncated: true,

144+

bytesRead: 5,

145+

});

146+

expect(cancel).toHaveBeenCalledTimes(1);

147+

expect(releaseLock).toHaveBeenCalledTimes(1);

148+

});

149+

});