@avi770/testteam 1.2.0

package/dist/helpers/screenshot.js

@@ -0,0 +1,76 @@
+import { mkdir, copyFile, readFile, writeFile, stat } from 'node:fs/promises';
+import { join, basename } from 'path';
+import pixelmatch from 'pixelmatch';
+import { PNG } from 'pngjs';
+async function fileExists(p) {
+    try {
+        await stat(p);
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * Captures a screenshot of the current page state and saves it to outputDir.
+ * Returns the absolute file path of the saved screenshot.
+ */
+export async function captureScreenshot(page, name, outputDir) {
+    const dirExists = await fileExists(outputDir);
+    if (!dirExists) {
+        await mkdir(outputDir, { recursive: true });
+    }
+    const safeName = name.replace(/[^a-zA-Z0-9_-]/g, '_');
+    const filePath = join(outputDir, `${safeName}.png`);
+    await page.screenshot({ path: filePath, fullPage: false });
+    return filePath;
+}
+/**
+ * Compares two PNG screenshots using pixelmatch.
+ * Returns match=true when the diff percentage is at or below the threshold.
+ * Default threshold: 0.1 (0.1% pixel difference).
+ * Writes a diff PNG alongside the current screenshot when images differ.
+ */
+export async function compareScreenshots(currentPath, baselinePath, threshold = 0.1) {
+    const currentData = await readFile(currentPath);
+    const baselineData = await readFile(baselinePath);
+    const current = PNG.sync.read(currentData);
+    const baseline = PNG.sync.read(baselineData);
+    if (current.width !== baseline.width || current.height !== baseline.height) {
+        return {
+            match: false,
+            diffPercentage: 100,
+        };
+    }
+    const { width, height } = current;
+    const diff = new PNG({ width, height });
+    const diffPixels = pixelmatch(current.data, baseline.data, diff.data, width, height, { threshold: 0.1 });
+    const totalPixels = width * height;
+    const diffPercentage = totalPixels > 0 ? (diffPixels / totalPixels) * 100 : 0;
+    const match = diffPercentage <= threshold;
+    let diffPath;
+    if (!match) {
+        diffPath = currentPath.replace(/\.png$/, '_diff.png');
+        await writeFile(diffPath, PNG.sync.write(diff));
+    }
+    return {
+        match,
+        diffPercentage,
+        ...(diffPath ? { diffPath } : {}),
+    };
+}
+/**
+ * Copies the current screenshot to the baseline directory, establishing it
+ * as the new reference image. Returns the path of the new baseline file.
+ */
+export async function updateBaseline(currentPath, baselineDir) {
+    const dirExists = await fileExists(baselineDir);
+    if (!dirExists) {
+        await mkdir(baselineDir, { recursive: true });
+    }
+    const fileName = basename(currentPath);
+    const baselinePath = join(baselineDir, fileName);
+    await copyFile(currentPath, baselinePath);
+    return baselinePath;
+}
+//# sourceMappingURL=screenshot.js.map

package/dist/helpers/screenshot.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"screenshot.js","sourceRoot":"","sources":["../../helpers/screenshot.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,QAAQ,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAC9E,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,MAAM,CAAC;AACtC,OAAO,UAAU,MAAM,YAAY,CAAC;AACpC,OAAO,EAAE,GAAG,EAAE,MAAM,OAAO,CAAC;AAS5B,KAAK,UAAU,UAAU,CAAC,CAAS;IACjC,IAAI,CAAC;QAAC,MAAM,IAAI,CAAC,CAAC,CAAC,CAAC;QAAC,OAAO,IAAI,CAAC;IAAC,CAAC;IAAC,MAAM,CAAC;QAAC,OAAO,KAAK,CAAC;IAAC,CAAC;AAC7D,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,iBAAiB,CACrC,IAAU,EACV,IAAY,EACZ,SAAiB;IAEjB,MAAM,SAAS,GAAG,MAAM,UAAU,CAAC,SAAS,CAAC,CAAC;IAC9C,IAAI,CAAC,SAAS,EAAE,CAAC;QACf,MAAM,KAAK,CAAC,SAAS,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC9C,CAAC;IAED,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAAC,iBAAiB,EAAE,GAAG,CAAC,CAAC;IACtD,MAAM,QAAQ,GAAG,IAAI,CAAC,SAAS,EAAE,GAAG,QAAQ,MAAM,CAAC,CAAC;IAEpD,MAAM,IAAI,CAAC,UAAU,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,QAAQ,EAAE,KAAK,EAAE,CAAC,CAAC;IAE3D,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,WAAmB,EACnB,YAAoB,EACpB,SAAS,GAAG,GAAG;IAEf,MAAM,WAAW,GAAG,MAAM,QAAQ,CAAC,WAAW,CAAC,CAAC;IAChD,MAAM,YAAY,GAAG,MAAM,QAAQ,CAAC,YAAY,CAAC,CAAC;IAElD,MAAM,OAAO,GAAG,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAC3C,MAAM,QAAQ,GAAG,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAE7C,IAAI,OAAO,CAAC,KAAK,KAAK,QAAQ,CAAC,KAAK,IAAI,OAAO,CAAC,MAAM,KAAK,QAAQ,CAAC,MAAM,EAAE,CAAC;QAC3E,OAAO;YACL,KAAK,EAAE,KAAK;YACZ,cAAc,EAAE,GAAG;SACpB,CAAC;IACJ,CAAC;IAED,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC;IAClC,MAAM,IAAI,GAAG,IAAI,GAAG,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC;IAExC,MAAM,UAAU,GAAG,UAAU,CAC3B,OAAO,CAAC,IAAI,EACZ,QAAQ,CAAC,IAAI,EACb,IAAI,CAAC,IAAI,EACT,KAAK,EACL,MAAM,EACN,EAAE,SAAS,EAAE,GAAG,EAAE,CACnB,CAAC;IAEF,MAAM,WAAW,GAAG,KAAK,GAAG,MAAM,CAAC;IACnC,MAAM,cAAc,GAAG,WAAW,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,UAAU,GAAG,WAAW,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC;IAC9E,MAAM,KAAK,GAAG,cAAc,IAAI,SAAS,CAAC;IAE1C,IAAI,QAA4B,CAAC;IACjC,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,QAAQ,GAAG,WAAW,CAAC,OAAO,CAAC,QAAQ,EAAE,WAAW,CAAC,CAAC;QACtD,MAAM,SAAS,CAAC,QAAQ,EAAE,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC;IAClD,CAAC;IAED,OAAO;QACL,KAAK;QACL,cAAc;QACd,GAAG,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;KAClC,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,cAAc,CAClC,WAAmB,EACnB,WAAmB;IAEnB,MAAM,SAAS,GAAG,MAAM,UAAU,CAAC,WAAW,CAAC,CAAC;IAChD,IAAI,CAAC,SAAS,EAAE,CAAC;QACf,MAAM,KAAK,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAChD,CAAC;IAED,MAAM,QAAQ,GAAG,QAAQ,CAAC,WAAW,CAAC,CAAC;IACvC,MAAM,YAAY,GAAG,IAAI,CAAC,WAAW,EAAE,QAAQ,CAAC,CAAC;IAEjD,MAAM,QAAQ,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC;IAE1C,OAAO,YAAY,CAAC;AACtB,CAAC"}

package/dist/helpers/seed-validator.d.ts

@@ -0,0 +1,15 @@
+export interface SeedValidationResult {
+    covered: string[];
+    missing: string[];
+}
+/**
+ * Validates that a Prisma seed file contains data creation entries for each
+ * of the supplied module IDs.
+ *
+ * Detection heuristic: for each moduleId the function looks for the module
+ * identifier (case-insensitive) appearing in a data-creation context — i.e.
+ * adjacent to keywords such as create, createMany, insert, upsert, seed, or
+ * model within 120 characters. This covers common Prisma and raw-SQL patterns.
+ */
+export declare function validateSeedCompleteness(seedFilePath: string, moduleIds: string[]): Promise<SeedValidationResult>;
+//# sourceMappingURL=seed-validator.d.ts.map

package/dist/helpers/seed-validator.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"seed-validator.d.ts","sourceRoot":"","sources":["../../helpers/seed-validator.ts"],"names":[],"mappings":"AAEA,MAAM,WAAW,oBAAoB;IACnC,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,OAAO,EAAE,MAAM,EAAE,CAAC;CACnB;AAED;;;;;;;;GAQG;AACH,wBAAsB,wBAAwB,CAC5C,YAAY,EAAE,MAAM,EACpB,SAAS,EAAE,MAAM,EAAE,GAClB,OAAO,CAAC,oBAAoB,CAAC,CAgB/B"}

package/dist/helpers/seed-validator.js

@@ -0,0 +1,53 @@
+import { readFileSync } from 'fs';
+/**
+ * Validates that a Prisma seed file contains data creation entries for each
+ * of the supplied module IDs.
+ *
+ * Detection heuristic: for each moduleId the function looks for the module
+ * identifier (case-insensitive) appearing in a data-creation context — i.e.
+ * adjacent to keywords such as create, createMany, insert, upsert, seed, or
+ * model within 120 characters. This covers common Prisma and raw-SQL patterns.
+ */
+export async function validateSeedCompleteness(seedFilePath, moduleIds) {
+    const content = readFileSync(seedFilePath, 'utf-8');
+    const contentLower = content.toLowerCase();
+    const covered = [];
+    const missing = [];
+    for (const moduleId of moduleIds) {
+        if (isCoveredByContent(contentLower, moduleId.toLowerCase())) {
+            covered.push(moduleId);
+        }
+        else {
+            missing.push(moduleId);
+        }
+    }
+    return { covered, missing };
+}
+/**
+ * Returns true if the seed content appears to create data for the given module.
+ * Checks for the module name near data-creation keywords or as a Prisma model reference.
+ */
+function isCoveredByContent(contentLower, moduleIdLower) {
+    const creationKeywords = ['create', 'createmany', 'insert', 'upsert', 'seed', 'model'];
+    let searchStart = 0;
+    const moduleIndex = contentLower.indexOf(moduleIdLower, searchStart);
+    if (moduleIndex === -1) {
+        return false;
+    }
+    // Scan all occurrences of the module name for a nearby creation keyword
+    let idx = moduleIndex;
+    while (idx !== -1) {
+        const windowStart = Math.max(0, idx - 120);
+        const windowEnd = Math.min(contentLower.length, idx + moduleIdLower.length + 120);
+        const window = contentLower.slice(windowStart, windowEnd);
+        for (const keyword of creationKeywords) {
+            if (window.includes(keyword)) {
+                return true;
+            }
+        }
+        searchStart = idx + 1;
+        idx = contentLower.indexOf(moduleIdLower, searchStart);
+    }
+    return false;
+}
+//# sourceMappingURL=seed-validator.js.map

package/dist/helpers/seed-validator.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"seed-validator.js","sourceRoot":"","sources":["../../helpers/seed-validator.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,IAAI,CAAC;AAOlC;;;;;;;;GAQG;AACH,MAAM,CAAC,KAAK,UAAU,wBAAwB,CAC5C,YAAoB,EACpB,SAAmB;IAEnB,MAAM,OAAO,GAAG,YAAY,CAAC,YAAY,EAAE,OAAO,CAAC,CAAC;IACpD,MAAM,YAAY,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;IAE3C,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,MAAM,OAAO,GAAa,EAAE,CAAC;IAE7B,KAAK,MAAM,QAAQ,IAAI,SAAS,EAAE,CAAC;QACjC,IAAI,kBAAkB,CAAC,YAAY,EAAE,QAAQ,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC;YAC7D,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACzB,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACzB,CAAC;IACH,CAAC;IAED,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC;AAC9B,CAAC;AAED;;;GAGG;AACH,SAAS,kBAAkB,CAAC,YAAoB,EAAE,aAAqB;IACrE,MAAM,gBAAgB,GAAG,CAAC,QAAQ,EAAE,YAAY,EAAE,QAAQ,EAAE,QAAQ,EAAE,MAAM,EAAE,OAAO,CAAC,CAAC;IAEvF,IAAI,WAAW,GAAG,CAAC,CAAC;IACpB,MAAM,WAAW,GAAG,YAAY,CAAC,OAAO,CAAC,aAAa,EAAE,WAAW,CAAC,CAAC;IAErE,IAAI,WAAW,KAAK,CAAC,CAAC,EAAE,CAAC;QACvB,OAAO,KAAK,CAAC;IACf,CAAC;IAED,wEAAwE;IACxE,IAAI,GAAG,GAAG,WAAW,CAAC;IACtB,OAAO,GAAG,KAAK,CAAC,CAAC,EAAE,CAAC;QAClB,MAAM,WAAW,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,GAAG,GAAG,GAAG,CAAC,CAAC;QAC3C,MAAM,SAAS,GAAG,IAAI,CAAC,GAAG,CAAC,YAAY,CAAC,MAAM,EAAE,GAAG,GAAG,aAAa,CAAC,MAAM,GAAG,GAAG,CAAC,CAAC;QAClF,MAAM,MAAM,GAAG,YAAY,CAAC,KAAK,CAAC,WAAW,EAAE,SAAS,CAAC,CAAC;QAE1D,KAAK,MAAM,OAAO,IAAI,gBAAgB,EAAE,CAAC;YACvC,IAAI,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC;gBAC7B,OAAO,IAAI,CAAC;YACd,CAAC;QACH,CAAC;QAED,WAAW,GAAG,GAAG,GAAG,CAAC,CAAC;QACtB,GAAG,GAAG,YAAY,CAAC,OAAO,CAAC,aAAa,EAAE,WAAW,CAAC,CAAC;IACzD,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC"}

package/helpers/__tests__/api-client.test.ts

@@ -0,0 +1,199 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { ApiClient } from '../api-client';
+import type { ApiResponse } from '../api-client';
+
+// ---------------------------------------------------------------------------
+// fetch mock helpers
+// ---------------------------------------------------------------------------
+
+function makeFetchResponse(
+  status: number,
+  body: unknown,
+  headers: Record<string, string> = {},
+): Response {
+  const responseHeaders = new Headers({
+    'content-type': 'application/json',
+    ...headers,
+  });
+
+  return {
+    status,
+    headers: responseHeaders,
+    json: async () => body,
+    text: async () => JSON.stringify(body),
+  } as unknown as Response;
+}
+
+// ---------------------------------------------------------------------------
+// Tests
+// ---------------------------------------------------------------------------
+
+describe('ApiClient', () => {
+  beforeEach(() => {
+    vi.stubGlobal('fetch', vi.fn());
+  });
+
+  afterEach(() => {
+    vi.unstubAllGlobals();
+  });
+
+  describe('Authorization header', () => {
+    it('includes Bearer token in GET requests when token is provided', async () => {
+      const mockFetch = vi.mocked(fetch);
+      mockFetch.mockResolvedValueOnce(makeFetchResponse(200, { ok: true }));
+
+      const client = new ApiClient('https://api.example.com', 'my-jwt-token');
+      await client.get('/api/data');
+
+      expect(mockFetch).toHaveBeenCalledOnce();
+      const [, init] = mockFetch.mock.calls[0];
+      const headers = init?.headers as Record<string, string>;
+      expect(headers['Authorization']).toBe('Bearer my-jwt-token');
+    });
+
+    it('omits Authorization header when no token is provided', async () => {
+      const mockFetch = vi.mocked(fetch);
+      mockFetch.mockResolvedValueOnce(makeFetchResponse(200, {}));
+
+      const client = new ApiClient('https://api.example.com');
+      await client.get('/api/public');
+
+      const [, init] = mockFetch.mock.calls[0];
+      const headers = init?.headers as Record<string, string>;
+      expect(headers['Authorization']).toBeUndefined();
+    });
+
+    it('includes Bearer token in POST requests', async () => {
+      const mockFetch = vi.mocked(fetch);
+      mockFetch.mockResolvedValueOnce(makeFetchResponse(201, { id: 1 }));
+
+      const client = new ApiClient('https://api.example.com', 'post-token');
+      await client.post('/api/items', { name: 'test' });
+
+      const [, init] = mockFetch.mock.calls[0];
+      const headers = init?.headers as Record<string, string>;
+      expect(headers['Authorization']).toBe('Bearer post-token');
+    });
+
+    it('includes Bearer token in PUT requests', async () => {
+      const mockFetch = vi.mocked(fetch);
+      mockFetch.mockResolvedValueOnce(makeFetchResponse(200, { updated: true }));
+
+      const client = new ApiClient('https://api.example.com', 'put-token');
+      await client.put('/api/items/1', { name: 'updated' });
+
+      const [, init] = mockFetch.mock.calls[0];
+      const headers = init?.headers as Record<string, string>;
+      expect(headers['Authorization']).toBe('Bearer put-token');
+    });
+
+    it('includes Bearer token in DELETE requests', async () => {
+      const mockFetch = vi.mocked(fetch);
+      mockFetch.mockResolvedValueOnce(makeFetchResponse(204, null));
+
+      const client = new ApiClient('https://api.example.com', 'delete-token');
+      await client.delete('/api/items/1');
+
+      const [, init] = mockFetch.mock.calls[0];
+      const headers = init?.headers as Record<string, string>;
+      expect(headers['Authorization']).toBe('Bearer delete-token');
+    });
+  });
+
+  describe('response timing', () => {
+    it('records a non-negative durationMs', async () => {
+      const mockFetch = vi.mocked(fetch);
+      mockFetch.mockResolvedValueOnce(makeFetchResponse(200, {}));
+
+      const client = new ApiClient('https://api.example.com');
+      const response: ApiResponse = await client.get('/api/ping');
+
+      expect(response.durationMs).toBeGreaterThanOrEqual(0);
+      expect(typeof response.durationMs).toBe('number');
+    });
+  });
+
+  describe('response shape', () => {
+    it('returns status, body, headers, and durationMs', async () => {
+      const mockFetch = vi.mocked(fetch);
+      mockFetch.mockResolvedValueOnce(
+        makeFetchResponse(200, { message: 'hello' }, { 'x-request-id': 'abc123' }),
+      );
+
+      const client = new ApiClient('https://api.example.com');
+      const response = await client.get('/api/hello');
+
+      expect(response.status).toBe(200);
+      expect(response.body).toEqual({ message: 'hello' });
+      expect(response.headers).toHaveProperty('content-type');
+      expect(typeof response.durationMs).toBe('number');
+    });
+  });
+
+  describe('HTTP methods', () => {
+    it('uses GET method for get()', async () => {
+      vi.mocked(fetch).mockResolvedValueOnce(makeFetchResponse(200, {}));
+      const client = new ApiClient('https://api.example.com');
+      await client.get('/api/test');
+      const [, init] = vi.mocked(fetch).mock.calls[0];
+      expect(init?.method).toBe('GET');
+    });
+
+    it('uses POST method and sends JSON body for post()', async () => {
+      vi.mocked(fetch).mockResolvedValueOnce(makeFetchResponse(201, {}));
+      const client = new ApiClient('https://api.example.com');
+      await client.post('/api/test', { key: 'value' });
+      const [, init] = vi.mocked(fetch).mock.calls[0];
+      expect(init?.method).toBe('POST');
+      expect(init?.body).toBe(JSON.stringify({ key: 'value' }));
+    });
+
+    it('uses PUT method for put()', async () => {
+      vi.mocked(fetch).mockResolvedValueOnce(makeFetchResponse(200, {}));
+      const client = new ApiClient('https://api.example.com');
+      await client.put('/api/test/1', { updated: true });
+      const [, init] = vi.mocked(fetch).mock.calls[0];
+      expect(init?.method).toBe('PUT');
+    });
+
+    it('uses DELETE method for delete()', async () => {
+      vi.mocked(fetch).mockResolvedValueOnce(makeFetchResponse(204, null));
+      const client = new ApiClient('https://api.example.com');
+      await client.delete('/api/test/1');
+      const [, init] = vi.mocked(fetch).mock.calls[0];
+      expect(init?.method).toBe('DELETE');
+    });
+  });
+
+  describe('createAuthenticated()', () => {
+    it('returns an authenticated client when login succeeds with access_token', async () => {
+      const mockFetch = vi.mocked(fetch);
+      mockFetch.mockResolvedValueOnce(
+        makeFetchResponse(200, { access_token: 'jwt-abc' }),
+      );
+
+      const client = await ApiClient.createAuthenticated('https://api.example.com', {
+        email: 'user@example.com',
+        password: 'secret',
+      });
+
+      // Verify the token is used on the next request
+      mockFetch.mockResolvedValueOnce(makeFetchResponse(200, {}));
+      await client.get('/api/me');
+      const [, init] = mockFetch.mock.calls[1];
+      const headers = init?.headers as Record<string, string>;
+      expect(headers['Authorization']).toBe('Bearer jwt-abc');
+    });
+
+    it('throws when the login response contains no token', async () => {
+      vi.mocked(fetch).mockResolvedValueOnce(makeFetchResponse(401, { error: 'Unauthorized' }));
+
+      await expect(
+        ApiClient.createAuthenticated('https://api.example.com', {
+          email: 'bad@example.com',
+          password: 'wrong',
+        }),
+      ).rejects.toThrow(/Authentication failed/);
+    });
+  });
+});

package/helpers/__tests__/element-discovery.test.ts

@@ -0,0 +1,202 @@
+import { describe, it, expect, vi } from 'vitest';
+import {
+  classifyAsDestructive,
+  discoverInteractiveElements,
+} from '../element-discovery';
+import type { DiscoveredElement } from '../element-discovery';
+import type { Page } from 'playwright';
+
+// ---------------------------------------------------------------------------
+// classifyAsDestructive
+// ---------------------------------------------------------------------------
+
+describe('classifyAsDestructive(text)', () => {
+  it('returns true for "Delete client"', () => {
+    expect(classifyAsDestructive('Delete client')).toBe(true);
+  });
+
+  it('returns false for "Save changes"', () => {
+    expect(classifyAsDestructive('Save changes')).toBe(false);
+  });
+
+  it('returns true for "Remove item"', () => {
+    expect(classifyAsDestructive('Remove item')).toBe(true);
+  });
+
+  it('returns true for "Archive"', () => {
+    expect(classifyAsDestructive('Archive')).toBe(true);
+  });
+
+  it('returns true for "Clear all"', () => {
+    expect(classifyAsDestructive('Clear all')).toBe(true);
+  });
+
+  it('returns true for "destroy"', () => {
+    expect(classifyAsDestructive('destroy')).toBe(true);
+  });
+
+  it('returns true for "Reset password" (case-insensitive)', () => {
+    expect(classifyAsDestructive('Reset password')).toBe(true);
+  });
+
+  it('returns false for "Submit"', () => {
+    expect(classifyAsDestructive('Submit')).toBe(false);
+  });
+
+  it('returns false for an empty string', () => {
+    expect(classifyAsDestructive('')).toBe(false);
+  });
+});
+
+// ---------------------------------------------------------------------------
+// discoverInteractiveElements — mock page
+// ---------------------------------------------------------------------------
+
+type EvaluateFn = (selector: string) => Array<{
+  tag: string;
+  role: string;
+  text: string;
+  ariaLabel: string | null;
+  testId: string | null;
+}>;
+
+function makeMockPage(evaluateResult: ReturnType<EvaluateFn>): Page {
+  return {
+    evaluate: vi.fn().mockResolvedValue(evaluateResult),
+  } as unknown as Page;
+}
+
+describe('discoverInteractiveElements(page)', () => {
+  it('returns a button element with correct type and selector', async () => {
+    const page = makeMockPage([
+      { tag: 'button', role: '', text: 'Save changes', ariaLabel: null, testId: null },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results).toHaveLength(1);
+    expect(results[0]?.type).toBe('button');
+    expect(results[0]?.isDestructive).toBe(false);
+    expect(results[0]?.selector).toContain('button');
+  });
+
+  it('returns a link element with type "link"', async () => {
+    const page = makeMockPage([
+      { tag: 'a', role: '', text: 'Go to dashboard', ariaLabel: null, testId: null },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results[0]?.type).toBe('link');
+  });
+
+  it('classifies a delete button as destructive', async () => {
+    const page = makeMockPage([
+      { tag: 'button', role: '', text: 'Delete client', ariaLabel: null, testId: null },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results[0]?.isDestructive).toBe(true);
+  });
+
+  it('prefers data-testid selector when present', async () => {
+    const page = makeMockPage([
+      { tag: 'button', role: '', text: 'Submit', ariaLabel: null, testId: 'submit-btn' },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results[0]?.selector).toBe('[data-testid="submit-btn"]');
+  });
+
+  it('falls back to aria-label selector when no data-testid', async () => {
+    const page = makeMockPage([
+      { tag: 'button', role: '', text: '', ariaLabel: 'Close dialog', testId: null },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results[0]?.selector).toBe('[aria-label="Close dialog"]');
+    expect(results[0]?.ariaLabel).toBe('Close dialog');
+  });
+
+  it('classifies role="tab" elements with type "tab"', async () => {
+    const page = makeMockPage([
+      { tag: 'div', role: 'tab', text: 'Overview', ariaLabel: null, testId: null },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results[0]?.type).toBe('tab');
+  });
+
+  it('classifies role="menuitem" elements with type "menuitem"', async () => {
+    const page = makeMockPage([
+      { tag: 'li', role: 'menuitem', text: 'Profile', ariaLabel: null, testId: null },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results[0]?.type).toBe('menuitem');
+  });
+
+  it('classifies input elements with type "input"', async () => {
+    const page = makeMockPage([
+      { tag: 'input', role: '', text: '', ariaLabel: 'Email', testId: null },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results[0]?.type).toBe('input');
+  });
+
+  it('deduplicates elements that resolve to the same selector', async () => {
+    const page = makeMockPage([
+      { tag: 'button', role: '', text: 'Save', ariaLabel: null, testId: 'save-btn' },
+      { tag: 'button', role: '', text: 'Save', ariaLabel: null, testId: 'save-btn' },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results).toHaveLength(1);
+  });
+
+  it('returns multiple distinct elements', async () => {
+    const page = makeMockPage([
+      { tag: 'button', role: '', text: 'Add client', ariaLabel: null, testId: 'add-client' },
+      { tag: 'button', role: '', text: 'Delete client', ariaLabel: null, testId: 'delete-client' },
+      { tag: 'a', role: '', text: 'Home', ariaLabel: null, testId: null },
+    ]);
+
+    const results: DiscoveredElement[] = await discoverInteractiveElements(page);
+
+    expect(results).toHaveLength(3);
+
+    const deleteEl = results.find((r) => r.selector === '[data-testid="delete-client"]');
+    expect(deleteEl?.isDestructive).toBe(true);
+
+    const addEl = results.find((r) => r.selector === '[data-testid="add-client"]');
+    expect(addEl?.isDestructive).toBe(false);
+  });
+
+  it('uses tag:text() selector as last resort', async () => {
+    const page = makeMockPage([
+      { tag: 'button', role: '', text: 'Confirm', ariaLabel: null, testId: null },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results[0]?.selector).toBe('button:text("Confirm")');
+  });
+
+  it('uses bare tag selector when there is no text, aria-label, or testId', async () => {
+    const page = makeMockPage([
+      { tag: 'button', role: '', text: '', ariaLabel: null, testId: null },
+    ]);
+
+    const results = await discoverInteractiveElements(page);
+
+    expect(results[0]?.selector).toBe('button');
+  });
+});