@yasserkhanorg/impact-gate 2.0.0

package/dist/esm/prompts/generation.js

@@ -0,0 +1,297 @@
+// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
+// See LICENSE.txt for license information.
+import { formatApiSurfaceForPrompt } from '../knowledge/api_surface.js';
+import { sanitizeForPrompt } from '../crew/sanitize.js';
+import { isMattermostProfile } from './generation_profile.js';
+function resolveRelevantPageObjects(apiSurface, decision) {
+    const relevant = [];
+    const familyHints = [
+        decision.routeFamily,
+        decision.featureId,
+        ...decision.userActions.join(' ').toLowerCase().split(/\s+/),
+    ]
+        .filter(Boolean)
+        .map((s) => s.toLowerCase().replace(/[^a-z]/g, ''));
+    for (const po of apiSurface.pageObjects) {
+        const nameLower = po.className.toLowerCase();
+        if (nameLower.includes('channels') ||
+            nameLower.includes('page') ||
+            familyHints.some((hint) => hint.length > 3 && nameLower.includes(hint))) {
+            relevant.push(po.className);
+        }
+    }
+    return [...new Set(relevant)].slice(0, 10);
+}
+function buildAssertionPatternsBlock(patterns) {
+    if (!patterns || patterns.length === 0) {
+        return [];
+    }
+    return [
+        'REQUIRED ASSERTION PATTERNS:',
+        'Your tests MUST include assertions that verify each of these behaviors.',
+        'Do NOT just check element visibility — verify the actual business outcome.',
+        ...patterns.map((p) => `  - [${p.type}] ${p.pattern}`),
+        '',
+    ];
+}
+export function buildGenerationPrompt(ctx) {
+    const profile = ctx.profile;
+    const isMM = profile ? isMattermostProfile(profile) : false;
+    const relevantClasses = resolveRelevantPageObjects(ctx.apiSurface, ctx.decision);
+    const apiBlock = relevantClasses.length > 0
+        ? formatApiSurfaceForPrompt(ctx.apiSurface, relevantClasses)
+        : 'No page objects available. Use raw Playwright selectors via page.getByRole/getByTestId.';
+    const scenariosBlock = (ctx.decision.scenariosToAdd || [])
+        .map((s, i) => `  ${i + 1}. ${s}`)
+        .join('\n');
+    const existingBlock = ctx.existingSpecContent
+        ? `\nEXISTING SPEC (extend this file):\n\`\`\`typescript\n${ctx.existingSpecContent}\n\`\`\``
+        : '';
+    const modeInstruction = ctx.mode === 'create_spec'
+        ? `Create a NEW spec file at: ${ctx.specPath}`
+        : `ADD scenarios to the EXISTING spec at: ${ctx.specPath}`;
+    const routeFamilyTag = ctx.decision.routeFamily;
+    // Build prompt based on profile
+    const projectName = profile?.projectName || 'Project';
+    const testFramework = profile?.testFramework || 'Playwright';
+    const importStatement = profile?.importStatement || '@playwright/test';
+    // API test mode prompt
+    if (profile?.testMode === 'api') {
+        return buildApiTestPrompt(ctx, profile, scenariosBlock, routeFamilyTag);
+    }
+    // Build rules from profile conventions or use Mattermost defaults
+    const rules = isMM
+        ? [
+            `1. Import ONLY from "${importStatement}" — no other test framework imports.`,
+            '2. Every test must call `await pw.initSetup()` first.',
+            '3. Use `await pw.testBrowser.login(user)` to log in — never hardcode credentials.',
+            '4. Use ONLY page object methods listed above. Do NOT invent methods that are not listed.',
+            '5. If a method is not available, use `page.getByRole()` or `page.getByTestId()`.',
+            `6. Tag every test: {tag: '@${routeFamilyTag}'}`,
+            '7. Write one test per scenario with a descriptive name of what the user does and what is verified.',
+            `8. Use \`expect\` from "${importStatement}" — do NOT import from "@playwright/test".`,
+            '9. Include the copyright header for new files.',
+            '10. NEVER fabricate test IDs (MM-TXXXX). Use descriptive names only.',
+        ]
+        : [
+            ...(profile?.conventions || []).map((c, i) => `${i + 1}. ${c}`),
+            `${(profile?.conventions?.length || 0) + 1}. Use ONLY page object methods listed above. Do NOT invent methods that are not listed.`,
+            `${(profile?.conventions?.length || 0) + 2}. If a method is not available, use \`page.getByRole()\` or \`page.getByTestId()\`.`,
+            `${(profile?.conventions?.length || 0) + 3}. Tag every test: {tag: '@${routeFamilyTag}'}`,
+        ];
+    // Build example block
+    const exampleBlock = isMM
+        ? [
+            'EXAMPLE SPEC STRUCTURE:',
+            '```typescript',
+            '// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.',
+            '// See LICENSE.txt for license information.',
+            '',
+            `import {expect, test} from '${importStatement}';`,
+            '',
+            'test(',
+            "    'descriptive name of what is tested',",
+            `    {tag: '@${routeFamilyTag}'},`,
+            '    async ({pw}) => {',
+            '        const {user} = await pw.initSetup();',
+            '        const {channelsPage} = await pw.testBrowser.login(user);',
+            '        await channelsPage.goto();',
+            '        await channelsPage.toBeVisible();',
+            '        // test steps...',
+            '    },',
+            ');',
+            '```',
+        ]
+        : [
+            'EXAMPLE SPEC STRUCTURE:',
+            '```typescript',
+            ...(profile?.copyrightHeader ? [profile.copyrightHeader, ''] : []),
+            `import {test, expect} from '${importStatement}';`,
+            '',
+            'test(',
+            "    'descriptive name of what is tested',",
+            `    {tag: '@${routeFamilyTag}'},`,
+            '    async ({page}) => {',
+            '        // test steps...',
+            '    },',
+            ');',
+            '```',
+        ];
+    return [
+        `You are generating ${projectName} ${testFramework} E2E test code.`,
+        '',
+        `TASK: ${modeInstruction}`,
+        '',
+        `FLOW: ${sanitizeForPrompt(ctx.decision.flowName)}`,
+        `Route Family: ${ctx.decision.routeFamily}${ctx.decision.featureId ? ` / ${ctx.decision.featureId}` : ''}`,
+        `Route: ${ctx.decision.specificRoute || '(not specified)'}`,
+        `Priority: ${ctx.decision.priority}`,
+        `Evidence: ${sanitizeForPrompt(ctx.decision.evidence)}`,
+        '',
+        'SCENARIOS TO IMPLEMENT:',
+        scenariosBlock || '  (implement core user actions for this flow)',
+        '',
+        'USER ACTIONS:',
+        ctx.decision.userActions.map((a) => `  - ${sanitizeForPrompt(a)}`).join('\n') || '  (none specified)',
+        '',
+        ...buildAssertionPatternsBlock(ctx.decision.assertionPatterns),
+        'AVAILABLE PAGE OBJECTS AND METHODS:',
+        apiBlock,
+        existingBlock,
+        '',
+        'MANDATORY RULES:',
+        ...rules,
+        '',
+        ...exampleBlock,
+        '',
+        'Return ONLY the TypeScript code. No explanations, no markdown fences.',
+    ].join('\n');
+}
+function buildApiTestPrompt(ctx, profile, scenariosBlock, routeFamilyTag) {
+    const modeInstruction = ctx.mode === 'create_spec'
+        ? `Create a NEW test file at: ${ctx.specPath}`
+        : `ADD test cases to the EXISTING file at: ${ctx.specPath}`;
+    const existingBlock = ctx.existingSpecContent
+        ? `\nEXISTING FILE (extend this):\n\`\`\`typescript\n${ctx.existingSpecContent}\n\`\`\``
+        : '';
+    return [
+        `You are generating ${profile.projectName} API test code using ${profile.testFramework}.`,
+        '',
+        `TASK: ${modeInstruction}`,
+        '',
+        `FLOW: ${sanitizeForPrompt(ctx.decision.flowName)}`,
+        `Route Family: ${ctx.decision.routeFamily}${ctx.decision.featureId ? ` / ${ctx.decision.featureId}` : ''}`,
+        `Endpoint: ${ctx.decision.specificRoute || '(not specified)'}`,
+        `Priority: ${ctx.decision.priority}`,
+        `Evidence: ${sanitizeForPrompt(ctx.decision.evidence)}`,
+        '',
+        'SCENARIOS TO IMPLEMENT:',
+        scenariosBlock || '  (implement core API endpoint tests)',
+        '',
+        'USER ACTIONS:',
+        ctx.decision.userActions.map((a) => `  - ${sanitizeForPrompt(a)}`).join('\n') || '  (none specified)',
+        '',
+        ...buildAssertionPatternsBlock(ctx.decision.assertionPatterns),
+        existingBlock,
+        '',
+        'MANDATORY RULES:',
+        ...profile.conventions.map((c, i) => `${i + 1}. ${c}`),
+        `${profile.conventions.length + 1}. Tag every test: {tag: '@${routeFamilyTag}'}`,
+        '',
+        'EXAMPLE TEST STRUCTURE:',
+        ...(profile.testFramework.toLowerCase().includes('pytest')
+            ? [
+                '```python',
+                ...(profile.copyrightHeader ? [profile.copyrightHeader, ''] : []),
+                'import pytest',
+                'import requests',
+                '',
+                `BASE_URL = 'http://localhost:3000'`,
+                '',
+                '',
+                `class Test${ctx.decision.routeFamily.replace(/[^a-zA-Z0-9]/g, '')}:`,
+                "    def test_should_return_200_for_valid_request(self):",
+                `        res = requests.get(f'{BASE_URL}${ctx.decision.specificRoute || '/api/endpoint'}')`,
+                '        assert res.status_code == 200',
+                '```',
+            ]
+            : [
+                '```typescript',
+                ...(profile.copyrightHeader ? [profile.copyrightHeader, ''] : []),
+                `import {describe, it, expect} from '${profile.importStatement}';`,
+                "import supertest from 'supertest';",
+                '',
+                "const request = supertest('http://localhost:3000');",
+                '',
+                `describe('${ctx.decision.routeFamily}', () => {`,
+                "    it('should return 200 for valid request', async () => {",
+                `        const res = await request.get('${ctx.decision.specificRoute || '/api/endpoint'}');`,
+                '        expect(res.status).toBe(200);',
+                '    });',
+                '});',
+                '```',
+            ]),
+        '',
+        ...(profile.testFramework.toLowerCase().includes('pytest')
+            ? ['Return ONLY the Python code. No explanations, no markdown fences.']
+            : ['Return ONLY the TypeScript code. No explanations, no markdown fences.']),
+    ].join('\n');
+}
+export function parseGenerationResponse(text, expectedPath, mode, flowId, profile) {
+    let code = text.trim();
+    const fenced = code.match(/^```(?:typescript|ts)?\s*([\s\S]*?)```\s*$/i);
+    if (fenced) {
+        code = fenced[1].trim();
+    }
+    if (!code.includes('test(') && !code.includes('it(') && !code.includes('describe(')) {
+        return null;
+    }
+    const importStatement = profile?.importStatement || '@playwright/test';
+    // Auto-add import if missing
+    if (!code.includes(importStatement)) {
+        if (profile?.testMode === 'api') {
+            code = `import {describe, it, expect} from '${importStatement}';\n\n${code}`;
+        }
+        else {
+            code = `import {expect, test} from '${importStatement}';\n\n${code}`;
+        }
+    }
+    return { specPath: expectedPath, code, mode, flowId };
+}
+const BUILT_IN_METHODS = new Set([
+    'click', 'fill', 'focus', 'hover', 'press', 'type', 'check', 'uncheck',
+    'selectOption', 'waitFor', 'waitForLoadState', 'evaluate', 'dispatchEvent',
+    'getAttribute', 'textContent', 'innerText', 'innerHTML',
+    'isVisible', 'isEnabled', 'isChecked', 'isHidden',
+    'toBeVisible', 'toBeEnabled', 'toBeChecked', 'toBeHidden',
+    'toContainText', 'toHaveText', 'toHaveURL', 'toHaveValue',
+    'not', 'nth', 'first', 'last', 'all',
+    'getByRole', 'getByText', 'getByLabel', 'getByPlaceholder',
+    'getByTestId', 'getByTitle', 'getByAltText',
+    'locator', 'frame', 'page', 'expect',
+    'goBack', 'goForward', 'reload', 'goto',
+    'keyboard', 'mouse', 'touchscreen', 'close', 'bringToFront',
+    'initSetup', 'login', 'waitUntil', 'skipIfNoLicense', 'ensureLicense',
+    'random', 'duration', 'isOutsideRemoteUserHour', 'setTimeout',
+    'skip', 'fixme', 'slow', 'fail',
+    // API testing built-ins
+    'get', 'post', 'put', 'patch', 'delete', 'send', 'set', 'query',
+    'toBe', 'toEqual', 'toBeDefined', 'toContain', 'toHaveProperty',
+    'toMatchObject', 'toHaveLength', 'toBeTruthy', 'toBeFalsy',
+]);
+/**
+ * Returns method names that appear in generated code but do not exist in the API surface.
+ * Detects all call patterns: await X.Y(), X.Y(), const z = X.Y(), chained calls.
+ */
+export function detectHallucinatedMethods(code, apiSurface) {
+    const allMethods = new Set(apiSurface.pageObjects.flatMap((po) => po.methods.map((m) => m.name)));
+    const suspected = new Set();
+    const callPatterns = [
+        /\bawait\s+\w+\.([a-zA-Z_]\w*)\s*\(/g,
+        /\b(?:const|let|var)\s+\w+\s*=\s*\w+\.([a-zA-Z_]\w*)\s*\(/g,
+        /\b\w+Page\.([a-zA-Z_]\w*)\s*\(/g, // any *Page object (channelsPage, settingsPage, etc.)
+        /\b(?:pw|page|this)\.\w*\.?([a-zA-Z_]\w*)\s*\(/g,
+    ];
+    const generalCallRe = /\b[a-zA-Z_]\w*\.([a-zA-Z_]\w*)\s*\(/g;
+    for (const re of callPatterns) {
+        let match;
+        while ((match = re.exec(code)) !== null) {
+            const methodName = match[1];
+            if (!BUILT_IN_METHODS.has(methodName) && !allMethods.has(methodName) && methodName.length > 3) {
+                suspected.add(methodName);
+            }
+        }
+    }
+    let match;
+    while ((match = generalCallRe.exec(code)) !== null) {
+        const methodName = match[1];
+        if (!BUILT_IN_METHODS.has(methodName) &&
+            !allMethods.has(methodName) &&
+            methodName.length > 3 &&
+            !methodName.startsWith('to') &&
+            !methodName.startsWith('get')) {
+            suspected.add(methodName);
+        }
+    }
+    return [...suspected];
+}

package/dist/esm/prompts/generation_profile.js

@@ -0,0 +1,147 @@
+// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
+// See LICENSE.txt for license information.
+import { UI_FRAMEWORKS, API_FRAMEWORKS } from '../adapters/framework_adapter.js';
+const MATTERMOST_PROFILE = {
+    projectName: 'Mattermost',
+    testFramework: 'Playwright',
+    importStatement: '@mattermost/playwright-lib',
+    conventions: [
+        'Import ONLY from "@mattermost/playwright-lib" — no other test framework imports.',
+        'Every test must call `await pw.initSetup()` first.',
+        'Use `await pw.testBrowser.login(user)` to log in — never hardcode credentials.',
+        'Use `expect` from "@mattermost/playwright-lib" — do NOT import from "@playwright/test".',
+        'Include the copyright header for new files.',
+    ],
+    copyrightHeader: [
+        '// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.',
+        '// See LICENSE.txt for license information.',
+    ].join('\n'),
+    testMode: 'ui',
+};
+const DEFAULT_PLAYWRIGHT_PROFILE = {
+    projectName: 'Project',
+    testFramework: 'Playwright',
+    importStatement: '@playwright/test',
+    conventions: [
+        'Import from "@playwright/test" for test and expect.',
+        'Use page fixtures provided by Playwright.',
+        'Prefer ARIA roles and data-testid attributes for selectors.',
+        'Write one test per scenario with a descriptive name.',
+    ],
+    testMode: 'ui',
+};
+const DEFAULT_API_PROFILE = {
+    projectName: 'Project',
+    testFramework: 'vitest + supertest',
+    importStatement: 'vitest',
+    conventions: [
+        'Import from "vitest" for test and expect.',
+        'Use supertest for HTTP request assertions.',
+        'Validate response status codes, headers, and body structure.',
+        'Test both success and error paths for each endpoint.',
+    ],
+    testMode: 'api',
+};
+/**
+ * Resolves the generation profile from config and optional KG metadata.
+ * - If profile='mattermost' or Mattermost is detected, returns Mattermost profile.
+ * - If KG is present, derives project-specific profile from it.
+ * - Otherwise, returns generic Playwright profile.
+ */
+export function resolveGenerationProfile(config, kg) {
+    // Explicit Mattermost profile
+    if (config?.profile === 'mattermost') {
+        return { ...MATTERMOST_PROFILE };
+    }
+    // KG-based profile derivation
+    if (kg) {
+        const frameworks = kg.project.frameworks.map((f) => f.toLowerCase());
+        const isMattermost = kg.project.name.toLowerCase().includes('mattermost') ||
+            frameworks.includes('@mattermost/playwright-lib');
+        if (isMattermost) {
+            return { ...MATTERMOST_PROFILE };
+        }
+        const testMode = config?.testMode || deriveTestMode(frameworks);
+        const testFramework = deriveTestFramework(frameworks, testMode);
+        const importStatement = deriveImportStatement(frameworks, testMode);
+        return {
+            projectName: kg.project.name || 'Project',
+            testFramework,
+            importStatement,
+            conventions: buildConventions(testFramework, importStatement, testMode),
+            testMode,
+        };
+    }
+    // Default profiles based on test mode
+    if (config?.testMode === 'api') {
+        return { ...DEFAULT_API_PROFILE };
+    }
+    return { ...DEFAULT_PLAYWRIGHT_PROFILE };
+}
+/**
+ * Checks if a profile is the Mattermost profile (for backward compatibility checks).
+ */
+export function isMattermostProfile(profile) {
+    return profile.importStatement === '@mattermost/playwright-lib';
+}
+// ---------------------------------------------------------------------------
+// Internal helpers
+// ---------------------------------------------------------------------------
+function deriveTestMode(frameworks) {
+    const uiSet = new Set(UI_FRAMEWORKS);
+    const apiSet = new Set(API_FRAMEWORKS);
+    const hasUiFramework = frameworks.some((f) => uiSet.has(f));
+    const hasApiFramework = frameworks.some((f) => apiSet.has(f));
+    if (hasUiFramework && hasApiFramework)
+        return 'both';
+    if (hasApiFramework && !hasUiFramework)
+        return 'api';
+    return 'ui';
+}
+function deriveTestFramework(frameworks, testMode) {
+    if (testMode === 'api') {
+        if (frameworks.includes('pytest'))
+            return 'pytest';
+        if (frameworks.includes('jest'))
+            return 'jest + supertest';
+        return 'vitest + supertest';
+    }
+    if (frameworks.includes('cypress'))
+        return 'Cypress';
+    if (frameworks.includes('selenium'))
+        return 'Selenium';
+    return 'Playwright';
+}
+function deriveImportStatement(frameworks, testMode) {
+    if (testMode === 'api') {
+        if (frameworks.includes('pytest'))
+            return 'pytest';
+        if (frameworks.includes('jest'))
+            return 'jest';
+        return 'vitest';
+    }
+    if (frameworks.includes('cypress'))
+        return 'cypress';
+    return '@playwright/test';
+}
+function buildConventions(testFramework, importStatement, testMode) {
+    const conventions = [];
+    if (testMode === 'api' || testMode === 'both') {
+        conventions.push(`Import from "${importStatement}" for test and expect.`);
+        conventions.push('Validate response status codes, headers, and body structure.');
+        conventions.push('Test both success and error paths for each endpoint.');
+    }
+    if (testMode === 'ui' || testMode === 'both') {
+        if (testFramework.includes('Playwright')) {
+            conventions.push('Import from "@playwright/test" for test and expect.');
+            conventions.push('Use page fixtures provided by Playwright.');
+        }
+        else if (testFramework.includes('Cypress')) {
+            conventions.push('Use cy.* commands for browser interaction.');
+        }
+        conventions.push('Prefer ARIA roles and data-testid attributes for selectors.');
+    }
+    conventions.push('Write one test per scenario with a descriptive name of what the user does and what is verified.');
+    conventions.push('NEVER fabricate test IDs. Use descriptive names only.');
+    return conventions;
+}

package/dist/esm/prompts/heal.js

@@ -0,0 +1,91 @@
+// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
+// See LICENSE.txt for license information.
+import { sanitizeForPrompt } from '../crew/sanitize.js';
+import { isMattermostProfile } from './generation_profile.js';
+/**
+ * Builds a route-family-aware heal prompt for the playwright-test-healer agent.
+ * Enriches the base healer constraints with flow context so the agent understands
+ * what the test is supposed to verify, reducing hallucination during selector repair.
+ */
+export function buildHealPrompt(ctx) {
+    const flowBlock = ctx.decision
+        ? [
+            '',
+            'FLOW CONTEXT (use to understand test intent — do not change test objectives):',
+            `  Flow: ${sanitizeForPrompt(ctx.decision.flowName)}`,
+            `  Route Family: ${ctx.decision.routeFamily}${ctx.decision.featureId ? ` / ${ctx.decision.featureId}` : ''}`,
+            `  Route: ${ctx.decision.specificRoute || '(family-level)'}`,
+            `  User Actions: ${sanitizeForPrompt(ctx.decision.userActions.join('; ')) || 'not specified'}`,
+            `  Evidence: ${sanitizeForPrompt(ctx.decision.evidence)}`,
+        ].join('\n')
+        : '';
+    const statusNote = ctx.status === 'flaky'
+        ? 'This test is FLAKY (passes sometimes, fails other times). Look for race conditions, missing waits, or order-dependent state.'
+        : 'This test is FAILING consistently. The selector, URL, or API call is likely broken.';
+    const failureBlock = ctx.failureDetail
+        ? `\nFailure detail:\n${sanitizeForPrompt(ctx.failureDetail)}`
+        : '';
+    const consoleBlock = ctx.consoleErrors && ctx.consoleErrors.length > 0
+        ? `\nRecent console errors from test run:\n${ctx.consoleErrors.slice(-3).map((e) => `  - ${sanitizeForPrompt(e)}`).join('\n')}`
+        : '';
+    const importLib = ctx.profile?.importStatement || '@playwright/test';
+    const isMM = ctx.profile ? isMattermostProfile(ctx.profile) : false;
+    const projectLabel = ctx.profile?.projectName || 'Project';
+    const frameworkLabel = ctx.profile?.testFramework || 'Playwright';
+    const constraints = isMM
+        ? [
+            `- Import ONLY from "${importLib}". Do not use "@playwright/test" directly.`,
+            '- Do not use test.describe or test.only.',
+            '- Keep a single tag string matching the route family (e.g. "@channels", "@scheduled_posts").',
+            `- Use only existing ${projectLabel} ${frameworkLabel} fixture and page-object APIs.`,
+            '- Do NOT invent new pw.* clients or page object methods that do not exist.',
+            '- Avoid brittle class selectors (.backstage-navbar, .admin-console__wrapper, .left-panel, .panel-card).',
+        ]
+        : [
+            `- Import from "${importLib}".`,
+            '- Keep a single tag string matching the route family.',
+            `- Use only existing ${projectLabel} ${frameworkLabel} page-object APIs.`,
+            '- Do NOT invent page object methods that do not exist.',
+        ];
+    return [
+        `Heal this specific ${frameworkLabel} test file and keep edits minimal.`,
+        '',
+        `Target test file: ${ctx.specPath}`,
+        `Status: ${ctx.status.toUpperCase()} — ${statusNote}`,
+        failureBlock,
+        consoleBlock,
+        flowBlock,
+        '',
+        'Healing constraints (must follow):',
+        ...constraints,
+        '- Prefer stable assertions using URL patterns, data-testid attributes, ARIA roles, and page-object methods.',
+        '- For flaky tests: add explicit waits (waitFor, expect().toBeVisible()) before interactions.',
+        '- Keep the test intent and scenario unchanged — only fix what is broken.',
+        '- If behavior is genuinely broken server-side, mark test.fixme with a clear comment explaining why.',
+        '',
+        'Run and fix this test until it compiles and passes, or mark test.fixme when the behavior is truly broken.',
+    ].filter((line) => line !== null).join('\n');
+}
+/**
+ * Builds a minimal quality-fix prompt for spec files that fail content validation
+ * (e.g. contain test.describe, test.only, wrong imports).
+ */
+export function buildQualityFixPrompt(specPath, qualityIssues, profile) {
+    const importLib = profile?.importStatement || '@playwright/test';
+    const frameworkLabel = profile?.testFramework || 'Playwright';
+    return [
+        `Fix quality issues in this ${frameworkLabel} spec file. Make minimal edits only.`,
+        '',
+        `Target file: ${specPath}`,
+        '',
+        'Issues to fix:',
+        ...qualityIssues.map((issue) => `  - ${issue}`),
+        '',
+        'Rules:',
+        `- Import only from "${importLib}".`,
+        '- Remove test.describe wrappers (flatten to top-level test() calls).',
+        '- Remove test.only calls.',
+        '- Ensure each test has exactly one tag string.',
+        '- Do not change test logic — only fix structural quality issues.',
+    ].join('\n');
+}

package/dist/esm/prompts/impact.js

@@ -0,0 +1,63 @@
+// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
+// See LICENSE.txt for license information.
+import { extractJsonFromResponse } from './json_extract.js';
+import { formatSpecsForPrompt } from '../knowledge/spec_index.js';
+import { formatApiSurfaceForPrompt } from '../knowledge/api_surface.js';
+export function buildImpactPrompt(ctx) {
+    const familyRoutes = ctx.family.routes.join(', ');
+    const featureNote = ctx.featureId ? `\nFEATURE: ${ctx.featureId}` : '';
+    const pageObjectNames = ctx.family.pageObjects || [];
+    const componentNames = ctx.family.components || [];
+    const allClassNames = [...pageObjectNames, ...componentNames];
+    const apiSurfaceBlock = allClassNames.length > 0
+        ? formatApiSurfaceForPrompt(ctx.apiSurface, allClassNames)
+        : 'No page objects or components mapped for this family.';
+    const specsBlock = ctx.existingSpecs.length > 0
+        ? formatSpecsForPrompt(ctx.existingSpecs)
+        : 'No existing specs found for this route family.';
+    const changedFilesBlock = ctx.changedFiles
+        .map((f) => {
+        if (f.snippet) {
+            return `${f.path}:\n\`\`\`\n${f.snippet}\n\`\`\``;
+        }
+        return f.path;
+    })
+        .join('\n\n');
+    return [
+        `You are analyzing code changes in ${ctx.projectName || 'the project'} to identify impacted user-facing flows.`,
+        '',
+        `ROUTE FAMILY: ${ctx.family.id}`,
+        `ROUTES: ${familyRoutes}`,
+        featureNote,
+        '',
+        `PAGE OBJECTS AND COMPONENTS:`,
+        apiSurfaceBlock,
+        '',
+        `EXISTING SPECS FOR THIS FAMILY (${ctx.existingSpecs.length}):`,
+        specsBlock,
+        '',
+        `CHANGED FILES (${ctx.changedFiles.length}):`,
+        changedFilesBlock,
+        '',
+        ctx.contextBlock,
+        '',
+        'For each changed file, identify impacted user-facing flows.',
+        '',
+        'Return strict JSON only with this shape:',
+        '{"flows":[{"id":"<flow_id>","name":"<human readable name>","route":"<specific route from ROUTES>","userActions":["<what the user does>"],"priority":"P0|P1|P2","confidence":0-100,"evidence":"<why this flow is impacted>","pageObjects":["<page object used>"],"changedFiles":["<files>"]}]}',
+        '',
+        'Rules:',
+        '- ONLY use routes listed in ROUTES above.',
+        '- ONLY reference page objects and components listed above.',
+        '- Each flow must describe a specific user action, not a generic category.',
+        '- If you cannot determine the impacted flow with high confidence, return:',
+        '  {"id":"unknown","name":"cannot determine","confidence":0,"evidence":"<reason>","userActions":[],"changedFiles":["<files>"]}',
+        '- Do NOT default to /admin_console/reporting/system_analytics unless the changed files are literally analytics code.',
+        '- Do NOT invent routes, page objects, or methods that are not listed above.',
+        '- Keep at most 8 flows.',
+        '- Prioritize true user-impacting flows; avoid low-value internal buckets.',
+    ].filter(Boolean).join('\n');
+}
+export function parseImpactResponse(text) {
+    return extractJsonFromResponse(text, (obj) => obj != null && typeof obj === 'object' && Array.isArray(obj.flows));
+}

package/dist/esm/prompts/json_extract.js

@@ -0,0 +1,36 @@
+// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
+// See LICENSE.txt for license information.
+/**
+ * Shared JSON extraction from LLM text responses.
+ * Handles fenced code blocks, bare JSON, and partial text.
+ */
+/**
+ * Extract and parse JSON from LLM response text.
+ * Tries fenced code blocks first, then raw text.
+ * Returns null if no valid JSON found.
+ *
+ * @param text - Raw LLM response text
+ * @param validate - Predicate to check if parsed object has the expected shape
+ */
+export function extractJsonFromResponse(text, validate) {
+    const fenced = text.match(/```(?:json)?\s*([\s\S]*?)```/i);
+    const candidates = fenced ? [fenced[1], text] : [text];
+    for (const candidate of candidates) {
+        const start = candidate.indexOf('{');
+        const end = candidate.lastIndexOf('}');
+        if (start < 0 || end <= start) {
+            continue;
+        }
+        const raw = candidate.slice(start, end + 1);
+        try {
+            const parsed = JSON.parse(raw);
+            if (validate(parsed)) {
+                return parsed;
+            }
+        }
+        catch {
+            continue;
+        }
+    }
+    return null;
+}