Fix copying of Config and CommandOptions

When copying these dataclasses we also need to copy the lists that they
hold, otherwise if there are in-place updates the copied instances will
be affected too.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

RELEASE_NOTES.md

@@ -52,7 +52,11 @@
 
 - The distribution package doesn't include tests and other useless files anymore.
 
-- nox: When discovering path *extra paths*, now paths will not be added if they are also *source paths*, as we don't want any duplicates.
+- nox
+
+  * When discovering path *extra paths*, now paths will not be added if they are also *source paths*, as we don't want any duplicates.
+
+  * Fix copying of `Config` and `CommandOptions` objects.
 
 ### Cookiecutter template
 

src/frequenz/repo/config/nox/config.py

@@ -53,7 +53,16 @@ def copy(self) -> Self:
         Returns:
             The copy of self.
         """
-        return _dataclasses.replace(self)
+        return _dataclasses.replace(
+            self,
+            black=self.black.copy(),
+            darglint=self.darglint.copy(),
+            isort=self.isort.copy(),
+            mypy=self.mypy.copy(),
+            pydocstyle=self.pydocstyle.copy(),
+            pylint=self.pylint.copy(),
+            pytest=self.pytest.copy(),
+        )
 
 
 @_dataclasses.dataclass(kw_only=True, slots=True)
@@ -101,7 +110,13 @@ def copy(self, /) -> Self:
         Returns:
             The copy of self.
         """
-        return _dataclasses.replace(self)
+        return _dataclasses.replace(
+            self,
+            opts=self.opts.copy(),
+            sessions=self.sessions.copy(),
+            source_paths=self.source_paths.copy(),
+            extra_paths=self.extra_paths.copy(),
+        )
 
     def path_args(self, session: _nox.Session, /) -> list[str]:
         """Return the file paths to run the checks on.

src/frequenz/repo/config/nox/default.py

@@ -24,8 +24,6 @@
 method.
 """
 
-import dataclasses
-
 from . import config as _config
 from . import util as _util
 
@@ -87,20 +85,20 @@
 api_command_options: _config.CommandsOptions = common_command_options.copy()
 """Default command-line options for APIs."""
 
-api_config: _config.Config = dataclasses.replace(
-    common_config,
-    opts=api_command_options,
-    # We don't check the sources at all because they are automatically generated.
-    source_paths=[],
-    # We adapt the path to the tests.
-    extra_paths=list(_util.replace(common_config.extra_paths, {"tests": "pytests"})),
-)
+api_config: _config.Config = common_config.copy()
 """Default configuration for APIs.
 
 Same as `common_config`, but with `source_paths` replacing `"src"` with `"py"`
 and `extra_paths` replacing `"tests"` with `"pytests"`.
 """
 
+# We don't check the sources at all because they are automatically generated.
+api_config.source_paths = []
+# We adapt the path to the tests.
+api_config.extra_paths = list(
+    _util.replace(common_config.extra_paths, {"tests": "pytests"})
+)
+
 app_command_options: _config.CommandsOptions = common_command_options.copy()
 """Default command-line options for applications."""