From af5d330f836ed4894d07035eaabaedeff42797ec Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Wed, 24 Jun 2026 10:07:49 +0000 Subject: [PATCH 1/2] Update CI parity for worker_flutter - Add `worker_flutter` definitions and executable commands to `AGENTS.md`. - Widen flutter sdk dependency bound in `worker_flutter/pubspec.yaml` to ensure compatibility with `stable` branch in CI. - Add `subosito/flutter-action@v2` action configuring the `stable` channel to `lint` and `test` jobs in `.github/workflows/ci.yml`. - Add executable lint and test commands to `scripts/lint.sh` and `scripts/test.sh`. --- .github/workflows/ci.yml | 8 ++++++++ AGENTS.md | 4 ++++ scripts/lint.sh | 4 ++++ scripts/test.sh | 4 ++++ worker_flutter/pubspec.lock | 10 +++++----- worker_flutter/pubspec.yaml | 2 +- 6 files changed, 26 insertions(+), 6 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index dcc5cbeb..e46cb542 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -21,6 +21,10 @@ jobs: api/package-lock.json worker/package-lock.json + - uses: subosito/flutter-action@v2 + with: + channel: 'stable' + - run: ./scripts/lint.sh build: @@ -62,6 +66,10 @@ jobs: api/package-lock.json worker/package-lock.json + - uses: subosito/flutter-action@v2 + with: + channel: 'stable' + - run: ./scripts/test.sh release-scripts: diff --git a/AGENTS.md b/AGENTS.md index fa0850de..db6491ad 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -9,6 +9,7 @@ - **Node.js**: 20+ (managed by `npm`) - **Docker**: Required for simulation engine - **Package Manager**: `npm` (Node) +- **Flutter**: `stable` channel ## Setup - **Install Node Dependencies**: `npm run install:all` @@ -29,6 +30,8 @@ - **Worker Install**: `cd worker && npm install` - **Worker Dev**: `cd worker && npm run dev` - **Worker Test**: `cd worker && npm run test:unit` +- **Worker Flutter Lint**: `cd worker_flutter && flutter analyze --no-fatal-infos` +- **Worker Flutter Test**: `cd worker_flutter && flutter test` ## Deployment - **CI**: `.github/workflows/ci.yml` (Runs on PR) @@ -38,3 +41,4 @@ - `frontend/`: React app - `api/`: Next.js API, ingestion, and analysis - `worker/`: Node.js + Docker simulation runner +- `worker_flutter/`: Cross-platform desktop worker diff --git a/scripts/lint.sh b/scripts/lint.sh index 528e316c..af4c5369 100755 --- a/scripts/lint.sh +++ b/scripts/lint.sh @@ -15,3 +15,7 @@ cd frontend && npm ci && npm run lint && cd .. # API echo "=== API lint ===" cd api && npm ci && npm run lint && cd .. + +# Worker Flutter +echo "=== Worker Flutter lint ===" +cd worker_flutter && flutter pub get && flutter analyze --no-fatal-infos && cd .. diff --git a/scripts/test.sh b/scripts/test.sh index 31b58634..3fa7a7cf 100755 --- a/scripts/test.sh +++ b/scripts/test.sh @@ -22,3 +22,7 @@ npm run test:ingestion && cd .. # Worker tests echo "=== Worker unit tests ===" cd worker && npm ci && npm run test:unit && cd .. + +# Worker Flutter +echo "=== Worker Flutter tests ===" +cd worker_flutter && flutter pub get && flutter test && cd .. diff --git a/worker_flutter/pubspec.lock b/worker_flutter/pubspec.lock index b7c9feee..d21e63a0 100644 --- a/worker_flutter/pubspec.lock +++ b/worker_flutter/pubspec.lock @@ -692,10 +692,10 @@ packages: dependency: transitive description: name: matcher - sha256: dc0b7dc7651697ea4ff3e69ef44b0407ea32c487a39fff6a4004fa585e901861 + sha256: "12956d0ad8390bbcc63ca2e1469c0619946ccb52809807067a7020d57e647aa6" url: "https://pub.dev" source: hosted - version: "0.12.19" + version: "0.12.18" material_color_utilities: dependency: transitive description: @@ -1193,10 +1193,10 @@ packages: dependency: transitive description: name: test_api - sha256: "8161c84903fd860b26bfdefb7963b3f0b68fee7adea0f59ef805ecca346f0c7a" + sha256: "93167629bfc610f71560ab9312acdda4959de4df6fac7492c89ff0d3886f6636" url: "https://pub.dev" source: hosted - version: "0.7.10" + version: "0.7.9" tray_manager: dependency: "direct main" description: @@ -1374,5 +1374,5 @@ packages: source: hosted version: "3.1.3" sdks: - dart: ">=3.11.5 <4.0.0" + dart: ">=3.11.0 <4.0.0" flutter: ">=3.38.4" diff --git a/worker_flutter/pubspec.yaml b/worker_flutter/pubspec.yaml index 5fb626dc..08b2bf83 100644 --- a/worker_flutter/pubspec.yaml +++ b/worker_flutter/pubspec.yaml @@ -10,7 +10,7 @@ publish_to: 'none' version: 0.2.0+1 environment: - sdk: ^3.11.5 + sdk: '>=3.11.0 <4.0.0' dependencies: flutter: From 61ff8cc9d7a6d882f1deccccb6c6d4aa579c156e Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Wed, 24 Jun 2026 10:20:24 +0000 Subject: [PATCH 2/2] Update CI parity for worker_flutter - Add `worker_flutter` definitions and executable commands to `AGENTS.md`. - Widen flutter sdk dependency bound in `worker_flutter/pubspec.yaml` to ensure compatibility with `stable` branch in CI. - Remove unused macos specific import from `worker_flutter/lib/main.dart` to satisfy flutter analyzer. - Add `subosito/flutter-action@v2` action configuring the `stable` channel to `lint` and `test` jobs in `.github/workflows/ci.yml`. - Add executable lint and test commands to `scripts/lint.sh` and `scripts/test.sh`. --- worker_flutter/lib/main.dart | 1 - 1 file changed, 1 deletion(-) diff --git a/worker_flutter/lib/main.dart b/worker_flutter/lib/main.dart index af23c0da..289325c9 100644 --- a/worker_flutter/lib/main.dart +++ b/worker_flutter/lib/main.dart @@ -19,7 +19,6 @@ import 'installer/install_progress_app.dart'; import 'installer/installer.dart'; import 'launch/auto_start_service.dart'; import 'launch/mode_picker_screen.dart'; -import 'macos/activation_policy.dart'; import 'offline/offline_app.dart'; import 'sentry_setup.dart'; import 'telemetry.dart';