Merge pull request Expensify#90612 from shubham1206agra/rename-website-web

[NoQA] Renamed .website to .web for simplicity

Author: roryabraham

config/eslint/eslint.seatbelt.tsv

@@ -109,8 +109,8 @@
 "../../src/components/MagicCodeInput.tsx"	"react-hooks/refs"	1
 "../../src/components/MagicCodeInput.tsx"	"react-hooks/set-state-in-effect"	3
 "../../src/components/MapView/MapView.tsx"	"react-hooks/set-state-in-effect"	1
-"../../src/components/MapView/MapView.website.tsx"	"react-hooks/set-state-in-effect"	1
-"../../src/components/MapView/MapViewImpl.website.tsx"	"react-hooks/set-state-in-effect"	1
+"../../src/components/MapView/MapView.web.tsx"	"react-hooks/set-state-in-effect"	1
+"../../src/components/MapView/MapViewImpl.web.tsx"	"react-hooks/set-state-in-effect"	1
 "../../src/components/MapView/ToggleDistanceUnitButton/index.android.tsx"	"no-restricted-syntax"	1
 "../../src/components/MessagesRow.tsx"	"no-restricted-syntax"	1
 "../../src/components/Modal/BaseModal.tsx"	"react-hooks/set-state-in-effect"	1
@@ -364,8 +364,8 @@
 "../../src/libs/actions/IOU/index.ts"	"@typescript-eslint/no-deprecated/getPolicyTagsData"	2
 "../../src/libs/actions/IOU/index.ts"	"rulesdir/no-onyx-connect"	10
 "../../src/libs/actions/ImportTransactions.ts"	"no-restricted-syntax"	1
-"../../src/libs/actions/InputFocus/index.website.ts"	"@typescript-eslint/no-deprecated/InteractionManager.runAfterInteractions"	1
-"../../src/libs/actions/InputFocus/index.website.ts"	"no-restricted-syntax"	1
+"../../src/libs/actions/InputFocus/index.web.ts"	"@typescript-eslint/no-deprecated/InteractionManager.runAfterInteractions"	1
+"../../src/libs/actions/InputFocus/index.web.ts"	"no-restricted-syntax"	1
 "../../src/libs/actions/Link.ts"	"@typescript-eslint/no-deprecated/InteractionManager.runAfterInteractions"	1
 "../../src/libs/actions/Link.ts"	"no-restricted-syntax"	3
 "../../src/libs/actions/MapboxToken.ts"	"no-restricted-syntax"	2
@@ -491,7 +491,7 @@
 "../../src/pages/Share/ShareRootPage.tsx"	"@typescript-eslint/no-deprecated/InteractionManager.runAfterInteractions"	1
 "../../src/pages/TransactionDuplicate/Confirmation.tsx"	"react-hooks/refs"	12
 "../../src/pages/Travel/TravelUpgrade.tsx"	"react-hooks/set-state-in-effect"	1
-"../../src/pages/ValidateLoginPage/index.website.tsx"	"react-hooks/set-state-in-effect"	1
+"../../src/pages/ValidateLoginPage/index.web.tsx"	"react-hooks/set-state-in-effect"	1
 "../../src/pages/domain/Groups/PreferredWorkspaceToggle.tsx"	"@typescript-eslint/no-deprecated/ConfirmModal"	1
 "../../src/pages/domain/Saml/SamlLoginSectionContent.tsx"	"@typescript-eslint/no-deprecated/ConfirmModal"	1
 "../../src/pages/inbox/DeleteTransactionNavigateBackHandler.tsx"	"@typescript-eslint/no-deprecated/InteractionManager.runAfterInteractions"	1
@@ -525,7 +525,7 @@
 "../../src/pages/inbox/report/ReportActionsList.tsx"	"react-hooks/set-state-in-effect"	3
 "../../src/pages/inbox/report/TripSummary.tsx"	"rulesdir/no-default-id-values"	1
 "../../src/pages/inbox/report/UserTypingEventListener.tsx"	"@typescript-eslint/no-deprecated/InteractionManager.runAfterInteractions"	4
-"../../src/pages/inbox/report/shouldUseEmojiPickerSelection/index.website.ts"	"no-restricted-syntax"	1
+"../../src/pages/inbox/report/shouldUseEmojiPickerSelection/index.web.ts"	"no-restricted-syntax"	1
 "../../src/pages/inbox/report/useReportActionsNewActionLiveTail.ts"	"@typescript-eslint/no-deprecated/InteractionManager.runAfterInteractions"	1
 "../../src/pages/inbox/report/withReportOrNotFound.tsx"	"react-hooks/refs"	3
 "../../src/pages/inbox/sidebar/SidebarLinksData.tsx"	"react-hooks/refs"	1
@@ -694,7 +694,7 @@
 "../../src/pages/workspace/workflows/approvals/WorkspaceWorkflowsApprovalsEditPage.tsx"	"@typescript-eslint/no-deprecated/InteractionManager.runAfterInteractions"	2
 "../../src/stories/ReportActionItemImages.stories.tsx"	"no-restricted-syntax"	1
 "../../src/stories/TextInput.stories.tsx"	"react-hooks/set-state-in-effect"	1
-"../../src/styles/utils/autoCompleteSuggestion/index.website.ts"	"no-restricted-syntax"	1
+"../../src/styles/utils/autoCompleteSuggestion/index.web.ts"	"no-restricted-syntax"	1
 "../../src/types/onyx/Account.ts"	"no-restricted-syntax"	1
 "../../src/types/onyx/AccountData.ts"	"no-restricted-syntax"	1
 "../../src/types/onyx/BankAccount.ts"	"no-restricted-syntax"	1

config/webpack/webpack.common.ts

@@ -396,10 +396,9 @@ const getCommonConfiguration = ({file = '.env', platform = 'web'}: Environment):
                 '@selectors': path.resolve(dirname, '../../src/selectors/'),
             },
 
-            // React Native libraries may have web-specific module implementations that appear with the extension `.web.js`
-            // without this, web will try to use native implementations and break in not very obvious ways.
-            // This is also why we have to use .website.js for our own web-specific files...
-            extensions: ['.web.js', '.website.js', '.js', '.jsx', '.web.ts', '.website.ts', '.website.tsx', '.ts', '.web.tsx', '.tsx'],
+            // Resolve web-specific implementations (`.web.*`) before bare files so React Native
+            // libraries and our own app-level overrides both pick up their browser variants.
+            extensions: ['.web.js', '.js', '.jsx', '.web.ts', '.web.tsx', '.ts', '.tsx'],
             fallback: {
                 'process/browser': require.resolve('process/browser'),
                 crypto: false,

contributingGuides/philosophies/CROSS-PLATFORM.md

@@ -28,11 +28,11 @@ Put all platform specific code in dedicated files and folders (see below) and re
 In most cases, the code written for this repo should be platform-independent. In such cases, each module should have a single file, `index.js`, which defines the module's exports. There are, however, some cases in which a feature is intrinsically tied to the underlying platform. In such cases, the following file extensions can be used to export platform-specific code from a module:
 - Mobile => `index.native.js`
 - iOS Native App/Android Native App => `index.ios.js`/`index.android.js`
-- Web => `index.website.js`
+- Web => `index.web.js`
 
 **Note:** `index.js` should be the default and only platform-specific implementations should be done in their respective files. i.e: If you have mobile-specific implementation in `index.native.js`, then the web implementation can be contained in a shared `index.js`.
 
-`index.ios.js` and `index.android.js` are used when the app is running natively on respective platforms. These files are not used when users access the app through mobile browsers, but `index.website.js` is used instead. `index.native.js` are for both iOS and Android native apps. `index.native.js` should not be included in the same module as `index.ios.js` or `index.android.js`.
+`index.ios.js` and `index.android.js` are used when the app is running natively on respective platforms. These files are not used when users access the app through mobile browsers, but `index.web.js` is used instead. `index.native.js` are for both iOS and Android native apps. `index.native.js` should not be included in the same module as `index.ios.js` or `index.android.js`.
 
 ### Supporting Mobile Web
 The above platform-specific files only work because they are compiled when the app is built for the different platforms. This means that a different mechanism needs to be used for Mobile Web (since "mobile web" and "web" are the same at build time).

…teGlobalBackgroundColor/index.website.ts …updateGlobalBackgroundColor/index.web.tssrc/components/CustomStatusBarAndBackground/updateGlobalBackgroundColor/index.website.ts renamed to src/components/CustomStatusBarAndBackground/updateGlobalBackgroundColor/index.web.ts src/components/CustomStatusBarAndBackground/updateGlobalBackgroundColor/index.website.ts renamed to src/components/CustomStatusBarAndBackground/updateGlobalBackgroundColor/index.web.ts

@@ -9,7 +9,7 @@ import type {SkeletonSpanReasonAttributes} from '@libs/telemetry/useSkeletonSpan
 import type {MapViewProps} from './MapViewTypes';
 import PendingMapView from './PendingMapView';
 
-const MapViewImpl = lazy(() => import('./MapViewImpl.website'));
+const MapViewImpl = lazy(() => import('./MapViewImpl.web'));
 
 function MapView({ref, ...props}: MapViewProps) {
     const {isOffline} = useNetwork();