chocolatito-code 1.0.0 → 1.1.0

package/extension/content.js

@@ -155,12 +155,43 @@
   function norm(s) {
     return (s || "")
       .normalize("NFD")
+      // Rango de diacriticos combinantes. Escrito con escapes y no con los
+      // caracteres literales: en el fuente eran invisibles y cualquier editor
+      // que "normalizara" el archivo se los llevaba por delante sin que nadie
+      // lo notara hasta que el filtro dejaba de casar acentos.
       .replace(/[̀-ͯ]/g, "")
       .toLowerCase();
   }
 
-  function buildSnapshot(startIndex, filter) {
-    if (startIndex === 0) refIdentity.clear();
+  /**
+   * Quita las marcas del snapshot anterior.
+   *
+   * data-choco-ref se escribia en los elementos de la pagina del usuario y no
+   * lo borraba nadie: los atributos se acumulaban snapshot tras snapshot, y un
+   * ref viejo podia seguir resolviendo a un elemento que ya no estaba en la
+   * lista actual.
+   */
+  function limpiarMarcas() {
+    for (const root of allRoots()) {
+      let marcados;
+      try {
+        marcados = root.querySelectorAll("[data-choco-ref]");
+      } catch (_) {
+        continue;
+      }
+      for (const el of marcados) el.removeAttribute("data-choco-ref");
+    }
+  }
+
+  function buildSnapshot(startIndex, filter, nuevo) {
+    // Antes esto era `startIndex === 0`, que solo es cierto en el PRIMER frame.
+    // Cada frame tiene su propio content script con su propia tabla, asi que
+    // los subframes no la limpiaban nunca: acumulaban identidades de snapshots
+    // viejos y un ref podia resolver a un boton que ya no era ese.
+    if (nuevo || startIndex === 0) {
+      refIdentity.clear();
+      limpiarMarcas();
+    }
     const out = [];
     let idx = startIndex;
     const f = norm(filter);
@@ -324,13 +355,52 @@
     return true;
   }
 
+  /**
+   * Codigos de tecla.
+   *
+   * La tabla tenia siete entradas y todo lo demas salia con keyCode 0, que para
+   * un manejador que mira `e.keyCode` o `e.which` -que son casi todos los que
+   * existen desde antes de 2018- equivale a "no se ha pulsado nada". Escribir
+   * una letra suelta o un Espacio con press no hacia absolutamente nada.
+   */
+  const CODIGOS = {
+    Enter: 13, Escape: 27, Esc: 27, Tab: 9, Backspace: 8, Delete: 46,
+    ArrowUp: 38, ArrowDown: 40, ArrowLeft: 37, ArrowRight: 39,
+    Home: 36, End: 35, PageUp: 33, PageDown: 34, Insert: 45,
+    " ": 32, Space: 32, Shift: 16, Control: 17, Alt: 18,
+    F1: 112, F2: 113, F3: 114, F4: 115, F5: 116, F6: 117,
+    F7: 118, F8: 119, F9: 120, F10: 121, F11: 122, F12: 123,
+  };
+
+  /** Nombre DOM del codigo fisico ("KeyA", "Digit1", "Space"...). */
+  function codigoFisico(key) {
+    if (key === " " || key === "Space") return "Space";
+    if (/^[a-zA-Z]$/.test(key)) return "Key" + key.toUpperCase();
+    if (/^[0-9]$/.test(key)) return "Digit" + key;
+    return key;
+  }
+
   function pressKey(el, key) {
     const target = el || document.activeElement || document.body;
-    const map = { Enter: 13, Escape: 27, Tab: 9, Backspace: 8, Delete: 46, ArrowUp: 38, ArrowDown: 40 };
-    const code = map[key] || 0;
-    const init = { key, code: key, keyCode: code, which: code, bubbles: true, cancelable: true, composed: true };
+    const nombre = key === "Space" ? " " : key;
+    const code = CODIGOS[key] !== undefined
+      ? CODIGOS[key]
+      : nombre.length === 1
+        ? nombre.toUpperCase().charCodeAt(0)
+        : 0;
+    const init = {
+      key: nombre,
+      code: codigoFisico(key),
+      keyCode: code,
+      which: code,
+      bubbles: true,
+      cancelable: true,
+      composed: true,
+    };
     target.dispatchEvent(new KeyboardEvent("keydown", init));
-    target.dispatchEvent(new KeyboardEvent("keypress", init));
+    // keypress solo existe para las teclas que producen caracter. Emitirlo para
+    // Escape o las flechas es inventarse un evento que el navegador nunca manda.
+    if (nombre.length === 1) target.dispatchEvent(new KeyboardEvent("keypress", init));
     target.dispatchEvent(new KeyboardEvent("keyup", init));
     // Si es un formulario y la tecla es Enter, se intenta enviar de verdad.
     if (key === "Enter" && target.form && typeof target.form.requestSubmit === "function") {
@@ -355,7 +425,7 @@
           break;
 
         case "snapshot": {
-          const items = buildSnapshot(msg.startIndex || 0, msg.filter || "");
+          const items = buildSnapshot(msg.startIndex || 0, msg.filter || "", msg.nuevo === true);
           sendResponse({ ok: true, items, url: location.href, title: document.title, top: IS_TOP });
           break;
         }
@@ -425,13 +495,26 @@
           break;
 
         case "eval": {
+          // Un objeto salia como "[object Object]", que no dice nada: quien
+          // pedia document.querySelectorAll('a').length lo tenia bien, pero
+          // quien pedia el objeto entero recibia ruido y volvia a intentarlo.
           let out;
           try {
-            out = String(eval(msg.script));
+            const valor = eval(msg.script);
+            if (valor === undefined) out = "undefined";
+            else if (valor === null) out = "null";
+            else if (typeof valor === "object") {
+              try {
+                out = JSON.stringify(valor, null, 2);
+                if (out === undefined) out = String(valor); // referencias circulares
+              } catch (_) {
+                out = String(valor);
+              }
+            } else out = String(valor);
           } catch (e) {
             out = "Error JS: " + (e && e.message ? e.message : e);
           }
-          sendResponse({ ok: true, result: out });
+          sendResponse({ ok: true, result: String(out).slice(0, 20000) });
           break;
         }
 

package/extension/manifest.json

@@ -1,7 +1,8 @@
 {
   "manifest_version": 3,
+  "minimum_chrome_version": "102",
   "name": "Chocolatito Code",
-  "version": "1.0.0",
+  "version": "1.2.0",
   "description": "Deja que Chocolatito Code trabaje dentro de tu Chrome, en segundo plano y con tus sesiones ya iniciadas.",
   "permissions": [
     "tabs",

package/extension/popup.js

@@ -1,12 +1,28 @@
-// Estado de la conexion con la CLI. El badge lo pone el service worker; aqui
-// solo se traduce a algo legible para el usuario.
-chrome.action.getBadgeText({}, (text) => {
-  const on = text === "on";
-  document.getElementById("dot").className = "dot" + (on ? " on" : "");
-  document.getElementById("state").textContent = on
-    ? "Conectado a la CLI"
-    : "Sin conexion con la CLI";
-  document.getElementById("hint").innerHTML = on
-    ? "El agente puede trabajar en tus pestanas en segundo plano."
-    : 'Arranca <code>chocolatito</code> en la terminal y esta pestana se conectara sola.';
-});
+/**
+ * Estado de la conexion con la CLI. El badge lo pone el service worker; aqui
+ * solo se traduce a algo legible.
+ *
+ * Aqui se comparaba el badge con la cadena "on", y el service worker nunca
+ * escribe eso: escribe el NUMERO de sesiones conectadas ("1", "2") o vacio. La
+ * comparacion no se cumplia jamas, asi que el popup decia "Sin conexion con la
+ * CLI" incluso con el agente trabajando delante. Es el unico sitio donde el
+ * usuario puede comprobar si la extension funciona, y siempre mentia.
+ */
+chrome.action.getBadgeText({}, (text) => {
+  const sesiones = Number(text);
+  const conectadas = Number.isFinite(sesiones) && sesiones > 0 ? sesiones : 0;
+
+  document.getElementById("dot").className = "dot" + (conectadas > 0 ? " on" : "");
+
+  document.getElementById("state").textContent =
+    conectadas === 0
+      ? "Sin conexion con la CLI"
+      : conectadas === 1
+        ? "Conectado a la CLI"
+        : `Conectado a ${conectadas} sesiones`;
+
+  document.getElementById("hint").innerHTML =
+    conectadas > 0
+      ? "El agente puede trabajar en tus pestanas en segundo plano."
+      : 'Arranca <code>chocolatito</code> en la terminal y esta pestana se conectara sola.';
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chocolatito-code",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "Agente autónomo de programación para la terminal, con control real del ordenador y del navegador. Desarrollado por Chocolatito.",
   "type": "module",
   "bin": {