Skip to content

Conversation

@mdjastrzebski
Copy link
Member

@mdjastrzebski mdjastrzebski commented Jan 8, 2026

Summary

Removes all unsafe sync functions from the public API:

  • unsafe_act
  • unsafe_renderSync
  • unsafe_fireEventSync
  • unsafe_renderHookSync

These functions were deprecated in favor of their async counterparts (act, render, fireEvent, renderHook). All tests and documentation updated.

Test plan

  • All tests pass (yarn test, yarn typecheck)
  • Removed functions are no longer accessible from public API
  • Migration guide updated with breaking change documentation

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.10%. Comparing base (2f582ac) to head (7d0cc8a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1855      +/-   ##
==========================================
- Coverage   91.23%   91.10%   -0.14%     
==========================================
  Files          93       92       -1     
  Lines        5432     5215     -217     
  Branches      907      869      -38     
==========================================
- Hits         4956     4751     -205     
+ Misses        471      459      -12     
  Partials        5        5              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mdjastrzebski mdjastrzebski merged commit 16e350f into main Jan 8, 2026
26 checks passed
@mdjastrzebski mdjastrzebski deleted the refactor/remove-unsafe-sync-functions branch January 8, 2026 22:02
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.

2 participants