fix: handle None from_entity in Context API RelationSummary

claude[bot] · groksrc · web-flow · commit a21c66eaa87c · 2025-06-23T04:30:15.000Z
Add null check before accessing from_entity.title to prevent AttributeError when relations exist but referenced entities are deleted or inconsistent. Follows the same pattern already used for to_entity handling. Fixes #165 Co-authored-by: Drew Cain <groksrc@users.noreply.github.com>
diff --git a/src/basic_memory/api/routers/utils.py b/src/basic_memory/api/routers/utils.py
@@ -52,7 +52,7 @@ async def to_summary(item: SearchIndexRow | ContextResultRow):
                     file_path=item.file_path,
                     permalink=item.permalink,  # pyright: ignore
                     relation_type=item.relation_type,  # pyright: ignore
-                    from_entity=from_entity.title,  # pyright: ignore
+                    from_entity=from_entity.title if from_entity else None,
                     to_entity=to_entity.title if to_entity else None,
                     created_at=item.created_at,
                 )

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ async def to_summary(item: SearchIndexRow \| ContextResultRow):`
`52`	`52`	`file_path=item.file_path,`
`53`	`53`	`permalink=item.permalink, # pyright: ignore`
`54`	`54`	`relation_type=item.relation_type, # pyright: ignore`
`55`		`- from_entity=from_entity.title, # pyright: ignore`
	`55`	`+ from_entity=from_entity.title if from_entity else None,`
`56`	`56`	`to_entity=to_entity.title if to_entity else None,`
`57`	`57`	`created_at=item.created_at,`
`58`	`58`	`)`