Skip to content

feat: add React Compiler support for hooks#8062

Draft
joshblack wants to merge 2 commits into
mainfrom
copilot/react-compiler-hooks
Draft

feat: add React Compiler support for hooks#8062
joshblack wants to merge 2 commits into
mainfrom
copilot/react-compiler-hooks

Conversation

@joshblack

Copy link
Copy Markdown
Member

Closes #

Adds React Compiler support for the remaining ignored shared hooks by removing them from the compiler unsupported list.

Changelog

New

  • React Compiler support for shared hooks

Changed

  • None

Removed

  • Removed the remaining shared hooks from the React Compiler unsupported list

Rollout strategy

  • Patch release
  • Minor release
  • Major release; if selected, include a written rollout or migration plan
  • None; if selected, include a brief description as to why

Testing & Reviewing

  • npx eslint --no-cache packages/react/src/hooks/useAnchoredPosition.ts packages/react/src/hooks/useFocusTrap.ts packages/react/src/hooks/useFocusZone.ts packages/react/src/hooks/useMenuInitialFocus.ts packages/react/src/hooks/useOnEscapePress.ts packages/react/src/hooks/useResizeObserver.ts packages/react/src/hooks/useSafeTimeout.ts packages/react/src/hooks/useScrollFlash.ts packages/react/src/hooks/useMergedRefs.ts
  • npm test -- --run packages/react/src/hooks/__tests__/useAnchoredPosition.test.tsx packages/react/src/hooks/__tests__/useMenuInitialFocus.test.tsx packages/react/src/hooks/__tests__/useMergedRefs.test.tsx packages/react/src/hooks/__tests__/useOnEscapePress.test.tsx packages/react/src/hooks/__tests__/useSafeTimeout.test.tsx
  • npm run build && npm test -- --run && npm run type-check && npm run lint && npm run lint:css && npm run format:diff

Merge checklist

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@changeset-bot

changeset-bot Bot commented Jun 25, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c7ec867

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/react Patch

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

@github-actions github-actions Bot added staff Author is a staff member integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm labels Jun 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Action required

👋 Hi, this pull request contains changes to the source code that github/github-ui depends on. If you are GitHub staff, test these changes with github/github-ui using the integration workflow. If this doesn't work, you can also use the original workflow here. Check the integration testing docs for step-by-step instructions. Or, apply the integration-tests: skipped manually label to skip these checks.

To publish a canary release for integration testing, apply the Canary Release label to this PR.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm staff Author is a staff member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant