From 048be11185ebe8f2b1c4a3976e9cfc1d49c7c466 Mon Sep 17 00:00:00 2001
From: Dariusz L <dl122@wp.pl>
Date: Sun, 2 Aug 2026 20:17:33 +0200
Subject: [PATCH] test: enforce workflow module boundaries

---
 tests/test_workflow_boundaries.py | 44 +++++++++++++++++++++++++++++++
 1 file changed, 44 insertions(+)
 create mode 100644 tests/test_workflow_boundaries.py

diff --git a/tests/test_workflow_boundaries.py b/tests/test_workflow_boundaries.py
new file mode 100644
index 0000000..548d3b3
--- /dev/null
+++ b/tests/test_workflow_boundaries.py
@@ -0,0 +1,44 @@
+import ast
+from pathlib import Path
+
+WORKFLOW_COMPONENTS = {
+    "widgets": set(),
+    "dynamic_widgets": {"widgets"},
+    "references": {"dynamic_widgets", "widgets"},
+    "subgraphs": {"references", "widgets"},
+    "analysis": {"references", "subgraphs"},
+    "inventory": {"analysis"},
+}
+
+
+def _sibling_imports(module_path):
+    tree = ast.parse(module_path.read_text(encoding="utf-8"))
+    imports = set()
+
+    for node in ast.walk(tree):
+        if not isinstance(node, ast.ImportFrom) or node.level != 1:
+            continue
+        if node.module:
+            imports.add(node.module.split(".", 1)[0])
+        else:
+            imports.update(alias.name.split(".", 1)[0] for alias in node.names)
+
+    return imports
+
+
+def test_workflow_components_follow_the_declared_dependency_layers():
+    workflow_directory = Path(__file__).parents[1] / "core" / "workflow"
+
+    for component_name, allowed_imports in WORKFLOW_COMPONENTS.items():
+        imports = _sibling_imports(workflow_directory / f"{component_name}.py")
+
+        assert imports <= allowed_imports, (
+            f"{component_name} imports unexpected workflow components: "
+            f"{sorted(imports - allowed_imports)}"
+        )
+
+
+def test_workflow_package_does_not_restore_the_removed_flat_module():
+    legacy_module = Path(__file__).parents[1] / "core" / "workflow_analyzer.py"
+
+    assert not legacy_module.exists()