feat: disable status checking by default

- adds a new tree.status setting to reenable
- change the default task icon color to task's primary color

Author: pd93

README.md

@@ -45,4 +45,5 @@
 | `checkForUpdates`    | `boolean` |                                   | `true`      | Check if there is a newer version of Task on startup.                                                                                         |
 | `doubleClickTimeout` | `number`  |                                   | `0`         | Time in milliseconds to consider a double-click. 0 disables double-click to run. 500 is a good starting point if you want to enable it.       |
 | `tree.nesting`       | `boolean` |                                   | `true`      | Whether to nest tasks by their namespace in the tree view.                                                                                    |
+| `tree.status`        | `boolean` |                                   | `false`     | Whether to show the status of tasks in the tree view (may be slow on large Taskfiles).                                                        |
 | `tree.sort`          | `sort`    | `default`, `alphanumeric`, `none` | `"default"` | The order in which to display tasks in the tree view.                                                                                         |

package.json

@@ -263,6 +263,16 @@
           "highContrast": "#ff9b05",
           "highContrastLight": "#ff9b05"
         }
+      },
+      {
+        "id": "vscodetask.primaryColor",
+        "description": "Color for primary elements.",
+        "defaults": {
+          "dark": "#43aba2",
+          "light": "#43aba2",
+          "highContrast": "#43aba2",
+          "highContrastLight": "#43aba2"
+        }
       }
     ],
     "configuration": {
@@ -314,6 +324,11 @@
                   "default": true,
                   "description": "Whether to nest tasks by their namespace in the tree view."
                 },
+                "status": {
+                  "type": "boolean",
+                  "default": false,
+                  "description": "Whether to show the status of tasks in the tree view (may be slow on large Taskfiles)."
+                },
                 "sort": {
                   "type": "string",
                   "enum": [

src/elements/treeItem.ts

@@ -42,10 +42,15 @@ export class TaskTreeItem extends vscode.TreeItem {
     ) {
         super(label, collapsibleState);
         this.description = this.task?.desc;
-        if (this.task.up_to_date) {
-            this.iconPath = new vscode.ThemeIcon('debug-breakpoint-log-unverified', new vscode.ThemeColor('vscodetask.upToDateIcon'));
-        } else {
-            this.iconPath = new vscode.ThemeIcon('debug-breakpoint-data-unverified', new vscode.ThemeColor('vscodetask.outOfDateIcon'));
+        switch (this.task.up_to_date) {
+            case true:
+                this.iconPath = new vscode.ThemeIcon('debug-breakpoint-data-unverified', new vscode.ThemeColor('vscodetask.upToDateIcon'));
+                break;
+            case false:
+                this.iconPath = new vscode.ThemeIcon('debug-breakpoint-data-unverified', new vscode.ThemeColor('vscodetask.outOfDateIcon'));
+                break;
+            default:
+                this.iconPath = new vscode.ThemeIcon('debug-breakpoint-data-unverified', new vscode.ThemeColor('vscodetask.primaryColor'));
         }
         this.contextValue = `taskTreeItem`;
     }

src/models/models.ts

@@ -12,7 +12,7 @@ export interface Task {
     desc: string;
     summary: string;
     // eslint-disable-next-line @typescript-eslint/naming-convention
-    up_to_date: boolean;
+    up_to_date: boolean | undefined;
     location: Location;
 }
 

src/services/taskfile.ts

@@ -177,7 +177,7 @@ class TaskfileService {
         }
     }
 
-    public async read(dir: string, nesting: boolean): Promise<Namespace | undefined> {
+    public async read(dir: string, nesting: boolean, status: boolean): Promise<Namespace | undefined> {
         log.info(`Searching for taskfile in: "${dir}"`);
         return await new Promise((resolve, reject) => {
             let flags = [
@@ -191,6 +191,9 @@ class TaskfileService {
             if (nesting) {
                 flags.push(`--nested`);
             }
+            if (!status) {
+                flags.push(`--no-status`);
+            }
             let command = this.command(`${flags.join(' ')}`);
             cp.exec(command, { cwd: dir }, (err: cp.ExecException | null, stdout: string, stderr: string) => {
                 if (err) {

src/task.ts

@@ -13,11 +13,13 @@ export class TaskExtension {
     private _watcher: vscode.FileSystemWatcher;
     private _changeTimeout: NodeJS.Timeout | null = null;
     private _nesting: boolean;
+    private _status: boolean;
 
     constructor() {
         this._activityBar = new ActivityBar();
         this._watcher = vscode.workspace.createFileSystemWatcher("**/*.{yml,yaml}");
         this._nesting = settings.tree.nesting;
+        this._status = settings.tree.status;
     }
 
     public async update(checkForUpdates?: boolean): Promise<void> {
@@ -36,7 +38,7 @@ export class TaskExtension {
                 // Read taskfiles
                 let p: Promise<Namespace | undefined>[] = [];
                 vscode.workspace.workspaceFolders?.forEach((folder) => {
-                    p.push(taskfileSvc.read(folder.uri.fsPath, this._nesting));
+                    p.push(taskfileSvc.read(folder.uri.fsPath, this._nesting, this._status));
                 });
 
                 return Promise.allSettled(p);
@@ -288,6 +290,10 @@ export class TaskExtension {
         log.info("Detected changes to configuration");
         if (event.affectsConfiguration("task")) {
             settings.update();
+            this._nesting = settings.tree.nesting;
+            this._status = settings.tree.status;
+            this.refresh(false);
+            vscode.commands.executeCommand('setContext', 'vscode-task:treeNesting', this._nesting);
         }
     }
 }

src/utils/settings.ts

@@ -50,6 +50,7 @@ export enum UpdateOn {
 class TreeSettings {
     private static _instance: TreeSettings;
     public nesting!: boolean;
+    public status!: boolean;
     public sort!: TreeSort;
 
     constructor() {
@@ -69,6 +70,7 @@ class TreeSettings {
 
         // Set the properties
         this.nesting = config.get("tree.nesting") ?? true;
+        this.status = config.get("tree.status") ?? false;
         this.sort = config.get("tree.sort") ?? TreeSort.default;
     }
 }