@l10nmonster/helpers-translated 3.1.0 → 3.1.1

package/CHANGELOG.md

@@ -1,3 +1,25 @@
+## @l10nmonster/helpers-translated [3.1.1](https://public-github/l10nmonster/l10nmonster/compare/@l10nmonster/helpers-translated@3.1.0...@l10nmonster/helpers-translated@3.1.1) (2025-12-23)
+
+
+### Bug Fixes
+
+* Improve type definitions and checks ([826b412](https://public-github/l10nmonster/l10nmonster/commit/826b412f0f7e761d404165a243b0c2b26c416ac1))
+
+## @l10nmonster/helpers-translated [3.1.1](https://public-github/l10nmonster/l10nmonster/compare/@l10nmonster/helpers-translated@3.1.0...@l10nmonster/helpers-translated@3.1.1) (2025-12-23)
+
+
+### Bug Fixes
+
+* Improve type definitions and checks ([826b412](https://public-github/l10nmonster/l10nmonster/commit/826b412f0f7e761d404165a243b0c2b26c416ac1))
+
+
+
+
+
+### Dependencies
+
+* **@l10nmonster/core:** upgraded to 3.1.1
+
 # @l10nmonster/helpers-translated [3.1.0](https://public-github/l10nmonster/l10nmonster/compare/@l10nmonster/helpers-translated@3.0.0...@l10nmonster/helpers-translated@3.1.0) (2025-12-20)
 
 

package/laraProvider.js

@@ -3,7 +3,6 @@ import { Credentials, Translator } from '@translated/lara';
 
 /**
  * @typedef {object} LaraProviderOptions
- * @extends ChunkedRemoteTranslationProviderOptions
  * @property {string} keyId - The Lara API key id. This is required.
  * @property {Promise<string>|string} [keySecret] - The Lara API key secret. Optional, but often needed for authentication.
  * @property {string | Array<string>} [adaptTo] - An optional single translation memory ID or an array of IDs to adapt translations to.
@@ -24,6 +23,7 @@ export class LaraProvider extends providers.ChunkedRemoteTranslationProvider {
      * @param {LaraProviderOptions} options - Configuration options for the provider.
      */
     constructor({ keyId, keySecret, adaptTo, glossaries, ...options }) {
+        // @ts-ignore - spread loses type info but parent class handles validation
         super({ maxChunkSize: 60, ...options }); // maximum number of strings sent to Lara is 128 including notes
         this.#keyId = keyId;
         this.#keySecret = keySecret;
@@ -36,11 +36,13 @@ export class LaraProvider extends providers.ChunkedRemoteTranslationProvider {
     }
 
     async #getLara() {
-        const credentials = new Credentials(this.#keyId, await (typeof this.#keySecret === 'function' ? this.#keySecret() : this.#keySecret));
+        // @ts-ignore - keySecret can be a function or value, TypeScript doesn't narrow correctly
+        const resolvedSecret = await (typeof this.#keySecret === 'function' ? this.#keySecret() : this.#keySecret);
+        const credentials = new Credentials(this.#keyId, resolvedSecret);
         return new Translator(credentials);
     }
 
-    prepareTranslateChunkArgs({ sourceLang, targetLang, xmlTus, instructions }) {
+    prepareTranslateChunkArgs({ sourceLang, targetLang, xmlTus, jobGuid, chunkNumber, instructions }) {
         const payload = xmlTus.map(xmlTu => {
             const textBlock = [];
             textBlock.push({ text: `bundle: ${xmlTu.bundle} key: ${xmlTu.key} notes: ${xmlTu.notes ?? ''}`, translatable: false });
@@ -48,7 +50,7 @@ export class LaraProvider extends providers.ChunkedRemoteTranslationProvider {
             return textBlock;
         }).flat(1);
         const translateOptions = instructions ? { ...this.#translateOptions, instructions: [...this.#translateOptions.instructions, instructions] } : this.#translateOptions;
-        return { payload, sourceLang, targetLang, translateOptions };
+        return { payload, sourceLang, targetLang, xmlTus, jobGuid, chunkNumber, translateOptions };
     }
 
     async startTranslateChunk(args) {

package/mmtProvider.js

@@ -3,7 +3,6 @@ import { ModernMT as MMTClient } from 'modernmt';
 
 /**
  * @typedef {object} MMTProviderOptions
- * @extends ChunkedRemoteTranslationProviderOptions
  * @property {string} [id] - Global identifier (by default 'MMTBatch' or 'MMTRealtime')
  * @property {Promise<string>|string} apiKey - The ModernMT API key.
  * @property {string} [webhook] - The webhook URL for batch translation.
@@ -26,6 +25,7 @@ export class MMTProvider extends providers.ChunkedRemoteTranslationProvider {
      */
     constructor({ id, apiKey, webhook, chunkFetcher, hints, glossaries, ignoreGlossaryCase, multiline = true, ...options }) {
         id ??= webhook ? 'MMTBatch' : 'MMTRealtime';
+        // @ts-ignore - spread loses type info but parent class handles validation
         super({ id, ...options });
         if (webhook) {
             if (chunkFetcher) {
@@ -51,8 +51,11 @@ export class MMTProvider extends providers.ChunkedRemoteTranslationProvider {
         return {
             sourceLang,
             targetLang,
+            xmlTus,
+            jobGuid,
+            chunkNumber,
             q: xmlTus.map(xmlTu => xmlTu.source),
-            hints:this.baseRequest.hints,
+            hints: this.baseRequest.hints,
             contextVector: undefined,
             options: this.baseRequest.options,
             webhook: this.baseRequest.webhook,
@@ -67,7 +70,9 @@ export class MMTProvider extends providers.ChunkedRemoteTranslationProvider {
     async startTranslateChunk(args) {
         const { sourceLang, targetLang, q, hints, contextVector, options, webhook, batchOptions } = args;
         try {
-            const mmt = new MMTClient(await (typeof this.#apiKey === 'function' ? this.#apiKey() : this.#apiKey), 'l10n.monster/MMT', '3.0');
+            // @ts-ignore - apiKey can be a function or value, TypeScript doesn't narrow correctly
+            const resolvedKey = await (typeof this.#apiKey === 'function' ? this.#apiKey() : this.#apiKey);
+            const mmt = new MMTClient(resolvedKey, 'l10n.monster/MMT', '3.0');
             if (webhook) {
                 const response = await mmt.batchTranslate(webhook, sourceLang, targetLang, q, hints, contextVector, batchOptions);
                 return { enqueued: response };

package/package.json

@@ -1,16 +1,17 @@
 {
     "name": "@l10nmonster/helpers-translated",
-    "version": "3.1.0",
+    "version": "3.1.1",
     "description": "Helpers to integrate with Translated.com",
     "main": "index.js",
     "type": "module",
     "scripts": {
-        "test": "node --test"
+        "test": "node --test",
+        "typecheck": "tsc --noEmit"
     },
     "author": "Diego Lagunas",
     "license": "MIT",
     "peerDependencies": {
-        "@l10nmonster/core": "3.1.0"
+        "@l10nmonster/core": "3.1.1"
     },
     "dependencies": {
         "@translated/lara": "^1.4.0",

package/tsconfig.json

@@ -0,0 +1,18 @@
+{
+  "extends": "../tsconfig.base.json",
+  "include": [
+    "*.js",
+    "**/*.js"
+  ],
+  "exclude": [
+    "node_modules",
+    "**/node_modules",
+    "test/**",
+    "tests/**",
+    "**/*.test.js",
+    "**/*.spec.js",
+    "dist/**",
+    "ui/**",
+    "types/**"
+  ]
+}

package/.releaserc.json

@@ -1,31 +0,0 @@
-{
-  "branches": [
-    "main",
-    {
-      "name": "next",
-      "prerelease": "alpha"
-    },
-    {
-      "name": "beta",
-      "prerelease": "beta"
-    }
-  ],
-  "tagFormat": "@l10nmonster/helpers-translated@${version}",
-  "plugins": [
-    "@semantic-release/commit-analyzer",
-    "@semantic-release/release-notes-generator",
-    {
-      "path": "@semantic-release/changelog",
-      "changelogFile": "CHANGELOG.md"
-    },
-    {
-      "path": "@semantic-release/npm",
-      "npmPublish": false
-    },
-    {
-      "path": "@semantic-release/git",
-      "assets": ["CHANGELOG.md", "package.json"],
-      "message": "chore(release): @l10nmonster/helpers-translated@${nextRelease.version} [skip ci]\n\n${nextRelease.notes}"
-    }
-  ]
-}