Skip to content

ci: Harden background-agent MVP git authentication loop - #49373

Merged
morgankrey merged 2 commits into
mainfrom
fix/background-agent-mvp-git-auth-refresh
Feb 17, 2026
Merged

ci: Harden background-agent MVP git authentication loop#49373
morgankrey merged 2 commits into
mainfrom
fix/background-agent-mvp-git-auth-refresh

Conversation

@morgankrey

@morgankrey morgankrey commented Feb 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • refresh git auth headers before each per-crash iteration in background_agent_mvp
  • add guarded git fetch/git checkout handling so one candidate auth failure doesn’t fail the entire run

Context

  • fixes the auth failure seen in workflow run 22106378719 (fatal: could not read Username for 'https://github.com')
  • related Linear issue: BIZOPS-853

Testing

  • ruby -e "require 'yaml'; YAML.load_file('.github/workflows/background_agent_mvp.yml')"

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Feb 17, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Feb 17, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be pulled in from another PR and unrelated to the proposed changes here.

git checkout -B "$BRANCH" origin/main
git config --local http.https://github.com/.extraheader "AUTHORIZATION: bearer ${GH_TOKEN}"

if ! git fetch origin main; then

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this fetch even needed? We use the checkout action just above which should ensure that the changes are based on the lates main already. If we are concerned that main is outdated during the n-th consecutive agent run, should we not instead throw this into a matrix job instead?

@MrSubidubi MrSubidubi changed the title ci: harden background-agent MVP git auth loop ci: Harden background-agent MVP git authentication loop Feb 17, 2026
@morgankrey
morgankrey force-pushed the fix/background-agent-mvp-git-auth-refresh branch from d4ce58f to a14e556 Compare February 17, 2026 17:43
morgankrey and others added 2 commits February 17, 2026 11:45
Reapply GitHub auth headers before each per-crash fetch and skip candidates when fetch/checkout fails so one auth drop does not fail the whole run.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Replace "affected crate(s)" with neutral wording in both the workflow and local runner prompt text.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@morgankrey
morgankrey force-pushed the fix/background-agent-mvp-git-auth-refresh branch from a14e556 to a6550f1 Compare February 17, 2026 17:45
@morgankrey
morgankrey marked this pull request as ready for review February 17, 2026 17:59
@morgankrey
morgankrey merged commit 7620c68 into main Feb 17, 2026
27 checks passed
@morgankrey
morgankrey deleted the fix/background-agent-mvp-git-auth-refresh branch February 17, 2026 17:59
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
…es#49373)

## Summary
- refresh git auth headers before each per-crash iteration in
`background_agent_mvp`
- add guarded `git fetch`/`git checkout` handling so one candidate auth
failure doesn’t fail the entire run

## Context
- fixes the auth failure seen in workflow run 22106378719 (`fatal: could
not read Username for 'https://github.com'`)
- related Linear issue: BIZOPS-853

## Testing
- ruby -e "require 'yaml';
YAML.load_file('.github/workflows/background_agent_mvp.yml')"

Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
…es#49373)

## Summary
- refresh git auth headers before each per-crash iteration in
`background_agent_mvp`
- add guarded `git fetch`/`git checkout` handling so one candidate auth
failure doesn’t fail the entire run

## Context
- fixes the auth failure seen in workflow run 22106378719 (`fatal: could
not read Username for 'https://github.com'`)
- related Linear issue: BIZOPS-853

## Testing
- ruby -e "require 'yaml';
YAML.load_file('.github/workflows/background_agent_mvp.yml')"

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants