i18n plugin: return actual translated count, more logging

Author: ivictbor

adminforth/documentation/docs/tutorial/05-Plugins/10-i18n.md

@@ -109,7 +109,7 @@ export default {
     {
       name: "en_string",
       type: AdminForthDataTypes.STRING,
-      title: 'English',
+      label: 'English',
     },
     {
       name: "created_at",
@@ -118,17 +118,17 @@ export default {
     {
       name: "uk_string",
       type: AdminForthDataTypes.STRING,
-      title: 'Ukrainian',
+      label: 'Ukrainian',
     },
     {
       name: "ja_string",
       type: AdminForthDataTypes.STRING,
-      title: 'Japanese',
+      label: 'Japanese',
     },
     {
       name: "fr_string",
       type: AdminForthDataTypes.STRING,
-      title: 'French',
+      label: 'French',
     },
     {
       name: "completedLangs",

adminforth/plugins/i18n/index.ts

@@ -318,18 +318,22 @@ export default class I18N extends AdminForthPlugin {
           confirm: 'Are you sure you want to translate selected items?',
           state: 'selected',
           action: async ({ selectedIds, tr }) => {
+            let translatedCount = 0;
             try {
-              await this.bulkTranslate({ selectedIds });
+              translatedCount = await this.bulkTranslate({ selectedIds });
             } catch (e) {
               if (e instanceof AiTranslateError) {
+                process.env.HEAVY_DEBUG && console.error('🪲⛔ bulkTranslate error', e);
                 return { ok: false, error: e.message };
               }
             }
-            console.log('🪲bulkTranslate done', selectedIds);
+            process.env.HEAVY_DEBUG && console.log('🪲bulkTranslate done', selectedIds);
             this.updateUntranslatedMenuBadge();
             return { 
               ok: true, error: undefined, 
-              successMessage: await tr(`Translated {count} items`, 'backend', {count: selectedIds.length}),
+              successMessage: await tr(`Translated {count} items`, 'backend', {
+                count: translatedCount,
+              }),
             };
           }
         }
@@ -411,7 +415,7 @@ export default class I18N extends AdminForthPlugin {
         let totalTranslated = 0;
         for (let i = 0; i < strings.length; i += maxKeysInOneReq) {
           const slicedStrings = strings.slice(i, i + maxKeysInOneReq);
-          console.log('🪲🔪slicedStrings ', slicedStrings);
+          process.env.HEAVY_DEBUG && console.log('🪲🔪slicedStrings len', slicedStrings.length);
           totalTranslated += await translateToLang(langIsoCode, slicedStrings, plurals);
         }
         return totalTranslated;
@@ -435,7 +439,7 @@ ${
 \`\`\`
 `;
 
-      process.env.HEAVY_DEBUG && console.log('llm prompt', prompt);
+      process.env.HEAVY_DEBUG && console.log('🧠 llm prompt', prompt);
 
       // call OpenAI
       const resp = await this.options.completeAdapter.complete(
@@ -444,7 +448,7 @@ ${
         300,
       );
 
-      process.env.HEAVY_DEBUG && console.log('llm resp', resp);
+      process.env.HEAVY_DEBUG && console.log('🧠 llm resp', resp);
 
       if (resp.error) {
         throw new AiTranslateError(resp.error);
@@ -460,7 +464,7 @@ ${
         res = resp.content.split("```json")[1].split("```")[0];
       } catch (e) {
         console.error('error in parsing OpenAI', resp);
-        return;
+        throw new AiTranslateError('Error in parsing OpenAI response');
       }
       res = JSON.parse(res);
 
@@ -486,7 +490,7 @@ ${
         }
       }
 
-      return res.length;
+      return Object.keys(updateStrings).length;
     }
 
     const langsInvolved = new Set(Object.keys(needToTranslateByLang));

adminforth/plugins/i18n/package-lock.json

@@ -1,12 +1,13 @@
 {
   "name": "@adminforth/i18n",
-  "version": "1.0.13",
+  "version": "1.0.18-next.1",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "type": "module",
   "scripts": {
-    "build": "tsc && rsync -av --exclude 'node_modules' custom dist/ && npm version patch",
-    "rollout": "npm run build && npm publish --access public",
+    "build": "tsc && rsync -av --exclude 'node_modules' custom dist/",
+    "rollout": "npm run build && npm version patch && npm publish --access public",
+    "rollout-next": "npm run build && npm version prerelease --preid=next && npm publish --tag next",
     "prepare": "npm link adminforth"
   },
   "keywords": [],