[Sky Island] Warped reinforcement is stopped by... something?

[Consoleable]

Describe the bug

When crafting the "warped reinforcement" upgrade for the main room of the sky island bunker, there is no room expansion

Attach save file

Bonsai-trimmed.tar.gz

Steps to reproduce

Craft "warped reinforcement"
No expansion of main room
Suffering

Expected behavior

Warped reinforcement should cause the nexus room of the bunker in SI mod to expand. It doesnt seem to want to.
Note that in the attached save file, there is another bug related to playernum and mission cleanup, HOWEVER this bug doesnt seem to prevent the other rooms from being expanded, so it seems to be an issue with warped reinforcement

Screenshots

No response

Versions and configuration

• OS: Windows
  • OS Version: 10.0.22631.3447 (23H2)
• Game Version: 2f373e7 [64-bit]
• Graphics Version: Tiles
• Game Language: System language []
• Mods loaded: [
  Dark Days Ahead [dda],
  Disable NPC Needs [no_npc_food],
  Portal Storms Ignore NPCs [personal_portal_storms],
  Slowdown Fungal Growth [no_fungal_growth],
  Translate Complex Dialogue [translate_dialogue],
  sees-player, run, crouch, bleed icons, retrodays [sees_player_retro],
  Mind Over Matter [mindovermatter],
  Sky Island [skyisland],
  Stats Through Kills [stats_through_kills],
  SpeedyDex [speedydex],
  Tamable Wildlife [Tamable_Wildlife]
  ]

Additional context

No response

[RenechCDDA]

As far as I can tell the mission's end is being called, EOC effects are activating, mission effects are firing.... it's unclear to me why the mapgen update is not running.

[RenechCDDA]

(oops, wrong wrong wrong)

So here's mission_type::parse_funcs() which is called when the mission ends and is supposed to run update_mapgen.

update_mapgen is a mission function - it should be in that vector called funcs. The vector is empty. Why? It's called by an eoc - the eoc is of course a talk function :)

[RenechCDDA]

After a great deal of stepping through functions, I've found out that... the appliances are getting in the way?!

Here, update_map returns false due to vehicle collision.

Cataclysm-DDA/src/mapgen.cpp

Line 7843 in ef430d2

bool const u = update_map( md, offset, verify );

Destroying all vehicles on the map is sufficient to allow it to work. I thought it was the wall light at first (it's in the path of replaced terrain) but destroying just the wall light is not sufficient. This is very strange (and also it should probably throw a warning on fail!)

[Procyonae]

Here, update_map returns false due to vehicle collision.

Ye I noticed that was a thing here #72947 but I weren't entirely sure how to solve it properly. It probably shouldn't throw an error bc map extras use mapgen updates and them failing to be placed is supposed to be silent but potentially we should have some way to call a forced mapgen update in json that doesn't verify and instead throws an error for stuff like this where you do expect it to place 100% of the time.

[physics-enthusiast]

Destroying all vehicles on the map is sufficient to allow it to work. I thought it was the wall light at first (it's in the path of replaced terrain) but destroying just the wall light is not sufficient. This is very strange (and also it should probably throw a warning on fail!)

Wait, is it mentioned ingame anywhere that this will destroy appliances? And is there a way to avoid doing so?

[Procyonae]

It doesn't destroy appliances it fails because there are appliances, Renech's saying if you do destroy all appliances it will work