feat: add registered bot weights

Author: kyleratti

src/controllers/cron/commentreply.ts

@@ -105,7 +105,7 @@ async function processCommentUpdates(comment: CommentReply) {
   // FIXME: due to an issue with snoowrap typings, the 'await' keyword causes compile errors. see https://github.com/DefinitelyTyped/DefinitelyTyped/issues/33139
   post = await redditapi.getSubmission(comment.redditPostId_Parent).fetch();
 
-  const mirrors = await AvailableMirror.find({
+  let mirrors = await AvailableMirror.find({
     where: {
       redditPostId: comment.redditPostId_Parent,
     },
@@ -116,6 +116,14 @@ async function processCommentUpdates(comment: CommentReply) {
 
   if (mirrors.length <= 0) return deleteComment(comment);
 
+  // This is a dirty workaround for sorting mirrors by the
+  // weight of their registered bot. This really should be done
+  // with an SQL query, but I opted to do this instead because
+  // there is already a bunch of Active Record stuff in this
+  // model and I don't want to break pattern. That's probably a
+  // bad reason. In fact, I know it is.
+  mirrors = mirrors.sort((a, b) => a.bot.weight - b.bot.weight);
+
   let commentBody = generateFormattedMirrors(mirrors);
 
   let reply: RedditComment;

src/db/database.ts

@@ -6,7 +6,7 @@ let db = getConnectionManager().create({
   database: path.resolve(
     process.env.DATABASE_LOCATION || "data/database.sqlite3"
   ),
-  synchronize: true,
+  synchronize: false,
   logging: true,
   entities: [__dirname + "/../entity/**{.ts,.js}"],
   migrations: [__dirname + "/../migration/**{.ts,.js}"],

src/entity/registeredbot.ts

@@ -32,6 +32,11 @@ export class RegisteredBot extends BaseEntity {
   })
   token: string;
 
+  @Column({
+    default: 0,
+  })
+  weight: number;
+
   @OneToMany((type) => AvailableMirror, (mirroredvideo) => mirroredvideo.bot)
   mirroredVideos: AvailableMirror[];
 

src/migration/1609376826000-AddWeightToBots.ts

@@ -0,0 +1,15 @@
+import { MigrationInterface, QueryRunner } from "typeorm";
+
+export class AddWeightToBots1609376826000 implements MigrationInterface {
+  async up(queryRunner: QueryRunner): Promise<any> {
+    await queryRunner.query(`UPDATE registered_bot SET weight = 0`);
+
+    await queryRunner.query(
+      `UPDATE registered_bot SET weight = -1 WHERE username = 'tuckbot'`
+    );
+  }
+
+  async down(queryRunner: QueryRunner): Promise<any> {
+    await queryRunner.query(`UPDATE registered_bot SET weight = 0`);
+  }
+}

src/server.ts

@@ -51,6 +51,9 @@ export class WebServer {
   async start() {
     try {
       await db.connect();
+      await db.query("PRAGMA foreign_keys=OFF");
+      await db.synchronize();
+      await db.query("PRAGMA foreign_keys=ON");
 
       logger.info(`database sucecssfully started`);
     } catch (err) {