Stop OmegaConfigLoader from reading config from hidden directory like ipynb_checkpoints

RELEASE.md

@@ -21,7 +21,8 @@
 * Updated `kedro pipeline create` and `kedro catalog create` to use new `/conf` file structure.
 * Converted `setup.py` in default template to `pyproject.toml` and moved flake8 configuration
   to dedicated file `.flake8`.
-
+* Fixed a bug that `OmegaConfigLoader` read config from hidden directory like `.ipynb_checkpoints`.
+* 
 ## Documentation changes
 * Revised the `data` section to restructure beginner and advanced pages about the Data Catalog and datasets.
 * Moved contributor documentation to the [GitHub wiki](https://github.com/kedro-org/kedro/wiki/Contribute-to-Kedro).

kedro/config/omegaconf_config.py

@@ -250,11 +250,12 @@ def load_and_merge_dir_config(  # noqa: too-many-arguments
                 f"or is not a valid directory: {conf_path}"
             )
 
-        paths = [
-            Path(each)
-            for pattern in patterns
-            for each in self._fs.glob(Path(f"{str(conf_path)}/{pattern}").as_posix())
-        ]
+        paths = []
+        for pattern in patterns:
+            for each in self._fs.glob(Path(f"{str(conf_path)}/{pattern}").as_posix()):
+                if not self._is_hidden(each):
+                    paths.append(Path(each))
+
         deduplicated_paths = set(paths)
         config_files_filtered = [
             path for path in deduplicated_paths if self._is_valid_config_path(path)
@@ -392,3 +393,11 @@ def _resolve_environment_variables(config: dict[str, Any]) -> None:
             OmegaConf.clear_resolver("oc.env")
         else:
             OmegaConf.resolve(config)
+
+    def _is_hidden(self, path: str):
+        """Check if path contains any hidden directory or is a hidden file"""
+        path = Path(path).resolve().as_posix()
+        parts = path.split(self._fs.sep)  # filesystem specific separator
+        HIDDEN = "."
+        # Check if any component (folder or file) starts with a dot (.)
+        return any(part.startswith(HIDDEN) for part in parts)

kedro/framework/hooks/manager.py

@@ -48,6 +48,8 @@ def _register_hooks(hook_manager: PluginManager, hooks: Iterable[Any]) -> None:
         if not hook_manager.is_registered(hooks_collection):
             hook_manager.register(hooks_collection)
 
+    hook_manager.check_pending() # Validate hook_impl respect hook_spec
+
 
 def _register_hooks_setuptools(
     hook_manager: PluginManager, disabled_plugins: Iterable[str]

tests/config/test_omegaconf_config.py

@@ -797,3 +797,43 @@ def test_bad_globals_underscore(self, tmp_path):
             match=r"Keys starting with '_' are not supported for globals.",
         ):
             conf["parameters"]["param2"]
+
+    @pytest.mark.parametrize(
+        "hidden_path", ["/User/.hidden/dummy.yml", "/User/dummy/.hidden.yml"]
+    )
+    def test_is_hidden_config(self, tmp_path, hidden_path):
+        conf = OmegaConfigLoader(str(tmp_path))
+        assert conf._is_hidden(hidden_path)
+
+    @pytest.mark.parametrize(
+        "hidden_path",
+        [
+            "/User/conf/base/catalog.yml",
+            "/User/conf/local/catalog/data_science.yml",
+            "/User/notebooks/../conf/base/catalog",
+        ],
+    )
+    def test_not_hidden_config(self, tmp_path, hidden_path):
+        conf = OmegaConfigLoader(str(tmp_path))
+        assert not conf._is_hidden(hidden_path)
+
+    def test_ignore_ipynb_checkpoints(self, tmp_path, mocker):
+        conf = OmegaConfigLoader(str(tmp_path), default_run_env=_BASE_ENV)
+        base_path = tmp_path / _BASE_ENV / "parameters.yml"
+        checkpoints_path = (
+            tmp_path / _BASE_ENV / ".ipynb_checkpoints" / "parameters.yml"
+        )
+
+        base_config = {"param1": "dummy"}
+        checkpoints_config = {"param1": "dummy"}
+
+        _write_yaml(base_path, base_config)
+        _write_yaml(checkpoints_path, checkpoints_config)
+
+        # read successfully
+        conf["parameters"]
+
+        mocker.patch.object(conf, "_is_hidden", return_value=False)  #
+        with pytest.raises(ValueError, match="Duplicate keys found in"):
+            # fail because of reading the hidden files and get duplicate keys
+            conf["parameters"]