fix: address code review feedback

jeanfbrito · jeanfbrito · commit 81b2d199154b · 2026-02-23T16:20:01.000-03:00
- privacy.ts: don't spread message in catch block to avoid leaking
  sensitive fields; return only safe metadata (level, date). Add one-time
  stderr warning when redaction fails for observability.
- store/index.ts: warn when watch() is called before store is initialized
  so callers aren't silently left in uninitialized state.
- hu.i18n.json: fix 'bolt' (shop) -&gt; 'tároló' (storage) for software
  store references. Add missing trailing period for consistency.
diff --git a/src/i18n/hu.i18n.json b/src/i18n/hu.i18n.json
@@ -293,7 +293,7 @@
       },
       "videoCallWindowPersistence": {
         "title": "A videohívás-ablak pozíciójának megjegyzése",
-        "description": "A videohívás-ablakok pozíciójának és méretének mentése és helyreállítása a munkamenetek között"
+        "description": "A videohívás-ablakok pozíciójának és méretének mentése és helyreállítása a munkamenetek között."
       },
       "transparentWindow": {
         "title": "Átlátszó ablak hatás",
@@ -497,12 +497,12 @@
       }
     },
     "status": {
-      "moduleNotAvailable": "A bolt modul nem érhető el a kiszolgáló környezetének leképezésénél",
-      "importFailed": "A bolt importálása nem sikerült"
+      "moduleNotAvailable": "A tároló modul nem érhető el a kiszolgáló környezetének leképezésénél",
+      "importFailed": "A tároló importálása nem sikerült"
     },
     "errors": {
       "configurationFailed": "Nem sikerült beállítani az elektronnaplót",
-      "storeUnavailable": "A bolt modul nem érhető el a kiszolgáló környezetének leképezésénél",
+      "storeUnavailable": "A tároló modul nem érhető el a kiszolgáló környezetének leképezésénél",
       "webContentsLoggingFailed": "Nem sikerült beállítani a webtartalmak naplózását",
       "rendererLogFailed": "Nem sikerült naplózni a megjelenítőből",
       "consoleOverrideFailed": "Nem sikerült felülbírálni a konzol metódusait"
diff --git a/src/logging/privacy.ts b/src/logging/privacy.ts
@@ -97,6 +97,8 @@ const redactObject = (obj: any, seen = new WeakSet()): any => {
   return obj;
 };
 
+let privacyHookFailed = false;
+
 export const createPrivacyHook = () => {
   return (message: any, _transport: any, _transportName?: string) => {
     try {
@@ -110,9 +112,15 @@ export const createPrivacyHook = () => {
       });
       return { ...message, data: sanitizedData };
     } catch {
-      // Don't emit raw data on failure - only safe placeholder
+      if (!privacyHookFailed) {
+        privacyHookFailed = true;
+        process.stderr.write(
+          '[privacy] redaction hook threw; falling back to placeholder\n'
+        );
+      }
       return {
-        ...message,
+        level: message.level,
+        date: message.date,
         data: ['[Privacy redaction failed]'],
       };
     }
diff --git a/src/store/index.ts b/src/store/index.ts
@@ -75,7 +75,10 @@ export const watch = <T>(
   selector: Selector<T>,
   watcher: (curr: T, prev: T | undefined) => void
 ): (() => void) => {
-  if (!reduxStore) return () => undefined;
+  if (!reduxStore) {
+    console.warn('[store] watch() called before store initialized');
+    return () => undefined;
+  }
   const initial = select(selector);
   watcher(initial, undefined);
 
