Make collectItems() more accurate

collectItems() was only checking if the item could be obtained. We need
to verify that we can leave the room the item is in, which wasn't being
accounted for before this check-in.

Author: BashPrime

src/electron/models/prime/world.ts

@@ -192,8 +192,19 @@ export class PrimeWorld extends World {
       const filledItemLocations = searchResults.getLocations().filter(location => location.hasItem());
 
       const searchLocations = new LocationCollection(filledItemLocations.filter(location => {
+        const visitedRegion = searchResults.getVisitedRegion(location.getParentRegion());
+
+        // Can't get the item, no point running the additional checks.
+        if (!location.itemRule(myItems, this.settings)) {
+          return false;
+        }
+
+        // If we can obtain the item, can we exit?
+        const locationExit = visitedRegion.entryPoint
+          ? visitedRegion.entryPoint.getOpposite()
+          : null;
 
-        return location.itemRule(myItems, this.settings);
+        return !locationExit || locationExit.accessRule(new PrimeItemCollection([...myItems.toArray(), location.getItem()]), this.settings);
       }));
 
       const foundItems = new PrimeItemCollection(searchLocations.getItems().toArray());