feat: named exports for esm users

[vaporoxx]

This PR modifies the way index.js re-exports everything so that ESM users can use named exports too. For reference, discord.js applied this exact change with discordjs/discord.js#6884.

Before, this resulted in the following:

import { CommandHandler } from 'discord-akairo';
         ^^^^^^^^^^^^^^
SyntaxError: Named export 'CommandHandler' not found. The requested module 'discord-akairo'
is a CommonJS module, which may not support all module.exports as named exports.

Resolves #122.

[GalGreenfield]

Looks great! I added it manually to my local installation, thank you!

[GalGreenfield]

Did you test this? 'cause it didn't work for me.

This doesn't support ESM completely - it does add exports to the various core files, since they, themeslves, have a dependency on various files, they all need to be ESM modules, themeslves, and therefore have to be renamed to .mjs to run on Node.js, and they need to have ES6 import/export syntax, per ESM specifications.

I'm working on converting the whole library's modules to ESM, with the addition of your exports.

[vaporoxx]

If you mean loading modules from your own files, that won't be implemented anyway (see #122 (comment)).

As a workaround, i'm loading all modules manually:

async loadModules() {
    for (const file of CommandHandler.readdirRecursive(this.commandHandler.directory)) {
        const { default: module } = await import(`../../${file}`);
        this.commandHandler.load(module);
    }

    for (const file of ListenerHandler.readdirRecursive(this.listenerHandler.directory)) {
        const { default: module } = await import(`../../${file}`);
        this.listenerHandler.load(module);
    }
}

[GalGreenfield]

The comment says:

import does not support reloading as of the moment. Its not a deal breaker, but I'd rather not touch it for now.

I don't understand - why and how is reloading of user-defined Command modules necessary?

Also, where do you use loadModules() - in your client?

[vaporoxx]

I don't understand - why and how is reloading of user-defined Command modules necessary?

¯\_(ツ)_/¯

Also, where do you use loadModules() - in your client?

I added it as a method to my client class, doesn't matter where/how you do it though

[GalGreenfield]

I don't understand - why and how is reloading of user-defined Command modules necessary?

¯_(ツ)_/¯

Also, where do you use loadModules() - in your client?

I added it as a method to my client class, doesn't matter where/how you do it though

Haha, okay. I guess I should ask the 1Computer1, then.
Thanks, that's what I thought I'll test it out and see if it works. 🙂

[GalGreenfield]

Also, just to make sure - your main file (index or whatever you called it) file and your client class file are both .mjs files and you're using type: "module" in your package.json, yes? I'm asking to make sure I'm reproducing exactly what you did.

[GalGreenfield]

Thanks, I tried it and it works great! :)