release(ui): cut @srl-labs/clab-ui v0.0.19

Author: FloSch62

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@srl-labs/clab-ui",
-  "version": "0.0.18",
+  "version": "0.0.19",
   "private": false,
   "description": "Shared UI package for containerlab webviews and topology editing.",
   "license": "Apache-2.0",

package.json

@@ -45,42 +45,55 @@ function toEndpointTab(id: string): EndpointTab {
   return id === "b" ? "b" : "a";
 }
 
+function buildFallbackEndpoint(options: {
+  node: string;
+  endpoint?: string;
+  mac?: string;
+  mtu?: string | number;
+  type?: string;
+  stats?: InterfaceStatsPayload;
+}): EndpointData {
+  return {
+    node: options.node,
+    interface: options.endpoint ?? "",
+    mac: options.mac ?? "",
+    mtu: options.mtu ?? "",
+    type: options.type ?? "",
+    stats: options.stats
+  };
+}
+
+function mergeEndpointData(fallback: EndpointData, override?: EndpointData): EndpointData {
+  return {
+    ...fallback,
+    ...(override ?? {}),
+    stats: override?.stats ?? fallback.stats
+  };
+}
+
 function getEndpoints(linkData: LinkInfoData): { a: EndpointData; b: EndpointData } {
   const extraData = linkData.extraData ?? {};
-  const sourceMac = getString(extraData.clabSourceMacAddress) ?? "";
-  const sourceType = getString(extraData.clabSourceType) ?? "";
-  const targetMac = getString(extraData.clabTargetMacAddress) ?? "";
-  const targetType = getString(extraData.clabTargetType) ?? "";
-
-  const fallbackA: EndpointData = {
+  const fallbackA = buildFallbackEndpoint({
     node: linkData.source,
-    interface: linkData.sourceEndpoint ?? "",
-    mac: sourceMac,
-    mtu: extraData.clabSourceMtu ?? "",
-    type: sourceType,
+    endpoint: linkData.sourceEndpoint,
+    mac: getString(extraData.clabSourceMacAddress) ?? "",
+    mtu: extraData.clabSourceMtu,
+    type: getString(extraData.clabSourceType) ?? "",
     stats: extraData.clabSourceStats
-  };
-  const a: EndpointData = {
-    ...fallbackA,
-    ...(linkData.endpointA ?? {}),
-    stats: linkData.endpointA?.stats ?? fallbackA.stats
-  };
-
-  const fallbackB: EndpointData = {
+  });
+  const fallbackB = buildFallbackEndpoint({
     node: linkData.target,
-    interface: linkData.targetEndpoint ?? "",
-    mac: targetMac,
-    mtu: extraData.clabTargetMtu ?? "",
-    type: targetType,
+    endpoint: linkData.targetEndpoint,
+    mac: getString(extraData.clabTargetMacAddress) ?? "",
+    mtu: extraData.clabTargetMtu,
+    type: getString(extraData.clabTargetType) ?? "",
     stats: extraData.clabTargetStats
-  };
-  const b: EndpointData = {
-    ...fallbackB,
-    ...(linkData.endpointB ?? {}),
-    stats: linkData.endpointB?.stats ?? fallbackB.stats
-  };
+  });
 
-  return { a, b };
+  return {
+    a: mergeEndpointData(fallbackA, linkData.endpointA),
+    b: mergeEndpointData(fallbackB, linkData.endpointB)
+  };
 }
 
 export const LinkInfoView: React.FC<LinkInfoViewProps> = ({ linkData }) => {

src/components/panels/context-panel/views/LinkInfoView.tsx

@@ -60,16 +60,18 @@ function handlePanelAction(
   switch (action) {
     case "edit-node":
       if (nodeId !== undefined) editNode(nodeId);
-      return;
+      break;
     case "edit-link":
       if (edgeId !== undefined) editEdge(edgeId);
-      return;
+      break;
     case "node-info":
       if (nodeId !== undefined) selectNode(nodeId);
-      return;
+      break;
     case "link-info":
       if (edgeId !== undefined) selectEdge(edgeId);
-      return;
+      break;
+    default:
+      break;
   }
 }
 
@@ -136,30 +138,30 @@ export function useTopoViewerMessageSubscription(): void {
       switch (event.type) {
         case "modeChanged":
           handleTopoModeChanged(event);
-          return;
+          break;
         case "panelAction":
           handlePanelAction(event);
-          return;
+          break;
         case "customNodesUpdated":
           handleCustomNodesUpdated(event);
-          return;
+          break;
         case "customNodeError":
           handleCustomNodeError(event);
-          return;
+          break;
         case "iconList":
           handleIconList(event);
-          return;
+          break;
         case "lifecycleLog":
           handleLifecycleLog(event);
-          return;
+          break;
         case "lifecycleStatus":
           handleLifecycleStatus(event);
-          return;
+          break;
         case "fitViewport":
           handleFitViewport();
-          return;
+          break;
         case "svgExportResult":
-          return;
+          break;
       }
     });
   }, [host]);

src/hooks/app/useTopoViewerMessageSubscription.ts

@@ -9,8 +9,7 @@ import {
   useGraphStore,
   useIsLocked
 } from "../../stores";
-import { collectNodeGroupMemberships } from "../../annotations/groupMembership";
-import { TRAFFIC_RATE_NODE_TYPE } from "../../annotations/annotationNodeConverters";
+import { collectNodeGroupMemberships, TRAFFIC_RATE_NODE_TYPE } from "../../annotations";
 import type { GroupStyleAnnotation } from "../../core/types/topology";
 
 import type { AnnotationContextValue } from "./annotationTypes";

src/hooks/canvas/useAnnotations.ts

@@ -281,16 +281,16 @@ export function createExplorerController(options: ExplorerControllerOptions): Ex
       switch (message.command) {
         case "ready":
           this.connect();
-          return;
+          break;
         case "setFilter":
           await this.setFilter(message.value);
-          return;
+          break;
         case "invokeAction":
           await this.invokeAction(message.actionRef);
-          return;
+          break;
         case "persistUiState":
           await this.persistUiState(message.state);
-          return;
+          break;
       }
     },