Skip to content

修复provider刷不出来的问题 - #13

Open
FaithZL wants to merge 1 commit into
SeeleAI:agent/dev/mvpfrom
FaithZL:agent/dev/mvp
Open

修复provider刷不出来的问题#13
FaithZL wants to merge 1 commit into
SeeleAI:agent/dev/mvpfrom
FaithZL:agent/dev/mvp

Conversation

@FaithZL

@FaithZL FaithZL commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Motivation

Changes

How Tested

  • Bug fix
  • New feature
  • Breaking change

@Royalvice Royalvice left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Blocking: the App hook must not turn every observed loading snapshot into refreshProvidersSnapshot({}). Settings mounts both ProvidersSection and nested ProviderCatalogList, each with an independent useProvidersSnapshot instance and local refresh guard. On a cold snapshot they can both force a full refresh.

The global refresh path calls clearCachedProviders(undefined), resets all snapshot scopes/in-flight loads, and starts force probes for every provider. That can discard earlier in-flight results, reset unrelated Workspace model snapshots to loading, and turn a user-requested provider-specific refresh into a global one.

Please move first-read catalog warm-up/recovery to the daemon ProviderSnapshotManager / ProviderCatalogSession, using the existing non-force, scope/provider-deduplicated warm-up path. The App should render the daemon snapshot and invoke force refresh only for an explicit user action. Please also add coverage for two same-scope consumers, an in-flight provider-specific refresh, scope isolation, and refresh failure/reconnect.

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.

2 participants