fix(useFocusVisible): use Reflect.defineProperty to wrap HTMLElement.prototype.focus

[antododo]

Closes #9649

✅ Pull Request Checklist:

• Included link to corresponding React Spectrum GitHub Issue.
• Added/updated unit tests and storybook for this change (for new code or code which already has tests).
• Filled out test instructions.
• Updated documentation (if it already exists for this component).
• Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

1. yarn jest packages/react-aria/test/interactions/useFocusVisible.test.js — all 18 tests pass, including the new regression test does not throw when HTMLElement.prototype.focus is an accessor-only property.
2. Confirm the regression: temporarily revert just the src/ change, keep the new test, and rerun the command above. The new test should fail with TypeError: Cannot set property focus of [object HTMLElement] which has only a getter. Re-apply the src/ change and the test passes.
3. Smoke-test the wider interactions suite to confirm no regression: yarn jest packages/react-aria/test/interactions/ — all 237 tests pass.
4. In a real browser, nothing should change — focus rings, keyboard vs. pointer modality, and screen-reader virtual focus detection behave identically (the Reflect.defineProperty call is equivalent to the previous assignment against the default HTMLElement.prototype.focus data descriptor).

🧢 Your Project:

While building components on react-aria-components for Swissquote.ch internal design system where consumers' Jest+JSDOM test suites using @testing-library/user-event's setup() crash on import of any RAC-using component.

[nwidynski]

At this point, why override at all instead of using an apply trap?

[antododo]

At this point, why override at all instead of using an apply trap?

@nwidynski The bug is the install mechanism (getter-only focus accessor breaks plain assignment), not the wrapper shape, so a Proxy would still need the same defineProperty + try/catch. Happy to switch if you see a concrete benefit.

[nwidynski]

@antododo Not with a Proxy, but via Reflect API. See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Reflect/defineProperty

That wont throw when failing to install the override.

[antododo]

@nwidynski thanks for the suggestion, PR updated.

[snowystinger]

I'm fine with this, hopefully userEvent gets their act together soon

[antododo]

@snowystinger Hello and thanks for the approval. How can we get a second one? Thanks in advance!