.

Author: sentrivana

MIGRATION_GUIDE.md

@@ -31,6 +31,7 @@ Looking to upgrade from Sentry SDK 2.x to 3.x? Here's a comprehensive list of wh
 - The option `attach_stacktrace` is now `True` by default, meaning the SDK will attach stack traces to messages.
 - The Django integration now creates spans for cache operations by default. Pass `DjangoIntegration(cache_spans=False)` to `sentry_sdk.init()` to turn them off.
 - The Django integration no longer force-enables cache spans when Spotlight is active and `settings.DEBUG` is `True`. The `cache_spans` option is now always respected as given.
+- Exception groups in exception chains are now properly unfurled.
 
 ### Logging
 

tests/test_exceptiongroup.py

@@ -317,6 +317,90 @@ def test_simple_exception():
     assert exception_values == expected_exception_values
 
 
+@minimum_python_311
+def test_exception_group_chained_with_context():
+    try:
+        try:
+            raise ExceptionGroup(
+                "group",
+                [
+                    ValueError("child1"),
+                    ExceptionGroup(
+                        "child2",
+                        [
+                            RuntimeError("grandchild1"),
+                            RuntimeError("grandchild2"),
+                        ],
+                    ),
+                ],
+            )
+        finally:
+            raise TypeError("bar")
+    except BaseException as e:
+        exc = e
+
+    (event, _) = event_from_exception(
+        exc,
+        client_options={
+            "include_local_variables": True,
+            "include_source_context": True,
+            "max_value_length": 1024,
+        },
+        mechanism={"type": "test_suite", "handled": False},
+    )
+
+    exception_values = event["exception"]["values"]
+
+    # innermost (oldest) to outermost (newest)
+    assert [(e["type"], e["value"]) for e in exception_values] == [
+        ("RuntimeError", "2"),
+        ("RuntimeError", "1"),
+        ("ExceptionGroup", "child2"),
+        ("ValueError", "child1"),
+        ("ExceptionGroup", "group"),
+        ("TypeError", "bar"),
+    ]
+
+    # TypeError("bar") is the outermost exception (exception_id=0)
+    type_error = exception_values[-1]
+    assert type_error["mechanism"]["type"] == "test_suite"
+    assert type_error["mechanism"]["exception_id"] == 0
+
+    # ExceptionGroup("group") is the __context__ of TypeError
+    group = exception_values[-2]
+    assert group["mechanism"]["type"] == "chained"
+    assert group["mechanism"]["source"] == "__context__"
+    assert group["mechanism"]["parent_id"] == 0
+    assert group["mechanism"]["is_exception_group"] is True
+
+    group_id = group["mechanism"]["exception_id"]
+
+    # ValueError("child1") and ExceptionGroup("child2") are children of "group"
+    child1 = exception_values[-3]
+    assert child1["type"] == "ValueError"
+    assert child1["mechanism"]["source"] == "exceptions[0]"
+    assert child1["mechanism"]["parent_id"] == group_id
+
+    child2 = exception_values[-4]
+    assert child2["type"] == "ExceptionGroup"
+    assert child2["mechanism"]["source"] == "exceptions[1]"
+    assert child2["mechanism"]["parent_id"] == group_id
+    assert child2["mechanism"]["is_exception_group"] is True
+
+    child2_id = child2["mechanism"]["exception_id"]
+
+    # RuntimeError("grandchild1") and RuntimeError("grandchild2") are children of "child2"
+    rt1 = exception_values[-5]
+    assert rt1["value"] == "grandchild1"
+    assert rt1["mechanism"]["source"] == "exceptions[0]"
+    assert rt1["mechanism"]["parent_id"] == child2_id
+
+    rt2 = exception_values[-6]
+    assert rt2["value"] == "grandchild2"
+    assert rt2["mechanism"]["source"] == "exceptions[1]"
+    assert rt2["mechanism"]["parent_id"] == child2_id
+
+
 @minimum_python_311
 def test_exceptiongroup_starlette_collapse():
     """