Skip to content

fix: correo de Cal.com, textos pegados y hora local (#394AA0) - #81

Draft
santmun wants to merge 1 commit into
mainfrom
cursor/fix-ticket-394aa0-7120
Draft

santmun wants to merge 1 commit into
mainfrom
cursor/fix-ticket-394aa0-7120

Conversation

@santmun

@santmun santmun commented Oct 8, 2026

Copy link
Copy Markdown
Owner

Qué cambia

Tres arreglos del ticket de soporte #394AA0, sin cambiar el comportamiento cuando Cal.com sí está configurado:

  1. Correo de Cal.com. scheduleAppointment solo se registra si hay CALCOM_API_KEY. Sin la llave, el esquema (pide attendeeEmail) ya no llega al modelo, así que el bot no pide un correo "para Cal.com" cuando las citas se agendan por otro lado. Con la llave, la tool y su descripción quedan igual.
  2. Textos pegados. Si el modelo escribe, usa una tool y vuelve a escribir, esos trozos se unían sin separador (…el nuevo horario 💛Ya quedó anotado). Ahora se unen con una línea en blanco. Un solo paso de texto no cambia.
  3. Hora local. El contexto del bot incluye la fecha y la frase La hora actual es HH:MM (24 h). Si existe BOT_TIMEZONE, esa es la zona; si no, sigue CALCOM_TIMEZONE o la de member/config.

Por qué

Ticket de soporte #394AA0 (versiones 1.0.75 en adelante). El miembro tenía que volver a parchar los tres a mano en cada actualización. El cliente final veía un pedido de correo que no correspondía, respuestas pegadas y saludos o reglas de anticipación sin la hora real.

Cómo lo probaste

  • pnpm test pasa (576 tests)
  • pnpm typecheck limpio
  • (si aplica) probado contra un bot real

Cobertura nueva:

  • Sin CALCOM_API_KEY (también llave en blanco) la tool no está en el registro ni en el canvas de Mi Agente. Con la llave, free y pro la registran y la descripción sigue pidiendo attendeeEmail para Cal.com.
  • Un turno con texto, tool y más texto devuelve los dos trozos separados por una línea en blanco. Un solo paso se deja igual.
  • A las 22:30 UTC, BOT_TIMEZONE=America/Bogota produce La hora actual es 17:30 y no usa Europe/Madrid. Sin BOT_TIMEZONE, la hora sigue la zona de Cal.com (15:00 UTC en Madrid → 17:00).

Checklist

  • No toqué la carpeta member/ (config de cada quien)
  • El PR es de un solo tema (enfocado y chico)
  • Si tu agente (Claude) hizo el PR, revisaste el diff tú mismo antes de abrirlo
  • No hay secrets ni API keys en el código
Open in Web Open in Cursor 

Sin CALCOM_API_KEY la tool de citas no se registra, así el bot no pide
un correo para Cal.com. El texto de varios pasos se une con una línea
en blanco, y el prompt incluye la hora actual en BOT_TIMEZONE.

Ticket #394AA0

Co-authored-by: santmun <santmun@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Oct 8, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants