Migrating codebase to ts (#40)

* refactor: moving to ts, wip resolving errors

* feat: resolve all ts errors

* refactor: fix test cases, remove babel

* build: set up tsc compiler properly

* refactor: fix eslint issue

* chore(package.json): update keywords

Co-authored-by: Valeriy Borodayev <[email protected]>

Author: borodayev

.babelrc.js

@@ -1,2 +1 @@
 lib/*
-flow-typed

.eslintignore

@@ -1,60 +1,29 @@
 module.exports = {
-  "extends": ["airbnb-base", "prettier"],
-  "parser": "babel-eslint",
-  "rules": {
-    "no-underscore-dangle": 0,
-    "arrow-body-style": 0,
-    "no-unused-expressions": 0,
-    "no-plusplus": 0,
-    "lines-between-class-members": 0,
-    "import/no-extraneous-dependencies": [
-      "error",
-      {
-        "devDependencies": [
-          "**/*-test.js",
-          "**/__mocks__/**",
-          "**/__fixtures__/**",
-          "**/demo/**"
-        ]
-      }
-    ],
-    "no-prototype-builtins": 0,
-    "no-restricted-syntax": 0,
-    "no-mixed-operators": 0,
-    "comma-dangle": [
-      "error",
-      {
-        "arrays": "always-multiline",
-        "objects": "always-multiline",
-        "imports": "always-multiline",
-        "exports": "always-multiline",
-        "functions": "ignore"
-      }
-    ],
-    "prettier/prettier": [
-      "error",
-      {
-        "printWidth": 100,
-        "singleQuote": true,
-        "trailingComma": "es5"
-      }
-    ],
-    "import/prefer-default-export": 0,
-    "arrow-parens": 0,
-    "prefer-destructuring": 0,
-    "no-use-before-define": 0
+  root: true,
+  parser: '@typescript-eslint/parser',
+  parserOptions: {
+    project: ['./tsconfig.eslint.json'],
   },
-  "env": {
-    "jasmine": true,
-    "jest": true
+  extends: [
+    'airbnb-typescript/base',
+    'prettier/@typescript-eslint',
+    'plugin:import/typescript',
+    'plugin:jest/all',
+    'plugin:prettier/recommended',
+  ],
+  plugins: [
+    '@typescript-eslint',
+    'jest',
+    'prettier',
+  ],
+  env: {
+    'jest/globals': true,
+    'jasmine': true,
+    'jest': true
   },
-  "plugins": ["flowtype", "prettier"],
-  "globals": {
-    "Class": true,
-    "Iterator": true,
-    "$Shape": true,
-    "$Keys": true,
-    "$FlowFixMe": true,
-    "fetch": true
+  rules: {
+    'prettier/prettier': ['error', { 'singleQuote': true }],
+    'lines-between-class-members': 0,
+    'no-underscore-dangle': 0,
   }
 }

.eslintrc.js

@@ -1,13 +1,13 @@
 // @flow
 /* eslint-disable no-param-reassign, func-names */
 
-import mongoose, { type MongooseSchema } from 'mongoose';
-import DiffPlugin from '../src/index';
+import mongoose, { Schema, Document, Model } from 'mongoose';
+import DiffPlugin, { IDiffModel } from '../src/index';
 import DB from './db';
 
 DB.init();
 
-export const PostSchema: MongooseSchema<PostDoc> = new mongoose.Schema(
+export const PostSchema: Schema<IPostDoc> = new mongoose.Schema(
   {
     title: {
       type: String,
@@ -34,15 +34,15 @@ export const PostSchema: MongooseSchema<PostDoc> = new mongoose.Schema(
   }
 );
 
-export class PostDoc /* :: extends Mongoose$Document */ {
+export interface IPostDoc extends Document {
   title: string;
   subjects: Array<{ name: string }>;
+}
 
-  // TODO: find out solution for flow to use `DiffModelT` instead of `any`
-  /* :: static diffModel(): any {}  */
+interface IPostModel extends Model<IPostDoc> {
+  diffModel(): IDiffModel;
 }
 
 PostSchema.plugin(DiffPlugin);
-PostSchema.loadClass(PostDoc);
 
-export const Post = DB.data.model('Post', PostSchema);
+export const Post = DB.data.model<IPostDoc, IPostModel>('Post', PostSchema);

.flowconfig

@@ -1,16 +1,13 @@
+/* eslint-disable no-param-reassign, jest/require-top-level-describe, jest/no-hooks */
 /* eslint-disable no-param-reassign */
-/* @flow */
 
-import MongoMemoryServer from 'mongodb-memory-server';
+import * as m from 'mongodb-memory-server';
 
-// $FlowFixMe
 const DB = require.requireActual('../db').default;
 
-const mongod = new MongoMemoryServer({
-  // debug: true,
-});
+const mongod = new m.MongoMemoryServer({});
 
-DB.autoIndexOnceInDev = opts => {
+DB.autoIndexOnceInDev = (opts: any) => {
   opts.config.autoIndex = false;
 };
 DB.consoleErr = () => {};

__fixtures__/Post.js renamed to __fixtures__/Post.ts

@@ -1,8 +1,6 @@
-// @flow
 /* eslint-disable no-console */
 
-import mongoose from 'mongoose';
-import type { MongooseConnection } from 'mongoose';
+import mongoose, { Connection } from 'mongoose';
 
 mongoose.Promise = global.Promise;
 
@@ -14,7 +12,7 @@ export default class DB {
   static consoleLog = console.log;
   static _connectionStr: string;
 
-  static data: MongooseConnection = mongoose.createConnection();
+  static data: Connection = mongoose.createConnection();
 
   static init(connectionStr?: string) {
     if (connectionStr) this._connectionStr = connectionStr;
@@ -25,24 +23,23 @@ export default class DB {
     return Promise.all([DB.closeDB('data')]);
   }
 
-  static openDB(name: DBNames = 'data'): Promise<MongooseConnection> {
+  static openDB(name: DBNames = 'data'): Promise<Connection> {
     return new Promise((resolve, reject) => {
       const uri = process.env.MONGO_CONNECTION_STRING || this._connectionStr;
-      const opts = {};
+      const opts: any = {};
 
       opts.promiseLibrary = global.Promise;
       opts.autoReconnect = true;
       opts.reconnectTries = Number.MAX_VALUE;
       opts.reconnectInterval = 1000;
       opts.useNewUrlParser = true;
 
-      // $FlowFixMe
       const db: any = DB[name];
 
       db.consoleErr = DB.consoleErr;
       db.consoleLog = DB.consoleLog;
 
-      db.on('error', e => {
+      db.on('error', (e: any) => {
         if (e.message.code === 'ETIMEDOUT') {
           db.consoleErr(Date.now(), e);
           db.connect(uri, opts);
@@ -64,7 +61,6 @@ export default class DB {
   }
 
   static closeDB(name: DBNames = 'data'): Promise<any> {
-    // $FlowFixMe
     if (DB[name]) {
       return DB[name].close();
     }