422 when creating pattern with a name that already exists

This seems like a regression from ✨ [PP-1] Integrate saving sections with the backend Postponed , where \Drupal\experience_builder\Entity\Pattern::generateId() was added, which literally contains:

    if ($id_exists) {
      $id = $id . '_' . (new Random())->machineName(8);
    }

So: seems like we're missing test coverage in \Drupal\Tests\experience_builder\Functional\XbConfigEntityHttpApiTest::testPattern()?

Instead of showing a 422, user should be able to save over the existing pattern.

Ah. I had missed this bit.

That feels like a task/feature request — not a bug.

Let's do the simpler thing here and make the current logic work. I had not realized we had already defined how this should work so I proposed the override based on an old design I found from Figma. In the long term it should be possible to edit sections when it becomes less relevant to be able to override them.