State message tooltips (#1100)

State messages can contain long errors and are frequently truncated. By default VSCode shows labels as tooltips, which is not useful. Also removed the empty node between workflows and pipelines.
databricks · Feb 28, 2024 · f0986e1 · f0986e1
1 parent ab214b4
commit f0986e1
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/packages/databricks-vscode/src/ui/bundle-resource-explorer/JobRunStatusTreeNode.ts b/packages/databricks-vscode/src/ui/bundle-resource-explorer/JobRunStatusTreeNode.ts
@@ -36,6 +36,7 @@ export class JobRunStatusTreeNode implements BundleResourceExplorerTreeNode {
                     {
                         label: "State Message",
                         description: this.runDetails.state?.state_message,
+                        tooltip: this.runDetails.state?.state_message,
                         contextValue: "state_message",
                     },
                     this

diff --git a/packages/databricks-vscode/src/ui/bundle-resource-explorer/ResourceTypeHeaderTreeNode.ts b/packages/databricks-vscode/src/ui/bundle-resource-explorer/ResourceTypeHeaderTreeNode.ts
@@ -9,7 +9,6 @@ import {
 } from "./types";
 import {ExtensionContext, TreeItemCollapsibleState} from "vscode";
 import {PipelineTreeNode} from "./PipelineTreeNode";
-import {TreeItemTreeNode} from "./TreeItemTreeNode";
 
 function humaniseResourceType(type: BundleResourceExplorerTreeNode["type"]) {
     switch (type) {
@@ -93,10 +92,6 @@ export class ResourceTypeHeaderTreeNode
         );
         if (pipelines.length > 0) {
             roots.push(
-                new TreeItemTreeNode(
-                    {collapsibleState: TreeItemCollapsibleState.None},
-                    undefined
-                ),
                 new ResourceTypeHeaderTreeNode(context, "pipelines", pipelines)
             );
         }

diff --git a/packages/databricks-vscode/src/ui/bundle-resource-explorer/TaskRunStatusTreeNode.ts b/packages/databricks-vscode/src/ui/bundle-resource-explorer/TaskRunStatusTreeNode.ts
@@ -45,6 +45,7 @@ export class TaskRunStatusTreeNode implements BundleResourceExplorerTreeNode {
                     {
                         label: "State Message",
                         description: this.runDetails.state?.state_message,
+                        tooltip: this.runDetails.state?.state_message,
                         contextValue: "state_message",
                     },
                     this