Refactor import logic

[mFragaBA]

Create an index file for each submodule that re-exports everything in the directory. This simplifies the code in the client a bit, and avoids using require in the middle of the Client initialization.

[josuebouchard]

Hola! Hace varios dias que se inicio este PR y no me gusta tenerlo colgado. Para poder avanzar y que esto no se estanque vamos a hacer lo siguiente:

• Puede ser que haya faltado cerrar la función?
• Sip, bien visto! Ahora lo arreglo

• Como ya habiamos hablado que habia un typo en esa funcion, commiteo la sugerencia que habia propuesto para resolverlo, asi no tenes que hacer otro commit más. Si de repente te das cuenta que malinterprete el typo, podes hacer otro commit tranquilamente.

• Pongo este PR listo para revisión y doy mi aprobado del PR ya que me parece buena la idea y luego del fix no le veo ningun error, pero voy a pedirle a @ilitteri que de el ultimo visto bueno para ver si esta de acuerdo con esta modificación.

Desde ya, gracias por el trabajo, en lo posible dentro de la semana voy a intentar de poder concluir este PR.

[josuebouchard]

Pido disculpas por el cambio repentino de curso, clonando el repo y analizando el PR en profundidad, note que todavia se presentan varios problemas, y varios importantes:

1. Existen muchos imports y exports que estan mal hechos. Cuando se importa o exporta algo de un archivo no se agrega el .ts al final, y siempre se agrega la ruta relativa de importacion.

Ejemplos en src/commands/index.ts:
Ejemplo incorrecto: export * from 'EmbedMessageCommand.ts';
Ejemplo correcto: export * from './EmbedMessageCommand';

2. El hecho de realizar export * trae varios conflictos. Por ejemplo, en src/commands/index.ts, typescript tira el siguiente error:

Module './EmbedMessageCommand' has already exported a member named 'data'. Consider explicitly re-exporting to resolve the ambiguity.ts(2308)
Module './EmbedMessageCommand' has already exported a member named 'execute'. Consider explicitly re-exporting to resolve the ambiguity.ts(2308)

Esto proviene del hecho que todos los commands tienen la misma estructura general, todos exportan un data y un execute, con lo cual habria que ver como se soluciona eso.

3. El codigo tiene inconsistencias en el uso de comillas simples y dobles e inconsistencias en la indentacion. (Este ultimo punto se puede resolver corriendo npx prettify --write ./src estando en el directorio raiz del proyecto).

Los puntos 1. y 3. son de facil solucion, pero lo que queda ver es como solucionar el punto 2.
Quizas cambiar export * from './EmbedMessageCommand'; por export * as EmbedMessageCommand from './EmbedMessageCommand'; resuelva el tema, pero no estoy seguro.

Si podes resolver estos temas @mFragaBA, despues me encargo de seguir con vos la aprobacion del PR.

[josuebouchard]

Como ultima nota, este PR busca mergear a release/0.2.1. En este repo no hacemos eso, sino que mergeamos contra develop, y cuando esta todo listo, se hace un release. Voy a cambiar eso ahora para no olvidarnos en el futuro.