fix(explore): Recurse into logic groups when extracting filter keys

extractFilterKeys only iterated top-level tokens, so filter keys nested
inside parenthesized groups (Token.LOGIC_GROUP) were never sent to the
validation API. Add a recursive walk to inspect inner tokens.

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

Author: nsdeschenes

static/app/views/explore/hooks/useAttributeValidation.spec.tsx

@@ -115,6 +115,42 @@ describe('useAttributeValidation', () => {
     expect(mockValidate).toHaveBeenCalledTimes(1);
   });
 
+  it('extracts filter keys inside logical groups', async () => {
+    const mockValidate = MockApiClient.addMockResponse({
+      url: `/organizations/${ORG_SLUG}/trace-items/attributes/validate/`,
+      method: 'POST',
+      body: {
+        attributes: {
+          'span.op': {valid: true},
+          'span.module': {valid: false, error: 'Unknown attribute'},
+        },
+      },
+    });
+
+    const {result} = renderHookWithProviders(
+      () =>
+        useAttributeValidation(
+          TraceItemDataset.SPANS,
+          '(span.op:db OR span.module:http)',
+          DEFAULT_SELECTION
+        ),
+      {organization}
+    );
+
+    await waitFor(() => {
+      expect(result.current.invalidFilterKeys).toEqual(['span.module']);
+    });
+
+    expect(mockValidate).toHaveBeenCalledWith(
+      expect.any(String),
+      expect.objectContaining({
+        data: expect.objectContaining({
+          attributes: expect.arrayContaining(['span.module', 'span.op']),
+        }),
+      })
+    );
+  });
+
   it('re-validates when selection changes', async () => {
     const mockValidate = MockApiClient.addMockResponse({
       url: `/organizations/${ORG_SLUG}/trace-items/attributes/validate/`,

static/app/views/explore/hooks/useAttributeValidation.tsx

@@ -40,11 +40,16 @@ function extractFilterKeys(parsedQuery: ParseResult | null): string[] {
     return EMPTY_KEYS;
   }
   const keySet = new Set<string>();
-  for (const token of parsedQuery) {
-    if (token.type === Token.FILTER) {
-      keySet.add(getKeyName(token.key));
+  function walk(tokens: ParseResult) {
+    for (const token of tokens) {
+      if (token.type === Token.FILTER) {
+        keySet.add(getKeyName(token.key));
+      } else if (token.type === Token.LOGIC_GROUP) {
+        walk(token.inner);
+      }
     }
   }
+  walk(parsedQuery);
   return keySet.size > 0 ? [...keySet].sort() : EMPTY_KEYS;
 }