From 137bad2c8b4143a6817c19a073af32f06aef8ad4 Mon Sep 17 00:00:00 2001 From: can1357 Date: Mon, 27 Jul 2026 16:17:04 +0200 Subject: [PATCH] style: applied biome formatting to review follow-up changes --- packages/coding-agent/src/mcp/manager.ts | 5 ++++- packages/coding-agent/src/session/model-controls.ts | 6 +++++- packages/coding-agent/src/session/turn-recovery.ts | 7 ++++++- packages/coding-agent/src/thinking.ts | 7 +------ packages/coding-agent/test/autolearn-learn-local.test.ts | 3 +-- packages/coding-agent/test/mcp-reconnect.test.ts | 2 +- 6 files changed, 18 insertions(+), 12 deletions(-) diff --git a/packages/coding-agent/src/mcp/manager.ts b/packages/coding-agent/src/mcp/manager.ts index 3c0b2a7f5..8316bbb00 100644 --- a/packages/coding-agent/src/mcp/manager.ts +++ b/packages/coding-agent/src/mcp/manager.ts @@ -1108,7 +1108,10 @@ export class MCPManager { listResourceTemplates(connection), ]); if (templatesResult.status === "rejected") { - logger.debug("Failed to list MCP resource templates", { path: `mcp:${name}`, error: templatesResult.reason }); + logger.debug("Failed to list MCP resource templates", { + path: `mcp:${name}`, + error: templatesResult.reason, + }); } if (resourcesResult.status === "rejected") throw resourcesResult.reason; const resources = resourcesResult.value; diff --git a/packages/coding-agent/src/session/model-controls.ts b/packages/coding-agent/src/session/model-controls.ts index dd79f23de..e0e47c4a3 100644 --- a/packages/coding-agent/src/session/model-controls.ts +++ b/packages/coding-agent/src/session/model-controls.ts @@ -148,7 +148,11 @@ export class ModelControls { this.#autoResolvedLevel = undefined; this.#thinkingLevel = level === AUTO_THINKING - ? clampThinkingLevelToCeiling(this.#model, resolveProvisionalAutoLevel(this.#model), this.#thinkingLevelCeiling) + ? clampThinkingLevelToCeiling( + this.#model, + resolveProvisionalAutoLevel(this.#model), + this.#thinkingLevelCeiling, + ) : resolveThinkingLevelForModel( this.#model, clampThinkingLevelToCeiling(this.#model, level, this.#thinkingLevelCeiling), diff --git a/packages/coding-agent/src/session/turn-recovery.ts b/packages/coding-agent/src/session/turn-recovery.ts index 83e0ab22c..15efa4a94 100644 --- a/packages/coding-agent/src/session/turn-recovery.ts +++ b/packages/coding-agent/src/session/turn-recovery.ts @@ -28,7 +28,12 @@ import type { RecoveredRetryError } from "../extensibility/shared-events"; import emptyStopRetryTemplate from "../prompts/system/empty-stop-retry.md" with { type: "text" }; import thinkingLoopRedirectTemplate from "../prompts/system/thinking-loop-redirect.md" with { type: "text" }; import unexpectedStopRetryTemplate from "../prompts/system/unexpected-stop-retry.md" with { type: "text" }; -import { AUTO_THINKING, clampThinkingLevelToCeiling, type ConfiguredThinkingLevel, modelSupportsEffortCeiling } from "../thinking"; +import { + AUTO_THINKING, + type ConfiguredThinkingLevel, + clampThinkingLevelToCeiling, + modelSupportsEffortCeiling, +} from "../thinking"; import type { AgentSessionEvent } from "./agent-session-events"; import type { InitialRetryFallbackState } from "./agent-session-types"; import { isEmptyErrorTurn } from "./messages"; diff --git a/packages/coding-agent/src/thinking.ts b/packages/coding-agent/src/thinking.ts index c95c3a4e4..4138779fc 100644 --- a/packages/coding-agent/src/thinking.ts +++ b/packages/coding-agent/src/thinking.ts @@ -327,12 +327,7 @@ export function clampThinkingLevelToCeiling( level: ThinkingLevel | undefined, ceiling: Effort | undefined, ): ThinkingLevel | undefined { - if ( - ceiling === undefined || - level === undefined || - level === ThinkingLevel.Off || - level === ThinkingLevel.Inherit - ) { + if (ceiling === undefined || level === undefined || level === ThinkingLevel.Off || level === ThinkingLevel.Inherit) { return level; } const maxIndex = THINKING_EFFORTS.indexOf(ceiling); diff --git a/packages/coding-agent/test/autolearn-learn-local.test.ts b/packages/coding-agent/test/autolearn-learn-local.test.ts index 6e88728d8..10c4e06ba 100644 --- a/packages/coding-agent/test/autolearn-learn-local.test.ts +++ b/packages/coding-agent/test/autolearn-learn-local.test.ts @@ -270,8 +270,7 @@ describe("learned-lesson read-back", () => { // Save a new lesson — must not destroy the header or prose, and the // new lesson lands newest-first at the head of the bullet run. await saveLearnedLesson(agentDir, settings.getCwd(), { content: "new lesson" }); - const expected = - "# Project Lessons\n\nRemember these conventions:\n\n- new lesson\n- existing 1\n- existing 2\n"; + const expected = "# Project Lessons\n\nRemember these conventions:\n\n- new lesson\n- existing 1\n- existing 2\n"; expect(await Bun.file(path.join(root, "learned.md")).text()).toBe(expected); // Saving the same lesson again is byte-idempotent: no duplicate entry // and no blank-line growth from the trailing-newline split artifact. diff --git a/packages/coding-agent/test/mcp-reconnect.test.ts b/packages/coding-agent/test/mcp-reconnect.test.ts index 60b89062d..4576ffffc 100644 --- a/packages/coding-agent/test/mcp-reconnect.test.ts +++ b/packages/coding-agent/test/mcp-reconnect.test.ts @@ -1,5 +1,4 @@ import { describe, expect, it, vi } from "bun:test"; -import { logger } from "@oh-my-pi/pi-utils"; import type { MCPReconnect } from "@oh-my-pi/pi-coding-agent/mcp/tool-bridge"; import { DeferredMCPTool, @@ -9,6 +8,7 @@ import { } from "@oh-my-pi/pi-coding-agent/mcp/tool-bridge"; import type { MCPServerConnection, MCPToolCallResult, MCPTransport } from "@oh-my-pi/pi-coding-agent/mcp/types"; import { ToolAbortError } from "@oh-my-pi/pi-coding-agent/tools/tool-errors"; +import { logger } from "@oh-my-pi/pi-utils"; // --------------------------------------------------------------------------- // Helpers