chore: Update Dependencies

[MSNev]

• @microsoft/rush
• @nevware21/ts-utils
• @nevware21/ts-async

To fix the unused variable issue, remove the creation of the unused default-import helper node_path__WEBPACK_IMPORTED_MODULE_0___default while preserving the actual import of node:path that is used. The __webpack_require__.n wrapper is only needed if the code will access node_path__WEBPACK_IMPORTED_MODULE_0___default() somewhere; if that never happens, we can delete that line without affecting functionality.

Concretely, in common/scripts/install-run-rush.js, locate the block where node:path and node:fs are imported into the Webpack bundle entry:

/* harmony import */ var node_path__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! node:path */ 176760);
/* harmony import */ var node_path__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(node_path__WEBPACK_IMPORTED_MODULE_0__);
/* harmony import */ var node_fs__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! node:fs */ 973024);
/* harmony import */ var node_fs__WEBPACK_IMPORTED_MODULE_1___default = /*#__PURE__*/__webpack_require__.n(node_fs__WEBPACK_IMPORTED_MODULE_1__);

Remove only the line that assigns node_path__WEBPACK_IMPORTED_MODULE_0___default, leaving the raw node_path__WEBPACK_IMPORTED_MODULE_0__ import and the node:fs imports untouched. No new methods, imports, or definitions are required, and existing behavior remains the same because the unused helper was never referenced.

In general, to fix an unused-variable issue, either remove the declaration if the value is truly never used, or update the code to use it where it was intended. Here, node_fs__WEBPACK_IMPORTED_MODULE_1___default is declared but never used, while node_fs__WEBPACK_IMPORTED_MODULE_1__ is used for readFileSync. The best non‑behavior‑changing fix is to remove the unused default helper variable and keep the namespace import intact.

Concretely, in common/scripts/install-run-rush.js, we adjust the import section around lines 114–117 so that we only define node_fs__WEBPACK_IMPORTED_MODULE_1__ (the object we actually use) and drop the node_fs__WEBPACK_IMPORTED_MODULE_1___default = ... line. We leave the node:path import unchanged because both its namespace and default helpers might be used elsewhere in the file; the only change is to the unused node_fs__WEBPACK_IMPORTED_MODULE_1___default variable.

In general, to fix an unused-variable issue you either remove the variable declaration (and any associated initializer) or start using the variable if it was meant to be used. Here, node_fs__WEBPACK_IMPORTED_MODULE_0___default is an automatically created alias for a default import of node:fs, but the code appears to use only the namespace import node_fs__WEBPACK_IMPORTED_MODULE_0__. The safest minimal fix is to remove the unused alias assignment while leaving the working namespace import intact.

Concretely, in common/scripts/install-run.js, in the npmrcUtilities.js section around lines 84–87, we will remove the line that defines node_fs__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(node_fs__WEBPACK_IMPORTED_MODULE_0__); and leave the rest unchanged. No new methods or imports are needed, and existing behavior will be preserved since the removed variable was never used.

In general, unused variables should be removed to improve readability and avoid confusion. Here, the unused symbol is node_path__WEBPACK_IMPORTED_MODULE_1___default, created by the line assigning __webpack_require__.n(...). To fix this without changing functionality, we should stop binding the result of __webpack_require__.n(node_path__WEBPACK_IMPORTED_MODULE_1__) to a variable that is never read. Since the import of node:path itself (node_path__WEBPACK_IMPORTED_MODULE_1__) is presumably still used elsewhere, we leave that line unchanged.

Concretely, in common/scripts/install-run.js, within the npmrcUtilities.js module header, remove the line that assigns to node_path__WEBPACK_IMPORTED_MODULE_1___default and keep the rest as-is. No additional methods, definitions, or imports are required.

In general, the best fix for an unused variable created by an import is to remove the unused binding while keeping any needed side effects of the import. Here, the only unused element is the locally scoped variable node_child_process__WEBPACK_IMPORTED_MODULE_0___default; the side‑effectful require call is already associated with node_child_process__WEBPACK_IMPORTED_MODULE_0__, which we must preserve. We therefore should eliminate just the extra __default variable assignment line and leave the rest of the imports untouched.

Concretely, in common/scripts/install-run.js, around lines 355–362, we will remove the line that declares and initializes node_child_process__WEBPACK_IMPORTED_MODULE_0___default while keeping the existing /* harmony import */ line for node_child_process__WEBPACK_IMPORTED_MODULE_0__ and all subsequent imports (node_fs__WEBPACK_IMPORTED_MODULE_1__, node_os__WEBPACK_IMPORTED_MODULE_2__, node_path__WEBPACK_IMPORTED_MODULE_3__, etc.) unchanged. No new methods, imports, or definitions are required.

In general, the correct fix for an unused variable in bundled code is to remove the declaration (and any immediately associated initialization) while leaving all actually used imports and logic intact. This preserves behavior and reduces noise and potential confusion.

Here, the pattern is:

/* harmony import */ var node_fs__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! node:fs */ 973024);
(/* harmony import */ var node_fs__WEBPACK_IMPORTED_MODULE_1___default = /*#__PURE__*/__webpack_require__.n(node_fs__WEBPACK_IMPORTED_MODULE_1__);)

CodeQL reports the node_fs__WEBPACK_IMPORTED_MODULE_1___default variable as unused. The underlying node_fs__WEBPACK_IMPORTED_MODULE_1__ import is presumably still used and must remain. The best minimal fix is to remove just the unused default-helper declaration line that assigns node_fs__WEBPACK_IMPORTED_MODULE_1___default, leaving the original import line for node_fs__WEBPACK_IMPORTED_MODULE_1__ untouched and leaving all subsequent code unchanged.

Concretely, in common/scripts/install-run.js, within the ./lib-esnext/scripts/install-run.js section around lines 355–362, delete the line that declares node_fs__WEBPACK_IMPORTED_MODULE_1___default. No new imports, methods, or definitions are required.

In general, to fix an unused-variable/import issue you either (a) remove the unused binding, or (b) actually use it if it was meant to be used. Here, the generated code imports the Node os module twice: once as the module itself (node_os__WEBPACK_IMPORTED_MODULE_2__) and once as a default-wrapper helper (node_os__WEBPACK_IMPORTED_MODULE_2___default = __webpack_require__.n(...)). Only the second binding is reported unused.

The minimal, behavior-preserving fix is to delete the unused default-wrapper binding, leaving the actual node:os module import intact. Concretely, in common/scripts/install-run.js, replace the two-line pattern:

/* harmony import */ var node_os__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(/*! node:os */ 848161);
(node_os__WEBPACK_IMPORTED_MODULE_2___default = /*#__PURE__*/__webpack_require__.n(node_os__WEBPACK_IMPORTED_MODULE_2__));

with a single import line that only declares node_os__WEBPACK_IMPORTED_MODULE_2__. This removes node_os__WEBPACK_IMPORTED_MODULE_2___default completely, satisfying CodeQL without altering any other logic.

In general, unused variables or imports should be removed so that every declared binding is actually needed. Here, the tool-generated line both requires the node:path module and assigns its default export to a variable that is never used: var node_path__WEBPACK_IMPORTED_MODULE_3___default = .... The best minimal fix without changing runtime behavior is to keep the __webpack_require__ call (so the module is still registered/available) but drop the unused variable binding.

Concretely, in common/scripts/install-run.js around line 361–362, replace the line that assigns __webpack_require__.n(node_path__WEBPACK_IMPORTED_MODULE_3__) to node_path__WEBPACK_IMPORTED_MODULE_3___default with a line that just calls __webpack_require__.n(node_path__WEBPACK_IMPORTED_MODULE_3__); for side effects, without assigning the result. All other imports and code remain unchanged.

[Copilot]

Pull request overview

This PR updates three key dependencies across the Application Insights JavaScript SDK monorepo: @microsoft/rush (build tool), @nevware21/ts-utils (utility library), and @nevware21/ts-async (async utilities library). The updates are minor version increments that maintain backward compatibility.

Changes:

• Updated @microsoft/rush from 5.153.1 to 5.166.0
• Updated @nevware21/ts-utils from >= 0.11.8 to >= 0.12.6 across all packages
• Updated @nevware21/ts-async from >= 0.5.4 to >= 0.5.5 across all packages

Reviewed changes

Copilot reviewed 29 out of 30 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
rush.json	Updated rushVersion to 5.166.0
package.json	Updated @microsoft/rush devDependency to 5.166.0
common/scripts/install-run.js	Auto-generated Rush script updated with Node.js protocol imports and improved Windows shell command handling
common/scripts/install-run-rush.js	Auto-generated Rush script updated with Node.js protocol imports
common/config/rush/npm-shrinkwrap.json	Updated lockfile with new dependency resolutions and integrity hashes
shared/AppInsightsCore/package.json	Updated ts-utils to 0.12.6 and ts-async to 0.5.5
shared/AppInsightsCommon/package.json	Updated ts-utils to 0.12.6
shared/1ds-core-js/package.json	Updated ts-utils to 0.12.6 and ts-async to 0.5.5 in both dependencies and devDependencies
AISKU/package.json	Updated ts-utils to 0.12.6 and ts-async to 0.5.5
AISKULight/package.json	Updated ts-utils to 0.12.6 and ts-async to 0.5.5
tools/shims/package.json	Updated ts-utils to 0.12.6
tools/chrome-debug-extension/package.json	Updated ts-utils to 0.12.6 and ts-async to 0.5.5
common/Tests/Framework/package.json	Updated ts-utils to 0.12.6 and ts-async to 0.5.5
channels/*/package.json (4 files)	Updated ts-utils to 0.12.6 and ts-async to 0.5.5 across all channel packages
extensions/*/package.json (9 files)	Updated ts-utils to 0.12.6 and ts-async to 0.5.5 where applicable across all extension packages
examples/*/package.json (5 files)	Updated ts-utils to 0.12.6 across all example packages

Files not reviewed (1)

• common/config/rush/npm-shrinkwrap.json: Language not supported

[Copilot]

Unused variable node_path__WEBPACK_IMPORTED_MODULE_0___default.

Suggested change

	/* harmony import */ var node_path__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(node_path__WEBPACK_IMPORTED_MODULE_0__);
	/* harmony import */ var node_fs__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! node:fs */ 973024);
	/* harmony import */ var node_fs__WEBPACK_IMPORTED_MODULE_1___default = /*#__PURE__*/__webpack_require__.n(node_fs__WEBPACK_IMPORTED_MODULE_1__);
	/* harmony import */ var node_fs__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! node:fs */ 973024);

[Copilot]

Unused variable node_fs__WEBPACK_IMPORTED_MODULE_1___default.

[Copilot]

Unused variable node_fs__WEBPACK_IMPORTED_MODULE_0___default.

Suggested change

/* harmony import */ var node_fs__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(node_fs__WEBPACK_IMPORTED_MODULE_0__);

[Copilot]

Unused variable node_path__WEBPACK_IMPORTED_MODULE_1___default.

Suggested change

/* harmony import */ var node_path__WEBPACK_IMPORTED_MODULE_1___default = /*#__PURE__*/__webpack_require__.n(node_path__WEBPACK_IMPORTED_MODULE_1__);

[Copilot]

Unused variable node_child_process__WEBPACK_IMPORTED_MODULE_0___default.

[Copilot]

Unused variable node_fs__WEBPACK_IMPORTED_MODULE_1___default.

Suggested change

/* harmony import */ var node_fs__WEBPACK_IMPORTED_MODULE_1___default = /*#__PURE__*/__webpack_require__.n(node_fs__WEBPACK_IMPORTED_MODULE_1__);

[Copilot]

Unused variable node_os__WEBPACK_IMPORTED_MODULE_2___default.

Suggested change

/* harmony import */ var node_os__WEBPACK_IMPORTED_MODULE_2___default = /*#__PURE__*/__webpack_require__.n(node_os__WEBPACK_IMPORTED_MODULE_2__);

[Copilot]

Unused variable node_path__WEBPACK_IMPORTED_MODULE_3___default.

Suggested change

/* harmony import */ var node_path__WEBPACK_IMPORTED_MODULE_3___default = /*#__PURE__*/__webpack_require__.n(node_path__WEBPACK_IMPORTED_MODULE_3__);