feat(github_copilot): add gpt-5.6 luna, terra, sol - #33966
Conversation
Greptile SummaryAdds three new GitHub Copilot
Confidence Score: 5/5Safe to merge — the change is purely additive JSON data with no logic modifications. The PR adds three new model entries to two JSON files and a corresponding mock-based unit test. The entries are consistent with each other and with the existing responses-mode catalog entries. No production logic is altered, and the test confirms the metadata is correct without making live API calls. No files require special attention.
|
| Filename | Overview |
|---|---|
| model_prices_and_context_window.json | Adds three new github_copilot gpt-5.6 model entries (luna, terra, sol) with consistent fields matching the pattern of existing responses-mode catalog entries |
| litellm/model_prices_and_context_window_backup.json | Identical additions to the backup file; the three new gpt-5.6 model entries are consistent with the primary JSON file |
| tests/test_litellm/llms/github_copilot/responses/test_github_copilot_responses_transformation.py | Adds a parametrized test that validates all three new model catalog entries; relies on the module-level autouse fixture to pin litellm.model_cost to the local backup file, so no real network calls are made |
Reviews (1): Last reviewed commit: "feat(github_copilot): add gpt-5.6 luna, ..." | Re-trigger Greptile
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
80b5d97 to
5f1986a
Compare
|
Any update on this? |
5f1986a to
52d4fa0
Compare
52d4fa0 to
3192c49
Compare
|
gpt-5.6 Luna, Terra and Sol via GitHub Copilot would really be appreciated here |
Relevant issues
Fixes #33893
Linear ticket
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
@greptileaito re-request a review after pushing changes)Screenshots / Proof of Fix
Type
🆕 New Feature
🐛 Bug Fix
Changes
added 5.6 models for GitHub Copilot in
model_prices_and_context_window.json