Merge pull request #237 from transifex/TX-16599_1

TX-16599: Native push --verbose command group created, updated, skipped, failed strings

Author: pablotransifex

packages/cli/src/api/utils.js

@@ -23,6 +23,26 @@ function mergeArrays(array1, array2) {
   return _.uniq(_.concat(array1 || [], array2 || []));
 }
 
+/**
+ * Normalize input to array format
+ *
+ * @param {*} value - Input that might be an array, string, or other type
+ * @returns {Array} Normalized array
+ */
+function normalizeArray(value) {
+  if (!value) return [];
+  if (Array.isArray(value)) return value;
+  if (typeof value === 'string') {
+    try {
+      const parsed = JSON.parse(value);
+      return Array.isArray(parsed) ? parsed : [];
+    } catch (_e) {
+      return [];
+    }
+  }
+  return [];
+}
+
 /**
  * Async/await sleep
  *
@@ -40,5 +60,6 @@ function sleep(msec) {
 module.exports = {
   stringToArray,
   mergeArrays,
+  normalizeArray,
   sleep,
 };

packages/cli/src/commands/push.js

@@ -11,7 +11,7 @@ const { CliUx } = require('@oclif/core');
 const { extractPhrases } = require('../api/extract');
 const { uploadPhrases, pollJob } = require('../api/upload');
 const { mergePayload } = require('../api/merge');
-const { stringToArray } = require('../api/utils');
+const { stringToArray, normalizeArray } = require('../api/utils');
 
 /**
  * Test if path is folder
@@ -161,6 +161,41 @@ class PushCommand extends Command {
         : 'Uploading content to Transifex';
 
       this.log('');
+
+      /**
+       * Print detailed verbose output for a group of items
+       *
+       * @param {String} label - Label for the group
+       * @param {Array} items - Items to display
+       * @param {String} groupColor - Color for the group label
+       */
+      const printVerboseGroup = (label, items, groupColor = 'white') => {
+        const arr = normalizeArray(items);
+        if (!arr.length) return;
+        this.log(`  ${label}: ${arr.length}`[groupColor]);
+        arr.forEach((item) => {
+          const {
+            key = '',
+            string = '',
+            context = [],
+            occurrences = [],
+          } = item;
+          if (string) {
+            if (key !== string) {
+              this.log(`    └─ ${key}: ${string.underline}`);
+            } else {
+              this.log(`    └─ ${string.underline}`);
+            }
+            if (occurrences.length) {
+              this.log(`      └─ occurrences: ${occurrences.join(', ')}`.gray);
+            }
+            if (context.length) {
+              this.log(`      └─ context: ${context}`.gray);
+            }
+          }
+        });
+      };
+
       CliUx.ux.action.start(uploadMessage, '', { stdout: true });
       try {
         let res = await uploadPhrases(payload, {
@@ -207,20 +242,29 @@ class PushCommand extends Command {
         if (status === 'completed') {
           CliUx.ux.action.stop('Success'.green);
           this.log(`${'✓'.green} Successfully pushed strings to Transifex:`);
-          if (res.created > 0) {
-            this.log(`  Created strings: ${res.created.toString().green}`);
-          }
-          if (res.updated > 0) {
-            this.log(`  Updated strings: ${res.updated.toString().green}`);
-          }
-          if (res.skipped > 0) {
-            this.log(`  Skipped strings: ${res.skipped.toString().green}`);
-          }
-          if (res.deleted > 0) {
-            this.log(`  Deleted strings: ${res.deleted.toString().green}`);
-          }
-          if (res.failed > 0) {
-            this.log(`  Failed strings: ${res.failed.toString().red}`);
+
+          if (res.verbose && flags.verbose) {
+            printVerboseGroup('Created strings', res.verbose.created, 'green');
+            printVerboseGroup('Updated strings', res.verbose.updated, 'yellow');
+            printVerboseGroup('Deleted strings', res.verbose.deleted, 'red');
+            printVerboseGroup('Skipped strings', res.verbose.skipped, 'green');
+            printVerboseGroup('Failed strings', res.verbose.failed, 'red');
+          } else {
+            if (res.created > 0) {
+              this.log(`  Created strings: ${res.created}`.green);
+            }
+            if (res.updated > 0) {
+              this.log(`  Updated strings: ${res.updated}`.yellow);
+            }
+            if (res.deleted > 0) {
+              this.log(`  Deleted strings: ${res.deleted}`.red);
+            }
+            if (res.skipped > 0) {
+              this.log(`  Skipped strings: ${res.skipped}`.green);
+            }
+            if (res.failed > 0) {
+              this.log(`  Failed strings: ${res.failed}`.red);
+            }
           }
         } else {
           CliUx.ux.action.stop('Failed'.red);