feat: null handling

src/commands/breakdown/render.tsx

@@ -20,6 +20,7 @@ import { translator } from '../../core/localization/translator';
 import calculateWN8 from '../../core/statistics/calculateWN8';
 import { StatFilters, filterStats } from '../../core/statistics/filterStats';
 import getWN8Percentile from '../../core/statistics/getWN8Percentile';
+import { UserError } from '../../hooks/userError';
 
 const ROWS_PER_PAGE = 8;
 const MAX_PAGES = 9;
@@ -37,7 +38,7 @@ export async function renderBreakdown(
   const { filteredOrder } = await filterStats(statsInPeriod, filters);
   const accountInfo = await getAccountInfo(region, id);
   const clanData = await getClanAccountInfo(region, id, ['clan']);
-  const tankStats = await getTankStats(region, id, locale);
+  const tankStats = await getTankStats(region, id);
   const filterDescriptions = await filtersToDescription(filters, locale);
   const orderedCurrentStats: AllStats[] = [];
   const orderedCareerStats: AllStats[] = [];
@@ -45,6 +46,10 @@ export async function renderBreakdown(
   const orderedCareerWN8: (number | undefined)[] = [];
   const orderedCareerWN8Full: (number | undefined)[] = [];
 
+  if (tankStats === null) {
+    throw new UserError(t`bot.common.errors.no_tank_stats`);
+  }
+
   filteredOrder.forEach((id) => {
     const tankAveragesEntry = awaitedTankAverages[id];
     const current = statsInPeriod.diff[id];

src/commands/ownedTanks.tsx

@@ -31,11 +31,16 @@ export const ownedTanksCommand = new Promise<CommandRegistry>((resolve) => {
       .addStringOption(addUsernameChoices),
 
     async handler(interaction) {
-      const { translate } = translator(interaction.locale);
+      const { t, translate } = translator(interaction.locale);
       const tier = Number(interaction.options.getString('tier'));
       const { id, region } = await resolvePlayerFromCommand(interaction);
       const accountInfo = await getAccountInfo(region, id);
-      const tankStats = await getTankStats(region, id, interaction.locale);
+      const tankStats = await getTankStats(region, id);
+
+      if (tankStats === null) {
+        return t`bot.common.errors.no_tank_stats`;
+      }
+
       const filteredTanks = (
         await Promise.all(
           tankStats.map(async (tankData) => ({

src/commands/research.tsx

@@ -126,7 +126,11 @@ export const researchCommand = new Promise<CommandRegistry>((resolve) => {
           ]);
         }
 
-        const tankStats = await getTankStats(region, id, interaction.locale);
+        const tankStats = await getTankStats(region, id);
+
+        if (tankStats === null) {
+          return t`bot.common.errors.no_tank_stats`;
+        }
 
         if (tankStats.some(({ tank_id }) => tank_id === targetTankId)) {
           return translate('bot.commands.research.errors.already_researched', [