Skip to content

Summarize AI chat responses within the output budget - #18

Merged
codoom1 merged 1 commit into
mainfrom
agent/summarize-ai-chat-responses
Jul 25, 2026
Merged

Summarize AI chat responses within the output budget#18
codoom1 merged 1 commit into
mainfrom
agent/summarize-ai-chat-responses

Conversation

@codoom1

@codoom1 codoom1 commented Jul 25, 2026

Copy link
Copy Markdown
Owner

What changed

  • Gives the chatbot a feature-specific 1,000-token response budget.
  • Requires answers of at most 120 words with no preamble.
  • Limits model-replacement comparisons to three concise alternatives and one decision sentence.
  • Adds regression coverage for both the summary contract and output budget.

Why

The shared 400-token limit includes GPT-5 mini reasoning tokens, so a concise visible answer could still end with an incomplete provider response. The new bounded chat budget leaves room for reasoning while the prompt keeps the user-facing answer short.

Validation

  • pytest -q tests: 132 passed
  • Focused chatbot and output-budget tests: 3 passed

@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
statistical-model-suggester Ready Ready Preview, Comment Jul 25, 2026 12:40am

@codoom1
codoom1 marked this pull request as ready for review July 25, 2026 00:44
Copilot AI review requested due to automatic review settings July 25, 2026 00:44
@codoom1
codoom1 merged commit 7e2a127 into main Jul 25, 2026
7 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

2 participants