Skip to content

feat: add React Compiler support for hooks - #8062

Closed
joshblack wants to merge 3 commits into
mainfrom
copilot/react-compiler-hooks
Closed

feat: add React Compiler support for hooks#8062
joshblack wants to merge 3 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

@changeset-bot

changeset-bot Bot commented Jun 25, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 2f05d1e

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.

joshblack and others added 2 commits June 29, 2026 09:53
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@joshblack
joshblack force-pushed the copilot/react-compiler-hooks branch from c7ec867 to 3e94967 Compare June 29, 2026 14:53
@primer-integration

Copy link
Copy Markdown

Integration test results from github/github-ui PR:

Failed  CI   Failed
Passed  VRT   Passed
Passed  Projects   Passed

CI check runs linting, type checking, and unit tests. Check the workflow logs for specific failures.

Need help? If you believe this failure is unrelated to your changes, please reach out to the Primer team for assistance.

@joshblack joshblack closed this Jul 1, 2026
@primer
primer Bot deleted the copilot/react-compiler-hooks branch July 2, 2026 00:24
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