blob: 844a2be59941754dd32bafba306cc9a94b5f77b3 [file] [log] [blame]
// Copyright (c) 2021, 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 'package:analyzer/dart/ast/ast.dart';
import 'package:analyzer/dart/ast/visitor.dart';
import '../analyzer.dart';
import '../ast.dart';
const _desc = r'Depend on referenced packages.';
const _details = r'''
**DO** Depend on referenced packages.
When importing a package, add a dependency on it to your pubspec.
Depending explicitly on packages that you reference ensures they will always
exist and allows you to put a dependency constraint on them to guard you
against breaking changes.
Whether this should be a regular dependency or dev_dependency depends on if it
is referenced from a public file (one under either `lib` or `bin`), or some
other private file.
**BAD:**
```dart
import 'package:a/a.dart';
```
```yaml
dependencies:
```
**GOOD:**
```dart
import 'package:a/a.dart';
```
```yaml
dependencies:
a: ^1.0.0
```
''';
class DependOnReferencedPackages extends LintRule implements NodeLintRule {
DependOnReferencedPackages()
: super(
name: 'depend_on_referenced_packages',
description: _desc,
details: _details,
group: Group.pub);
@override
void registerNodeProcessors(
NodeLintRegistry registry, LinterContext context) {
// Only lint if we have a pubspec.
var package = context.package;
if (package is! PubWorkspacePackage) return;
var pubspec = package.pubspec;
if (pubspec == null) return;
var name = pubspec.name?.value.text;
if (name == null) return;
var dependencies = pubspec.dependencies;
var devDependencies = pubspec.devDependencies;
var availableDeps = [
name,
if (dependencies != null)
for (var dep in dependencies)
if (dep.name?.text != null) dep.name!.text!,
if (devDependencies != null &&
!isInPublicDir(context.currentUnit.unit, context.package))
for (var dep in devDependencies)
if (dep.name?.text != null) dep.name!.text!,
];
var visitor = _Visitor(this, availableDeps);
registry.addImportDirective(this, visitor);
registry.addExportDirective(this, visitor);
}
}
class _Visitor extends SimpleAstVisitor {
final DependOnReferencedPackages rule;
final List<String> availableDeps;
_Visitor(this.rule, this.availableDeps);
void _checkDirective(UriBasedDirective node) {
// Is it a package: uri?
var uriContent = node.uriContent;
if (uriContent == null) return;
if (!uriContent.startsWith('package:')) return;
// The package name is the first segment of the uri, find the first slash.
var firstSlash = uriContent.indexOf('/');
if (firstSlash == -1) return;
var packageName = uriContent.substring(8, firstSlash);
if (availableDeps.contains(packageName)) return;
rule.reportLint(node.uri);
}
@override
void visitImportDirective(ImportDirective node) => _checkDirective(node);
@override
void visitExportDirective(ExportDirective node) => _checkDirective(node);
}