Repository navigation
Conversation
Materialize host-resolvable skill links before Docker sync, allow Pi's installed version to be pinned, and warn when MCP definitions live outside the synced Pi root without exposing their contents.
|
@asantos00 is attempting to deploy a commit to the madarco's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Thanks for your pull request. Before it can be merged, please read our Contributor License Agreement and sign it by posting the comment below. You only ever have to do this once. I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
|
Looks great thanks @asantos00! Need to ask, have you personally tested this code, or is this an AI generated submission? |
Remote Docker prepares a shared base rather than a host-specific snapshot, so seed selected agents' static config and shared skills after provisioning. Keep failures inside the create cleanup guard and document the per-box size setting. Bump the additive provider SDK surface to 4.4.0.
|
This is an AI-assisted submission. I have not done a separate line-by-line manual review myself, so I do not want to imply otherwise. The branch has been exercised rather than submitted from generated output alone: the affected test suites, full typecheck/lint/build, and provider SDK pack test pass; I also ran disposable local Docker and Remote Docker trials. The remote trial created a real linux/amd64 box over SSH, verified Pi 0.85.1, materialized an external skill, confirmed a fake external MCP secret was absent, enforced 1 CPU/2 GiB limits, passed Chrome/agent-browser smoke and private preview/artifact retrieval, survived stop/start, and then removed the container and volume. The PR body has the full validation list. |
Summary
~/.pi/agentremain usable in Linux boxespi.versionsetting for exact Pi version or npm dist-tag selection4.4.0Why
Pi configurations often link
~/.pi/agent/skills/*into other host directories. The Docker helper sees the source at/src, so those links lose their host path context and arrive broken. Cloud orchestration also assumed every provider baked static host config during prepare, while Remote Docker only prepared a shared base image. A remote Pi box therefore received neither the host's Pi config nor its skills.Pi and supporting tool versions also drift when installs always select
latest. Finally, MCP definitions may live in shared or Claude configuration outside~/.pi/agent, which makes a box appear to have lost its MCP setup.This keeps credential migration explicit: the doctor check points users at the mismatch, but does not copy external MCP files or inspect server names, commands, or environment values. Remote static seeding uses the existing snapshot filters, selects only the requested agents plus shared
~/.agentsskills, and runs inside the create cleanup guard.Validation
pnpm typecheck— 48 taskspnpm lint— 26 taskspnpm build— 26 taskspnpm --filter @madarco/agentbox test— 1,662 passed, 1 skippedpnpm --filter @agentbox/sandbox-core test— 564 passedpnpm --filter @agentbox/config test— 237 passedpnpm --filter @agentbox/agent-registry test— 87 passedpnpm --filter @agentbox/agent-pi test— 8 passedpnpm --filter @agentbox/sandbox-remote-docker test— 70 passed4.4.0build and isolatedpack:testgit diff --check0.85.1, external skill materialized as real files, fake external MCP secret absent from the box and doctor output, 4 GiB/2 CPU limits, custom relay portSKILL.md, zero remaining symlinks; two broken host links were excluded0.85.1, selected external skill materialized, fake MCP secret absent, Docker-enforced 1 CPU/2 GiB limits, repository branch seeded, Chrome/agent-browser smoke passed, valid PNG copied back, private loopback preview passed, stop/start persistence passed, and final container/volume cleanup passedThe monorepo-wide cloud package test command also reaches two existing
sandbox-cloudcustody tests whose expected normalized GitHub URL differs from this host's result. The focused cloud static tests and other affected suites pass, and this change does not touch custody code.