Add undefined to optional properties, Regenerator to Rimraf (DefinitelyTyped#54353)

In preparation for exactOptionalPropertyTypes in Typescript 4.4, add undefined to all optional properties. #no-publishing-comment

This PR covers widely used packages, those with more then 100,000 packages per month, from regenerator -- rimraf.

microsoft/dtslint#335

Author: sandersn

types/regenerator-runtime/index.d.ts

@@ -33,10 +33,10 @@ export interface CompletionRecord {
 
 export interface TryEntry {
     readonly tryLoc: number;
-    readonly catchLoc?: number;
-    readonly finallyLoc?: number;
-    readonly afterLoc?: ContextLocation;
-    completion?: CompletionRecord;
+    readonly catchLoc?: number | undefined;
+    readonly finallyLoc?: number | undefined;
+    readonly afterLoc?: ContextLocation | undefined;
+    completion?: CompletionRecord | undefined;
 }
 
 export interface DelegatedIterator {

types/registry-auth-token/index.d.ts

@@ -17,7 +17,7 @@ declare namespace auth {
          * Wether or not url's path parts are recursively trimmed from the registry
          * url when searching for tokens
          */
-        recursive?: boolean;
+        recursive?: boolean | undefined;
         /**
          * An npmrc configuration object used when searching for tokens. If no object is provided,
          * the `.npmrc` file at the base of the project is used.
@@ -26,12 +26,12 @@ declare namespace auth {
             /**
              * A registry url used for matching
              */
-            registry?: string;
+            registry?: string | undefined;
             /**
              * Registry url's with token information
              */
             [registryUrls: string]: string | undefined;
-        };
+        } | undefined;
     }
     /**
      * The generated authentication information
@@ -48,11 +48,11 @@ declare namespace auth {
         /**
          * The username used in `Basic`
          */
-        username?: string;
+        username?: string | undefined;
         /**
          * The password used in `Basic`
          */
-        password?: string;
+        password?: string | undefined;
     }
 }
 

types/regression/index.d.ts

@@ -14,13 +14,13 @@ export interface Options {
      * This is used to round the calculated fitting coefficients,
      * the output predictions, and the value of r^2.
      */
-    precision?: number;
+    precision?: number | undefined;
     /**
      * The number of terms to solve for (and therefore
      * the number of coefficients to calculate). Only
      * relevant for polynomial fitting.
      */
-    order?: number;
+    order?: number | undefined;
 }
 
 export interface Result {

types/relateurl/index.d.ts

@@ -12,23 +12,23 @@ declare namespace RelateUrl {
          *
          * Extend the list with any ports you need. Any URLs containing these default ports will have them removed. Example: http://example.com:80/ will become http://example.com/.
          */
-        defaultPorts?: Object;
+        defaultPorts?: Object | undefined;
 
         /**
          * Type: Array
          * Default value: ["index.html"]
          *
          * Extend the list with any resources you need. Works with options.removeDirectoryIndexes.
          */
-        directoryIndexes?: Array<string>;
+        directoryIndexes?: Array<string> | undefined;
 
         /**
          * Type: Boolean
          * Default value: false
          *
          * This will, for example, consider any domains containing http://www.example.com/ to be related to any that contain http://example.com/.
          */
-        ignore_www?: boolean;
+        ignore_www?: boolean | undefined;
 
         /**
          * Type: constant or String
@@ -41,71 +41,71 @@ declare namespace RelateUrl {
          * RelateUrl.ROOT_RELATIVE will produce something like /child-of-root/etc/.
          * RelateUrl.SHORTEST will choose whichever is shortest between root- and path-relative.
          */
-        output?: string;
+        output?: string | undefined;
 
         /**
          * Type: Array
          * Default value: ["data","javascript","mailto"]
          *
          * Extend the list with any additional schemes. Example: javascript:something will not be modified.
          */
-        rejectedSchemes?: Array<string>;
+        rejectedSchemes?: Array<string> | undefined;
 
         /**
          * Type: Boolean
          * Default value: false
          *
          * Remove user authentication information from the output URL.
          */
-        removeAuth?: boolean;
+        removeAuth?: boolean | undefined;
 
         /**
          * Type: Boolean
          * Default value: true
          *
          * Remove any resources that match any found in options.directoryIndexes.
          */
-        removeDirectoryIndexes?: boolean;
+        removeDirectoryIndexes?: boolean | undefined;
 
         /**
          * Type: Boolean
          * Default value: false
          *
          * Remove empty query variables. Example: http://domain.com/?var1&var2=&var3=asdf will become http://domain.com/?var3=adsf. This does not apply to unrelated URLs (with other protocols, auths, hosts and/or ports).
          */
-        removeEmptyQueries?: boolean;
+        removeEmptyQueries?: boolean | undefined;
 
         /**
          * Type: Boolean
          * Default value: true
          *
          * Remove trailing slashes from root paths. Example: http://domain.com/?var will become http://domain.com?var while http://domain.com/dir/?var will not be modified.
          */
-        removeRootTrailingSlash?: boolean;
+        removeRootTrailingSlash?: boolean | undefined;
 
         /**
          * Type: Boolean
          * Default value: true
          *
          * Output URLs relative to the scheme. Example: http://example.com/ will become //example.com/.
          */
-        schemeRelative?: boolean;
+        schemeRelative?: boolean | undefined;
 
         /**
          * Type: String
          * Default value: undefined
          *
          * An options-based version of the from argument. If both are specified, from takes priority.
          */
-        site?: string;
+        site?: string | undefined;
 
         /**
          * Type: Boolean
          * Default value: true
          *
          * Passed to Node's url.parse.
          */
-        slashesDenoteHost?: boolean;
+        slashesDenoteHost?: boolean | undefined;
     }
 }
 

types/relaxed-json/index.d.ts

@@ -7,10 +7,10 @@ export type Reviver = (this: {}, key: string, value: any) => any;
 export function transform(text: string): string;
 export function parse(text: string, reviver: Reviver): {};
 export function parse(text: string, opts?: {
-    reviver?: Reviver,
-    relaxed?: boolean,
-    warnings?: boolean,
-    tolerant?: boolean,
-    duplicate?: boolean,
+    reviver?: Reviver | undefined,
+    relaxed?: boolean | undefined,
+    warnings?: boolean | undefined,
+    tolerant?: boolean | undefined,
+    duplicate?: boolean | undefined,
 }): {};
 export function stringify(obj: any): string;

types/relay-runtime/lib/handlers/connection/ConnectionHandler.d.ts

@@ -11,7 +11,7 @@ export interface ConnectionMetadata {
     direction: string | null | undefined; // 'forward' | 'backward' | 'bidirectional' | null | undefined;
     cursor: string | null | undefined;
     count: string | null | undefined;
-    stream?: boolean;
+    stream?: boolean | undefined;
 }
 
 export function buildConnectionEdge(

types/relay-runtime/lib/mutations/RelayDeclarativeMutationConfig.d.ts

@@ -13,36 +13,36 @@ export type RangeBehaviors = RangeBehaviorsFunction | RangeBehaviorsObject;
 
 export interface RangeAddConfig {
     type: 'RANGE_ADD';
-    parentName?: string;
-    parentID?: string;
+    parentName?: string | undefined;
+    parentID?: string | undefined;
     connectionInfo?: ReadonlyArray<{
         key: string;
-        filters?: Variables;
+        filters?: Variables | undefined;
         rangeBehavior: string;
-    }>;
-    connectionName?: string;
+    }> | undefined;
+    connectionName?: string | undefined;
     edgeName: string;
-    rangeBehaviors?: RangeBehaviors;
+    rangeBehaviors?: RangeBehaviors | undefined;
 }
 
 export interface RangeDeleteConfig {
     type: 'RANGE_DELETE';
-    parentName?: string;
-    parentID?: string;
+    parentName?: string | undefined;
+    parentID?: string | undefined;
     connectionKeys?: ReadonlyArray<{
         key: string;
-        filters?: Variables;
-    }>;
-    connectionName?: string;
+        filters?: Variables | undefined;
+    }> | undefined;
+    connectionName?: string | undefined;
     deletedIDFieldName: string | ReadonlyArray<string>;
     pathToConnection: ReadonlyArray<string>;
 }
 
 export interface NodeDeleteConfig {
     type: 'NODE_DELETE';
-    parentName?: string;
-    parentID?: string;
-    connectionName?: string;
+    parentName?: string | undefined;
+    parentID?: string | undefined;
+    connectionName?: string | undefined;
     deletedIDFieldName: string;
 }
 

types/relay-runtime/lib/mutations/applyOptimisticMutation.d.ts

@@ -4,11 +4,11 @@ import { GraphQLTaggedNode } from '../query/RelayModernGraphQLTag';
 import { SelectorStoreUpdater, Environment } from '../store/RelayStoreTypes';
 
 export interface OptimisticMutationConfig {
-    configs?: ReadonlyArray<DeclarativeMutationConfig> | null;
+    configs?: ReadonlyArray<DeclarativeMutationConfig> | null | undefined;
     mutation: GraphQLTaggedNode;
     variables: Variables;
-    optimisticUpdater?: SelectorStoreUpdater | null;
-    optimisticResponse?: object;
+    optimisticUpdater?: SelectorStoreUpdater | null | undefined;
+    optimisticResponse?: object | undefined;
 }
 
 /**

types/relay-runtime/lib/mutations/commitMutation.d.ts

@@ -7,22 +7,22 @@ import { Environment, SelectorStoreUpdater } from '../store/RelayStoreTypes';
 export interface MutationParameters {
     readonly response: {};
     readonly variables: {};
-    readonly rawResponse?: {};
+    readonly rawResponse?: {} | undefined;
 }
 
 export interface MutationConfig<TOperation extends MutationParameters> {
-    configs?: DeclarativeMutationConfig[];
-    cacheConfig?: CacheConfig;
+    configs?: DeclarativeMutationConfig[] | undefined;
+    cacheConfig?: CacheConfig | undefined;
     mutation: GraphQLTaggedNode;
-    onError?: ((error: Error) => void) | null;
+    onError?: ((error: Error) => void) | null | undefined;
     onCompleted?:
         | ((response: TOperation['response'], errors: ReadonlyArray<PayloadError> | null | undefined) => void)
-        | null;
-    onUnsubscribe?: () => void | null | undefined;
-    optimisticResponse?: TOperation['response'];
-    optimisticUpdater?: SelectorStoreUpdater<TOperation['response']> | null;
-    updater?: SelectorStoreUpdater<TOperation['response']> | null;
-    uploadables?: UploadableMap | null;
+        | null | undefined;
+    onUnsubscribe?: (() => void | null | undefined) | undefined;
+    optimisticResponse?: TOperation['response'] | undefined;
+    optimisticUpdater?: SelectorStoreUpdater<TOperation['response']> | null | undefined;
+    updater?: SelectorStoreUpdater<TOperation['response']> | null | undefined;
+    uploadables?: UploadableMap | null | undefined;
     variables: TOperation['variables'];
 }
 

types/relay-runtime/lib/network/RelayNetworkTypes.d.ts

@@ -20,8 +20,8 @@ export interface PayloadError {
     locations?: Array<{
         line: number;
         column: number;
-    }>;
-    severity?: 'CRITICAL' | 'ERROR' | 'WARNING'; // Not officially part of the spec, but used at Facebook
+    }> | undefined;
+    severity?: 'CRITICAL' | 'ERROR' | 'WARNING' | undefined; // Not officially part of the spec, but used at Facebook
 }
 
 export interface PayloadExtensions {
@@ -34,17 +34,17 @@ export interface PayloadExtensions {
  */
 export interface GraphQLResponseWithData {
     data: PayloadData;
-    errors?: PayloadError[];
-    extensions?: PayloadExtensions;
-    label?: string;
-    path?: Array<string | number>;
+    errors?: PayloadError[] | undefined;
+    extensions?: PayloadExtensions | undefined;
+    label?: string | undefined;
+    path?: Array<string | number> | undefined;
 }
 export interface GraphQLResponseWithoutData {
-    data?: PayloadData;
+    data?: PayloadData | undefined;
     errors: PayloadError[];
-    extensions?: PayloadExtensions;
-    label?: string;
-    path?: Array<string | number>;
+    extensions?: PayloadExtensions | undefined;
+    label?: string | undefined;
+    path?: Array<string | number> | undefined;
 }
 export interface GraphQLResponseWithExtensionsOnly {
     // Per https://spec.graphql.org/June2018/#sec-Errors
@@ -91,9 +91,9 @@ export type FetchFunction = (
 ) => ObservableFromValue<GraphQLResponse>;
 
 export interface LegacyObserver<T> {
-    onCompleted?: () => void;
-    onError?: (error: Error) => void;
-    onNext?: (data: T) => void;
+    onCompleted?: (() => void) | undefined;
+    onError?: ((error: Error) => void) | undefined;
+    onNext?: ((data: T) => void) | undefined;
 }
 
 /**