Merge pull request #13 from Countly/ar2rsawseen/main

Update publish workflow

Author: ar2rsawseen

.github/workflows/npm-publish.yml

@@ -4,6 +4,9 @@ on:
     tags:
       - '*'
   workflow_dispatch:
+permissions:
+  id-token: write  # Required for OIDC
+  contents: read
 jobs:
   publish:
     runs-on: ubuntu-latest
@@ -15,11 +18,11 @@ jobs:
         with:
           node-version: '20'
           registry-url: 'https://registry.npmjs.org/'
+      - name: Update npm
+        run: npm install -g npm@latest
       - name: Install dependencies
         run: npm ci
       - name: Build
         run: npm run build
       - name: Publish to npm
         run: npm publish --access public
-        env:
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

.github/workflows/test.yml

@@ -8,6 +8,27 @@ on:
   workflow_dispatch:
 
 jobs:
+  lint:
+    runs-on: ubuntu-latest
+    
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+        
+      - name: Set up Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: '20'
+          cache: 'npm'
+          
+      - name: Install dependencies
+        run: npm ci
+        
+      - name: Run ESLint
+        run: npm run lint
+        
+      - name: Run TypeScript compiler check
+        run: npm run build
   test:
     runs-on: ubuntu-latest
     steps:

.husky/pre-commit

@@ -0,0 +1 @@
+npx lint-staged

.vscode/extensions.json

@@ -0,0 +1,6 @@
+{
+    "recommendations": [
+        "dbaeumer.vscode-eslint",
+        "esbenp.prettier-eslint"
+    ]
+}

.vscode/settings.json

@@ -0,0 +1,28 @@
+{
+    // Enable ESLint
+    "eslint.enable": true,
+    // Validate these file types with ESLint
+    "eslint.validate": [
+        "javascript",
+        "javascriptreact",
+        "typescript",
+        "typescriptreact"
+    ],
+    // Format on save
+    "editor.formatOnSave": true,
+    // Use ESLint to fix issues on save
+    "editor.codeActionsOnSave": {
+        "source.fixAll.eslint": "explicit"
+    },
+    // Set default formatter for TypeScript files
+    "[typescript]": {
+        "editor.defaultFormatter": "dbaeumer.vscode-eslint"
+    },
+    "[javascript]": {
+        "editor.defaultFormatter": "dbaeumer.vscode-eslint"
+    },
+    // Optional: Auto-save files
+    "files.autoSave": "onFocusChange",
+    // Show ESLint status in the status bar
+    "eslint.alwaysShowStatus": true
+}

eslint.config.js

@@ -0,0 +1,70 @@
+import tseslint from '@typescript-eslint/eslint-plugin';
+import tsparser from '@typescript-eslint/parser';
+import importPlugin from 'eslint-plugin-import';
+
+export default [
+  {
+    ignores: [
+      'build/**',
+      'node_modules/**',
+      'coverage/**',
+      '**/*.js',
+      '**/*.d.ts',
+      'dist/**',
+      'tests/**',
+      '*.config.ts',
+      '*.config.js'
+    ]
+  },
+  {
+    files: ['src/**/*.ts'],
+    languageOptions: {
+      parser: tsparser,
+      parserOptions: {
+        ecmaVersion: 2022,
+        sourceType: 'module',
+        project: './tsconfig.json'
+      },
+      globals: {
+        console: 'readonly',
+        process: 'readonly',
+        Buffer: 'readonly',
+        __dirname: 'readonly',
+        __filename: 'readonly',
+        module: 'readonly',
+        require: 'readonly'
+      }
+    },
+    plugins: {
+      '@typescript-eslint': tseslint,
+      'import': importPlugin
+    },
+    rules: {
+      // TypeScript specific rules
+      '@typescript-eslint/no-unused-vars': ['error', { 
+        argsIgnorePattern: '^_',
+        varsIgnorePattern: '^_',
+        destructuredArrayIgnorePattern: '^_'
+      }],
+      '@typescript-eslint/no-explicit-any': 'off', // Turned off for now - can be enabled gradually
+      '@typescript-eslint/explicit-function-return-type': 'off',
+      '@typescript-eslint/explicit-module-boundary-types': 'off',
+      '@typescript-eslint/no-floating-promises': 'error',
+      '@typescript-eslint/no-misused-promises': 'error',
+      '@typescript-eslint/await-thenable': 'warn', // Downgraded to warning
+      
+      // General rules
+      'no-console': ['warn', { allow: ['warn', 'error'] }],
+      'prefer-const': 'error',
+      'no-var': 'error',
+      'eqeqeq': ['error', 'always'],
+      'curly': ['error', 'all'],
+      'brace-style': ['error', '1tbs'],
+      
+      // Import rules - relaxed
+      'import/order': 'off', // Too strict for now
+      'import/newline-after-import': 'off',
+      'import/no-duplicates': 'error'
+    }
+  }
+];