Skip to content

[TOO-1171] feat(arcade-core): add Sentry OAuth2 provider class#874

Draft
EricGustin wants to merge 1 commit into
mainfrom
ericgustin/sentry-auth-provider
Draft

[TOO-1171] feat(arcade-core): add Sentry OAuth2 provider class#874
EricGustin wants to merge 1 commit into
mainfrom
ericgustin/sentry-auth-provider

Conversation

@EricGustin

Copy link
Copy Markdown
Member

Add the Sentry OAuth2 provider class

Adds class Sentry(OAuth2) (provider_id = "sentry") so toolkit authors can declare requires_auth=Sentry(), mirroring the existing well-known provider classes, and re-exports it from arcade_tdk.auth and arcade_mcp_server.auth.

Sister to the Engine well-known provider (monorepo, TOO-1170) and the docs page (TOO-1175).

Patch coverage proof

The new class adds a super().__init__ line that only runs on instantiation. libs/tests/core/test_auth_providers.py (added here) parametrizes over every OAuth2 subclass in arcade_core.auth and constructs each, so a newly added provider is covered with no per-provider edit:

Name                                          Stmts   Miss  Cover
libs/arcade-core/arcade_core/auth.py            106      0   100%
============================== 46 passed ==============================

ruff check on the changed files: clean.

Closes TOO-1171

🤖 Generated with Claude Code

Adds `class Sentry(OAuth2)` (provider_id "sentry") so toolkit authors can declare
`requires_auth=Sentry()`, mirroring the existing well-known provider classes, and
re-exports it from the arcade-tdk and arcade-mcp-server auth modules.

Sister to the Engine well-known provider (TOO-1170). Adds the dynamic
test_auth_providers.py, which constructs every OAuth2 subclass so a newly added
provider is covered with no per-provider edit; arcade_core/auth.py reports 100%
coverage with Sentry included.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Jun 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
libs/arcade-core/arcade_core/auth.py 100.00% <100.00%> (+14.70%) ⬆️
...cade-mcp-server/arcade_mcp_server/auth/__init__.py 100.00% <ø> (ø)
libs/arcade-tdk/arcade_tdk/auth/__init__.py 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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