From 77bc2b38d5eb174be4fca157f4b5b872d32595c0 Mon Sep 17 00:00:00 2001 From: Matt Chapman Date: Fri, 17 Apr 2026 10:50:37 -0700 Subject: [PATCH 1/3] Remove outdated comment. Removed outdated comments regarding v1 capabilities in PiProvider. --- packages/providers/src/community/pi/provider.ts | 6 ------ 1 file changed, 6 deletions(-) diff --git a/packages/providers/src/community/pi/provider.ts b/packages/providers/src/community/pi/provider.ts index e054c3f2ce..6378b03eeb 100644 --- a/packages/providers/src/community/pi/provider.ts +++ b/packages/providers/src/community/pi/provider.ts @@ -69,12 +69,6 @@ function lookupPiModel(provider: string, modelId: string): Model | undefine * coding-agent harness. Each `sendQuery()` call creates a fresh session * (no reuse) with in-memory auth/session/settings, so the server never * touches `~/.pi/` and concurrent calls don't collide. - * - * v1 capabilities are all false (see `capabilities.ts`): sessionResume, - * thinkingControl, skills, mcp, etc. map to Pi features but require - * intentional wiring before they can be declared. Under-declaring is - * honest; the dag-executor emits warnings for any nodeConfig field not - * supported. */ export class PiProvider implements IAgentProvider { async *sendQuery( From 7467fed747689f92a5d40a79c0c9a12fda71036f Mon Sep 17 00:00:00 2001 From: Matt Chapman Date: Fri, 17 Apr 2026 11:33:31 -0700 Subject: [PATCH 2/3] Updated comments in capabilities.ts Updated comments to reflect changes in roadmap and capabilities. --- packages/providers/src/community/pi/capabilities.ts | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/packages/providers/src/community/pi/capabilities.ts b/packages/providers/src/community/pi/capabilities.ts index 6a5ffbb97a..fa1c52a5da 100644 --- a/packages/providers/src/community/pi/capabilities.ts +++ b/packages/providers/src/community/pi/capabilities.ts @@ -1,13 +1,17 @@ import type { ProviderCapabilities } from '../../types'; /** - * Pi v1 capabilities — intentionally conservative. Declared flags must reflect + * Pi capabilities — intentionally conservative. Declared flags must reflect * wired-up behavior, not potential support. The dag-executor uses these to * warn users when a workflow node specifies a feature the provider ignores. * - * Roadmap (v2+): thinkingControl, skills, envInjection can be flipped once - * the corresponding nodeConfig fields are intentionally translated to Pi's - * runtime options. + * Roadmap (v3+): + * The pi maintainer has expressed some opposition to supporting structured + * output, (https://github.com/badlogic/pi-mono/issues/1086) so that is + * unlikely to be added apart from an extension. + * + * Similarly, hooks, fallbackModel, and sandbox can be implemented with + * extensions, but probably not off-the-shelf pi. */ export const PI_CAPABILITIES: ProviderCapabilities = { sessionResume: true, From 4b9b556c512c1d813aa97dc0cc5f31581cd4d299 Mon Sep 17 00:00:00 2001 From: Matt Chapman Date: Fri, 17 Apr 2026 20:28:58 -0700 Subject: [PATCH 3/3] Trim whitespace. --- packages/providers/src/community/pi/capabilities.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/providers/src/community/pi/capabilities.ts b/packages/providers/src/community/pi/capabilities.ts index fa1c52a5da..a42d936a38 100644 --- a/packages/providers/src/community/pi/capabilities.ts +++ b/packages/providers/src/community/pi/capabilities.ts @@ -5,9 +5,9 @@ import type { ProviderCapabilities } from '../../types'; * wired-up behavior, not potential support. The dag-executor uses these to * warn users when a workflow node specifies a feature the provider ignores. * - * Roadmap (v3+): + * Roadmap (v3+): * The pi maintainer has expressed some opposition to supporting structured - * output, (https://github.com/badlogic/pi-mono/issues/1086) so that is + * output, (https://github.com/badlogic/pi-mono/issues/1086) so that is * unlikely to be added apart from an extension. * * Similarly, hooks, fallbackModel, and sandbox can be implemented with