chore: update dependencies and improve code structure

- Updated various dependencies in package.json for better performance and security.
- Refactored saveData.ts to simplify variable declarations and improve readability.
- Modified Frame.tsx and WUDetails.tsx to streamline state management and component rendering.
- Cleaned up session.ts by removing unnecessary variable initialization.
- Enhanced extension tests in extension.test.ts for better clarity and error handling.
- Simplified test suite execution in index.ts and removed Mocha dependency from tsconfig.test.json.

Author: GordonSmith

package-lock.json

@@ -56,73 +56,74 @@
     "exceljs": "4.4.0",
     "html-to-text": "9.0.5"
   },
+  "overrides": {
+    "vite": "7.3.2"
+  },
   "devDependencies": {
-    "@eslint/js": "9.39.4",
+    "@eslint/js": "10.0.1",
     "@fluentui/react": "8.125.5",
     "@fluentui/react-hooks": "8.10.2",
-    "@hpcc-js/api": "3.4.15",
-    "@hpcc-js/common": "3.7.5",
-    "@hpcc-js/comms": "3.15.4",
-    "@hpcc-js/dgrid": "3.7.5",
+    "@hpcc-js/api": "3.4.16",
+    "@hpcc-js/common": "3.7.6",
+    "@hpcc-js/comms": "3.15.6",
+    "@hpcc-js/dgrid": "3.7.6",
     "@hpcc-js/dgrid-shim": "3.4.5",
     "@hpcc-js/esbuild-plugins": "1.8.7",
     "@hpcc-js/observable-shim": "3.1.0",
-    "@hpcc-js/observablehq-compiler": "3.7.8",
-    "@hpcc-js/util": "3.5.4",
+    "@hpcc-js/observablehq-compiler": "3.7.9",
+    "@hpcc-js/util": "3.5.5",
     "@hpcc-js/wasm-expat": "1.13.2",
     "@hpcc-js/wasm-llama": "1.0.0",
     "@hpcc-js/wasm-zstd": "1.12.2",
     "@langchain/community": "1.1.27",
-    "@langchain/core": "1.1.39",
-    "@langchain/classic": "1.0.28",
+    "@langchain/core": "1.1.40",
+    "@langchain/classic": "1.0.31",
     "@langchain/ollama": "1.2.6",
     "@observablehq/inspector": "5.0.1",
     "@observablehq/parser": "6.1.0",
     "@observablehq/runtime": "6.0.0",
     "@observablehq/stdlib": "5.8.8",
     "@octokit/rest": "22.0.1",
-    "@types/mocha": "10.0.10",
     "@types/node": "24.12.2",
     "@types/react": "18.3.28",
     "@types/react-dom": "18.3.7",
     "@types/tmp": "0.2.6",
     "@types/vscode": "1.105.0",
     "@types/vscode-notebook-renderer": "1.72.4",
     "@vscode/test-electron": "2.5.2",
-    "@vitest/coverage-v8": "4.1.3",
-    "@vitest/ui": "4.1.3",
+    "@vitest/coverage-v8": "4.1.5",
+    "@vitest/ui": "4.1.5",
     "@vscode/debugadapter": "1.68.0",
     "@vscode/debugprotocol": "1.68.0",
     "@vscode/extension-telemetry": "1.5.1",
-    "@vscode/vsce": "3.7.1",
+    "@vscode/vsce": "3.9.1",
     "adm-zip": "0.5.17",
     "antlr4": "4.13.2",
     "assert": "2.1.0",
     "copy-to-clipboard": "3.3.3",
     "copyfiles": "2.4.1",
     "csv-writer": "1.6.0",
-    "diff": "8.0.4",
+    "diff": "9.0.0",
     "esbuild": "0.28.0",
     "esbuild-copy-static-files": "0.1.0",
     "esbuild-plugin-text-replace": "1.3.0",
     "esbuild-plugin-umd-wrapper": "3.0.0",
-    "eslint": "9.39.4",
-    "eslint-plugin-react-hooks": "7.0.1",
+    "eslint": "10.2.1",
+    "eslint-plugin-react-hooks": "7.1.1",
     "glob": "13.0.6",
     "jsdom": "29.0.2",
-    "langchain": "1.3.0",
-    "mocha": "11.7.5",
+    "langchain": "1.3.3",
     "npm-run-all": "4.1.5",
     "react": "18.3.1",
     "react-dom": "18.3.1",
     "rimraf": "6.1.3",
     "run-script-os": "1.1.6",
     "tslib": "2.8.1",
-    "typescript": "6.0.2",
-    "typescript-eslint": "8.58.0",
+    "typescript": "6.0.3",
+    "typescript-eslint": "8.59.0",
     "use-resize-observer": "9.1.0",
-    "uuid": "13.0.0",
-    "vitest": "4.1.3",
+    "uuid": "14.0.0",
+    "vitest": "4.1.5",
     "yargs": "18.0.0"
   },
   "author": {

package.json

@@ -108,8 +108,6 @@ export class SaveData {
         const numPieces = Math.floor(totalRows / pieceSize);
         const pieces = numPieces == 0 ? 1 : numPieces;
         let i = 0;
-        let number = 0;
-        let displayNum = 0;
 
         if (this._type == SaveFileType.CSV) {
             this._createCsvWriter = createArrayCsvWriter;
@@ -120,8 +118,8 @@ export class SaveData {
         }
 
         for (let s = 0; i < pieces; s += pieceSize, i++) {
-            number = totalRows < pieceSize ? totalRows : pieceSize;
-            displayNum = s + pieceSize > totalRows ? totalRows : s + pieceSize;
+            const number = totalRows < pieceSize ? totalRows : pieceSize;
+            const displayNum = s + pieceSize > totalRows ? totalRows : s + pieceSize;
             eclStatusBar.showClientTools(`${result.Name} (${displayNum} of ${totalRows})...`, `${localize("Saving data to")} ${this._filePath}`);
             await this.fetchRows(result, s, number);
         }
@@ -202,13 +200,10 @@ export class SaveData {
             if (result.Total > maxRowsFound)
                 maxRowsFound = result.Total;
         }
-        let userLimit = -1;
         if (maxRowsFound > this._maxRows) {
-            userLimit = await this.promptUserMaxRows(maxRowsFound);
-        } else {
-            userLimit = this._maxRows;
+            return this.promptUserMaxRows(maxRowsFound);
         }
-        return userLimit;
+        return this._maxRows;
     }
 
     async promptUserMaxRows(rowCount: number): Promise<number> {

src/ecl/saveData.ts

@@ -14,16 +14,11 @@ interface FrameProps {
 export const Frame: React.FunctionComponent<FrameProps> = () => {
 
     const state = useMessageReceiver();
-    const [name, setName] = React.useState<string>(state?.resultName ?? "");
-
-    React.useEffect(() => {
-        setName(state?.resultName ?? "");
-    }, [state?.wuid, state?.resultName]);
 
     return <ThemeProvider style={{ height: "100%" }}>
         {
             state ?
-                <WUDetails opts={state} wuid={state.wuid} name={name} setName={setName}></WUDetails> :
+                <WUDetails key={`${state.wuid}-${state.resultName}`} opts={state} wuid={state.wuid} initialName={state.resultName ?? ""}></WUDetails> :
                 <div>...loading...</div>
         }
     </ThemeProvider >;

src/eclwatch/Frame.tsx

@@ -24,19 +24,18 @@ const pivotStyles: IStyleFunctionOrObject<IPivotStyleProps, IPivotStyles> = {
 export interface WUDetailsProps {
     opts: IOptions;
     wuid: string;
-    name: string;
-    setName: React.Dispatch<React.SetStateAction<string>>;
+    initialName: string;
 }
 
 export const WUDetails: React.FunctionComponent<WUDetailsProps> = ({
     opts,
     wuid,
-    name,
-    setName
+    initialName
 }) => {
 
     const pivotRef = React.useRef<HTMLDivElement>(null);
 
+    const [name, setName] = React.useState(initialName);
     const [selectedValue, setSelectedValue] = React.useState<string | undefined>();
     const [spinnerMessage, setSpinnerMessage] = React.useState("Loading...");
     const [complete, setComplete] = React.useState(false);
@@ -67,8 +66,6 @@ export const WUDetails: React.FunctionComponent<WUDetailsProps> = ({
         }
 
         if (wuid) {
-            setSpinnerMessage("Loading...");
-            update(false, [], []);
             const wu = Workunit.attach(opts, wuid);
             wu.refresh().then(() => {
                 if (wu.isComplete()) {
@@ -82,20 +79,17 @@ export const WUDetails: React.FunctionComponent<WUDetailsProps> = ({
                 setSpinnerMessage(e.message);
                 update(true, [], []);
             });
-        } else {
-            setSpinnerMessage("");
-            update(true, [], []);
         }
 
         return () => {
             canceled = true;
         };
-    }, [name, opts, setName, wuid]);
+    }, [name, opts, wuid]);
 
     const handleLinkClick = React.useCallback((item?: PivotItem) => {
         setName(item?.props.itemKey ?? "");
         setSelectedValue(item?.props["data-result"]?.Value);
-    }, [setName]);
+    }, []);
 
     const hasIssues = exceptions.length > 0;
     const hasResults = results.length > 0;

src/eclwatch/WUDetails.tsx

@@ -559,7 +559,6 @@ export class SessionManager {
         const activeUri: string = vscode.window.activeTextEditor?.document?.uri.toString(true) || "";
         if (activeUri) {
             const eclConfig = vscode.workspace.getConfiguration("ecl", null);
-            isPinned = false;
             const pinnedLaunchConfigurations = eclConfig.get<object>("pinnedLaunchConfigurations");
             isPinned = !!pinnedLaunchConfigurations[activeUri];
         }

src/hpccplatform/session.ts

@@ -1,34 +1,36 @@
 import * as assert from 'assert';
 import * as vscode from 'vscode';
 
-suite('Extension Activation Test', () => {
-    test('extension manifest is discoverable', () => {
+export async function run(): Promise<void> {
+    console.log('Test: extension manifest is discoverable');
+    {
         const ext = vscode.extensions.getExtension('hpcc-systems.ecl');
         assert.ok(ext, 'Extension should be found in the extension manifest');
-    });
+        console.log('  PASS');
+    }
 
-    test('extension can activate when dependencies are present', async function () {
+    console.log('Test: extension can activate when dependencies are present');
+    {
         const ext = vscode.extensions.getExtension('hpcc-systems.ecl');
         assert.ok(ext, 'Extension should be found');
 
         // Check if required dependency is present
         const requiredDep = vscode.extensions.getExtension('GordonSmith.observable-js');
         if (!requiredDep) {
-            console.log('⚠️  Test skipped: GordonSmith.observable-js extension not installed (expected in CI/minimal environments)');
-            this.skip();
-            return;
-        }
+            console.log('  SKIPPED: GordonSmith.observable-js extension not installed (expected in CI/minimal environments)');
+        } else {
+            // Attempt activation
+            try {
+                await ext?.activate();
+            } catch (e: unknown) {
+                const msg = (e as Error)?.message || String(e);
+                throw new Error(`Extension activation failed: ${msg}`);
+            }
 
-        // Attempt activation
-        try {
-            await ext?.activate();
-        } catch (e: unknown) {
-            const msg = (e as Error)?.message || String(e);
-            throw new Error(`Extension activation failed: ${msg}`);
+            const commands = await vscode.commands.getCommands(true);
+            assert.ok(commands.includes('ecl.submit'), 'ecl.submit command should be registered');
+            assert.ok(commands.includes('ecl.compile'), 'ecl.compile command should be registered');
+            console.log('  PASS');
         }
-
-        const commands = await vscode.commands.getCommands(true);
-        assert.ok(commands.includes('ecl.submit'), 'ecl.submit command should be registered');
-        assert.ok(commands.includes('ecl.compile'), 'ecl.compile command should be registered');
-    });
-});
+    }
+}

test/integration/suite/extension.test.ts

@@ -1,31 +1,25 @@
 import * as path from 'path';
-import Mocha from 'mocha';
 import { globSync } from 'glob';
 
-export function run(): Promise<void> {
-    const mocha = new Mocha({
-        ui: 'tdd',
-        color: true,
-        timeout: 20000
-    });
-
+export async function run(): Promise<void> {
     const testsRoot = path.resolve(__dirname);
+    const files = globSync('**/*.test.js', { cwd: testsRoot });
+
+    let failures = 0;
 
-    return new Promise((resolve, reject) => {
-        try {
-            const files = globSync('**/*.test.js', { cwd: testsRoot });
-            for (const f of files) {
-                mocha.addFile(path.resolve(testsRoot, f));
+    for (const f of files) {
+        const mod = require(path.resolve(testsRoot, f));
+        if (typeof mod.run === 'function') {
+            try {
+                await mod.run();
+            } catch (e) {
+                failures++;
+                console.error(`FAIL: ${f}`, e);
             }
-            mocha.run((failures: number) => {
-                if (failures > 0) {
-                    reject(new Error(`${failures} tests failed.`));
-                } else {
-                    resolve();
-                }
-            });
-        } catch (e) {
-            reject(e);
         }
-    });
+    }
+
+    if (failures > 0) {
+        throw new Error(`${failures} test files failed.`);
+    }
 }

test/integration/suite/index.ts

@@ -9,8 +9,7 @@
         "skipLibCheck": true,
         "types": [
             "node",
-            "vscode",
-            "mocha"
+            "vscode"
         ],
         "noEmitOnError": true
     },