Merge pull request #23 from LangStream/21-handle-the-decision-of-name…

…-id-unknown-better

Added decision of name, id, or unknown

CHANGELOG.md

@@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.2.1]
+
+### Changed
+
+For pipeline and agent, created a decision hierarchy to display either its name, id, or "unknown" 
+
+Contributors: ddieruf
+
 ## [0.2.0]
 
 ### Added

package.json

@@ -2,7 +2,7 @@
   "name": "langstream",
   "displayName": "LangStream AI",
   "description": "Develop streaming generative AI applications with LangStream",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "publisher": "DataStax",
   "engines": {
     "vscode": "^1.77.0"

src/providers/controlPlaneTreeData/nodes/agent.ts

@@ -13,9 +13,10 @@ export interface IAgentNode extends vscode.TreeItem {
 
 export class AgentNode extends vscode.TreeItem implements IAgentNode {
   constructor(readonly agent: lsModels.AgentConfiguration) {
-    super(agent.name ?? "unknown", vscode.TreeItemCollapsibleState.None);
+    super("unknown", vscode.TreeItemCollapsibleState.None);
     this.description = `Agent`;
     this.contextValue = Constants.CONTEXT_VALUES.agent;
+    this.label = this.decideLabel(agent);
 
     switch(agent.errors?.retries ?? 0) {
       case 0:
@@ -32,6 +33,34 @@ export class AgentNode extends vscode.TreeItem implements IAgentNode {
         break;
     }
   }
+
+  private decideLabel(agent: lsModels.AgentConfiguration): string {
+    const name: string | undefined | null = agent.name?.trim()
+      .replace(/'null'/g, "")
+      .replace(/"null"/g, "")
+      .replace(/^\bnull\b$/i, "")
+      .replace(/"/g, "")
+      .replace(/'/g, "")
+      .replace(/`/g, "");
+
+    const id: string | undefined | null = agent.id?.trim()
+      .replace(/'null'/g, "")
+      .replace(/"null"/g, "")
+      .replace(/^\bnull\b$/i, "")
+      .replace(/"/g, "")
+      .replace(/'/g, "")
+      .replace(/`/g, "");
+
+    if(name !== undefined && name !== null && name.length > 0){
+      return name;
+    }
+
+    if(id !== undefined && id !== null && id.length > 0){
+      return id;
+    }
+
+    return "unknown";
+  }
 }
 
 export default class AgentTree {

src/providers/controlPlaneTreeData/nodes/pipeline.ts

@@ -12,9 +12,38 @@ export interface IPipelineNode extends vscode.TreeItem {
 
 export class PipelineNode extends vscode.TreeItem implements IPipelineNode {
   constructor(readonly pipeline: lsModels.Pipeline) {
-    super(pipeline.name ?? "unknown", vscode.TreeItemCollapsibleState.Collapsed);
+    super("unknown", vscode.TreeItemCollapsibleState.Collapsed);
     this.description = `Pipeline`;
     this.contextValue = `${Constants.CONTEXT_VALUES.pipeline}.${(pipeline.errors?.retries ?? 0) > 0 ? "errors" : "healthy"}`;
+    this.label = this.decideLabel(pipeline);
+  }
+
+  private decideLabel(pipeline: lsModels.Pipeline): string {
+    const name: string | undefined | null = pipeline.name?.trim()
+                  .replace(/'null'/g, "")
+                  .replace(/"null"/g, "")
+                  .replace(/^\bnull\b$/i, "")
+                  .replace(/"/g, "")
+                  .replace(/'/g, "")
+                  .replace(/`/g, "");
+
+    const id: string | undefined | null = pipeline.id?.trim()
+      .replace(/'null'/g, "")
+      .replace(/"null"/g, "")
+      .replace(/^\bnull\b$/i, "")
+      .replace(/"/g, "")
+      .replace(/'/g, "")
+      .replace(/`/g, "");
+
+    if(name !== undefined && name !== null && name.length > 0){
+      return name;
+    }
+
+    if(id !== undefined && id !== null && id.length > 0){
+      return id;
+    }
+
+    return "unknown";
   }
 }