Merge pull request #51 from outis1one/claude/read-repo-YMu21

fix: reveal pairs oscillating, bump 0.9.14

manifest.firefox.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "Silent Send",
-  "version": "0.9.13",
+  "version": "0.9.14",
   "description": "Intercepts personal info and substitutes it with user-defined replacements before sending to AI services.",
   "browser_specific_settings": {
     "gecko": {

manifest.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "Silent Send",
-  "version": "0.9.13",
+  "version": "0.9.14",
   "description": "Intercepts personal info and substitutes it with user-defined replacements before sending to AI services.",
   "permissions": [
     "storage",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "silent-send",
-  "version": "0.9.13",
+  "version": "0.9.14",
   "private": true,
   "license": "MIT",
   "description": "Browser extension that substitutes personal data before sending to AI services",

src/content/content.js

@@ -1473,13 +1473,17 @@
   // session, preventing false positives (e.g. "user" in AI prose).
   function buildRevealPairs() {
     const pairs = [];
+    const added = new Set();
 
     // Helper: only add if this substitute was actually sent
     function addIfUsed(from, to, caseSensitive) {
       if (!from || !to) return;
-      const entry = sessionSubstitutions.get(from.toLowerCase());
+      const key = from.toLowerCase();
+      if (added.has(key)) return;
+      const entry = sessionSubstitutions.get(key);
       if (entry) {
         pairs.push({ from, to, caseSensitive });
+        added.add(key);
       }
     }
 
@@ -1492,9 +1496,9 @@
       for (const e of (identity.emails || [])) {
         addIfUsed(e.substitute, e.real);
       }
-      for (const n of (identity.names || [])) {
+      // For names: DON'T add individual first/last — the smart pattern engine
-        addIfUsed(n.substitute, n.real);
+      // combines them (e.g. "Ademo Demo" for "John Smith"). The catch-all
-      }
+      // below picks up the combined form from sessionSubstitutions.
       for (const u of (identity.usernames || [])) {
         addIfUsed(u.substitute, u.real);
       }
@@ -1506,10 +1510,13 @@
       }
     }
 
-    // Also add auto-detect and auto-redact substitutions from this session
+    // Catch-all: add any session substitution not already covered above.
+    // This picks up combined names ("Ademo Demo" → "John Smith"),
+    // auto-detected PII, and auto-redacted secrets.
     for (const [key, entry] of sessionSubstitutions) {
-      if (!pairs.some(p => p.from.toLowerCase() === key)) {
+      if (!added.has(key)) {
         pairs.push({ from: entry.replaced, to: entry.original });
+        added.add(key);
       }
     }
 
@@ -1517,11 +1524,13 @@
     return pairs;
   }
 
-  // Cache — invalidate when config changes or new substitutions happen
+  // Cache — invalidate when identity/mappings change or new substitutions happen
+  // Settings-only updates (e.g. reveal toggle) do NOT invalidate
   let _revealPairsCache = null;
-  let _revealPairsCacheSize = 0;
   window.addEventListener('message', (event) => {
-    if (event.data?.type === 'ss:config-updated') _revealPairsCache = null;
+    if (event.data?.type === 'ss:config-updated') {
+      if (event.data.mappings || event.data.identity) _revealPairsCache = null;
+    }
     if (event.data?.type === 'ss:substitution-performed') _revealPairsCache = null;
   });
 

src/options/options.html

@@ -631,7 +631,7 @@
     </section>
 
     <footer>
-      <p>Silent Send v0.9.13</p>
+      <p>Silent Send v0.9.14</p>
       <p style="font-size:11px;color:#9ca3af;margin-top:6px;max-width:600px">
         Silent Send is a convenience tool, not a security guarantee. Third-party sites may change how they send data at any time, which can cause missed substitutions without warning. You are responsible for verifying your data before sending. See the <a href="https://github.com/outis1one/silent-send/blob/main/LICENSE" target="_blank" style="color:#6b7280">LICENSE</a> for full terms.
       </p>