fix(seg-fault): attempt

Author: Daniele Briggi

src/sqlite_rag/models/document_result.py

@@ -42,8 +42,9 @@ def get_preview(
             return self.chunk_content[:max_chars]
 
         # Sort by start_offset to maintain document order
-        top_sentences.sort(
-            key=lambda s: s.start_offset if s.start_offset is not None else -1
+        top_sentences = sorted(
+            top_sentences,
+            key=lambda s: s.start_offset if s.start_offset is not None else -1,
         )
 
         preview_parts = []

tests/models/test_document_result.py

@@ -177,6 +177,47 @@ def test_get_preview_orders_sentences_by_offset(self):
         # Should be in document order despite rank order
         assert "First sentence. [...] Third sentence." == preview
 
+    def test_get_preview_orders_sentences_stress_test(self):
+        """Stress test to detect race conditions in sentence ordering."""
+        import gc
+
+        for _ in range(100):
+            doc = Document(uri="test.txt", content="test content")
+
+            sentences = [
+                SentenceResult(
+                    chunk_id=1,
+                    id=3,
+                    content="Third sentence.",
+                    rank=1,
+                    distance=0.1,
+                    start_offset=66,
+                    end_offset=82,
+                ),
+                SentenceResult(
+                    chunk_id=1,
+                    id=1,
+                    content="First sentence.",
+                    rank=2,
+                    distance=0.2,
+                    start_offset=0,
+                    end_offset=15,
+                ),
+            ]
+
+            result = DocumentResult(
+                document=doc,
+                chunk_id=1,
+                combined_rank=1.0,
+                sentences=sentences,
+            )
+
+            preview = result.get_preview(max_chars=400)
+            assert "First sentence. [...] Third sentence." == preview
+
+            # Force garbage collection to expose potential memory issues
+            gc.collect()
+
     def test_get_preview_limits_to_top_k_sentences(self):
         """Test get_preview respects top_k_sentences parameter."""
         doc = Document(uri="test.txt", content="test content")