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

推荐订阅源

罗磊的独立博客
U
Unit 42
N
Netflix TechBlog - Medium
人人都是产品经理
人人都是产品经理
Hugging Face - Blog
Hugging Face - Blog
腾讯CDC
小众软件
小众软件
V
Visual Studio Blog
T
Tailwind CSS Blog
Engineering at Meta
Engineering at Meta
博客园 - 叶小钗
GbyAI
GbyAI
爱范儿
爱范儿
雷峰网
雷峰网
Microsoft Azure Blog
Microsoft Azure Blog
D
DataBreaches.Net
博客园_首页
D
Docker
A
About on SuperTechFans
G
Google Developers Blog
I
InfoQ
T
The Blog of Author Tim Ferriss
V
V2EX
博客园 - Franky

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(gateway): scope tools.effective global agent lookup ·...
Pick-cat · 2026-06-29 · via Recent Commits to openclaw:main
1+

// Integration proof for tools.effective global sessions scoped to non-default agents.

2+

import path from "node:path";

3+

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

4+

import { installGatewayTestHooks, testState, writeSessionStore } from "../test-helpers.js";

5+

import { getGatewayConfigModule, sessionStoreEntry } from "../test/server-sessions.test-helpers.js";

6+

import { testing, toolsEffectiveHandlers } from "./tools-effective.js";

7+8+

const inventoryMocks = vi.hoisted(() => ({

9+

resolveEffectiveToolInventory: vi.fn(

10+

(params: { agentId: string; modelProvider?: string; modelId?: string }) => ({

11+

agentId: params.agentId,

12+

profile: "coding",

13+

groups: [

14+

{

15+

id: "core",

16+

label: "Built-in tools",

17+

source: "core",

18+

tools: [

19+

{

20+

id: "exec",

21+

label: "Exec",

22+

description: "Run shell commands",

23+

source: "core",

24+

},

25+

],

26+

},

27+

],

28+

modelProvider: params.modelProvider,

29+

modelId: params.modelId,

30+

}),

31+

),

32+

resolveEffectiveToolInventoryRuntimeModelContext: vi.fn(() => ({

33+

modelApi: "openai-responses",

34+

runtimeModel: {

35+

id: "work-model",

36+

name: "Work model",

37+

provider: "openai",

38+

api: "openai-responses",

39+

baseUrl: "https://api.openai.com/v1",

40+

},

41+

})),

42+

}));

43+44+

vi.mock("./tools-effective.runtime.js", async (importOriginal) => {

45+

const actual = await importOriginal<typeof import("./tools-effective.runtime.js")>();

46+

return {

47+

...actual,

48+

resolveEffectiveToolInventory: inventoryMocks.resolveEffectiveToolInventory,

49+

resolveEffectiveToolInventoryRuntimeModelContext:

50+

inventoryMocks.resolveEffectiveToolInventoryRuntimeModelContext,

51+

peekSessionMcpRuntime: vi.fn(() => undefined),

52+

resolveSessionMcpConfigSummary: vi.fn(() => ({ fingerprint: "mcp:0", serverNames: [] })),

53+

buildBundleMcpToolsFromCatalog: vi.fn(() => []),

54+

applyFinalEffectiveToolPolicy: vi.fn(

55+

(params: { bundledTools: unknown[] }) => params.bundledTools,

56+

),

57+

getActivePluginRegistryVersion: vi.fn(() => 1),

58+

getActivePluginChannelRegistryVersion: vi.fn(() => 1),

59+

};

60+

});

61+62+

installGatewayTestHooks();

63+64+

describe("tools.effective global agent integration", () => {

65+

let mainStorePath = "";

66+

let workStorePath = "";

67+

let getRuntimeConfig: Awaited<ReturnType<typeof getGatewayConfigModule>>["getRuntimeConfig"];

68+69+

async function seedSelectedGlobalStores() {

70+

const stateDir = process.env.OPENCLAW_STATE_DIR;

71+

if (!stateDir) {

72+

throw new Error("OPENCLAW_STATE_DIR is required");

73+

}

74+

const dir = path.join(stateDir, "session-stores", `tools-effective-${Date.now()}`);

75+

const storeTemplate = path.join(dir, "{agentId}", "sessions.json");

76+

testState.sessionStorePath = storeTemplate;

77+

testState.sessionConfig = { scope: "global" };

78+

testState.agentsConfig = { list: [{ id: "main", default: true }, { id: "work" }] };

79+

mainStorePath = storeTemplate.replace("{agentId}", "main");

80+

workStorePath = storeTemplate.replace("{agentId}", "work");

81+

const configModule = await getGatewayConfigModule();

82+

configModule.clearRuntimeConfigSnapshot();

83+

configModule.clearConfigCache();

84+

getRuntimeConfig = configModule.getRuntimeConfig;

85+

}

86+87+

beforeEach(async () => {

88+

testing.resetToolsEffectiveCacheForTest();

89+

vi.clearAllMocks();

90+

await seedSelectedGlobalStores();

91+

});

92+93+

it("resolves tools.effective for global session scoped to a non-default agent store", async () => {

94+

await writeSessionStore({

95+

storePath: mainStorePath,

96+

entries: {

97+

global: sessionStoreEntry("sess-main-global", {

98+

modelProvider: "openai",

99+

model: "main-model",

100+

}),

101+

},

102+

});

103+

await writeSessionStore({

104+

storePath: workStorePath,

105+

agentId: "work",

106+

entries: {

107+

global: sessionStoreEntry("sess-work-global", {

108+

modelProvider: "openai",

109+

model: "work-model",

110+

}),

111+

},

112+

});

113+114+

const respond = vi.fn();

115+

await toolsEffectiveHandlers["tools.effective"]({

116+

params: { sessionKey: "global", agentId: "work" },

117+

respond: respond as never,

118+

context: { getRuntimeConfig } as never,

119+

client: null,

120+

req: { type: "req", id: "req-tools-effective-global", method: "tools.effective" },

121+

isWebchatConnect: () => false,

122+

});

123+124+

const call = respond.mock.calls[0] as [boolean, { agentId?: string }?, unknown?] | undefined;

125+

expect(call?.[0]).toBe(true);

126+

expect(call?.[1]?.agentId).toBe("work");

127+

expect(inventoryMocks.resolveEffectiveToolInventory).toHaveBeenCalledWith(

128+

expect.objectContaining({

129+

agentId: "work",

130+

sessionKey: "global",

131+

modelProvider: "openai",

132+

modelId: "work-model",

133+

}),

134+

);

135+

});

136+137+

// Negative control on the real session-resolution path: a non-global key owned

138+

// by `main` must keep rejecting a mismatched configured agent. Before the

139+

// ownership-narrowing fix the requested agent overrode session-agent resolution

140+

// here, so this request would have succeeded under `work`.

141+

it("rejects a mismatched configured agent for a non-global session key", async () => {

142+

await seedNonGlobalMainStore();

143+144+

await writeSessionStore({

145+

storePath: mainStorePath,

146+

entries: {

147+

"agent:main:abc": sessionStoreEntry("sess-main-agent", {

148+

modelProvider: "openai",

149+

model: "main-model",

150+

}),

151+

},

152+

});

153+154+

const respond = vi.fn();

155+

await toolsEffectiveHandlers["tools.effective"]({

156+

params: { sessionKey: "agent:main:abc", agentId: "work" },

157+

respond: respond as never,

158+

context: { getRuntimeConfig } as never,

159+

client: null,

160+

req: { type: "req", id: "req-tools-effective-mismatch", method: "tools.effective" },

161+

isWebchatConnect: () => false,

162+

});

163+164+

const call = respond.mock.calls[0] as

165+

| [boolean, unknown?, { code: number; message: string }?]

166+

| undefined;

167+

expect(call?.[0]).toBe(false);

168+

expect(call?.[2]?.message).toBe('agent id "work" does not match session agent "main"');

169+

expect(inventoryMocks.resolveEffectiveToolInventory).not.toHaveBeenCalled();

170+

});

171+172+

async function seedNonGlobalMainStore() {

173+

const stateDir = process.env.OPENCLAW_STATE_DIR;

174+

if (!stateDir) {

175+

throw new Error("OPENCLAW_STATE_DIR is required");

176+

}

177+

const dir = path.join(stateDir, "session-stores", `tools-effective-nonglobal-${Date.now()}`);

178+

const storeTemplate = path.join(dir, "{agentId}", "sessions.json");

179+

testState.sessionStorePath = storeTemplate;

180+

testState.sessionConfig = undefined;

181+

testState.agentsConfig = { list: [{ id: "main", default: true }, { id: "work" }] };

182+

mainStorePath = storeTemplate.replace("{agentId}", "main");

183+

const configModule = await getGatewayConfigModule();

184+

configModule.clearRuntimeConfigSnapshot();

185+

configModule.clearConfigCache();

186+

getRuntimeConfig = configModule.getRuntimeConfig;

187+

}

188+

});