We should likely split this into two separate hook handlers, both of them only being registered when their respective options are on (triforce hunt & the randomized swimming)

I was thinking that it might be better to have only one hook handler for each hook in randomizer and split if off inside the hook to prevent registering too many hooks, if it's preferred to use separate hooks, I'll change it to that.

This is by far the most expensive hook we can register, we need to be very careful and precise about how and when this is registered. Each of the different target actors should be specified by the RegisterGameHookForID hook, and these hooks should only be registered when their respective options are on

eg

onFileLoad(() {
  // unregister skeletonKeyDoorUpdate

  if (skeletonKeyEnabled) {
    // Because we're doing RegisterGameHookForID this will only ever be called for door updates
    GameInteractor::Instance->RegisterGameHookForID<GameInteractor::OnActorUpdate>(ACTOR_EN_DOOR, skeletonKeyDoorUpdate);
  }
})

Similarly here, we should only register this hook handler when the infinite upgrades option is enabled in the current save

This doesn't seem to be working, I'm not getting the GCBK after reloading in after the credits sequence.

Oh I missed this, yeah this should be migrated to just doing a flag set and letting the queue handle the give like everything else

The problem is that it's not a check so we can't use randomizerQueuedChecks.

This might be scope creep, but this made me think of the "ganon boss key on LACS" situation, and how seperating the conditional GCBK gives from LACS would be a good idea. Making a new type of GCBK give would solve both that and this.

Upon second look at all the changes here in this file, I don't think we need any of them. You should be able to override the destination of the blue warp the same way we are for rando & cutscene skips with the VB_PLAY_TRANSITION_CS