Skip to content

fix(pi-coding-agent): install missing sdk with npm#939

Merged
christso merged 1 commit intomainfrom
fix/922-pi-sdk-npm-install
Apr 5, 2026
Merged

fix(pi-coding-agent): install missing sdk with npm#939
christso merged 1 commit intomainfrom
fix/922-pi-sdk-npm-install

Conversation

@christso
Copy link
Copy Markdown
Collaborator

@christso christso commented Apr 5, 2026

Summary

  • switch pi-coding-agent auto-install from bun to npm
  • check project-local, managed install root, and global npm installs before prompting
  • install missing Pi SDK into an isolated managed directory outside the repo workspace so npm works on Windows and in Bun worktrees

Verification

  • bun test packages/core/test/evaluation/providers/pi-coding-agent.test.ts
  • manual Windows E2E on source CLI with pi-sdk-azure against Azure
  • RED: missing-dependency prompt path failed when npm installed inside the workspace due to workspace:*
  • GREEN: same prompt path accepted y, installed @mariozechner/pi-coding-agent via npm into C:\Users\Christopher.Tso\AppData\Local\agentv\deps\pi-sdk, and the smoke eval passed with score 1.000

Closes #922

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 5, 2026

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: 920eade
Status: ✅  Deploy successful!
Preview URL: https://7e68358e.agentv.pages.dev
Branch Preview URL: https://fix-922-pi-sdk-npm-install.agentv.pages.dev

View logs

@christso christso force-pushed the fix/922-pi-sdk-npm-install branch from 1c7bec3 to 920eade Compare April 5, 2026 05:07
@christso christso marked this pull request as ready for review April 5, 2026 05:10
@christso christso merged commit 965ffdc into main Apr 5, 2026
4 checks passed
@christso christso deleted the fix/922-pi-sdk-npm-install branch April 5, 2026 05:10
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.

pi-cli azure: empty assistant content in agent_end JSONL

1 participant