feat!: Redesign configuration layerfor full typing and OOP friendliness

Split Config into schema/loader and resolver components to enable fully typed, OOP-friendly configuration without existing temporary workarounds. Introduces explicit runtime loading for rules and formatters instead of lazy attributes in Config class.

Author: bhirsz

docs/generate_rules_list.py

@@ -11,6 +11,7 @@
 import robocop
 from docs.rules_metadata import GROUPS_LOOKUP
 from robocop.linter.rules import SeverityThreshold
+from robocop.runtime.resolver import DocumentationImporter
 
 RULES_LIST_TEMPLATE = """
 # Rules list
@@ -120,7 +121,7 @@
 
 def get_checker_docs() -> tuple[list[tuple], int]:
     """Load rules for dynamic docs generation"""
-    doc_importer = robocop.linter.rules.DocumentationImporter()
+    doc_importer = DocumentationImporter()
     rules_count = 0
     for _, rule in doc_importer.get_builtin_rules():
         rules_count += 1

docs/linter/custom_rules.md

@@ -414,7 +414,7 @@ be used to run any code, for example, analysis of the project dependencies and a
 Example project checker:
 
 ```python title="project_checker.py"
-from robocop.config_manager import ConfigManager
+from robocop.config.manager import ConfigManager
 from robocop.linter.rules import Rule, ProjectChecker, RuleSeverity
 
 

noxfile.py

@@ -56,8 +56,7 @@ def install_doc_deps(session: nox.Session, robot_version: str) -> None:
 @nox.session()
 def docs(session: nox.Session) -> None:
     install_doc_deps(session, "7.2.2")
-    # session.run("sphinx-build", "-a", "-E", "-b", "html", "docs", "docs/_build/")
-    command = ["sphinx-build", "-a", "-E", "--verbose", "-b", "html", "docs/source", "docs/_build/"]
+    command = ["mkdocs", "build", "--clean", "--verbose", "--config-file", "mkdocs.yml"]
     session.run(*command)
 
 
@@ -95,4 +94,4 @@ def performance(session: nox.Session, robocop_version: str) -> None:
             f"--python={session.virtualenv.location}",
             env={"UV_PROJECT_ENVIRONMENT": session.virtualenv.location},
         )
-    session.run("python", "-m", "tests.performance.generate_reports", external=True, silent=False)
+    session.run("python", "-m", "tests.performance.generate_reports", robocop_version, external=True, silent=False)

src/robocop/cache.py

@@ -8,22 +8,20 @@
 from __future__ import annotations
 
 from dataclasses import dataclass, field
-from pathlib import Path
 from typing import TYPE_CHECKING, Any
 
 import msgpack
 
 from robocop import __version__
+from robocop.config import defaults
 from robocop.source_file import SourceFile
 
 if TYPE_CHECKING:
+    from pathlib import Path
+
     from robocop.config import Config
     from robocop.linter.diagnostics import Diagnostic
-
-
-CACHE_VERSION = "1.0"
-CACHE_DIR_NAME = ".robocop_cache"
-CACHE_FILE_NAME = "cache.msgpack"
+    from robocop.runtime.resolved_config import ResolvedConfig
 
 
 @dataclass(frozen=True)
@@ -201,7 +199,6 @@ def from_dict(cls, data: dict[str, Any]) -> FormatterCacheEntry:
 class CacheData:
     """Mutable container for cache data."""
 
-    version: str = CACHE_VERSION
     robocop_version: str = field(default_factory=lambda: __version__)
     linter: dict[str, LinterCacheEntry] = field(default_factory=dict)
     formatter: dict[str, FormatterCacheEntry] = field(default_factory=dict)
@@ -215,7 +212,6 @@ def to_dict(self) -> dict[str, Any]:
 
         """
         return {
-            "version": self.version,
             "robocop_version": self.robocop_version,
             "linter": {path: entry.to_dict() for path, entry in self.linter.items()},
             "formatter": {path: entry.to_dict() for path, entry in self.formatter.items()},
@@ -231,7 +227,6 @@ def from_dict(cls, data: dict[str, Any]) -> CacheData:
 
         """
         return cls(
-            version=data.get("version", CACHE_VERSION),
             robocop_version=data.get("robocop_version", ""),
             linter={path: LinterCacheEntry.from_dict(entry) for path, entry in data.get("linter", {}).items()},
             formatter={path: FormatterCacheEntry.from_dict(entry) for path, entry in data.get("formatter", {}).items()},
@@ -248,9 +243,9 @@ class RobocopCache:
 
     def __init__(
         self,
-        cache_dir: Path | None = None,
-        enabled: bool = True,
-        verbose: bool = False,
+        cache_dir: Path,
+        enabled: bool,
+        verbose: bool,
     ) -> None:
         """
         Initialize the cache.
@@ -262,7 +257,7 @@ def __init__(
 
         """
         self.enabled = enabled
-        self.cache_dir = cache_dir or Path.cwd() / CACHE_DIR_NAME
+        self.cache_dir = cache_dir
         self.verbose = verbose
         self._data: CacheData | None = None
         self._dirty = False
@@ -282,7 +277,7 @@ def _load(self) -> None:
             self._data = CacheData()
             return
 
-        cache_file = self.cache_dir / CACHE_FILE_NAME
+        cache_file = self.cache_dir / defaults.CACHE_FILE_NAME
 
         if not cache_file.is_file():
             self._data = CacheData()
@@ -315,7 +310,7 @@ def save(self) -> None:
 
         self.cache_dir.mkdir(parents=True, exist_ok=True)
         self._create_gitignore()
-        cache_file = self.cache_dir / CACHE_FILE_NAME
+        cache_file = self.cache_dir / defaults.CACHE_FILE_NAME
 
         try:
             cache_file.write_bytes(msgpack.packb(self._data.to_dict(), use_bin_type=True))
@@ -517,6 +512,7 @@ def restore_diagnostics(
     cached_entry: LinterCacheEntry,
     source: Path,
     config: Config,
+    resolved_config: ResolvedConfig,
 ) -> list[Diagnostic] | None:
     """
     Restore Diagnostic objects from cached data.
@@ -525,6 +521,7 @@ def restore_diagnostics(
         cached_entry: The cached linter entry.
         source: The source file path (Path object for consistency with normal diagnostics).
         config: Configuration associated with the source file.
+        resolved_config: ResolvedConfig with loaded runtime objects such as rules.
 
     Returns:
         List of restored diagnostics, or None if restoration failed
@@ -536,9 +533,9 @@ def restore_diagnostics(
     restored = []
     for cached_diag in cached_entry.diagnostics:
         # Try to find rule by ID first, fall back to name
-        rule = config.linter.rules.get(cached_diag.rule_id)
+        rule = resolved_config.rules.get(cached_diag.rule_id)
         if rule is None:
-            rule = config.linter.rules.get(cached_diag.rule_name)
+            rule = resolved_config.rules.get(cached_diag.rule_name)
 
         if rule is None:
             # Rule no longer exists - invalidate cache entry