feat: add integration testing setup and esbuild for bundling

- Introduced `.vscode-test.mjs` for running integration tests within a VS Code environment.
- Added `esbuild.js` for building and bundling the extension, replacing the previous TypeScript compilation method.
- Updated `package.json` scripts to include new build and test commands.
- Modified `.vscodeignore` to exclude new build artifacts and test files.
- Created initial integration tests in `src/integration/extension.itest.ts` to verify extension activation and command registration.

Author: Arul1998

.github/workflows/ci.yml

@@ -5,9 +5,17 @@ on:
     branches: [main]
   pull_request:
 
+permissions:
+  contents: read
+
 jobs:
-  build:
-    runs-on: ubuntu-latest
+  test:
+    name: Test (${{ matrix.os }})
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, windows-latest, macos-latest]
 
     steps:
       - uses: actions/checkout@v4
@@ -20,14 +28,45 @@ jobs:
       - name: Install dependencies
         run: npm ci
 
-      - name: Compile
+      - name: Type-check and bundle
         run: npm run compile
 
       - name: Lint
         run: npm run lint
 
-      - name: Test
+      - name: Test (unit)
         run: npm test
 
-      - name: Package (smoke check)
+      - name: Test (integration, Linux)
+        if: runner.os == 'Linux'
+        run: xvfb-run -a npm run test:integration
+
+      - name: Test (integration, Windows/macOS)
+        if: runner.os != 'Linux'
+        run: npm run test:integration
+
+  package:
+    name: Package (smoke check)
+    runs-on: ubuntu-latest
+    needs: test
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Package VSIX
         run: npx --yes @vscode/vsce package --out angular-code-quality-toolkit.vsix
+
+      - name: Upload VSIX artifact
+        uses: actions/upload-artifact@v4
+        with:
+          name: angular-code-quality-toolkit-vsix
+          path: angular-code-quality-toolkit.vsix
+          if-no-files-found: error

.vscode-test.mjs

@@ -0,0 +1,13 @@
+import { defineConfig } from '@vscode/test-cli';
+
+// Runs the compiled integration tests (src/integration/*.itest.ts -> out/integration)
+// inside a real VS Code Extension Development Host. Unit tests live in src/test and
+// run separately via `npm test` (node:test); this file drives Mocha for the host tests.
+export default defineConfig({
+  files: 'out/integration/**/*.itest.js',
+  version: 'stable',
+  mocha: {
+    ui: 'tdd',
+    timeout: 20000,
+  },
+});

.vscodeignore

@@ -1,13 +1,15 @@
-out/test/**
+out/**
 src/**
 .vscode/**
+.vscode-test/**
 **/*.map
 **/*.ts
 node_modules/**
+esbuild.js
+.vscode-test.mjs
 eslint.config.mjs
 tsconfig.json
 package-lock.json
 .gitignore
 .github/**
 assets/angular-code-quality-icon.png
-!.vscode-test/**

README.md

@@ -85,7 +85,7 @@ To catch unused variables and parameters, add the rule to your ESLint config so
 | `angularCodeQuality.stylelint.useJsonFormat` | `true` | Ask stylelint for JSON output. |
 | `angularCodeQuality.depcheck.ignoreAngularImplicit` | `true` | Hide false "unused" hits for packages Angular uses implicitly (`@angular/*`, `zone.js`, `rxjs`, `tslib`, `typescript`, karma/jasmine, builders). |
 | `angularCodeQuality.depcheck.ignores` | `[]` | Extra packages to hide (`*` wildcard, e.g. `@my-scope/*`). |
-| `angularCodeQuality.revealOutputOnRun` | `true` | Auto-open the Output channel on each run. |
+| `angularCodeQuality.revealOutputOnRun` | `false` | Auto-open the Output channel on each run. Off by default — findings go to the Problems panel; enable this only to watch raw tool logs. |
 
 ---
 

esbuild.js

@@ -0,0 +1,57 @@
+const esbuild = require('esbuild');
+
+const production = process.argv.includes('--production');
+const watch = process.argv.includes('--watch');
+
+/**
+ * Reports esbuild problems in a format VS Code's task matcher understands and
+ * fails the build (non-zero exit) when a bundle error occurs outside watch mode.
+ * @type {import('esbuild').Plugin}
+ */
+const esbuildProblemMatcherPlugin = {
+  name: 'esbuild-problem-matcher',
+  setup(build) {
+    build.onStart(() => {
+      console.log('[watch] build started');
+    });
+    build.onEnd((result) => {
+      result.errors.forEach(({ text, location }) => {
+        console.error(`✘ [ERROR] ${text}`);
+        if (location) {
+          console.error(`    ${location.file}:${location.line}:${location.column}:`);
+        }
+      });
+      console.log('[watch] build finished');
+    });
+  },
+};
+
+async function main() {
+  const ctx = await esbuild.context({
+    entryPoints: ['src/extension.ts'],
+    bundle: true,
+    format: 'cjs',
+    minify: production,
+    sourcemap: !production,
+    sourcesContent: false,
+    platform: 'node',
+    target: 'node18',
+    outfile: 'dist/extension.js',
+    // The 'vscode' module is provided by the VS Code runtime, never bundled.
+    external: ['vscode'],
+    logLevel: 'silent',
+    plugins: [esbuildProblemMatcherPlugin],
+  });
+
+  if (watch) {
+    await ctx.watch();
+  } else {
+    await ctx.rebuild();
+    await ctx.dispose();
+  }
+}
+
+main().catch((e) => {
+  console.error(e);
+  process.exit(1);
+});