Skip to content

Builderbot dev#1211

Merged
leifermendez merged 14 commits intocodigoencasa:builderbot-devfrom
joajo13:builderbot-dev
Feb 23, 2026
Merged

Builderbot dev#1211
leifermendez merged 14 commits intocodigoencasa:builderbot-devfrom
joajo13:builderbot-dev

Conversation

@joajo13
Copy link

@joajo13 joajo13 commented Feb 23, 2026

Que tipo de Pull Request es?

  • Mejoras
  • Docs / tests

Descripción

Corregi CI para el test. Se agrega el Provider Gupshup. El provider esta parcialmente creado. El CLI no esta completo, unicamente permite crear la instancia con memory, ningun otro tipo de memoria.

@leifermendez
Copy link
Member

buenas puedes. Asegurarte que funcione con todos. Tienes un script que genera todo los templates, no tienes que hacerlo a mano

@joajo13
Copy link
Author

joajo13 commented Feb 23, 2026

Ya validé la generación y pruebas de templates.
Todo pasa correctamente y, en el caso de Gupshup, solo se habilita intencionalmente la combinación --provider=gupshup --language=ts --database=memory, porque fue la única que implementé en esta iteración.
Por eso, el resto de combinaciones de Gupshup se omiten intencionalmente, mediante validateTemplateCombination en packages/cli/src/configuration/index.ts (GUPSHUP_SUPPORTED_TEMPLATE_COMBINATIONS).

@leifermendez leifermendez merged commit 7329b25 into codigoencasa:builderbot-dev Feb 23, 2026
@leifermendez
Copy link
Member

1.3.15-alpha.17

@joajo13
Copy link
Author

joajo13 commented Feb 25, 2026

Buenos dias Leifer, disculpa las molestias. Al intentar crear un bot con el Alpha se me rompe.

  • @builderbot/provider-gupshup (alpha.16 y alpha.17) se instala sin dist/ (solo trae README.md, LICENSE.md, package.json).
  • tsc/rollup no puede resolver @builderbot/provider-gupshup y lanza TS2307 en src/app.ts:4.
  • Con npm pack --dry-run: el tarball oficial no incluye los archivos compilados.

A que puede deberse el error?

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