Improve LogContext Sidebar WIP

Author: ddelemeny

src/LogContext/components/LogContextQueryBuilderSidebar.tsx

@@ -1,17 +1,21 @@
 import React, { useEffect, useMemo, useState } from "react";
-import { Field } from '@grafana/data';
+// import { Field } from '@grafana/data';
 import { CollapsableSection } from '@grafana/ui';
 import { LogContextProps } from "./LogContextUI";
 import { css } from "@emotion/css";
 import { QuickwitQuery } from 'LogContext/QueryBuilder';
 
-const filterFields = [
-  'attributes.appname',
-  'attributes.proc_id',
-  'attributes.hostname',
-  'attributes.priority',
-  'severity_number',
-  'severity_text'
+
+// TODO : define sensible defaults here
+const excludedFields = [
+  '_source',
+  'sort',
+  'attributes',
+  'attributes.message',
+  'body',
+  'body.message',
+  'observed_timestamp_nanos',
+  'timestamp_nanos',
 ];
 
 type FieldContingency = { [value: string]: {
@@ -69,29 +73,33 @@ export function LogContextQueryBuilderSidebar(props: LogContextProps & QueryBuil
 
   const {row, parsedQuery, updateQuery} = props;
   const [fields, setFields] = useState<Array<{ name: string; contingency: FieldContingency; }>>([]);
+
+
   const filteredFields = useMemo(() => (
-    row.dataFrame.fields.filter((f: Field) => (filterFields.includes(f.name)))
+    row.dataFrame.fields
+      // sort fields by name
+      .sort((f1, f2)=> (f1.name>f2.name ? 1 : -1))
+      // exclude some low-filterability fields
+      .filter((f)=> !excludedFields.includes(f.name)
+      )
   ), [row.dataFrame.fields]);
 
-
   useEffect(() => {
-    const fields = filteredFields.map((f) => {
-      const contingency: FieldContingency = {};
-
-      f.values.toArray().forEach((attrName, i) => {
-        if (!contingency[attrName]) {
-           contingency[attrName] = {
-            count: 0,
-            pinned: i === row.rowIndex,
-            active: parsedQuery ? parsedQuery.findFilter(`${f.name}:\"${attrName}\"`) > -1 : false
+    const fields = filteredFields
+      .map((f) => {
+        const contingency: FieldContingency = {};
+        f.values.toArray().forEach((attrName, i) => {
+          if (!contingency[attrName]) {
+            contingency[attrName] = {
+              count: 0,
+              pinned: i === row.rowIndex,
+              active: parsedQuery ? parsedQuery.findFilter(`${f.name}:\"${attrName}\"`) > -1 : false
+            }
           }
-        }
-        contingency[attrName].count += 1;  
-      });
-      return {
-        name: f.name,
-        contingency };
-    })
+          contingency[attrName].count += 1;  
+        });
+        return { name: f.name, contingency };
+      })
 
     setFields(fields);
   }, [filteredFields, row.rowIndex, parsedQuery]);