@depup/react-native 0.86.0-depup.49 → 0.87.0-depup.0

package/Libraries/Image/RCTImageLoader.mm

@@ -68,8 +68,8 @@ static NSError *addResponseHeadersToError(NSError *originalError, NSHTTPURLRespo
 @end
 
 @implementation RCTImageLoader {
-  std::atomic<BOOL> _isLoaderSetup;
-  std::mutex _loaderSetupLock;
+  std::atomic<BOOL> _didSetup;
+  std::mutex _setupLock;
   NSArray<id<RCTImageURLLoader>> * (^_loadersProvider)(RCTModuleRegistry *);
   NSArray<id<RCTImageDataDecoder>> * (^_decodersProvider)(RCTModuleRegistry *);
   NSArray<id<RCTImageURLLoader>> *_loaders;
@@ -82,7 +82,6 @@ static NSError *addResponseHeadersToError(NSError *originalError, NSHTTPURLRespo
   NSMutableArray *_pendingDecodes;
   NSInteger _scheduledDecodes;
   NSUInteger _activeBytes;
-  std::mutex _loadersMutex;
   __weak id<RCTImageRedirectProtocol> _redirectDelegate;
 }
 
@@ -108,7 +107,7 @@ RCT_EXPORT_MODULE()
 {
   if (self = [super init]) {
     _redirectDelegate = redirectDelegate;
-    _isLoaderSetup = NO;
+    _didSetup = NO;
   }
   return self;
 }
@@ -126,15 +125,56 @@ RCT_EXPORT_MODULE()
 
 - (void)setUp
 {
-  std::lock_guard<std::mutex> guard(_loaderSetupLock);
-  if (!_isLoaderSetup) {
+  std::lock_guard<std::mutex> guard(_setupLock);
+  if (!_didSetup) {
     // Set defaults
     _maxConcurrentLoadingTasks = _maxConcurrentLoadingTasks ?: 4;
     _maxConcurrentDecodingTasks = _maxConcurrentDecodingTasks ?: 2;
     _maxConcurrentDecodingBytes = _maxConcurrentDecodingBytes ?: 30 * 1024 * 1024; // 30MB
 
     _URLRequestQueue = dispatch_queue_create("com.facebook.react.ImageLoaderURLRequestQueue", DISPATCH_QUEUE_SERIAL);
-    _isLoaderSetup = YES;
+
+    // Get loaders, sorted in reverse priority order (highest priority first)
+    if (_loadersProvider != nil) {
+      _loaders = _loadersProvider(self.moduleRegistry);
+    } else {
+      RCTAssert(_bridge, @"Trying to find RCTImageURLLoaders and bridge not set.");
+      _loaders = [_bridge modulesConformingToProtocol:@protocol(RCTImageURLLoader)];
+    }
+    _loaders =
+        [_loaders sortedArrayUsingComparator:^NSComparisonResult(id<RCTImageURLLoader> a, id<RCTImageURLLoader> b) {
+          float priorityA = [a respondsToSelector:@selector(loaderPriority)] ? [a loaderPriority] : 0;
+          float priorityB = [b respondsToSelector:@selector(loaderPriority)] ? [b loaderPriority] : 0;
+          if (priorityA > priorityB) {
+            return NSOrderedAscending;
+          } else if (priorityA < priorityB) {
+            return NSOrderedDescending;
+          } else {
+            return NSOrderedSame;
+          }
+        }];
+
+    // Get decoders, sorted in reverse priority order (highest priority first)
+    if (_decodersProvider != nil) {
+      _decoders = _decodersProvider(self.moduleRegistry);
+    } else {
+      RCTAssert(_bridge, @"Trying to find RCTImageDataDecoders and bridge not set.");
+      _decoders = [_bridge modulesConformingToProtocol:@protocol(RCTImageDataDecoder)];
+    }
+    _decoders = [_decoders
+        sortedArrayUsingComparator:^NSComparisonResult(id<RCTImageDataDecoder> a, id<RCTImageDataDecoder> b) {
+          float priorityA = [a respondsToSelector:@selector(decoderPriority)] ? [a decoderPriority] : 0;
+          float priorityB = [b respondsToSelector:@selector(decoderPriority)] ? [b decoderPriority] : 0;
+          if (priorityA > priorityB) {
+            return NSOrderedAscending;
+          } else if (priorityA < priorityB) {
+            return NSOrderedDescending;
+          } else {
+            return NSOrderedSame;
+          }
+        }];
+
+    _didSetup = YES;
   }
 }
 
@@ -163,41 +203,17 @@ RCT_EXPORT_MODULE()
 
 - (id<RCTImageURLLoader>)imageURLLoaderForURL:(NSURL *)URL
 {
-  if (!_isLoaderSetup) {
+  if (!_didSetup) {
     [self setUp];
   }
 
-  if (!_loaders) {
-    std::unique_lock<std::mutex> guard(_loadersMutex);
-    if (!_loaders) {
-      // Get loaders, sorted in reverse priority order (highest priority first)
-      if (_loadersProvider) {
-        _loaders = _loadersProvider(self.moduleRegistry);
-      } else {
-        RCTAssert(_bridge, @"Trying to find RCTImageURLLoaders and bridge not set.");
-        _loaders = [_bridge modulesConformingToProtocol:@protocol(RCTImageURLLoader)];
-      }
-
-      _loaders =
-          [_loaders sortedArrayUsingComparator:^NSComparisonResult(id<RCTImageURLLoader> a, id<RCTImageURLLoader> b) {
-            float priorityA = [a respondsToSelector:@selector(loaderPriority)] ? [a loaderPriority] : 0;
-            float priorityB = [b respondsToSelector:@selector(loaderPriority)] ? [b loaderPriority] : 0;
-            if (priorityA > priorityB) {
-              return NSOrderedAscending;
-            } else if (priorityA < priorityB) {
-              return NSOrderedDescending;
-            } else {
-              return NSOrderedSame;
-            }
-          }];
-    }
-  }
+  NSArray<id<RCTImageURLLoader>> *loaders = _loaders;
 
   if (RCT_DEBUG) {
     // Check for handler conflicts
     float previousPriority = 0;
     id<RCTImageURLLoader> previousLoader = nil;
-    for (id<RCTImageURLLoader> loader in _loaders) {
+    for (id<RCTImageURLLoader> loader in loaders) {
       float priority = [loader respondsToSelector:@selector(loaderPriority)] ? [loader loaderPriority] : 0;
       if (previousLoader && priority < previousPriority) {
         return previousLoader;
@@ -224,7 +240,7 @@ RCT_EXPORT_MODULE()
   }
 
   // Normal code path
-  for (id<RCTImageURLLoader> loader in _loaders) {
+  for (id<RCTImageURLLoader> loader in loaders) {
     if ([loader canLoadImageURL:URL]) {
       return loader;
     }
@@ -236,39 +252,17 @@ RCT_EXPORT_MODULE()
 
 - (id<RCTImageDataDecoder>)imageDataDecoderForData:(NSData *)data
 {
-  if (!_isLoaderSetup) {
+  if (!_didSetup) {
     [self setUp];
   }
 
-  if (!_decoders) {
-    // Get decoders, sorted in reverse priority order (highest priority first)
-
-    if (_decodersProvider) {
-      _decoders = _decodersProvider(self.moduleRegistry);
-    } else {
-      RCTAssert(_bridge, @"Trying to find RCTImageDataDecoders and bridge not set.");
-      _decoders = [_bridge modulesConformingToProtocol:@protocol(RCTImageDataDecoder)];
-    }
-
-    _decoders = [_decoders
-        sortedArrayUsingComparator:^NSComparisonResult(id<RCTImageDataDecoder> a, id<RCTImageDataDecoder> b) {
-          float priorityA = [a respondsToSelector:@selector(decoderPriority)] ? [a decoderPriority] : 0;
-          float priorityB = [b respondsToSelector:@selector(decoderPriority)] ? [b decoderPriority] : 0;
-          if (priorityA > priorityB) {
-            return NSOrderedAscending;
-          } else if (priorityA < priorityB) {
-            return NSOrderedDescending;
-          } else {
-            return NSOrderedSame;
-          }
-        }];
-  }
+  NSArray<id<RCTImageDataDecoder>> *decoders = _decoders;
 
   if (RCT_DEBUG) {
     // Check for handler conflicts
     float previousPriority = 0;
     id<RCTImageDataDecoder> previousDecoder = nil;
-    for (id<RCTImageDataDecoder> decoder in _decoders) {
+    for (id<RCTImageDataDecoder> decoder in decoders) {
       float priority = [decoder respondsToSelector:@selector(decoderPriority)] ? [decoder decoderPriority] : 0;
       if (previousDecoder && priority < previousPriority) {
         return previousDecoder;
@@ -297,7 +291,7 @@ RCT_EXPORT_MODULE()
   }
 
   // Normal code path
-  for (id<RCTImageDataDecoder> decoder in _decoders) {
+  for (id<RCTImageDataDecoder> decoder in decoders) {
     if ([decoder canDecodeImageData:data]) {
       return decoder;
     }
@@ -621,7 +615,7 @@ static RCTImageLoaderCancellationBlock RCTLoadImageURLFromLoader(
   }
 
   // All access to URL cache must be serialized
-  if (!_isLoaderSetup) {
+  if (!_didSetup) {
     [self setUp];
   }
 
@@ -1040,7 +1034,7 @@ static RCTImageLoaderCancellationBlock RCTLoadImageURLFromLoader(
       });
     };
 
-    if (!_isLoaderSetup) {
+    if (!_didSetup) {
       [self setUp];
     }
     dispatch_async(_URLRequestQueue, ^{
@@ -1173,6 +1167,9 @@ static RCTImageLoaderCancellationBlock RCTLoadImageURLFromLoader(
     return NO;
   }
 
+  if (!_didSetup) {
+    [self setUp];
+  }
   for (id<RCTImageURLLoader> loader in _loaders) {
     // Don't use RCTImageURLLoader protocol for modules that already conform to
     // RCTURLRequestHandler as it's inefficient to decode an image and then

package/Libraries/Image/RCTImageStoreManager.h

@@ -11,8 +11,6 @@
 #import <React/RCTBridgeProxy.h>
 #import <React/RCTURLRequestHandler.h>
 
-RCT_EXTERN void RCTEnableImageStoreManagerStorageQueue(BOOL enabled);
-
 @interface RCTImageStoreManager : NSObject <RCTURLRequestHandler>
 
 /**

package/Libraries/Image/RelativeImageStub.js

@@ -13,7 +13,7 @@
 // This is a stub for flow to make it understand require('./icon.png')
 // See metro/src/Bundler/index.js
 
-const AssetRegistry = require('@react-native/assets-registry/registry');
+const {AssetRegistry} = require('../../src/private/assets/AssetRegistry');
 
 const RelativeImageStub = AssetRegistry.registerAsset({
   __packager_asset: true,

package/Libraries/Image/resolveAssetSource.js

@@ -16,10 +16,10 @@ import type {ImageSource} from './ImageSource';
 
 import SourceCode from '../NativeModules/specs/NativeSourceCode';
 
+const {AssetRegistry} = require('../../src/private/assets/AssetRegistry');
 const AssetSourceResolver: AssetSourceResolverT =
   require('./AssetSourceResolver').default;
 const {pickScale} = require('./AssetUtils');
-const AssetRegistry = require('@react-native/assets-registry/registry');
 
 type CustomSourceTransformer = (
   resolver: AssetSourceResolver,

package/Libraries/Interaction/PanResponder.js

@@ -25,101 +25,6 @@ const previousCentroidYOfTouchesChangedAfter =
 const currentCentroidX = TouchHistoryMath.currentCentroidX;
 const currentCentroidY = TouchHistoryMath.currentCentroidY;
 
-/**
- * `PanResponder` reconciles several touches into a single gesture. It makes
- * single-touch gestures resilient to extra touches, and can be used to
- * recognize simple multi-touch gestures.
- *
- * It provides a predictable wrapper of the responder handlers provided by the
- * [gesture responder system](docs/gesture-responder-system.html).
- * For each handler, it provides a new `gestureState` object alongside the
- * native event object:
- *
- * ```
- * onPanResponderMove: (event, gestureState) => {}
- * ```
- *
- * A native event is a synthetic touch event with the following form:
- *
- *  - `nativeEvent`
- *      + `changedTouches` - Array of all touch events that have changed since the last event
- *      + `identifier` - The ID of the touch
- *      + `locationX` - The X position of the touch, relative to the element
- *      + `locationY` - The Y position of the touch, relative to the element
- *      + `pageX` - The X position of the touch, relative to the root element
- *      + `pageY` - The Y position of the touch, relative to the root element
- *      + `target` - The node id of the element receiving the touch event
- *      + `timestamp` - A time identifier for the touch, useful for velocity calculation
- *      + `touches` - Array of all current touches on the screen
- *
- * A `gestureState` object has the following:
- *
- *  - `stateID` - ID of the gestureState- persisted as long as there at least
- *     one touch on screen
- *  - `moveX` - the latest screen coordinates of the recently-moved touch
- *  - `moveY` - the latest screen coordinates of the recently-moved touch
- *  - `x0` - the screen coordinates of the responder grant
- *  - `y0` - the screen coordinates of the responder grant
- *  - `dx` - accumulated distance of the gesture since the touch started
- *  - `dy` - accumulated distance of the gesture since the touch started
- *  - `vx` - current velocity of the gesture
- *  - `vy` - current velocity of the gesture
- *  - `numberActiveTouches` - Number of touches currently on screen
- *
- * ### Basic Usage
- *
- * ```
- *   componentWillMount: function() {
- *     this._panResponder = PanResponder.create({
- *       // Ask to be the responder:
- *       onStartShouldSetPanResponder: (evt, gestureState) => true,
- *       onStartShouldSetPanResponderCapture: (evt, gestureState) => true,
- *       onMoveShouldSetPanResponder: (evt, gestureState) => true,
- *       onMoveShouldSetPanResponderCapture: (evt, gestureState) => true,
- *
- *       onPanResponderGrant: (evt, gestureState) => {
- *         // The gesture has started. Show visual feedback so the user knows
- *         // what is happening!
- *
- *         // gestureState.d{x,y} will be set to zero now
- *       },
- *       onPanResponderMove: (evt, gestureState) => {
- *         // The most recent move distance is gestureState.move{X,Y}
- *
- *         // The accumulated gesture distance since becoming responder is
- *         // gestureState.d{x,y}
- *       },
- *       onPanResponderTerminationRequest: (evt, gestureState) => true,
- *       onPanResponderRelease: (evt, gestureState) => {
- *         // The user has released all touches while this view is the
- *         // responder. This typically means a gesture has succeeded
- *       },
- *       onPanResponderTerminate: (evt, gestureState) => {
- *         // Another component has become the responder, so this gesture
- *         // should be cancelled
- *       },
- *       onShouldBlockNativeResponder: (evt, gestureState) => {
- *         // Returns whether this component should block native components from becoming the JS
- *         // responder. Returns true by default. Is currently only supported on android.
- *         return true;
- *       },
- *     });
- *   },
- *
- *   render: function() {
- *     return (
- *       <View {...this._panResponder.panHandlers} />
- *     );
- *   },
- *
- * ```
- *
- * ### Working Example
- *
- * To see it in action, try the
- * [PanResponder example in RNTester](https://github.com/facebook/react-native/blob/HEAD/packages/rn-tester/js/examples/PanResponder/PanResponderExample.js)
- */
-
 export type PanResponderGestureState = {
   /**
    * ID of the gestureState - persisted as long as there at least one touch on screen
@@ -225,9 +130,106 @@ export type PanResponderCallbacks = Readonly<{
   onShouldBlockNativeResponder?: ?ActiveCallback,
 }>;
 
+/**
+ * Reconciles several touches into a single gesture. Makes single-touch
+ * gestures resilient to extra touches and can recognize basic multi-touch
+ * gestures.
+ *
+ * It provides a predictable wrapper of the responder handlers provided by the
+ * [gesture responder system](docs/gesture-responder-system.html).
+ * For each handler, it provides a new `gestureState` object alongside the
+ * native event object:
+ *
+ * ```
+ * onPanResponderMove: (event, gestureState) => {}
+ * ```
+ *
+ * A native event is a synthetic touch event with the following form:
+ *
+ *  - `nativeEvent`
+ *      + `changedTouches` - Array of all touch events that have changed since the last event
+ *      + `identifier` - The ID of the touch
+ *      + `locationX` - The X position of the touch, relative to the element
+ *      + `locationY` - The Y position of the touch, relative to the element
+ *      + `pageX` - The X position of the touch, relative to the root element
+ *      + `pageY` - The Y position of the touch, relative to the root element
+ *      + `target` - The node id of the element receiving the touch event
+ *      + `timestamp` - A time identifier for the touch, useful for velocity calculation
+ *      + `touches` - Array of all current touches on the screen
+ *
+ * A `gestureState` object has the following:
+ *
+ *  - `stateID` - ID of the gestureState- persisted as long as there at least
+ *     one touch on screen
+ *  - `moveX` - the latest screen coordinates of the recently-moved touch
+ *  - `moveY` - the latest screen coordinates of the recently-moved touch
+ *  - `x0` - the screen coordinates of the responder grant
+ *  - `y0` - the screen coordinates of the responder grant
+ *  - `dx` - accumulated distance of the gesture since the touch started
+ *  - `dy` - accumulated distance of the gesture since the touch started
+ *  - `vx` - current velocity of the gesture
+ *  - `vy` - current velocity of the gesture
+ *  - `numberActiveTouches` - Number of touches currently on screen
+ *
+ * ### Basic Usage
+ *
+ * Example:
+ *
+ * ```tsx
+ *   componentWillMount: function() {
+ *     this._panResponder = PanResponder.create({
+ *       // Ask to be the responder:
+ *       onStartShouldSetPanResponder: (evt, gestureState) => true,
+ *       onStartShouldSetPanResponderCapture: (evt, gestureState) => true,
+ *       onMoveShouldSetPanResponder: (evt, gestureState) => true,
+ *       onMoveShouldSetPanResponderCapture: (evt, gestureState) => true,
+ *
+ *       onPanResponderGrant: (evt, gestureState) => {
+ *         // The gesture has started. Show visual feedback so the user knows
+ *         // what is happening!
+ *
+ *         // gestureState.d{x,y} will be set to zero now
+ *       },
+ *       onPanResponderMove: (evt, gestureState) => {
+ *         // The most recent move distance is gestureState.move{X,Y}
+ *
+ *         // The accumulated gesture distance since becoming responder is
+ *         // gestureState.d{x,y}
+ *       },
+ *       onPanResponderTerminationRequest: (evt, gestureState) => true,
+ *       onPanResponderRelease: (evt, gestureState) => {
+ *         // The user has released all touches while this view is the
+ *         // responder. This typically means a gesture has succeeded
+ *       },
+ *       onPanResponderTerminate: (evt, gestureState) => {
+ *         // Another component has become the responder, so this gesture
+ *         // should be cancelled
+ *       },
+ *       onShouldBlockNativeResponder: (evt, gestureState) => {
+ *         // Returns whether this component should block native components from becoming the JS
+ *         // responder. Returns true by default. Is currently only supported on android.
+ *         return true;
+ *       },
+ *     });
+ *   },
+ *
+ *   render: function() {
+ *     return (
+ *       <View {...this._panResponder.panHandlers} />
+ *     );
+ *   },
+ *
+ * ```
+ *
+ * ### Working Example
+ *
+ * To see it in action, try the
+ * [PanResponder example in RNTester](https://github.com/facebook/react-native/blob/HEAD/packages/rn-tester/js/examples/PanResponder/PanResponderExample.js)
+ *
+ * @see https://reactnative.dev/docs/panresponder
+ */
 const PanResponder = {
   /**
-   *
    * A graphical explanation of the touch data flow:
    *
    * +----------------------------+             +--------------------------------+
@@ -366,11 +368,14 @@ const PanResponder = {
   },
 
   /**
+   * Create a `PanResponder` instance with a config of gesture callback
+   * handlers.
+   *
    * @param {object} config Enhanced versions of all of the responder callbacks
    * that provide not only the typical `ResponderSyntheticEvent`, but also the
    * `PanResponder` gesture state.  Simply replace the word `Responder` with
    * `PanResponder` in each of the typical `onResponder*` callbacks. For
-   * example, the `config` object would look like:
+   * example, the `config` object looks like:
    *
    *  - `onMoveShouldSetPanResponder: (e, gestureState) => {...}`
    *  - `onMoveShouldSetPanResponderCapture: (e, gestureState) => {...}`

package/Libraries/LayoutAnimation/LayoutAnimation.js

@@ -159,18 +159,19 @@ const Presets = {
 };
 
 /**
- * Automatically animates views to their new positions when the
- * next layout happens.
- *
- * A common way to use this API is to call it before calling `setState`.
+ * Automatically animates views to their new positions when the next layout
+ * happens. Call before updating state to animate the transition.
  *
  * Note that in order to get this to work on **Android** you need to set the following flags via `UIManager`:
  *
  *     UIManager.setLayoutAnimationEnabledExperimental && UIManager.setLayoutAnimationEnabledExperimental(true);
+ *
+ * @see https://reactnative.dev/docs/layoutanimation
  */
 const LayoutAnimation = {
   /**
-   * Schedules an animation to happen on the next layout.
+   * Schedule an animation for the next layout. Config includes duration,
+   * create/update/delete sub-configs.
    *
    * @param config Specifies animation properties:
    *
@@ -183,10 +184,16 @@ const LayoutAnimation = {
    * @param onError Called on error. Only supported on iOS.
    */
   configureNext,
+
   /**
-   * Helper for creating a config for `configureNext`.
+   * Helper that creates a config object for `configureNext`.
    */
   create: createLayoutAnimation,
+
+  /**
+   * Enum of animation types (spring, linear, easeInEaseOut, easeIn, easeOut,
+   * keyboard).
+   */
   Types: Object.freeze({
     spring: 'spring',
     linear: 'linear',
@@ -195,15 +202,24 @@ const LayoutAnimation = {
     easeOut: 'easeOut',
     keyboard: 'keyboard',
   }) as LayoutAnimationTypes,
+
+  /**
+   * Enum of layout properties to animate (opacity, scaleX, scaleY, scaleXY).
+   */
   Properties: Object.freeze({
     opacity: 'opacity',
     scaleX: 'scaleX',
     scaleY: 'scaleY',
     scaleXY: 'scaleXY',
   }) as LayoutAnimationProperties,
+
   checkConfig(...args: Array<unknown>) {
     console.error('LayoutAnimation.checkConfig(...) has been disabled.');
   },
+
+  /**
+   * Predefined configs (easeInEaseOut, linear, spring).
+   */
   Presets,
   easeInEaseOut: configureNext.bind(null, Presets.easeInEaseOut) as (
     onAnimationDidEnd?: OnAnimationDidEndCallback,

package/Libraries/Linking/Linking.js

@@ -27,10 +27,7 @@ class LinkingImpl extends NativeEventEmitter<LinkingEventDefinitions> {
   }
 
   /**
-   * Add a handler to Linking changes by listening to the `url` event type
-   * and providing the handler
-   *
-   * See https://reactnative.dev/docs/linking#addeventlistener
+   * Listen for incoming URL changes.
    */
   addEventListener<K extends keyof LinkingEventDefinitions>(
     eventType: K,
@@ -40,9 +37,14 @@ class LinkingImpl extends NativeEventEmitter<LinkingEventDefinitions> {
   }
 
   /**
-   * Try to open the given `url` with any of the installed apps.
+   * Open the given URL with any installed app that can handle it. This
+   * includes URLs such as locations (e.g. "geo:37.484847,-122.148386"),
+   * contacts, or any other URL that can be opened with installed apps.
    *
-   * See https://reactnative.dev/docs/linking#openurl
+   * This method will fail if the system doesn't know how to open the
+   * specified URL. If you're passing in a non-http(s) URL, it's best to
+   * check `canOpenURL` first. For web URLs, the protocol ("http://",
+   * "https://") must be set accordingly.
    */
   openURL(url: string): Promise<void> {
     this._validateURL(url);
@@ -54,9 +56,10 @@ class LinkingImpl extends NativeEventEmitter<LinkingEventDefinitions> {
   }
 
   /**
-   * Determine whether or not an installed app can handle a given URL.
-   *
-   * See https://reactnative.dev/docs/linking#canopenurl
+   * Check whether an installed app can handle a given URL. For web URLs,
+   * the protocol ("http://", "https://") must be set accordingly. As of
+   * iOS 9, your app needs to provide the `LSApplicationQueriesSchemes`
+   * key inside Info.plist.
    */
   canOpenURL(url: string): Promise<boolean> {
     this._validateURL(url);
@@ -68,9 +71,8 @@ class LinkingImpl extends NativeEventEmitter<LinkingEventDefinitions> {
   }
 
   /**
-   * Open app settings.
-   *
-   * See https://reactnative.dev/docs/linking#opensettings
+   * Open the device Settings app and display the app’s custom settings, if
+   * it has any.
    */
   openSettings(): Promise<void> {
     if (Platform.OS === 'android') {
@@ -81,10 +83,9 @@ class LinkingImpl extends NativeEventEmitter<LinkingEventDefinitions> {
   }
 
   /**
-   * If the app launch was triggered by an app link,
-   * it will give the link url, otherwise it will give `null`
-   *
-   * See https://reactnative.dev/docs/linking#getinitialurl
+   * Get the URL that launched the app, or `null` if it was not launched from
+   * a link. To support deep linking on Android, see
+   * https://developer.android.com/training/app-indexing/deep-linking.html#handling-intents.
    */
   getInitialURL(): Promise<?string> {
     return Platform.OS === 'android'
@@ -92,12 +93,12 @@ class LinkingImpl extends NativeEventEmitter<LinkingEventDefinitions> {
       : nullthrows(NativeLinkingManager).getInitialURL();
   }
 
-  /*
-   * Launch an Android intent with extras (optional)
+  /**
+   * Launch an Android intent with optional extras. Useful for deep-linking
+   * to settings pages, opening an SMS app with a message draft in place,
+   * and more. See https://developer.android.com/reference/kotlin/android/content/Intent.
    *
    * @platform android
-   *
-   * See https://reactnative.dev/docs/linking#sendintent
    */
   sendIntent(
     action: string,
@@ -126,9 +127,9 @@ class LinkingImpl extends NativeEventEmitter<LinkingEventDefinitions> {
 const Linking: LinkingImpl = new LinkingImpl();
 
 /**
- * `Linking` gives you a general interface to interact with both incoming
- * and outgoing app links.
+ * General interface to interact with both incoming and outgoing app links,
+ * including deep links and universal links.
  *
- * See https://reactnative.dev/docs/linking
+ * @see https://reactnative.dev/docs/linking
  */
 export default Linking;