Closed catloversg closed 1 week ago
My personal stance has always been to lean towards (2). For instance, infil rewards could be scaled based on how many successful infiltrations you've done in the past 15 minutes. Done properly, it wouldn't affect manual infil rewards, while reducing the profit of automated infil to balancable levels. This would open the door to infil eventually becoming an official singularity API.
I think @Snarling needs to weigh in here.
As a player, I would really prefer to have some built-in ways to develop this myself rather than downloading someone else's script. I 100% expected to be able to do it as part of singularity.
It's already clear that automating infiltration is not officially supported:
Why not beef up the anti-automation protections?
Why not officially support automation?
Other thoughts:
The changes in this PR can also be worked around, so I'd be fine with merging this just to spice things up and give an extra hurdle to players who want to automate infiltration.
Infiltration has never been balanced to be played by an automation script. In our code base, we prohibit it with the checks in
src\Infiltration\ui\Game.tsx
andsrc\Infiltration\ui\KeyHandler.tsx
, but our checks are wrong:press
: It does not check ifevent
is a "real" event or a synthetic one.press
: It does not callonFailure({ automated: true })
, so the checkoptions?.automated
inonFailure
is useless.With these bugs, our standpoint on automated infiltration is weird. We prohibit it, but we don't do anything to prevent it. There are 3 ways to deal with it:
[3] is a big overhaul, so that's not the job of this PR. This PR uses [1]. This is a controversial change because the automated infiltration script is widespread, and many people like it. If the maintainers want, I will change it to [2].