fix(repo): filter empty optional fields from installed manifest

Empty optional fields (exports: {}, imports: {}, etc.) in the repo
element manifest caused constant updates in the universal agent due to
a diff between target and actual resources on every iteration.

- Add MANIFEST_OPTIONAL_FIELDS to InstalledManifest and filter empty
  values in from_repo_element
- Simplify _make_installed_manifest to use a loop over the shared
  MANIFEST_OPTIONAL_FIELDS tuple instead of individual if-blocks
- Add regression tests for empty and non-empty optional field handling

Signed-off-by: Anton Kremenetsky <anton.kremenetsky@gmail.com>

Author: akremenetsky

exordos_core/repo/agents/universal/drivers/repo_element.py

@@ -85,20 +85,10 @@ def _make_installed_manifest(
             project_id=em_manifest.project_id,
         )
 
-        if em_manifest.openapi_spec is not None:
-            installed_manifest.manifest["openapi_spec"] = em_manifest.openapi_spec
-
-        if em_manifest.exports:
-            installed_manifest.manifest["exports"] = em_manifest.exports
-
-        if em_manifest.imports:
-            installed_manifest.manifest["imports"] = em_manifest.imports
-
-        if em_manifest.requirements:
-            installed_manifest.manifest["requirements"] = em_manifest.requirements
-
-        if em_manifest.resources:
-            installed_manifest.manifest["resources"] = em_manifest.resources
+        for field in re_builder.InstalledManifest.MANIFEST_OPTIONAL_FIELDS:
+            value = getattr(em_manifest, field)
+            if value:
+                installed_manifest.manifest[field] = value
 
         return installed_manifest
 

exordos_core/repo/builders/element.py

@@ -65,6 +65,14 @@ class InstalledManifest(
             linked.
     """
 
+    MANIFEST_OPTIONAL_FIELDS = (
+        "openapi_spec",
+        "exports",
+        "imports",
+        "requirements",
+        "resources",
+    )
+
     version = properties.property(
         ra_types.String(min_length=5, max_length=64), required=True
     )
@@ -101,12 +109,21 @@ def get_resource_target_fields(self) -> tp.Collection[str]:
     @classmethod
     def from_repo_element(cls, element: models.RepoElement) -> "InstalledManifest":
         uuid = sys_uuid.UUID(str(element.manifest.get("uuid") or element.uuid))
+        # Filter out empty optional fields to keep the manifest consistent
+        # with _make_installed_manifest in the repo element driver, which
+        # also skips empty values. Without this, the agent detects a diff
+        # between target and actual resources on every iteration.
+        manifest = {
+            k: v
+            for k, v in element.manifest.items()
+            if k not in cls.MANIFEST_OPTIONAL_FIELDS or v
+        }
         return cls(
             uuid=uuid,
             name=element.name,
             description=element.manifest.get("description", ""),
             version=element.version,
-            manifest=element.manifest,
+            manifest=manifest,
             project_id=element.project_id,
         )
 

exordos_core/tests/unit/repo/test_element_builder.py

@@ -502,6 +502,50 @@ def test_from_repo_element_without_manifest_uuid(self):
         assert result.name == element.name
         assert result.version == "2.0.0"
 
+    def test_from_repo_element_filters_empty_optional_fields(self):
+        """Empty optional fields should be excluded from the manifest."""
+        element_uuid = sys_uuid.uuid4()
+        element = FakeElement(version="1.0.0")
+        element.uuid = element_uuid
+        element.manifest = {
+            "uuid": element_uuid,
+            "key": "value",
+            "exports": {},
+            "imports": {},
+            "requirements": {},
+            "resources": {},
+            "openapi_spec": None,
+        }
+        element.project_id = sys_uuid.uuid4()
+
+        result = builder_element.InstalledManifest.from_repo_element(element)
+
+        assert "exports" not in result.manifest
+        assert "imports" not in result.manifest
+        assert "requirements" not in result.manifest
+        assert "resources" not in result.manifest
+        assert "openapi_spec" not in result.manifest
+        assert result.manifest["key"] == "value"
+
+    def test_from_repo_element_keeps_non_empty_optional_fields(self):
+        """Non-empty optional fields should be preserved in the manifest."""
+        element_uuid = sys_uuid.uuid4()
+        element = FakeElement(version="1.0.0")
+        element.uuid = element_uuid
+        element.manifest = {
+            "uuid": element_uuid,
+            "exports": {"exp1": {}},
+            "resources": {"res1": {}},
+            "openapi_spec": {"paths": {}},
+        }
+        element.project_id = sys_uuid.uuid4()
+
+        result = builder_element.InstalledManifest.from_repo_element(element)
+
+        assert result.manifest["exports"] == {"exp1": {}}
+        assert result.manifest["resources"] == {"res1": {}}
+        assert result.manifest["openapi_spec"] == {"paths": {}}
+
     def test_get_resource_target_fields(self):
         manifest = builder_element.InstalledManifest(
             name="test",

exordos_core/tests/unit/repo/test_repo_element_driver.py

@@ -146,6 +146,25 @@ def test_without_repo_element_without_em_element(self):
         }
         assert result.manifest == expected_manifest
 
+    def test_with_non_empty_optional_fields(self):
+        """Should include non-empty optional fields in the manifest."""
+        client = self._make_client()
+        em_manifest = self._make_em_manifest()
+        em_manifest.exports = {"exp1": {}}
+        em_manifest.imports = {"imp1": {}}
+        em_manifest.requirements = {"dep": {"from_version": "1.0.0"}}
+        em_manifest.resources = {"res1": {}}
+        em_manifest.openapi_spec = {"paths": {}}
+        em_element = self._make_em_element()
+
+        result = client._make_installed_manifest(em_manifest, em_element)
+
+        assert result.manifest["exports"] == {"exp1": {}}
+        assert result.manifest["imports"] == {"imp1": {}}
+        assert result.manifest["requirements"] == {"dep": {"from_version": "1.0.0"}}
+        assert result.manifest["resources"] == {"res1": {}}
+        assert result.manifest["openapi_spec"] == {"paths": {}}
+
 
 class TestMakeEmManifest:
     """Tests for RepoEmBackendClient._make_em_manifest."""