Skip to content

agentHost: Adapt SDK download notification to available models - #334672

Merged
TylerLeonhardt merged 1 commit into
tyleonha/agent-sdk-download-offerfrom
tyleonha/agent-sdk-download-notification
Sep 5, 2026
Merged

agentHost: Adapt SDK download notification to available models#334672
TylerLeonhardt merged 1 commit into
tyleonha/agent-sdk-download-offerfrom
tyleonha/agent-sdk-download-notification

Conversation

@TylerLeonhardt

Copy link
Copy Markdown
Member

Fixes #334668.

Stacked on #334316. This PR targets tyleonha/agent-sdk-download-offer and contains only the notification follow-up. Retarget to main after the base PR merges.

Summary

  • Explain that users can click Download or send a message when models are available for the agent.
  • Update the visible notification as models appear or disappear, using the existing model-change subscription and content deduplication.
  • Preserve the required-download wording when no models are available and always keep the Download action.

Validation

  • npm run transpile-client
  • Targeted SDK setup notification and session target picker unit tests: 45 passing. The two new initial/reactive model-availability tests failed before the fix.
  • ESLint and hygiene checks for both changed files; pre-commit hygiene passed.
  • git diff --check

Explain download-on-use when models are available, react to model availability changes, and preserve the Download action and no-model wording.

Fixes #334668

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@TylerLeonhardt
TylerLeonhardt merged commit 5867b9e into tyleonha/agent-sdk-download-offer Sep 5, 2026
6 of 7 checks passed
@TylerLeonhardt
TylerLeonhardt deleted the tyleonha/agent-sdk-download-notification branch September 5, 2026 00:23
@vs-code-engineering vs-code-engineering Bot added this to the 1.137.0 milestone Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant