Merge pull request #3 from jaronpate/ft/v0.0.4

Added workflow and fixed bug with command handling

.github/workflows/ci.yml

@@ -1,4 +1,4 @@
-name: CI
+name: CI (template)
 on: workflow_dispatch
 jobs:
   build:

.github/workflows/publish.yml

@@ -4,14 +4,15 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v1
+      # Pull repo
-      - uses: actions/setup-node@v1
+      - uses: actions/checkout@v2
-        with:
+      # Install bun
-          node-version: 14
+      - uses: oven-sh/setup-bun@v2
-          registry-url: https://registry.npmjs.org/
+      # Run tests
-      - run: yarn
+      # TODO: Add tests
-      - run: yarn build
+      # Build
-      # - run: yarn test
+      - run: bun build
-      - run: yarn publish --access public
+      # Publish
+      - run: bun publish
         env:
-          NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
+          NPM_CONFIG_REGISTRY: https://registry.npmjs.org/_authToken=${{ secrets.NPM_TOKEN }}

package.json

@@ -1,7 +1,7 @@
 {
   "name": "unicord",
-  "description": "A simple discord library primarily for TypeScript. REST like and easy to use.",
+  "description": "A simple discord library primarily for TypeScript. Web server like and easy to use.",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "module": "src/index.ts",
   "type": "module",
   "main": "out/index.js",
@@ -17,7 +17,7 @@
     "build:packages": "bun run build:types && bun run build:core -- --packages external",
     "build:nocheck": "bun run build:types:nocheck && bun run build:core",
     
-    "build:core": "bun build src/index.ts --outdir out --drop=console --drop=debugger",
+    "build:core": "bun build src/index.ts --outdir out --drop=debugger",
     "build:types": "tsc --emitDeclarationOnly --project tsconfig.json",
     "build:types:nocheck": "bun run build:types -- --noCheck",
     

src/services/bus.ts

@@ -5,6 +5,7 @@ const EventBus = new EventEmitter() as Emitter;
 
 export type Emitter = EventEmitter & {
     emit: (event_name: string, context: Context | null, ...args: any[]) => boolean
+    on: (event_name: string, listener: Function) => Emitter
 }
 
 export { EventBus };

src/services/connectors/gateway.ts

@@ -205,15 +205,15 @@ export class Gateway {
             // Generate context
             const context = new Context(this.client, this.api, { message });
             // Dispatch event for the command
-            this.bus.emit(`command:${command}`, context, args);
+            // this.bus.emit(`${HandlerType.ChatCommands}:${command}`, context, args);
             // // Check if command exists
-            // if (this.processor[HandlerType.ChatCommands].has(command)) {
+            if (this.processor[HandlerType.ChatCommands].has(command)) {
-            //     // Execute the handler
+                // Execute the handler
-            //     this.processor[HandlerType.ChatCommands].execute(command, context, args);
+                this.processor[HandlerType.ChatCommands].execute(command, context, args);
-            // } else {
+            } else {
-            //     // TODO: Custom unknown command handler
+                // TODO: Custom unknown command handler
                 // context.reply(`Unknown command: ${command}`);
-            // }
+            }
         }
     };
 

src/services/processor.ts

@@ -64,6 +64,9 @@ export class Processor {
                         throw new Error('Event handlers cannot be a CommandHandler');
                     }
                     // Validate and resolve the args
+                    // TODO: There's no reason this can't use the hydrator system
+                    // Hydrator should have a more genric subcomponent that can be used anywhere and is more dumb.
+                    // Basically a function that accepts id's and returns values.
                     const resolvedArgs = await this.validateAndResolveArgs(argsOrPayload, handler.args);
                     // Execute the handler
                     await Promise.resolve(handler[Trait.execute](context!, resolvedArgs));