Skip to content

Enforce keyword-only arguments#2977

Merged
adamtheturtle merged 1 commit intomainfrom
add-keyword-only-args
Feb 21, 2026
Merged

Enforce keyword-only arguments#2977
adamtheturtle merged 1 commit intomainfrom
add-keyword-only-args

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Feb 21, 2026

Summary

  • Add * separator to enforce keyword-only arguments in functions with 2+ parameters
  • Update all call sites to use keyword arguments
  • Applies to validator functions and route decorators

Test plan

  • Run existing test suite to verify no regressions

🤖 Generated with Claude Code


Note

Medium Risk
Moderate risk because tightening function signatures and adding runtime type enforcement can break downstream callers or raise new exceptions at runtime if any call sites/types are mismatched.

Overview
Enforces keyword-only arguments on several multi-parameter functions (notably query/service validators and the route decorators) by adding *, requiring call sites to pass explicit keywords.

Expands runtime type checking by adding @beartype to additional Flask handlers (vws.py, vwq.py, target_manager.py) and to shared utility types (RequestData, Route, _Route, MissingSchemeError).

Written by Cursor Bugbot for commit 4f7859b. This will update automatically on new commits. Configure here.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@adamtheturtle adamtheturtle reopened this Feb 21, 2026
@adamtheturtle adamtheturtle enabled auto-merge (squash) February 21, 2026 09:47
@adamtheturtle adamtheturtle merged commit c7ae974 into main Feb 21, 2026
210 checks passed
@adamtheturtle adamtheturtle deleted the add-keyword-only-args branch February 21, 2026 10:00
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.

1 participant