From ef3c4328a3a969c7c078202c80c73b2c2ec87b88 Mon Sep 17 00:00:00 2001 From: Jan Kassens Date: Thu, 5 Jan 2023 17:31:19 -0500 Subject: [PATCH 1/8] flow codemod annotate-empty-array --write . --- packages/react-debug-tools/src/ReactDebugHooks.js | 4 ++-- .../src/backend/legacy/renderer.js | 6 +++--- packages/react-devtools-shared/src/backend/renderer.js | 2 +- packages/react-devtools-shared/src/backend/utils.js | 4 ++-- packages/react-devtools-shared/src/devtools/utils.js | 2 +- .../react-devtools-shared/src/devtools/views/utils.js | 2 +- packages/react-devtools-shared/src/hooks/astUtils.js | 2 +- .../react-devtools-shared/src/hooks/generateHookMap.js | 2 +- packages/react-devtools-shell/src/app/index.js | 2 +- .../src/TimelineSearchContext.js | 2 +- .../src/server/ReactDOMServerFormatConfig.js | 6 +++--- packages/react-reconciler/src/ReactFiberScope.js | 4 ++-- .../src/ReactFlightWebpackNodeLoader.js | 2 +- .../src/ReactFlightWebpackPlugin.js | 2 +- packages/react-server/src/ReactFizzServer.js | 10 +++++----- packages/react-server/src/ReactFlightServer.js | 6 +++--- packages/react/src/ReactChildren.js | 2 +- packages/scheduler/src/forks/SchedulerMock.js | 2 +- 18 files changed, 31 insertions(+), 31 deletions(-) diff --git a/packages/react-debug-tools/src/ReactDebugHooks.js b/packages/react-debug-tools/src/ReactDebugHooks.js index fbe4dc2f360a4..56c6e376dabb6 100644 --- a/packages/react-debug-tools/src/ReactDebugHooks.js +++ b/packages/react-debug-tools/src/ReactDebugHooks.js @@ -524,7 +524,7 @@ function buildTree( readHookLog: Array, includeHooksSource: boolean, ): HooksTree { - const rootChildren = []; + const rootChildren: Array = []; let prevStack = null; let levelChildren = rootChildren; let nativeHookID = 0; @@ -557,7 +557,7 @@ function buildTree( // The remaining part of the new stack are custom hooks. Push them // to the tree. for (let j = stack.length - commonSteps - 1; j >= 1; j--) { - const children = []; + const children: Array = []; const stackFrame = stack[j]; const levelChild: HooksNode = { id: null, diff --git a/packages/react-devtools-shared/src/backend/legacy/renderer.js b/packages/react-devtools-shared/src/backend/legacy/renderer.js index 1b667cf122185..69a193ecb08fc 100644 --- a/packages/react-devtools-shared/src/backend/legacy/renderer.js +++ b/packages/react-devtools-shared/src/backend/legacy/renderer.js @@ -785,7 +785,7 @@ export function attach( let owner = element._owner; if (owner) { - owners = []; + owners = ([]: Array); while (owner != null) { owners.push({ displayName: getData(owner).displayName || 'Unknown', @@ -807,8 +807,8 @@ export function attach( } // Not implemented - const errors = []; - const warnings = []; + const errors: Array<[string, number]> = []; + const warnings: Array<[string, number]> = []; return { id, diff --git a/packages/react-devtools-shared/src/backend/renderer.js b/packages/react-devtools-shared/src/backend/renderer.js index 54f95a2fbea6a..b09d303624fd4 100644 --- a/packages/react-devtools-shared/src/backend/renderer.js +++ b/packages/react-devtools-shared/src/backend/renderer.js @@ -3252,7 +3252,7 @@ export function attach( let owners = null; if (_debugOwner) { - owners = []; + owners = ([]: Array); let owner: null | Fiber = _debugOwner; while (owner !== null) { owners.push(fiberToSerializedElement(owner)); diff --git a/packages/react-devtools-shared/src/backend/utils.js b/packages/react-devtools-shared/src/backend/utils.js index 5c0a9d7800bc6..03e7800819af6 100644 --- a/packages/react-devtools-shared/src/backend/utils.js +++ b/packages/react-devtools-shared/src/backend/utils.js @@ -20,8 +20,8 @@ export function cleanForBridge( path?: Array = [], ): DehydratedData | null { if (data !== null) { - const cleanedPaths = []; - const unserializablePaths = []; + const cleanedPaths: Array> = []; + const unserializablePaths: Array> = []; const cleanedData = dehydrate( data, cleanedPaths, diff --git a/packages/react-devtools-shared/src/devtools/utils.js b/packages/react-devtools-shared/src/devtools/utils.js index 2e16d23c989bc..304483fd03f7f 100644 --- a/packages/react-devtools-shared/src/devtools/utils.js +++ b/packages/react-devtools-shared/src/devtools/utils.js @@ -198,7 +198,7 @@ const STACK_SOURCE_LOCATION = /([^\s]+) \((.+):(.+):(.+)\)/; export function stackToComponentSources( stack: string, ): Array<[string, ?Stack]> { - const out = []; + const out: Array<[string, ?Stack]> = []; stack .split(STACK_DELIMETER) .slice(1) diff --git a/packages/react-devtools-shared/src/devtools/views/utils.js b/packages/react-devtools-shared/src/devtools/views/utils.js index 9f7aeccea32d4..d75c21bf802b7 100644 --- a/packages/react-devtools-shared/src/devtools/views/utils.js +++ b/packages/react-devtools-shared/src/devtools/views/utils.js @@ -134,7 +134,7 @@ export function serializeDataForCopy(props: Object): string { export function serializeHooksForCopy(hooks: HooksTree | null): string { // $FlowFixMe "HooksTree is not an object" - const cloned = Object.assign([], hooks); + const cloned = Object.assign(([]: Array), hooks); const queue = [...cloned]; diff --git a/packages/react-devtools-shared/src/hooks/astUtils.js b/packages/react-devtools-shared/src/hooks/astUtils.js index 24cd0e5ea4611..80b8a9fa3e248 100644 --- a/packages/react-devtools-shared/src/hooks/astUtils.js +++ b/packages/react-devtools-shared/src/hooks/astUtils.js @@ -362,7 +362,7 @@ function getPotentialHookDeclarationsFromAST(sourceAST: File): NodePath[] { export function getHookNamesMappingFromAST( sourceAST: File, ): $ReadOnlyArray<{name: string, start: Position}> { - const hookStack = []; + const hookStack: Array<{name: string, start: any}> = []; const hookNames = []; const pushFrame = (name: string, node: Node) => { const nameInfo = {name, start: {...node.loc.start}}; diff --git a/packages/react-devtools-shared/src/hooks/generateHookMap.js b/packages/react-devtools-shared/src/hooks/generateHookMap.js index 8d88ecc4c682e..be81581c92745 100644 --- a/packages/react-devtools-shared/src/hooks/generateHookMap.js +++ b/packages/react-devtools-shared/src/hooks/generateHookMap.js @@ -63,7 +63,7 @@ export function generateHookMap(sourceAST: File): HookMap { const hookNamesMapping = getHookNamesMappingFromAST(sourceAST); const namesMap: Map = new Map(); const names = []; - const mappings = []; + const mappings: Array = []; let currentLine = null; hookNamesMapping.forEach(({name, start}) => { diff --git a/packages/react-devtools-shell/src/app/index.js b/packages/react-devtools-shell/src/app/index.js index 6568fa7d3f019..03f3f0c9c3391 100644 --- a/packages/react-devtools-shell/src/app/index.js +++ b/packages/react-devtools-shell/src/app/index.js @@ -33,7 +33,7 @@ ignoreErrors([ ignoreWarnings(['Warning: componentWillReceiveProps has been renamed']); ignoreLogs([]); -const unmountFunctions = []; +const unmountFunctions: Array<(() => void) | (() => boolean)> = []; function createContainer() { const container = document.createElement('div'); diff --git a/packages/react-devtools-timeline/src/TimelineSearchContext.js b/packages/react-devtools-timeline/src/TimelineSearchContext.js index 830f98f1c35f7..604dbf180607a 100644 --- a/packages/react-devtools-timeline/src/TimelineSearchContext.js +++ b/packages/react-devtools-timeline/src/TimelineSearchContext.js @@ -40,7 +40,7 @@ type Action = type Dispatch = (action: Action) => void; -const EMPTY_ARRAY = []; +const EMPTY_ARRAY: Array = []; function reducer(state: State, action: Action): State { let {searchIndex, searchRegExp, searchResults, searchText} = state; diff --git a/packages/react-dom-bindings/src/server/ReactDOMServerFormatConfig.js b/packages/react-dom-bindings/src/server/ReactDOMServerFormatConfig.js index 843c4bee1ff6d..921424e852e3f 100644 --- a/packages/react-dom-bindings/src/server/ReactDOMServerFormatConfig.js +++ b/packages/react-dom-bindings/src/server/ReactDOMServerFormatConfig.js @@ -191,7 +191,7 @@ export function createResponseState( : stringToPrecomputedChunk( '