Hay muchos prints en el código utilizados a modo de debug (imprimiendo cosas como "hola", "recibí algo") que estaría bueno eliminar o "formalizar" más. También hay inconsistencia en los logs ya que hay varios en español y varios en inglés. Otro detalle más a tener en cuenta es la cantidad de código comentado que debería eliminarse para un release final.

No hay un Graceful Quit en los clientes, ya que sólo se lanza una excepción de SigTerm pero no se liberan recursos al catchearla.

No se le informa al Puppeteer cuando los Puppets ya han recibido su ID, evitando así que cierre esas init queues.

Los clientes (Posts, Comments y Results) están definidos en el init.py del package cuando por buenas prácticas sería mejor tenerlos definidos en archivos propios.

No se chequea en todos los casos si el connect_retry devuelve una conexión válida o no, sólo se hace en los Puppet aunque debería ser un comportamiento compartido con otros nodos como los Clients y el Puppeteer.

Hay algunos assert en el código con los que hay que tener cuidado, ya que en caso de fallar la aserción el sistema rompería por un error. Estos casos sería mejor controlarlos manualmente y, en caso de detectar una falla, loguearla pero no hacer que el sistema se detenga.

Sería bueno contar con un README con instrucciones sobre cómo levantar y correr el sistema, para facilitar el análisis y la comprensión de un tercero que llegue al código.

No se descargan los bytes del meme con mejor análisis de sentimient

Cuando a través de una misma queue llegan distintos tipos de mensajes, se utilizan criterios muy ad-hoc para saber si vienen de un flujo u otro (por ejemplo, el Sentiment Meme chequea si hay un atributo meme_url o comment). Esto no es recomendable ya que vuelve al sistema menos escalable, si se empieza a agregar dicho campo en el otro flujo va a empezar a romper y va a ser difícil darse cuenta porque fue. En estos casos, es mejor tipar el mensaje directamente con algún ID especial (¿P para Posts, C para comments?) que sabemos que nunca va a cambiar, para evitar posibles problemas futuros.

Algunas variables a lo largo del código (cómo los parámetros de delay, backoff y retries de la conexión con Rabbit) están hardcodeados en lugar de ser configurables. Estos deberían permitirse cargar dinámicamente o al menos setearse con variables definidas en algún archivo de properties.