Adapt package:messages to be loading mechanism independent (#843)

* Squashed commit

* Add newline

* Add g.dart files to generated

* Upgrade to dart:asset

* Add changelogs

* Add wip

* Run tests only on dev

* Add logger

* Update examples

* Do not use dart:asset

* Use a dart run script instead of hooks

* Also output code in dart run

* Adapt workflow

* Fixes

* Add licenses

* typo

* Fix issues

* Sort on key

* Refactor to list

* Sort

* Clean ups

* sort deps

* Changelog

* Remove build_runner remaisn

* Delete old examples

* Add new examples

* Fix flutter specific package loading

* Fix zero word case

* Switch to flutter in wf

* Fix CI

* Setup flutter

* use dart not flutter

* do not check flutter examples

* Changes as per review

* Update CI sha

* Use main channel

* use beta channel

* Use new CI tool

* Rename

* Add licenses

* Use dart run messages again

Author: mosuem

.gitattributes

@@ -1 +1,2 @@
 pkgs/intl4x/lib/src/bindings/* linguist-generated=true
+pkgs/intl4x/**/*.g.dart linguist-generated=true

.github/workflows/messages.yml

@@ -21,28 +21,34 @@ jobs:
     defaults:
       run:
         working-directory: pkgs/messages
-    strategy:
-      matrix:
-        sdk: [stable, dev] # {pkgs.versions}
-        dependencies: [path, published]
-        include:
-          - sdk: stable
-            run-tests: true
 
     steps:
       - uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9
-      - uses: dart-lang/setup-dart@0a8a0fc875eb934c15d08629302413c671d3f672
+
+      - uses: flutter-actions/setup-flutter@61f93c6f11e0234fa9ec4df59362c9699b979660
         with:
-          sdk: ${{matrix.sdk}}
+          channel: beta
+
+      - run: dart pub get
 
       - run: dart pub get
+        working-directory: pkgs/messages/example
 
-      - run: (cd example_json; dart pub get)
+      - run: flutter pub get
+        working-directory: pkgs/messages/examples_flutter/my_application
+
+      - run: flutter pub get
+        working-directory: pkgs/messages/examples_flutter/my_shopping_cart
 
       - run: dart analyze --fatal-infos
 
       - run: dart format --output=none --set-exit-if-changed .
-        if: ${{matrix.run-tests}}
 
       - run: dart test
-        if: ${{matrix.run-tests}}
+
+      - name: Regenerate and run example
+        working-directory: pkgs/messages/example
+        run: |
+          dart run messages
+          git diff --exit-code
+          dart run

.github/workflows/messages_builder.yml

@@ -23,9 +23,9 @@ jobs:
         working-directory: pkgs/messages_builder
     strategy:
       matrix:
-        sdk: [stable, dev] # {pkgs.versions}
+        sdk: [dev]
         include:
-          - sdk: stable
+          - sdk: dev
             run-tests: true
     steps:
       - uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9

.github/workflows/messages_serializer.yml

@@ -23,9 +23,9 @@ jobs:
         working-directory: pkgs/messages_serializer
     strategy:
       matrix:
-        sdk: [stable, dev] # {pkgs.versions}
+        sdk: [dev]
         include:
-          - sdk: stable
+          - sdk: dev
             run-tests: true
     steps:
       - uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9

.github/workflows/messages_shrinker.yml

@@ -23,9 +23,9 @@ jobs:
         working-directory: pkgs/messages_shrinker
     strategy:
       matrix:
-        sdk: [stable, dev] # {pkgs.versions}
+        sdk: [dev]
         include:
-          - sdk: stable
+          - sdk: dev
             run-tests: true
     steps:
       - uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9

pkgs/messages/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.3.0-wip
+
+- Adapt to output data files to assets.
+
 ## 0.2.0
 
 - Remove `IntlObject` interface.

pkgs/messages/README.md

@@ -34,10 +34,9 @@ The `builder` to generate the named methods and data files from the input `arb`
 The logic for serializing `arb` message files into data files.
 
 ## Example
-Add `package:messages` and `package:messages_builder` to your dependencies:
+Add `package:messages` to your dependencies:
 ```bash
 dart pub add messages
-dart pub add dev:messages_builder
 ```
 
 Given translation message files in two languages:
@@ -68,7 +67,7 @@ This translated file was created by a translator given the reference `en.arb`.
 ```
 you can then run
 
-`dart run build_runner build -d`
+`dart run messages`
 
 This will generate both code to call your messages, as well as data files which will be shipped with your application. You can then use these generated files by importing the generated files:
 
@@ -105,4 +104,4 @@ package_options:
       // multi
       // line
       // header
-```
+```

pkgs/messages/bin/messages.dart

@@ -0,0 +1,31 @@
+// Copyright (c) 2024, the Dart project authors.  Please see the AUTHORS file
+// for details. 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:io';
+
+Future<void> main(List<String> args) async {
+  final runBuilder = await Process.run('dart', ['run', 'messages_builder']);
+  stdout.write(runBuilder.stdout as String);
+
+  final runBuilderStdErr = runBuilder.stderr as String;
+  final messagesBuilderNotInDeps =
+      runBuilderStdErr.contains('Could not find package `messages_builder`');
+  if (messagesBuilderNotInDeps) {
+    print('Adding `package:messages_builder` to dev dependencies...');
+    final addBuilder = await runDart(['pub', 'add', 'dev:messages_builder']);
+    if (addBuilder.exitCode == 0) {
+      print('Re-running message generation');
+      await runDart(['run', 'messages_builder']);
+    }
+  } else {
+    stderr.write(runBuilderStdErr);
+  }
+}
+
+Future<ProcessResult> runDart(List<String> arguments) async {
+  final processResult = await Process.run('dart', arguments);
+  stdout.write(processResult.stdout as String);
+  stderr.write(processResult.stderr as String);
+  return processResult;
+}

pkgs/messages/example_json/.gitignore renamed to pkgs/messages/example/.gitignore

@@ -4,3 +4,4 @@
 
 # Conventional directory for build output.
 build/
+bin/example/