ModelPreferences reports an invalid speedPriority as costPriority
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
Research direction
Locate the ModelPreferences validation and its existing tests, then compare the speedPriority and costPriority error-message checks. Update the incorrect message and add an assertion that an invalid speedPriority reports speedPriority; run the relevant ModelPreferences tests to verify the result.
Written by the indexing model from the issue text.
Description
ModelPreferences validates speedPriority, but the error message says costPriority must be in 0.0 <= x <= 1.0 value range: it was copied from the costPriority check. The tests only assert the exception type.
Expected: the message names speedPriority, and a test checks it.
- Dominant language
- Kotlin
- Stars
- 1.5k
- Forks
- 261
- Avg merge
- 2d 9h
- Merged PRs (30d)
- 36
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from modelcontextprotocol/kotlin-sdk
-
Client.connect does not apply the negotiated protocol version to Streamable HTTP transportPossibly taken @xu-baolin claimed this 24 days ago. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
modelcontextprotocol/kotlin-sdk#1090 ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
modelcontextprotocol/kotlin-sdk#1088 ·
Maintainers usually reply within 1 day
-
An empty URI matches a single-variable resource templatePossibly taken @liyifan9642-creator claimed this 12 days ago. Openbug P3
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
modelcontextprotocol/kotlin-sdk#1085 ·
Maintainers usually reply within 1 day
-
AbstractClientTransport.checkState builds a broken error messagePossibly taken @mvanhorn claimed this 5 days ago. Openapi bug P3
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
modelcontextprotocol/kotlin-sdk#1083 · 1 comment ·
Maintainers usually reply within 1 day
-
Dropped resource-updated notifications are never loggedPossibly taken @leeshengis claimed this 18 days ago. Openbug P3
Difficulty 1/5 Under an hour Newbie friendliness 86/100
modelcontextprotocol/kotlin-sdk#1082 ·
Maintainers usually reply within 1 day
All issues in modelcontextprotocol/kotlin-sdk
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
SimonHalvdansson/Harmonic-HN#399 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
[Request] 李跳跳Openrequest-adaption
Difficulty 2/5 Under an hour Newbie friendliness 78/100
BetterAndroid/android-notification-icon-project#799 ·
Maintainers usually reply within 1 day
-
6.4.0快进问题Open
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
open-ani/animeko#3664 · 4 comments ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100