@parca/profile 0.16.304 → 0.16.306

package/CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [0.16.306](https://github.com/parca-dev/parca/compare/@parca/profile@0.16.305...@parca/profile@0.16.306) (2023-11-16)
+
+**Note:** Version bump only for package @parca/profile
+
+## 0.16.305 (2023-11-15)
+
+**Note:** Version bump only for package @parca/profile
+
 ## [0.16.304](https://github.com/parca-dev/parca/compare/@parca/profile@0.16.303...@parca/profile@0.16.304) (2023-11-09)
 
 **Note:** Version bump only for package @parca/profile

package/dist/GraphTooltipArrow/useGraphTooltip/index.js

@@ -14,7 +14,7 @@ import { divide, valueFormatter } from '@parca/utilities';
 import { FIELD_CUMULATIVE, FIELD_DIFF, FIELD_LOCATION_ADDRESS, } from '../../ProfileIcicleGraph/IcicleGraphArrow';
 import { getTextForCumulative, nodeLabel } from '../../ProfileIcicleGraph/IcicleGraphArrow/utils';
 export const useGraphTooltip = ({ table, unit, total, totalUnfiltered, row, level, }) => {
-    if (row === null) {
+    if (row === null || row === 0) {
         return null;
     }
     const locationAddress = table.getChild(FIELD_LOCATION_ADDRESS)?.get(row) ?? 0n;

package/dist/ProfileIcicleGraph/IcicleGraphArrow/IcicleGraphNodes.js

@@ -57,8 +57,8 @@ export const IcicleNode = React.memo(function IcicleNodeNoMemo({ table, row, map
     // get the actual values from the columns
     const mappingFile = arrowToString(mappingColumn?.get(row));
     const functionName = arrowToString(functionNameColumn?.get(row));
-    const cumulative = BigInt(cumulativeColumn?.get(row));
-    const diff = BigInt(diffColumn?.get(row));
+    const cumulative = cumulativeColumn?.get(row) !== null ? BigInt(cumulativeColumn?.get(row)) : 0n;
+    const diff = diffColumn?.get(row) !== null ? BigInt(diffColumn?.get(row)) : null;
     const childRows = Array.from(table.getChild(FIELD_CHILDREN)?.get(row) ?? []);
     // TODO: Maybe it's better to pass down the sorter function as prop instead of figuring this out here.
     switch (sortBy) {

package/dist/ProfileSelector/index.js

@@ -139,7 +139,8 @@ const ProfileSelector = ({ queryClient, querySelection, selectProfile, selectQue
         queryExpressionString === '' ||
         queryExpressionString === '{}';
     const compareDisabled = selectedProfileName === '' || querySelection.expression === undefined;
-    return (_jsxs(_Fragment, { children: [_jsxs("div", { className: "mb-2 flex gap-2", children: [_jsxs("div", { className: "flex w-full flex-wrap content-start items-end justify-between gap-2", children: [_jsxs("div", { children: [_jsx("label", { className: "text-xs", children: "Profile type" }), _jsx(ProfileTypeSelector, { profileTypesData: profileTypesData, loading: profileTypesLoading, selectedKey: selectedProfileName, onSelection: setProfileName, error: error })] }), _jsxs("div", { className: "w-full flex-1", children: [_jsxs("div", { className: "mb-[2px] flex items-center justify-between", children: [_jsx("label", { className: "text-xs", children: "Query" }), query.matchers.length > 0 && viewComponent !== undefined && (_jsx("div", { children: viewComponent(query.matchersString()) }))] }), _jsx(MatchersInput, { queryClient: queryClient, setMatchersString: setMatchersString, runQuery: setQueryExpression, currentQuery: query })] }), _jsxs("div", { children: [_jsx("label", { className: "text-xs", children: "Period" }), _jsx(DateTimeRangePicker, { onRangeSelection: setTimeRangeSelection, range: timeRangeSelection })] }), _jsxs(ButtonGroup, { children: [!searchDisabled && (_jsx(_Fragment, { children: !comparing && (_jsx(CompareButton, { disabled: compareDisabled, onClick: handleCompareClick })) })), _jsx(Button, { disabled: searchDisabled, onClick: (e) => {
+    return (_jsxs(_Fragment, { children: [_jsxs("div", { className: "mb-2 flex gap-2", children: [_jsxs("div", { className: "flex w-full flex-wrap content-start items-end justify-between gap-2", children: [_jsxs("div", { children: [_jsx("label", { className: "text-xs", children: "Profile type" }), _jsx(ProfileTypeSelector, { profileTypesData: profileTypesData, loading: profileTypesLoading, selectedKey: selectedProfileName, onSelection: setProfileName, error: error })] }), _jsxs("div", { className: "w-full flex-1", children: [_jsxs("div", { className: "mb-[2px] flex items-center justify-between", children: [_jsx("label", { className: "text-xs", children: "Query" }), (query.matchers.length > 0 || query.inputMatcherString.length > 0) &&
+                                                viewComponent !== undefined && _jsx("div", { children: viewComponent(query.toString()) })] }), _jsx(MatchersInput, { queryClient: queryClient, setMatchersString: setMatchersString, runQuery: setQueryExpression, currentQuery: query })] }), _jsxs("div", { children: [_jsx("label", { className: "text-xs", children: "Period" }), _jsx(DateTimeRangePicker, { onRangeSelection: setTimeRangeSelection, range: timeRangeSelection })] }), _jsxs(ButtonGroup, { children: [!searchDisabled && (_jsx(_Fragment, { children: !comparing && (_jsx(CompareButton, { disabled: compareDisabled, onClick: handleCompareClick })) })), _jsx(Button, { disabled: searchDisabled, onClick: (e) => {
                                             e.preventDefault();
                                             setQueryExpression();
                                         }, id: "matcher-search-button", children: "Search" })] })] }), _jsx("div", { children: comparing && _jsx(IconButton, { onClick: () => closeProfile(), icon: _jsx(CloseIcon, {}) }) })] }), _jsx("div", { className: "rounded bg-white shadow dark:border-gray-500 dark:bg-gray-700", children: _jsx("div", { style: { height: heightStyle }, children: querySelection.expression !== undefined &&

package/dist/ProfileTypeSelector/index.d.ts

@@ -10,6 +10,7 @@ interface WellKnownProfiles {
     [key: string]: WellKnownProfile;
 }
 export declare const wellKnownProfiles: WellKnownProfiles;
+export declare function flexibleWellKnownProfileMatching(name: string): WellKnownProfile | undefined;
 export declare function profileSelectElement(name: string, flexibleKnownProfilesDetection: boolean): SelectElement;
 export declare const constructProfileName: (type: ProfileType) => string;
 export declare const normalizeProfileTypesData: (types: ProfileType[]) => string[];

package/dist/ProfileTypeSelector/index.js

@@ -79,7 +79,7 @@ export const wellKnownProfiles = {
         help: 'CPU profile samples observed by Parca Agent.',
     },
 };
-function flexibleWellKnownProfileMatching(name) {
+export function flexibleWellKnownProfileMatching(name) {
     const prefixExcludedName = name.split(':').slice(1).join(':');
     const deltaExcludedName = prefixExcludedName.replace(/:delta$/, '');
     const requiredKey = Object.keys(wellKnownProfiles).find(key => {

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@parca/profile",
-  "version": "0.16.304",
+  "version": "0.16.306",
   "description": "Profile viewing libraries",
   "dependencies": {
     "@parca/client": "^0.16.98",
-    "@parca/components": "^0.16.226",
+    "@parca/components": "^0.16.227",
     "@parca/dynamicsize": "^0.16.60",
     "@parca/hooks": "^0.0.35",
     "@parca/parser": "^0.16.66",
@@ -50,5 +50,5 @@
     "access": "public",
     "registry": "https://registry.npmjs.org/"
   },
-  "gitHead": "a342b32b3ed89bee6659d1f4fd073c6bcd52cb16"
+  "gitHead": "5b4010db90992ace68168bc7786e45bd242d5a56"
 }

package/src/GraphTooltipArrow/useGraphTooltip/index.ts

@@ -48,7 +48,7 @@ export const useGraphTooltip = ({
   row,
   level,
 }: Props): GraphTooltipData | null => {
-  if (row === null) {
+  if (row === null || row === 0) {
     return null;
   }
 

package/src/ProfileIcicleGraph/IcicleGraphArrow/IcicleGraphNodes.tsx

@@ -194,8 +194,8 @@ export const IcicleNode = React.memo(function IcicleNodeNoMemo({
   // get the actual values from the columns
   const mappingFile: string | null = arrowToString(mappingColumn?.get(row));
   const functionName: string | null = arrowToString(functionNameColumn?.get(row));
-  const cumulative = BigInt(cumulativeColumn?.get(row));
-  const diff: bigint | null = BigInt(diffColumn?.get(row));
+  const cumulative = cumulativeColumn?.get(row) !== null ? BigInt(cumulativeColumn?.get(row)) : 0n;
+  const diff: bigint | null = diffColumn?.get(row) !== null ? BigInt(diffColumn?.get(row)) : null;
   const childRows: number[] = Array.from(table.getChild(FIELD_CHILDREN)?.get(row) ?? []);
 
   // TODO: Maybe it's better to pass down the sorter function as prop instead of figuring this out here.

package/src/ProfileSelector/index.tsx

@@ -241,9 +241,8 @@ const ProfileSelector = ({
             <div className="mb-[2px] flex items-center justify-between">
               <label className="text-xs">Query</label>
 
-              {query.matchers.length > 0 && viewComponent !== undefined && (
-                <div>{viewComponent(query.matchersString())}</div>
-              )}
+              {(query.matchers.length > 0 || query.inputMatcherString.length > 0) &&
+                viewComponent !== undefined && <div>{viewComponent(query.toString())}</div>}
             </div>
 
             <MatchersInput

package/src/ProfileTypeSelector/index.tsx

@@ -94,7 +94,7 @@ export const wellKnownProfiles: WellKnownProfiles = {
   },
 };
 
-function flexibleWellKnownProfileMatching(name: string): WellKnownProfile | undefined {
+export function flexibleWellKnownProfileMatching(name: string): WellKnownProfile | undefined {
   const prefixExcludedName = name.split(':').slice(1).join(':');
   const deltaExcludedName = prefixExcludedName.replace(/:delta$/, '');
   const requiredKey = Object.keys(wellKnownProfiles).find(key => {