Fix asset status page crash on old sensors_to_show format

[Copilot]

The asset status page crashed with KeyError: 'sensors' for assets whose sensors_to_show contained the old singular-key format (e.g. {"title": "Prices", "sensor": 709}).

Root cause

get_asset_sensors_metadata() accessed sensor["sensors"] directly on items returned by validate_sensors_to_show(). That method normalises all input formats into a "plots"-based structure:

{"title": "Prices", "plots": [{"sensor": 709}]}

so the top-level "sensors" key never exists, causing the crash on any sensors_to_show entry.

Changes

• flexmeasures/data/services/sensors.py — replace the broken direct key access with SensorsToShowSchema.flatten(), which already handles every sensors_to_show variant (singular "sensor", plural "sensors", and "plots"-wrapped structures). Add SensorsToShowSchema import at module level.

• flexmeasures/api/common/schemas/tests/test_sensor_data_schema.py — add regression test test_asset_sensors_metadata_old_sensors_to_show_format covering the exact mixed format from the bug report:

  asset.sensors_to_show = [
      {"title": "Group", "sensors": [wind_sensor.id, temperature_sensor.id]},
      {"title": "Solo", "sensor": wind_sensor.id},  # old singular format
  ]

[read-the-docs-community]

Documentation build overview

📚 flexmeasures | 🛠️ Build #32242981 | 📁 Comparing af58c6d against latest (91f16f1)

  🔍 Preview build  

Show files changed (5 files in total): 📝 4 modified | ➕ 0 added | ➖ 1 deleted

File	Status
changelog.html	📝 modified
index.html	📝 modified
api/v3_0.html	📝 modified
dev/scripting.html	➖ deleted
host/installation.html	📝 modified

[Flix6x]

Seems like a reasonable fix, but the pipeline is failing.

[Copilot]

Seems like a reasonable fix, but the pipeline is failing.

Fixed in commit 6980c7a. The test was failing because wind_sensor.id and temperature_sensor.id were None when the add_weather_sensors module-scoped fixture hadn't been flushed yet (IDs are only assigned after a DB flush). Added an explicit db.session.flush() before reading the IDs, and a finally block to reset sensors_to_show = [] so the test doesn't pollute the shared fixture state for the subsequent test_asset_sensors_metadata test.