Skip to content

fix(pi): load MCP adapter in isolated runs - #115

Merged
drewstone merged 2 commits into
mainfrom
fix/pi-credential-isolation-20260803
Aug 4, 2026
Merged

fix(pi): load MCP adapter in isolated runs#115
drewstone merged 2 commits into
mainfrom
fix/pi-credential-isolation-20260803

Conversation

@drewstone

Copy link
Copy Markdown
Owner

What changed

  • explicitly load the installed pi-mcp-adapter when a scoped Pi request declares MCP servers
  • expose the installed extension dependency tree read-only inside the filesystem jail
  • retain local-path adapter discovery while refusing installs with no loadable path

Why

PR #114 points Pi at a fresh credential-free config directory. That also removed ambient extension discovery, so request-level MCP passed --mcp-config without loading the adapter. The filesystem jail then needed the adapter dependency tree, not only the package directory.

Proof

  • regression test before fix: missing --extension
  • real Pi 0.83.0 before fix: Unknown option: --mcp-config and hidden zod dependency
  • real Pi 0.83.0 after fix: request MCP completed through the installed adapter under read confinement
  • pnpm test: 641 passed, 7 skipped
  • pnpm typecheck: passed
  • git merge-tree --write-tree origin/main HEAD: dfc0e33adb04525d691ea83511f55ced96d3ac4c

@drewstone
drewstone merged commit b03311d into main Aug 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant