fix(seer): Replace internal Unicode operators with readable labels in LLM context

Widget conditions use \uf00d-delimited operators (e.g. Containsqueue.task)
that are invisible but garble the Seer agent's understanding, causing
broken tool calls. Replace these with readable labels (e.g. " contains ")
via simple string replacement that preserves all query structure — AND,
OR, parens, free text, and comparison operators pass through unchanged.

Co-Authored-By: Claude Opus 4.6 <noreply@example.com>

Author: Mihir-Mavalankar

static/app/views/dashboards/widgetCard/index.tsx

@@ -64,7 +64,7 @@ import {
   useTransactionsDeprecationWarning,
 } from './widgetCardContextMenu';
 import {WidgetFrame} from './widgetFrame';
-import {getWidgetQueryLLMHint} from './widgetLLMContext';
+import {getWidgetQueryLLMHint, readableConditions} from './widgetLLMContext';
 
 export type OnDataFetchedParams = {
   tableResults?: TableDataWithTitle[];
@@ -165,7 +165,7 @@ function WidgetCard(props: Props) {
     queryHint: getWidgetQueryLLMHint(resolvedDisplayType),
     queries: props.widget.queries.map(q => ({
       name: q.name,
-      conditions: q.conditions,
+      conditions: readableConditions(q.conditions),
       aggregates: q.aggregates,
       columns: q.columns,
       orderby: q.orderby,

static/app/views/dashboards/widgetCard/widgetLLMContext.spec.tsx

@@ -1,6 +1,6 @@
 import {DisplayType} from 'sentry/views/dashboards/types';
 
-import {getWidgetQueryLLMHint} from './widgetLLMContext';
+import {getWidgetQueryLLMHint, readableConditions} from './widgetLLMContext';
 
 describe('getWidgetQueryLLMHint', () => {
   it.each([
@@ -26,3 +26,125 @@ describe('getWidgetQueryLLMHint', () => {
     expect(getWidgetQueryLLMHint(DisplayType.WHEEL)).toContain('table query');
   });
 });
+
+describe('readableConditions', () => {
+  it('replaces Contains operator with readable label', () => {
+    expect(readableConditions('span.name:\uf00dContains\uf00dfoo')).toBe(
+      'span.name: contains foo'
+    );
+  });
+
+  it('replaces Contains with IN list brackets', () => {
+    expect(readableConditions('span.name:\uf00dContains\uf00d[a,b,c]')).toBe(
+      'span.name: contains [a,b,c]'
+    );
+  });
+
+  it('replaces DoesNotContain operator', () => {
+    expect(readableConditions('key:\uf00dDoesNotContain\uf00dval')).toBe(
+      'key: does not contain val'
+    );
+  });
+
+  it('replaces StartsWith and EndsWith operators', () => {
+    expect(readableConditions('key:\uf00dStartsWith\uf00d/api')).toBe(
+      'key: starts with /api'
+    );
+    expect(readableConditions('key:\uf00dEndsWith\uf00d.json')).toBe(
+      'key: ends with .json'
+    );
+  });
+
+  it('replaces DoesNotStartWith and DoesNotEndWith operators', () => {
+    expect(readableConditions('key:\uf00dDoesNotStartWith\uf00d/api')).toBe(
+      'key: does not start with /api'
+    );
+    expect(readableConditions('key:\uf00dDoesNotEndWith\uf00d.json')).toBe(
+      'key: does not end with .json'
+    );
+  });
+
+  it('preserves negated filter prefix', () => {
+    expect(readableConditions('!path:\uf00dContains\uf00dfoo')).toBe(
+      '!path: contains foo'
+    );
+  });
+
+  it('replaces multiple operators in one string', () => {
+    const input =
+      'span.name:\uf00dContains\uf00dqueue.task !trigger_path:\uf00dContains\uf00dold_seer';
+    expect(readableConditions(input)).toBe(
+      'span.name: contains queue.task !trigger_path: contains old_seer'
+    );
+  });
+
+  it('passes through plain filters unchanged', () => {
+    expect(readableConditions('browser.name:Firefox')).toBe('browser.name:Firefox');
+  });
+
+  it('passes through free text unchanged', () => {
+    expect(readableConditions('some free text')).toBe('some free text');
+  });
+
+  it('passes through empty string', () => {
+    expect(readableConditions('')).toBe('');
+  });
+
+  it('preserves OR and parentheses', () => {
+    expect(readableConditions('(a:1 OR b:2) error')).toBe('(a:1 OR b:2) error');
+  });
+
+  it('preserves comparison operators', () => {
+    expect(readableConditions('count():>100 duration:<=5s')).toBe(
+      'count():>100 duration:<=5s'
+    );
+  });
+
+  it('handles real-world widget query', () => {
+    const input =
+      'span.description:\uf00dContains\uf00d[sentry.tasks.autofix.generate_issue_summary_only,sentry.tasks.autofix.run_automation_only_task] span.name:\uf00dContains\uf00dqueue.task.taskworker !trigger_path:\uf00dContains\uf00dold_seer_automation';
+    expect(readableConditions(input)).toBe(
+      'span.description: contains [sentry.tasks.autofix.generate_issue_summary_only,sentry.tasks.autofix.run_automation_only_task] span.name: contains queue.task.taskworker !trigger_path: contains old_seer_automation'
+    );
+  });
+
+  it('does not replace DoesNotContain partially as Contains', () => {
+    // DoesNotContain must be replaced before Contains to avoid partial match
+    expect(readableConditions('key:\uf00dDoesNotContain\uf00dval')).toBe(
+      'key: does not contain val'
+    );
+    // Should NOT produce "key: does not contains val" or "key:DoesNot contains val"
+    expect(readableConditions('key:\uf00dDoesNotContain\uf00dval')).not.toContain(
+      '\uf00d'
+    );
+  });
+
+  it('handles mixed operator types in one query', () => {
+    const input =
+      'url:\uf00dStartsWith\uf00d/api span.description:\uf00dContains\uf00dfoo !path:\uf00dDoesNotEndWith\uf00d.js';
+    expect(readableConditions(input)).toBe(
+      'url: starts with /api span.description: contains foo !path: does not end with .js'
+    );
+  });
+
+  it('handles the same operator appearing multiple times', () => {
+    const input =
+      'a:\uf00dContains\uf00dfoo b:\uf00dContains\uf00dbar c:\uf00dContains\uf00dbaz';
+    expect(readableConditions(input)).toBe(
+      'a: contains foo b: contains bar c: contains baz'
+    );
+  });
+
+  it('preserves OR with wildcard operators inside parens', () => {
+    const input =
+      '(span.name:\uf00dContains\uf00dfoo OR span.name:\uf00dContains\uf00dbar)';
+    expect(readableConditions(input)).toBe(
+      '(span.name: contains foo OR span.name: contains bar)'
+    );
+  });
+
+  it('does not replace literal "Contains" text without unicode markers', () => {
+    // The word "Contains" in a value or free text should NOT be replaced
+    expect(readableConditions('message:Contains error')).toBe('message:Contains error');
+  });
+});

static/app/views/dashboards/widgetCard/widgetLLMContext.tsx

@@ -1,5 +1,22 @@
 import {DisplayType} from 'sentry/views/dashboards/types';
 
+/**
+ * Replace internal \uf00d-delimited wildcard operators with readable labels
+ * so the Seer Explorer agent can understand widget filter conditions.
+ *
+ * All other query structure (AND, OR, parens, free text, comparison operators)
+ * passes through unchanged since only wildcard operators use \uf00d markers.
+ */
+export function readableConditions(query: string): string {
+  return query
+    .replaceAll('\uf00dDoesNotContain\uf00d', ' does not contain ')
+    .replaceAll('\uf00dDoesNotStartWith\uf00d', ' does not start with ')
+    .replaceAll('\uf00dDoesNotEndWith\uf00d', ' does not end with ')
+    .replaceAll('\uf00dContains\uf00d', ' contains ')
+    .replaceAll('\uf00dStartsWith\uf00d', ' starts with ')
+    .replaceAll('\uf00dEndsWith\uf00d', ' ends with ');
+}
+
 /**
  * Returns a hint for the Seer Explorer agent describing how to re-query this
  * widget's data using a tool call, if the user wants to dig deeper.