Unify configFile

pkgs/native_assets_cli/lib/src/model/build_config.dart

@@ -6,23 +6,23 @@ part of '../api/build_config.dart';
 
 final class BuildConfigImpl extends HookConfigImpl implements BuildConfig {
   @override
+  Hook get hook => Hook.build;
+
+  @override
+  //TODO: Should be removed once migration to `hook/` is complete.
   Uri get script {
-    final hookScript =
-        packageRoot.resolve('hook/').resolve(Hook.build.scriptName);
+    final hookScript = packageRoot.resolve('hook/').resolve(hook.scriptName);
     if (File.fromUri(hookScript).existsSync()) {
       return hookScript;
     } else {
-      return packageRoot.resolve(Hook.build.scriptName);
+      return packageRoot.resolve(hook.scriptName);
     }
   }
 
   @override
   String get outputName =>
       version > Version(1, 1, 0) ? 'build_output.json' : 'build_output.yaml';
 
-  @override
-  Uri get configFile => outputDirectory.resolve('../config.json');
-
   /// The folder in which all output and intermediate artifacts should be
   /// placed.
   @override

pkgs/native_assets_cli/lib/src/model/hook.dart

@@ -4,15 +4,17 @@
 
 /// The two types of scripts which are hooked into the compilation process.
 ///
-/// The `build.dart` script runs before, and the `link.dart` script after
-/// compilation.
+/// The `build.dart` hook runs before, and the `link.dart` hook after
+/// compilation. This enum holds static information about these hooks.
 enum Hook {
-  link('link'),
-  build('build');
+  link('link', 'link_config'),
+  build('build', 'config');
 
   final String _scriptName;
+  final String _configName;
 
   String get scriptName => '$_scriptName.dart';
+  String get configName => '$_configName.json';
 
-  const Hook(this._scriptName);
+  const Hook(this._scriptName, this._configName);
 }

pkgs/native_assets_cli/lib/src/model/hook_config.dart

@@ -4,14 +4,20 @@
 
 import 'package:pub_semver/pub_semver.dart';
 
+import 'hook.dart';
+
 abstract class HookConfigImpl {
-  Uri get configFile;
+  Hook get hook;
+
+  Uri get configFile => outputDirectory.resolve('../${hook.configName}');
 
   Uri get outputFile => outputDirectory.resolve(outputName);
 
   Uri get outputDirectory;
 
-  Uri get script;
+  // This is currently overriden by [BuildConfig], do account for older versions
+  // still using a top-level build.dart.
+  Uri get script => packageRoot.resolve('hook/').resolve(hook.scriptName);
 
   String toJsonString();
 

pkgs/native_assets_cli/lib/src/model/link_config.dart

@@ -10,6 +10,9 @@ part of '../api/link_config.dart';
 /// [assets] from the build step, and the [resources] generated during the
 /// kernel compilation.
 class LinkConfigImpl extends HookConfigImpl implements LinkConfig {
+  @override
+  Hook get hook => Hook.link;
+
   @override
   final List<AssetImpl> assets;
 
@@ -28,9 +31,6 @@ class LinkConfigImpl extends HookConfigImpl implements LinkConfig {
   })  : _buildConfig = buildConfig,
         resources = fromIdentifiers(resourceIdentifiers);
 
-  @override
-  Uri get configFile => outputDirectory.resolve('../link_config.json');
-
   @override
   Uri get outputDirectory => _buildConfig.outputDirectory;
 
@@ -43,9 +43,6 @@ class LinkConfigImpl extends HookConfigImpl implements LinkConfig {
   @override
   Uri get packageRoot => _buildConfig.packageRoot;
 
-  @override
-  Uri get script => packageRoot.resolve('hook/').resolve(Hook.link.scriptName);
-
   @override
   String toJsonString() =>
       const JsonEncoder.withIndent('  ').convert(_args.toJson());