fix(triggers): add copilot as a trigger type#3191
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
|
Additional Comments (2)
These mutations set Also appears at:
|
|
@cursor review |
|
@greptile |
Summary
Type of Change
Testing
Tested manually
Checklist