Skip to content

Fix agent default model not picking up after authentication resolve - #54397

Merged
bennetbo merged 3 commits into
mainfrom
fix-agent-default-model-not-picking-up
Apr 21, 2026
Merged

Fix agent default model not picking up after authentication resolve#54397
bennetbo merged 3 commits into
mainfrom
fix-agent-default-model-not-picking-up

Conversation

@smitbarmase

@smitbarmase smitbarmase commented Apr 21, 2026

Copy link
Copy Markdown
Member

Regression in #54125

Release Notes:

  • agent: Fixed an issue where the default Zed model would not get selected after sign-in completed

Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Apr 21, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Apr 21, 2026
);
});

cx.subscribe(

@bennetbo bennetbo Apr 21, 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.

We removed this because it causes flickering issues, e.g. if multiple providers authenticate one after another. Instead we only want to update the fallback once after all authenticate tasks completed. And because the Zed provider now actually returns a task we can rely on only calling update_environment_fallback_model once all have authenticated

Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
@bennetbo bennetbo changed the title Fix agent default model not picking up after authetication resolve Fix agent default model not picking up after authentication resolve Apr 21, 2026
@bennetbo

Copy link
Copy Markdown
Member

/cherry-pick preview

@bennetbo
bennetbo enabled auto-merge (squash) April 21, 2026 10:44
@bennetbo
bennetbo merged commit 51fc26d into main Apr 21, 2026
33 checks passed
@bennetbo
bennetbo deleted the fix-agent-default-model-not-picking-up branch April 21, 2026 10:49
zed-zippy Bot added a commit that referenced this pull request Apr 21, 2026
…54397) (cherry-pick to preview) (#54399)

Cherry-pick of #54397 to preview

----
Regression in #54125

Release Notes:

- agent: Fixed an issue where the default Zed model would not get
selected after sign-in completed

---------

Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>

Co-authored-by: Smit Barmase <heysmitbarmase@gmail.com>
Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
pull Bot pushed a commit to y1024/zed that referenced this pull request Apr 27, 2026
…-industries#54826)

This PR fixes an issue introduced in
zed-industries#54397 where the Zed Cloud
provider would not be reflected as "authenticated" if a connection to
Collab was attempted, but could not be established.

This was especially noticable when running Zed against a local version
of Cloud and not having Collab running.

This restores the original logic prior to that change.

Release Notes:

- N/A
ebaah46 pushed a commit to ebaah46/zed that referenced this pull request May 6, 2026
…-industries#54826)

This PR fixes an issue introduced in
zed-industries#54397 where the Zed Cloud
provider would not be reflected as "authenticated" if a connection to
Collab was attempted, but could not be established.

This was especially noticable when running Zed against a local version
of Cloud and not having Collab running.

This restores the original logic prior to that change.

Release Notes:

- N/A
kathbigra pushed a commit to kathbigra/zed that referenced this pull request May 10, 2026
…ed-industries#54397)

Regression in zed-industries#54125

Release Notes:

- agent: Fixed an issue where the default Zed model would not get
selected after sign-in completed

---------

Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
kathbigra pushed a commit to kathbigra/zed that referenced this pull request May 10, 2026
…-industries#54826)

This PR fixes an issue introduced in
zed-industries#54397 where the Zed Cloud
provider would not be reflected as "authenticated" if a connection to
Collab was attempted, but could not be established.

This was especially noticable when running Zed against a local version
of Cloud and not having Collab running.

This restores the original logic prior to that change.

Release Notes:

- N/A
jasonsmithio pushed a commit to paddleboarddev/paddleboard that referenced this pull request May 31, 2026
…826)

This PR fixes an issue introduced in
zed-industries/zed#54397 where the Zed Cloud
provider would not be reflected as "authenticated" if a connection to
Collab was attempted, but could not be established.

This was especially noticable when running Zed against a local version
of Cloud and not having Collab running.

This restores the original logic prior to that change.

Release Notes:

- N/A
Zenor27 pushed a commit to Zenor27/zed that referenced this pull request Jul 4, 2026
…ed-industries#54397)

Regression in zed-industries#54125

Release Notes:

- agent: Fixed an issue where the default Zed model would not get
selected after sign-in completed

---------

Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
…ed-industries#54397)

Regression in zed-industries#54125

Release Notes:

- agent: Fixed an issue where the default Zed model would not get
selected after sign-in completed

---------

Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
…-industries#54826)

This PR fixes an issue introduced in
zed-industries#54397 where the Zed Cloud
provider would not be reflected as "authenticated" if a connection to
Collab was attempted, but could not be established.

This was especially noticable when running Zed against a local version
of Cloud and not having Collab running.

This restores the original logic prior to that change.

Release Notes:

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

Regression in zed-industries#54125

Release Notes:

- agent: Fixed an issue where the default Zed model would not get
selected after sign-in completed

---------

Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
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