Skip to content

fix: temperature validation boundaries + short message session titles (closes #347, closes #82)#406

Open
mayank-dev-15 wants to merge 1 commit into
imDarshanGK:mainfrom
mayank-dev-15:fix/temperature-validation-short-titles
Open

fix: temperature validation boundaries + short message session titles (closes #347, closes #82)#406
mayank-dev-15 wants to merge 1 commit into
imDarshanGK:mainfrom
mayank-dev-15:fix/temperature-validation-short-titles

Conversation

@mayank-dev-15

Copy link
Copy Markdown

Changes

Temperature Validation (#347)

  • Added ge=0.0, le=2.0 to AppSettings.temperature field in schemas.py
  • ChatRequest already had these bounds; now settings endpoint matches

Short Message Session Titles (#82)

  • Skip auto-title when stripped message is under 3 characters
  • Prevents useless titles like Hi... or Ok

Closes #347
Closes #82

- schemas.py: add ge=0.0, le=2.0 to AppSettings temperature (imDarshanGK#347)
- db_service.py: skip auto-title for messages under 3 chars (imDarshanGK#82)
@imDarshanGK imDarshanGK added invalid This doesn't seem right unassigned PR opened without issue assignment labels Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right unassigned PR opened without issue assignment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix: session title auto-update doesn't work for very short messages

2 participants