fix: workflow & address lookup fetchNames

.github/workflows/test.yml

@@ -5,7 +5,7 @@ on: [push, pull_request]
 jobs:
   test:
     runs-on: ubuntu-latest
-    container: node:16
+    container: node:20
 
     services:
       mariadb:
@@ -30,10 +30,10 @@ jobs:
         uses: actions/checkout@v2
 
       - name: Install dependencies
-        run: npm install --legacy-peer-deps
+        run: pnpm install --frozen-lockfile
 
       - name: Run tests
-        run: npm test
+        run: pnpm run test
         env: 
           DB_HOST: mariadb
           DB_PORT: ${{ job.services.mariadb.ports[3306] }}

src/krist/addresses/lookup.ts

@@ -21,25 +21,32 @@
 
 import { QueryTypes, sql } from "@sequelize/core";
 import { Address, db } from "../../database/index.js";
+import { AddressWithNames } from "./index.js";
 
 export async function lookupAddresses(
   addressList: string[],
   fetchNames?: boolean
 ): Promise<Address[]> {
   if (fetchNames) {
-    return db.query(sql`
+    const rows: AddressWithNames[] = await db.query(sql`
       SELECT
         \`addresses\`.*,
         COUNT(\`names\`.\`id\`) AS \`names\`
       FROM \`addresses\`
       LEFT JOIN \`names\` ON \`addresses\`.\`address\` = \`names\`.\`owner\`
-      WHERE \`addresses\`.\`address\` IN (:addresses)
+      WHERE \`addresses\`.\`address\` IN :addresses
       GROUP BY \`addresses\`.\`address\`
       ORDER BY \`names\` DESC
     `, {
-      replacements: { addresses: addressList },
+      replacements: { addresses: sql.list(addressList) },
       type: QueryTypes.SELECT
     });
+
+    return rows.map(row => {
+      row.firstseen = new Date(row.firstseen);
+      row.names = Number(row.names);
+      return row;
+    });
   } else {
     return Address.findAll({ where: { address: addressList } });
   }