Skip to content

Centralize backend ordering logic#50

Merged
CSRessel merged 2 commits intomainfrom
improve-backend-ordering
Nov 17, 2025
Merged

Centralize backend ordering logic#50
CSRessel merged 2 commits intomainfrom
improve-backend-ordering

Conversation

@CSRessel
Copy link
Copy Markdown
Collaborator

Summary

🤖 Generated with Nori

  • Created a single BACKEND_OPTIONS constant containing all backend metadata (name, availability check, factory function)
  • Eliminated disconnected agents and backend_availability vectors that could get out of sync
  • Updated get_backend() to use centralized options instead of hardcoded match statement
  • Updated UI and installation handling to use the centralized system
  • Added comprehensive tests to verify backend ordering and instantiation

Test Plan

  • All existing tests pass
  • New tests verify backend ordering consistency
  • New tests verify correct backend instantiation for each index
  • Integration tests verify UI displays backends correctly

Share Nori with your team: https://www.npmjs.com/package/nori-ai

- Add BackendOption struct and BACKEND_OPTIONS constant
- Remove separate agents and backend_availability vectors from Model
- Update get_backend() to use centralized options
- Update UI and installation handling
- Add comprehensive tests for backend ordering
@CSRessel CSRessel merged commit bba48b0 into main Nov 17, 2025
3 checks passed
@CSRessel CSRessel deleted the improve-backend-ordering branch November 17, 2025 19:46
CSRessel pushed a commit that referenced this pull request Nov 19, 2025
Signed-off-by: Thibault Sottiaux <tibo@openai.com>
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