fix: align agent id normalization
This commit is contained in:
@@ -162,7 +162,7 @@ describe("cron cli", () => {
|
|||||||
|
|
||||||
const updateCall = callGatewayFromCli.mock.calls.find((call) => call[0] === "cron.update");
|
const updateCall = callGatewayFromCli.mock.calls.find((call) => call[0] === "cron.update");
|
||||||
const patch = updateCall?.[2] as { patch?: { agentId?: unknown } };
|
const patch = updateCall?.[2] as { patch?: { agentId?: unknown } };
|
||||||
expect(patch?.patch?.agentId).toBe("Ops");
|
expect(patch?.patch?.agentId).toBe("ops");
|
||||||
|
|
||||||
callGatewayFromCli.mockClear();
|
callGatewayFromCli.mockClear();
|
||||||
await program.parseAsync(["cron", "edit", "job-2", "--clear-agent"], {
|
await program.parseAsync(["cron", "edit", "job-2", "--clear-agent"], {
|
||||||
|
|||||||
@@ -38,7 +38,7 @@ describe("normalizeCronJobCreate", () => {
|
|||||||
},
|
},
|
||||||
}) as unknown as Record<string, unknown>;
|
}) as unknown as Record<string, unknown>;
|
||||||
|
|
||||||
expect(normalized.agentId).toBe("Ops");
|
expect(normalized.agentId).toBe("ops");
|
||||||
|
|
||||||
const cleared = normalizeCronJobCreate({
|
const cleared = normalizeCronJobCreate({
|
||||||
name: "agent-clear",
|
name: "agent-clear",
|
||||||
|
|||||||
@@ -665,7 +665,7 @@ async function handleSendAction(ctx: ResolvedActionContext): Promise<MessageActi
|
|||||||
threadId: threadId ?? slackAutoThreadId,
|
threadId: threadId ?? slackAutoThreadId,
|
||||||
})
|
})
|
||||||
: null;
|
: null;
|
||||||
if (outboundRoute && !dryRun) {
|
if (outboundRoute && !dryRun && agentId) {
|
||||||
await ensureOutboundSessionEntry({
|
await ensureOutboundSessionEntry({
|
||||||
cfg,
|
cfg,
|
||||||
agentId,
|
agentId,
|
||||||
|
|||||||
@@ -180,7 +180,7 @@ describe("resolveAgentRoute", () => {
|
|||||||
accountId: undefined,
|
accountId: undefined,
|
||||||
peer: { kind: "dm", id: "+1000" },
|
peer: { kind: "dm", id: "+1000" },
|
||||||
});
|
});
|
||||||
expect(defaultRoute.agentId).toBe("defaultAcct");
|
expect(defaultRoute.agentId).toBe("defaultacct");
|
||||||
expect(defaultRoute.matchedBy).toBe("binding.account");
|
expect(defaultRoute.matchedBy).toBe("binding.account");
|
||||||
|
|
||||||
const otherRoute = resolveAgentRoute({
|
const otherRoute = resolveAgentRoute({
|
||||||
|
|||||||
Reference in New Issue
Block a user