fix(outbound): keep direct-only targets out of group sess...

Recent Commits to openclaw:main

scotthuang · 2026-06-19 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -467,6 +467,7 @@ describe("resolveDeliveryTarget", () => {`
`467`	`467`	`id: "alpha",`
`468`	`468`	`outbound: createStubOutbound("Alpha"),`
`469`	`469`	`messaging: { targetPrefixes: ["alpha"] },`
	`470`	`+capabilities: { chatTypes: ["group"] },`
`470`	`471`	`}),`
`471`	`472`	`directory: {`
`472`	`473`	`listGroups: async () => [`
`@@ -503,6 +504,7 @@ describe("resolveDeliveryTarget", () => {`
`503`	`504`	`id: "alpha",`
`504`	`505`	`outbound: createStubOutbound("Alpha"),`
`505`	`506`	`messaging: { targetPrefixes: ["alpha"] },`
	`507`	`+capabilities: { chatTypes: ["group"] },`
`506`	`508`	`}),`
`507`	`509`	`directory: {`
`508`	`510`	`listGroups: async () => {`

Original file line number	Diff line number	Diff line change
`@@ -498,6 +498,23 @@ describe("resolveOutboundSessionRoute", () => {`
`498`	`498`	`chatType: "direct",`
`499`	`499`	`},`
`500`	`500`	`},`
	`501`	`+{`
	`502`	`+name: "uses resolved direct-only channel user targets to avoid phantom group sessions",`
	`503`	`+target: "wxid_abc123@im.wechat",`
	`504`	`+channel: "openclaw-weixin",`
	`505`	`+resolvedTarget: {`
	`506`	`+to: "wxid_abc123@im.wechat",`
	`507`	`+kind: "user" as const,`
	`508`	`+source: "normalized" as const,`
	`509`	`+resolutionSource: "normalized" as const,`
	`510`	`+},`
	`511`	`+expected: {`
	`512`	`+sessionKey: "agent:main:openclaw-weixin:direct:wxid_abc123@im.wechat",`
	`513`	`+from: "openclaw-weixin:wxid_abc123@im.wechat",`
	`514`	`+to: "user:wxid_abc123@im.wechat",`
	`515`	`+chatType: "direct",`
	`516`	`+},`
	`517`	`+},`
`501`	`518`	`])("$name", async ({ channel = "guildchat", target, threadId, resolvedTarget, expected }) => {`
`502`	`519`	`const route = await resolveOutboundSessionRoute({`
`503`	`520`	`cfg: perChannelPeerSessionCfg,`

Original file line number	Diff line number	Diff line change
`@@ -142,12 +142,18 @@ describe("resolveMessagingTarget (directory fallback)", () => {`
`142`	`142`	`{ kind: "group", id: "replacement-id", name: "support" } satisfies ChannelDirectoryEntry,`
`143`	`143`	`]);`
`144`	`144`	`const firstPlugin = {`
`145`		`- ...createChannelTestPluginBase({ id: "richchat" }),`
	`145`	`+ ...createChannelTestPluginBase({`
	`146`	`+id: "richchat",`
	`147`	`+capabilities: { chatTypes: ["group"] },`
	`148`	`+}),`
`146`	`149`	`directory: { listGroups: firstListGroups },`
`147`	`150`	`messaging: { targetResolver: {} },`
`148`	`151`	`} satisfies ChannelPlugin;`
`149`	`152`	`const replacementPlugin = {`
`150`		`- ...createChannelTestPluginBase({ id: "richchat" }),`
	`153`	`+ ...createChannelTestPluginBase({`
	`154`	`+id: "richchat",`
	`155`	`+capabilities: { chatTypes: ["group"] },`
	`156`	`+}),`
`151`	`157`	`directory: { listGroups: replacementListGroups },`
`152`	`158`	`messaging: { targetResolver: {} },`
`153`	`159`	`} satisfies ChannelPlugin;`
`@@ -218,6 +224,37 @@ describe("resolveMessagingTarget (directory fallback)", () => {`
`218`	`224`	`expect(mocks.listGroupsLive).not.toHaveBeenCalled();`
`219`	`225`	`});`
`220`	`226`
	`227`	`+it("defaults bare id-like targets to user for direct-only channel plugins", async () => {`
	`228`	`+const directOnlyPlugin = {`
	`229`	`+ ...createChannelTestPluginBase({`
	`230`	`+id: "openclaw-weixin",`
	`231`	`+capabilities: { chatTypes: ["direct"] },`
	`232`	`+}),`
	`233`	`+messaging: {`
	`234`	`+targetResolver: {`
	`235`	`+looksLikeId: (raw: string) => raw.endsWith("@im.wechat"),`
	`236`	`+},`
	`237`	`+},`
	`238`	`+} satisfies ChannelPlugin;`
	`239`	`+`
	`240`	`+const result = await expectOkResolution({`
	`241`	`+ cfg,`
	`242`	`+channel: "openclaw-weixin",`
	`243`	`+input: "wxid_abc123@im.wechat",`
	`244`	`+plugin: directOnlyPlugin,`
	`245`	`+});`
	`246`	`+`
	`247`	`+expect(result.target).toEqual({`
	`248`	`+to: "wxid_abc123@im.wechat",`
	`249`	`+kind: "user",`
	`250`	`+display: "wxid_abc123@im.wechat",`
	`251`	`+source: "normalized",`
	`252`	`+resolutionSource: "normalized",`
	`253`	`+});`
	`254`	`+expect(mocks.listGroups).not.toHaveBeenCalled();`
	`255`	`+expect(mocks.listGroupsLive).not.toHaveBeenCalled();`
	`256`	`+});`
	`257`	`+`
`221`	`258`	`it("uses catalog plugin target grammar for unloaded numeric topic ids", async () => {`
`222`	`259`	`mocks.getLoadedChannelPlugin.mockReturnValue(undefined);`
`223`	`260`	`mocks.getChannelPlugin.mockReturnValue({`

Original file line number	Diff line number	Diff line change
`@@ -188,6 +188,11 @@ function detectTargetKind(`
`188`	`188`	`return "group";`
`189`	`189`	`}`
`190`	`190`
	`191`	`+const chatTypes = plugin?.capabilities?.chatTypes ?? [];`
	`192`	`+if (chatTypes.length > 0 && chatTypes.every((chatType) => chatType === "direct")) {`
	`193`	`+return "user";`
	`194`	`+}`
	`195`	`+`
`191`	`196`	`return "group";`
`192`	`197`	`}`
`193`	`198`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main