Skip to content

feat: add portable request controls and validation - #6

Merged
jkyberneees merged 6 commits into
mainfrom
cursor/chat-controls-validation-55a2
Sep 5, 2026
Merged

feat: add portable request controls and validation#6
jkyberneees merged 6 commits into
mainfrom
cursor/chat-controls-validation-55a2

Conversation

@jkyberneees

@jkyberneees jkyberneees commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add portable TopP and Stop controls across OpenAI, Gemini, and Anthropic
  • complete documented Thinking: "max" behavior and honor explicit budgets across active presets
  • reject unsigned Anthropic thinking replay before any network request
  • safely encode opaque Anthropic and Gemini model-pagination tokens
  • return typed configuration errors for nil requests
  • remove deprecated dialer configuration, correct learn-once documentation, and add explicit test timeouts

Verification

  • RED-first cross-format request, thinking, replay-validation, and pagination tests added
  • make quality passes
  • go test -race -count=1 -timeout 120s ./... passes
  • make lint passes with zero issues
  • statement coverage: 97.8%
Open in Web Open in Cursor 

cursoragent and others added 3 commits September 5, 2026 10:10
Co-authored-by: Rolando Santamaria Maso <jkyberneees@users.noreply.github.com>
Co-authored-by: Rolando Santamaria Maso <jkyberneees@users.noreply.github.com>
Co-authored-by: Rolando Santamaria Maso <jkyberneees@users.noreply.github.com>
@jkyberneees
jkyberneees marked this pull request as ready for review September 5, 2026 10:14
cursoragent and others added 3 commits September 5, 2026 10:15
Co-authored-by: Rolando Santamaria Maso <jkyberneees@users.noreply.github.com>
Co-authored-by: Rolando Santamaria Maso <jkyberneees@users.noreply.github.com>
Co-authored-by: Rolando Santamaria Maso <jkyberneees@users.noreply.github.com>
@jkyberneees
jkyberneees merged commit 58c2ef3 into main Sep 5, 2026
7 checks passed
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