Update husky, simplify configurations

.gitignore

@@ -1,10 +1,7 @@
+.husky/_
 .yarn
 coverage
-dist
 node_modules
-index.js
-index.js.map
-index.d.ts
 lib/*.js
 lib/*.js.map
 lib/*.d.ts

.husky/pre-commit

@@ -0,0 +1 @@
+yarn lint && yarn build && yarn test

eslint.config.js

@@ -6,23 +6,8 @@ module.exports = config([
     languageOptions: {
       parserOptions: {
         tsconfigRootDir: __dirname,
-        project: [ './tsconfig.eslint.json' ],
+        project: [ 'tsconfig.eslint.json' ],
       },
     },
   },
-  {
-    files: [ '**/*.ts', 'webpack.config.js' ],
-    rules: {
-      'import/no-nodejs-modules': 'off',
-      // The naming convention rules complain about the public static properties, as well
-      'ts/naming-convention': 'off',
-    },
-  },
-  {
-    // The readme contains examples with console logging
-    files: [ 'README.md/*' ],
-    rules: {
-      'no-console': 'off',
-    },
-  },
 ]);

lib/ReadableFromWeb.ts

@@ -1,10 +1,10 @@
 import { Readable, type ReadableOptions } from 'readable-stream';
 
-class ReadableFromWeb<R> extends Readable {
-  private readonly reader: ReadableStreamDefaultReader<R>;
+class ReadableFromWeb<T> extends Readable {
+  private readonly reader: ReadableStreamDefaultReader<T>;
   private readerClosed: boolean;
 
-  public constructor(stream: ReadableStream<R>, options?: ReadableOptions) {
+  public constructor(stream: ReadableStream<T>, options?: ReadableOptions) {
     super(options);
     this.reader = stream.getReader();
     this.readerClosed = false;
@@ -16,6 +16,7 @@ class ReadableFromWeb<R> extends Readable {
     });
   }
 
+  // eslint-disable-next-line ts/naming-convention
   public _read(): void {
     this.reader.read()
       .then(chunk => this.push(chunk.done ? null : chunk.value))
@@ -34,7 +35,7 @@ class ReadableFromWeb<R> extends Readable {
   }
 }
 
-function readableFromWeb<R>(stream: ReadableStream<R>, options?: ReadableOptions): Readable {
+function readableFromWeb<T>(stream: ReadableStream<T>, options?: ReadableOptions): Readable {
   return new ReadableFromWeb(stream, options);
 }
 

package.json

@@ -1,5 +1,5 @@
 {
-  "name": "@comunica/readable-from-web",
+  "name": "readable-from-web",
   "type": "commonjs",
   "version": "1.0.0",
   "packageManager": "[email protected]",
@@ -20,7 +20,8 @@
   "scripts": {
     "lint": "eslint .",
     "build": "tsc",
-    "test": "jest"
+    "test": "jest",
+    "prepare": "husky"
   },
   "dependencies": {
     "readable-stream": "^4.0.0"
@@ -29,18 +30,13 @@
     "@rubensworks/eslint-config": "^3.0.0",
     "@types/jest": "^29.0.0",
     "@types/readable-stream": "^4.0.0",
-    "eslint": "8.48.0",
-    "husky": "^4.0.0",
+    "eslint": "^8.0.0",
+    "husky": "^9.0.0",
     "jest": "^29.0.0",
     "readable-stream-node-to-web": "^1.0.0",
     "stream-to-string": "^1.0.0",
     "streamify-string": "^1.0.0",
     "ts-jest": "^29.0.0",
     "typescript": "^5.0.0"
-  },
-  "husky": {
-    "hooks": {
-      "pre-commit": "yarn run lint && yarn run build && yarn run test"
-    }
   }
 }

tsconfig.eslint.json

@@ -1,7 +1,6 @@
 {
   "extends": "./tsconfig.json",
   "include": [
-    "index.ts",
     "lib/*.ts",
     "test/*.ts"
   ]

tsconfig.json

@@ -17,7 +17,6 @@
     "sourceMap": true
   },
   "include": [
-    "index.ts",
     "lib/*.ts"
   ],
   "exclude": [