refactor(explore): remove duplicate React keys in ViewQueryModal

Addresses PR feedback:
- Remove redundant keys from Alert and ViewQuery components (Fragment already has key)
- Improve Fragment key to use content-based identifiers (query/error content) instead of array index

This follows React best practices by:
1. Avoiding duplicate keys on conditionally rendered children
2. Using stable, content-based keys instead of index-based keys

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

Author: sadpandajoe

superset-frontend/src/explore/components/controls/ViewQueryModal.tsx

@@ -89,26 +89,24 @@ const ViewQueryModal: FC<Props> = ({ latestQueryFormData }) => {
 
   return (
     <ViewQueryModalContainer>
-      {result.map((item, index) => (
-        <Fragment key={index}>
-          {item.error && (
-            <Alert
-              key={`error-${index}`}
-              type="error"
-              message={item.error}
-              closable={false}
-            />
-          )}
-          {item.query && (
-            <ViewQuery
-              key={`query-${index}`}
-              datasource={latestQueryFormData.datasource}
-              sql={item.query}
-              language={item.language}
-            />
-          )}
-        </Fragment>
-      ))}
+      {result.map((item, index) => {
+        // Use content-based key when available, fall back to index
+        const key = item.query || item.error || `result-${index}`;
+        return (
+          <Fragment key={key}>
+            {item.error && (
+              <Alert type="error" message={item.error} closable={false} />
+            )}
+            {item.query && (
+              <ViewQuery
+                datasource={latestQueryFormData.datasource}
+                sql={item.query}
+                language={item.language}
+              />
+            )}
+          </Fragment>
+        );
+      })}
     </ViewQueryModalContainer>
   );
 };