Revert "fix: only commit 'Missions' on successful completion (#2866)"

This reverts commit f5c1b8359895df5966c50d9a6aac733b22c25450.

src/services/missionInventoryUpdateService.ts

@@ -310,6 +310,9 @@ export const addMissionInventoryUpdates = async (
                 }
                 break;
             }
+            case "Missions":
+                addMissionComplete(inventory, value);
+                break;
             case "LastRegionPlayed":
                 if (!(config.unfaithfulBugFixes?.ignore1999LastRegionPlayed && value === "1999MapName")) {
                     inventory.LastRegionPlayed = value;
@@ -1242,9 +1245,6 @@ export const addMissionRewards = async (
     if (missions && missions.Tag in ExportRegions) {
         const node = ExportRegions[missions.Tag];
 
-        // cannot add this with normal updates because { Tier: 1 } would mark the SP node as completed even on a failure
-        addMissionComplete(inventory, missions);
-
         //node based credit rewards for mission completion
         if (isEligibleForCreditReward(rewardInfo, missions, node)) {
             const levelCreditReward = getLevelCreditRewards(node);