From 2b8ed2016a47f0becf347dc8c0c3c69171922581 Mon Sep 17 00:00:00 2001
From: Ben Hollis <ben@benhollis.net>
Date: Thu, 16 May 2024 11:58:44 -0700
Subject: [PATCH] Put back tag_cleanup

Edited from the GitHub UI, might be wrong. This reverts #213
---
 api/routes/update.ts | 23 +++++++++++------------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/api/routes/update.ts b/api/routes/update.ts
index 2defd0b..da3a830 100644
--- a/api/routes/update.ts
+++ b/api/routes/update.ts
@@ -2,7 +2,7 @@ import asyncHandler from 'express-async-handler';
 import { ClientBase } from 'pg';
 import { transaction } from '../db/index.js';
 import {
-  // deleteItemAnnotationList,
+  deleteItemAnnotationList,
   updateItemAnnotation as updateItemAnnotationInDb,
 } from '../db/item-annotations-queries.js';
 import { updateItemHashTag as updateItemHashTagInDb } from '../db/item-hash-tags-queries.js';
@@ -362,18 +362,17 @@ async function updateItemAnnotation(
 }
 
 async function tagCleanup(
-  _client: ClientBase,
-  _bungieMembershipId: number,
-  _inventoryItemIds: string[],
+  client: ClientBase,
+  bungieMembershipId: number,
+  inventoryItemIds: string[],
 ): Promise<ProfileUpdateResult> {
-  // TEMPORARY: Disable tag cleanup while bungie.net is being weird
-  //const start = new Date();
-  //await deleteItemAnnotationList(
-  //  client,
-  //  bungieMembershipId,
-  //  inventoryItemIds.filter(isValidItemId),
-  //);
-  //metrics.timing('update.tagCleanup', start);
+  const start = new Date();
+  await deleteItemAnnotationList(
+    client,
+    bungieMembershipId,
+    inventoryItemIds.filter(isValidItemId),
+  );
+  metrics.timing('update.tagCleanup', start);
 
   return { status: 'Success' };
 }