fix missing types and jsDocs

.eslintrc.json

@@ -3,8 +3,7 @@
     "es2021": true,
     "node": true
   },
-  "extends": ["eslint:recommended", "plugin:@typescript-eslint/recommended"],
-  "parser": "@typescript-eslint/parser",
+  "extends": "standard",
   "parserOptions": {
     "ecmaVersion": "latest",
     "sourceType": "module"

.github/workflows/ci.yml

@@ -21,4 +21,5 @@ jobs:
         with:
           node-version: ${{ matrix.node-version }}
       - run: npm install
+      - run: npx tsc
       - run: npm test

package.json

@@ -38,8 +38,6 @@
     "@types/content-type": "^1.1.8",
     "@types/mocha": "latest",
     "@types/readable-stream": "^4.0.11",
-    "@typescript-eslint/eslint-plugin": "^6.18.1",
-    "@typescript-eslint/parser": "^6.14.0",
     "chai": "^5.0.3",
     "eslint": "^8.56.0",
     "eslint-config-standard": "^17.1.0",

src/index.js

@@ -1,25 +1,21 @@
-import * as poParser from './poparser.js';
+import { poParse, poStream } from './poparser.js';
 import poCompiler from './pocompiler.js';
 import moParser from './moparser.js';
 import moCompiler from './mocompiler.js';
 
 /**
  * Translation parser and compiler for PO files
  * @see https://www.gnu.org/software/gettext/manual/html_node/PO.html
- *
- * @type {import("./index.d.ts").po} po
  */
 export const po = {
-  parse: poParser.parse,
-  createParseStream: poParser.stream,
+  parse: poParse,
+  createParseStream: poStream,
   compile: poCompiler
 };
 
 /**
- * Translation parser and compiler for PO files
+ * Translation parser and compiler for MO files
  * @see https://www.gnu.org/software/gettext/manual/html_node/MO.html
- *
- * @type {import("./index.d.ts").mo} mo
  */
 export const mo = {
   parse: moParser,

src/mocompiler.js

@@ -1,4 +1,4 @@
 import encoding from 'encoding';
 import { HEADERS, formatCharset, generateHeader, compareMsgid } from './shared.js';
 import contentType from 'content-type';
 
@@ -6,7 +6,7 @@
  * Exposes general compiler function. Takes a translation
  * object as a parameter and returns binary MO object
  *
- * @param {import('./index.d.ts').GetTextTranslations} table Translation object
+ * @param {import('./types.js').GetTextTranslations} table Translation object
  * @return {Buffer} Compiled binary MO object
  */
 export default function (table) {
@@ -16,37 +16,42 @@
 }
 
 /**
- * Creates a MO compiler object.
- *
- * @constructor
- * @param {import('./index.d.ts').GetTextTranslations} table Translation table as defined in the README
- * @return {import('./index.d.ts').Compiler} Compiler
+ * Prepare the header object to be compatible with MO compiler
+ * @param {{[key: string]: string}} headers the headers
+ * @return {{[key: string]: string}} The prepared header
  */
-function Compiler (table = {}) {
-  this._table = table;
-
-  let { headers = {}, translations = {} } = this._table;
-
-  headers = Object.keys(headers).reduce((result, key) => {
+function prepareMoHeaders (headers) {
+  return Object.keys(headers).reduce((/** @type {{[key: string]: string}} */ result, key) => {
     const lowerKey = key.toLowerCase();
 
     if (HEADERS.has(lowerKey)) {
       // POT-Creation-Date is removed in MO (see https://savannah.gnu.org/bugs/?49654)
       if (lowerKey !== 'pot-creation-date') {
-        result[HEADERS.get(lowerKey)] = headers[key];
+        const value = HEADERS.get(lowerKey);
+        if (value) {
+          result[value] = headers[key];
+        }
       }
     } else {
       result[key] = headers[key];
     }
 
     return result;
   }, {});
+}
 
-  // filter out empty translations
-  translations = Object.keys(translations).reduce((result, msgctxt) => {
+/**
+ * Prepare the translation object to be compatible with MO compiler
+ * @param {import('./types.js').GetTextTranslations['translations']} translations
+ * @return {import('./types.js').GetTextTranslations['translations']}
+ */
+function prepareTranslations (translations) {
+  return Object.keys(translations).reduce((result, msgctxt) => {
     const context = translations[msgctxt];
-    const msgs = Object.keys(context).reduce((result, msgid) => {
-      const hasTranslation = context[msgid].msgstr.some(item => !!item.length);
+    const msgs = Object.keys(context).reduce((/** @type {{[key: string]: string}} */ result, msgid) => {
+      /** @type {import('./types.js').GetTextTranslation[]} */
+      const TranslationMsgstr = context[msgid].msgstr;
+      const hasTranslation = TranslationMsgstr.some(item => !!item.length);
 
       if (hasTranslation) {
         result[msgid] = context[msgid];
@@ -56,26 +61,36 @@
     }, {});
 
     if (Object.keys(msgs).length) {
       result[msgctxt] = msgs;
     }
 
     return result;
   }, {});
+}
 
-  this._table.translations = translations;
-  this._table.headers = headers;
+/**
+ * Creates a MO compiler object.
+ *
+ * @param {import('./types.js').GetTextTranslations} [table] Translation table as defined in the README
+ */
+function Compiler (table) {
+  /** @type {import('./types.js').GetTextTranslations} _table The translation table */
+  this._table = {
+    charset: undefined,
+    translations: prepareTranslations(table?.translations ?? {}),
+    headers: prepareMoHeaders(table?.headers ?? {})
+  };
 
   this._translations = [];
 
-  this._writeFunc = 'writeUInt32LE';
-
   this._handleCharset();
-}
 
-/**
- * Magic bytes for the generated binary data
- */
-Compiler.prototype.MAGIC = 0x950412de;
+  /**
+   * Magic bytes for the generated binary data
+   * @type {number} MAGIC file header magic value of mo file
+   */
+  this.MAGIC = 0x950412de;
+}
 
 /**
  * Handles header values, replaces or adds (if needed) a charset property
@@ -96,17 +111,19 @@
 
 /**
  * Generates an array of translation strings
- * in the form of [{msgid:... , msgstr:...}]
+ * in the form of [{msgid:..., msgstr: ...}]
  *
- * @return {Array} Translation strings array
  */
 Compiler.prototype._generateList = function () {
+  /** @type {import('./types.js').GetTextTranslation[]} */
   const list = [];
 
-  list.push({
-    msgid: Buffer.alloc(0),
-    msgstr: encoding.convert(generateHeader(this._table.headers), this._table.charset)
-  });
+  if ('headers' in this._table) {
+    list.push({
+      msgid: Buffer.alloc(0),
+      msgstr: encoding.convert(generateHeader(this._table.headers), this._table.charset)
+    });
+  }
 
   Object.keys(this._table.translations).forEach(msgctxt => {
     if (typeof this._table.translations[msgctxt] !== 'object') {
@@ -148,20 +165,19 @@
 /**
  * Calculate buffer size for the final binary object
  *
- * @param {import('./index.d.ts').GetTextTranslations} list An array of translation strings from _generateList
- * @return {Object} Size data of {msgid, msgstr, total}
+ * @param {import('./types.js').GetTextTranslation[]} list An array of translation strings from _generateList
+ * @return {{msgid: number, msgstr: number, total: number}} Size data of {msgid, msgstr, total}
  */
 Compiler.prototype._calculateSize = function (list) {
   let msgidLength = 0;
   let msgstrLength = 0;
-  let totalLength = 0;
 
   list.forEach(translation => {
     msgidLength += translation.msgid.length + 1; // + extra 0x00
     msgstrLength += translation.msgstr.length + 1; // + extra 0x00
   });
 
-  totalLength = 4 + // magic number
+  const totalLength = 4 + // magic number
         4 + // revision
         4 + // string count
         4 + // original string table offset
@@ -183,52 +199,53 @@
 /**
  * Generates the binary MO object from the translation list
  *
- * @param {import('./index.d.ts').GetTextTranslations} list translation list
- * @param {Object} size Byte size information
- * @return {Buffer} Compiled MO object
+ * @param {import('./types.js').GetTextTranslation[]} list translation list
+ *  @param {{ msgid: number; msgstr: number; total: number }} size Byte size information
+ *  @return {Buffer} Compiled MO object
  */
 Compiler.prototype._build = function (list, size) {
+  /** @type {Buffer} returnBuffer */
   const returnBuffer = Buffer.alloc(size.total);
   let curPosition = 0;
   let i;
   let len;
 
   // magic
-  returnBuffer[this._writeFunc](this.MAGIC, 0);
+  returnBuffer.writeUInt32LE(this.MAGIC, 0);
 
   // revision
-  returnBuffer[this._writeFunc](0, 4);
+  returnBuffer.writeUInt32LE(0, 4);
 
   // string count
-  returnBuffer[this._writeFunc](list.length, 8);
+  returnBuffer.writeUInt32LE(list.length, 8);
 
   // original string table offset
-  returnBuffer[this._writeFunc](28, 12);
+  returnBuffer.writeUInt32LE(28, 12);
 
   // translation string table offset
-  returnBuffer[this._writeFunc](28 + (4 + 4) * list.length, 16);
+  returnBuffer.writeUInt32LE(28 + (4 + 4) * list.length, 16);
 
   // hash table size
-  returnBuffer[this._writeFunc](0, 20);
+  returnBuffer.writeUInt32LE(0, 20);
 
   // hash table offset
-  returnBuffer[this._writeFunc](28 + (4 + 4) * list.length * 2, 24);
+  returnBuffer.writeUInt32LE(28 + (4 + 4) * list.length * 2, 24);
 
-  // build originals table
+  // Build original table
   curPosition = 28 + 2 * (4 + 4) * list.length;
   for (i = 0, len = list.length; i < len; i++) {
     list[i].msgid.copy(returnBuffer, curPosition);
-    returnBuffer[this._writeFunc](list[i].msgid.length, 28 + i * 8);
-    returnBuffer[this._writeFunc](curPosition, 28 + i * 8 + 4);
+    returnBuffer.writeUInt32LE(list[i].msgid.length, 28 + i * 8);
+    returnBuffer.writeUInt32LE(curPosition, 28 + i * 8 + 4);
     returnBuffer[curPosition + list[i].msgid.length] = 0x00;
     curPosition += list[i].msgid.length + 1;
   }
 
-  // build translations table
+  // build translation table
   for (i = 0, len = list.length; i < len; i++) {
     list[i].msgstr.copy(returnBuffer, curPosition);
-    returnBuffer[this._writeFunc](list[i].msgstr.length, 28 + (4 + 4) * list.length + i * 8);
-    returnBuffer[this._writeFunc](curPosition, 28 + (4 + 4) * list.length + i * 8 + 4);
+    returnBuffer.writeUInt32LE(list[i].msgstr.length, 28 + (4 + 4) * list.length + i * 8);
+    returnBuffer.writeUInt32LE(curPosition, 28 + (4 + 4) * list.length + i * 8 + 4);
     returnBuffer[curPosition + list[i].msgstr.length] = 0x00;
     curPosition += list[i].msgstr.length + 1;
   }
@@ -237,8 +254,9 @@
 };
 
 /**
- * Compiles translation object into a binary MO object
+ * Compiles a translation object into a binary MO object
  *
+ * @interface
  * @return {Buffer} Compiled MO object
  */
 Compiler.prototype.compile = function () {