fix(typings): queryInterface.addIndex

types/lib/query-interface.d.ts

@@ -4,6 +4,7 @@ import { Promise } from './promise';
 import QueryTypes = require('./query-types');
 import { Sequelize, RetryOptions } from './sequelize';
 import { Transaction } from './transaction';
+import { SetRequired } from './../type-helpers/set-required';
 
 type BindOrReplacements = { [key: string]: unknown } | unknown[];
 type FieldMap = { [key: string]: string };
@@ -403,7 +404,7 @@ export class QueryInterface {
   ): Promise<void>;
   public addIndex(
     tableName: string,
-    options: QueryInterfaceIndexOptions & { fields: string[] },
+    options: SetRequired<QueryInterfaceIndexOptions, 'fields'>,
     rawTablename?: string
   ): Promise<void>;
 

types/test/query-interface.ts

@@ -145,6 +145,15 @@ queryInterface.addIndex('Person', ['firstname', 'lastname'], {
   type: 'UNIQUE',
 });
 
+queryInterface.addIndex('Foo', {
+  name: 'foo_a',
+  fields: [
+    { name: 'foo_b', order: 'DESC' },
+    'foo_c',
+    { name: 'foo_d', order: 'ASC', collate: 'foobar', length: 42 }
+  ],
+});
+
 queryInterface.removeIndex('Person', 'SuperDuperIndex');
 
 // or

types/type-helpers/set-required.d.ts

@@ -0,0 +1,16 @@
+/**
+ * Full credits to sindresorhus/type-fest
+ * 
+ * https://github.com/sindresorhus/type-fest/blob/v0.8.1/source/set-required.d.ts
+ * 
+ * Thank you!
+ */
+export type SetRequired<BaseType, Keys extends keyof BaseType = keyof BaseType> =
+	// Pick just the keys that are not required from the base type.
+	Pick<BaseType, Exclude<keyof BaseType, Keys>> &
+	// Pick the keys that should be required from the base type and make them required.
+	Required<Pick<BaseType, Keys>> extends
+	// If `InferredType` extends the previous, then for each key, use the inferred type key.
+	infer InferredType
+		? {[KeyType in keyof InferredType]: InferredType[KeyType]}
+		: never;