-
Notifications
You must be signed in to change notification settings - Fork 3.3k
fix(triggers): add copilot as a trigger type #3191
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile OverviewGreptile SummaryThis PR introduces a new core trigger type,
Overall, the change fits existing trigger handling patterns (core trigger enum → execute endpoint validation → LoggingSession metadata → log filtering/display). Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant CopilotClient as Copilot Client
participant ClientExec as executeWorkflowWithFullLogging()
participant ExecuteAPI as POST /api/workflows/:id/execute
participant Orchestrator as Copilot Orchestrator
participant ExecWorkflow as executeWorkflow()
participant CoreExec as executeWorkflowCore
participant Logs as LoggingSession
CopilotClient->>ClientExec: executeRunToolOnClient(...)
ClientExec->>ExecuteAPI: fetch(triggerType="copilot", useDraftState=true)
ExecuteAPI->>Logs: new LoggingSession(triggerType="copilot")
ExecuteAPI->>CoreExec: executeWorkflowCore(...)
CoreExec-->>ExecuteAPI: streaming events + result
ExecuteAPI-->>ClientExec: SSE response
ClientExec-->>CopilotClient: update tool state + reportCompletion
Orchestrator->>ExecWorkflow: executeRunWorkflow(..., workflowTriggerType="copilot")
ExecWorkflow->>Logs: new LoggingSession(triggerType="copilot")
ExecWorkflow->>CoreExec: executeWorkflowCore(...)
CoreExec-->>ExecWorkflow: ExecutionResult
ExecWorkflow-->>Orchestrator: ToolCallResult
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
7 files reviewed, 2 comments
Additional Comments (2)
These mutations set Also appears at:
|
|
@cursor review |
|
@greptile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
8 files reviewed, no comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Summary
Type of Change
Testing
Tested manually
Checklist