feat(FEC-13371): Refactor: Migration from 'flow' to Type Script' (#225)

Refactor: Migration to TypeScript'

solves FEC-13371

Related PRs
kaltura/kaltura-player-js#661
kaltura/playkit-js#726
kaltura/playkit-js-hls#213
kaltura/playkit-js-dash#254
kaltura/playkit-js-ui#815
kaltura/playkit-js-browserslist-config#1

Co-authored-by: JonathanTGold <jonathan.gold@[email protected]>

.eslintrc.json

@@ -1,26 +1,60 @@
 {
-  "parser": "babel-eslint",
-  "extends": ["eslint:recommended", "plugin:flowtype/recommended"],
-  "plugins": ["prettier", "import", "flowtype", "mocha-no-only"],
-  "env": {
-    "browser": true,
-    "es6": true,
-    "mocha": true,
-    "amd": true,
-    "commonjs": true
+  "root": true,
+  "parser": "@typescript-eslint/parser",
+  "extends": ["eslint:recommended", "plugin:@typescript-eslint/recommended"],
+  "plugins": ["@typescript-eslint"],
+  "rules": {
+    "indent": ["error", 2],
+    "react/prefer-stateless-function": "off",
+    "max-len": ["warn", { "code": 500 }],
+    "eol-last": "off",
+    "@typescript-eslint/explicit-function-return-type": "warn",
+    "@typescript-eslint/no-non-null-assertion": "off",
+    "@typescript-eslint/no-unused-vars": "error",
+    //    "@typescript-eslint/no-explicit-any": "warn",
+    "@typescript-eslint/no-explicit-any": "off",
+    "@typescript-eslint/explicit-member-accessibility": [
+      "error",
+      {
+        "accessibility": "explicit",
+        "overrides": {
+          "accessors": "explicit",
+          "constructors": "no-public",
+          "methods": "explicit",
+          "properties": "explicit",
+          "parameterProperties": "explicit"
+        }
+      }
+    ],
+    "block-scoped-var": "error",
+    "eqeqeq": "error",
+    "no-var": "error",
+    "no-console": "error",
+    "prefer-const": "error",
+    "prefer-arrow-callback": "error",
+    "no-trailing-spaces": "error",
+    "quotes": ["warn", "single", { "avoidEscape": true }],
+    "no-restricted-properties": [
+      "error",
+      {
+        "object": "describe",
+        "property": "only"
+      },
+      {
+        "object": "it",
+        "property": "only"
+      }
+    ]
   },
-  "globals": {
-    "should": true,
-    "sinon": true,
-    "__VERSION__": true,
-    "__NAME__": true,
-    "process": true,
-    "__dirname": true
+  "overrides": [],
+  "settings": {
+    "jest": {
+      "version": 26
+    }
   },
-  "rules": {
-    "prettier/prettier": "error",
-    "mocha-no-only/mocha-no-only": "off",
-    "require-jsdoc": ["error"],
-    "valid-jsdoc": ["error"]
+  "env": {
+    "browser": true,
+    "commonjs": true,
+    "es6": true
   }
 }

.github/workflows/run_canary.yaml

@@ -13,5 +13,6 @@ jobs:
     uses: kaltura/playkit-js-common/.github/workflows/canary_dependency.yaml@master
     secrets: inherit
     with:
-      schema-type: "playerV3Versions"
-      tests-yarn-run-to-execute: 'build eslint flow test'
+      node-version: '20.x'
+      schema-type: 'playerV3Versions'
+      tests-yarn-run-to-execute: 'build lint type-check test'

.github/workflows/run_prod.yaml

@@ -18,7 +18,8 @@ jobs:
       PLAYER_MSTEAMS_WEBHOOK: ${{ secrets.PLAYER_MSTEAMS_WEBHOOK }}
       PLAYER_GITHUB_BOT_TOKEN: ${{ secrets.PLAYER_GITHUB_BOT_TOKEN }}
     with:
-      type: "dependency"
-      env: "prod"
-      schema-type: "playerV3Versions"
-      tests-yarn-run-to-execute: 'build eslint flow test'
+      node-version: '20.x'
+      type: 'dependency'
+      env: 'prod'
+      schema-type: 'playerV3Versions'
+      tests-yarn-run-to-execute: 'build lint type-check test'

.github/workflows/run_tests.yaml

@@ -5,20 +5,41 @@ run-name: Player And Plugin Tests
 on:
   pull_request:
     branches:
-      - "*"
+      - '*'
 
 jobs:
-  running-tests:
+  build:
     uses: kaltura/ovp-pipelines-pub/.github/workflows/player_tests.yaml@master
     with:
-      yarn-run-to-execute: 'build eslint flow test'
+      node-version: '20.x'
+      yarn-run-to-execute: 'build'
+  test:
+    uses: kaltura/ovp-pipelines-pub/.github/workflows/player_tests.yaml@master
+    with:
+      node-version: '20.x'
+      yarn-run-to-execute: 'test'
+  type-check:
+    uses: kaltura/ovp-pipelines-pub/.github/workflows/player_tests.yaml@master
+    with:
+      node-version: '20.x'
+      yarn-run-to-execute: 'type-check'
+  build-types:
+    uses: kaltura/ovp-pipelines-pub/.github/workflows/player_tests.yaml@master
+    with:
+      node-version: '20.x'
+      yarn-run-to-execute: 'build:types'
+  lint:
+    uses: kaltura/ovp-pipelines-pub/.github/workflows/player_tests.yaml@master
+    with:
+      node-version: '20.x'
+      yarn-run-to-execute: 'lint'
   notification:
     if: always()
     uses: kaltura/ovp-pipelines-pub/.github/workflows/notification.yaml@master
-    needs: running-tests
+    needs: [build, test, type-check, lint]
     secrets:
       PLAYER_MSTEAMS_WEBHOOK: ${{ secrets.PLAYER_MSTEAMS_WEBHOOK }}
     with:
       failure-status: ${{ contains(needs.*.result, 'failure') }}
       cancelled-status: ${{ contains(needs.*.result, 'cancelled') }}
-      is-test: 'true'
+      is-test: 'true'

.gitignore

@@ -1,41 +1,9 @@
-# Logs
-logs
-*.log
-npm-debug.log*
-
-# Runtime data
-pids
-*.pid
-*.seed
-
-# Directory for instrumented libs generated by jscoverage/JSCover
-lib-cov
-
-# Coverage directory used by tools like istanbul
-coverage
-
-# nyc test coverage
-.nyc_output
-
-# Grunt intermediate storage (http://gruntjs.com/creating-plugins#storing-task-files)
-.grunt
-
-# node-waf configuration
-.lock-wscript
-
-# Compiled binary addons (http://nodejs.org/api/addons.html)
-build/Release
-
-# Dependency directories
 node_modules
-jspm_packages
-
-# Optional npm cache directory
-.npm
-
-# Optional REPL history
-.node_repl_history
-
+dist
+coverage
 .idea
-
-/dist/
+*.log
+lib
+api-extractor/report/
+api-extractor/report-temp/
+api-extractor/playkit-js-providers.api.json

api-extractor.json

@@ -0,0 +1,37 @@
+{
+  "$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
+  "mainEntryPointFilePath": "<projectFolder>/lib/types/index.d.ts",
+  "bundledPackages": [],
+  "compiler": {},
+  "apiReport": {
+    "enabled": true,
+    "reportFolder": "<projectFolder>/api-extractor/report",
+    "reportTempFolder": "<projectFolder>/api-extractor/report-temp"
+  },
+  "docModel": {
+    "enabled": true,
+    "apiJsonFilePath": "<projectFolder>/api-extractor/<unscopedPackageName>.api.json"
+  },
+  "dtsRollup": {
+    "enabled": true,
+    "untrimmedFilePath": "<projectFolder>/dist/index.d.ts"
+  },
+  "tsdocMetadata": {},
+  "messages": {
+    "compilerMessageReporting": {
+      "default": {
+        "logLevel": "warning"
+      }
+    },
+    "extractorMessageReporting": {
+      "default": {
+        "logLevel": "warning"
+      }
+    },
+    "tsdocMessageReporting": {
+      "default": {
+        "logLevel": "warning"
+      }
+    }
+  }
+}