@gmod/tabix 1.5.14 → 1.5.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gmod/tabix",
-  "version": "1.5.14",
+  "version": "1.5.15",
   "description": "Read Tabix-indexed files, supports both .tbi and .csi indexes",
   "license": "MIT",
   "repository": "GMOD/tabix-js",
@@ -20,16 +20,17 @@
     "src"
   ],
   "scripts": {
-    "test": "jest",
-    "coverage": "npm test -- --coverage",
-    "lint": "eslint --report-unused-disable-directives --max-warnings 0 src test",
+    "test": "vitest",
+    "lint": "eslint --report-unused-disable-directives --max-warnings 0",
+    "format": "prettier --write .",
     "clean": "rimraf dist esm",
     "prebuild": "npm run clean && npm run lint",
     "build:esm": "tsc --target es2018 --outDir esm",
     "build:es5": "tsc --target es2015 --module commonjs --outDir dist",
     "build": "npm run build:esm && npm run build:es5",
-    "preversion": "npm run lint && npm test && npm run build",
-    "prepublishOnly": "npm run lint && npm test && npm run build",
+    "postbuild": "webpack",
+    "preversion": "npm run lint && npm test run && npm run build",
+    "prepublishOnly": "npm run lint && npm test run && npm run build",
     "version": "standard-changelog && git add CHANGELOG.md",
     "postversion": "git push --follow-tags"
   },
@@ -47,23 +48,25 @@
   },
   "devDependencies": {
     "@eslint/eslintrc": "^3.1.0",
-    "@eslint/js": "^9.7.0",
-    "@types/jest": "^29.5.12",
+    "@eslint/js": "^9.9.0",
     "@types/long": "^4.0.0",
     "@types/node": "^20.11.16",
-    "@typescript-eslint/eslint-plugin": "^7.17.0",
-    "@typescript-eslint/parser": "^7.17.0",
-    "eslint": "^9.0.0",
+    "@typescript-eslint/eslint-plugin": "^8.0.1",
+    "@typescript-eslint/parser": "^8.0.1",
+    "@vitest/coverage-v8": "^2.0.5",
+    "buffer": "^6.0.3",
+    "eslint": "^9.9.0",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-prettier": "^5.0.1",
-    "eslint-plugin-unicorn": "^54.0.0",
-    "jest": "^29.7.0",
-    "jest-environment-jsdom": "^29.5.0",
-    "prettier": "^3.2.5",
+    "eslint-plugin-unicorn": "^55.0.0",
+    "prettier": "^3.3.3",
     "rimraf": "^6.0.1",
     "standard-changelog": "^6.0.0",
-    "ts-jest": "^29.1.1",
-    "typescript": "^5.3.3"
+    "typescript": "^5.3.3",
+    "typescript-eslint": "^8.0.1",
+    "vitest": "^2.0.5",
+    "webpack": "^5.93.0",
+    "webpack-cli": "^5.1.4"
   },
   "publishConfig": {
     "access": "public"

package/src/chunk.ts

@@ -20,7 +20,7 @@ export default class Chunk {
   }
 
   toUniqueString() {
-    // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
+     
     return `${this.minv}..${this.maxv} (bin ${
       this.bin
     }, fetchedSize ${this.fetchedSize()})`

package/src/csi.ts

@@ -30,10 +30,10 @@ export default class CSI extends IndexFile {
   }
   async lineCount(refName: string, opts: Options = {}): Promise<number> {
     const indexData = await this.parse(opts)
-    if (!indexData) {
+    const refId = indexData.refNameToId[refName]
+    if (refId === undefined) {
       return -1
     }
-    const refId = indexData.refNameToId[refName]
     const idx = indexData.indices[refId]
     if (!idx) {
       return -1
@@ -44,6 +44,7 @@ export default class CSI extends IndexFile {
     }
     return -1
   }
+
   indexCov() {
     throw new Error('CSI indexes do not support indexcov')
   }
@@ -206,10 +207,10 @@ export default class CSI extends IndexFile {
     }
 
     const indexData = await this.parse(opts)
-    if (!indexData) {
+    const refId = indexData.refNameToId[refName]
+    if (refId === undefined) {
       return []
     }
-    const refId = indexData.refNameToId[refName]
     const ba = indexData.indices[refId]
     if (!ba) {
       return []
@@ -258,7 +259,7 @@ export default class CSI extends IndexFile {
           `query ${beg}-${end} is too large for current binning scheme (shift ${this.minShift}, depth ${this.depth}), try a smaller query or a coarser index binning scheme`,
         )
       }
-      bins.push([b, e])
+      bins.push([b, e] as const)
     }
     return bins
   }

package/src/indexFile.ts

@@ -65,7 +65,7 @@ export default abstract class IndexFile {
 
   async parse(opts: Options = {}) {
     if (!this.parseP) {
-      this.parseP = this._parse(opts).catch(e => {
+      this.parseP = this._parse(opts).catch((e: unknown) => {
         this.parseP = undefined
         throw e
       })

package/src/tabixIndexedFile.ts

@@ -1,7 +1,7 @@
 import AbortablePromiseCache from '@gmod/abortable-promise-cache'
 import LRU from 'quick-lru'
 import { Buffer } from 'buffer'
-import { GenericFilehandle, LocalFile } from 'generic-filehandle'
+import { GenericFilehandle, RemoteFile, LocalFile } from 'generic-filehandle'
 import { unzip, unzipChunkSlice } from '@gmod/bgzf-filehandle'
 import { checkAbortSignal } from './util'
 import IndexFile, { Options, IndexData } from './indexFile'
@@ -33,44 +33,57 @@ function timeout(time: number) {
 export default class TabixIndexedFile {
   private filehandle: GenericFilehandle
   private index: IndexFile
-  private chunkSizeLimit: number
   private yieldTime: number
   private renameRefSeq: (n: string) => string
   private chunkCache: AbortablePromiseCache<Chunk, ReadChunk>
 
   /**
    * @param {object} args
+   *
    * @param {string} [args.path]
+   *
    * @param {filehandle} [args.filehandle]
+   *
    * @param {string} [args.tbiPath]
+   *
    * @param {filehandle} [args.tbiFilehandle]
+   *
    * @param {string} [args.csiPath]
+   *
    * @param {filehandle} [args.csiFilehandle]
-   * @param {number} [args.yieldTime] yield to main thread after N milliseconds if reading features is taking a long time to avoid hanging main thread
-   * @param {function} [args.renameRefSeqs] optional function with sig `string => string` to transform
-   * reference sequence names for the purpose of indexing and querying. note that the data that is returned is
-   * not altered, just the names of the reference sequences that are used for querying.
+   *
+   * @param {number} [args.yieldTime] yield to main thread after N milliseconds
+   * if reading features is taking a long time to avoid hanging main thread
+   *
+   * @param {function} [args.renameRefSeqs] optional function with sig `string
+   * => string` to transform reference sequence names for the purpose of
+   * indexing and querying. note that the data that is returned is not altered,
+   * just the names of the reference sequences that are used for querying.
    */
   constructor({
     path,
     filehandle,
+    url,
     tbiPath,
+    tbiUrl,
     tbiFilehandle,
     csiPath,
+    csiUrl,
     csiFilehandle,
     yieldTime = 500,
-    chunkSizeLimit = 50000000,
     renameRefSeqs = n => n,
     chunkCacheSize = 5 * 2 ** 20,
   }: {
     path?: string
     filehandle?: GenericFilehandle
+    url?: string
     tbiPath?: string
+    tbiUrl?: string
     tbiFilehandle?: GenericFilehandle
     csiPath?: string
+    csiUrl?: string
     csiFilehandle?: GenericFilehandle
     yieldTime?: number
-    chunkSizeLimit?: number
     renameRefSeqs?: (n: string) => string
     chunkCacheSize?: number
   }) {
@@ -78,6 +91,8 @@ export default class TabixIndexedFile {
       this.filehandle = filehandle
     } else if (path) {
       this.filehandle = new LocalFile(path)
+    } else if (url) {
+      this.filehandle = new RemoteFile(url)
     } else {
       throw new TypeError('must provide either filehandle or path')
     }
@@ -107,13 +122,24 @@ export default class TabixIndexedFile {
         filehandle: new LocalFile(`${path}.tbi`),
         renameRefSeqs,
       })
+    } else if (csiUrl) {
+      this.index = new CSI({
+        filehandle: new RemoteFile(csiUrl),
+      })
+    } else if (tbiUrl) {
+      this.index = new TBI({
+        filehandle: new RemoteFile(tbiUrl),
+      })
+    } else if (url) {
+      this.index = new TBI({
+        filehandle: new RemoteFile(`${url}.tbi`),
+      })
     } else {
       throw new TypeError(
-        'must provide one of tbiFilehandle, tbiPath, csiFilehandle, or csiPath',
+        'must provide one of tbiFilehandle, tbiPath, csiFilehandle, csiPath, tbiUrl, csiUrl',
       )
     }
 
-    this.chunkSizeLimit = chunkSizeLimit
     this.renameRefSeq = renameRefSeqs
     this.yieldTime = yieldTime
     this.chunkCache = new AbortablePromiseCache<Chunk, ReadChunk>({
@@ -125,10 +151,16 @@ export default class TabixIndexedFile {
 
   /**
    * @param refName name of the reference sequence
+   *
    * @param start start of the region (in 0-based half-open coordinates)
+   *
    * @param end end of the region (in 0-based half-open coordinates)
-   * @param opts callback called for each line in the region. can also pass a object param containing obj.lineCallback, obj.signal, etc
-   * @returns promise that is resolved when the whole read is finished, rejected on error
+   *
+   * @param opts callback called for each line in the region. can also pass a
+   * object param containing obj.lineCallback, obj.signal, etc
+   *
+   * @returns promise that is resolved when the whole read is finished,
+   * rejected on error
    */
   async getLines(
     refName: string,
@@ -139,9 +171,7 @@ export default class TabixIndexedFile {
     let signal: AbortSignal | undefined
     let options: Options = {}
     let callback: (line: string, lineOffset: number) => void
-    if (opts === undefined) {
-      throw new TypeError('line callback must be provided')
-    }
+
     if (typeof opts === 'function') {
       callback = opts
     } else {
@@ -149,12 +179,6 @@ export default class TabixIndexedFile {
       callback = opts.lineCallback
       signal = opts.signal
     }
-    if (refName === undefined) {
-      throw new TypeError('must provide a reference sequence name')
-    }
-    if (!callback) {
-      throw new TypeError('line callback must be provided')
-    }
 
     const metadata = await this.index.getMetadata(options)
     checkAbortSignal(signal)
@@ -172,17 +196,6 @@ export default class TabixIndexedFile {
     const chunks = await this.index.blocksForRange(refName, start, end, options)
     checkAbortSignal(signal)
 
-    // check the chunks for any that are over the size limit.  if
-    // any are, don't fetch any of them
-    for (const chunk of chunks) {
-      const size = chunk.fetchedSize()
-      if (size > this.chunkSizeLimit) {
-        throw new Error(
-          `Too much data. Chunk size ${size.toLocaleString()} bytes exceeds chunkSizeLimit of ${this.chunkSizeLimit.toLocaleString()}.`,
-        )
-      }
-    }
-
     // now go through each chunk and parse and filter the lines out of it
     let last = Date.now()
     for (const c of chunks) {
@@ -204,8 +217,9 @@ export default class TabixIndexedFile {
         const b = buffer.slice(blockStart, n)
         const line = decoder?.decode(b) ?? b.toString()
 
+        // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
         if (dpositions) {
-          while (blockStart + c.minv.dataPosition >= dpositions[pos++]) {}
+          while (blockStart + c.minv.dataPosition >= dpositions[pos++]!) {}
           pos--
         }
 
@@ -242,8 +256,8 @@ export default class TabixIndexedFile {
             // then the blockStart-dpositions is an uncompressed file offset from
             // that bgzip block boundary, and since the cpositions are multiplied by
             // (1 << 8) these uncompressed offsets get a unique space
-            cpositions[pos] * (1 << 8) +
-              (blockStart - dpositions[pos]) +
+            cpositions[pos]! * (1 << 8) +
+              (blockStart - dpositions[pos]!) +
               c.minv.dataPosition +
               1,
           )
@@ -270,14 +284,15 @@ export default class TabixIndexedFile {
   }
 
   /**
-   * get a buffer containing the "header" region of
-   * the file, which are the bytes up to the first
-   * non-meta line
+   * get a buffer containing the "header" region of the file, which are the
+   * bytes up to the first non-meta line
    */
   async getHeaderBuffer(opts: Options = {}) {
     const { firstDataLine, metaChar, maxBlockSize } =
       await this.getMetadata(opts)
     checkAbortSignal(opts.signal)
+
+    // eslint-disable-next-line @typescript-eslint/restrict-plus-operands
     const maxFetch = (firstDataLine?.blockPosition || 0) + maxBlockSize
     // TODO: what if we don't have a firstDataLine, and the header
     // actually takes up more than one block? this case is not covered here
@@ -305,8 +320,8 @@ export default class TabixIndexedFile {
   }
 
   /**
-   * get a string containing the "header" region of the
-   * file, is the portion up to the first non-meta line
+   * get a string containing the "header" region of the file, is the portion up
+   * to the first non-meta line
    *
    * @returns {Promise} for a string
    */
@@ -316,9 +331,8 @@ export default class TabixIndexedFile {
   }
 
   /**
-   * get an array of reference sequence names, in the order in which
-   * they occur in the file. reference sequence renaming is not applied
-   * to these names.
+   * get an array of reference sequence names, in the order in which they occur
+   * in the file. reference sequence renaming is not applied to these names.
    */
   async getReferenceSequenceNames(opts: Options = {}) {
     const metadata = await this.getMetadata(opts)
@@ -326,12 +340,17 @@ export default class TabixIndexedFile {
   }
 
   /**
-   * @param {object} metadata metadata object from the parsed index,
-   * containing columnNumbers, metaChar, and format
+   * @param {object} metadata metadata object from the parsed index, containing
+   * columnNumbers, metaChar, and format
+   *
    * @param {string} regionRefName
+   *
    * @param {number} regionStart region start coordinate (0-based-half-open)
+   *
    * @param {number} regionEnd region end coordinate (0-based-half-open)
+   *
    * @param {array[string]} line
+   *
    * @returns {object} like `{startCoordinate, overlaps}`. overlaps is boolean,
    * true if line is a data line that overlaps the given region
    */
@@ -364,9 +383,9 @@ export default class TabixIndexedFile {
     }
     const maxColumn = Math.max(ref, start, end)
 
-    // this code is kind of complex, but it is fairly fast.
-    // basically, we want to avoid doing a split, because if the lines are really long
-    // that could lead to us allocating a bunch of extra memory, which is slow
+    // this code is kind of complex, but it is fairly fast. basically, we want
+    // to avoid doing a split, because if the lines are really long that could
+    // lead to us allocating a bunch of extra memory, which is slow
 
     let currentColumnNumber = 1 // cols are numbered starting at 1 in the index metadata
     let currentColumnStart = 0
@@ -450,8 +469,11 @@ export default class TabixIndexedFile {
   }
 
   /**
-   * return the approximate number of data lines in the given reference sequence
+   * return the approximate number of data lines in the given reference
+   * sequence
+   *
    * @param refSeq reference sequence name
+   *
    * @returns number of data lines present on that reference sequence
    */
   async lineCount(refName: string, opts: Options = {}) {
@@ -476,8 +498,8 @@ export default class TabixIndexedFile {
    * contiguous bgzip blocks) of the file
    */
   async readChunk(c: Chunk, opts: Options = {}) {
-    // fetch the uncompressed data, uncompress carefully a block at a time,
-    // and stop when done
+    // fetch the uncompressed data, uncompress carefully a block at a time, and
+    // stop when done
 
     const data = await this._readRegion(
       c.minv.blockPosition,

package/src/tbi.ts

@@ -10,7 +10,8 @@ const TBI_MAGIC = 21578324 // TBI\1
 const TAD_LIDX_SHIFT = 14
 
 /**
- * calculate the list of bins that may overlap with region [beg,end) (zero-based half-open)
+ * calculate the list of bins that may overlap with region [beg,end)
+ * (zero-based half-open)
  */
 function reg2bins(beg: number, end: number) {
   beg += 1 // < convert to 1-based closed
@@ -22,16 +23,16 @@ function reg2bins(beg: number, end: number) {
     [73 + (beg >> 20), 73 + (end >> 20)],
     [585 + (beg >> 17), 585 + (end >> 17)],
     [4681 + (beg >> 14), 4681 + (end >> 14)],
-  ]
+  ] as const
 }
 
 export default class TabixIndex extends IndexFile {
   async lineCount(refName: string, opts: Options = {}) {
     const indexData = await this.parse(opts)
-    if (!indexData) {
+    const refId = indexData.refNameToId[refName]
+    if (refId === undefined) {
       return -1
     }
-    const refId = indexData.refNameToId[refName]
     const idx = indexData.indices[refId]
     if (!idx) {
       return -1
@@ -194,10 +195,10 @@ export default class TabixIndex extends IndexFile {
     }
 
     const indexData = await this.parse(opts)
-    if (!indexData) {
+    const refId = indexData.refNameToId[refName]
+    if (refId === undefined) {
       return []
     }
-    const refId = indexData.refNameToId[refName]
     const ba = indexData.indices[refId]
     if (!ba) {
       return []

package/src/util.ts

@@ -15,14 +15,15 @@ class AbortError extends Error {
   public code: string | undefined
 }
 /**
- * Properly check if the given AbortSignal is aborted.
- * Per the standard, if the signal reads as aborted,
- * this function throws either a DOMException AbortError, or a regular error
- * with a `code` attribute set to `ERR_ABORTED`.
+ * Properly check if the given AbortSignal is aborted. Per the standard, if the
+ * signal reads as aborted, this function throws either a DOMException
+ * AbortError, or a regular error with a `code` attribute set to `ERR_ABORTED`.
  *
  * For convenience, passing `undefined` is a no-op
  *
- * @param {AbortSignal} [signal] an AbortSignal, or anything with an `aborted` attribute
+ * @param {AbortSignal} [signal] an AbortSignal, or anything with an `aborted`
+ * attribute
+ *
  * @returns nothing
  */
 export function checkAbortSignal(signal?: AbortSignal) {
@@ -59,7 +60,7 @@ export function canMergeBlocks(chunk1: Chunk, chunk2: Chunk) {
   )
 }
 
-export function optimizeChunks(chunks: Chunk[], lowest: VirtualOffset) {
+export function optimizeChunks(chunks: Chunk[], lowest?: VirtualOffset) {
   const mergedChunks: Chunk[] = []
   let lastChunk: Chunk | null = null
 

package/src/virtualOffset.ts

@@ -16,20 +16,6 @@ export default class VirtualOffset {
       this.blockPosition - b.blockPosition || this.dataPosition - b.dataPosition
     )
   }
-
-  static min(...args: VirtualOffset[]) {
-    let min
-    let i = 0
-    for (; !min; i += 1) {
-      min = args[i]
-    }
-    for (; i < args.length; i += 1) {
-      if (min.compareTo(args[i]) > 0) {
-        min = args[i]
-      }
-    }
-    return min
-  }
 }
 export function fromBytes(bytes: Buffer, offset = 0, bigendian = false) {
   if (bigendian) {
@@ -37,12 +23,12 @@ export function fromBytes(bytes: Buffer, offset = 0, bigendian = false) {
   }
 
   return new VirtualOffset(
-    bytes[offset + 7] * 0x10000000000 +
-      bytes[offset + 6] * 0x100000000 +
-      bytes[offset + 5] * 0x1000000 +
-      bytes[offset + 4] * 0x10000 +
-      bytes[offset + 3] * 0x100 +
-      bytes[offset + 2],
-    (bytes[offset + 1] << 8) | bytes[offset],
+    bytes[offset + 7]! * 0x10000000000 +
+      bytes[offset + 6]! * 0x100000000 +
+      bytes[offset + 5]! * 0x1000000 +
+      bytes[offset + 4]! * 0x10000 +
+      bytes[offset + 3]! * 0x100 +
+      bytes[offset + 2]!,
+    (bytes[offset + 1]! << 8) | bytes[offset]!,
   )
 }