[sequelize] should use commonjs export format, not ES Module

Sequelize uses commonjs export format. The libdef here uses the ES Module format.

Importing as follows:

const Sequelize = require('sequelize');

Doesn’t work in Flow (though the code runs just fine), you have to import as:

const Sequelize = require('sequelize').default;

Luckily, Sequelize manually added a default prop to their module.exports, so the 2nd version works.

However, it’d be better if the types were congruent with the actual export format.

[retyui]

I was mistaken and eventually found a solution

[jedwards1211]

@zeorin another reason I think using ES module declarations is a better tradeoff – since a transpiler is required to strip flow type annotations (unless you put annotations in comments), most devs are probably already able to use import statements, and probably prefer to since they look better next to import type statements, and they can mix in import Sequelize, {type FindOptions} from 'sequelize'.