Skip to content

fix: resolve free-mode providers from live economics - #12750

Closed
keeltrace wants to merge 5 commits into
diegosouzapw:mainfrom
keeltrace:fix/free-mode-provider-resolution-main-20260904
Closed

fix: resolve free-mode providers from live economics#12750
keeltrace wants to merge 5 commits into
diegosouzapw:mainfrom
keeltrace:fix/free-mode-provider-resolution-main-20260904

Conversation

@keeltrace

@keeltrace keeltrace commented Sep 4, 2026

Copy link
Copy Markdown

Summary

  • preserve provider-declared/zero-price free status in synced model metadata and scope it per connection
  • let auto/*:free, hidePaidModels, and strict-zero-cost routing honor live free economics and explicit operator free-tier overrides
  • discover Nous Portal recommendations dynamically, keep the full curated Nous catalog, and treat live :free variants as strict-zero-cost routes
  • add regression coverage for Nous rotation, W&B-style operator overrides, per-connection free status, and discovery persistence

Validation

  • exact PR head: 853865b369116b286ade67d82af80e8a389fca90
  • exact base: c0b2253f21c2e70c5d73581ae1be6f60a4ac5647
  • code quality, both normal Vitest shards, backend build/tests, Rustfmt, Clippy, Rust tests, frontend E2E, security audit, AppImage, MSI, Coverage, Quality Ratchet, and CI Dashboard passed
  • the shared test:vitest:ui lane ran 66 tests: both autoCombo suites passed; its only failures were two unrelated ProviderSettings password/remount tests timing out at 10s
  • the exact base SHA passed the same Vitest (MCP / autoCombo / UI components) lane on main in CI run 33819289311, job 100858366268
  • the separate Build failure ended in runner/system shutdown (SIGTERM), not a demonstrated source assertion/build failure
  • contributor workflow rerun is unavailable (403: Resource not accessible by integration); maintainer rerun/authorization is the remaining CI action

No source change is warranted solely to perturb CI: the directly affected autoCombo coverage passed on this head and the failing assertions are outside this PR's change surface. No merge requested.

@keeltrace

Copy link
Copy Markdown
Author

Exact-head CI diagnosis for 853865b369116b286ade67d82af80e8a389fca90 (base c0b2253f21c2e70c5d73581ae1be6f60a4ac5647): the directly affected autoCombo UI suites passed. The shared test:vitest:ui lane ran 66 tests; its only failures were two ProviderSettings password/remount tests timing out at 10s. The exact base SHA passed this same UI lane on main in CI run 33819289311 / job 100858366268. The separate Build failure terminated via runner/system shutdown (SIGTERM), not a demonstrated source assertion/build failure.

The contributor integration cannot rerun the upstream workflow (403: Resource not accessible by integration). No no-op/source perturbation is justified just to retrigger CI. Remaining action is a maintainer rerun/authorization plus review; no merge requested.

@keeltrace

Copy link
Copy Markdown
Author

Superseded by #12763 on the active release/v3.8.51 line. Closing this duplicate so there is only one review surface.

@keeltrace
keeltrace marked this pull request as draft September 5, 2026 06:17
@keeltrace keeltrace closed this Sep 5, 2026
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