Skip to content

Show inactive model providers - #2144

Merged
DanielButler1 merged 2 commits into
mainfrom
fix/show-inactive-fable-providers-20260902
Sep 1, 2026
Merged

Show inactive model providers#2144
DanielButler1 merged 2 commits into
mainfrom
fix/show-inactive-fable-providers-20260902

Conversation

@DanielButler1

Copy link
Copy Markdown
Member

Summary

  • show known public provider routes on model pricing pages even when Phaseo routing is disabled
  • keep inactive routes explicitly non-routable through the existing is_active_gateway calculation
  • add a database contract check covering inactive-provider visibility

This lets Fable 5.1 display its tracked Bedrock, Vertex, Foundry, and regional Anthropic offers without enabling any of them for gateway traffic.

Validation

  • node scripts/validate-supabase-migrations.mjs origin/main
  • git diff --check origin/main...HEAD

Release impact

  • Requires DB migration
  • No migration / no special steps
  • No package changeset: this changes the public catalogue projection without versioning an SDK or package.

Created with Codex

@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
phaseo Ready Ready Preview Sep 1, 2026 11:47pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Sep 1, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 6 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 8402c5ed-1d53-4797-81c8-95acd6ffedab

📥 Commits

Reviewing files that changed from the base of the PR and between 4b83433 and 328558d.

📒 Files selected for processing (2)
  • supabase/migrations/20260902120000_show_inactive_provider_routes_in_pricing.sql
  • supabase/tests/stealth_catalogue_security_smoke.sql

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@DanielButler1 DanielButler1 added the Impact: High Impact Impact: High impact label Sep 1, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a0ad1b789f

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@DanielButler1

Copy link
Copy Markdown
Member Author

Addressed the inactive-variant review in 328558d: variant status and routing_enabled now participate in is_active_gateway, so disabled variants remain visible without becoming routable.

@DanielButler1
DanielButler1 merged commit 2c57db2 into main Sep 1, 2026
51 checks passed
@DanielButler1
DanielButler1 deleted the fix/show-inactive-fable-providers-20260902 branch September 1, 2026 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Impact: High Impact Impact: High impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant