Merge pull request #28 from Flagsmith/chore/npm-test-on-pr

chore: Add build and test workflow, fix npm audit issues

Author: kyle-ssg

.github/actions/install/action.yml

@@ -0,0 +1,14 @@
+name: "Clone and Install"
+description: "Clones repository and sets up Node environment"
+runs:
+  using: "composite"
+  steps:
+    - name: Set up Node
+      uses: actions/setup-node@v4
+      with:
+        node-version: '18.x'
+        registry-url: 'https://registry.npmjs.org'
+
+    - name: Install dependencies
+      run: npm install
+      shell: bash

.github/workflows/publish.yml

@@ -11,15 +11,12 @@ jobs:
     name: Publish NPM Package
 
     steps:
-      - name: Cloning repo
+      - name: Check out repository
         uses: actions/checkout@v4
 
-      - uses: actions/setup-node@v4
-        with:
-          node-version: '18.x'
-          registry-url: 'https://registry.npmjs.org'
+      - name: Clone and Install
+        uses: ./.github/actions/install
 
-      - run: npm i
       - run: npm run deploy
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

.github/workflows/pull-request.yml

@@ -0,0 +1,25 @@
+name: Build and Test
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  build_and_test:
+    concurrency:
+      group: ${{ github.ref }}
+      cancel-in-progress: true
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Check out repository
+        uses: actions/checkout@v4
+
+      - name: Clone and Install
+        uses: ./.github/actions/install
+
+      - name: Build
+        run: npm run build

package.json

@@ -17,29 +17,29 @@
     "/oclif.manifest.json"
   ],
   "dependencies": {
-    "@oclif/core": "^1.16.0",
-    "@oclif/plugin-help": "^5",
-    "@oclif/plugin-plugins": "^2.0.1",
+    "@oclif/core": "^1.26.2",
+    "@oclif/plugin-help": "^6.2.27",
+    "@oclif/plugin-plugins": "^5.4.36",
     "flagsmith": "^9.0.5",
     "node-fetch": "^2.6.6"
   },
   "devDependencies": {
     "@oclif/test": "^2.1.1",
     "@types/chai": "^4",
     "@types/mocha": "^9.0.0",
-    "@types/node": "^16.9.4",
-    "@types/node-fetch": "^2.6.2",
+    "@types/node": "^18.19.83",
+    "@types/node-fetch": "^2.6.6",
     "chai": "^4",
     "eslint": "^7.32.0",
     "eslint-config-oclif": "^4",
     "eslint-config-oclif-typescript": "^1.0.2",
     "globby": "^11",
-    "mocha": "^9",
-    "oclif": "^3",
+    "mocha": "^11.1.0",
+    "oclif": "^4.17.41",
     "shx": "^0.3.3",
     "ts-node": "^10.2.1",
     "tslib": "^2.3.1",
-    "typescript": "4.4.4"
+    "typescript": "5.8.2"
   },
   "oclif": {
     "bin": "flagsmith",

tsconfig.json

@@ -6,7 +6,8 @@
     "outDir": "dist",
     "rootDir": "src",
     "strict": true,
-    "target": "es2019"
+    "target": "es2019",
+    "skipLibCheck": true
   },
   "include": [
     "src/**/*"