fix: update import path for Collection type definition and use generics

Ref: bde4671

Author: kgryte

lib/node_modules/@stdlib/utils/async/count-by/docs/types/index.d.ts

@@ -20,12 +20,12 @@
 
 /// <reference types="@stdlib/types"/>
 
-import { Collection } from '@stdlib/types/object';
+import { Collection } from '@stdlib/types/array';
 
 /**
 * Interface defining function options.
 */
-interface Options {
+interface Options<T, V> {
 	/**
 	* The maximum number of pending invocations at any one time.
 	*/
@@ -39,72 +39,82 @@ interface Options {
 	/**
 	* Execution context.
 	*/
-	thisArg?: any;
+	thisArg?: ThisParameterType<Indicator<T, V>>;
+}
+
+/**
+* Interface defining a results object.
+*/
+interface Results {
+	/**
+	* Results for an individual group.
+	*/
+	[ key: string ]: number;
 }
 
 /**
 * Callback invoked either upon processing all collection elements or upon encountering an error.
 */
-type DoneNullary = () => void;
+type Nullary = () => void;
 
 /**
 * Callback invoked either upon processing all collection elements or upon encountering an error.
 *
 * @param error - encountered error or null
 */
-type DoneUnary = ( error: Error | null ) => void;
+type Unary = ( error: Error | null ) => void;
 
 /**
 * Callback invoked either upon processing all collection elements or upon encountering an error.
 *
 * @param error - encountered error or null
 * @param result - counts
 */
-type DoneBinary = ( error: Error | null, result: any ) => void;
+type Binary = ( error: Error | null, result: Results ) => void;
 
 /**
 * Callback invoked either upon processing all collection elements or upon encountering an error.
 *
 * @param error - encountered error or null
 * @param result - counts
 */
-type DoneCallback = DoneNullary | DoneUnary | DoneBinary;
+type Callback = Nullary | Unary | Binary;
 
 /**
-* Callback function.
+* Callback function to invoke once the predicate function has finished processing a collection value.
 */
-type Nullary = () => void;
+type NullaryNext = () => void;
 
 /**
-* Callback function.
+* Callback function to invoke once the predicate function has finished processing a collection value.
 *
 * @param error - encountered error or null
 */
-type Unary = ( error: Error | null ) => void;
+type UnaryNext = ( error: Error | null ) => void;
 
 /**
-* Callback function.
+* Callback function to invoke once the predicate function has finished processing a collection value.
 *
 * @param error - encountered error or null
 * @param group - value group
 */
-type Binary = ( error: Error | null, group: string ) => void;
+type BinaryNext = ( error: Error | null, group: string | Symbol | number ) => void;
 
 /**
-* Callback function.
+* Callback function to invoke once the predicate function has finished processing a collection value.
 *
 * @param error - encountered error or null
 * @param group - value group
 */
-type Callback = Nullary | Unary | Binary;
+type Next = NullaryNext | UnaryNext | BinaryNext;
 
 /**
 * Checks whether an element in a collection passes a test.
 *
 * @param value - collection value
 * @param next - a callback to be invoked after processing a collection `value`
 */
-type BinaryIndicator = ( value: any, next: Callback ) => void;
+type BinaryIndicator<T, V> = ( this: V, value: T, next: Next ) => void;
 
 /**
 * Checks whether an element in a collection passes a test.
@@ -113,7 +123,7 @@ type BinaryIndicator = ( value: any, next: Callback ) => void;
 * @param index - collection index
 * @param next - a callback to be invoked after processing a collection `value`
 */
-type TernaryIndicator = ( value: any, index: number, next: Callback ) => void;
+type TernaryIndicator<T, V> = ( this: V, value: T, index: number, next: Next ) => void;
 
 /**
 * Checks whether an element in a collection passes a test.
@@ -123,7 +133,7 @@ type TernaryIndicator = ( value: any, index: number, next: Callback ) => void;
 * @param collection - input collection
 * @param next - a callback to be invoked after processing a collection `value`
 */
-type QuaternaryIndicator = ( value: any, index: number, collection: Collection, next: Callback ) => void; // tslint-disable-line max-line-length
+type QuaternaryIndicator<T, V> = ( this: V, value: T, index: number, collection: Collection<T>, next: Next ) => void;
 
 /**
 * Checks whether an element in a collection passes a test.
@@ -133,15 +143,15 @@ type QuaternaryIndicator = ( value: any, index: number, collection: Collection,
 * @param collection - input collection
 * @param next - a callback to be invoked after processing a collection `value`
 */
-type Indicator = Unary | BinaryIndicator | TernaryIndicator | QuaternaryIndicator; // tslint-disable-line max-line-length
+type Indicator<T, V> = BinaryIndicator<T, V> | TernaryIndicator<T, V> | QuaternaryIndicator<T, V>;
 
 /**
 * Invokes an indicator function for each element in a collection.
 *
 * @param collection - input collection
 * @param done - function to invoke upon completion
 */
-type FactoryFunction = ( collection: Collection, done: DoneCallback ) => void;
+type FactoryFunction<T> = ( collection: Collection<T>, done: Callback ) => void;
 
 /**
 * Interface for `countByAsync`.
@@ -154,7 +164,6 @@ interface CountByAsync {
 	*
 	* -   This function does **not** guarantee that execution is asynchronous. To do so, wrap the `done` callback in a function which either executes at the end of the current stack (e.g., `nextTick`) or during a subsequent turn of the event loop (e.g., `setImmediate`, `setTimeout`).
 	*
-	*
 	* @param collection - input collection
 	* @param options - function options
 	* @param options.thisArg - execution context
@@ -195,7 +204,7 @@ interface CountByAsync {
 	*
 	* countByAsync( files, indicator, done );
 	*/
-	( collection: Collection, options: Options, indicator: Indicator, done: DoneCallback ): void; // tslint-disable-line max-line-length
+	<T = unknown, V = unknown>( collection: Collection<T>, options: Options<T, V>, indicator: Indicator<T, V>, done: Callback ): void;
 
 	/**
 	* Groups values according to an indicator function and returns group counts.
@@ -204,7 +213,6 @@ interface CountByAsync {
 	*
 	* -   This function does **not** guarantee that execution is asynchronous. To do so, wrap the `done` callback in a function which either executes at the end of the current stack (e.g., `nextTick`) or during a subsequent turn of the event loop (e.g., `setImmediate`, `setTimeout`).
 	*
-	*
 	* @param collection - input collection
 	* @param indicator - indicator function specifying which group an element in the input collection belongs to
 	* @param done - function to invoke upon completion
@@ -241,7 +249,7 @@ interface CountByAsync {
 	*
 	* countByAsync( files, indicator, done );
 	*/
-	( collection: Collection, indicator: Indicator, done: DoneCallback ): void;
+	<T = unknown, V = unknown>( collection: Collection<T>, indicator: Indicator<T, V>, done: Callback ): void; // tslint:disable-line:no-unnecessary-generics
 
 	/**
 	* Returns a function for grouping values according to an indicator function and returns group counts.
@@ -250,7 +258,6 @@ interface CountByAsync {
 	*
 	* -   This function does **not** guarantee that execution is asynchronous. To do so, wrap the `done` callback in a function which either executes at the end of the current stack (e.g., `nextTick`) or during a subsequent turn of the event loop (e.g., `setImmediate`, `setTimeout`).
 	*
-	*
 	* @param options - function options
 	* @param options.thisArg - execution context
 	* @param options.limit - maximum number of pending invocations at any one time
@@ -300,7 +307,7 @@ interface CountByAsync {
 	* // Try to read each element in `files`:
 	* countByAsync( files, done );
 	*/
-	factory( options: Options, indicator: Indicator ): FactoryFunction;
+	factory<T = unknown, V = unknown>( options: Options<T, V>, indicator: Indicator<T, V> ): FactoryFunction<T>;
 
 	/**
 	* Returns a function for grouping values according to an indicator function and returns group counts.
@@ -309,7 +316,6 @@ interface CountByAsync {
 	*
 	* -   This function does **not** guarantee that execution is asynchronous. To do so, wrap the `done` callback in a function which either executes at the end of the current stack (e.g., `nextTick`) or during a subsequent turn of the event loop (e.g., `setImmediate`, `setTimeout`).
 	*
-	*
 	* @param options - function options
 	* @param options.thisArg - execution context
 	* @param options.limit - maximum number of pending invocations at any one time
@@ -355,7 +361,7 @@ interface CountByAsync {
 	* // Try to read each element in `files`:
 	* countByAsync( files, done );
 	*/
-	factory( indicator: Indicator ): FactoryFunction;
+	factory<T = unknown, V = unknown>( indicator: Indicator<T, V> ): FactoryFunction<T>; // tslint:disable-line:no-unnecessary-generics
 }
 
 /**
@@ -365,7 +371,6 @@ interface CountByAsync {
 *
 * -   This function does **not** guarantee that execution is asynchronous. To do so, wrap the `done` callback in a function which either executes at the end of the current stack (e.g., `nextTick`) or during a subsequent turn of the event loop (e.g., `setImmediate`, `setTimeout`).
 *
-*
 * @param collection - input collection
 * @param options - function options
 * @param options.thisArg - execution context

lib/node_modules/@stdlib/utils/async/count-by/docs/types/test.ts

@@ -106,8 +106,8 @@ const done = ( error: Error | null, result: any ) => {
 
 // Attached to main export is a `factory` method which returns a function...
 {
-	countByAsync.factory( indicator ); // $ExpectType FactoryFunction
-	countByAsync.factory( { 'series': true }, indicator ); // $ExpectType FactoryFunction
+	countByAsync.factory( indicator ); // $ExpectType FactoryFunction<number>
+	countByAsync.factory( { 'series': true }, indicator ); // $ExpectType FactoryFunction<number>
 }
 
 // The compiler throws an error if the `factory` method is provided an options argument which is not an object...