Skip to content

Fix native component definitions to codegen in all situations#4027

Open
JasonVMo wants to merge 5 commits intomainfrom
user/jasonvmo/fix-native-components
Open

Fix native component definitions to codegen in all situations#4027
JasonVMo wants to merge 5 commits intomainfrom
user/jasonvmo/fix-native-components

Conversation

@JasonVMo
Copy link
Contributor

Platforms Impacted

  • all

Description of changes

Codegen's ability to find things like type files varies quite a bit. When running directly from typescript files:

  • it will not be able to pull in types from other files reliably
  • it will not generate correctly for things defined using types instead of interfaces
  • react-native now exports most of the types needed that we had to redefine previously.

This fixes bundling against 81 (combined with the default exports fix), allowing codegen to run as part of the bundle and build.

Verification

Combination of manual steps, and automated tests.

@JasonVMo JasonVMo requested a review from acoates-ms February 28, 2026 00:52
@changeset-bot
Copy link

changeset-bot bot commented Feb 28, 2026

🦋 Changeset detected

Latest commit: 7f526e8

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

This PR includes changesets to release 16 packages
Name Type
@fluentui-react-native/contextual-menu Patch
@fluentui-react-native/vibrancy-view Patch
@fluentui-react-native/focus-trap-zone Patch
@fluentui-react-native/menu-button Patch
@fluentui-react-native/focus-zone Patch
@fluentui-react-native/tooltip Patch
@fluentui-react-native/experimental-avatar Patch
@fluentui-react-native/callout Patch
@fluentui-react-native/button Patch
@fluentui-react-native/experimental-menu-button Patch
@fluentui/react-native Patch
@fluentui-react-native/menu Patch
@fluentui-react-native/radio-group Patch
@fluentui-react-native/tablist Patch
@fluentui-react-native/dropdown Patch
@fluentui-react-native/notification 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

@JasonVMo JasonVMo requested a review from Saadnajmi February 28, 2026 00:52
Comment on lines +10 to +13
"prettier.singleQuote": true,
"prettier.tabWidth": 2,
"prettier.printWidth": 140,
"prettier.trailingComma": "all",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if oxfmt respects this? Regardless probably delete? There's a way to make oxfmt so format on save

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For now we have the prettier formatter still being the one in vscode. I've found the oxc plugin is kind of crashy right now so until we make that work this is a stopgap so that we don't get immediate conflicts on save.

| 'rightCenter'
| 'rightBottomEdge'
| 'bottonLeftEdge'
| 'bottomLeftEdge'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a breaking change to internal code. Particularly Excel.

@@ -1,15 +1,13 @@
import type * as React from 'react';
import type { KeyboardMetrics, ViewStyle, ViewProps } from 'react-native';
import type { KeyboardMetrics, ViewStyle } from 'react-native';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A note... Callout never got a version with the v1 framework. Would be good AI fodder one day..

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