CLOUDP-286235: Improved tooling (ESLint/Prettier) and GH actions (#290)

Author: lovisaberggren

.github/workflows/code-health-tools.yml

@@ -55,6 +55,19 @@ jobs:
           sparse-checkout: |
             .github
             tools
+      - name: Setup Node
+        uses: actions/setup-node@v4
+        with:
+          node-version: '20.x'
+          cache: 'npm'
+      - name: Install npm dependencies
+        run: npm install
+      - name: Run Prettier
+        run: |
+          npm run format-check
+      - name: Run ESLint on JS files
+        run: |
+          npm run lint-js
       - name: Install Go
         uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed
         with:

.github/workflows/required-spec-validations.yml

@@ -26,13 +26,10 @@ jobs:
         - name: Setup Node
           uses: actions/setup-node@v4
           with:
-            node-version: '20.x'  
+            node-version: '20.x'
+            cache: 'npm'
         - name: Install npm dependencies
-          uses: nick-fields/retry@v3
-          with:
-            timeout_minutes: 10
-            max_attempts: 3
-            command: npm install
+          run: npm install
         - name: Download openapi-foas
           uses: actions/download-artifact@v4
           with:

.gitignore

@@ -13,7 +13,7 @@
 *.env
 
 /tools/postman/openapi
-/tools/postman/node_modules
+/node_modules
 
 # Tool generated files
 *.idea

.prettierignore

@@ -0,0 +1,6 @@
+# Ignore artifacts:
+*.md
+*.json
+*.yaml
+*.yml
+*.html

.prettierrc.json

@@ -0,0 +1,9 @@
+{
+  "trailingComma": "es5",
+  "tabWidth": 2,
+  "printWidth": 120,
+  "semi": true,
+  "singleQuote": true,
+  "bracketSpacing": true,
+  "bracketSameLine": false
+}

eslint.config.mjs

@@ -0,0 +1,17 @@
+import globals from 'globals';
+import pluginJs from '@eslint/js';
+
+/** @type {import('eslint').Linter.Config[]} */
+export default [
+  { languageOptions: { globals: globals.browser } },
+  pluginJs.configs.recommended,
+  {
+    languageOptions: {
+      ecmaVersion: 2022,
+      sourceType: 'module',
+    },
+  },
+  {
+    ignores: ['node-modules'],
+  },
+];