fix: type declarations

Type declarations were not working in cjs because exports were types instead of classes.

Author: Gozala

.gitignore

@@ -1 +1,2 @@
 node_modules
+test/**/package-lock.json

package.json

@@ -23,7 +23,11 @@
     "test:browser": "playwright-test test/test-*.cjs",
     "test:es": "mocha test/test-*.spec.js",
     "test:cjs": "npm run test:node && npm run test:browser",
-    "test": "npm run test:es && npm run test:cjs"
+    "test:types:ts": "npm test --prefix test/ts-use",
+    "test:types:esm": "npm test --prefix test/esm-use",
+    "test:types:cjs": "npm test --prefix test/cjs-use",
+    "test:types": "npm run test:types:ts && npm run test:types:esm && npm run test:types:cjs",
+    "test": "npm run test:es && npm run test:cjs && npm run test:types"
   },
   "license": "MIT",
   "author": "Irakli Gozalishvili <[email protected]>",

src/lib.d.ts

@@ -1,4 +1,4 @@
-type Encoder = TextEncoder
-type Decoder = TextDecoder
+declare var TextEncoder: typeof self.TextEncoder
+declare var TextDecoder: typeof self.TextDecoder
 
-export { Encoder as TextEncoder, Decoder as TextDecoder }
+export { TextEncoder, TextDecoder }

test/cjs-use/package.json

@@ -0,0 +1,10 @@
+{
+  "name": "cjs-use",
+  "private": true,
+  "dependencies": {
+    "web-encoding": "file:../.."
+  },
+  "scripts": {
+    "test": "npm install && npx typescript --noEmit --allowJs --checkJs src/main.js"
+  }
+}

test/cjs-use/src/main.js

@@ -0,0 +1,19 @@
+// @ts-check
+
+const { TextEncoder, TextDecoder } = require("web-encoding")
+
+/**
+ * @param {string} text
+ * @returns {Uint8Array}
+ */
+const encode = (text) =>
+  new TextEncoder().encode(text)
+
+/**
+ * @param {Uint8Array} bytes
+ * @returns {string}
+ */
+const decode = (bytes) =>
+  new TextDecoder().decode(bytes)
+
+module.exports = { encode, decode }

test/esm-use/package.json

@@ -0,0 +1,10 @@
+{
+  "name": "esm-use",
+  "private": true,
+  "dependencies": {
+    "web-encoding": "file:../.."
+  },
+  "scripts": {
+    "test": "npm install && npx typescript --noEmit --allowJs --checkJs src/main.js"
+  }
+}

test/esm-use/src/main.js

@@ -0,0 +1,17 @@
+// @ts-check
+
+import { TextEncoder, TextDecoder } from "web-encoding"
+
+/**
+ * @param {string} text
+ * @returns {Uint8Array}
+ */
+export const encode = (text) =>
+  new TextEncoder().encode(text)
+
+/**
+ * @param {Uint8Array} bytes
+ * @returns {string}
+ */
+export const decode = (bytes) =>
+  new TextDecoder().decode(bytes)

test/ts-use/package.json

@@ -0,0 +1,10 @@
+{
+  "name": "ts-use",
+  "private": true,
+  "dependencies": {
+    "web-encoding": "file:../.."
+  },
+  "scripts": {
+    "test": "npm install && npx typescript --noEmit"
+  }
+}

test/ts-use/src/main.ts

@@ -0,0 +1,7 @@
+import { TextEncoder, TextDecoder } from "web-encoding"
+
+export const encode = (text:string):Uint8Array =>
+  new TextEncoder().encode(text)
+
+export const decode = (bytes:Uint8Array):string =>
+  new TextDecoder().decode(bytes)

test/ts-use/tsconfig.json

@@ -0,0 +1,10 @@
+{
+  "compilerOptions": {
+    "noImplicitAny": true,
+    "strict": true,
+    "strictNullChecks": true,
+    "strictFunctionTypes": true,
+    "strictPropertyInitialization": true,
+    "strictBindCallApply": true
+  }
+}