Skip to content

Add missing @beartype decorators#2978

Merged
adamtheturtle merged 1 commit intomainfrom
add-missing-beartype
Feb 21, 2026
Merged

Add missing @beartype decorators#2978
adamtheturtle merged 1 commit intomainfrom
add-missing-beartype

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Feb 21, 2026

Summary

  • Add @beartype decorator to functions, classes, and dataclasses that have type annotations but were missing runtime type checking
  • Covers files in _mock_common, _flask_server, _services_validators, _query_validators, and _requests_mock_server

Test plan

  • Run existing test suite to verify no regressions

🤖 Generated with Claude Code


Note

Medium Risk
Mostly mechanical decorator/signature changes, but new runtime type enforcement and keyword-only parameters can raise errors or break callers that previously passed arguments positionally or with unexpected types.

Overview
Adds missing runtime type checking across the mock Vuforia stack by applying @beartype to several Flask route/hooks (vws.py, vwq.py, target_manager.py), shared dataclasses (_mock_common.py), request-mock server utilities (e.g. MissingSchemeError, route decorators), and services/query validator entrypoints.

Tightens a few validator and routing helper APIs by making some parameters keyword-only (adding * to functions like validate_content_type_header, validate_image_is_image, validate_include_target_data, validate_project_state, and run_services_validators, plus route in the requests-mock APIs), which can break positional-call sites but improves call clarity and type enforcement.

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

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@adamtheturtle adamtheturtle enabled auto-merge (squash) February 21, 2026 09:47
@adamtheturtle adamtheturtle merged commit c4c432d into main Feb 21, 2026
12 checks passed
@adamtheturtle adamtheturtle deleted the add-missing-beartype branch February 21, 2026 10:03
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