feat: support typescript linting and type checking (#474)

* feat: support typescript eslint

* feat: run tsc --noEmit on typescript projects

* chore: remove unused babel-eslint package

config/eslint-react.config.js

@@ -1,5 +1,5 @@
 module.exports = {
-    parser: 'babel-eslint',
+    parser: '@typescript-eslint/parser',
 
     plugins: ['react-hooks'],
 

config/eslint.config.js

@@ -24,6 +24,19 @@ module.exports = {
         },
         sourceType: 'module',
     },
+    // Limit TypeScript linting to TS/TSX
+    overrides: [
+        {
+            files: ['**/*.{ts,tsx}'],
+            extends: [
+                'plugin:@typescript-eslint/recommended',
+                'plugin:import/typescript',
+            ],
+            rules: {
+                'import/extensions': 'off',
+            },
+        },
+    ],
 
     rules: {
         'max-params': [

package.json

@@ -30,12 +30,13 @@
         "@commitlint/config-conventional": "^13.1.0",
         "@dhis2/cli-helpers-engine": "^3.0.0",
         "@ls-lint/ls-lint": "^1.10.0",
-        "babel-eslint": "^10.1.0",
-        "eslint": "^7.32.0",
-        "eslint-config-prettier": "^8.3.0",
+        "@typescript-eslint/eslint-plugin": "^8.9.0",
+        "@typescript-eslint/parser": "^8.9.0",
+        "eslint": "^8",
+        "eslint-config-prettier": "^9.1.0",
         "eslint-plugin-import": "^2.22.1",
-        "eslint-plugin-react": "^7.31.10",
-        "eslint-plugin-react-hooks": "^4.6.2",
+        "eslint-plugin-react": "^7.37.1",
+        "eslint-plugin-react-hooks": "^5.0.0",
         "fast-glob": "^3.2.5",
         "find-up": "^5.0.0",
         "fs-extra": "^10.0.0",
@@ -49,6 +50,7 @@
         "semver": "^7.3.5",
         "stylelint": "^16.3.1",
         "stylelint-use-logical": "^2.1.2",
+        "typescript": "^5.6.3",
         "yargs": "^16.2.0"
     },
     "publishConfig": {

src/commands/types/javascript.js

@@ -4,6 +4,7 @@ const { exit } = require('@dhis2/cli-helpers-engine')
 const { eslint } = require('../../tools/eslint.js')
 const { prettier } = require('../../tools/prettier.js')
 const { stylelint } = require('../../tools/stylelint.js')
+const { typescriptCheck } = require('../../tools/typescript-check.js')
 const { configured } = require('../../utils/config.js')
 const { selectFiles } = require('../../utils/files.js')
 const {
@@ -52,8 +53,22 @@ exports.handler = (argv, callback) => {
         return
     }
 
+    const isTypescriptProject = configured('typescript')
+
+    if (isTypescriptProject) {
+        log.info('typescript > tsc --noEmit')
+        typescriptCheck({ callback: finalStatus })
+
+        if (finalStatus() === 0) {
+            log.print('Typescript check completed successfully.')
+            log.print('')
+        }
+    }
+
     if (configured('eslint')) {
-        log.info('javascript > eslint')
+        log.info(
+            `${isTypescriptProject ? 'typescript' : 'javascript'} > eslint`
+        )
         eslint({
             apply,
             files: jsFiles,

src/tools/typescript-check.js

@@ -0,0 +1,8 @@
+const { bin } = require('@dhis2/cli-helpers-engine').exec
+const { PROJECT_ROOT } = require('../utils/paths.js')
+
+exports.typescriptCheck = ({ callback }) => {
+    const cmd = 'tsc'
+
+    bin(cmd, { args: ['--noEmit'], PROJECT_ROOT }, callback)
+}

src/utils/config.js

@@ -32,6 +32,7 @@ const packageConfigs = {
  * project.
  */
 const projectConfigs = {
+    typescript: path.join(PROJECT_ROOT, 'tsconfig.json'),
     editorconfig: path.join(PROJECT_ROOT, '.editorconfig'),
     eslint: path.join(PROJECT_ROOT, '.eslintrc.js'),
     prettier: path.join(PROJECT_ROOT, '.prettierrc.js'),