Look for pubspec.yaml next to .dart_tool/package_config.json instead of only current dir

Author: sigurdm

webdev/lib/src/command/build_command.dart

@@ -52,9 +52,8 @@ class BuildCommand extends Command<int> {
 
     List<String> arguments;
     try {
-      var pubspecLock = await readPubspecLock(configuration);
-      arguments = buildRunnerArgs(pubspecLock, configuration)
-        ..addAll(validExtraArgs);
+      await validatePubspecLock(configuration);
+      arguments = buildRunnerArgs(configuration)..addAll(validExtraArgs);
     } on PackageException catch (e) {
       logWriter(logging.Level.SEVERE, 'Pubspec errors: ',
           error: '${e.details}');

webdev/lib/src/command/daemon_command.dart

@@ -66,9 +66,8 @@ class DaemonCommand extends Command<int> {
             launchInChrome: true, debug: true, autoRun: false, release: false));
     configureLogWriter(configuration.verbose);
     // Validate the pubspec first to ensure we are in a Dart project.
-    PubspecLock? pubspecLock;
     try {
-      pubspecLock = await readPubspecLock(configuration);
+      await validatePubspecLock(configuration);
     } on PackageException catch (e) {
       logWriter(Level.SEVERE, 'Pubspec errors: ', error: '${e.details}');
       rethrow;
@@ -104,7 +103,7 @@ class DaemonCommand extends Command<int> {
         }
       });
       daemon.registerDomain(daemonDomain);
-      var buildOptions = buildRunnerArgs(pubspecLock, configuration);
+      var buildOptions = buildRunnerArgs(configuration);
       var extraArgs = argResults?.rest ?? [];
       var directoryArgs =
           extraArgs.where((arg) => !arg.startsWith('-')).toList();

webdev/lib/src/command/serve_command.dart

@@ -95,16 +95,15 @@ refresh: Performs a full page refresh.
     Configuration configuration;
     configuration = Configuration.fromArgs(argResults);
     configureLogWriter(configuration.verbose);
-    PubspecLock? pubspecLock;
     try {
-      pubspecLock = await readPubspecLock(configuration);
+      await validatePubspecLock(configuration);
     } on PackageException catch (e) {
       logWriter(Level.SEVERE, 'Pubspec errors: ', error: '${e.details}');
       rethrow;
     }
     // Forward remaining arguments as Build Options to the Daemon.
     // This isn't documented. Should it be advertised?
-    var buildOptions = buildRunnerArgs(pubspecLock, configuration)
+    var buildOptions = buildRunnerArgs(configuration)
       ..addAll(argResults!.rest.where((arg) => arg.startsWith('-')).toList());
     var extraArgs = argResults?.rest ?? [];
     var directoryArgs = extraArgs.where((arg) => !arg.startsWith('-')).toList();

webdev/lib/src/command/shared.dart

@@ -79,8 +79,7 @@ void addSharedArgs(ArgParser argParser,
 
 /// Parses the provided [Configuration] to return a list of
 /// `package:build_runner` appropriate arguments.
-List<String> buildRunnerArgs(
-    PubspecLock pubspecLock, Configuration configuration) {
+List<String> buildRunnerArgs(Configuration configuration) {
   var arguments = <String>[];
   if (configuration.release) {
     arguments.add('--$releaseFlag');
@@ -103,11 +102,10 @@ List<String> buildRunnerArgs(
   return arguments;
 }
 
-Future<PubspecLock> readPubspecLock(Configuration configuration) async {
+Future<void> validatePubspecLock(Configuration configuration) async {
   var pubspecLock = await PubspecLock.read();
   await checkPubspecLock(pubspecLock,
       requireBuildWebCompilers: configuration.requireBuildWebCompilers);
-  return pubspecLock;
 }
 
 /// Checks that the normalized form of [path] is a top level directory under

webdev/lib/src/pubspec.dart

@@ -7,6 +7,7 @@ import 'dart:convert';
 import 'dart:io';
 
 import 'package:http/http.dart';
+import 'package:path/path.dart' as p;
 import 'package:pub_semver/pub_semver.dart';
 import 'package:pubspec_parse/pubspec_parse.dart';
 import 'package:yaml/yaml.dart';
@@ -31,12 +32,6 @@ class PackageExceptionDetails {
       {this.description, bool missingDependency = false})
       : _missingDependency = missingDependency;
 
-  static const noPubspecLock =
-      PackageExceptionDetails._('`pubspec.lock` does not exist.',
-          description: 'Run `$appName` in a Dart package directory. '
-              'Run `dart pub get` first.',
-          missingDependency: true);
-
   static PackageExceptionDetails missingDep(
           String pkgName, VersionConstraint constraint) =>
       PackageExceptionDetails._(
@@ -75,9 +70,26 @@ class PubspecLock {
 
   static Future<PubspecLock> read() async {
     await _runPubDeps();
+    var dir = p.absolute(p.current);
+    while (true) {
+      final candidate = p.join(
+        dir,
+        '.dart_tool',
+        'package_config.json',
+      );
+      if (File(candidate).existsSync()) break;
+      final next = p.dirname(dir);
+      if (next == dir) {
+        // Give up.
+        dir = p.current;
+        break;
+      }
+      dir = next;
+    }
 
-    var pubspecLock =
-        loadYaml(await File('pubspec.lock').readAsString()) as YamlMap;
+    var pubspecLock = loadYaml(
+            await File(p.relative(p.join(dir, 'pubspec.lock'))).readAsString())
+        as YamlMap;
 
     var packages = pubspecLock['packages'] as YamlMap?;
     return PubspecLock(packages);