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

推荐订阅源

L
LangChain Blog
J
Java Code Geeks
P
Proofpoint News Feed
Recent Announcements
Recent Announcements
罗磊的独立博客
H
Hackread – Cybersecurity News, Data Breaches, AI and More
博客园_首页
Hugging Face - Blog
Hugging Face - Blog
MongoDB | Blog
MongoDB | Blog
人人都是产品经理
人人都是产品经理
博客园 - 【当耐特】
雷峰网
雷峰网
D
DataBreaches.Net
B
Blog RSS Feed
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
博客园 - 聂微东
V
Visual Studio Blog
Apple Machine Learning Research
Apple Machine Learning Research
N
Netflix TechBlog - Medium
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
Martin Fowler
Martin Fowler
有赞技术团队
有赞技术团队
Blog — PlanetScale
Blog — PlanetScale
Engineering at Meta
Engineering at Meta

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(plugins): scope tool callbacks during materialization...
steipete · 2026-05-31 · via Recent Commits to openclaw:main

@@ -40,6 +40,8 @@ let resetPluginRuntimeStateForTest: typeof import("./runtime.js").resetPluginRun

4040

let setActivePluginRegistry: typeof import("./runtime.js").setActivePluginRegistry;

4141

let clearCurrentPluginMetadataSnapshot: typeof import("./current-plugin-metadata-snapshot.js").clearCurrentPluginMetadataSnapshot;

4242

let setCurrentPluginMetadataSnapshot: typeof import("./current-plugin-metadata-snapshot.js").setCurrentPluginMetadataSnapshot;

43+

let getPluginRuntimeGatewayRequestScope: typeof import("./runtime/gateway-request-scope.js").getPluginRuntimeGatewayRequestScope;

44+

let withPluginRuntimeGatewayRequestScope: typeof import("./runtime/gateway-request-scope.js").withPluginRuntimeGatewayRequestScope;

43454446

function makeTool(name: string) {

4547

return {

@@ -480,6 +482,8 @@ describe("resolvePluginTools optional tools", () => {

480482

resetPluginRuntimeStateForTest,

481483

setActivePluginRegistry,

482484

} = await import("./runtime.js"));

485+

({ getPluginRuntimeGatewayRequestScope, withPluginRuntimeGatewayRequestScope } =

486+

await import("./runtime/gateway-request-scope.js"));

483487

({ clearCurrentPluginMetadataSnapshot, setCurrentPluginMetadataSnapshot } =

484488

await import("./current-plugin-metadata-snapshot.js"));

485489

});

@@ -510,6 +514,155 @@ describe("resolvePluginTools optional tools", () => {

510514

vi.useRealTimers();

511515

});

512516517+

it("runs plugin tool factories, prepare callbacks, and execute callbacks under the owning plugin scope", async () => {

518+

const context = createContext();

519+

const observed: Array<{

520+

phase: "factory" | "prepare" | "execute";

521+

pluginId?: string;

522+

pluginSource?: string;

523+

}> = [];

524+525+

setRegistry(

526+

["multi", "optional-demo"].map((pluginId) => ({

527+

pluginId,

528+

optional: false,

529+

source: `/tmp/${pluginId}.js`,

530+

names: [`${pluginId}_tool`],

531+

factory: () => {

532+

const scope = getPluginRuntimeGatewayRequestScope();

533+

observed.push({

534+

phase: "factory",

535+

pluginId: scope?.pluginId,

536+

pluginSource: scope?.pluginSource,

537+

});

538+

return {

539+

name: `${pluginId}_tool`,

540+

description: `${pluginId} tool`,

541+

parameters: { type: "object", properties: {} },

542+

prepareArguments(args: unknown) {

543+

const prepareScope = getPluginRuntimeGatewayRequestScope();

544+

observed.push({

545+

phase: "prepare",

546+

pluginId: prepareScope?.pluginId,

547+

pluginSource: prepareScope?.pluginSource,

548+

});

549+

return args;

550+

},

551+

async execute() {

552+

const executeScope = getPluginRuntimeGatewayRequestScope();

553+

observed.push({

554+

phase: "execute",

555+

pluginId: executeScope?.pluginId,

556+

pluginSource: executeScope?.pluginSource,

557+

});

558+

return { content: [{ type: "text", text: pluginId }] };

559+

},

560+

};

561+

},

562+

})),

563+

);

564+565+

await withPluginRuntimeGatewayRequestScope(

566+

{

567+

pluginId: "outer",

568+

pluginSource: "/tmp/outer.js",

569+

isWebchatConnect: () => false,

570+

},

571+

async () => {

572+

const tools = resolvePluginTools(createResolveToolsParams({ context }));

573+

expect(tools.map((tool) => tool.name)).toEqual(["multi_tool", "optional-demo_tool"]);

574+

for (const tool of tools) {

575+

await tool.execute(`call-${tool.name}`, tool.prepareArguments?.({}) ?? {}, undefined);

576+

expect(getPluginRuntimeGatewayRequestScope()).toMatchObject({

577+

pluginId: "outer",

578+

pluginSource: "/tmp/outer.js",

579+

});

580+

}

581+

},

582+

);

583+584+

expect(getPluginRuntimeGatewayRequestScope()).toBeUndefined();

585+

expect(observed).toEqual([

586+

{ phase: "factory", pluginId: "multi", pluginSource: "/tmp/multi.js" },

587+

{

588+

phase: "factory",

589+

pluginId: "optional-demo",

590+

pluginSource: "/tmp/optional-demo.js",

591+

},

592+

{ phase: "prepare", pluginId: "multi", pluginSource: "/tmp/multi.js" },

593+

{ phase: "execute", pluginId: "multi", pluginSource: "/tmp/multi.js" },

594+

{

595+

phase: "prepare",

596+

pluginId: "optional-demo",

597+

pluginSource: "/tmp/optional-demo.js",

598+

},

599+

{

600+

phase: "execute",

601+

pluginId: "optional-demo",

602+

pluginSource: "/tmp/optional-demo.js",

603+

},

604+

]);

605+

});

606+607+

it("wraps every array tool callback and restores caller scope after errors", async () => {

608+

const context = createContext();

609+

const observed: Array<{ name: string; pluginId?: string; pluginSource?: string }> = [];

610+

setRegistry([

611+

{

612+

pluginId: "multi",

613+

optional: false,

614+

source: "/tmp/multi.js",

615+

names: ["array_first", "array_second"],

616+

factory: () =>

617+

["array_first", "array_second"].map((name) => ({

618+

name,

619+

description: `${name} tool`,

620+

parameters: { type: "object", properties: {} },

621+

prepareArguments() {

622+

const scope = getPluginRuntimeGatewayRequestScope();

623+

observed.push({ name: `${name}:prepare`, pluginId: scope?.pluginId });

624+

if (name === "array_second") {

625+

throw new Error("bad args");

626+

}

627+

return {};

628+

},

629+

async execute() {

630+

const scope = getPluginRuntimeGatewayRequestScope();

631+

observed.push({

632+

name,

633+

pluginId: scope?.pluginId,

634+

pluginSource: scope?.pluginSource,

635+

});

636+

return { content: [{ type: "text", text: name }] };

637+

},

638+

})),

639+

},

640+

]);

641+642+

await withPluginRuntimeGatewayRequestScope(

643+

{

644+

pluginId: "outer",

645+

pluginSource: "/tmp/outer.js",

646+

isWebchatConnect: () => false,

647+

},

648+

async () => {

649+

const tools = resolvePluginTools(createResolveToolsParams({ context }));

650+

await tools[0]?.execute("call-first", tools[0].prepareArguments?.({}) ?? {}, undefined);

651+

expect(() => tools[1]?.prepareArguments?.({})).toThrow("bad args");

652+

expect(getPluginRuntimeGatewayRequestScope()).toMatchObject({

653+

pluginId: "outer",

654+

pluginSource: "/tmp/outer.js",

655+

});

656+

},

657+

);

658+659+

expect(observed).toEqual([

660+

{ name: "array_first:prepare", pluginId: "multi" },

661+

{ name: "array_first", pluginId: "multi", pluginSource: "/tmp/multi.js" },

662+

{ name: "array_second:prepare", pluginId: "multi" },

663+

]);

664+

});

665+513666

it("does not load plugin-owned tools whose manifest metadata has no available signal", () => {

514667

const config = createContext().config;

515668

installToolManifestSnapshot({