fix(console): stop offering the inert native OAuth catalog (#822) - #828
1 finding
The new diff (docs split + types.ts comment) is accurate and aligned with the code. The README-to-connections.md split follows the 500-line Markdown cap. Both prior findings still stand: the connected-only tail can produce a tile with an empty name, and the empty-state message tells the operator that connected providers "still appear here" in a branch where, by construction, none can. (2 earlier finding(s) still open) Code retrieval was unavailable (model: openrouter embeddings returned 404 Not Found: {"error":{"message":"No allowed providers are available for the selected model. Providers serving openai/text-embedding-3-small: openai, azure, but your account's allowed-providers setting permits only: deepinfra, streamlake, digitalocean. To change your allowed providers, visit: https://openrouter.ai/settings/privacy.","code":404,"metadata":{"available_providers":["openai","azure"],"requested_pro), so this review saw the diff alone.
| Finding | Where | |
|---|---|---|
| Connected tail renders a blank-named tile when no local metadata exists | (pull request description) |
 Connected tail renders a blank-named tile when no local metadata exists
[RULE] Document intentionally untested edge cases in the PR description.
The connected-only tail builds a tile with an empty name when the host reports a provider the console has no metadata for:
connectedOnly.push({
slug,
name: tile?.name ?? "",
description: tile?.description ?? "",
logo: null,
categories: [],
});The test lists a connected provider the console has no metadata for exercises exactly this path with zoom-pro and asserts canDisconnect and providerId but never checks the label. A tile rendered with name: "" has no visible heading, so an operator seeing it on the grid gets a blank tile with a Disconnect button addressed to a provider they cannot identify by name. Falling back to the raw state.provider (or a title-cased slug) would at least show something.