Address GitHub Copilot review feedback

- Remove Q.Promise type annotation and cast in extensionCache.ts
- Migrate UNSAFE_componentWillReceiveProps to componentDidUpdate
- Update VS Code tasks.json to use bun commands instead of grunt
- Remove legacy tasks.json.old backup file
- Add explicit userMode type handling in widget configuration
- Use flushSync for accurate render timing telemetry
- Remove unused state property from CompletionDropdown

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: BenGWeeks

.vscode/tasks.json

@@ -1,23 +1,42 @@
-// A task runner configuration.
+// VS Code task runner configuration
 {
     "version": "2.0.0",
-    "command": "grunt",
     "tasks": [
         {
-            "label": "build",
-            "type": "grunt",
-            "task": "build",
+            "label": "Build (Dev)",
+            "type": "shell",
+            "command": "bun run package-dev",
             "problemMatcher": "$msCompile",
             "group": {
-                "_id": "build",
-                "isDefault": false
+                "kind": "build",
+                "isDefault": true
             }
         },
         {
-            "label": "publish",
-            "type": "grunt",
-            "task": "publish",
-            "problemMatcher": "$msCompile"
+            "label": "Build (Release)",
+            "type": "shell",
+            "command": "bun run package-release",
+            "problemMatcher": "$msCompile",
+            "group": "build"
+        },
+        {
+            "label": "Lint",
+            "type": "shell",
+            "command": "bun run lint",
+            "problemMatcher": "$eslint-stylish"
+        },
+        {
+            "label": "Format",
+            "type": "shell",
+            "command": "bun run format",
+            "problemMatcher": []
+        },
+        {
+            "label": "Dev Server",
+            "type": "shell",
+            "command": "bun run dev",
+            "problemMatcher": [],
+            "isBackground": true
         }
     ]
-}
+}

.vscode/tasks.json.old

@@ -168,8 +168,11 @@ VSS.require(
           notifyChange = widgetConfigurationContext.notify.bind(widgetConfigurationContext);
 
           const settings = parseSettings(widgetSettings.customSettings.data);
-          const baseFilter = settings.filter || (await defaultFilter.getValue());
-          currentFilter = { ...baseFilter, userMode: baseFilter.userMode || "specific" };
+          const baseFilter: IContributionFilter & Partial<{ userMode: UserMode }> =
+            settings.filter || (await defaultFilter.getValue());
+          const userMode: UserMode =
+            ("userMode" in baseFilter && baseFilter.userMode) || "specific";
+          currentFilter = { ...baseFilter, userMode };
 
           if (!root && container) {
             root = createRoot(container);

scripts/contributionsWidgetConfiguration.tsx

@@ -17,20 +17,15 @@ interface ICompletionDropdownProps {
   label?: string;
 }
 
-interface ICompletionDropdownState {
-  selected?: ITag[];
-}
-
 export class CompletionDropdown extends React.Component<
   ICompletionDropdownProps,
-  ICompletionDropdownState
+  Record<string, never>
 > {
   private static counter = 0;
   private readonly key: number;
   constructor(props: ICompletionDropdownProps) {
     super(props);
     this.key = CompletionDropdown.counter++;
-    this.state = { selected: props.selected };
   }
   render() {
     return (

scripts/controls/CompletionDropdown.tsx

@@ -1,5 +1,6 @@
 import * as React from "react";
 import { createRoot, Root } from "react-dom/client";
+import { flushSync } from "react-dom";
 import { DelayedFunction } from "VSS/Utils/Core";
 
 import { IUserContributions } from "../data/contracts";
@@ -30,9 +31,11 @@ export function renderGraphs(filter: IContributionFilter) {
       const loadingContributions = filter.identities.map(
         (user): IUserContributions => ({ key: -1, data: {}, user })
       );
-      root.render(
-        <Graphs contributions={loadingContributions} loading={true} sharedScale={false} />
-      );
+      flushSync(() => {
+        root.render(
+          <Graphs contributions={loadingContributions} loading={true} sharedScale={false} />
+        );
+      });
       timings.measure("drawSpinner");
     }
   });
@@ -42,9 +45,15 @@ export function renderGraphs(filter: IContributionFilter) {
       showSpinner.cancel();
       if (currentRender === renderNum) {
         timings.measure("getContributions");
-        root.render(
-          <Graphs contributions={contributions} loading={false} sharedScale={filter.sharedScale} />
-        );
+        flushSync(() => {
+          root.render(
+            <Graphs
+              contributions={contributions}
+              loading={false}
+              sharedScale={filter.sharedScale}
+            />
+          );
+        });
         timings.measure("drawGraph");
         trackEvent("loadGraph", filterToIProperties(filter), timings.measurements);
       }

scripts/controls/showGraphs.tsx

@@ -33,8 +33,13 @@ export class TimeWindow extends React.Component<
     super(props);
     this.state = { contributions: this.getContributions(props) };
   }
-  UNSAFE_componentWillReceiveProps(props: ITimeWindowProps) {
-    this.setState({ contributions: this.getContributions(props) });
+  componentDidUpdate(prevProps: ITimeWindowProps) {
+    if (
+      prevProps.selected !== this.props.selected ||
+      prevProps.allContributions.key !== this.props.allContributions.key
+    ) {
+      this.setState({ contributions: this.getContributions(this.props) });
+    }
   }
   render() {
     const { selected } = this.props;

scripts/controls/timeWindow/timeWindow.tsx

@@ -1,5 +1,3 @@
-import * as Q from "q";
-
 const collection = "extension-cache";
 
 interface IExtensionCacheEntry<T> {
@@ -46,15 +44,15 @@ export async function get<T>(key: string, hardGet: () => Promise<IHardGetValue<T
       }
       return doc.value;
     },
-    (error: TfsError): Q.Promise<T> => {
+    (error: TfsError) => {
       const status = Number(error.status);
       // If collection has not been created yet;
       if (
         status === 404 ||
         // User does not have permissions
         status === 401
       ) {
-        return hardGetAndStore() as unknown as Q.Promise<T>;
+        return hardGetAndStore();
       }
       throw error;
     }