chore(release v1.22.0): enforce stricter linting, refactor utils for nullish safety, update dependencies and publish with npm provenance

Author: vasu31dev

.github/workflows/publish-release.yml

@@ -8,18 +8,19 @@ jobs:
     runs-on: ubuntu-latest
     permissions:
       contents: read
+      id-token: write
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
+
       - name: Setup Node.js environment
         uses: actions/setup-node@v4
         with:
-          node-version: '22'
+          node-version: '24'
           registry-url: 'https://registry.npmjs.org/'
 
       - name: Install Libraries (ignore scripts)
         run: npm ci --ignore-scripts # This prevents postinstall and other scripts from running
-      - name: Publish to NPM
-        run: npm publish
-        env:
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
+
+      - name: Publish to NPM with provenance
+        run: npm publish --provenance --access public

CHANGELOG.md

@@ -1,6 +1,32 @@
 # Changelog
 
-## [1.21.0] - 2025-02-06
+## [1.22.0] - 2026-02-09
+
+### Release
+
+- Updated npm publish workflow to use Node.js `24`.
+- Added `id-token: write` GitHub Actions permission.
+- Switched publish step to `npm publish --provenance --access public`.
+
+### Linting and Quality
+
+- Enforced stricter ESLint and TypeScript rules, including nullish-coalescing and async safety checks.
+- Added Playwright recommended rule set from `plugin:playwright/playwright-test`.
+- Tightened import and general best-practice rules (`import/first`, `no-var`, `prefer-const`, `eqeqeq`, and others).
+
+### Refactors
+
+- Refactored utility functions to prefer nullish coalescing (`??`) over logical OR (`||`) where fallback-on-null/undefined is required.
+
+### Updated Dependencies
+
+- Updated `@types/node` from `^25.2.1` to `^25.2.2` and`eslint-plugin-jsdoc` from `^62.5.3` to `^62.5.4`.
+
+### Removed Dependencies
+
+- Removed `allure-commandline`, `allure-playwright`, `axios`.
+
+## [1.21.0] - 2026-02-06
 
 ### Updated Dependencies
 

eslint.config.js

@@ -39,34 +39,68 @@ module.exports = [
       'prettier/prettier': 'error',
       'no-trailing-spaces': 'error',
       'no-multiple-empty-lines': ['error', { max: 1, maxEOF: 0 }],
+      'eol-last': ['error', 'always'],
 
-      // TypeScript Rules
+      // TypeScript Rules - Base
       ...typescript.configs['eslint-recommended'].overrides[0].rules,
       ...typescript.configs.recommended.rules,
       '@typescript-eslint/no-unsafe-assignment': 'warn',
       '@typescript-eslint/no-unsafe-member-access': 'warn',
       '@typescript-eslint/no-unsafe-return': 'warn',
+      '@typescript-eslint/no-unsafe-call': 'warn',
       '@typescript-eslint/no-floating-promises': 'error',
       '@typescript-eslint/no-unnecessary-type-assertion': 'error',
       '@typescript-eslint/no-explicit-any': 'warn',
       '@typescript-eslint/explicit-module-boundary-types': 'off',
 
+      // TypeScript Rules - Additional Strict Rules
+      '@typescript-eslint/no-unused-vars': ['error', { argsIgnorePattern: '^_', varsIgnorePattern: '^_' }],
+      '@typescript-eslint/no-unused-expressions': 'error',
+      '@typescript-eslint/no-empty-function': 'warn',
+      '@typescript-eslint/prefer-nullish-coalescing': 'error',
+      '@typescript-eslint/prefer-optional-chain': 'error',
+      '@typescript-eslint/prefer-as-const': 'error',
+      '@typescript-eslint/no-non-null-assertion': 'warn',
+      '@typescript-eslint/no-duplicate-enum-values': 'error',
+      '@typescript-eslint/no-inferrable-types': ['error', { ignoreParameters: false, ignoreProperties: false }],
+
       // Import Rules
       'import/no-unresolved': 'error',
       'import/named': 'error',
       'import/default': 'error',
       'import/no-absolute-path': 'error',
       'import/no-self-import': 'error',
+      // 'import/no-duplicates': 'error',
+      'import/first': 'error',
+      'import/no-mutable-exports': 'error',
       'sort-imports': ['error', { ignoreDeclarationSort: true }],
 
-      // General Rules
-      'require-await': 'error',
+      // General Rules - Best Practices
+      'require-await': 'off', // Turned off in favor of @typescript-eslint version
+      '@typescript-eslint/require-await': 'error',
+      '@typescript-eslint/await-thenable': 'error',
+      '@typescript-eslint/no-misused-promises': 'error',
       complexity: ['warn', { max: 11 }],
+      'no-console': ['warn', { allow: ['warn', 'error', 'info'] }],
+      'no-debugger': 'error',
+      'no-var': 'error',
+      'prefer-const': 'error',
+      'prefer-template': 'error',
+      'object-shorthand': 'error',
+      // 'no-else-return': ['error', { allowElseIf: false }],
+      'no-lonely-if': 'error',
+      'no-useless-return': 'error',
+      'no-nested-ternary': 'warn',
+      eqeqeq: ['error', 'always', { null: 'ignore' }],
+      'no-throw-literal': 'error',
 
       // JSDoc Rules
       'jsdoc/check-alignment': 'warn',
       'jsdoc/check-indentation': 'warn',
 
+      // Playwright recommended rules (from plugin:playwright/playwright-test)
+      ...playwright.configs['playwright-test'].rules,
+
       // Playwright Rules
       'playwright/missing-playwright-await': ['error'],
       'playwright/no-focused-test': 'error',
@@ -85,6 +119,7 @@ module.exports = [
       'playwright/expect-expect': 'off',
       'playwright/no-conditional-in-test': 'off',
       'playwright/no-skipped-test': 'off',
+      'playwright/consistent-spacing-between-blocks': 'off',
     },
   },
   { files: ['src/**/*.ts'], rules: { '@typescript-eslint/explicit-module-boundary-types': 'warn' } },