thread-stream 2.7.0 → 3.0.1

package/.github/workflows/ci.yml

@@ -38,15 +38,11 @@ jobs:
       contents: read
     strategy:
       matrix:
-        node-version: [14, 16, 18, 20]
+        node-version: [18, 20, 22]
         os: [macos-latest, ubuntu-latest, windows-latest]
         exclude:
           - os: windows-latest
-            node-version: 14
-          - os: macos-latest
-            node-version: 14
-          - os: macos-latest
-            node-version: 16
+            node-version: 22
 
     steps:
       - name: Check out repo

package/index.d.ts

@@ -1,5 +1,5 @@
 import { EventEmitter } from 'events'
-import workerThreads from 'worker_threads'
+import * as workerThreads from 'worker_threads'
 
 interface ThreadStreamOptions {
   /**
@@ -76,9 +76,8 @@ declare class ThreadStream extends EventEmitter {
    * @param args the arguments to be passed to the event handlers.
    * @returns {boolean} `true` if the event had listeners, `false` otherwise.
    */
-  emit(eventName: string | symbol, ...args: any[]): boolean {
-    return super.emit(eventName, ...args);
-  }
+  emit(eventName: string | symbol, ...args: any[]): boolean;
+
   /**
    * Post a message to the Worker with specified data and an optional list of transferable objects.
    *

package/lib/worker.js

@@ -159,7 +159,6 @@ process.once('exit', exitCode => {
     process.exit(exitCode)
     return
   }
-
   if (destination?.writableNeedDrain && !destination?.writableEnded) {
     parentPort.postMessage({
       code: 'WARNING',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thread-stream",
-  "version": "2.7.0",
+  "version": "3.0.1",
   "description": "A streaming way to send data to a Node.js Worker Thread",
   "main": "index.js",
   "types": "index.d.ts",
@@ -15,7 +15,7 @@
     "fastbench": "^1.0.1",
     "husky": "^9.0.6",
     "pino-elasticsearch": "^8.0.0",
-    "sonic-boom": "^3.0.0",
+    "sonic-boom": "^4.0.1",
     "standard": "^17.0.0",
     "tap": "^16.2.0",
     "ts-node": "^10.8.0",
@@ -23,7 +23,8 @@
     "why-is-node-running": "^2.2.2"
   },
   "scripts": {
-    "test": "standard && npm run transpile && tap \"test/**/*.test.*js\" && tap --ts test/*.test.*ts",
+    "build": "tsc --noEmit",
+    "test": "standard && npm run build && npm run transpile && tap \"test/**/*.test.*js\" && tap --ts test/*.test.*ts",
     "test:ci": "standard && npm run transpile && npm run test:ci:js && npm run test:ci:ts",
     "test:ci:js": "tap --no-check-coverage --timeout=120 --coverage-report=lcovonly \"test/**/*.test.*js\"",
     "test:ci:ts": "tap --ts --no-check-coverage --coverage-report=lcovonly \"test/**/*.test.*ts\"",

package/test/never-drain.test.js

@@ -22,7 +22,9 @@ function retryUntilTimeout (fn, timeout) {
   })
 }
 
-test('emit warning when the worker gracefully exit without the stream ended', async function (t) {
+const isNode18 = process.version.indexOf('v18') === 0
+
+test('emit warning when the worker gracefully exit without the stream ended', { skip: !isNode18 }, async function (t) {
   const expectedWarning = 'ThreadStream: process exited before destination stream was drained. this may indicate that the destination stream try to write to a another missing stream'
   const stream = new ThreadStream({
     filename: join(__dirname, 'to-next.js')

package/test/pkg/pkg.test.js

@@ -14,6 +14,8 @@ test('worker test when packaged into executable using pkg', async (t) => {
   // package the app into several node versions, check config for more info
   const filePath = `${join(__dirname, packageName)}.js`
   const configPath = join(__dirname, 'pkg.config.json')
+  process.env.NODE_OPTIONS ||= ''
+  process.env.NODE_OPTIONS = '--no-warnings'
   const { stderr } = await exec(`npx pkg ${filePath} --config ${configPath}`)
 
   // there should be no error when packaging

package/tsconfig.json

@@ -0,0 +1,8 @@
+{
+    "compilerOptions": {
+        "esModuleInterop": true
+    },
+    "files": [
+        "index.d.ts"
+    ],
+}