Replace eslint-plugin-import with eslint-plugin-import-x

Modern fork with better performance and fewer dependencies.
Updates eslint config to use import-x rules.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Author: cmdcolin

eslint.config.mjs

@@ -1,6 +1,6 @@
 import eslint from '@eslint/js'
 import { defineConfig } from 'eslint/config'
-import importPlugin from 'eslint-plugin-import'
+import importPlugin from 'eslint-plugin-import-x'
 import eslintPluginUnicorn from 'eslint-plugin-unicorn'
 import tseslint from 'typescript-eslint'
 
@@ -98,9 +98,9 @@ export default defineConfig(
       // keep off: non-null assertions are used intentionally on bounds-checked array access
       '@typescript-eslint/no-non-null-assertion': 'off',
 
-      'import/no-unresolved': 'off',
-      'import/extensions': ['error', 'ignorePackages'],
-      'import/order': [
+      'import-x/no-unresolved': 'off',
+      'import-x/extensions': ['error', 'ignorePackages'],
+      'import-x/order': [
         'error',
         {
           named: true,

package.json

@@ -50,7 +50,7 @@
     "@typescript-eslint/parser": "^8.59.0",
     "@vitest/coverage-v8": "^4.1.5",
     "eslint": "^10.2.1",
-    "eslint-plugin-import": "^2.32.0",
+    "eslint-plugin-import-x": "^4.16.2",
     "eslint-plugin-unicorn": "^64.0.0",
     "prettier": "^3.8.3",
     "puppeteer": "^24.42.0",

pnpm-lock.yaml

@@ -55,10 +55,10 @@ export default class BgzFilehandle {
       blockNum += 1
     ) {
       const uncompressedBuffer = await this._readAndUncompressBlock(
-        blockPositions[blockNum]![0],
-        blockPositions[blockNum + 1]![0],
+        blockPositions[blockNum][0],
+        blockPositions[blockNum + 1][0],
       )
-      const [, uncompressedPosition] = blockPositions[blockNum]!
+      const [, uncompressedPosition] = blockPositions[blockNum]
       const sourceOffset =
         uncompressedPosition >= position ? 0 : position - uncompressedPosition
       const sourceEnd =

src/bgzFilehandle.ts

@@ -93,7 +93,7 @@ export default class GziIndex {
 
     let comparison = compare(
       position,
-      entries[searchPosition]!,
+      entries[searchPosition],
       entries[searchPosition + 1],
     )
     while (comparison !== 0) {
@@ -105,17 +105,17 @@ export default class GziIndex {
       searchPosition = Math.ceil((upperBound - lowerBound) / 2) + lowerBound
       comparison = compare(
         position,
-        entries[searchPosition]!,
+        entries[searchPosition],
         entries[searchPosition + 1],
       )
     }
 
     // here's where we read forward
-    relevant.push(entries[searchPosition]!)
+    relevant.push(entries[searchPosition])
     let i = searchPosition + 1
     for (; i < entries.length; i += 1) {
-      relevant.push(entries[i]!)
-      if (entries[i]![UNCOMPRESSED_POSITION] >= endPosition) {
+      relevant.push(entries[i])
+      if (entries[i][UNCOMPRESSED_POSITION] >= endPosition) {
         break
       }
     }

src/gziIndex.ts

@@ -3,14 +3,14 @@ const TWO_PWR_32_DBL = TWO_PWR_16_DBL * TWO_PWR_16_DBL
 
 export function longFromBytesToUnsigned(source: Uint8Array, i = 0) {
   const low =
-    source[i]! |
-    (source[i + 1]! << 8) |
-    (source[i + 2]! << 16) |
-    (source[i + 3]! << 24)
+    source[i] |
+    (source[i + 1] << 8) |
+    (source[i + 2] << 16) |
+    (source[i + 3] << 24)
   const high =
-    source[i + 4]! |
-    (source[i + 5]! << 8) |
-    (source[i + 6]! << 16) |
-    (source[i + 7]! << 24)
+    source[i + 4] |
+    (source[i + 5] << 8) |
+    (source[i + 6] << 16) |
+    (source[i + 7] << 24)
   return (high >>> 0) * TWO_PWR_32_DBL + (low >>> 0)
 }

src/long.ts

@@ -171,7 +171,7 @@ test('cpositions increment correctly from large starting position', async () =>
 
   // Subsequent positions should be increasing
   for (let i = 1; i < cpositions.length; i++) {
-    expect(cpositions[i]).toBeGreaterThan(cpositions[i - 1]!)
+    expect(cpositions[i]).toBeGreaterThan(cpositions[i - 1])
   }
 
   // All positions should be large (>5TB)