feat(results): direct-push auto_push to base branch#1201
Merged
Conversation
Change auto_push behavior to push results directly to the configured repo's base branch instead of creating a feature branch and draft PR. This makes results immediately visible in Studio for shared/containerized deployments without requiring PR merges. The direct push handles non-fast-forward conflicts with pull --rebase retry (up to 3 attempts), which is safe because each run writes to a unique timestamped directory. Closes #1200 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Deploying agentv with
|
| Latest commit: |
6522e8f
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f916826b.agentv.pages.dev |
| Branch Preview URL: | https://feat-1200-direct-push-result.agentv.pages.dev |
Remove branch_prefix from config example and update description to reflect direct-push semantics. Add note about concurrent push safety. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
1 participant
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
Changes
auto_push: trueto push results directly to the configured repo's base branch instead of creating a feature branch + draft PR.Closes #1200
Changes
packages/core/src/evaluation/results-repo.ts: AddeddirectPushResults()— clones/fetches the results repo, stages artifacts on the base branch, commits, and pushes directly. Handles non-fast-forward with pull-rebase retry (3 attempts).apps/cli/src/commands/results/remote.ts: RewrotemaybeAutoExportRunArtifacts()to calldirectPushResults()instead of the branch+PR flow. Removed dead code (branch naming, PR body generation).packages/core/src/index.ts: ExportdirectPushResults.Design Rationale (per AGENTS.md)
auto_push: truesemantics.results.export.auto_push: trueis configured.auto_pushno longer creates PRs. This is intentional per the issue's request.Config (unchanged shape)
Concurrency
Each run writes to a unique timestamped directory, so content conflicts are impossible. Non-fast-forward (concurrent pushes) is handled by
git pull --rebase+ retry.Testing