Skip to content
This repository was archived by the owner on Jun 12, 2026. It is now read-only.

fix: move oauth flow intent literal into proto and align oauth state typing - #175

Merged
mplemay merged 1 commit into
mainfrom
fix-oauth-state-type-mismatch
May 3, 2026
Merged

fix: move oauth flow intent literal into proto and align oauth state typing#175
mplemay merged 1 commit into
mainfrom
fix-oauth-state-type-mismatch

Conversation

@mplemay

@mplemay mplemay commented May 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • Moved OAuthFlowIntent into belgie_proto and re-exported it through the existing public proto surface.
  • Updated OAuthStateMixin to use Mapped[OAuthFlowIntent] and Mapped[JSONValue] so it matches OAuthStateProtocol.
  • Kept the SQLAlchemy column definitions unchanged while fixing the type-level contract that blocked BelgieAdapter(..., oauth_state=OAuthState).

Testing

  • uv run pytest packages/belgie-alchemy/src/belgie_alchemy/__tests__/unit/core/adapter/test_adapter.py passed.
  • uv run ty check passed for an isolated BelgieAdapter(..., oauth_state=OAuthState) constructor check.
  • uv run ruff check passed for the edited proto, oauth, and alchemy files.

@mplemay
mplemay merged commit 4041521 into main May 3, 2026
4 checks passed
@mplemay
mplemay deleted the fix-oauth-state-type-mismatch branch May 3, 2026 19:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant