packages/react-reconciler/src/ReactFiberCommitWork.old.js (36 lines): - line 347: // TODO: Should wrap this in flags check, too, as optimization - line 400: // TODO: Move this out of the hot path using a dedicated effect tag. - line 428: // TODO: revisit this when we implement resuming. - line 504: // TODO (effects) It would be nice to avoid calling doesFiberContain() - line 731: // TODO: revisit this when we implement resuming. - line 784: // TODO: revisit this when we implement resuming. - line 838: // TODO: I think this is now always non-null by the time it reaches the - line 873: // TODO: revisit this when we implement resuming. - line 879: // TODO: I think this is now always non-null by the time it reaches the - line 1005: // TODO: This is a temporary solution that allowed us to transition away - line 1020: // TODO (Offscreen) Check: flags & LayoutStatic - line 1277: // TODO: this is recursive. - line 1442: // TODO: Move to `commitPassiveUnmountInsideDeletedTreeOnFiber` instead. - line 1520: // TODO: Find a more efficient way to do this. - line 1830: // TODO: Check if we're inside an Offscreen subtree that disappeared - line 1946: // TODO: Type the updateQueue to be specific to host components. - line 2024: // TODO: Move this to passive phase - line 2110: // TODO: Use an effect tag. - line 2151: // TODO: Should wrap this in flags check, too, as optimization - line 2203: // TODO: The factoring of this phase could probably be improved. Consider - line 2245: // TODO: This is a temporary solution that allowed us to transition away - line 2262: // TODO: Move to passive phase - line 2276: // TODO: This needs to run whenever there's an insertion or update - line 2296: // TODO: Move re-appear call here for symmetry? - line 2315: // TODO: findDOMNode doesn't rely on this any more but isMounted does - line 2392: // TODO (Offscreen) Also check: subtreeFlags & LayoutMask - line 2481: // TODO (Offscreen) Check: flags & (RefStatic | LayoutStatic) - line 2504: // TODO (Offscreen) Check: flags & RefStatic - line 2530: // TODO (Offscreen) Check: subtreeFlags & LayoutStatic - line 2574: // TODO (Offscreen) Check: subtreeFlags & LayoutStatic - line 2590: // TODO (Offscreen) Check: flags & LayoutStatic - line 2893: // TODO: Check if fiber has a PassiveStatic flag - line 2899: // TODO: Only traverse subtree if it has a PassiveStatic flag. (But, if we - line 2978: // TODO: run passive unmount effects when unmounting a root. - line 3030: // TODO: Remove this assignment once we're confident that it won't break - line 3035: // TODO: Reuse reappearLayoutEffects traversal here? packages/react-reconciler/src/ReactFiberCommitWork.new.js (36 lines): - line 347: // TODO: Should wrap this in flags check, too, as optimization - line 400: // TODO: Move this out of the hot path using a dedicated effect tag. - line 428: // TODO: revisit this when we implement resuming. - line 504: // TODO (effects) It would be nice to avoid calling doesFiberContain() - line 731: // TODO: revisit this when we implement resuming. - line 784: // TODO: revisit this when we implement resuming. - line 838: // TODO: I think this is now always non-null by the time it reaches the - line 873: // TODO: revisit this when we implement resuming. - line 879: // TODO: I think this is now always non-null by the time it reaches the - line 1005: // TODO: This is a temporary solution that allowed us to transition away - line 1020: // TODO (Offscreen) Check: flags & LayoutStatic - line 1277: // TODO: this is recursive. - line 1442: // TODO: Move to `commitPassiveUnmountInsideDeletedTreeOnFiber` instead. - line 1520: // TODO: Find a more efficient way to do this. - line 1830: // TODO: Check if we're inside an Offscreen subtree that disappeared - line 1946: // TODO: Type the updateQueue to be specific to host components. - line 2024: // TODO: Move this to passive phase - line 2110: // TODO: Use an effect tag. - line 2151: // TODO: Should wrap this in flags check, too, as optimization - line 2203: // TODO: The factoring of this phase could probably be improved. Consider - line 2245: // TODO: This is a temporary solution that allowed us to transition away - line 2262: // TODO: Move to passive phase - line 2276: // TODO: This needs to run whenever there's an insertion or update - line 2296: // TODO: Move re-appear call here for symmetry? - line 2315: // TODO: findDOMNode doesn't rely on this any more but isMounted does - line 2392: // TODO (Offscreen) Also check: subtreeFlags & LayoutMask - line 2481: // TODO (Offscreen) Check: flags & (RefStatic | LayoutStatic) - line 2504: // TODO (Offscreen) Check: flags & RefStatic - line 2530: // TODO (Offscreen) Check: subtreeFlags & LayoutStatic - line 2574: // TODO (Offscreen) Check: subtreeFlags & LayoutStatic - line 2590: // TODO (Offscreen) Check: flags & LayoutStatic - line 2893: // TODO: Check if fiber has a PassiveStatic flag - line 2899: // TODO: Only traverse subtree if it has a PassiveStatic flag. (But, if we - line 2978: // TODO: run passive unmount effects when unmounting a root. - line 3030: // TODO: Remove this assignment once we're confident that it won't break - line 3035: // TODO: Reuse reappearLayoutEffects traversal here? packages/react-reconciler/src/ReactFiberWorkLoop.old.js (31 lines): - line 483: // TODO: Move this type conversion to the event priority module. - line 494: // TODO: Move this type conversion to the event priority module. - line 587: // TODO: Consolidate with `isInterleavedUpdate` check - line 608: // TODO: Make sure this doesn't override pings that happen while we've - line 704: // TODO: Optimize slightly by comparing to root that fiber belongs to. - line 765: // TODO: Temporary until we confirm this warning is not fired. - line 898: // TODO: We only check `didTimeout` defensively, to account for a Scheduler - line 942: // TODO: It's possible that even a concurrent render may never have yielded - line 1006: // TODO: I think what we should do is set ForceClientRender inside - line 1090: // FIXME: What if the suspended lanes are Idle? Should not restart. - line 1124: // TODO: Is the JND optimization worth the added complexity? If this is - line 1215: // TODO: Lol maybe there's a better way to factor this besides this - line 1500: // TODO: I found and added this missing line while investigating a - line 1516: // TODO: Consider calling `unwindWork` to pop the contexts. - line 1630: // TODO: We should probably always mark the root as suspended immediately - line 1958: // TODO: This no longer makes any sense. We already wrap the mutation and - line 1985: // TODO: Might be better if `flushPassiveEffects` did not automatically - line 2064: // TODO: Delete all other places that schedule the passive effect callback - line 2084: // TODO: This is left over from the effect list implementation, where we had - line 2189: // TODO: Maybe there's a better way to report this. - line 2213: // TODO: This is part of the `componentDidCatch` implementation. Its purpose - line 2272: // TODO: We can optimize this by not scheduling the callback earlier. Since we - line 2355: // TODO: Combine this check with the one in flushPassiveEFfectsImpl. We should - line 2413: // TODO: This is sometimes out of sync with rootWithPendingPassiveEffects. - line 2438: // TODO: Move to commitPassiveMountEffects - line 2471: // TODO: Move to commitPassiveMountEffects - line 2572: // TODO: Until we re-land skipUnmountedBoundaries (see #20147), this warning - line 2613: // TODO: If we're rendering sync either due to Sync, Batched or expired, - line 2645: // TODO: Assign this to `suspenseState.retryLane`? to avoid - line 2649: // TODO: Special case idle priority? - line 2767: // TODO (StrictEffects) Should we set a marker on the root if it contains strict effects packages/react-reconciler/src/ReactFiberWorkLoop.new.js (31 lines): - line 483: // TODO: Move this type conversion to the event priority module. - line 494: // TODO: Move this type conversion to the event priority module. - line 587: // TODO: Consolidate with `isInterleavedUpdate` check - line 608: // TODO: Make sure this doesn't override pings that happen while we've - line 704: // TODO: Optimize slightly by comparing to root that fiber belongs to. - line 765: // TODO: Temporary until we confirm this warning is not fired. - line 898: // TODO: We only check `didTimeout` defensively, to account for a Scheduler - line 942: // TODO: It's possible that even a concurrent render may never have yielded - line 1006: // TODO: I think what we should do is set ForceClientRender inside - line 1090: // FIXME: What if the suspended lanes are Idle? Should not restart. - line 1124: // TODO: Is the JND optimization worth the added complexity? If this is - line 1215: // TODO: Lol maybe there's a better way to factor this besides this - line 1500: // TODO: I found and added this missing line while investigating a - line 1516: // TODO: Consider calling `unwindWork` to pop the contexts. - line 1630: // TODO: We should probably always mark the root as suspended immediately - line 1958: // TODO: This no longer makes any sense. We already wrap the mutation and - line 1985: // TODO: Might be better if `flushPassiveEffects` did not automatically - line 2064: // TODO: Delete all other places that schedule the passive effect callback - line 2084: // TODO: This is left over from the effect list implementation, where we had - line 2189: // TODO: Maybe there's a better way to report this. - line 2213: // TODO: This is part of the `componentDidCatch` implementation. Its purpose - line 2272: // TODO: We can optimize this by not scheduling the callback earlier. Since we - line 2355: // TODO: Combine this check with the one in flushPassiveEFfectsImpl. We should - line 2413: // TODO: This is sometimes out of sync with rootWithPendingPassiveEffects. - line 2438: // TODO: Move to commitPassiveMountEffects - line 2471: // TODO: Move to commitPassiveMountEffects - line 2572: // TODO: Until we re-land skipUnmountedBoundaries (see #20147), this warning - line 2613: // TODO: If we're rendering sync either due to Sync, Batched or expired, - line 2645: // TODO: Assign this to `suspenseState.retryLane`? to avoid - line 2649: // TODO: Special case idle priority? - line 2767: // TODO (StrictEffects) Should we set a marker on the root if it contains strict effects packages/react-reconciler/src/ReactFiberBeginWork.new.js (22 lines): - line 359: // TODO: current can be non-null here even if the component - line 558: // TODO: current can be non-null here even if the component - line 613: // TODO: Move the reset at in beginWork out of the common path so that - line 779: // TODO: Optimize this to use the OffscreenComponent fiber instead of - line 1093: // TODO This way of resetting the error boundary state is a hack. - line 1241: // TODO: Warn in a future release. - line 1292: // TODO: Restructure so we never read values from the instance. - line 1345: // FIXME: Slipped past code review. This is not a safe mutation: - line 2018: // TODO: Probably should inline this back - line 2047: // TODO: Should not remove render lanes that were pinged during this render - line 2714: // TODO This time should be the time at which the server rendered response that is - line 2745: // TODO: When we delete legacy mode, we should make this error argument - line 2760: // TODO: The server should serialize the error message so we can log it - line 2773: // TODO: Factoring is a little weird, since we check this right below, too. - line 2803: // TODO: Ideally this would inherit the event time of the current render - line 3189: // TODO: We could fast path by just rendering the tail now. - line 3228: // TODO: If workInProgress.child is null, we can continue on the tail immediately. - line 3270: // TODO: Consider unifying this with how the root works. - line 3456: // TODO: Once we add back resuming, we should check if the children are - line 3696: // TODO: We use `childLanes` as a heuristic for whether there is - line 3757: // TODO: Probably should refactor `beginWork` to split the bailout - line 3863: // TODO: This assumes that we're about to evaluate the component and process packages/react-reconciler/src/ReactFiberBeginWork.old.js (22 lines): - line 359: // TODO: current can be non-null here even if the component - line 558: // TODO: current can be non-null here even if the component - line 613: // TODO: Move the reset at in beginWork out of the common path so that - line 779: // TODO: Optimize this to use the OffscreenComponent fiber instead of - line 1093: // TODO This way of resetting the error boundary state is a hack. - line 1241: // TODO: Warn in a future release. - line 1292: // TODO: Restructure so we never read values from the instance. - line 1345: // FIXME: Slipped past code review. This is not a safe mutation: - line 2018: // TODO: Probably should inline this back - line 2047: // TODO: Should not remove render lanes that were pinged during this render - line 2714: // TODO This time should be the time at which the server rendered response that is - line 2745: // TODO: When we delete legacy mode, we should make this error argument - line 2760: // TODO: The server should serialize the error message so we can log it - line 2773: // TODO: Factoring is a little weird, since we check this right below, too. - line 2803: // TODO: Ideally this would inherit the event time of the current render - line 3189: // TODO: We could fast path by just rendering the tail now. - line 3228: // TODO: If workInProgress.child is null, we can continue on the tail immediately. - line 3270: // TODO: Consider unifying this with how the root works. - line 3456: // TODO: Once we add back resuming, we should check if the children are - line 3696: // TODO: We use `childLanes` as a heuristic for whether there is - line 3757: // TODO: Probably should refactor `beginWork` to split the bailout - line 3863: // TODO: This assumes that we're about to evaluate the component and process fixtures/attribute-behavior/src/attributes.js (14 lines): - line 371: {name: 'contextMenu', read: getAttribute('contextmenu')}, // TODO: Read the property by rendering a menu with the ID. - line 740: {name: 'form', read: getAttribute('form')}, // TODO: Read the property by rendering into a form with i - line 879: // TODO: re-enable when we no longer compare to 15. - line 957: {name: 'inputMode', tagName: 'input', read: getAttribute('inputmode')}, // TODO: Should use property but it's not implemented in Chrome - line 969: read: getAttribute('is'), // TODO: This could check if this is an extended custom element but this is a controversial spec. - line 1099: {name: 'list', read: getAttribute('list')}, // TODO: This should match the ID of a datalist element and then read property. - line 1208: {name: 'mediaGroup', tagName: 'video', read: getAttribute('mediagroup')}, // TODO: Not yet implemented in Chrome. - line 1257: {name: 'on-click'}, // TODO: Check for event subscriptions - line 1258: {name: 'on-unknownevent'}, // TODO: Check for event subscriptions - line 1259: {name: 'onclick'}, // TODO: Check for event subscriptions - line 1260: {name: 'onClick'}, // TODO: Check for event subscriptions - line 1261: {name: 'onunknownevent'}, // TODO: Check for event subscriptions - line 1262: {name: 'onUnknownEvent'}, // TODO: Check for event subscriptions - line 1535: {name: 'results', tagName: 'input', read: getAttribute('results')}, // TODO: Should use property but it's not supported in Chrome. packages/react-reconciler/src/ReactChildFiber.old.js (13 lines): - line 110: // TODO: Clean this up once we turn on the string ref warning for - line 292: // TODO: For the shouldClone case, this could be micro-optimized a bit by - line 805: // TODO: This breaks on empty slots like null children. That's - line 823: // TODO: Move out of the loop. This only happens for the first run. - line 826: // TODO: Defer siblings if we're not at the right index for this slot. - line 856: // TODO: Move out of the loop. This only happens for the first run. - line 1007: // TODO: This breaks on empty slots like null children. That's - line 1025: // TODO: Move out of the loop. This only happens for the first run. - line 1028: // TODO: Defer siblings if we're not at the right index for this slot. - line 1058: // TODO: Move out of the loop. This only happens for the first run. - line 1152: // TODO: If key === null and child.key === null, then this only applies to - line 1230: // TODO: If key === null and child.key === null, then this only applies to - line 1308: // TODO: This function is supposed to be non-recursive. packages/react-reconciler/src/ReactFiberHooks.old.js (13 lines): - line 197: // TODO: Maybe there's some way to consolidate this with - line 401: // TODO Warn if no hooks are used at all during mount, then some are used during update. - line 567: // TODO: Don't need to reset the flags here, because they're reset in the - line 830: // TODO: Don't need to accumulate this. Instead, we can remove - line 956: // TODO: Not sure if this is the desired semantics, but it's what we - line 1342: // TODO: We can move this to the passive phase once we add a pre-commit - line 1818: // TODO: If deps are provided, should we skip comparing the ref itself? - line 1856: // TODO: If deps are provided, should we skip comparing the ref itself? - line 2063: // TODO: In Fizz, id generation is specific to each server config. Maybe we - line 2120: // TODO: Does Cache work in legacy mode? Should decide and write a test. - line 2121: // TODO: Consider warning if the refresh is at discrete priority, or if we - line 2135: // TODO: If a refresh never commits, the new cache created here must be - line 2157: // TODO: Warn if unmounted? packages/react-reconciler/src/ReactChildFiber.new.js (13 lines): - line 110: // TODO: Clean this up once we turn on the string ref warning for - line 292: // TODO: For the shouldClone case, this could be micro-optimized a bit by - line 805: // TODO: This breaks on empty slots like null children. That's - line 823: // TODO: Move out of the loop. This only happens for the first run. - line 826: // TODO: Defer siblings if we're not at the right index for this slot. - line 856: // TODO: Move out of the loop. This only happens for the first run. - line 1007: // TODO: This breaks on empty slots like null children. That's - line 1025: // TODO: Move out of the loop. This only happens for the first run. - line 1028: // TODO: Defer siblings if we're not at the right index for this slot. - line 1058: // TODO: Move out of the loop. This only happens for the first run. - line 1152: // TODO: If key === null and child.key === null, then this only applies to - line 1230: // TODO: If key === null and child.key === null, then this only applies to - line 1308: // TODO: This function is supposed to be non-recursive. packages/react-reconciler/src/ReactFiberHooks.new.js (13 lines): - line 197: // TODO: Maybe there's some way to consolidate this with - line 401: // TODO Warn if no hooks are used at all during mount, then some are used during update. - line 567: // TODO: Don't need to reset the flags here, because they're reset in the - line 830: // TODO: Don't need to accumulate this. Instead, we can remove - line 956: // TODO: Not sure if this is the desired semantics, but it's what we - line 1342: // TODO: We can move this to the passive phase once we add a pre-commit - line 1818: // TODO: If deps are provided, should we skip comparing the ref itself? - line 1856: // TODO: If deps are provided, should we skip comparing the ref itself? - line 2063: // TODO: In Fizz, id generation is specific to each server config. Maybe we - line 2120: // TODO: Does Cache work in legacy mode? Should decide and write a test. - line 2121: // TODO: Consider warning if the refresh is at discrete priority, or if we - line 2135: // TODO: If a refresh never commits, the new cache created here must be - line 2157: // TODO: Warn if unmounted? packages/react-reconciler/src/ReactFiberCompleteWork.old.js (11 lines): - line 194: // TODO: If we move the `hadNoMutationsEffects` call after `bubbleProperties` - line 272: // TODO: Split the update API as separate for the props vs. children. - line 276: // TODO: Experiencing an error where oldProps is null. Suggests a host - line 287: // TODO: Type this specific to this type of component. - line 954: // TODO: Move createInstance to beginWork and keep it on a context - line 960: // TODO: Move this and createInstance step into the beginPhase - line 1190: // TODO: This will still suspend a synchronous tree if anything - line 1194: // TODO: Move this back to throwException because this is too late - line 1226: // TODO: Move to passive phase - line 1236: // TODO: Move to passive phase - line 1468: // TODO: We can probably just avoid popping it instead and only packages/react-reconciler/src/ReactFiberCompleteWork.new.js (11 lines): - line 194: // TODO: If we move the `hadNoMutationsEffects` call after `bubbleProperties` - line 272: // TODO: Split the update API as separate for the props vs. children. - line 276: // TODO: Experiencing an error where oldProps is null. Suggests a host - line 287: // TODO: Type this specific to this type of component. - line 954: // TODO: Move createInstance to beginWork and keep it on a context - line 960: // TODO: Move this and createInstance step into the beginPhase - line 1190: // TODO: This will still suspend a synchronous tree if anything - line 1194: // TODO: Move this back to throwException because this is too late - line 1226: // TODO: Move to passive phase - line 1236: // TODO: Move to passive phase - line 1468: // TODO: We can probably just avoid popping it instead and only packages/eslint-plugin-react-hooks/src/ExhaustiveDeps.js (7 lines): - line 559: // TODO: we could potentially ignore early returns. - line 770: // TODO: also add an import? - line 772: // TODO: ideally we'd gather deps here but it would require - line 782: // TODO: What if the function needs to change on every render anyway? - line 785: // TODO: Why not report this at the dependency site? - line 1103: // TODO: consider preserving the comments or formatting? - line 1148: // TODO: Can this have a suggestion? scripts/rollup/bundles.js (6 lines): - line 105: // TODO: use on WWW. - line 402: global: 'ReactFlightDOMRelayServer', // TODO: Rename to Writer - line 417: global: 'ReactFlightDOMRelayClient', // TODO: Rename to Reader - line 432: global: 'ReactFlightNativeRelayServer', // TODO: Rename to Writer - line 448: global: 'ReactFlightNativeRelayClient', // TODO: Rename to Reader - line 927: // TODO: it's awkward to create a bundle for this but if we don't, the package packages/react-reconciler/src/ReactFiberLane.new.js (6 lines): - line 16: // TODO: Ideally these types would be opaque but that doesn't work well with - line 229: // TODO: Consider warning in this path if a fallback timer is not scheduled. - line 285: // TODO: Reconsider this. The counter-argument is that the partial work - line 370: // TODO: Retries should be allowed to expire if they are CPU bound for - line 396: // TODO: This gets called every time we yield. We can optimize by storing - line 584: // TODO: We really only need to unsuspend only lanes that are in the packages/react-reconciler/src/ReactFiberLane.old.js (6 lines): - line 16: // TODO: Ideally these types would be opaque but that doesn't work well with - line 229: // TODO: Consider warning in this path if a fallback timer is not scheduled. - line 285: // TODO: Reconsider this. The counter-argument is that the partial work - line 370: // TODO: Retries should be allowed to expire if they are CPU bound for - line 396: // TODO: This gets called every time we yield. We can optimize by storing - line 584: // TODO: We really only need to unsuspend only lanes that are in the fixtures/blocks/src/Router.js (5 lines): - line 18: // TODO: can't really import a server component on the client. - line 23: // TODO: use this for invalidation. - line 38: // TODO: cancel previous fetch? - line 63: // TODO: Here, There, and Everywhere. - line 64: // TODO: Instant Transitions, somehow. packages/react/index.modern.fb.js (4 lines): - line 21: SuspenseList as unstable_SuspenseList, // TODO: Remove once call sights updated to SuspenseList - line 34: startTransition as unstable_startTransition, // TODO: Remove once call sights updated to startTransition - line 48: useDeferredValue as unstable_useDeferredValue, // TODO: Remove once call sights updated to useDeferredValue - line 61: useTransition as unstable_useTransition, // TODO: Remove once call sights updated to useTransition scripts/rollup/build.js (4 lines): - line 116: // TODO: Remove object spread from the source. Prefer Object.assign instead. - line 122: // TODO: Remove for...of from the source. It requires a runtime to be embedded. - line 124: // TODO: Remove array spread from the source. Prefer .apply instead. - line 127: // TODO: Remove array destructuring from the source. Requires runtime. packages/react/index.classic.fb.js (4 lines): - line 21: SuspenseList as unstable_SuspenseList, // TODO: Remove once call sights updated to SuspenseList - line 35: startTransition as unstable_startTransition, // TODO: Remove once call sights updated to startTransition - line 49: useDeferredValue as unstable_useDeferredValue, // TODO: Remove once call sights updated to useDeferredValue - line 62: useTransition as unstable_useTransition, // TODO: Remove once call sights updated to useTransition packages/shared/ReactVersion.js (4 lines): - line 8: // TODO: this is special because it gets imported during build. - line 10: // TODO: 18.0.0 has not been released to NPM; - line 12: // When we next publish a release, update the matching TODO in backend/renderer.js - line 13: // TODO: This module is used both by the release scripts and to expose a version scripts/jest/TestFlags.js (3 lines): - line 39: // TODO: Should "experimental" also imply "modern"? Maybe we should - line 50: FIXME: false, - line 93: // TODO: Maybe we should error on conflicts? Or we could namespace packages/react-reconciler/src/ReactFiberHydrationContext.old.js (3 lines): - line 447: // TODO: Type this specific to this type of component. - line 487: // TODO: Delete this argument when we remove the legacy root API. - line 502: // TODO: Delete this argument when we remove the legacy root API. packages/react-reconciler/src/ReactFiberNewContext.old.js (3 lines): - line 205: // TODO: This path is only used by Cache components. Update - line 252: // TODO: Because we don't have a work-in-progress, this will add the - line 391: // TODO: Compare selected values to bail out early. packages/react-reconciler/src/ReactUpdateQueue.old.js (3 lines): - line 118: // TODO: Temporary field. Will remove this by storing a map of - line 497: // TODO: Pass `current` as argument - line 518: // TODO: Don't need to accumulate this. Instead, we can remove renderLanes packages/react-reconciler/src/ReactFiberNewContext.new.js (3 lines): - line 205: // TODO: This path is only used by Cache components. Update - line 252: // TODO: Because we don't have a work-in-progress, this will add the - line 391: // TODO: Compare selected values to bail out early. packages/react-reconciler/src/ReactFiberHydrationContext.new.js (3 lines): - line 447: // TODO: Type this specific to this type of component. - line 487: // TODO: Delete this argument when we remove the legacy root API. - line 502: // TODO: Delete this argument when we remove the legacy root API. scripts/error-codes/transform-error-messages.js (3 lines): - line 59: // /* FIXME (minify-errors-in-prod): Unminified error message in production build! */ - line 69: // TODO: Since this only detects one of many ways to disable a lint - line 90: '! FIXME (minify-errors-in-prod): Unminified error message in production build!' packages/react-reconciler/src/ReactUpdateQueue.new.js (3 lines): - line 118: // TODO: Temporary field. Will remove this by storing a map of - line 497: // TODO: Pass `current` as argument - line 518: // TODO: Don't need to accumulate this. Instead, we can remove renderLanes scripts/jest/setupTests.js (2 lines): - line 16: // TODO: Stop using spyOn in all the test since that seem deprecated. - line 69: // TODO: Consider consolidating this with `yieldValue`. In both cases, tests packages/react-reconciler/src/ReactFiberThrow.old.js (2 lines): - line 143: // TODO: Warn in strict mode if getDerivedStateFromError is - line 424: // TODO: I think we can remove this, since we now use `DidCapture` in packages/jest-react/src/internalAct.js (2 lines): - line 65: // TODO: This would be way simpler if 1) we required a promise to be - line 101: // TODO: Let's not support non-async scopes at all in our tests. Need to packages/shared/ReactFeatureFlags.js (2 lines): - line 40: // TODO: Finish rolling out in www - line 43: // TODO: Finish rolling out in www packages/react-reconciler/src/ReactFiberSyncTaskQueue.new.js (2 lines): - line 42: // TODO: There's only a single type of callback: performSyncOnWorkOnRoot. So - line 60: // TODO: Is this necessary anymore? The only user code that runs in this packages/react-reconciler/src/ReactFiberThrow.new.js (2 lines): - line 143: // TODO: Warn in strict mode if getDerivedStateFromError is - line 424: // TODO: I think we can remove this, since we now use `DidCapture` in scripts/release/shared-commands/download-build-artifacts.js (2 lines): - line 42: // TODO: Consider staging the release in a different directory from the CI - line 50: // TODO: Rename release channel to `next` packages/react-reconciler/src/ReactFiberSyncTaskQueue.old.js (2 lines): - line 42: // TODO: There's only a single type of callback: performSyncOnWorkOnRoot. So - line 60: // TODO: Is this necessary anymore? The only user code that runs in this packages/react/src/ReactElementValidator.js (2 lines): - line 341: // TODO: Drop this when these are no longer allowed as the type argument. - line 465: // TODO: Drop this when these are no longer allowed as the type argument. fixtures/blocks/src/server/ProfilePage.js (2 lines): - line 15: // TODO: Replace with asset reference. - line 18: // TODO: Router component? scripts/rollup/build-all-release-channels.js (2 lines): - line 231: // TODO: If the module is not in the version map, we should exclude it - line 258: // TODO: Remove this special case. packages/react-reconciler/src/ReactFiberReconciler.new.js (2 lines): - line 274: // TODO: Remove `callback` when we delete legacy mode. - line 299: // TODO: Move this to FiberRoot constructor fixtures/blocks/src/server/App.js (2 lines): - line 14: // TODO: Replace with asset reference. - line 17: // TODO: Router component? scripts/perf-counters/src/portability.h (2 lines): - line 34: // TODO: does clang define __GNUC__ ? - line 133: // TODO: FIXME! Without this implemented properly, the JIT packages/react-reconciler/src/ReactFiber.new.js (2 lines): - line 117: // TODO: Consider warning about bad polyfills - line 451: // TODO: Flip this flag and rename it something like "forceConcurrentByDefaultForTesting" scripts/rollup/validate/index.js (2 lines): - line 17: // TODO: Should we lint them? - line 26: // TODO: Should we we lint them? packages/react-reconciler/src/ReactFiber.old.js (2 lines): - line 117: // TODO: Consider warning about bad polyfills - line 451: // TODO: Flip this flag and rename it something like "forceConcurrentByDefaultForTesting" packages/react-reconciler/src/ReactFiberReconciler.old.js (2 lines): - line 274: // TODO: Remove `callback` when we delete legacy mode. - line 299: // TODO: Move this to FiberRoot constructor scripts/jest/jest-cli.js (2 lines): - line 13: // TODO: These configs are separate but should be rolled into the configs above - line 233: // TODO: We could build this if it hasn't been built yet. fixtures/legacy-jsx-runtimes/react-15/cjs/react-jsx-dev-runtime.development.js (1 line): - line 819: // TODO: Drop this when these are no longer allowed as the type argument. packages/shared/ReactErrorUtils.js (1 line): - line 59: * TODO: See if caughtError and rethrowError can be unified. fixtures/blocks/src/server/Comments.js (1 line): - line 12: // TODO: Replace with asset reference. scripts/release/publish-commands/print-follow-up-instructions.js (1 line): - line 39: // FIXME: New build script does not output build-info.json. It's only used packages/react-reconciler/src/ReactMutableSource.new.js (1 line): - line 101: // TODO Clear this data once all pending hydration work is finished. packages/react-reconciler/src/ReactFiberSuspenseComponent.old.js (1 line): - line 28: // TODO: Add "unstable_" prefix? packages/shared/forks/ReactFeatureFlags.native-fb-dynamic.js (1 line): - line 19: // TODO: __VARIANT__ isn't supported for React Native flags yet. You can set the packages/react/src/jsx/ReactJSXElementValidator.js (1 line): - line 355: // TODO: Drop this when these are no longer allowed as the type argument. packages/react/src/React.js (1 line): - line 70: // TODO: Move this branching into the other module instead and just re-export. fixtures/legacy-jsx-runtimes/react-15/cjs/react-jsx-runtime.development.js (1 line): - line 823: // TODO: Drop this when these are no longer allowed as the type argument. packages/react/src/ReactNoopUpdateQueue.js (1 line): - line 91: * properties which is confusing. TODO: Expose pendingState or don't use it fixtures/concurrent/time-slicing/src/index.js (1 line): - line 105: // TODO: useTransition hook instead. scripts/jest/setupHostConfigs.js (1 line): - line 18: // TODO: Fix tests to not use ReactDOM.render or gate them. Then we can remove this. packages/scheduler/src/forks/SchedulerMock.js (1 line): - line 18: // TODO: Use symbols? packages/react-reconciler/src/ReactFiberTransition.new.js (1 line): - line 61: // are originally created (TODO: except during refreshes, until we implement packages/react-reconciler/src/ReactFiberDevToolsHook.new.js (1 line): - line 15: // TODO: This import doesn't work because the DevTools depend on the DOM version of React packages/react-reconciler/src/ReactFiberSuspenseComponent.new.js (1 line): - line 28: // TODO: Add "unstable_" prefix? packages/react-reconciler/src/ReactFiberRoot.new.js (1 line): - line 136: // TODO: We have several of these arguments that are conceptually part of the packages/react/src/ReactHooks.js (1 line): - line 59: // TODO: add a more generic warning for invalid values. scripts/jest/setupTests.www.js (1 line): - line 13: // TODO: Many tests were written before we started running them against the packages/react-reconciler/src/ReactFiberTransitionPool.old.js (1 line): - line 61: // are originally created (TODO: except during refreshes, until we implement packages/react-reconciler/src/ReactFiberContext.new.js (1 line): - line 186: // TODO (bvaughn) Replace this behavior with an invariant() in the future. packages/react-reconciler/src/ReactFiberTransition.old.js (1 line): - line 61: // are originally created (TODO: except during refreshes, until we implement packages/react-reconciler/src/ReactFiberHotReloading.old.js (1 line): - line 184: // TODO: if it was but can no longer be simple, packages/eslint-plugin-react-hooks/npm/index.js (1 line): - line 3: // TODO: this doesn't make sense for an ESLint rule. packages/react-reconciler/src/ReactFiberDevToolsHook.old.js (1 line): - line 15: // TODO: This import doesn't work because the DevTools depend on the DOM version of React packages/react-reconciler/src/ReactMutableSource.old.js (1 line): - line 101: // TODO Clear this data once all pending hydration work is finished. packages/react-reconciler/src/ReactFiberRoot.old.js (1 line): - line 136: // TODO: We have several of these arguments that are conceptually part of the packages/shared/forks/ReactFeatureFlags.www-dynamic.js (1 line): - line 54: // TODO: These flags are hard-coded to the default values used in open source. fixtures/legacy-jsx-runtimes/react-17/cjs/react-jsx-dev-runtime.development.js (1 line): - line 1156: // TODO: Drop this when these are no longer allowed as the type argument. scripts/shared/inlinedHostConfigs.js (1 line): - line 83: isFlowTyped: false, // TODO: type it. fixtures/legacy-jsx-runtimes/setupTests.js (1 line): - line 181: // TODO Decide whether we need to support nested toWarn* expectations. packages/shared/ReactTypes.js (1 line): - line 92: // TODO: figure out the API for cross-renderer implementation. packages/react-reconciler/src/ReactFiberContext.old.js (1 line): - line 186: // TODO (bvaughn) Replace this behavior with an invariant() in the future. packages/react-reconciler/src/ReactFiberTreeContext.new.js (1 line): - line 72: // TODO: Use the unified fiber stack module instead of this local one? packages/scheduler/src/forks/Scheduler.js (1 line): - line 23: // TODO: Use symbols? fixtures/blocks/src/server/Post.js (1 line): - line 13: // TODO: Replace with asset reference. packages/react-reconciler/src/ReactFiberAct.old.js (1 line): - line 49: // TODO: Include link to relevant documentation page. packages/react-reconciler/src/ReactFiberClassComponent.new.js (1 line): - line 1212: // TODO: In some cases, we'll end up checking if context has changed twice, fixtures/legacy-jsx-runtimes/react-17/cjs/react-jsx-runtime.development.js (1 line): - line 1156: // TODO: Drop this when these are no longer allowed as the type argument. fixtures/devtools/standalone/index.html (1 line): - line 5: TODO List packages/react-reconciler/src/ReactFiberAct.new.js (1 line): - line 49: // TODO: Include link to relevant documentation page. packages/react-reconciler/src/ReactFiberHotReloading.new.js (1 line): - line 184: // TODO: if it was but can no longer be simple, scripts/jest/matchers/toWarnDev.js (1 line): - line 147: // TODO Decide whether we need to support nested toWarn* expectations. packages/react/src/ReactChildren.js (1 line): - line 45: * TODO: Test that a single child and an array with one item have the same key packages/react-reconciler/src/ReactFiberTreeContext.old.js (1 line): - line 72: // TODO: Use the unified fiber stack module instead of this local one? fixtures/legacy-jsx-runtimes/react-14/cjs/react-jsx-runtime.development.js (1 line): - line 818: // TODO: Drop this when these are no longer allowed as the type argument. packages/shared/forks/consoleWithStackDev.www.js (1 line): - line 49: // TODO: don't ignore level and pass it down somewhere too. packages/react/src/BadMapPolyfill.js (1 line): - line 20: // TODO: Consider warning about bad polyfills fixtures/legacy-jsx-runtimes/react-14/cjs/react-jsx-dev-runtime.development.js (1 line): - line 814: // TODO: Drop this when these are no longer allowed as the type argument. scripts/flow/runFlow.js (1 line): - line 14: // TODO: This generates all the renderer configs at once. Originally this was packages/react-reconciler/src/ReactFiberUnwindWork.old.js (1 line): - line 106: // TODO: popHydrationState scripts/perf-counters/src/perf-counters.cpp (1 line): - line 17: // TODO: Allow customizing recorded events scripts/perf-counters/src/jsc-perf.cpp (1 line): - line 87: // TODO: Allow customizing recorded events packages/react/src/ReactContext.js (1 line): - line 15: // TODO: Second argument used to be an optional `calculateChangedBits` scripts/rollup/validate/eslintrc.umd.js (1 line): - line 21: // TODO: this is too permissive. fixtures/legacy-jsx-runtimes/react-16/cjs/react-jsx-runtime.development.js (1 line): - line 846: // TODO: Drop this when these are no longer allowed as the type argument. packages/scheduler/src/SchedulerPriorities.js (1 line): - line 12: // TODO: Use symbols? packages/use-sync-external-store/src/useSyncExternalStoreShimClient.js (1 line): - line 111: // TODO: Because there is no cross-renderer API for batching updates, it's fixtures/blocks/src/client/Shell.js (1 line): - line 11: // TODO: Error Boundaries. fixtures/dom/src/toWarnDev.js (1 line): - line 162: // TODO Decide whether we need to support nested toWarn* expectations. fixtures/legacy-jsx-runtimes/react-16/cjs/react-jsx-dev-runtime.development.js (1 line): - line 842: // TODO: Drop this when these are no longer allowed as the type argument. packages/react-reconciler/src/ReactFiberClassComponent.old.js (1 line): - line 1212: // TODO: In some cases, we'll end up checking if context has changed twice, packages/shared/forks/ReactFeatureFlags.www.js (1 line): - line 78: // TODO: www currently relies on this feature. It's disabled in open source. packages/react-reconciler/src/ReactFiberUnwindWork.new.js (1 line): - line 106: // TODO: popHydrationState fixtures/blocks/src/client/ProfileNav.js (1 line): - line 14: // TODO: Don't hardcode ID. scripts/release/shared-commands/get-build-id-for-commit.js (1 line): - line 50: // TODO: The location of the retry loop is a bit weird. We should scripts/jest/preprocessor.js (1 line): - line 39: // TODO: I have not verified that this actually works.