[DEV-76] Auto-fix CI failures on DevIntern-created PRs - #127
Merged
Conversation
Contributor
Author
|
Updated after the fine-grained PAT review: CI detection now uses the Actions API (workflow runs, jobs, and job logs) plus legacy commit statuses, with no Checks API dependency. Also merged current main and resolved the fleet runner return-type conflict. |
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
[workspace].ci_failure_fixsetting (default:false)Replacement for #51
This ports DEV-76 onto the current
mainafter the original PR became stale and conflicted. It also fixes issues found during the review of #51:logs_url304responsesWatch lifecycle
The watcher reconciles once at worker startup. Pending, failing, changed, and not-yet-reported CI is polled at the configured workspace interval. Once a PR is terminal green and remains unchanged, that PR progressively backs off to 5, 15, and 30 minutes; an observed PR or CI change restores the configured cadence. Green PRs continue to be checked so delayed reruns and newly added workflows are detected.
It watches only open PRs in the local
agent_prsregistry and stops when the PR closes, the repository is removed, the feature is disabled, or the worker stops. Retry/escalation state survives worker restarts; a restart performs an immediate reconciliation.Configuration
The setting defaults to
falseand live-reloads.GitHub permissions
When enabled, the local token or customer-owned GitHub App needs:
Existing customer-owned App installations must be re-approved after these permissions are added.
GitHub does not currently expose its separate Checks permission for fine-grained PATs. GitHub Actions is therefore detected through the Actions API, which provides run/job conclusions and logs with
Actions: Read. CI providers that publish only Checks API check runs are outside this version's watch scope unless they also publish a legacy commit status.Validation
mainand resolved the fleet runner contract conflict@getdevintern/code: 1,393 tests passed@devintern/dashboard-ui: 100 tests passed