Add a CI workflow to check the tool/builder package (#25)

* add a CI workflow to check the tool/builder package

* correct the builder workflow setup action

* check formatting on stable

.github/dependabot.yml

@@ -0,0 +1,9 @@
+# Dependabot configuration file.
+# See https://docs.github.com/en/code-security/dependabot/dependabot-version-updates
+
+version: 2
+updates: 
+  - package-ecosystem: github-actions
+    directory: /
+    schedule:
+      interval: monthly

.github/workflows/builder.yaml

@@ -0,0 +1,38 @@
+name: builder
+
+# Declare default permissions as read only.
+permissions: read-all
+
+on:
+  pull_request:
+    branches: [ main ]
+  push:
+    branches: [ main ]
+  schedule:
+    - cron: '0 0 * * 0' # weekly
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    defaults:
+      run:
+        working-directory: tool/builder
+    strategy:
+      fail-fast: false
+      matrix:
+        sdk: [stable]
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dart-lang/setup-dart@v1
+        with:
+          sdk: ${{ matrix.sdk }}
+
+      - run: dart pub get
+
+      - run: dart analyze --fatal-infos
+
+      - run: dart format --output=none --set-exit-if-changed .
+        if: ${{ matrix.sdk == 'stable' }}
+
+      # Ensure the tool can run without errors.
+      - run: dart bin/main.dart -h

.github/workflows/main.yaml

@@ -4,11 +4,10 @@ name: Main Branch CI
 permissions: read-all
 
 on:
-  push:
-    branches: [main]
   pull_request:
+    branches: [ main ]
+  push:
     branches: [main, ffi-wrapper, ffi-wrapper-text-pkg]
-  workflow_dispatch:
   schedule:
     - cron: "0 0 * * *" # Every day at midnight
 

tool/builder/analysis_options.yaml

@@ -3,3 +3,15 @@ include: package:lints/recommended.yaml
 analyzer:
   errors:
     todo: ignore
+
+linter:
+  rules:
+    - always_declare_return_types
+    - avoid_dynamic_calls
+    - comment_references
+    - directives_ordering
+    - library_annotations
+    - only_throw_errors
+    - prefer_relative_imports
+    - prefer_single_quotes
+    - sort_pub_dependencies

tool/builder/bin/main.dart

@@ -10,7 +10,7 @@ import 'package:builder/sync_headers.dart';
 final runner = CommandRunner(
   'build',
   'Performs build operations for google/flutter-mediapipe that '
-      'depend on contents in this repository',
+      'depend on contents in this repository.',
 )
   ..addCommand(DownloadModelCommand())
   ..addCommand(SdksFinderCommand())

tool/builder/lib/download_model.dart

@@ -3,12 +3,14 @@
 // found in the LICENSE file.
 
 import 'dart:io' as io;
+
 import 'package:args/command_runner.dart';
-import 'package:builder/repo_finder.dart';
 import 'package:http/http.dart' as http;
 import 'package:logging/logging.dart';
 import 'package:path/path.dart' as path;
 
+import 'repo_finder.dart';
+
 final _log = Logger('DownloadModelCommand');
 
 enum Model {
@@ -86,7 +88,7 @@ class DownloadModelCommand extends Command with RepoFinderMixin {
   Model get model {
     final value = Model.values.asNameMap()[argResults!['model']];
     if (value == null) {
-      throw ('Unexpected Model value ${argResults!['model']}');
+      throw Exception('Unexpected Model value ${argResults!['model']}');
     }
     return value;
   }

tool/builder/lib/extensions.dart

@@ -1,3 +1,7 @@
+// Copyright 2024 The Flutter Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
 import 'dart:convert';
 import 'dart:io';
 

tool/builder/lib/repo_finder.dart

@@ -3,11 +3,12 @@
 // found in the LICENSE file.
 
 import 'dart:io' as io;
+
 import 'package:args/args.dart';
 import 'package:args/command_runner.dart';
+import 'package:io/ansi.dart';
 import 'package:logging/logging.dart';
 import 'package:path/path.dart' as path;
-import 'package:io/ansi.dart';
 
 /// Mixin to help [Command] subclasses locate both `google/mediapipe` and
 /// the root of `google/flutter-mediapipe` (this repository).

tool/builder/lib/sdks_finder.dart

@@ -1,15 +1,20 @@
+// Copyright 2024 The Flutter Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
 import 'dart:convert';
-import 'dart:io';
 import 'dart:io' as io;
+import 'dart:io';
 
 import 'package:args/command_runner.dart';
-import 'package:builder/extensions.dart';
-import 'package:builder/repo_finder.dart';
 import 'package:io/ansi.dart';
 import 'package:logging/logging.dart';
 import 'package:native_assets_cli/native_assets_cli.dart';
 import 'package:path/path.dart' as path;
 
+import 'extensions.dart';
+import 'repo_finder.dart';
+
 final _log = Logger('SDKsFinder');
 
 /// Structure of flattened build locations suitable for JSON serialization.
@@ -71,7 +76,7 @@ class SdksFinderCommand extends Command with RepoFinderMixin {
   }
   @override
   String description =
-      'Updates MediaPipe SDK manifest files for the current build target';
+      'Updates MediaPipe SDK manifest files for the current build target.';
 
   @override
   String name = 'sdks';
@@ -257,7 +262,7 @@ class _OsFinder {
     );
   }
 
-  /// Receives a [Path] like ".../[os_folder]/release/[build_number]/[date]/"
+  /// Receives a path like `".../[os_folder]/release/[build_number]/[date]/"`
   /// and yields all matching architecture folders within.
   Stream<String> _getArchitectectures(String path) async* {
     final pathsWithinBuild = <String>[];

tool/builder/lib/sync_headers.dart

@@ -4,13 +4,15 @@
 
 import 'dart:convert';
 import 'dart:io' as io;
+
 import 'package:args/command_runner.dart';
-import 'package:builder/extensions.dart';
-import 'package:builder/repo_finder.dart';
 import 'package:io/ansi.dart';
 import 'package:path/path.dart' as path;
 import 'package:process/process.dart';
 
+import 'extensions.dart';
+import 'repo_finder.dart';
+
 /// Relative header paths (in both repositories)
 final containers = 'mediapipe/tasks/c/components/containers';
 final processors = 'mediapipe/tasks/c/components/processors';
@@ -49,7 +51,7 @@ List<(String, String, String, Function(io.File)?)> headerPaths = [
 /// disrupted by the move.
 class SyncHeadersCommand extends Command with RepoFinderMixin {
   @override
-  String description = 'Syncs header files to google/flutter-mediapipe';
+  String description = 'Syncs header files to google/flutter-mediapipe.';
   @override
   String name = 'headers';
 

tool/builder/pubspec.yaml

@@ -1,12 +1,11 @@
 name: builder
-description: Performs build operations for google/flutter-mediapipe that depend
-  on contents in this repository.
-version: 1.0.0
-# repository: https://github.com/my_org/my_repo
+description: Performs build operations for google/flutter-mediapipe.
+
+publish_to: none
+
 environment:
   sdk: ^3.1.5
 
-# Add regular dependencies here.
 dependencies:
   args: ^2.4.2
   http: ^1.1.0