Skip to content

feat(clients): add Pi and OMP support - #474

Open
christso wants to merge 3 commits into
mainfrom
feat/460-pi-omp-support
Open

feat(clients): add Pi and OMP support#474
christso wants to merge 3 commits into
mainfrom
feat/460-pi-omp-support

Conversation

@christso

@christso christso commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds Pi and OMP as first-class AllAgents clients at project and user scope.

  • syncs skills and agent instructions to each client's native project/user paths
  • adds real Pi package and OMP marketplace-plugin install, update, uninstall, inspection, and status flows
  • generalizes native resource ownership/state handling across Claude, Copilot, Pi, and OMP
  • fails closed on project trust, ambiguous ownership, malformed inventories, and unsafe/symlinked destinations
  • exposes native effects through human and JSON CLI output

Stack

This is PR 1 of 7 for #460. Later layers add global Pi/OMP profile materialization, per-client profile adapters, and the published profile schemas.

Design decisions

  • Native operations use an explicit (client, scope, root, environment) context instead of ambient process state.
  • Pi supports npm, Git, and local package sources on Pi 0.85.1–0.85.x. Project installs require trust recorded outside the project boundary.
  • OMP supports scoped marketplace plugins on OMP 18.1.17+. Ordinary operations clear inherited profile selectors while preserving HOME/XDG roots.
  • Marketplace registrations are retained on uninstall; AllAgents removes only corroborated managed resources.
  • Pi MCP adapter support in this layer is inspection-only. Profile materialization arrives later in the stack.

Validation

Automated:

bun run build
bun run typecheck
bun run lint
bun test                     # 1586 passed, 6 skipped
bun run test:e2e             # 142 passed, 4 skipped
bun run docs:build

Real-runtime E2E used the built dist/index.js, Pi 0.85.1, and OMP 18.1.20 in isolated temporary HOME/XDG roots:

  1. Synced one plugin skill into project .pi/skills/ and .omp/skills/ paths.
  2. Synced the same skill into user ~/.pi/agent/skills/ and ~/.omp/agent/skills/ paths.
  3. Launched Pi's real TUI and confirmed /skills discovered dogfood-shared.
  4. Installed a local Pi package and compound-engineering@EveryInc/compound-engineering-plugin through AllAgents native mode.
  5. Confirmed the resources with real pi list and omp plugin list --json inventories.
  6. Ran targeted updates for both resources and observed updated: 1, skipped: 0, plus changed native effects.
  7. Uninstalled both resources through AllAgents and confirmed both real client inventories were empty.
  8. Confirmed Pi rejects project-controlled trust before mutation.
  9. Confirmed a symlinked client destination cannot write outside the project.
  10. Confirmed allagents status is read-only and reports managed native ownership without creating marketplace state.

Related: #460

@christso christso changed the title feat/460 pi omp support feat(clients): add Pi and OMP support Sep 14, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 14, 2026

Copy link
Copy Markdown

Deploying allagents with  Cloudflare Pages  Cloudflare Pages

Latest commit: 20a7008
Status: ✅  Deploy successful!
Preview URL: https://1f71368d.allagents.pages.dev
Branch Preview URL: https://feat-460-pi-omp-support.allagents.pages.dev

View logs

@christso
christso force-pushed the feat/460-pi-omp-support branch from 375af68 to 20a7008 Compare September 14, 2026 20:43
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