refactor(pi-coding-agent): migrate from CLI subprocess to SDK

.agentv/targets.yaml

@@ -4,7 +4,6 @@
 targets:
   - name: default
     provider: pi-coding-agent
-    executable: ${{ PI_CLI_PATH }}
     subprovider: openrouter
     model: z-ai/glm-4.7
     api_key: ${{ OPENROUTER_API_KEY }}
@@ -37,4 +36,4 @@ targets:
   - name: openrouter
     provider: openrouter
     api_key: ${{ OPENROUTER_API_KEY }}
-    model: ${{ OPENROUTER_MODEL }}
+    model: ${{ OPENROUTER_MODEL }}

apps/cli/package.json

@@ -33,8 +33,6 @@
     "@github/copilot-sdk": "^0.1.25",
     "@hono/node-server": "^1.19.11",
     "@inquirer/prompts": "^8.2.1",
-    "@mariozechner/pi-agent-core": "^0.54.2",
-    "@mariozechner/pi-ai": "^0.54.2",
     "@openai/codex-sdk": "^0.104.0",
     "cmd-ts": "^0.14.3",
     "dotenv": "^16.4.5",
@@ -45,6 +43,12 @@
     "semver": "^7.7.4",
     "yaml": "^2.6.1"
   },
+  "peerDependencies": {
+    "@mariozechner/pi-coding-agent": "^0.62.0"
+  },
+  "peerDependenciesMeta": {
+    "@mariozechner/pi-coding-agent": { "optional": true }
+  },
   "devDependencies": {
     "@agentv/core": "workspace:*",
     "@types/semver": "^7.7.1",

examples/features/.agentv/targets.yaml

@@ -48,10 +48,9 @@ targets:
     api_key: ${{ GOOGLE_GENERATIVE_AI_API_KEY }}
     model: ${{ GEMINI_MODEL_NAME }}
 
-  # Pi Coding Agent - autonomous coding CLI from pi-mono
+  # Pi Coding Agent - autonomous coding agent from pi-mono
   - name: pi
     provider: pi-coding-agent
-    executable: ${{ PI_CLI_PATH }}  # Optional: defaults to `pi` if omitted
     subprovider: openrouter
     model: openai/gpt-5.4
     api_key: ${{ OPENROUTER_API_KEY }}

package.json

@@ -43,8 +43,6 @@
     "yaml": "^2.6.1"
   },
   "dependencies": {
-    "@mariozechner/pi-agent-core": "^0.54.2",
-    "@mariozechner/pi-ai": "^0.54.2",
     "@openrouter/ai-sdk-provider": "^2.3.3"
   }
 }

packages/core/package.json

@@ -48,8 +48,6 @@
     "@ai-sdk/openai": "^3.0.0",
     "@anthropic-ai/claude-agent-sdk": "^0.2.49",
     "@github/copilot-sdk": "^0.1.25",
-    "@mariozechner/pi-agent-core": "^0.54.2",
-    "@mariozechner/pi-ai": "^0.54.2",
     "@openai/codex-sdk": "^0.104.0",
     "@openrouter/ai-sdk-provider": "^2.3.1",
     "ai": "^6.0.0",
@@ -67,6 +65,12 @@
     "@opentelemetry/sdk-trace-node": "^2.5.1",
     "@opentelemetry/semantic-conventions": "^1.39.0"
   },
+  "peerDependencies": {
+    "@mariozechner/pi-coding-agent": "^0.62.0"
+  },
+  "peerDependenciesMeta": {
+    "@mariozechner/pi-coding-agent": { "optional": true }
+  },
   "devDependencies": {
     "@types/micromatch": "^4.0.10",
     "zod-to-json-schema": "^3.25.1"

packages/core/src/evaluation/evaluators/skill-trigger.ts

@@ -107,9 +107,7 @@ const PROVIDER_TOOL_SEMANTICS: Partial<Record<ProviderKind, ToolMatcher>> = {
   'claude-sdk': CLAUDE_MATCHER,
   codex: CODEX_MATCHER,
   'pi-coding-agent': PI_CODING_AGENT_MATCHER,
-  // pi-agent-sdk has no tools, so skill detection is a no-op. Kept for completeness.
-  // TODO: consider removing pi-agent-sdk provider entirely.
-  'pi-agent-sdk': PI_CODING_AGENT_MATCHER,
+  'pi-cli': PI_CODING_AGENT_MATCHER,
   'copilot-cli': COPILOT_MATCHER,
   'copilot-sdk': COPILOT_MATCHER,
   vscode: COPILOT_MATCHER,

packages/core/src/evaluation/providers/index.ts

@@ -14,7 +14,7 @@ import { CodexProvider } from './codex.js';
 import { CopilotCliProvider } from './copilot-cli.js';
 import { CopilotSdkProvider } from './copilot-sdk.js';
 import { MockProvider } from './mock.js';
-import { PiAgentSdkProvider } from './pi-agent-sdk.js';
+import { PiCliProvider } from './pi-cli.js';
 import { PiCodingAgentProvider } from './pi-coding-agent.js';
 import { ProviderRegistry } from './provider-registry.js';
 import type { ResolvedTarget } from './targets.js';
@@ -48,7 +48,7 @@ export type {
   MockResolvedConfig,
   OpenAIResolvedConfig,
   OpenRouterResolvedConfig,
-  PiAgentSdkResolvedConfig,
+  PiCliResolvedConfig,
   PiCodingAgentResolvedConfig,
   ResolvedTarget,
   VSCodeResolvedConfig,
@@ -100,8 +100,7 @@ export function createBuiltinProviderRegistry(): ProviderRegistry {
     .register('copilot-sdk', (t) => new CopilotSdkProvider(t.name, t.config as never))
     .register('copilot-cli', (t) => new CopilotCliProvider(t.name, t.config as never))
     .register('pi-coding-agent', (t) => new PiCodingAgentProvider(t.name, t.config as never))
-    // TODO: consider removing pi-agent-sdk — it has no tools and is superseded by pi-coding-agent
-    .register('pi-agent-sdk', (t) => new PiAgentSdkProvider(t.name, t.config as never))
+    .register('pi-cli', (t) => new PiCliProvider(t.name, t.config as never))
     // claude-cli is the new default subprocess provider; claude is an alias
     .register('claude-cli', (t) => new ClaudeCliProvider(t.name, t.config as never))
     .register('claude', (t) => new ClaudeCliProvider(t.name, t.config as never))