chore: migrate vscode-logging

.changeset/config.json

@@ -11,7 +11,12 @@
       "sample-action-client",
       "vscode-using-workspace-api"
     ],
-    ["yeoman-ui", "yeoman-ui-frontend"]
+    ["yeoman-ui", "yeoman-ui-frontend"],
+    [
+      "@vscode-logging/logger",
+      "@vscode-logging/types",
+      "@vscode-logging/wrapper"
+    ]
   ],
   "linked": [],
   "access": "restricted",

.eslintrc.js

@@ -85,6 +85,17 @@ module.exports = {
         "eslint-comments/require-description": "off",
       },
     },
+    {
+      // vscode-logging was integrated with these rules relaxed for the migrated code.
+      // TODO: clean up violations and tighten incrementally.
+      files: ["projects/vscode-logging/**"],
+      rules: {
+        "eslint-comments/require-description": "off",
+        // base rule for .js sources/tests (the @typescript-eslint variant is
+        // handled in the projects/vscode-logging/**/*.ts override below)
+        "no-unused-vars": "off",
+      },
+    },
     {
       // Use this package's tsconfig so type-aware linting sees its newer (@types/node) typings.
       files: ["packages/vsix-zst/**/*.ts"],
@@ -178,6 +189,19 @@ module.exports = {
         "@typescript-eslint/no-misused-promises": "off",
       },
     },
+    {
+      // Additional TypeScript rules for vscode-logging packages.
+      // TODO: clean up violations and tighten incrementally.
+      files: ["projects/vscode-logging/**/*.ts"],
+      rules: {
+        // public API surface (api.d.ts) intentionally exposes `any`
+        "@typescript-eslint/no-explicit-any": "off",
+        // chai assertions (e.g. expect(x).to.be.true) read as unused expressions
+        "@typescript-eslint/no-unused-expressions": "off",
+        "@typescript-eslint/no-unused-vars": "off",
+        "@typescript-eslint/no-unsafe-argument": "off",
+      },
+    },
     {
       // For Vue frontend sub-packages.
       files: ["*.vue"],

.gitignore

@@ -30,6 +30,9 @@ coverage
 # nyc test coverage
 .nyc_output
 
+# logger test log output
+.log-out
+
 # Dependency directories
 node_modules/
 

package.json

@@ -6,6 +6,7 @@
       "packages/*",
       "projects/*/packages/*",
       "projects/*",
+      "projects/*/examples/*",
       "examples/*"
     ]
   },
@@ -42,7 +43,7 @@
     "@types/deep-equal-in-any-order": "1.0.4",
     "@types/lodash": "4.14.177",
     "@types/mocha": "10.0.7",
-    "@types/node": "16.11.10",
+    "@types/node": "22.20.1",
     "@types/sinon": "17.0.4",
     "@types/vscode": "1.75.0",
     "@typescript-eslint/eslint-plugin": "8.60.1",

packages/app-studio-toolkit/src/devspace-manager/landscape/landscape.ts

@@ -20,7 +20,7 @@ const LBL_ADD_LANDSCAPE = "Add another landscape";
 const DEFAULT_REFRESH_RATE = 10 * 1000; // 10 sec default
 const DEFAULT_TIMEOUT = 2 * 60 * 1000; // 2 min default
 
-const autoRefreshTimerArray: NodeJS.Timer[] = [];
+const autoRefreshTimerArray: NodeJS.Timeout[] = [];
 
 export function clearAutoRefreshTimers(): void {
   autoRefreshTimerArray.forEach((interval) => {
@@ -34,7 +34,7 @@ export function autoRefresh(
   timeOut = DEFAULT_TIMEOUT
 ): void {
   let refreshedTime = 0;
-  const refreshInterval: NodeJS.Timer = setInterval(() => {
+  const refreshInterval: NodeJS.Timeout = setInterval(() => {
     getLandscapes()
       .then((landscapes) => {
         if (refreshedTime < timeOut && !isEmpty(landscapes)) {