Scheduled sync: main → develop

[github-actions]

Automated sync needs review:

• Merge conflicted: true
• Direct push exit code: n/a
  Please resolve and merge to bring develop up to date with main.

---

Note

Medium Risk
Adds new runtime guardrails around ImportedSurface usage and entity private_attribute_id requirements, which can cause previously-accepted calls to raise errors if callers relied on implicit behavior. Also changes how ImportedSurface ids are generated, which may affect persistence/lookup semantics across sessions.

Overview
Tightens DraftContext requirements for imported surface dependencies. Project._run() now raises a Flow360ValueError when SimulationParams.private_attribute_asset_cache.imported_surfaces is non-empty but no active DraftContext exists, preventing submissions that would otherwise proceed with inconsistent draft-dependent state.

Hardens entity tracking for draft features. CoordinateSystemManager.assign() and MirrorManager.create_mirror_of() now reject entities with private_attribute_id=None, and ImportedSurface auto-populates a deterministic private_attribute_id derived from name to keep coordinate-system/mirror bookkeeping stable across reconstructions.

Tests updated/added to cover the new guard clause, the new error paths for id-less entities, and deterministic ImportedSurface id behavior.

^{Written by Cursor Bugbot for commit 5b9ea2b. This will update automatically on new commits. Configure here.}