fix: only commit 'Missions' on successful completion #2866

Merged
Sainan merged 1 commits from sp-node-fix into main 2025-10-06 22:57:19 -07:00

View File

@ -309,9 +309,6 @@ export const addMissionInventoryUpdates = async (
} }
break; break;
} }
case "Missions":
addMissionComplete(inventory, value);
break;
case "LastRegionPlayed": case "LastRegionPlayed":
if (!(config.unfaithfulBugFixes?.ignore1999LastRegionPlayed && value === "1999MapName")) { if (!(config.unfaithfulBugFixes?.ignore1999LastRegionPlayed && value === "1999MapName")) {
inventory.LastRegionPlayed = value; inventory.LastRegionPlayed = value;
@ -1208,6 +1205,9 @@ export const addMissionRewards = async (
if (missions && missions.Tag in ExportRegions) { if (missions && missions.Tag in ExportRegions) {
const node = ExportRegions[missions.Tag]; 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 //node based credit rewards for mission completion
if (isEligibleForCreditReward(rewardInfo, missions, node)) { if (isEligibleForCreditReward(rewardInfo, missions, node)) {
const levelCreditReward = getLevelCreditRewards(node); const levelCreditReward = getLevelCreditRewards(node);