Skip to content

fix(server): unavailable pool accounts no longer hide Claude models - #35

Merged
ScriptedAlchemy merged 1 commit into
mainfrom
fix/claude-pool-unavailable-catalog
Oct 31, 2026
Merged

ScriptedAlchemy merged 1 commit into
mainfrom
fix/claude-pool-unavailable-catalog

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Owner

An unavailable Claude account publishes an empty placeholder catalog. The account pool intersects that with the healthy account's catalog, so it can report ready while offering zero models. The composer hides the individual accounts behind the pool, leaving Claude's picker empty; agent capabilities also list no Claude models.

Build the pool's model, command, and skill intersection from available accounts. Keep every configured member's identity so recovery and saved selections continue to work. Option compatibility checks still apply to available accounts, and routing already excludes unavailable accounts.

Validation:

  • Extended the existing pool health test with an unavailable account; it failed before the fix because the pool returned no models.
  • 50 focused catalog, routing, and failover tests passed.
  • vp run check:changed passed for both changed files. The pre-existing untracked .metadata_never_index marker was excluded from this invocation with a temporary Git excludes file.
  • Runtime reproduction used a read-only cached Claude account and a simulated unavailable initialization placeholder: before ready / 0 models; after ready / 13 models, with routing selecting the available account. No live state was changed and no paid agent turn was started.

Model: gpt-6.1-sol. Harness: Codex in T3 Code.

@changeset-bot

changeset-bot Bot commented Oct 31, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 2cedfc6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy
ScriptedAlchemy merged commit e747cef into main Oct 31, 2026
13 of 15 checks passed
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