Please address the comments from this code review:
## Individual Comments

### Comment 1
<location> `cms/wizards/wizard_base.py:30` </location>
<code_context>
+            # do something with a wizard...
+    """
+    wizards = apps.get_app_config('cms').cms_extension.wizards
+    return sorted(wizards.values(), key=lambda e: e.weight)
+
+
</code_context>

<issue_to_address>
Sorting now returns only values, not (id, wizard) tuples.

The return type no longer matches the docstring and example, which may break code relying on the previous (id, wizard) tuple format.
</issue_to_address>

### Comment 2
<location> `cms/cms_config.py:51` </location>
<code_context>
+        """
+
+        wizards = {}
+        for iterable in self.lazy_wizards:
+            new_wizards = {wizard.id: wizard for wizard in iterable}
+            if wizard := next((wizard for wizard in new_wizards.values() if not isinstance(wizard, Wizard)), None):
+                # If any wizard in the iterable is not an instance of Wizard, raise an exception
                 raise ImproperlyConfigured(
-                    "All wizards defined in cms_wizards must inherit "
-                    "from cms.wizards.wizard_base.Wizard"
+                    f"cms_wizards must be iterable of Wizard instances, got {type(wizard)}"
                 )
-            elif wizard.id in self.wizards:
-                msg = f"Wizard for model {wizard.get_model()} has already been registered"
-                logger.warning(msg)
-            else:
-                self.wizards[wizard.id] = wizard
+            wizards |= new_wizards
+        return wizards

</code_context>

<issue_to_address>
Merging dictionaries with |= may overwrite Wizards with duplicate IDs.

This can lead to unintentional data loss. Please add a check for duplicate IDs and raise an error or warning if found.

Suggested implementation:

```python
        wizards = {}

```

```python
        for iterable in self.lazy_wizards:
            new_wizards = {wizard.id: wizard for wizard in iterable}
            if wizard := next((wizard for wizard in new_wizards.values() if not isinstance(wizard, Wizard)), None):
                # If any wizard in the iterable is not an instance of Wizard, raise an exception
                raise ImproperlyConfigured(
                    f"cms_wizards must be iterable of Wizard instances, got {type(wizard)}"
                )
            duplicate_ids = set(new_wizards.keys()) & set(wizards.keys())
            if duplicate_ids:
                raise ImproperlyConfigured(
                    f"Duplicate wizard IDs found: {', '.join(str(i) for i in duplicate_ids)}"
                )
            wizards |= new_wizards
        return wizards

```
</issue_to_address>

### Comment 3
<location> `cms/tests/test_cms_config_wizards.py:52` </location>
<code_context>
         wizard = Mock(id=3, spec=object)
         cms_config = Mock(
             cms_enabled=True, cms_wizards=[wizard])
+        extensions.configure_wizards(cms_config)
         with self.assertRaises(ImproperlyConfigured):
-            extensions.configure_wizards(cms_config)
+            extensions.wizards
+

</code_context>

<issue_to_address>
Test for exception when non-Wizard instance is present now triggers on property access.

Consider adding a comment or separate test to document that the exception is now raised on 'wizards' property access, not during configuration, to clarify intent and timing.
</issue_to_address>

<suggested_fix>
<<<<<<< SEARCH
        wizard = Mock(id=3, spec=object)
        cms_config = Mock(
            cms_enabled=True, cms_wizards=[wizard])
        extensions.configure_wizards(cms_config)
        with self.assertRaises(ImproperlyConfigured):
            extensions.wizards

=======
        # Note: Exception is now raised on 'wizards' property access, not during configuration.
        wizard = Mock(id=3, spec=object)
        cms_config = Mock(
            cms_enabled=True, cms_wizards=[wizard])
        extensions.configure_wizards(cms_config)
        with self.assertRaises(ImproperlyConfigured):
            extensions.wizards

    def test_raises_exception_on_wizards_property_access_with_non_wizard(self):
        """
        Test that ImproperlyConfigured is raised when a non-Wizard instance is present,
        and that the exception is triggered on 'wizards' property access, not during configuration.
        """
        wizard = Mock(id=3, spec=object)
        cms_config = Mock(
            cms_enabled=True, cms_wizards=[wizard])
        extensions.configure_wizards(cms_config)
        with self.assertRaises(ImproperlyConfigured):
            _ = extensions.wizards
>>>>>>> REPLACE

</suggested_fix>