Skip to content

refactor: add type-safe useAppContext hook #2480

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
afonsojramos merged 1 commit into from
Dec 29, 2025
Merged

refactor: add type-safe useAppContext hook #2480

afonsojramos merged 1 commit into from
Dec 29, 2025

Conversation

@afonsojramos
Copy link
Member

@afonsojramos afonsojramos commented Dec 28, 2025

Summary

  • Adds useAppContext() hook for type-safe context access
  • Replaces manual useContext(AppContext) calls throughout codebase

Benefits

  • Type-safe: returns AppContextState without casting
  • Fail-fast: throws if used outside provider
  • Cleaner imports: one hook instead of context + useContext

Test plan

  • All existing tests pass
  • Lint passes

@github-actions github-actions bot added the refactor Refactoring of existing feature label Dec 28, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 28, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
93.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@afonsojramos afonsojramos mentioned this pull request Dec 28, 2025
Draft
6 tasks
setchy
setchy approved these changes Dec 29, 2025
View reviewed changes
Copy link
Member

@setchy setchy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@afonsojramos afonsojramos merged commit 9a5bd05 into main Dec 29, 2025
14 checks passed
@afonsojramos afonsojramos deleted the use-app-context-hook branch December 29, 2025 00:13
@github-actions github-actions bot added this to the Release 6.15.0 milestone Dec 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@setchy setchy setchy approved these changes

@bmulholland bmulholland Awaiting requested review from bmulholland bmulholland is a code owner

Assignees

No one assigned

Labels

refactor Refactoring of existing feature

Milestone

Release 6.15.0

Development

Successfully merging this pull request may close these issues.

3 participants

@afonsojramos @setchy