Fix CLI coverage update branch drift#117
Open
IlyaasK wants to merge 3 commits into
Open
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit a85a94f. Configure here.
|
Created a monitoring plan for this PR. What this PR does: Refactors the automated CLI coverage workflow to create a fresh, per-SDK-version branch in Intended effect:
Risks:
Status updates will be posted automatically on this PR as monitoring progresses. |
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.

What
Fixes
update-cli-coverage.ymlso it no longer reuses the stale evergreencli-coverage-updatebranch before running the Cursor CLI updater.The workflow now:
cli-coverage-v0.62.0kernel/clifresh frommainmaingo list -m, notgrepWhy
The existing workflow is currently failing before Cursor runs. It checks out the stale
cli-coverage-updatebranch, tries to mergeorigin/main, ignores merge conflicts with|| true, then reads a conflictedgo.mod.That produces two SDK versions:
GitHub then rejects the multiline output:
Starting from fresh CLI
mainper SDK version removes the conflict drift and prevents old SDK pins from being carried forward.Verification
YAML.load_filebash -nover each embeddedrunscript after scrubbing GitHub expressionsgit diff --check -- .github/workflows/update-cli-coverage.ymlgo list -m -f '{{.Version}}' github.com/kernel/kernel-go-sdkreturns a single SDK version inkernel/cliNote
Low Risk
CI workflow and agent instructions only; no runtime product code, though push semantics changed for automation branches.
Overview
The Update CLI coverage workflow no longer reuses the long-lived
cli-coverage-updatebranch. It derives a per-SDK branch (e.g.cli-coverage-v0.62.0), cloneskernel/clishallow from main, creates that branch fresh, and optionally fetches an existing remote branch of the same name—avoiding merge-conflictedgo.modand invalid multiline workflow outputs.SDK version steps run earlier and emit the branch name; shell steps use
set -euo pipefailand quoted$GITHUB_OUTPUT. The CLI’s current SDK pin is read withgo list -minstead of greppinggo.mod. The Cursor agent prompt now targets the versioned branch, forbids the old evergreen branch, and uses force-with-lease when pushing to an existing remote branch; PR creation/listing uses the per-SDK head branch instead ofcli-coverage-update.Reviewed by Cursor Bugbot for commit 5fa3793. Bugbot is set up for automated code reviews on this repo. Configure here.