@parca/profile 0.19.6 → 0.19.7

package/CHANGELOG.md

@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [0.19.7](https://github.com/parca-dev/parca/compare/@parca/profile@0.19.6...@parca/profile@0.19.7) (2025-06-24)
+
+**Note:** Version bump only for package @parca/profile
+
 ## [0.19.6](https://github.com/parca-dev/parca/compare/@parca/profile@0.19.5...@parca/profile@0.19.6) (2025-06-23)
 
 **Note:** Version bump only for package @parca/profile

package/dist/ProfileExplorer/index.d.ts

@@ -6,6 +6,7 @@ interface ProfileExplorerProps {
     navigateTo: NavigateFunction;
 }
 export declare const getExpressionAsAString: (expression: string | []) => string;
+export declare const filterEmptyParams: (o: Record<string, any>) => Record<string, any>;
 declare const ProfileExplorer: ({ queryClient, queryParams, navigateTo, }: ProfileExplorerProps) => JSX.Element;
 export default ProfileExplorer;
 //# sourceMappingURL=index.d.ts.map

package/dist/ProfileExplorer/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/ProfileExplorer/index.tsx"],"names":[],"mappings":"AAiBA,OAAO,EAAC,kBAAkB,EAAC,MAAM,eAAe,CAAC;AAIjD,OAAO,EAIL,KAAK,gBAAgB,EACtB,MAAM,kBAAkB,CAAC;AAU1B,UAAU,oBAAoB;IAC5B,WAAW,EAAE,kBAAkB,CAAC;IAChC,WAAW,EAAE,GAAG,CAAC;IACjB,UAAU,EAAE,gBAAgB,CAAC;CAC9B;AAaD,eAAO,MAAM,sBAAsB,eAAgB,MAAM,GAAG,EAAE,KAAG,MAGhE,CAAC;AAiXF,QAAA,MAAM,eAAe,8CAIlB,oBAAoB,KAAG,GAAG,CAAC,OAkB7B,CAAC;AAEF,eAAe,eAAe,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/ProfileExplorer/index.tsx"],"names":[],"mappings":"AAiBA,OAAO,EAAC,kBAAkB,EAAC,MAAM,eAAe,CAAC;AAIjD,OAAO,EAIL,KAAK,gBAAgB,EACtB,MAAM,kBAAkB,CAAC;AAU1B,UAAU,oBAAoB;IAC5B,WAAW,EAAE,kBAAkB,CAAC;IAChC,WAAW,EAAE,GAAG,CAAC;IACjB,UAAU,EAAE,gBAAgB,CAAC;CAC9B;AAaD,eAAO,MAAM,sBAAsB,eAAgB,MAAM,GAAG,EAAE,KAAG,MAGhE,CAAC;AAiBF,eAAO,MAAM,iBAAiB,MAAO,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,KAAG,MAAM,CAAC,MAAM,EAAE,GAAG,CAiB5E,CAAC;AA+UF,QAAA,MAAM,eAAe,8CAIlB,oBAAoB,KAAG,GAAG,CAAC,OAkB7B,CAAC;AAEF,eAAe,eAAe,CAAC"}

package/dist/ProfileExplorer/index.js

@@ -41,7 +41,7 @@ const sanitizeDateRange = (time_selection_a, from_a, to_a) => {
     return { time_selection_a: range.getRangeKey(), from_a, to_a };
 };
 /* eslint-enable @typescript-eslint/naming-convention */
-const filterEmptyParams = (o) => {
+export const filterEmptyParams = (o) => {
     return Object.fromEntries(Object.entries(o)
         .filter(([_, value]) => value !== '' && value !== undefined && (Array.isArray(value) ? value.length > 0 : true))
         .map(([key, value]) => {

package/dist/ProfileView/hooks/useResetStateOnNewSearch.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useResetStateOnNewSearch.d.ts","sourceRoot":"","sources":["../../../src/ProfileView/hooks/useResetStateOnNewSearch.ts"],"names":[],"mappings":"AAeA,eAAO,MAAM,wBAAwB,QAAO,CAAC,MAAM,IAAI,CAQtD,CAAC"}
+{"version":3,"file":"useResetStateOnNewSearch.d.ts","sourceRoot":"","sources":["../../../src/ProfileView/hooks/useResetStateOnNewSearch.ts"],"names":[],"mappings":"AAeA,eAAO,MAAM,wBAAwB,QAAO,CAAC,MAAM,IAAI,CAWtD,CAAC"}

package/dist/ProfileView/hooks/useResetStateOnNewSearch.js

@@ -12,9 +12,12 @@
 // limitations under the License.
 import { useURLState } from '@parca/components';
 export const useResetStateOnNewSearch = () => {
-    const [, setCurPath] = useURLState('cur_path');
+    const [val, setCurPath] = useURLState('cur_path');
     return () => {
         setTimeout(() => {
+            if (val === undefined) {
+                return;
+            }
             setCurPath(undefined);
         });
     };

package/dist/ProfileView/hooks/useResetStateOnProfileTypeChange.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useResetStateOnProfileTypeChange.d.ts","sourceRoot":"","sources":["../../../src/ProfileView/hooks/useResetStateOnProfileTypeChange.ts"],"names":[],"mappings":"AAeA,eAAO,MAAM,gCAAgC,QAAO,CAAC,MAAM,IAAI,CAgB9D,CAAC"}
+{"version":3,"file":"useResetStateOnProfileTypeChange.d.ts","sourceRoot":"","sources":["../../../src/ProfileView/hooks/useResetStateOnProfileTypeChange.ts"],"names":[],"mappings":"AAeA,eAAO,MAAM,gCAAgC,QAAO,CAAC,MAAM,IAAI,CA0B9D,CAAC"}

package/dist/ProfileView/hooks/useResetStateOnProfileTypeChange.js

@@ -12,18 +12,28 @@
 // limitations under the License.
 import { useURLState } from '@parca/components';
 export const useResetStateOnProfileTypeChange = () => {
-    const [, setGroupBy] = useURLState('group_by');
-    const [, setFilterByFunction] = useURLState('filter_by_function');
-    const [, setExcludeFunction] = useURLState('exclude_function');
-    const [, setSearchString] = useURLState('search_string');
-    const [, setCurPath] = useURLState('cur_path');
+    const [groupBy, setGroupBy] = useURLState('group_by');
+    const [filterByFunction, setFilterByFunction] = useURLState('filter_by_function');
+    const [excludeFunction, setExcludeFunction] = useURLState('exclude_function');
+    const [searchString, setSearchString] = useURLState('search_string');
+    const [curPath, setCurPath] = useURLState('cur_path');
     return () => {
         setTimeout(() => {
-            setGroupBy(undefined);
-            setFilterByFunction(undefined);
-            setExcludeFunction(undefined);
-            setSearchString(undefined);
-            setCurPath(undefined);
+            if (groupBy !== undefined) {
+                setGroupBy(undefined);
+            }
+            if (filterByFunction !== undefined) {
+                setFilterByFunction(undefined);
+            }
+            if (excludeFunction !== undefined) {
+                setExcludeFunction(undefined);
+            }
+            if (searchString !== undefined) {
+                setSearchString(undefined);
+            }
+            if (curPath !== undefined) {
+                setCurPath(undefined);
+            }
         });
     };
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@parca/profile",
-  "version": "0.19.6",
+  "version": "0.19.7",
   "description": "Profile viewing libraries",
   "dependencies": {
     "@floating-ui/react": "^0.27.12",
@@ -78,5 +78,5 @@
     "access": "public",
     "registry": "https://registry.npmjs.org/"
   },
-  "gitHead": "8d547b91b5f02fa8b2a1fdd48102b6cb2f844d6c"
+  "gitHead": "4bcbd7e4e41cbb9d57aee3160784e74a40997526"
 }

package/src/ProfileExplorer/index.test.ts

@@ -0,0 +1,97 @@
+// Copyright 2022 The Parca Authors
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+import {describe, expect, it} from 'vitest';
+
+import {filterEmptyParams} from './index';
+
+describe('filterEmptyParams', () => {
+  it('should return an array with 2 elements when given object with 2 valid and multiple invalid values', () => {
+    const input = {
+      validString: 'hello',
+      validArray: ['item1', 'item2'],
+      emptyString: '',
+      undefinedValue: undefined,
+      emptyArray: [],
+      anotherEmptyString: '',
+    };
+
+    const result = filterEmptyParams(input);
+    const resultEntries = Object.entries(result);
+
+    expect(resultEntries).toHaveLength(2);
+    expect(result).toEqual({
+      validString: 'hello',
+      validArray: ['item1', 'item2'],
+    });
+  });
+
+  it('should filter out empty strings', () => {
+    const input = {
+      valid: 'test',
+      empty: '',
+    };
+
+    const result = filterEmptyParams(input);
+    expect(result).toEqual({valid: 'test'});
+  });
+
+  it('should filter out undefined values', () => {
+    const input = {
+      valid: 'test',
+      notDefined: undefined,
+    };
+
+    const result = filterEmptyParams(input);
+    expect(result).toEqual({valid: 'test'});
+  });
+
+  it('should filter out empty arrays', () => {
+    const input = {
+      valid: 'test',
+      emptyArray: [],
+      nonEmptyArray: ['item'],
+    };
+
+    const result = filterEmptyParams(input);
+    expect(result).toEqual({
+      valid: 'test',
+      nonEmptyArray: ['item'],
+    });
+  });
+
+  it('should keep all valid values including numbers, booleans, and objects', () => {
+    const input = {
+      string: 'test',
+      number: 0,
+      boolean: false,
+      object: {key: 'value'},
+      array: ['item'],
+    };
+
+    const result = filterEmptyParams(input);
+    expect(result).toEqual(input);
+  });
+
+  it('should return empty object when all values are invalid', () => {
+    const input = {
+      empty1: '',
+      empty2: '',
+      undefined1: undefined,
+      emptyArray: [],
+    };
+
+    const result = filterEmptyParams(input);
+    expect(result).toEqual({});
+  });
+});

package/src/ProfileExplorer/index.tsx

@@ -71,7 +71,7 @@ const sanitizeDateRange = (
 };
 /* eslint-enable @typescript-eslint/naming-convention */
 
-const filterEmptyParams = (o: Record<string, any>): Record<string, any> => {
+export const filterEmptyParams = (o: Record<string, any>): Record<string, any> => {
   return Object.fromEntries(
     Object.entries(o)
       .filter(

package/src/ProfileView/hooks/useResetStateOnNewSearch.ts

@@ -14,10 +14,13 @@
 import {useURLState} from '@parca/components';
 
 export const useResetStateOnNewSearch = (): (() => void) => {
-  const [, setCurPath] = useURLState('cur_path');
+  const [val, setCurPath] = useURLState('cur_path');
 
   return () => {
     setTimeout(() => {
+      if (val === undefined) {
+        return;
+      }
       setCurPath(undefined);
     });
   };

package/src/ProfileView/hooks/useResetStateOnProfileTypeChange.ts

@@ -14,19 +14,29 @@
 import {useURLState} from '@parca/components';
 
 export const useResetStateOnProfileTypeChange = (): (() => void) => {
-  const [, setGroupBy] = useURLState('group_by');
-  const [, setFilterByFunction] = useURLState('filter_by_function');
-  const [, setExcludeFunction] = useURLState('exclude_function');
-  const [, setSearchString] = useURLState('search_string');
-  const [, setCurPath] = useURLState('cur_path');
+  const [groupBy, setGroupBy] = useURLState('group_by');
+  const [filterByFunction, setFilterByFunction] = useURLState('filter_by_function');
+  const [excludeFunction, setExcludeFunction] = useURLState('exclude_function');
+  const [searchString, setSearchString] = useURLState('search_string');
+  const [curPath, setCurPath] = useURLState('cur_path');
 
   return () => {
     setTimeout(() => {
-      setGroupBy(undefined);
-      setFilterByFunction(undefined);
-      setExcludeFunction(undefined);
-      setSearchString(undefined);
-      setCurPath(undefined);
+      if (groupBy !== undefined) {
+        setGroupBy(undefined);
+      }
+      if (filterByFunction !== undefined) {
+        setFilterByFunction(undefined);
+      }
+      if (excludeFunction !== undefined) {
+        setExcludeFunction(undefined);
+      }
+      if (searchString !== undefined) {
+        setSearchString(undefined);
+      }
+      if (curPath !== undefined) {
+        setCurPath(undefined);
+      }
     });
   };
 };