@gtrabanco/pi-agentic-workflow 0.34.2 → 0.35.0

package/README.md

@@ -146,16 +146,27 @@ set of routes, and you can define as many as you like under `profiles`:
   The built-in `nan` profile is shown but not editable — create a profile to
   override it.
 
-## Serena (ships by default)
+## Serena (native MCP registration)
 
-Installing this package also gives you **Serena's language-server tools** —
-`serena_find_symbol`, `serena_find_referencing_symbols`,
-`serena_get_symbols_overview`, `serena_find_declaration`, diagnostics and the
-rest. Two declarations make that work, and they ship together: the exact-pinned
-`@bacnh85/pi-serena` dependency, and a second `pi.extensions` entry pointing at
-that dependency's own extension. Pi installs a package's dependencies but
-activates only the package's own `pi.extensions` entries, so the pin alone would
-install a toolset nothing ever loaded.
+Serena's language-server tools reach pi through Serena's own MCP server — the
+interface Serena documents, tests and keeps version-stable — not through a
+bundled bridge. This package ships no Serena dependency; provisioning is one
+consented step:
+
+```sh
+pi mcp add serena -- serena start-mcp-server --context claude-code
+pi mcp list   # verifies the connection
+```
+
+That writes the user-level `~/.pi/agent/mcp.json` (add `--local` for a
+project-level `.pi/mcp.json`, read only after project trust is granted). The
+tools appear under the `mcp__serena__<tool>` namespace — `find_symbol`,
+`find_referencing_symbols`, `get_symbols_overview`, `get_diagnostics` and the
+rest — and pi's per-server `toolExposure` config narrows the surface if the
+full toolset is more than a session needs. The machine half is the same local
+tool as before: `uv tool install -p 3.13 serena-agent` (or `/init-workspace`,
+which offers the install, the registration and the global trust append in one
+consented question).
 
 **You must run `/init-workspace` once per project.** This package never writes or
 repairs `.serena/project.yml` — `init-workspace` is its only writer, so there is
@@ -165,27 +176,12 @@ missing, lists no `language_servers`, or was provisioned by an older bundled
 `/init-workspace`; the latch lives in this package's state file, so a restart does
 not re-nag.
 
-Serena itself is a local tool: install the machine half with `uv tool install -p
-3.13 serena-agent` (or `/init-workspace`, which offers it). Nothing is uploaded
-and no key is configured.
-
-**Switching it off.** pi owns package resources, so the off switch is pi's, not a
-key of ours: `pi config` opens the resource picker (tab switches scope) and lets
-you disable the package's resources, and the same filter is expressible in
-settings under `packages`:
-
-```json
-{
-  "packages": [
-    { "source": "npm:@gtrabanco/pi-agentic-workflow", "extensions": ["./dist/extension/index.js"] }
-  ]
-}
-```
+Nothing is uploaded and no key is configured.
 
-That keeps the workflow's own extension and commands and drops the Serena
-extension entry, so the `serena_*` tools are never registered. There is no
-settings key inside this package for it: the entry is declared in the manifest,
-and a config key cannot gate a manifest-declared extension.
+**Switching it off.** Serena is an MCP server now, so pi's own MCP controls are
+the off switch: `pi mcp remove serena` unregisters it, `/mcp` inside a session
+disables or re-enables it, or set `"enabled": false` on the entry in
+`~/.pi/agent/mcp.json` to keep it listed but disconnected.
 
 ## Path protection (optional)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gtrabanco/pi-agentic-workflow",
-  "version": "0.34.2",
+  "version": "0.35.0",
   "description": "Pi package: canonical agentic-workflow skills, friendly slash commands, and per-command model routing.",
   "keywords": [
     "pi-package",
@@ -21,8 +21,7 @@
   "type": "module",
   "pi": {
     "extensions": [
-      "./dist/extension/index.js",
-      "./node_modules/@bacnh85/pi-serena/extensions/index.ts"
+      "./dist/extension/index.js"
     ],
     "skills": [
       "./skills"
@@ -58,7 +57,6 @@
     "@earendil-works/pi-tui": "*"
   },
   "dependencies": {
-    "@bacnh85/pi-serena": "0.10.0",
     "@gtrabanco/agentic-workflow": "0.4.0",
     "@gtrabanco/agentic-workflow-schema": "4.8.0"
   },

package/skills/init-workspace/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: init-workspace
 user-invocable: true
-version: 4.2.0
+version: 4.3.0
 argument-hint: <target-dir>
 author: "Gabriel Trabanco <1969593+gtrabanco@users.noreply.github.com>"
 license: MIT

package/skills/init-workspace/references/SERENA.md

@@ -172,6 +172,29 @@ entry, never another line.
 | Cursor | `.cursor/mcp.json` → `mcpServers.serena` | `{"command":"serena","args":["start-mcp-server","--context=ide"]}` |
 | JetBrains Copilot | the global `mcp.json` Copilot opens | `{"type":"stdio","command":"serena","args":["start-mcp-server","--context=jb-copilot-plugin"]}` |
 | Codex | `~/.codex/config.toml` → `[mcp_servers.serena]` | `command = "serena"`, `args = ["start-mcp-server","--project-from-cwd","--context=codex"]` |
+| **pi** (this agent harness) | `pi mcp add serena -- serena start-mcp-server --context claude-code` (user-level default → `~/.pi/agent/mcp.json`; project-level: add `--local` → `.pi/mcp.json`) | command `serena` |
+
+**pi-specific notes:** the native MCP registration replaces the old `@bacnh85/pi-serena` bridge (removed by issue #321). Two options:
+
+- **User-level (default)** — `pi mcp add serena -- serena start-mcp-server --context claude-code` writes `~/.pi/agent/mcp.json`; fresh clones are zero-config. Ask first, like every other global write.
+- **Project-level** — `pi mcp add --local serena -- serena start-mcp-server --context claude-code` writes `.pi/mcp.json` (read only after project trust is granted), keeping the choice in git:
+  ```json
+  {
+    "mcpServers": {
+      "serena": {
+        "command": "serena",
+        "args": ["start-mcp-server", "--context", "claude-code"]
+      }
+    }
+  }
+  ```
+  Same consent rule as the other per-workspace files (`.vscode/mcp.json`, `.cursor/mcp.json`).
+
+Verify either option with `pi mcp list` (it connects and exits 1 on a broken entry). Tools are namespaced `mcp__serena__<tool>` (e.g. `mcp__serena__find_symbol`), so generic native names cannot collide with other MCP servers. If the full tool surface is too wide for the session, pi's per-server `toolExposure` config narrows it — the same knob the bridge solved by shipping a curated 22.
+
+The `--context claude-code` value resolves issue #321's open question: pi is a terminal agent harness like Claude Code, so it uses Serena's `claude-code` context (not the REPL `agent_interface`, which stays out of scope — the classic `tools` interface remains the default).
+
+**Consent gating:** the registration is asked inside the same consented question as the tool install and trust append. A project-level `.pi/mcp.json` write additionally commits a file to the repo, so it needs the same explicit yes as any other write.
 
 ### Trust — the fresh clone (global config, consent-gated)