[OPIK-839] [FE] Remove not leaves nodes(subtrees) from the autocomple…

…te component in the rules popup (#1099)
comet-ml · Jan 21, 2025 · f230645 · f230645
1 parent 8a665a7
commit f230645
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 18 deletions.
diff --git a/...nd/src/components/pages-shared/traces/TracesPathsAutocomplete/TracesPathsAutocomplete.tsx b/...nd/src/components/pages-shared/traces/TracesPathsAutocomplete/TracesPathsAutocomplete.tsx
@@ -37,22 +37,19 @@ const TracesPathsAutocomplete: React.FC<TracesPathsAutocompleteProps> = ({
 
   const items = useMemo(() => {
     return uniq(
-      (data?.content || []).reduce<string[]>(
-        (acc, d) => {
-          return acc.concat(
-            rootKeys.reduce<string[]>(
-              (internalAcc, key) =>
-                internalAcc.concat(
-                  isObject(d[key]) || isArray(d[key])
-                    ? getJSONPaths(d[key], key)
-                    : [],
-                ),
-              [],
-            ),
-          );
-        },
-        [...rootKeys],
-      ),
+      (data?.content || []).reduce<string[]>((acc, d) => {
+        return acc.concat(
+          rootKeys.reduce<string[]>(
+            (internalAcc, key) =>
+              internalAcc.concat(
+                isObject(d[key]) || isArray(d[key])
+                  ? getJSONPaths(d[key], key)
+                  : [],
+              ),
+            [],
+          ),
+        );
+      }, []),
     )
       .filter((p) =>
         value ? p.toLowerCase().includes(value.toLowerCase()) : true,

diff --git a/apps/opik-frontend/src/lib/utils.ts b/apps/opik-frontend/src/lib/utils.ts
@@ -57,10 +57,8 @@ export const getJSONPaths = (
         : key;
 
       if (isArray(value)) {
-        results.push(path);
         getJSONPaths(value, path, results);
       } else if (isObject(value)) {
-        results.push(path);
         getJSONPaths(value, path, results);
       } else {
         results.push(path);