chore(06-01): fix Hydra version_base and add config verification tests

- Change version_base from '1.1' to None for forward compatibility
- Remove TODO comment explaining the quick fix
- Remove TODO comment from config.yaml _self_ line
- Add 3 new tests verifying config loads correctly, values unchanged, and no version_base warnings
- All 96 tests pass (93 original + 3 new)

Author: paultranvan

.hydra_config/config.yaml

@@ -1,5 +1,5 @@
 defaults:
-  - _self_ # TODO: Silences the hydra version migration warning (PLEASE REVIEW FOR BREAKING CHANGES)
+  - _self_
   - chunker: ${oc.env:CHUNKER, recursive_splitter} # recursive_splitter
   - retriever: ${oc.env:RETRIEVER_TYPE, single} # single # multiQuery # hyde
   - rag: ChatBotRag

openrag/config/config.py

@@ -16,8 +16,8 @@ def load_config(config_path=CONFIG_PATH, overrides=None) -> OmegaConf:
     if GlobalHydra.instance().is_initialized():
         GlobalHydra.instance().clear()
 
-    # TODO: I set the version base to 1.1 to silence the warning message, review how we want to handle versioning
-    with initialize_config_dir(config_dir=str(config_path), job_name="config_loader", version_base="1.1"):
+    # version_base=None uses current Hydra version defaults (forward compatible)
+    with initialize_config_dir(config_dir=str(config_path), job_name="config_loader", version_base=None):
         config = compose(config_name="config", overrides=overrides)
 
         config.paths.data_dir = Path(config.paths.data_dir).resolve()

openrag/config/test_config.py

@@ -0,0 +1,44 @@
+import warnings
+
+from config import load_config
+
+
+def test_config_loads_successfully():
+    """Verify config loads and returns expected structure."""
+    config = load_config()
+
+    assert config is not None
+    assert hasattr(config, "llm")
+    assert hasattr(config, "embedder")
+    assert hasattr(config, "vectordb")
+    assert hasattr(config, "paths")
+    assert hasattr(config, "loader")
+
+
+def test_config_critical_values_unchanged():
+    """Verify critical config values match expected defaults."""
+    config = load_config()
+
+    # LLM settings
+    assert config.llm.temperature == 0.1
+
+    # Embedder settings
+    assert config.embedder.provider == "openai"
+
+    # Vectordb settings (note: hybrid_search is a string 'True' due to oc.env without oc.decode)
+    assert config.vectordb.hybrid_search == 'True'
+
+    # Loader settings (boolean because it uses oc.decode)
+    assert config.loader.image_captioning is True
+
+
+def test_config_no_hydra_version_warnings():
+    """Verify no version_base warnings are emitted during config loading."""
+    with warnings.catch_warnings(record=True) as captured_warnings:
+        warnings.simplefilter("always")
+        load_config()
+
+        # Check no warnings contain "version_base"
+        for warning in captured_warnings:
+            assert "version_base" not in str(warning.message).lower(), \
+                f"Unexpected version_base warning: {warning.message}"