Jannik/PaperClipAI
1
0 Fork 0
mirror of https://github.com/paperclipai/paperclip.git synced 2026-10-23 09:07:46 +02:00
Code Issues Packages Projects Releases Wiki Activity

Bind native Pi launches to their provider asset package

Browse Source 
Co-Authored-By: Paperclip <noreply@paperclip.ing>
This commit is contained in:
DottaandPaperclip committed 2026-10-21 16:17:54 -05:00
1 parent 4a8c6c703a
commit 6e7076cc0a
2 files changed
+13 -9
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

packages/paperclip-runner/src/live/runnerd-codex-transport.test.ts
+10 -6
View File
@@ -1892,13 +1892,17 @@ it.each([false, true])("binds a vendored source server to its pnpm dependency ro
expect(runnerdLaunchProfileInternals.acpxProviderPackageAuthority(
join(server, "dist/vendor/paperclip-runner/cli/acpx-runtime-sidecar.cjs"),
)).toEqual({ root: selfRooted ? server : workspace, manifest: join(server, "package.json") });
expect(runnerdLaunchProfileInternals.acpxProviderPackageAuthority(
join(server, "dist/vendor/paperclip-runner/cli/acpx-runtime-sidecar.cjs"), workspace, "pi",
)).toEqual({ root: server, manifest: join(server, "package.json") });
} finally {
await rm(workspace, { recursive: true, force: true });
}
});
it.each(["source", "scoped-npm", "deployment"])(
"admits the controller-selected Copilot package authority in a %s layout",
async (layout) => {
it.each((["copilot", "pi"] as const).flatMap(agent =>
["source", "scoped-npm", "deployment"].map(layout => ({ agent, layout }))))(
"admits the controller-selected $agent package authority in a $layout layout",
async ({ agent, layout }) => {
const directory = await realpath(await mkdtemp(join(tmpdir(), "copilot-package-authority-")));
const runnerRoot = layout === "source"
? join(directory, "packages/paperclip-runner")
@@ -1909,13 +1913,13 @@ it.each(["source", "scoped-npm", "deployment"])(
await writeFile(join(runnerRoot, "package.json"), JSON.stringify({ name: "@paperclipai/paperclip-runner" }));
try {
const authority = runnerdLaunchProfileInternals.acpxProviderPackageAuthority(
join(runnerRoot, "dist/cli/acpx-runtime-sidecar.cjs"), runnerRoot, "copilot",
join(runnerRoot, "dist/cli/acpx-runtime-sidecar.cjs"), runnerRoot, agent,
);
expect(authority).toEqual({ root: runnerRoot, manifest: join(runnerRoot, "package.json") });
vi.stubEnv("PAPERCLIP_ACPX_PROVIDER_PACKAGE_ROOT", authority.root);
vi.stubEnv("PAPERCLIP_ACPX_PROVIDER_PACKAGE_MANIFEST", authority.manifest);
expect(resolveRunnerProviderAssetsRoot("file:///proc/self/fd/18", "copilot"))
.toBe(join(runnerRoot, "provider-assets/copilot"));
expect(resolveRunnerProviderAssetsRoot("file:///proc/self/fd/18", agent))
.toBe(join(runnerRoot, `provider-assets/${agent}`));
} finally {
vi.unstubAllEnvs();
await rm(directory, { recursive: true, force: true });
packages/paperclip-runner/src/live/runnerd-codex-transport.ts
+3 -3
View File
@@ -3023,7 +3023,7 @@ function acpxProviderPackageAuthority(
const workspaceRoot = dirname(serverRoot);
// Source/Docker builds share the workspace's pnpm store. Published npm
// bundles and self-rooted pnpm deployments retain their server authority.
if (basename(serverRoot) === "server" &&
if (agent !== "pi" && basename(serverRoot) === "server" &&
!existsSync(resolve(serverRoot, "node_modules/.pnpm")) &&
existsSync(resolve(workspaceRoot, "node_modules/.pnpm")) &&
existsSync(resolve(workspaceRoot, "pnpm-workspace.yaml")) &&
@@ -3046,10 +3046,10 @@ function acpxProviderPackageAuthority(
);
}
const sidecarPackageRoot = resolve(cliDirectory, "../..");
// Copilot's native closure belongs to the runner package itself. Its asset
// Native Pi and Copilot closures belong to the runner package itself. Their asset
// guard requires the bound manifest directly inside that same package root;
// the workspace dependency root used by JS providers is a different owner.
if (agent === "copilot") {
if (agent === "copilot" || agent === "pi") {
return {
root: sidecarPackageRoot,
manifest: resolve(sidecarPackageRoot, "package.json"),