Misc

Author: cmdcolin

eslint.config.mjs

@@ -1,5 +1,6 @@
 import eslint from '@eslint/js'
 import eslintPluginUnicorn from 'eslint-plugin-unicorn'
+import importPlugin from 'eslint-plugin-import'
 import tseslint from 'typescript-eslint'
 
 export default tseslint.config(
@@ -18,6 +19,7 @@ export default tseslint.config(
   ...tseslint.configs.recommended,
   ...tseslint.configs.stylisticTypeChecked,
   ...tseslint.configs.strictTypeChecked,
+  importPlugin.flatConfigs.recommended,
   eslintPluginUnicorn.configs['flat/recommended'],
   {
     rules: {
@@ -34,6 +36,38 @@ export default tseslint.config(
       '@typescript-eslint/ban-ts-comment': 0,
       'unicorn/expiring-todo-comments': 0,
       semi: ['error', 'never'],
+
+      'import/no-unresolved': 'off',
+      'import/order': [
+        'error',
+        {
+          named: true,
+          'newlines-between': 'always',
+          alphabetize: {
+            order: 'asc',
+          },
+          groups: [
+            'builtin',
+            ['external', 'internal'],
+            ['parent', 'sibling', 'index', 'object'],
+            'type',
+          ],
+          pathGroups: [
+            {
+              group: 'builtin',
+              pattern: 'react',
+              position: 'before',
+            },
+            {
+              group: 'external',
+              pattern: '@mui/icons-material',
+              position: 'after',
+            },
+          ],
+
+          pathGroupsExcludedImportTypes: ['react'],
+        },
+      ],
     },
   },
 )

package.json

@@ -29,6 +29,7 @@
     "@typescript-eslint/eslint-plugin": "^8.4.0",
     "@typescript-eslint/parser": "^8.4.0",
     "eslint": "^9.0.0",
+    "eslint-plugin-import": "^2.31.0",
     "eslint-plugin-unicorn": "^58.0.0",
     "generic-filehandle2": "^1.0.0",
     "prettier": "^3.3.3",

src/dedupe.ts

@@ -0,0 +1,18 @@
+type Hasher<T> = (input: T) => string
+
+// from https://github.com/seriousManual/dedupe/blob/master/LICENSE
+export function dedupe<T>(list: T[], hasher: Hasher<T> = JSON.stringify) {
+  const clone: T[] = []
+  const lookup = new Set<string>()
+
+  for (const entry of list) {
+    const hashed = hasher(entry)
+
+    if (!lookup.has(hashed)) {
+      clone.push(entry)
+      lookup.add(hashed)
+    }
+  }
+
+  return clone
+}

src/index.ts

@@ -1,21 +1,14 @@
-import type { GenericFilehandle } from 'generic-filehandle2'
+import { dedupe } from './dedupe'
 import { concatUint8Array } from './util'
 
+import type { GenericFilehandle } from 'generic-filehandle2'
+
 const CHUNK_SIZE = 65536
 
 // this is the number of hex characters to use for the address in ixixx, see
 // https://github.com/GMOD/ixixx-js/blob/master/src/index.ts#L182
 const ADDRESS_SIZE = 10
 
-// https://stackoverflow.com/a/9229821/2129219
-function uniqBy<T>(a: T[], key: (elt: T) => string) {
-  const seen = new Set()
-  return a.filter(item => {
-    const k = key(item)
-    return seen.has(k) ? false : seen.add(k)
-  })
-}
-
 export default class Trix {
   constructor(
     public ixxFile: GenericFilehandle,
@@ -93,8 +86,8 @@ export default class Trix {
       }
     }
 
-    // deduplicate results based on the detail column (resultArr[1])
-    return uniqBy(resultArr, elt => elt[1]).slice(0, this.maxResults)
+    // de-duplicate results based on the detail column (resultArr[1])
+    return dedupe(resultArr, elt => elt[1]).slice(0, this.maxResults)
   }
 
   private async getIndex(opts?: { signal?: AbortSignal }) {

test/pneumobase.test.ts

@@ -1,6 +1,7 @@
-import { test, expect } from 'vitest'
-import Trix from '../src/index'
 import { LocalFile } from 'generic-filehandle2'
+import { expect, test } from 'vitest'
+
+import Trix from '../src/index'
 
 test('can find pneumobase features', async () => {
   const trix1 = new Trix(

test/search.test.ts

@@ -1,6 +1,7 @@
-import { describe, it, expect } from 'vitest'
-import Trix from '../src/index'
 import { LocalFile } from 'generic-filehandle2'
+import { describe, expect, it } from 'vitest'
+
+import Trix from '../src/index'
 
 describe('Test a search of test1 ix file', () => {
   const searchTerms = [