chore(pi): organize mode implementations - #6362
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Import paths are updated everywhere that referenced the old flat filenames (e.g.
Reviewed by Cursor Bugbot for commit 6c94d74. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR reorganizes the PI executor implementation into explicit core, cloud, and local areas without changing its public handler interface.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/executor/handlers/pi/pi-handler.ts | Preserves the PI handler entry point while redirecting mode and shared-module imports to the new folder structure. |
| apps/sim/executor/handlers/pi/cloud/authoring/backend.ts | Relocates cloud authoring execution and updates its internal dependencies without behavioral changes. |
| apps/sim/executor/handlers/pi/cloud/review/backend.ts | Relocates cloud review execution and consistently updates imports for review tools, GitHub helpers, and core services. |
| apps/sim/executor/handlers/pi/cloud/babysit/backend.ts | Relocates babysit execution and updates its cloud and core dependency paths. |
| apps/sim/executor/handlers/pi/local/backend.ts | Relocates local PI execution and redirects shared and SSH-tool imports to the organized modules. |
| apps/sim/executor/handlers/pi/core/backend.ts | Relocates shared PI backend contracts while preserving their definitions and updating type dependencies. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
Registry[Executor handler registry] --> Handler[PI handler]
Handler --> Core[core: contracts, context, events, keys, SDK, redaction]
Handler --> CloudAuthoring[cloud/authoring]
Handler --> CloudReview[cloud/review]
Handler --> Local[local: backend and tools]
CloudAuthoring --> CloudShared[cloud/shared]
CloudAuthoring --> Babysit[cloud/babysit]
CloudAuthoring --> GitHub[cloud/github-pr]
CloudReview --> CloudShared
CloudReview --> GitHub
CloudAuthoring --> Core
CloudReview --> Core
Local --> Core
Handler --> Search[search]
Search --> Core
Reviews (3): Last reviewed commit: "chore(pi): organize mode implementations" | Re-trigger Greptile
6bb3442 to
c193a14
Compare
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit c193a14. Configure here.
c193a14 to
c212383
Compare
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit c212383. Configure here.
…olders # Conflicts: # apps/sim/executor/handlers/pi/cloud/authoring/backend.ts # apps/sim/executor/handlers/pi/cloud/babysit/round.ts # apps/sim/executor/handlers/pi/cloud/review/backend.ts # apps/sim/executor/handlers/pi/local/backend.test.ts # apps/sim/executor/handlers/pi/local/backend.ts
Summary
Type of Change
Testing
Checklist