Repository navigation
perf(react-compiler): enable Checkbox, Tooltip, and useForceUpdate - #8511
Conversation
🦋 Changeset detectedLatest commit: 45bb6ac The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
Integration test results from github/github-ui PR:
All checks passed! |
a636f8e to
62392d2
Compare
62392d2 to
9953239
Compare
9953239 to
7929a9d
Compare
|
7929a9d to
8fe5d18
Compare
|
|
There was a problem hiding this comment.
🟢 Approval recommended
The narrowly scoped exclusions update has no identified blocking issues, and reported validation covers the affected behavior.
0 open findings
What changed in this PR
Extends React Compiler adoption in @primer/react without source refactors or public API changes.
Changes:
- Removes compiler exclusions for Checkbox, legacy Tooltip,
useForceUpdate, and Autocomplete test fixtures. - Adds a minor changeset describing component rendering optimizations.
| File | Description |
|---|---|
packages/react/script/react-compiler.mjs |
Enables compilation for four previously excluded files. |
.changeset/react-compiler-low-risk-batch-2.md |
Records the component optimizations for release. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
6e1d8a3 to
1428f99
Compare
|
1428f99 to
45bb6ac
Compare
|
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: cf031929-064e-4482-961d-25594325eb1b
45bb6ac to
1f834ae
Compare
|
No issue to close.
Enable React Compiler for Checkbox, the legacy Tooltip, and
useForceUpdate. These files already compile without source refactors or public API changes.Changelog
New
None.
Changed
useForceUpdate.Removed
Remove the corresponding compiler exclusions. No public APIs are removed.
Rollout strategy
Minor release for coordinated compiler optimizations, with no intended visual or accessibility changes.
Testing & Reviewing
Compiler checks, the full test suite, build, type checking, lint, and formatting pass. Checkbox and legacy Tooltip stories render successfully in Chromium.
Review Checkbox's controlled, uncontrolled, and indeterminate states, ref forwarding with both existing feature-flag settings, and Tooltip's accessible labels.
ActionMenu and SelectPanel test files remain excluded: enabling their compilation exposed anchor replacement and initial empty-state behavior that needs separate work.
Dedicated VRT/AVT and github-ui integration were not run.