Skip to content

Never list hidden models - #57

Merged
irl-dan merged 1 commit into
mainfrom
fix/hide-hidden-models
Oct 9, 2026
Merged

irl-dan merged 1 commit into
mainfrom
fix/hide-hidden-models

Conversation

@rawwerks

@rawwerks rawwerks commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

I'm an AI agent (Claude Code) opening this PR on behalf of Ray (@rawwerks). He asked for this change and approved opening the PR, but he did not write the text below.

Why

The service marks some older model ids hidden: still accepted when named, but no longer offered. The product rule is that hidden models are not shown in any interface. The web app's model picker already omits them. But prose cli model list printed them in an "Also accepted (not recommended)" section, with their successors, and kept them in result.catalog.

What changes

  • The listing skips hidden entries. model list's catalog projection now skips hidden entries, as it already did deprecated ones, in both the Rust and Bun ports. The human "Also accepted (not recommended)" section is gone.
  • Acceptance is unchanged. A hidden model named with --model, or as a job spec's model, still passes the local check and reaches the service. The premium and acceptance checks read the full catalog, not the listing. runs/submit-preview-hidden-model-accepted keeps pinning this.
  • Nothing else changes. service status, service triage, capabilities, help and the guide didn't enumerate hidden models.
  • Corpus:
    • discovery/model-list-catalog-json loses the hidden entry.
    • discovery/model-list-catalog-human loses the section.
    • Both descriptions are updated.
  • Also updated: the Rust unit test, docs/service/discovery.md, the discovery schema's description and the CHANGELOG.

Evidence

Local gates pass:

  • shared-contracts, service-help, public-surface-files and service-operations-corpus
  • differential-conformance and service-coverage
  • rust-format and rust-clippy
  • service-operations-rust and service-operations-bun: 1089 cases, 0 failures each
  • bun-typecheck

The Rust discovery unit tests pass, and the two changed cases were accepted only where Rust and Bun produce identical output.

🤖 Generated with Claude Code

model list printed an "Also accepted (not recommended)" section and kept
hidden entries in result.catalog. Hidden models are accepted but not
shown in any interface, so the listing now skips them like deprecated
models. Acceptance is unchanged: a named hidden model still passes the
local --model check and reaches the service.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@irl-dan
irl-dan merged commit 540c09f into main Oct 9, 2026
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.

2 participants