smashimo/oh-my-pi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(agent): clarify internal steering skip message

Browse Source 
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Fatih Al-Aziz
2026-08-03 12:54:08 +07:00
parent 06343fef42
commit c61ed66dd3
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/agent/src/agent-loop.ts
+2 -2
View File
@@ -2873,8 +2873,8 @@ function createSkippedToolResult(
reason = "queued user message";
blocker = "queued message";
} else if (source === "system") {
reason = "pending system advisory";
blocker = "advisory";
reason = "pending internal steering message";
blocker = "steering message";
} else if (source === "irc") {
reason = "pending peer interrupt";
blocker = "interrupt";
packages/agent/test/agent-loop.test.ts
+4 -2
View File
@@ -1532,7 +1532,7 @@ describe("agentLoop with AgentMessage", () => {
expect(sawInterruptInContext).toBe(true);
});
it("should skip remaining tool calls with system advisory wording when advisor steering is queued", async () => {
it("should skip remaining tool calls with internal steering wording when non-user steering is queued", async () => {
const toolSchema = type({ value: "string" });
const executed: string[] = [];
const tool: AgentTool<typeof toolSchema, { value: string }> = {
@@ -1610,7 +1610,9 @@ describe("agentLoop with AgentMessage", () => {
const skippedContent = toolEnds[1].result.content[0];
expect(skippedContent?.type).toBe("text");
if (skippedContent?.type !== "text") throw new Error("skipped tool result must be text");
expect(skippedContent.text).toContain("Skipped due to pending system advisory");
expect(skippedContent.text).toContain("Skipped due to pending internal steering message");
expect(skippedContent.text).toContain("After the steering message is handled on the next step");
expect(skippedContent.text).not.toContain("advisory");
expect(skippedContent.text).not.toContain("queued user message");
expect(skippedContent.text).toContain("Do not count this skipped result as completed work");
expect(skippedContent.text).toContain("retry the skipped tool if it is still needed");