Remove strategy, leave fingerprinting

Author: mujacica

src/sentry/grouping/strategies/configurations.py

@@ -72,16 +72,3 @@ def register_grouping_config(id: str, **kwargs) -> type[StrategyConfiguration]:
     initial_context={},
     enhancements_base="all-platforms:2025-11-21",
 )
-
-# Experimental config with thread-aware grouping
-THREAD_GROUPING_CONFIG = "newstyle:2025-with-threads"
-register_grouping_config(
-    id=THREAD_GROUPING_CONFIG,
-    base=FALL_2025_GROUPING_CONFIG,
-    initial_context={
-        # Enable thread name in automatic grouping
-        # When enabled, errors in different threads (by name) will create separate issues
-        "include_thread_name_in_grouping": True
-    },
-    enhancements_base="all-platforms:2025-11-21",
-)

src/sentry/grouping/strategies/newstyle.py

@@ -8,7 +8,6 @@
 from typing import TYPE_CHECKING, Any
 
 from sentry.grouping.component import (
-    BaseGroupingComponent,
     ChainedExceptionGroupingComponent,
     ContextLineGroupingComponent,
     ErrorTypeGroupingComponent,
@@ -23,7 +22,6 @@
     NSErrorDomainGroupingComponent,
     NSErrorGroupingComponent,
     StacktraceGroupingComponent,
-    ThreadNameGroupingComponent,
     ThreadsGroupingComponent,
 )
 from sentry.grouping.strategies.base import (
@@ -615,10 +613,19 @@ def single_exception(
             )
 
         values: list[ExceptionGroupingComponentChildren] = []
+
         if ns_error_component is not None:
-            values = [stacktrace_component, type_component, ns_error_component, value_component]
+            values = [type_component, value_component, ns_error_component, stacktrace_component]
         else:
-            values = [stacktrace_component, type_component, value_component]
+            values = [type_component, value_component, stacktrace_component]
+
+        # TODO: Once we're fully transitioned off of the `newstyle:2023-01-11` config, the code here
+        # (and the option controlling it) can be deleted
+        if context.get("use_legacy_exception_subcomponent_order"):
+            if ns_error_component is not None:
+                values = [stacktrace_component, type_component, ns_error_component, value_component]
+            else:
+                values = [stacktrace_component, type_component, value_component]
 
         exception_components_by_variant[variant_name] = ExceptionGroupingComponent(
             values=values, frame_counts=stacktrace_component.frame_counts
@@ -875,46 +882,24 @@ def threads(
 def _get_thread_components(
     threads: list[dict[str, Any]], event: Event, context: GroupingContext, **kwargs: dict[str, Any]
 ) -> ComponentsByVariant | None:
-    # Only process single-thread scenarios (existing behavior)
-    # Multiple threads are ambiguous for grouping
     if len(threads) != 1:
         return None
 
-    thread = threads[0]
-    stacktrace = thread.get("stacktrace")
-
-    # Check if config enables thread name in grouping via initial_context
-    include_thread_name = context.get("include_thread_name_in_grouping", False)
-
-    # Collect thread metadata components (only thread name, as IDs are random)
-    thread_metadata: list[BaseGroupingComponent[str]] = []
-    if include_thread_name and thread.get("name"):
-        thread_metadata.append(
-            ThreadNameGroupingComponent(
-                values=[thread["name"]],
-                contributes=True,
-                hint="thread name included in grouping",
-            )
-        )
-
-    # If no stacktrace, return early (thread metadata can't group without frames)
+    stacktrace = threads[0].get("stacktrace")
     if not stacktrace:
         return {
             "app": ThreadsGroupingComponent(
                 contributes=False, hint="ignored because thread has no stacktrace"
             )
         }
 
-    # Build stacktrace components for each variant with thread metadata
     thread_components_by_variant = {}
 
     for variant_name, stacktrace_component in context.get_grouping_components_by_variant(
         stacktrace, event=event, **kwargs
     ).items():
         thread_components_by_variant[variant_name] = ThreadsGroupingComponent(
-            values=[stacktrace_component],
-            frame_counts=stacktrace_component.frame_counts,
-            metadata=thread_metadata,
+            values=[stacktrace_component], frame_counts=stacktrace_component.frame_counts
         )
 
     return thread_components_by_variant