Skip to content

chore(pi): upgrade agent to 1.0.0 - #8585

Open
BillLeoutsakosvl346 wants to merge 2 commits into
stagingfrom
chore/pi-1-upgrade
Open

BillLeoutsakosvl346 wants to merge 2 commits into
stagingfrom
chore/pi-1-upgrade

Conversation

@BillLeoutsakosvl346

@BillLeoutsakosvl346 BillLeoutsakosvl346 commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Upgrade the embedded Pi SDK and sandbox recipes from 0.80.10 to exact 1.0.0 while preserving the existing Local Dev/Review Code SDK and Plan/Create PR/Update PR/Babysit CLI architecture.

  • Adapt the sealed resource loader to the new prompt-source interface and explicitly disable cache warming in embedded sessions and private CLI settings.
  • Regenerate the installed chat-model catalog and keep Sim's provider/model filtering. Saved workflows using models removed by Pi fail before credential resolution or remote setup and ask users to choose a supported Pi model; there are no substitutions or legacy definitions.
  • Check the sandbox CLI version before every agent invocation. Existing Pi images must be rebuilt or updated to 1.0.0 before these cloud modes can run. Node requirements and sandbox sizing stay unchanged.
  • Keep the test diff focused on removed-model handling, stale sandbox runtime rejection, and private CLI settings. Reuse existing backend, credential, event, and search coverage. No new public block fields, operations, outputs, or migrations.

Live Pi testing and cloud image rollout deferred.

Type of Change

  • Breaking change
  • Other: dependency upgrade

Testing

  • Focused Pi, search, provider/model-option, and sandbox tests pass; regression cases were demonstrated failing before fixes or with their corresponding guards removed.
  • Root bun run test passes: 19 workspace tasks, including 34,648 passing app tests (25 existing skips).
  • Root workspace type checking, lint, all 58 audits, block-registry validation against origin/staging, docs-manifest checks, and whitespace checks pass.
  • Pi catalog generation is byte-identical on repeat. Required in-repo artifact generation completed, and unrelated generated sandbox-bundle changes were excluded.
  • Offline checks against the real installed SDK/CLI passed using a loopback provider; validation tooling and reports remain outside Git.
  • Production Next standalone build passes. A copied artifact outside the checkout executes a real offline SDK tool cycle with the packaged 1.0.0 runtime and assets.
  • Trigger.dev's deploy bundler completes without deployment or cloud API calls; its worker package manifest pins both Pi dependencies to 1.0.0. Existing externalization is retained.

Remaining acceptance/rollout: build versioned E2B and Daytona images, verify CLI 1.0.0, exercise both providers and every Pi mode with disposable resources, check real model access/cancellation/usage/cost/search/Babysit continuation, and coordinate app/worker image configuration with rollback references before deployment.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@BillLeoutsakosvl346
BillLeoutsakosvl346 requested a review from a team as a code owner October 2, 2026 20:37
@vercel

vercel Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Oct 2, 2026 8:55pm UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 19 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[High risk] Upgrades Pi agent SDK to version 1.0.0 with API changes.

The PR appears mergeable, though restoring an in-repository real-runtime regression check would make the Pi 1.0 upgrade safer to maintain.

Findings

  1. P2 Real Pi coverage removed ▶

Summary

The PR upgrades Pi to 1.0.0, updates its model catalog and sandbox version gate, and configures private, cache-warming-off sessions.

  • Removed models are rejected with guidance rather than substituted.
  • The final test changes narrow coverage to sandbox boundary checks and remove the real installed-runtime wire tests.

Reviews (2) · Last reviewed commit: "chore(pi): narrow upgrade regression cov..."

Comment thread apps/sim/executor/handlers/pi/cloud/review/backend.test.ts Outdated
@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 16 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

Comment thread apps/sim/executor/handlers/pi/cloud/cli-runtime.test.ts

This branch was previously deployed

1 inactive deployment
Preview — d553acd3 Deployed Oct 2, 2026 by vercel[bot]
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