test: guard agent seven-hit mock calls · openclaw/openclaw@e4498e0

Recent Commits to openclaw:main

steipete · 2026-05-12 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -437,7 +437,7 @@ describe("runEmbeddedPiAgent incomplete-turn safety", () => {`
`437`	`437`	`});`
`438`	`438`
`439`	`439`	`expect(mockedRunEmbeddedAttempt).toHaveBeenCalledTimes(2);`
`440`		`-const secondCall = mockedRunEmbeddedAttempt.mock.calls[1]?.[0] as { prompt?: string };`
	`440`	`+const secondCall = mockedRunEmbeddedAttempt.mock.calls.at(1)?.[0] as { prompt?: string };`
`441`	`441`	`expect(secondCall.prompt).toContain(REASONING_ONLY_RETRY_INSTRUCTION);`
`442`	`442`	`expectWarnMessageWith("reasoning-only assistant turn detected");`
`443`	`443`	`});`
`@@ -472,7 +472,7 @@ describe("runEmbeddedPiAgent incomplete-turn safety", () => {`
`472`	`472`	`});`
`473`	`473`
`474`	`474`	`expect(mockedRunEmbeddedAttempt).toHaveBeenCalledTimes(1);`
`475`		`-const onlyCall = mockedRunEmbeddedAttempt.mock.calls[0]?.[0] as { prompt?: string };`
	`475`	`+const onlyCall = mockedRunEmbeddedAttempt.mock.calls.at(0)?.[0] as { prompt?: string };`
`476`	`476`	`expect(onlyCall.prompt).not.toContain(REASONING_ONLY_RETRY_INSTRUCTION);`
`477`	`477`	`expect(onlyCall.prompt).not.toContain(EMPTY_RESPONSE_RETRY_INSTRUCTION);`
`478`	`478`	`expectNoWarnMessageWith("reasoning-only assistant turn detected");`
`@@ -641,7 +641,7 @@ describe("runEmbeddedPiAgent incomplete-turn safety", () => {`
`641`	`641`	`});`
`642`	`642`
`643`	`643`	`expect(mockedRunEmbeddedAttempt).toHaveBeenCalledTimes(2);`
`644`		`-const secondCall = mockedRunEmbeddedAttempt.mock.calls[1]?.[0] as { prompt?: string };`
	`644`	`+const secondCall = mockedRunEmbeddedAttempt.mock.calls.at(1)?.[0] as { prompt?: string };`
`645`	`645`	`expect(secondCall.prompt).toContain(REASONING_ONLY_RETRY_INSTRUCTION);`
`646`	`646`	`expectWarnMessageWith("reasoning-only assistant turn detected");`
`647`	`647`	`});`
`@@ -681,7 +681,7 @@ describe("runEmbeddedPiAgent incomplete-turn safety", () => {`
`681`	`681`	`});`
`682`	`682`
`683`	`683`	`expect(mockedRunEmbeddedAttempt).toHaveBeenCalledTimes(2);`
`684`		`-const secondCall = mockedRunEmbeddedAttempt.mock.calls[1]?.[0] as { prompt?: string };`
	`684`	`+const secondCall = mockedRunEmbeddedAttempt.mock.calls.at(1)?.[0] as { prompt?: string };`
`685`	`685`	`expect(secondCall.prompt).toContain(EMPTY_RESPONSE_RETRY_INSTRUCTION);`
`686`	`686`	`expectWarnMessageWith("empty response detected");`
`687`	`687`	`});`
`@@ -735,7 +735,7 @@ describe("runEmbeddedPiAgent incomplete-turn safety", () => {`
`735`	`735`	`});`
`736`	`736`
`737`	`737`	`expect(mockedRunEmbeddedAttempt).toHaveBeenCalledTimes(2);`
`738`		`-const secondCall = mockedRunEmbeddedAttempt.mock.calls[1]?.[0] as { prompt?: string };`
	`738`	`+const secondCall = mockedRunEmbeddedAttempt.mock.calls.at(1)?.[0] as { prompt?: string };`
`739`	`739`	`expect(secondCall.prompt).toContain(EMPTY_RESPONSE_RETRY_INSTRUCTION);`
`740`	`740`	`expectWarnMessageWith("empty response detected");`
`741`	`741`	`});`
`@@ -804,7 +804,7 @@ describe("runEmbeddedPiAgent incomplete-turn safety", () => {`
`804`	`804`	`});`
`805`	`805`
`806`	`806`	`expect(mockedRunEmbeddedAttempt).toHaveBeenCalledTimes(2);`
`807`		`-const secondCall = mockedRunEmbeddedAttempt.mock.calls[1]?.[0] as { prompt?: string };`
	`807`	`+const secondCall = mockedRunEmbeddedAttempt.mock.calls.at(1)?.[0] as { prompt?: string };`
`808`	`808`	`expect(secondCall.prompt).toContain(EMPTY_RESPONSE_RETRY_INSTRUCTION);`
`809`	`809`	`expectWarnMessageWith("empty response detected");`
`810`	`810`	`});`
`@@ -1923,7 +1923,7 @@ describe("runEmbeddedPiAgent incomplete-turn safety", () => {`
`1923`	`1923`	`});`
`1924`	`1924`
`1925`	`1925`	`expect(mockedRunEmbeddedAttempt).toHaveBeenCalledTimes(1);`
`1926`		`-const onlyCall = mockedRunEmbeddedAttempt.mock.calls[0]?.[0] as { prompt?: string };`
	`1926`	`+const onlyCall = mockedRunEmbeddedAttempt.mock.calls.at(0)?.[0] as { prompt?: string };`
`1927`	`1927`	`expect(onlyCall.prompt).not.toContain(REASONING_ONLY_RETRY_INSTRUCTION);`
`1928`	`1928`	`expect(onlyCall.prompt).not.toContain(EMPTY_RESPONSE_RETRY_INSTRUCTION);`
`1929`	`1929`	`expect(result.payloads).toEqual([{ text: "NO_REPLY" }]);`

Original file line number	Diff line number	Diff line change
`@@ -257,7 +257,7 @@ describe("runEmbeddedPiAgent overflow compaction trigger routing", () => {`
`257`	`257`	`});`
`258`	`258`
`259`	`259`	`expect(mockedEnsureAuthProfileStore).not.toHaveBeenCalled();`
`260`		`-const authStoreCall = mockedEnsureAuthProfileStoreWithoutExternalProfiles.mock.calls[0] as`
	`260`	`+const authStoreCall = mockedEnsureAuthProfileStoreWithoutExternalProfiles.mock.calls.at(0) as`
`261`	`261`	`\| [string \| undefined, { allowKeychainPrompt?: boolean } \| undefined]`
`262`	`262`	`\| undefined;`
`263`	`263`	`expect(typeof authStoreCall?.[0]).toBe("string");`
`@@ -297,7 +297,7 @@ describe("runEmbeddedPiAgent overflow compaction trigger routing", () => {`
`297`	`297`	`expect(typeof forwardedTools.normalize).toBe("function");`
`298`	`298`	`const forwardedTransport = expectRecordFields(forwardedPlan.transport, {});`
`299`	`299`	`expect(typeof forwardedTransport.resolveExtraParams).toBe("function");`
`300`		`-const attemptParams = mockedRunEmbeddedAttempt.mock.calls[0]?.[0] as`
	`300`	`+const attemptParams = mockedRunEmbeddedAttempt.mock.calls.at(0)?.[0] as`
`301`	`301`	`\| EmbeddedRunAttemptParams`
`302`	`302`	`\| undefined;`
`303`	`303`	`expect(attemptParams?.runtimePlan).toBe(runtimePlan);`
`@@ -395,7 +395,7 @@ describe("runEmbeddedPiAgent overflow compaction trigger routing", () => {`
`395`	`395`	`forwardedAuthProfileId: "openai-codex:work",`
`396`	`396`	`},`
`397`	`397`	`});`
`398`		`-const harnessParams = pluginRunAttempt.mock.calls[0]?.[0];`
	`398`	`+const harnessParams = pluginRunAttempt.mock.calls.at(0)?.[0];`
`399`	`399`	`expect(harnessParams?.runtimePlan).toBe(runtimePlan);`
`400`	`400`	`expect(Object.keys(harnessParams?.authProfileStore.profiles ?? {})).toEqual([`
`401`	`401`	`"openai-codex:work",`
`@@ -472,7 +472,7 @@ describe("runEmbeddedPiAgent overflow compaction trigger routing", () => {`
`472`	`472`	`forwardedAuthProfileId: "openai-codex:work",`
`473`	`473`	`},`
`474`	`474`	`});`
`475`		`-const harnessParams = pluginRunAttempt.mock.calls[0]?.[0];`
	`475`	`+const harnessParams = pluginRunAttempt.mock.calls.at(0)?.[0];`
`476`	`476`	`expect(harnessParams?.runtimePlan).toBe(runtimePlan);`
`477`	`477`	`expect(mockedMarkAuthProfileSuccess).toHaveBeenCalledTimes(1);`
`478`	`478`	`const [[successParams]] = mockedMarkAuthProfileSuccess.mock.calls as unknown as Array<`
`@@ -549,7 +549,7 @@ describe("runEmbeddedPiAgent overflow compaction trigger routing", () => {`
`549`	`549`	`forwardedAuthProfileId: "openai-codex:default",`
`550`	`550`	`},`
`551`	`551`	`});`
`552`		`-const harnessParams = pluginRunAttempt.mock.calls[0]?.[0];`
	`552`	`+const harnessParams = pluginRunAttempt.mock.calls.at(0)?.[0];`
`553`	`553`	`expect(harnessParams?.runtimePlan).toBe(runtimePlan);`
`554`	`554`	`});`
`555`	`555`
`@@ -622,7 +622,7 @@ describe("runEmbeddedPiAgent overflow compaction trigger routing", () => {`
`622`	`622`	`forwardedAuthProfileId: "openai-codex:default",`
`623`	`623`	`},`
`624`	`624`	`});`
`625`		`-const harnessParams = pluginRunAttempt.mock.calls[0]?.[0];`
	`625`	`+const harnessParams = pluginRunAttempt.mock.calls.at(0)?.[0];`
`626`	`626`	`expect(harnessParams?.runtimePlan).toBe(runtimePlan);`
`627`	`627`	`});`
`628`	`628`
`@@ -707,7 +707,7 @@ describe("runEmbeddedPiAgent overflow compaction trigger routing", () => {`
`707`	`707`	`forwardedAuthProfileId: "openai:personal",`
`708`	`708`	`},`
`709`	`709`	`});`
`710`		`-const harnessParams = pluginRunAttempt.mock.calls[0]?.[0];`
	`710`	`+const harnessParams = pluginRunAttempt.mock.calls.at(0)?.[0];`
`711`	`711`	`expect(harnessParams?.runtimePlan).toBe(runtimePlan);`
`712`	`712`	`expect(Object.keys(harnessParams?.authProfileStore.profiles ?? {})).toEqual([`
`713`	`713`	`"openai:personal",`

Original file line number	Diff line number	Diff line change
`@@ -1308,7 +1308,7 @@ describe("compaction-safeguard recent-turn preservation", () => {`
`1308`	`1308`
`1309`	`1309`	`expect(result.cancel).not.toBe(true);`
`1310`	`1310`	`expect(mockSummarizeInStages).toHaveBeenCalledTimes(1);`
`1311`		`-const droppedCall = mockSummarizeInStages.mock.calls[0]?.[0];`
	`1311`	`+const droppedCall = mockSummarizeInStages.mock.calls.at(0)?.[0];`
`1312`	`1312`	`expect(droppedCall?.customInstructions).toContain(`
`1313`	`1313`	`"Produce a compact, factual summary with these exact section headings:",`
`1314`	`1314`	`);`
`@@ -1351,7 +1351,7 @@ describe("compaction-safeguard recent-turn preservation", () => {`
`1351`	`1351`
`1352`	`1352`	`await compactionHandler(event, mockContext);`
`1353`	`1353`
`1354`		`-const call = mockSummarizeInStages.mock.calls[0]?.[0];`
	`1354`	`+const call = mockSummarizeInStages.mock.calls.at(0)?.[0];`
`1355`	`1355`	`expect(call?.reserveTokens).toBe(128_000);`
`1356`	`1356`	`});`
`1357`	`1357`
`@@ -1539,7 +1539,7 @@ describe("compaction-safeguard recent-turn preservation", () => {`
`1539`	`1539`
`1540`	`1540`	`expect(result.cancel).not.toBe(true);`
`1541`	`1541`	`expect(mockSummarizeInStages).toHaveBeenCalledTimes(2);`
`1542`		`-const secondCall = mockSummarizeInStages.mock.calls[1]?.[0];`
	`1542`	`+const secondCall = mockSummarizeInStages.mock.calls.at(1)?.[0];`
`1543`	`1543`	`expect(secondCall?.customInstructions).toContain("Quality check feedback");`
`1544`	`1544`	`expect(secondCall?.customInstructions).toContain("missing_section:## Decisions");`
`1545`	`1545`	`});`
`@@ -1626,7 +1626,7 @@ describe("compaction-safeguard recent-turn preservation", () => {`
`1626`	`1626`
`1627`	`1627`	`expect(result.cancel).not.toBe(true);`
`1628`	`1628`	`expect(mockSummarizeInStages).toHaveBeenCalledTimes(2);`
`1629`		`-const secondCall = mockSummarizeInStages.mock.calls[1]?.[0];`
	`1629`	`+const secondCall = mockSummarizeInStages.mock.calls.at(1)?.[0];`
`1630`	`1630`	`expect(secondCall?.customInstructions).toContain("latest_user_ask_not_reflected");`
`1631`	`1631`	`});`
`1632`	`1632`
`@@ -1817,7 +1817,7 @@ describe("compaction-safeguard recent-turn preservation", () => {`
`1817`	`1817`
`1818`	`1818`	`expect(result.cancel).not.toBe(true);`
`1819`	`1819`	`expect(mockSummarizeInStages).toHaveBeenCalledTimes(1);`
`1820`		`-const call = mockSummarizeInStages.mock.calls[0]?.[0];`
	`1820`	`+const call = mockSummarizeInStages.mock.calls.at(0)?.[0];`
`1821`	`1821`	`expect(call?.previousSummary).toBeUndefined();`
`1822`	`1822`	`expect(JSON.stringify(call?.messages[0])).toContain("<previous-compaction-summary>");`
`1823`	`1823`	`expect(JSON.stringify(call?.messages[0])).toContain("Old duplicated section");`
`@@ -1880,14 +1880,14 @@ describe("compaction-safeguard recent-turn preservation", () => {`
`1880`	`1880`	`const compaction = expectCompactionResult(result);`
`1881`	`1881`	`expect(getApiKeyAndHeadersMock).not.toHaveBeenCalled();`
`1882`	`1882`	`expect(mockSummarizeInStages).not.toHaveBeenCalled();`
`1883`		`-const providerInput = providerSummarize.mock.calls[0]?.[0];`
	`1883`	`+const providerInput = providerSummarize.mock.calls.at(0)?.[0];`
`1884`	`1884`	`expect(providerInput?.previousSummary).toBe("previous provider summary");`
`1885`	`1885`	`expect(providerInput?.customInstructions).toContain("Keep milestone names.");`
`1886`	`1886`	`expect(providerInput?.summarizationInstructions).toEqual({`
`1887`	`1887`	`identifierPolicy: "custom",`
`1888`	`1888`	`identifierInstructions: "Preserve ticket IDs exactly.",`
`1889`	`1889`	`});`
`1890`		`-const providerMessages = providerSummarize.mock.calls[0]?.[0]?.messages ?? [];`
	`1890`	`+const providerMessages = providerSummarize.mock.calls.at(0)?.[0]?.messages ?? [];`
`1891`	`1891`	`expect(JSON.stringify(providerMessages)).not.toContain("openclaw.runtime-context");`
`1892`	`1892`	`expect(JSON.stringify(providerMessages)).not.toContain("secret runtime context");`
`1893`	`1893`	`expect(compaction.summary).toContain("provider summary body");`

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ function expectFields(value: unknown, expected: Record<string, unknown>): void {`
`115`	`115`	`}`
`116`	`116`
`117`	`117`	`function firstCallArg(mock: ReturnType<typeof vi.fn>): Record<string, unknown> {`
`118`		`-const [arg] = mock.mock.calls[0] ?? [];`
	`118`	`+const [arg] = mock.mock.calls.at(0) ?? [];`
`119`	`119`	`if (!arg \|\| typeof arg !== "object") {`
`120`	`120`	`throw new Error("expected first call argument object");`
`121`	`121`	`}`
`@@ -206,8 +206,8 @@ describe("subagent registry lifecycle hardening", () => {`
`206`	`206`	`).resolves.toBeUndefined();`
`207`	`207`
`208`	`208`	`expect(warn).toHaveBeenCalledTimes(1);`
`209`		`-expect(warn.mock.calls[0]?.[0]).toBe("failed to finalize subagent background task state");`
`210`		`-expectFields(warn.mock.calls[0]?.[1], {`
	`209`	`+expect(warn.mock.calls.at(0)?.[0]).toBe("failed to finalize subagent background task state");`
	`210`	`+expectFields(warn.mock.calls.at(0)?.[1], {`
`211`	`211`	`error: { name: "Error", message: "task store boom" },`
`212`	`212`	`runId: "***",`
`213`	`213`	`childSessionKey: "agent:main:…",`
`@@ -250,10 +250,10 @@ describe("subagent registry lifecycle hardening", () => {`
`250`	`250`	`).resolves.toBeUndefined();`
`251`	`251`
`252`	`252`	`expect(warn).toHaveBeenCalledTimes(1);`
`253`		`-expect(warn.mock.calls[0]?.[0]).toBe(`
	`253`	`+expect(warn.mock.calls.at(0)?.[0]).toBe(`
`254`	`254`	`"failed to update subagent background task delivery state",`
`255`	`255`	`);`
`256`		`-expectFields(warn.mock.calls[0]?.[1], {`
	`256`	`+expectFields(warn.mock.calls.at(0)?.[1], {`
`257`	`257`	`error: { name: "Error", message: "delivery state boom" },`
`258`	`258`	`runId: "***",`
`259`	`259`	`childSessionKey: "agent:main:…",`
`@@ -677,10 +677,10 @@ describe("subagent registry lifecycle hardening", () => {`
`677`	`677`	`).resolves.toBeUndefined();`
`678`	`678`
`679`	`679`	`expect(warn).toHaveBeenCalledTimes(1);`
`680`		`-expect(warn.mock.calls[0]?.[0]).toBe(`
	`680`	`+expect(warn.mock.calls.at(0)?.[0]).toBe(`
`681`	`681`	`"failed to update subagent background task delivery state",`
`682`	`682`	`);`
`683`		`-expectFields(warn.mock.calls[0]?.[1], {`
	`683`	`+expectFields(warn.mock.calls.at(0)?.[1], {`
`684`	`684`	`error: { name: "Error", message: "delivery status boom" },`
`685`	`685`	`deliveryStatus: "delivered",`
`686`	`686`	`});`

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

推荐订阅源

Recent Commits to openclaw:main