champollion 0.3.4 → 0.4.0

package/lib/diff.js

@@ -130,15 +130,49 @@ function diffLocale(sourceFlat, targetFlat, fallbackPrefix = '[EN] ', forceKeys
 }
 
 /**
- * Generate a human-readable label for the diff result.
+ * The queue's reasons with each key under ONE of them — the first that
+ * applies, in this order: missing, [EN] fallback, untranslated (source
+ * echo), changed, forced. A key named for a redo that is also missing is
+ * "missing" (Round 10, i18next persona: verify's one-key repair was logged
+ * "1 missing + 1 forced"). `diff` itself keeps the full lists — the named
+ * keys still drive what a redo replaces.
+ *
+ * @param {{ missing: string[], needsTranslation: string[], untranslated?: string[], changed?: string[], forced?: string[] }} diff
+ * @returns {{ missing: string[], needsTranslation: string[], untranslated: string[], changed: string[], forced: string[] }}
+ */
+function queueReasons(diff) {
+  const seen = new Set();
+  const take = (keys) => (keys || []).filter((k) => {
+    if (seen.has(k)) return false;
+    seen.add(k);
+    return true;
+  });
+  return {
+    missing: take(diff.missing),
+    needsTranslation: take(diff.needsTranslation),
+    untranslated: take(diff.untranslated),
+    changed: take(diff.changed),
+    forced: take(diff.forced),
+  };
+}
+
+/**
+ * Generate a human-readable label for the diff result. Each queued key is
+ * counted once, under its first reason (queueReasons).
  *
  * @param {import('./types.js').DiffResult} diff - Diff result from diffLocale
  * @returns {string} Human-readable summary (e.g., '3 missing + 1 [EN] fallback(s)')
  */
 function diffLabel(diff) {
-  const { missing, needsTranslation, untranslated, changed, forced, noTranslate } = diff;
+  const { noTranslate } = diff;
+  const { missing, needsTranslation, untranslated, changed, forced } = queueReasons(diff);
   const parts = [];
-  if (missing.length > 0) parts.push(`${missing.length} missing`);
+  // gettext entries flagged fuzzy read as missing (lib/po.js); the caller
+  // passes them as `fuzzy` (a subset of `missing`) so they are named as such.
+  const fuzzy = (diff.fuzzy || []).filter(k => missing.includes(k));
+  const absent = missing.length - fuzzy.length;
+  if (absent > 0) parts.push(`${absent} missing`);
+  if (fuzzy.length > 0) parts.push(`${fuzzy.length} fuzzy (source changed — ${diff.fuzzyVerb || 're-translating'})`);
   if (needsTranslation.length > 0) parts.push(`${needsTranslation.length} [EN] fallback(s)`);
   if (untranslated && untranslated.length > 0) parts.push(`${untranslated.length} untranslated`);
   if (changed && changed.length > 0) parts.push(`${changed.length} changed`);
@@ -152,4 +186,4 @@ function diffLabel(diff) {
   return 'fully synced';
 }
 
-export { diffLocale, diffLabel };
+export { diffLocale, diffLabel, queueReasons };