cognite-neat 1.0.33__py3-none-any.whl → 1.0.35__py3-none-any.whl

cognite/neat/_data_model/_analysis.py

@@ -2,17 +2,17 @@ import math
 from collections import defaultdict
 from dataclasses import dataclass
 from enum import Enum
+from functools import cached_property
 from itertools import chain, combinations
 from typing import Literal, TypeAlias, TypeVar
 
 import networkx as nx
-from pyparsing import cached_property
 
 from cognite.neat._client.data_classes import SpaceStatisticsResponse
 from cognite.neat._data_model._constants import COGNITE_SPACES
 from cognite.neat._data_model._snapshot import SchemaSnapshot
 from cognite.neat._data_model.models.dms._constraints import RequiresConstraintDefinition
-from cognite.neat._data_model.models.dms._container import ContainerRequest
+from cognite.neat._data_model.models.dms._container import ContainerPropertyDefinition, ContainerRequest
 from cognite.neat._data_model.models.dms._data_types import DirectNodeRelation
 from cognite.neat._data_model.models.dms._limits import SchemaLimits
 from cognite.neat._data_model.models.dms._references import (
@@ -55,6 +55,21 @@ class RequiresChangeStatus(Enum):
     NO_MODIFIABLE_CONTAINERS = "no_modifiable_containers"  # All containers are immutable
 
 
+@dataclass
+class ResolvedReverseDirectRelation:
+    """Resolved context for a reverse direct relation, including container-level information."""
+
+    reverse_view_ref: ViewReference
+    reverse_property_id: str
+    direct_view_ref: ViewReference
+    through_property_id: str
+    direct_property: ViewCorePropertyRequest
+    container_ref: ContainerReference
+    container_property_id: str
+    container: ContainerRequest | None
+    container_property: ContainerPropertyDefinition | None
+
+
 @dataclass
 class RequiresChangesForView:
     """Result of computing requires constraint changes for a view."""
@@ -365,6 +380,83 @@ class ValidationResources:
 
         return bidirectional_connections
 
+    @staticmethod
+    def normalize_through_reference(
+        source_view_ref: ViewReference, through: ContainerDirectReference | ViewDirectReference
+    ) -> ViewDirectReference:
+        """Normalize through reference to ViewDirectReference for consistent processing.
+
+        When a reverse direct relation uses a ContainerDirectReference, we convert it to
+        a ViewDirectReference using the source view. This enables consistent handling
+        in validators regardless of how the 'through' was originally specified.
+        """
+        if isinstance(through, ContainerDirectReference):
+            return ViewDirectReference(source=source_view_ref, identifier=through.identifier)
+        return through
+
+    @cached_property
+    def resolved_reverse_direct_relations(self) -> list[ResolvedReverseDirectRelation]:
+        """Get all reverse direct relations with their resolved context.
+
+        This property traverses from reverse direct relation → source view → container,
+        resolving all references along the way.
+
+        Includes reverse direct relations where view-level resolution succeeded:
+        - The source view exists and can be expanded
+        - The through property exists in the source view
+        - The through property is a ViewCorePropertyRequest (direct relation)
+
+        Container-level fields may be None if resolution failed at that level.
+        """
+        result: list[ResolvedReverseDirectRelation] = []
+
+        for (target_view_ref, reverse_prop_id), (
+            source_view_ref,
+            through,
+        ) in self.reverse_to_direct_mapping.items():
+            through_normalized = self.normalize_through_reference(source_view_ref, through)
+
+            # Get expanded source view to include inherited properties
+            source_view_expanded = self.expand_view_properties(source_view_ref)
+            if not source_view_expanded or not source_view_expanded.properties:
+                continue
+
+            if through_normalized.identifier not in source_view_expanded.properties:
+                continue
+
+            source_property = source_view_expanded.properties[through_normalized.identifier]
+
+            # Must be a core property (direct relation)
+            if not isinstance(source_property, ViewCorePropertyRequest):
+                continue
+
+            container_ref = source_property.container
+            container_property_id = source_property.container_property_identifier
+
+            # Resolve container - may be None if missing
+            container = self.select_container(container_ref, container_property_id)
+
+            # Resolve container property - may be None if container missing or property missing
+            container_property = None
+            if container and container.properties:
+                container_property = container.properties.get(container_property_id)
+
+            result.append(
+                ResolvedReverseDirectRelation(
+                    reverse_view_ref=target_view_ref,
+                    reverse_property_id=reverse_prop_id,
+                    direct_view_ref=source_view_ref,
+                    through_property_id=through_normalized.identifier,
+                    direct_property=source_property,
+                    container_ref=container_ref,
+                    container_property_id=container_property_id,
+                    container=container,
+                    container_property=container_property,
+                )
+            )
+
+        return result
+
     @property
     def connection_end_node_types(self) -> dict[tuple[ViewReference, str], ViewReference | None]:
         """Get a mapping of view references to their corresponding ViewRequest objects."""

cognite/neat/_data_model/models/conceptual/_concept.py

@@ -1,7 +1,7 @@
 from collections import Counter
+from typing import cast
 
 from pydantic import Field, ValidationInfo, field_validator
-from pyparsing import cast
 
 from cognite.neat._data_model.models.entities import ConceptEntity
 from cognite.neat._data_model.models.entities._constants import PREFIX_PATTERN, SUFFIX_PATTERN, VERSION_PATTERN

cognite/neat/_data_model/rules/dms/__init__.py

@@ -39,6 +39,7 @@ from ._limits import (
 from ._orchestrator import DmsDataModelRulesOrchestrator
 from ._performance import (
     MissingRequiresConstraint,
+    MissingReverseDirectRelationTargetIndex,
     SuboptimalRequiresConstraint,
     UnresolvableQueryPerformance,
 )
@@ -60,6 +61,7 @@ __all__ = [
     "ExternalContainerPropertyDoesNotExist",
     "ImplementedViewNotExisting",
     "MissingRequiresConstraint",
+    "MissingReverseDirectRelationTargetIndex",
     "RequiredContainerDoesNotExist",
     "RequiresConstraintCycle",
     "ReverseConnectionContainerMissing",

cognite/neat/_data_model/rules/dms/_connections.py

@@ -4,7 +4,6 @@ from dataclasses import dataclass
 
 from cognite.neat._data_model.models.dms._data_types import DirectNodeRelation
 from cognite.neat._data_model.models.dms._references import (
-    ContainerDirectReference,
     ViewDirectReference,
     ViewReference,
 )
@@ -119,15 +118,6 @@ class ReverseConnectionContext:
     source_view_ref: ViewReference
 
 
-def _normalize_through_reference(
-    source_view_ref: ViewReference, through: ContainerDirectReference | ViewDirectReference
-) -> ViewDirectReference:
-    """Normalize through reference to ViewDirectReference for consistent processing."""
-    if isinstance(through, ContainerDirectReference):
-        return ViewDirectReference(source=source_view_ref, identifier=through.identifier)
-    return through
-
-
 class ReverseConnectionSourceViewMissing(DataModelRule):
     """Validates that source view referenced in reverse connection exist.
 
@@ -153,7 +143,7 @@ class ReverseConnectionSourceViewMissing(DataModelRule):
             source_view_ref,
             through,
         ) in self.validation_resources.reverse_to_direct_mapping.items():
-            through = _normalize_through_reference(source_view_ref, through)
+            through = self.validation_resources.normalize_through_reference(source_view_ref, through)
             source_view = self.validation_resources.select_view(source_view_ref, through.identifier)
 
             if not source_view:
@@ -198,7 +188,7 @@ class ReverseConnectionSourcePropertyMissing(DataModelRule):
             source_view_ref,
             through,
         ) in self.validation_resources.reverse_to_direct_mapping.items():
-            through = _normalize_through_reference(source_view_ref, through)
+            through = self.validation_resources.normalize_through_reference(source_view_ref, through)
             source_view = self.validation_resources.select_view(source_view_ref, through.identifier)
 
             if not source_view:
@@ -250,7 +240,7 @@ class ReverseConnectionSourcePropertyWrongType(DataModelRule):
             source_view_ref,
             through,
         ) in self.validation_resources.reverse_to_direct_mapping.items():
-            through = _normalize_through_reference(source_view_ref, through)
+            through = self.validation_resources.normalize_through_reference(source_view_ref, through)
             source_view = self.validation_resources.select_view(source_view_ref, through.identifier)
 
             if not source_view:
@@ -301,40 +291,16 @@ class ReverseConnectionContainerMissing(DataModelRule):
     def validate(self) -> list[ConsistencyError]:
         errors: list[ConsistencyError] = []
 
-        for (target_view_ref, reverse_prop_name), (
-            source_view_ref,
-            through,
-        ) in self.validation_resources.reverse_to_direct_mapping.items():
-            through = _normalize_through_reference(source_view_ref, through)
-            source_view = self.validation_resources.select_view(source_view_ref, through.identifier)
-
-            if not source_view:
-                continue  # Handled by ReverseConnectionSourceViewMissing
-
-            if not (source_view_expanded := self.validation_resources.expand_view_properties(source_view_ref)):
-                raise RuntimeError(f"{type(self).__name__}: View {source_view_ref!s} not found. This is a bug in NEAT.")
-
-            if not source_view_expanded.properties or through.identifier not in source_view_expanded.properties:
-                continue  # Handled by ReverseConnectionSourcePropertyMissing
-
-            source_property = source_view_expanded.properties[through.identifier]
-
-            if not isinstance(source_property, ViewCorePropertyRequest):
-                continue  # Handled by ReverseConnectionSourcePropertyWrongType
-
-            container_ref = source_property.container
-            container_property_id = source_property.container_property_identifier
-
-            source_container = self.validation_resources.select_container(container_ref, container_property_id)
-            if not source_container:
+        for resolved in self.validation_resources.resolved_reverse_direct_relations:
+            if resolved.container is None:
                 errors.append(
                     ConsistencyError(
                         message=(
-                            f"Container {container_ref!s} is missing in both the data model and CDF. "
-                            f"This container is required by view {source_view_ref!s}"
-                            f" property '{through.identifier}', "
-                            f"which configures the reverse connection '{reverse_prop_name}'"
-                            f" in target view {target_view_ref!s}."
+                            f"Container {resolved.container_ref!s} is missing in both the data model and CDF. "
+                            f"This container is required by view {resolved.direct_view_ref!s}"
+                            f" property '{resolved.through_property_id}', "
+                            f"which configures the reverse connection '{resolved.reverse_property_id}'"
+                            f" in target view {resolved.reverse_view_ref!s}."
                         ),
                         fix="Define the missing container",
                         code=self.code,
@@ -366,43 +332,18 @@ class ReverseConnectionContainerPropertyMissing(DataModelRule):
     def validate(self) -> list[ConsistencyError]:
         errors: list[ConsistencyError] = []
 
-        for (target_view_ref, reverse_prop_name), (
-            source_view_ref,
-            through,
-        ) in self.validation_resources.reverse_to_direct_mapping.items():
-            through = _normalize_through_reference(source_view_ref, through)
-            source_view = self.validation_resources.select_view(source_view_ref, through.identifier)
-
-            if not source_view:
-                continue  # Handled by ReverseConnectionSourceViewMissing
-
-            if not (source_view_expanded := self.validation_resources.expand_view_properties(source_view_ref)):
-                raise RuntimeError(f"{type(self).__name__}: View {source_view_ref!s} not found. This is a bug in NEAT.")
-
-            if not source_view_expanded.properties or through.identifier not in source_view_expanded.properties:
-                continue  # Handled by ReverseConnectionSourcePropertyMissing
-
-            source_property = source_view_expanded.properties[through.identifier]
-
-            if not isinstance(source_property, ViewCorePropertyRequest):
-                continue  # Handled by ReverseConnectionSourcePropertyWrongType
-
-            container_ref = source_property.container
-            container_property_id = source_property.container_property_identifier
-
-            source_container = self.validation_resources.select_container(container_ref, container_property_id)
-            if not source_container:
-                continue  # Handled by ReverseConnectionContainerMissing
-
-            if not source_container.properties or container_property_id not in source_container.properties:
+        for resolved in self.validation_resources.resolved_reverse_direct_relations:
+            # Container must exist but property is missing
+            if resolved.container is not None and resolved.container_property is None:
                 errors.append(
                     ConsistencyError(
                         message=(
-                            f"Container {container_ref!s} is missing property '{container_property_id}'. "
-                            f"This property is required by the source view {source_view_ref!s}"
-                            f" property '{through.identifier}', "
-                            f"which configures the reverse connection '{reverse_prop_name}' "
-                            f"in target view {target_view_ref!s}."
+                            f"Container {resolved.container_ref!s} is missing "
+                            f"property '{resolved.container_property_id}'. "
+                            f"This property is required by the source view {resolved.direct_view_ref!s}"
+                            f" property '{resolved.through_property_id}', "
+                            f"which configures the reverse connection '{resolved.reverse_property_id}' "
+                            f"in target view {resolved.reverse_view_ref!s}."
                         ),
                         fix="Add the missing property to the container",
                         code=self.code,
@@ -434,46 +375,21 @@ class ReverseConnectionContainerPropertyWrongType(DataModelRule):
     def validate(self) -> list[ConsistencyError]:
         errors: list[ConsistencyError] = []
 
-        for (target_view_ref, reverse_prop_name), (
-            source_view_ref,
-            through,
-        ) in self.validation_resources.reverse_to_direct_mapping.items():
-            through = _normalize_through_reference(source_view_ref, through)
-            source_view = self.validation_resources.select_view(source_view_ref, through.identifier)
-
-            if not source_view:
-                continue  # Handled by ReverseConnectionSourceViewMissing
-
-            if not (source_view_expanded := self.validation_resources.expand_view_properties(source_view_ref)):
-                raise RuntimeError(f"{type(self).__name__}: View {source_view_ref!s} not found. This is a bug in NEAT.")
-
-            if not source_view_expanded.properties or through.identifier not in source_view_expanded.properties:
-                continue  # Handled by ReverseConnectionSourcePropertyMissing
-
-            source_property = source_view_expanded.properties[through.identifier]
-
-            if not isinstance(source_property, ViewCorePropertyRequest):
-                continue  # Handled by ReverseConnectionSourcePropertyWrongType
-
-            container_ref = source_property.container
-            container_property_id = source_property.container_property_identifier
-
-            source_container = self.validation_resources.select_container(container_ref, container_property_id)
-            if not source_container or not source_container.properties:
-                continue  # Handled by other validators
-
-            container_property = source_container.properties.get(container_property_id)
-            if not container_property:
+        for resolved in self.validation_resources.resolved_reverse_direct_relations:
+            if not resolved.container_property:
                 continue  # Handled by ReverseConnectionContainerPropertyMissing
 
-            if not isinstance(container_property.type, DirectNodeRelation):
+            if not isinstance(resolved.container_property.type, DirectNodeRelation):
                 errors.append(
                     ConsistencyError(
                         message=(
-                            f"Container property '{container_property_id}' in container {container_ref!s} "
-                            f"must be a direct connection, but found type '{container_property.type!s}'. "
-                            f"This property is used by source view {source_view_ref!s} property '{through.identifier}' "
-                            f"to configure reverse connection '{reverse_prop_name}' in target view {target_view_ref!s}."
+                            f"Container property '{resolved.container_property_id}' "
+                            f"in container {resolved.container_ref!s} "
+                            f"must be a direct connection, but found type '{resolved.container_property.type!s}'. "
+                            f"This property is used by source view {resolved.direct_view_ref!s} "
+                            f"property '{resolved.through_property_id}' "
+                            f"to configure reverse connection '{resolved.reverse_property_id}' "
+                            f"in target view {resolved.reverse_view_ref!s}."
                         ),
                         fix="Change container property type to be a direct connection",
                         code=self.code,
@@ -504,39 +420,17 @@ class ReverseConnectionTargetMissing(DataModelRule):
     def validate(self) -> list[Recommendation]:
         recommendations: list[Recommendation] = []
 
-        for (target_view_ref, reverse_prop_name), (
-            source_view_ref,
-            through,
-        ) in self.validation_resources.reverse_to_direct_mapping.items():
-            through = _normalize_through_reference(source_view_ref, through)
-            source_view = self.validation_resources.select_view(source_view_ref, through.identifier)
-
-            if not source_view:
-                continue  # Handled by ReverseConnectionSourceViewMissing
-
-            if not (source_view_expanded := self.validation_resources.expand_view_properties(source_view_ref)):
-                raise RuntimeError(f"{type(self).__name__}: View {source_view_ref!s} not found. This is a bug in NEAT.")
-
-            if not source_view_expanded.properties or through.identifier not in source_view_expanded.properties:
-                continue  # Handled by ReverseConnectionSourcePropertyMissing
-
-            source_property = source_view_expanded.properties[through.identifier]
-
-            if not isinstance(source_property, ViewCorePropertyRequest):
-                continue  # Handled by ReverseConnectionSourcePropertyWrongType
-
-            actual_target_view = source_property.source
-
-            if not actual_target_view:
+        for resolved in self.validation_resources.resolved_reverse_direct_relations:
+            if not resolved.direct_property.source:
                 recommendations.append(
                     Recommendation(
                         message=(
-                            f"Source view {source_view_ref!s} property '{through.identifier}' "
+                            f"Source view {resolved.direct_view_ref!s} property '{resolved.through_property_id}' "
                             f"has no target view specified (value type is None). "
-                            f"This property is used for reverse connection '{reverse_prop_name}' "
-                            f"in target view {target_view_ref!s}. "
+                            f"This property is used for reverse connection '{resolved.reverse_property_id}' "
+                            f"in target view {resolved.reverse_view_ref!s}. "
                             f"While this works as a hack for multi-value relations in CDF Search, "
-                            f"it's recommended to explicitly define the target view as {target_view_ref!s}."
+                            f"it's recommended to explicitly define the target view as {resolved.reverse_view_ref!s}."
                         ),
                         fix="Set the property's value type to the target view for better clarity",
                         code=self.code,
@@ -568,41 +462,24 @@ class ReverseConnectionPointsToAncestor(DataModelRule):
     def validate(self) -> list[Recommendation]:
         recommendations: list[Recommendation] = []
 
-        for (target_view_ref, reverse_prop_name), (
-            source_view_ref,
-            through,
-        ) in self.validation_resources.reverse_to_direct_mapping.items():
-            through = _normalize_through_reference(source_view_ref, through)
-            source_view = self.validation_resources.select_view(source_view_ref, through.identifier)
-
-            if not source_view:
-                continue  # Handled by ReverseConnectionSourceViewMissing
-
-            if not (source_view_expanded := self.validation_resources.expand_view_properties(source_view_ref)):
-                raise RuntimeError(f"{type(self).__name__}: View {source_view_ref!s} not found. This is a bug in NEAT.")
-
-            if not source_view_expanded.properties or through.identifier not in source_view_expanded.properties:
-                continue  # Handled by ReverseConnectionSourcePropertyMissing
-
-            source_property = source_view_expanded.properties[through.identifier]
-
-            if not isinstance(source_property, ViewCorePropertyRequest):
-                continue  # Handled by other validators
-
-            actual_target_view = source_property.source
+        for resolved in self.validation_resources.resolved_reverse_direct_relations:
+            actual_target_view = resolved.direct_property.source
 
             if not actual_target_view:
                 continue  # Handled by ReverseConnectionTargetMissing
 
-            if self.validation_resources.is_ancestor(target_view_ref, actual_target_view):
+            if self.validation_resources.is_ancestor(resolved.reverse_view_ref, actual_target_view):
                 recommendations.append(
                     Recommendation(
                         message=(
-                            f"The direct connection property '{through.identifier}' in view {source_view_ref!s} "
-                            f"configures the reverse connection '{reverse_prop_name}' in {target_view_ref!s}. "
-                            f"Therefore, it is expected that '{through.identifier}' points to {target_view_ref!s}. "
+                            f"The direct connection property '{resolved.through_property_id}' "
+                            f"in view {resolved.direct_view_ref!s} "
+                            f"configures the reverse connection '{resolved.reverse_property_id}' "
+                            f"in {resolved.reverse_view_ref!s}. "
+                            f"Therefore, it is expected that '{resolved.through_property_id}' "
+                            f"points to {resolved.reverse_view_ref!s}. "
                             f"However, it currently points to {actual_target_view!s}, which is an ancestor of "
-                            f"{target_view_ref!s}. "
+                            f"{resolved.reverse_view_ref!s}. "
                             "While this will allow for model to be valid, it can be a source of confusion and mistakes."
                         ),
                         fix="Update the direct connection property to point to the target view instead of its ancestor",
@@ -635,42 +512,24 @@ class ReverseConnectionTargetMismatch(DataModelRule):
     def validate(self) -> list[Recommendation]:
         recommendations: list[Recommendation] = []
 
-        for (target_view_ref, reverse_prop_name), (
-            source_view_ref,
-            through,
-        ) in self.validation_resources.reverse_to_direct_mapping.items():
-            through = _normalize_through_reference(source_view_ref, through)
-            source_view = self.validation_resources.select_view(source_view_ref, through.identifier)
-
-            if not source_view:
-                continue  # Handled by ReverseConnectionSourceViewMissing
-
-            if not (source_view_expanded := self.validation_resources.expand_view_properties(source_view_ref)):
-                raise RuntimeError(f"{type(self).__name__}: View {source_view_ref!s} not found. This is a bug in NEAT.")
-
-            if not source_view_expanded.properties or through.identifier not in source_view_expanded.properties:
-                continue  # Handled by ReverseConnectionSourcePropertyMissing
-
-            source_property = source_view_expanded.properties[through.identifier]
-
-            if not isinstance(source_property, ViewCorePropertyRequest):
-                continue  # Handled by other validators
-
-            actual_target_view = source_property.source
+        for resolved in self.validation_resources.resolved_reverse_direct_relations:
+            actual_target_view = resolved.direct_property.source
 
             if not actual_target_view:
                 continue  # Handled by ReverseConnectionTargetMissing
 
-            if self.validation_resources.is_ancestor(target_view_ref, actual_target_view):
-                continue  # Handled by ReverseConnectionTargetAncestor
+            if self.validation_resources.is_ancestor(resolved.reverse_view_ref, actual_target_view):
+                continue  # Handled by ReverseConnectionPointsToAncestor
 
-            if actual_target_view != target_view_ref:
+            if actual_target_view != resolved.reverse_view_ref:
                 recommendations.append(
                     Recommendation(
                         message=(
-                            f"The reverse connection '{reverse_prop_name}' in view {target_view_ref!s} "
-                            f"expects its corresponding direct connection in view {source_view_ref!s} "
-                            f"(property '{through.identifier}') to point back to {target_view_ref!s}, "
+                            f"The reverse connection '{resolved.reverse_property_id}' "
+                            f"in view {resolved.reverse_view_ref!s} "
+                            f"expects its corresponding direct connection in view {resolved.direct_view_ref!s} "
+                            f"(property '{resolved.through_property_id}') "
+                            f"to point back to {resolved.reverse_view_ref!s}, "
                             f"but it actually points to {actual_target_view!s}."
                         ),
                         fix="Update the direct connection property to point back to the correct target view",

cognite/neat/_data_model/rules/dms/_containers.py

@@ -1,6 +1,6 @@
 """Validators for checking containers in the data model."""
 
-from pyparsing import cast
+from typing import cast
 
 from cognite.neat._data_model.models.dms._constraints import Constraint, RequiresConstraintDefinition
 from cognite.neat._data_model.models.dms._view_property import ViewCorePropertyRequest

cognite/neat/_data_model/rules/dms/_performance.py

@@ -2,6 +2,8 @@
 
 from cognite.neat._data_model._analysis import RequiresChangeStatus
 from cognite.neat._data_model._constants import COGNITE_SPACES
+from cognite.neat._data_model.models.dms._data_types import DirectNodeRelation
+from cognite.neat._data_model.models.dms._indexes import BtreeIndex
 from cognite.neat._data_model.rules.dms._base import DataModelRule
 from cognite.neat._issues import Recommendation
 
@@ -193,3 +195,76 @@ class UnresolvableQueryPerformance(DataModelRule):
                 )
 
         return recommendations
+
+
+class MissingReverseDirectRelationTargetIndex(DataModelRule):
+    """
+    Recommends adding a cursorable index on direct relation properties that are
+    targets of reverse direct relations for query performance.
+
+    ## What it does
+    Identifies direct relation properties that are referenced by reverse direct relations
+    but lack a cursorable B-tree index. When querying through a reverse direct relation,
+    CDF needs to look up nodes that have the direct relation pointing to the
+    source nodes. Without an index, this requires scanning many nodes inefficiently.
+
+    ## Why is this important?
+    Traversing a reverse direct relation (inwards direction) requires an index on the
+    target direct relation property. Without this index, queries will be inefficient,
+    potentially leading to timeouts over time, as they won't scale well with data volume.
+
+    The exception is when the target direct relation is a list property. In that case,
+    this validator will not flag them, as reverse direct relations targeting lists of
+    direct relations needs to be traversed using the `instances/search` endpoint instead,
+    which does not directly benefit from adding indexes to container properties.
+
+    ## Example
+    View `WindFarm` has a reverse property `turbines` through `WindTurbine.windFarm`.
+    Container `WindTurbine` should have a cursorable B-tree index on the `windFarm`
+    property to enable efficient traversal from WindFarm to its turbines.
+    """
+
+    code = f"{BASE_CODE}-004"
+    issue_type = Recommendation
+    alpha = True
+
+    def validate(self) -> list[Recommendation]:
+        recommendations: list[Recommendation] = []
+
+        for resolved in self.validation_resources.resolved_reverse_direct_relations:
+            # Skip if container or container property couldn't be resolved
+            if not resolved.container or not resolved.container_property:
+                continue
+
+            # Must be a DirectNodeRelation type (other types handled by ReverseConnectionContainerPropertyWrongType)
+            if not isinstance(resolved.container_property.type, DirectNodeRelation):
+                continue
+
+            # Skip if this is a list direct relation - indexes are not supported for list properties
+            if resolved.container_property.type.list:
+                continue
+
+            # Skip if there's already a cursorable B-tree index on this property
+            if resolved.container.indexes and any(
+                isinstance(index, BtreeIndex)
+                and index.cursorable
+                and resolved.container_property_id in index.properties
+                for index in resolved.container.indexes.values()
+            ):
+                continue
+
+            recommendations.append(
+                Recommendation(
+                    message=(
+                        f"View '{resolved.reverse_view_ref!s}' has a reverse direct relation "
+                        f"'{resolved.reverse_property_id}' that points to container "
+                        f"'{resolved.container_ref!s}' property '{resolved.container_property_id}'. "
+                        f"Add a cursorable B-tree index on this target container property "
+                        f"to enable efficient query traversal."
+                    ),
+                    fix="Add a cursorable B-tree index on the target direct relation property",
+                    code=self.code,
+                )
+            )
+
+        return recommendations

cognite/neat/_version.py

@@ -1,2 +1,2 @@
-__version__ = "1.0.33"
+__version__ = "1.0.35"
 __engine__ = "^2.0.4"

{cognite_neat-1.0.33.dist-info → cognite_neat-1.0.35.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cognite-neat
-Version: 1.0.33
+Version: 1.0.35
 Summary: Knowledge graph transformation
 Author: Nikola Vasiljevic, Anders Albert
 Author-email: Nikola Vasiljevic <nikola.vasiljevic@cognite.com>, Anders Albert <anders.albert@cognite.com>

{cognite_neat-1.0.33.dist-info → cognite_neat-1.0.35.dist-info}/RECORD

@@ -17,7 +17,7 @@ cognite/neat/_client/statistics_api.py,sha256=3KNVsD2OV-EDFb1eqIrCXfYwjvyCDO80bP
 cognite/neat/_client/views_api.py,sha256=YMaw7IaxU4gmixpd_t1u9JK9BHfNerf5DMNinGPCAa0,3692
 cognite/neat/_config.py,sha256=IJS_R-86M4SLxdo644LZ0dE9h0jg2XBL1A4QKWJj0TQ,10160
 cognite/neat/_data_model/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-cognite/neat/_data_model/_analysis.py,sha256=sZO2NATkyuSzlo3uRgXJ6B3btKWkwblNos7SXT6LHLc,43779
+cognite/neat/_data_model/_analysis.py,sha256=VUheUE1xZcfXxOzF9wW6iOmsbWWIg_GB6X6HoD0PYbU,47799
 cognite/neat/_data_model/_constants.py,sha256=pPDtRwBsOq80IkYXRJ0XaRfFOb7CPvxz81IQzcPgAuc,1889
 cognite/neat/_data_model/_identifiers.py,sha256=lDLvMvYDgRNFgk5GmxWzOUunG7M3synAciNjzJI0m_o,1913
 cognite/neat/_data_model/_shared.py,sha256=H0gFqa8tKFNWuvdat5jL6OwySjCw3aQkLPY3wtb9Wrw,1302
@@ -48,7 +48,7 @@ cognite/neat/_data_model/importers/_table_importer/source.py,sha256=h7u5ur5oetmv
 cognite/neat/_data_model/models/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 cognite/neat/_data_model/models/conceptual/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 cognite/neat/_data_model/models/conceptual/_base.py,sha256=SFkoBJDM51pqew_isHFJoB20OgfofpwVRnTrg-rKkNY,710
-cognite/neat/_data_model/models/conceptual/_concept.py,sha256=0Pk4W2TJ_Y0Z7oPHpzely1kPXrAkmkyqw6a0n3il6LY,2248
+cognite/neat/_data_model/models/conceptual/_concept.py,sha256=eQMOb3tJJHvopr3QqP3XkP5GjBO4UXTKoVy_eSEwzEk,2245
 cognite/neat/_data_model/models/conceptual/_data_model.py,sha256=zvJZi0OqOFMviwY84Am3Oz_RmF1g3tOzANFHlrldBuU,1673
 cognite/neat/_data_model/models/conceptual/_properties.py,sha256=CpF37vJYBTLT4DH4ZOu2U-JyWtkb_27V8fw52qiaE_k,4007
 cognite/neat/_data_model/models/conceptual/_property.py,sha256=blSZQxX52zaILAtjUkldPzPeysz7wnG-UGSNU5tacI8,4138
@@ -81,15 +81,15 @@ cognite/neat/_data_model/rules/cdf/__init__.py,sha256=Bn9_DkkO4OPUYh0CsM3Pf1fD_U
 cognite/neat/_data_model/rules/cdf/_base.py,sha256=59fqmFi_Gvzx3swLsIyc2ACkHAzKAKiqNlwwGnW5bDg,128
 cognite/neat/_data_model/rules/cdf/_orchestrator.py,sha256=egzcYUuFDd8Vm7neBpZFofqZBfYjuElAjYwJWAScutc,2250
 cognite/neat/_data_model/rules/cdf/_spaces.py,sha256=Lsfrp2e7yzgRimkD0qynJ5M2S7Zwzplfg_4Pim1voHk,1605
-cognite/neat/_data_model/rules/dms/__init__.py,sha256=ThCvtSCPzEhciJNnzBKDoTWMimw5U74MbfUwoF6e7F8,2950
+cognite/neat/_data_model/rules/dms/__init__.py,sha256=LiZNGYClXHrgSbG0K6tk8nYN2B0VX3JlKn_minyCiI8,3042
 cognite/neat/_data_model/rules/dms/_ai_readiness.py,sha256=1KvOk3Q8aQkE8G6oHml7O-SHiTeIBNmyKPWr3hPcCCw,16133
 cognite/neat/_data_model/rules/dms/_base.py,sha256=BnzBELon389lEXJi7dlb9t3SPjiXgVquiI445I7-APU,134
-cognite/neat/_data_model/rules/dms/_connections.py,sha256=vi_cJp5S4MpfX-jA1eqZClnHdKznkQMDvZk9iIj42mM,30343
+cognite/neat/_data_model/rules/dms/_connections.py,sha256=yvlcb_-lBILZpMZ9Zcp8R1sC-G6DhCdyqjffwOnlmUI,23416
 cognite/neat/_data_model/rules/dms/_consistency.py,sha256=7u-EDQk08eV4CJoY5nRbrYXs2cbkQkH6TQCcPOyUGI8,2425
-cognite/neat/_data_model/rules/dms/_containers.py,sha256=eb0eLy13n7OozKp2vrE_oXqD6m2uyX-9fhoCqU_gJqc,10542
+cognite/neat/_data_model/rules/dms/_containers.py,sha256=R67rAlA2V9oMbp0bQEpHu8VF1Yr3f8xXgox7NkQtWk8,10539
 cognite/neat/_data_model/rules/dms/_limits.py,sha256=L6dAsLWVBEpfVFZgw3SvJk1PnEGf4lhJ3SOXjDUsY2Q,14820
 cognite/neat/_data_model/rules/dms/_orchestrator.py,sha256=tTXxLH1yThAeTW_GdDeD-to19faK72c_z8QL9kzgGJc,3069
-cognite/neat/_data_model/rules/dms/_performance.py,sha256=7LTs1E86xA-hvL4fqiIhAHcgIeohwb1pODzh3o6tuzQ,8601
+cognite/neat/_data_model/rules/dms/_performance.py,sha256=Tp7D5cCEOqjfst9tLKuA9X2F9f4lQLJHEiGojm-rQUI,12229
 cognite/neat/_data_model/rules/dms/_views.py,sha256=Tp4VW2512eGSGgeMRoUFh7hqK2vsZD8ryN_EJ7QJoEg,6335
 cognite/neat/_exceptions.py,sha256=hOjPL1vFNNAZzqAHFB9l9ek-XJEBKcqiaPk0onwLPns,2540
 cognite/neat/_issues.py,sha256=wH1mnkrpBsHUkQMGUHFLUIQWQlfJ_qMfdF7q0d9wNhY,1871
@@ -333,9 +333,9 @@ cognite/neat/_v0/session/_template.py,sha256=BNcvrW5y7LWzRM1XFxZkfR1Nc7e8UgjBClH
 cognite/neat/_v0/session/_to.py,sha256=AnsRSDDdfFyYwSgi0Z-904X7WdLtPfLlR0x1xsu_jAo,19447
 cognite/neat/_v0/session/_wizard.py,sha256=baPJgXAAF3d1bn4nbIzon1gWfJOeS5T43UXRDJEnD3c,1490
 cognite/neat/_v0/session/exceptions.py,sha256=jv52D-SjxGfgqaHR8vnpzo0SOJETIuwbyffSWAxSDJw,3495
-cognite/neat/_version.py,sha256=1SKg9QyerZ0e2dpVZc97ky2WseNfvIsmQKsSm-Ot5NM,45
+cognite/neat/_version.py,sha256=pfiLq3MLQ7wyaAim2tr-d61GfU9oqrmB9vFVh7JRzNY,45
 cognite/neat/legacy.py,sha256=DMFeLCSBLT2enk-nm1KfX1rKR2DQDpxY-w6ThY0y9c8,421
 cognite/neat/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-cognite_neat-1.0.33.dist-info/WHEEL,sha256=e_m4S054HL0hyR3CpOk-b7Q7fDX6BuFkgL5OjAExXas,80
-cognite_neat-1.0.33.dist-info/METADATA,sha256=2xWjKg7ErSy3X3GJbvybZB952bXdMJPT5ETumyMEahw,6872
-cognite_neat-1.0.33.dist-info/RECORD,,
+cognite_neat-1.0.35.dist-info/WHEEL,sha256=5DEXXimM34_d4Gx1AuF9ysMr1_maoEtGKjaILM3s4w4,80
+cognite_neat-1.0.35.dist-info/METADATA,sha256=ra9SuJzQhAi_Yw6BM6qV-nnNNQ8DUCtCDG2RCe1hVxs,6872
+cognite_neat-1.0.35.dist-info/RECORD,,

{cognite_neat-1.0.33.dist-info → cognite_neat-1.0.35.dist-info}/WHEEL

@@ -1,4 +1,4 @@
 Wheel-Version: 1.0
-Generator: uv 0.9.27
+Generator: uv 0.9.29
 Root-Is-Purelib: true
 Tag: py3-none-any