fix: disable schedule/event trigger UI (pending GitHub App Copilot entitlements) - #937
Closed
sabbour wants to merge 2 commits into
Closed
fix: disable schedule/event trigger UI (pending GitHub App Copilot entitlements)#937sabbour wants to merge 2 commits into
sabbour wants to merge 2 commits into
Conversation
…pp Copilot entitlements) Adds AUTOMATION_TRIGGERS_ENABLED = false constant in WorkflowsPage.tsx. Schedule and event trigger buttons are replaced with disabled variants showing a tooltip linking to github/copilot-sdk#551. Backend WorkflowScheduleTriggerService returns early in ExecuteAsync when AutomationTriggersEnabled is false, preventing stored schedules from firing without a Copilot identity. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 006365ea-32d3-4bd1-9abc-1a373fc4b208
Contributor
📝 Docs sync reminderThis PR changes code in doc-relevant paths but does not touch Changed source filesHow to update fast
This is a non-blocking reminder — it will not prevent merge. See |
… triggers Beat 3.1: 'Schedule recurring dependency sweeps' -> 'Run dependency sweep on demand' Beat 3.2: 'Trigger bug triage from GitHub' (webhook config) -> 'Manually trigger bug triage' Scheduled and event-based triggers disabled in UI pending GitHub App Copilot entitlements. Demo now shows on-demand manual runs via the 'Run now' button on the Workflows page, using the real 'Queued a run for "..."' confirmation toast. Removed the now-unused AGENTWEAVER_DEMO_GITHUB_TRIAGE_ISSUE_URL and AGENTWEAVER_DEMO_GITHUB_TRIAGE_RUN_URL preflight externalArtifacts entries since beat 3.2 no longer configures a webhook/event trigger. NOTE: Narration WAVs and narration/blueprint-narration.md for beats 3.1 and 3.2 need re-recording/rewriting to match the new on-demand narrative. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 006365ea-32d3-4bd1-9abc-1a373fc4b208
sabbour
marked this pull request as draft
August 27, 2026 18:20
Owner
Author
|
Closing as superseded by Fleet automation activation in #948. This temporary automation-disable design must not merge. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Scheduled and event-based (webhook) triggers cannot safely run without a live user's Copilot identity. GitHub Apps don't yet have Copilot entitlements, so automation triggers have no valid identity source.
Changes:
WorkflowsPage.tsx: Add schedule/Edit schedule and event trigger buttons disabled, replaced with a tooltip explaining this is a future enhancement and linking to github/copilot-sdk#551WorkflowScheduleTriggerService.cs: Backend guard (AutomationTriggersEnabled = false) makes ExecuteAsync return early, preventing the hosted background loop from ever firing stored schedulesWorkflowsPage.test.tsx: updated tests that previously drove the schedule/event dialogs via the (now disabled) buttons to instead assert the buttons render disabled with the coming-soon tooltip; badge-rendering coverage is unchangedRe-enabling: Flip AUTOMATION_TRIGGERS_ENABLED in WorkflowsPage.tsx and AutomationTriggersEnabled in WorkflowScheduleTriggerService.cs to rue once GitHub ships Copilot entitlements for GitHub Apps.
Closes: N/A (tracks github/copilot-sdk#551)
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com