Skip to content

feat(combos): add custom context_length configuration for model combos - #58

Closed
mahdiwafy wants to merge 5 commits into
Vanszs:mainfrom
mahdiwafy:feat/combo-custom-context-length
Closed

feat(combos): add custom context_length configuration for model combos#58
mahdiwafy wants to merge 5 commits into
Vanszs:mainfrom
mahdiwafy:feat/combo-custom-context-length

Conversation

@mahdiwafy

Copy link
Copy Markdown

Summary

Adds support for configuring and advertising an optional custom context_length on model combos.

Changes

  1. Schema & DB Repo: Expose optional context_length column on combos table (SCHEMA_VERSION bumped to 2).
  2. API: Accept and update context_length in /api/combos CRUD endpoints.
  3. Model Listing: Expose context_length in /v1/models output for combo entries when set.
  4. Dashboard UI: Add an optional "Advertised Context Length" input field in Combo create/edit modal and render context length badges on combo cards.

@mahdiwafy

Copy link
Copy Markdown
Author

Heads up: this PR touches src/app/api/combos/route.js and src/lib/db/repos/combosRepo.js to add the context_length field — and #61 touches the exact same two files/functions to add the alias field (both insert into the same SQL statements). Whichever of these two merges first will cause a straightforward-but-real text conflict for the other on merge.

Not a blocker, just flagging it so whoever merges second knows to expect it and can resolve by keeping both fields (verified locally: manually merging both changes together works cleanly, no functional conflict — just needs both columns/params present in the same INSERT/UPDATE).

@mahdiwafy mahdiwafy mentioned this pull request Jul 26, 2026
@mahdiwafy

Copy link
Copy Markdown
Author

Superceded and merged locally into the main working branch (fix/claude-5-models). Closing to clean up PR list.

@mahdiwafy mahdiwafy closed this Jul 29, 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