Skip to content

Show a saved Agent's model provider in the console - #137

Merged
SaladDay merged 3 commits into
mainfrom
codex/web-agent-model-provider
Sep 25, 2026
Merged

SaladDay merged 3 commits into
mainfrom
codex/web-agent-model-provider

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Shows a saved Agent's model provider in the console and guards the Agents page against the regression that #136 fixed. Real acceptance found that one Agent saved with x_agents_core.model_provider made the whole project's Agents list fail to load ("Could not load ").

Changes

  • Agent detail: a Model provider section, shown when model_provider is present. It lists the protocol, the base URL, and whether an API key is configured, plus the context window and max output tokens when present. The key is never read or shown.
  • No saved harness: an Agent with a provider but no harness keeps showing Core default, which is accurate, since Core then uses the deployment's default harness.
  • Test data: the e2e fixture and preview mocks gain two Agents, one with a provider only and one with a harness plus a provider.

Tests

Results: typecheck ✓, unit tests 339/339, build ✓, full web e2e 23/23.

The one docs/web line, docs/web/protocol-coverage.md:55, was reviewed by the backend session.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith with what you need. Autofix is disabled.

An Agent's detail page now has a Model provider section when the Agent
saved x_agents_core.model_provider: its protocol (Anthropic Messages or
OpenAI Responses), base URL, whether an API key is configured, and the
context window and max output tokens when Core reports them. The read
view has no key and the page never shows one.

A saved Agent can have a provider and no harness, so the Agents list and
the detail show "—" for a missing harness instead of "Core default".

The e2e fixture adds two Production Agents, one with only a provider and
one with a harness and a provider, appended after the seeded data so the
rest of the demo is unchanged. One acceptance test checks that the
project's Agents load, the provider-only Agent's harness reads "—", its
provider section shows every field, and no key appears.
An Agent that saved no harness runs on the deployment's default harness,
so "Core default" is accurate, and it matches how the page shows unset
generation settings. The acceptance test now expects it for the
provider-only Agent.
@SaladDay
SaladDay merged commit 697c6a9 into main Sep 25, 2026
2 checks passed
@SaladDay
SaladDay deleted the codex/web-agent-model-provider branch October 7, 2026 06:38
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