create-panal-agent 0.6.0 → 0.6.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-panal-agent",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "Crea un agente de IA para Panal, funcionando y cobrando on-chain, en cinco minutos",
   "type": "module",
   "license": "MIT",

package/template/src/register.ts

@@ -25,15 +25,19 @@ const PERFIL = {
 
   // Una frase que diga qué resuelves. Concreta gana a genérica: "traduzco
   // documentación técnica EN<->ES" se contrata más que "asistente de IA".
-  description: 'Describe aquí qué hace tu agente',
+  description: 'CAMBIA-ESTO: una frase que diga qué resuelves',
 
-  // Por estas palabras te van a encontrar. Piensa en lo que escribiría alguien
-  // que necesita tu servicio, no en cómo describirías tú tu tecnología.
-  skills: ['skill-1', 'skill-2'],
+  // Por estas palabras te van a encontrar, y son las que deciden en qué
+  // categoría del mercado apareces. Piensa en lo que escribiría alguien que
+  // necesita tu servicio, no en cómo describirías tú tu tecnología.
+  skills: ['CAMBIA-ESTO', 'las-palabras-por-las-que-te-buscan'],
 
   // Tu endpoint público HTTPS. Sin esto el cliente no puede mandarte el brief
   // ni descargarse el resultado, así que el agente queda casi inútil.
-  botUrl: process.env.PUBLIC_URL ?? 'https://cambia-esto.example.com',
+  // `||` y no `??` a propósito: el .env trae `PUBLIC_URL=` vacío, y una cadena
+  // vacía NO la sustituye `??`. Con `??` el botUrl acababa siendo '' y la
+  // comprobación de "no has puesto tu URL" no saltaba nunca.
+  botUrl: process.env.PUBLIC_URL?.trim() || 'https://cambia-esto.example.com',
 };
 
 /** Lo que cobras por tarea. */
@@ -44,6 +48,109 @@ const MONEDA = NATIVE_CURRENCY;
 
 // ────────────────────────────────────────────────────────────────────────────
 
+/**
+ * La marca que lleva todo lo que trae la plantilla sin rellenar.
+ *
+ * Una sola palabra en un solo sitio, a propósito. La primera versión de esto
+ * guardaba una copia de cada texto de ejemplo para compararlos, y con el texto
+ * duplicado en dos puntos del archivo bastaba un buscar-y-reemplazar para
+ * cambiar los dos a la vez: el perfil quedaba "relleno" y la comprobación
+ * seguía dándolo por vacío, porque su copia había cambiado igual.
+ */
+const SIN_RELLENAR = /cambia-esto/i;
+
+/**
+ * Lo que falta por rellenar del perfil, o null si está listo.
+ *
+ * Registrarse con los valores de ejemplo no falla: te deja en el escaparate con
+ * una ficha que no dice nada. Y no es solo feo — las skills son por lo que el
+ * mercado te clasifica, así que con las de la plantilla acabas en el cajón por
+ * defecto y quien busca lo que tú haces no te encuentra. Le pasó a un agente
+ * real que estructuraba JSON y estaba archivado entre los de código.
+ *
+ * Se exporta para poder probarlo sin firmar nada.
+ */
+export function loQueFaltaDelPerfil(perfil: typeof PERFIL): string | null {
+  if (SIN_RELLENAR.test(perfil.botUrl)) {
+    return (
+      'falta tu URL pública. Ponla en PERFIL.botUrl, o en PUBLIC_URL del .env.\n' +
+      '  Sin endpoint no puedes recibir encargos ni entregar: el agente se queda de adorno.'
+    );
+  }
+  const desc = perfil.description.trim();
+  if (!desc || SIN_RELLENAR.test(desc)) {
+    return (
+      'falta tu descripción: sigue la de la plantilla.\n' +
+      '  Es la frase que lee quien decide si contratarte. Concreta gana a genérica.'
+    );
+  }
+  const skills = perfil.skills.map((s) => s.trim()).filter(Boolean);
+  if (!skills.length || skills.some((s) => SIN_RELLENAR.test(s))) {
+    return (
+      'faltan tus skills: siguen las de la plantilla.\n' +
+      '  Por esas palabras te encuentran, y son las que deciden en qué categoría\n' +
+      '  del mercado apareces. Con las de ejemplo no te encuentra nadie.'
+    );
+  }
+  return null;
+}
+
+/**
+ * ¿Tu endpoint responde, y es TUYO?
+ *
+ * Se pide `GET /agent.json`, que es lo que sirve tu propio servidor, y se
+ * compara la dirección que anuncia con la de esta wallet. Así se cazan las dos
+ * formas de registrar un agente roto: una URL que todavía no está levantada, y
+ * una URL que sí responde pero es de otro (copiada de un ejemplo, o de otro
+ * agente tuyo).
+ *
+ * Importa porque el estado que evita es el peor de todos: aparecer en el
+ * mercado, que alguien te contrate y que su encargo no llegue a ninguna parte.
+ * Su dinero se queda bloqueado hasta que vence el plazo.
+ */
+async function compruebaEndpoint(botUrl: string, yo: string): Promise<string | null> {
+  let url: string;
+  try {
+    url = new URL('/agent.json', botUrl).toString();
+  } catch {
+    return `PERFIL.botUrl no es una URL válida: ${botUrl}`;
+  }
+  if (!url.startsWith('https://')) {
+    return (
+      `tu endpoint no es https (${botUrl}).\n` +
+      '  Por ahí viaja el encargo del cliente con su firma, y en claro lo lee cualquiera.'
+    );
+  }
+
+  let res: Response;
+  try {
+    res = await fetch(url, { signal: AbortSignal.timeout(10_000) });
+  } catch (err) {
+    return (
+      `tu endpoint no responde (${url}).\n` +
+      `  ${err instanceof Error ? err.message : err}\n` +
+      '  Arranca el agente y expón el puerto con https ANTES de registrarte.'
+    );
+  }
+  if (!res.ok) return `tu endpoint respondió ${res.status} en ${url}, y debería devolver tu tarjeta.`;
+
+  let card: { agent?: string };
+  try {
+    card = (await res.json()) as { agent?: string };
+  } catch {
+    return `${url} no devuelve JSON. ¿Seguro que ahí está tu agente y no otra cosa?`;
+  }
+  if (!card.agent) return `${url} responde, pero no anuncia ninguna dirección. ¿Es tu agente de Panal?`;
+  if (card.agent.toLowerCase() !== yo.toLowerCase()) {
+    return (
+      `esa URL es de OTRO agente.\n` +
+      `  ${url} dice ser ${card.agent}\n` +
+      `  y tú te estás registrando como ${yo}.`
+    );
+  }
+  return null;
+}
+
 async function main(): Promise<void> {
   const key = process.env.AGENT_PRIVATE_KEY?.trim();
   if (!key || !/^0x[0-9a-fA-F]{64}$/.test(key)) {
@@ -53,16 +160,34 @@ async function main(): Promise<void> {
   const account = privateKeyToAccount(key as `0x${string}`);
   const panal = createPanalClient({ account, rpcUrl: process.env.RPC_URL });
 
-  if (PERFIL.botUrl.includes('cambia-esto')) {
-    console.error(
-      'Antes de registrarte, pon tu URL pública en PERFIL.botUrl (o en PUBLIC_URL del .env).\n' +
-        'Sin endpoint no puedes recibir encargos ni entregar: el agente se quedaría de adorno.',
-    );
+  // Lo primero, porque no cuesta nada y es lo que más se olvida.
+  const falta = loQueFaltaDelPerfil(PERFIL);
+  if (falta) {
+    console.error(`No te registro todavía: ${falta}\n\nEstá todo en src/register.ts, arriba del todo.`);
     process.exit(1);
   }
 
-  const balance = await panal.publicClient.getBalance({ address: account.address });
   console.log(`Wallet:  ${account.address}`);
+
+  // El endpoint ANTES que el saldo, y no al revés: levantar un servidor con
+  // https es la parte larga, y mandar gas la corta. Decirle a alguien que le
+  // falta MON cuando su agente ni siquiera responde le hace resolver lo fácil
+  // para chocarse con lo difícil después.
+  if (process.env.REGISTRO_SIN_COMPROBAR === '1') {
+    console.log('\n(REGISTRO_SIN_COMPROBAR=1: no compruebo tu endpoint. Tú sabrás.)');
+  } else {
+    const roto = await compruebaEndpoint(PERFIL.botUrl, account.address);
+    if (roto) {
+      console.error(
+        `\nNo te registro: ${roto}\n\n` +
+          'Si sabes lo que haces y quieres registrarte igual, repite con REGISTRO_SIN_COMPROBAR=1.',
+      );
+      process.exit(1);
+    }
+    console.log(`Endpoint: ${PERFIL.botUrl} responde y es tuyo.`);
+  }
+
+  const balance = await panal.publicClient.getBalance({ address: account.address });
   console.log(`Saldo:   ${formatEther(balance)} MON`);
   if (balance === 0n) {
     console.error('\nSin MON no puedes ni pagar el gas del registro. Manda un poco a esa dirección.');