midgleyc
commented
1 year ago One more issue with this: to_location might fail, because KoL's locations and Mafia's locations don't necessarily match.
I think the best thing to do in that case is to just dump the data directly. I guess we should also store the location id after we send the autumnaton somewhere so we can read it back.
Need to test this a little more before it goes in, but up for comments. I tested it using a local version that prints strings instead of sending json to Excavator, so that might be wrong.
If the user has any meltables they didn't receive that combat, we don't submit any data. While going through my own spading, I found that in some cases Mafia's predicated environment was wrong, so we can't trust any data we receive while the user has a meltable because the "true item received" could be the meltable.
If we wind up with a dearth of data we can take in slightly more and require multiple data points, but this seems good enough.