@skyramp/mcp 0.2.1 → 0.2.3

package/build/playwright/registerPlaywrightTools.js

@@ -40,13 +40,23 @@ export async function registerPlaywrightTools(server, options) {
         'browser_select_option',
         'browser_hover',
         'browser_drag',
+        'browser_mouse_move_xy',
+        'browser_mouse_click_xy',
+        'browser_mouse_drag_xy',
+        'browser_mouse_down',
+        'browser_mouse_up',
+        'browser_file_upload',
+        'browser_evaluate',
         'browser_tabs',
         'browser_navigate_back',
         'browser_wait_for',
         'browser_take_screenshot',
         'browser_assert',
         'browser_assert_api_request',
+        'browser_assert_table_cell',
         'skyramp_export_zip',
+        'skyramp_load_trace',
+        'browser_mouse_action',
         // DOM Analyzer tools (Phase C)
         'browser_blueprint',
         'browser_blueprint_diff',

package/build/prompts/test-recommendation/scopeAssessment.js

@@ -1,24 +1,125 @@
+import * as fs from "fs";
+import * as path from "path";
 import { MAX_RECOMMENDATIONS, MAX_TESTS_TO_GENERATE } from "./recommendationSections.js";
 // .vue and .svelte cannot be route handlers in any framework — always frontend.
 const ALWAYS_FRONTEND_EXT = /\.(vue|svelte)$/i;
+// .dart is always frontend in Flutter projects (no server-side Dart web framework
+// in scope). Gated on hasFlutterPubspec so non-Flutter Dart files (rare, but
+// possible — e.g. Dart-on-server backends) don't accidentally classify as UI.
+const FLUTTER_DART_EXT = /\.dart$/i;
 // .tsx/.jsx are usually UI components but can be Next.js API handlers (pages/api/*.tsx).
 const LIKELY_FRONTEND_EXT = /\.(tsx|jsx)$/i;
 // Ambiguous extensions need directory context to distinguish frontend from backend.
 const AMBIGUOUS_FRONTEND_PATTERN = /\.(tsx?|jsx?|css|scss|less|html?|erb|jsp|asp|jinja2?|twig)$/i;
 const API_DIR_PATTERN = /\/(api|routes?|controllers?|routers?|handlers?|endpoints?|server)\//;
 const FRONTEND_DIR_PATTERN = /(^|\/)(components?|pages?|views?|layouts?|app|src\/app|frontend|client|public|styles?|templates?)\//i;
+/**
+ * Returns true if the repository root's `pubspec.yaml` declares a Flutter
+ * SDK dependency (i.e., a `flutter:` key with `sdk: flutter` under
+ * `dependencies:`). This is the load-bearing signal that the project
+ * is a Flutter UI app or package, NOT a pure Dart server / CLI / library
+ * (those have a `pubspec.yaml` too but no `sdk: flutter` dep).
+ *
+ * Used as the gate for treating `.dart` files as frontend in
+ * `isFrontendFile`. Pure-Dart-backend repos correctly fall through to
+ * the existing classifier behaviour (no `.dart` recognition).
+ *
+ * Synchronous + best-effort: returns false on any read/parse error.
+ * Cheap because pubspec.yaml is small and we only call this once per
+ * tool invocation (computed at the budget-driving boundary, then passed
+ * through as an option).
+ */
+/**
+ * Common subdir locations for `pubspec.yaml` in real Flutter repos:
+ *   - root (single-app — birdle, customer SPA)
+ *   - app/, mobile/, frontend/, web/ (mixed-stack repos)
+ *   - apps/<name>/, packages/<name>/ (monorepos — depth-2 walked)
+ *
+ * We stop at the first pubspec we find that declares `sdk: flutter`. Pure-Dart
+ * subprojects (e.g. a CLI tool in `tools/`) are correctly skipped because their
+ * pubspec doesn't have the Flutter SDK line.
+ *
+ * Less-common shapes that are NOT auto-discovered: `services/mobile/`,
+ * `clients/<name>/`, etc. If a customer's repo uses one of those, add the
+ * parent dir here or tell them to symlink `pubspec.yaml` at the root.
+ */
+const PUBSPEC_SEARCH_DIRS = [
+    ".",
+    "app",
+    "mobile",
+    "frontend",
+    "web",
+    "client",
+];
+const PUBSPEC_MONOREPO_PARENTS = ["apps", "packages"];
+export function hasFlutterSdkDep(repositoryPath) {
+    for (const subdir of PUBSPEC_SEARCH_DIRS) {
+        if (checkPubspec(path.join(repositoryPath, subdir, "pubspec.yaml"))) {
+            return true;
+        }
+    }
+    // Monorepo shape: apps/<name>/pubspec.yaml, packages/<name>/pubspec.yaml.
+    // One level of fan-out — enough for typical Flutter monorepos, cheap to walk.
+    for (const parent of PUBSPEC_MONOREPO_PARENTS) {
+        const parentPath = path.join(repositoryPath, parent);
+        let entries;
+        try {
+            entries = fs.readdirSync(parentPath, { withFileTypes: true });
+        }
+        catch {
+            continue;
+        }
+        for (const entry of entries) {
+            if (!entry.isDirectory())
+                continue;
+            if (checkPubspec(path.join(parentPath, entry.name, "pubspec.yaml"))) {
+                return true;
+            }
+        }
+    }
+    return false;
+}
+function checkPubspec(pubspecPath) {
+    try {
+        if (!fs.existsSync(pubspecPath))
+            return false;
+        const content = fs.readFileSync(pubspecPath, "utf8");
+        // Detect the `flutter:` SDK dep — proper YAML parsing would be overkill.
+        // The canonical pattern is:
+        //   dependencies:
+        //     flutter:
+        //       sdk: flutter
+        // The `sdk: flutter` line under a `flutter:` key is unique to Flutter
+        // projects; pure-Dart projects use `sdk: dart` or no sdk: line at all.
+        return /^\s+sdk:\s*flutter\s*$/m.test(content);
+    }
+    catch {
+        return false;
+    }
+}
 /**
  * Returns true if the file path is a frontend file.
  *
- * Three tiers:
+ * Four tiers:
  * 1. .vue / .svelte — always frontend; these file types cannot be route handlers.
- * 2. .tsx / .jsx — frontend unless in an API/backend directory (e.g. pages/api/).
- * 3. Ambiguous extensions (.ts, .js, .css, .html, …) — require a recognised
+ * 2. .dart — always frontend IFF `hasFlutterSdkDep` is true (Flutter project).
+ *    Sits in tier 1 because there's no server-side Dart web framework in scope
+ *    when the Flutter SDK dep is present, so `API_DIR_PATTERN` doesn't apply.
+ * 3. .tsx / .jsx — frontend unless in an API/backend directory (e.g. pages/api/).
+ * 4. Ambiguous extensions (.ts, .js, .css, .html, …) — require a recognised
  *    frontend directory AND must not be in an API/backend directory.
+ *
+ * The `hasFlutterSdkDep` opt is passed in by the budget-driving callers
+ * (uiAnalyzeChangesTool, analyzeChangesTool) which have repository context.
+ * Pure-path callers (test-recommendation-prompt, analysisOutputPrompt,
+ * scopeAssessment's own buildScopeAssessmentSection) default to false —
+ * they don't drive the budget.
  */
-export function isFrontendFile(filePath) {
+export function isFrontendFile(filePath, { hasFlutterSdkDep = false } = {}) {
     if (ALWAYS_FRONTEND_EXT.test(filePath))
         return true;
+    if (hasFlutterSdkDep && FLUTTER_DART_EXT.test(filePath))
+        return true;
     if (API_DIR_PATTERN.test(filePath))
         return false;
     if (LIKELY_FRONTEND_EXT.test(filePath))
@@ -113,7 +214,7 @@ Read the Changed Files list and endpoint changes above, then work through the fo
 
 **Step A — Classify changed files:**
 Count each type from the diff context (ignore generated test files, lock files, and build artifacts):
-- **Frontend files**: .vue / .svelte anywhere (always UI components). .tsx / .jsx anywhere except in api/, routes/, routers/, controllers/, handlers/, endpoints/, or server/ directories. .ts / .js / .html / .css / .scss / .less / .erb / .jsp / .asp / .jinja2 / .twig only when in a frontend directory (components/, pages/, views/, layouts/, app/, frontend/, client/, styles/, templates/).
+- **Frontend files**: .vue / .svelte anywhere (always UI components). .dart anywhere (always UI in a Flutter project — repo has a \`pubspec.yaml\` with \`sdk: flutter\`). .tsx / .jsx anywhere except in api/, routes/, routers/, controllers/, handlers/, endpoints/, or server/ directories. .ts / .js / .html / .css / .scss / .less / .erb / .jsp / .asp / .jinja2 / .twig only when in a frontend directory (components/, pages/, views/, layouts/, app/, frontend/, client/, styles/, templates/).
 - **Backend files**: route handlers, controllers, services, models, API modules, middleware, config with business logic
 - **Non-application** (exclude from test value): CSS-only, copy/string changes, README, CI config with no logic
 

package/build/prompts/test-recommendation/scopeAssessment.test.js

@@ -1,7 +1,10 @@
 jest.mock("@skyramp/skyramp", () => ({
     WorkspaceConfigManager: { create: jest.fn() },
 }));
-import { isFrontendFile, isTestFile, buildScopeAssessmentSection } from "./scopeAssessment.js";
+import * as fs from "fs";
+import * as path from "path";
+import * as os from "os";
+import { hasFlutterSdkDep, isFrontendFile, isTestFile, buildScopeAssessmentSection } from "./scopeAssessment.js";
 // ---------------------------------------------------------------------------
 // isFrontendFile
 // ---------------------------------------------------------------------------
@@ -56,6 +59,130 @@ describe("isFrontendFile", () => {
     it("returns false for a plain TS API client in utils/ (no frontend dir)", () => {
         expect(isFrontendFile("utils/apiClient.ts")).toBe(false);
     });
+    // Flutter / Dart support — gated on hasFlutterSdkDep.
+    // Without the flag, .dart is unrecognised (legacy behaviour). With it,
+    // .dart is always-frontend (tier 2, before API_DIR_PATTERN).
+    describe(".dart files (Flutter)", () => {
+        it("returns false for .dart without hasFlutterSdkDep (default)", () => {
+            expect(isFrontendFile("lib/main.dart")).toBe(false);
+        });
+        it("returns false for .dart with hasFlutterSdkDep: false", () => {
+            expect(isFrontendFile("lib/main.dart", { hasFlutterSdkDep: false })).toBe(false);
+        });
+        it("returns true for .dart with hasFlutterSdkDep: true", () => {
+            expect(isFrontendFile("lib/main.dart", { hasFlutterSdkDep: true })).toBe(true);
+        });
+        it("returns true for nested .dart files in a Flutter project", () => {
+            expect(isFrontendFile("lib/widgets/game_board.dart", { hasFlutterSdkDep: true })).toBe(true);
+        });
+        it("returns true for .dart even in a directory that would otherwise look API-like (Flutter has no server-side Dart in scope)", () => {
+            // .dart sits in tier 2 (above API_DIR_PATTERN) so it's frontend even
+            // under api/ — there's no server-side Dart web framework that would
+            // make this ambiguous in the Flutter case.
+            expect(isFrontendFile("lib/api/client.dart", { hasFlutterSdkDep: true })).toBe(true);
+        });
+        it("returns true for game.dart at repo root in a Flutter project", () => {
+            expect(isFrontendFile("game.dart", { hasFlutterSdkDep: true })).toBe(true);
+        });
+    });
+});
+// ---------------------------------------------------------------------------
+// hasFlutterSdkDep
+// ---------------------------------------------------------------------------
+describe("hasFlutterSdkDep", () => {
+    let tmpDir;
+    beforeEach(() => {
+        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "flutter-pubspec-"));
+    });
+    afterEach(() => {
+        fs.rmSync(tmpDir, { recursive: true, force: true });
+    });
+    it("returns false when pubspec.yaml is absent", () => {
+        expect(hasFlutterSdkDep(tmpDir)).toBe(false);
+    });
+    it("returns true when pubspec.yaml has the canonical Flutter SDK dep", () => {
+        fs.writeFileSync(path.join(tmpDir, "pubspec.yaml"), `name: birdle
+description: A Flutter app.
+dependencies:
+  flutter:
+    sdk: flutter
+  cupertino_icons: ^1.0.6
+`);
+        expect(hasFlutterSdkDep(tmpDir)).toBe(true);
+    });
+    it("returns false for a pure Dart project (no flutter SDK dep)", () => {
+        // Pure Dart server (e.g. shelf, dart_frog) has pubspec.yaml but no flutter SDK
+        fs.writeFileSync(path.join(tmpDir, "pubspec.yaml"), `name: my_dart_server
+description: A Dart server using shelf.
+environment:
+  sdk: ">=3.0.0 <4.0.0"
+dependencies:
+  shelf: ^1.4.0
+  shelf_router: ^1.1.4
+`);
+        expect(hasFlutterSdkDep(tmpDir)).toBe(false);
+    });
+    it("returns false for a Dart CLI tool (no flutter SDK dep)", () => {
+        fs.writeFileSync(path.join(tmpDir, "pubspec.yaml"), `name: my_cli
+description: A Dart CLI.
+environment:
+  sdk: ^3.5.0
+dependencies:
+  args: ^2.5.0
+`);
+        expect(hasFlutterSdkDep(tmpDir)).toBe(false);
+    });
+    it("returns false on a malformed pubspec.yaml (graceful failure)", () => {
+        fs.writeFileSync(path.join(tmpDir, "pubspec.yaml"), "::: not yaml ::: \x00");
+        // Our regex-based check still works on string content; this is more of
+        // a "doesn't throw" test. Returns false because the malformed content
+        // doesn't match the sdk: flutter pattern.
+        expect(hasFlutterSdkDep(tmpDir)).toBe(false);
+    });
+    it("returns true even with extra whitespace and comments in pubspec.yaml", () => {
+        fs.writeFileSync(path.join(tmpDir, "pubspec.yaml"), `name: birdle
+# This is the SDK dep
+dependencies:
+  flutter:
+    sdk:    flutter
+`);
+        expect(hasFlutterSdkDep(tmpDir)).toBe(true);
+    });
+    // Subdir search — common shapes in real Flutter repos.
+    it("returns true when pubspec.yaml lives in app/", () => {
+        fs.mkdirSync(path.join(tmpDir, "app"));
+        fs.writeFileSync(path.join(tmpDir, "app", "pubspec.yaml"), "name: app\ndependencies:\n  flutter:\n    sdk: flutter\n");
+        expect(hasFlutterSdkDep(tmpDir)).toBe(true);
+    });
+    it("returns true when pubspec.yaml lives in mobile/", () => {
+        fs.mkdirSync(path.join(tmpDir, "mobile"));
+        fs.writeFileSync(path.join(tmpDir, "mobile", "pubspec.yaml"), "name: app\ndependencies:\n  flutter:\n    sdk: flutter\n");
+        expect(hasFlutterSdkDep(tmpDir)).toBe(true);
+    });
+    it("returns true for monorepo with apps/<name>/pubspec.yaml", () => {
+        fs.mkdirSync(path.join(tmpDir, "apps", "customer-app"), { recursive: true });
+        fs.writeFileSync(path.join(tmpDir, "apps", "customer-app", "pubspec.yaml"), "name: customer_app\ndependencies:\n  flutter:\n    sdk: flutter\n");
+        expect(hasFlutterSdkDep(tmpDir)).toBe(true);
+    });
+    it("returns true for monorepo with packages/<name>/pubspec.yaml", () => {
+        fs.mkdirSync(path.join(tmpDir, "packages", "ui"), { recursive: true });
+        fs.writeFileSync(path.join(tmpDir, "packages", "ui", "pubspec.yaml"), "name: ui\ndependencies:\n  flutter:\n    sdk: flutter\n");
+        expect(hasFlutterSdkDep(tmpDir)).toBe(true);
+    });
+    it("returns false when monorepo subdirs only contain pure-Dart packages", () => {
+        // Mixed monorepo: a Dart CLI in packages/cli/, no Flutter app anywhere.
+        fs.mkdirSync(path.join(tmpDir, "packages", "cli"), { recursive: true });
+        fs.writeFileSync(path.join(tmpDir, "packages", "cli", "pubspec.yaml"), "name: cli\ndependencies:\n  args: ^2.5.0\n");
+        expect(hasFlutterSdkDep(tmpDir)).toBe(false);
+    });
+    it("does not walk arbitrary subdirs (e.g. node_modules-style nested deps)", () => {
+        // A pubspec deep under an unsupported subdir should NOT be picked up.
+        // Without this guard, a transitive dependency's pubspec could falsely
+        // flag a non-Flutter repo as Flutter.
+        fs.mkdirSync(path.join(tmpDir, "vendor", "nested", "lib"), { recursive: true });
+        fs.writeFileSync(path.join(tmpDir, "vendor", "nested", "lib", "pubspec.yaml"), "name: nested\ndependencies:\n  flutter:\n    sdk: flutter\n");
+        expect(hasFlutterSdkDep(tmpDir)).toBe(false);
+    });
 });
 // ---------------------------------------------------------------------------
 // isTestFile

package/build/prompts/testbot/testbot-prompts.js

@@ -127,7 +127,7 @@ ${userPrompt ? "Generate only the tests that the user requested from the Additio
   Keep advancing until you have created exactly ${maxGenerate} new test files OR exhausted all candidates.
 - Example: If enrichment reveals that sending \`discount_value\` without \`discount_type\` silently orphans the value (a concrete bug), complete all planned GENERATE items first, then generate this discovered scenario as an extra test and report it in \`newTestsCreated\`.
 - Total generated: Follow the "Budget: N generate" line in the Execution Plan. Process every GENERATE-tagged item in order. Backfill from ADDITIONAL candidates (highest-ranked first) until \`newTestsCreated\` reaches ${maxGenerate} or all candidates are exhausted.
-- **UI test priority**: If the diff contains frontend/UI changes (e.g. \`.tsx\`, \`.jsx\`, \`.vue\`, \`.svelte\` files), you MUST attempt to generate at least one UI test. Use \`browser_navigate\` to the app's base URL — if the app responds, record a trace and generate the test.
+- **UI test priority**: If the PR scope assessment shows any UI/E2E budget OR \`uiContext.changedFrontendFiles\` is non-empty (the deterministic server signal — populated for all supported frontend file types including \`.tsx\`/\`.jsx\`/\`.vue\`/\`.svelte\`/\`.dart\`), you MUST attempt to generate at least one UI test. Use \`browser_navigate\` to the app's base URL — if the app responds, record a trace and generate the test.
   **Skip only if one of these conditions is met:**
   - **(a) App is unreachable** — \`browser_navigate\` fails or connection is refused.
   - **(b) Unintegrated non-route component** — the changed file is a leaf component (not a framework route/entrypoint) that has no integration point in the running app. To confirm:
@@ -273,7 +273,7 @@ If a test **generation** tool call fails:
 1. **Retry once** with the same parameters.
 2. If it fails again, **skip** that candidate and move to the next ranked candidate.
 3. If all candidates in the GENERATE set fail, fall back to generating the **simplest possible test**: a single contract test for the highest-scored endpoint (GET → 200 or POST → 201).
-   **Exception — frontend-only PRs**: If the diff modifies ONLY frontend files (\`.tsx\`, \`.jsx\`, \`.vue\`, \`.svelte\`, \`.css\`, \`.html\`) AND browser recording was not possible, do NOT generate a backend fallback contract test — it is irrelevant to the PR. Instead move ALL GENERATE candidates to \`additionalRecommendations\` and proceed to Task 3.
+   **Exception — frontend-only PRs**: If the diff modifies ONLY frontend files (\`.tsx\`, \`.jsx\`, \`.vue\`, \`.svelte\`, \`.dart\`, \`.css\`, \`.html\`) AND browser recording was not possible, do NOT generate a backend fallback contract test — it is irrelevant to the PR. Instead move ALL GENERATE candidates to \`additionalRecommendations\` and proceed to Task 3.
 4. Log skipped candidates in \`issuesFound\` with the error message.
 
 If a test **execution** (\`skyramp_execute_test\`) fails for a newly generated test:

package/build/prompts/testbot/testbot-prompts.test.js

@@ -360,4 +360,25 @@ describe("testbot prompt blueprint-grounded recommendations (slice 4)", () => {
         // Make sure we removed the old capturedBlueprints threading directive.
         expect(prompt).not.toMatch(/capturedBlueprints/);
     });
+    // Flutter support — both the generalised UI trigger wording and the
+    // canvas/empty-ARIA issuesFound rule should appear in the prompt.
+    it("UI test priority defers to the deterministic server signal (uiContext.changedFrontendFiles), not just hard-coded extensions", () => {
+        const prompt = getTestbotPrompt(baseArgs.prTitle, baseArgs.prDescription, baseArgs.summaryOutputFile, baseArgs.repositoryPath);
+        // The trigger must reference the server signal (changedFrontendFiles)
+        // — that's what makes it framework-agnostic. Hard-coded extension lists
+        // become illustrative, not gating.
+        expect(prompt).toMatch(/uiContext.*changedFrontendFiles|changedFrontendFiles.*uiContext/);
+        // .dart should appear in the supported-types example list
+        expect(prompt).toMatch(/\.dart/);
+    });
+    it("frontend-only PR exception lists .dart alongside other frontend extensions", () => {
+        const prompt = getTestbotPrompt(baseArgs.prTitle, baseArgs.prDescription, baseArgs.summaryOutputFile, baseArgs.repositoryPath);
+        // The frontend-only-PR exception block (Task 2 backend-fallback skip)
+        // must include .dart so a Flutter-only PR doesn't emit a backend
+        // contract test as a fallback.
+        const exceptionBlock = prompt.slice(prompt.indexOf("Exception — frontend-only PRs"), prompt.indexOf("Exception — frontend-only PRs") + 800);
+        expect(exceptionBlock).toContain(".dart");
+        expect(exceptionBlock).toContain(".tsx");
+        expect(exceptionBlock).toContain(".vue");
+    });
 });

package/build/tools/test-management/analyzeChangesTool.js

@@ -10,7 +10,7 @@ import { parseWorkspaceAuthType, getDefaultAuthHeader, WorkspaceAuthType, readWo
 import { AnalyticsService } from "../../services/AnalyticsService.js";
 import { StateManager, registerSession, storeSessionData, setTestsRepoDir, } from "../../utils/AnalysisStateManager.js";
 import { buildRecommendationPrompt } from "../../prompts/test-recommendation/test-recommendation-prompt.js";
-import { isFrontendFile, isTestFile } from "../../prompts/test-recommendation/scopeAssessment.js";
+import { hasFlutterSdkDep, isFrontendFile, isTestFile } from "../../prompts/test-recommendation/scopeAssessment.js";
 import { enumerateCandidateUiPages } from "../../utils/uiPageEnumerator.js";
 import { MAX_RECOMMENDATIONS, MAX_TESTS_TO_GENERATE } from "../../prompts/test-recommendation/recommendationSections.js";
 import { TestDiscoveryService } from "../../services/TestDiscoveryService.js";
@@ -1123,7 +1123,13 @@ Combines API endpoint scanning, branch diff computation, and test discovery into
                 // pass isFrontendFile (any .ts under a frontend directory matches
                 // the tier-3 rule) but aren't UI source we'd want to ground page
                 // enumeration in.
-                const frontendFiles = changedFiles.filter((f) => isFrontendFile(f) && !isTestFile(f));
+                // Compute hasFlutterSdkDep once and pass it through so .dart files
+                // in a Flutter project are recognised as frontend. See Confluence
+                // "Flutter support in Testbot" — this is the second budget-driving
+                // call site that must thread the flag (the other is
+                // uiAnalyzeChangesTool). Both must agree to avoid silent divergence.
+                const flutterSdk = hasFlutterSdkDep(params.repositoryPath);
+                const frontendFiles = changedFiles.filter((f) => isFrontendFile(f, { hasFlutterSdkDep: flutterSdk }) && !isTestFile(f));
                 if (frontendFiles.length === 0)
                     return undefined;
                 const candidateUiPages = await enumerateCandidateUiPages(params.repositoryPath, frontendFiles);

package/build/tools/test-management/uiAnalyzeChangesTool.js

@@ -3,7 +3,7 @@ import * as path from "path";
 import { z } from "zod";
 import { logger } from "../../utils/logger.js";
 import { enumerateCandidateUiPages } from "../../utils/uiPageEnumerator.js";
-import { isFrontendFile, isTestFile } from "../../prompts/test-recommendation/scopeAssessment.js";
+import { hasFlutterSdkDep, isFrontendFile, isTestFile } from "../../prompts/test-recommendation/scopeAssessment.js";
 import { parseChangedFilesFromDiff } from "../../utils/branchDiff.js";
 import { toolText } from "../../utils/utils.js";
 import { isTestbotEnabled } from "../../utils/featureFlags.js";
@@ -49,7 +49,13 @@ export async function runUiAnalyzeChanges(params) {
             instructions: DIFF_FILE_MISSING_INSTRUCTIONS,
         };
     }
-    const frontendFiles = changedFiles.filter((f) => isFrontendFile(f) && !isTestFile(f));
+    // Compute hasFlutterSdkDep once at the tool boundary; pass into isFrontendFile
+    // so .dart files are recognised as frontend in Flutter repos. See Confluence
+    // "Flutter support in Testbot" — this is one of two budget-driving call sites
+    // that must thread the flag (the other is analyzeChangesTool). Without it,
+    // a Flutter PR shows zero frontend files and never enters the UI pipeline.
+    const flutterSdk = hasFlutterSdkDep(repoPath);
+    const frontendFiles = changedFiles.filter((f) => isFrontendFile(f, { hasFlutterSdkDep: flutterSdk }) && !isTestFile(f));
     if (frontendFiles.length === 0) {
         const uiContext = {
             changedFrontendFiles: [],

package/build/tools/test-management/uiAnalyzeChangesTool.test.js

@@ -97,4 +97,51 @@ describe("runUiAnalyzeChanges", () => {
         expect(result.instructions).toMatch(/source-grounded/i);
         fs.rmSync(repo, { recursive: true, force: true });
     });
+    // Flutter support — the headline case the Confluence plan validates.
+    // Without the .dart classifier change, a Flutter PR returns
+    // changedFrontendFiles: [] → "No UI changes detected" → agent skips
+    // browser_navigate entirely. With the change + a Flutter pubspec, the
+    // diff registers as frontend and the agent gets told to navigate.
+    it("recognises .dart files as frontend in a Flutter project (with pubspec.yaml)", async () => {
+        readSpy.mockResolvedValue({
+            services: [
+                {
+                    serviceName: "birdle-frontend",
+                    language: "typescript",
+                    framework: "playwright",
+                    api: { baseUrl: "http://localhost:8080" },
+                },
+            ],
+        });
+        const repo = makeRepoWithDiff(["lib/main.dart"]);
+        // Write a Flutter pubspec.yaml at the repo root so hasFlutterSdkDep returns true
+        fs.writeFileSync(path.join(repo, "pubspec.yaml"), `name: birdle\ndependencies:\n  flutter:\n    sdk: flutter\n`);
+        const result = await runUiAnalyzeChanges({ repositoryPath: repo });
+        expect(result.uiContext.changedFrontendFiles).toContain("lib/main.dart");
+        expect(result.uiContext.candidateUiPages.length).toBeGreaterThan(0);
+        expect(result.instructions).toMatch(/browser_navigate/);
+        fs.rmSync(repo, { recursive: true, force: true });
+    });
+    it("does NOT recognise .dart files as frontend without a Flutter pubspec.yaml", async () => {
+        // Pure Dart project (server/CLI) — pubspec.yaml absent or missing the SDK dep.
+        // .dart files should NOT enter the UI pipeline.
+        readSpy.mockResolvedValue({ services: [] });
+        const repo = makeRepoWithDiff(["bin/server.dart"]);
+        // No pubspec.yaml written at all
+        const result = await runUiAnalyzeChanges({ repositoryPath: repo });
+        expect(result.uiContext.changedFrontendFiles).toEqual([]);
+        expect(result.uiContext.candidateUiPages).toEqual([]);
+        expect(result.instructions).toMatch(/no UI changes/i);
+        fs.rmSync(repo, { recursive: true, force: true });
+    });
+    it("does NOT recognise .dart files as frontend in a pure Dart project (pubspec without flutter SDK dep)", async () => {
+        // pubspec.yaml exists but declares a Dart server (e.g. shelf) — no flutter SDK
+        readSpy.mockResolvedValue({ services: [] });
+        const repo = makeRepoWithDiff(["bin/server.dart"]);
+        fs.writeFileSync(path.join(repo, "pubspec.yaml"), `name: my_dart_server\ndependencies:\n  shelf: ^1.4.0\n`);
+        const result = await runUiAnalyzeChanges({ repositoryPath: repo });
+        expect(result.uiContext.changedFrontendFiles).toEqual([]);
+        expect(result.uiContext.candidateUiPages).toEqual([]);
+        fs.rmSync(repo, { recursive: true, force: true });
+    });
 });