Add a few init related lints from recommended set

analysis_options.yaml

@@ -43,6 +43,7 @@ linter:
     - prefer_final_locals
     - prefer_for_elements_to_map_fromIterable
     - prefer_if_null_operators
+    - prefer_initializing_formals
     - prefer_null_aware_operators
     - prefer_single_quotes
     - prefer_spread_collections
@@ -63,3 +64,4 @@ linter:
     - unnecessary_this
     - use_function_type_syntax_for_parameters
     - use_rethrow_when_possible
+    - use_super_parameters

app/lib/fake/backend/fake_pub_worker.dart

@@ -314,7 +314,7 @@ final class UploadException implements Exception {
 }
 
 final class IntermittentUploadException extends UploadException {
-  IntermittentUploadException(String message) : super(message);
+  IntermittentUploadException(super.message);
 }
 
 Summary _emptySummary(String package, String version) {

app/lib/package/api_export/exported_api.dart

@@ -482,10 +482,10 @@ final class ExportedJsonFile<T> extends ExportedObject {
   final Duration _maxAge;
 
   ExportedJsonFile._(
-    ExportedApi _owner,
-    String _objectName,
+    super._owner,
+    super._objectName,
     this._maxAge,
-  ) : super._(_owner, _objectName);
+  ) : super._();
 
   ObjectMetadata _metadata() {
     return ObjectMetadata(
@@ -531,12 +531,12 @@ final class ExportedBlob extends ExportedObject {
   final String _filename;
 
   ExportedBlob._(
-    ExportedApi _owner,
-    String _objectName,
+    super._owner,
+    super._objectName,
     this._filename,
     this._contentType,
     this._maxAge,
-  ) : super._(_owner, _objectName);
+  ) : super._();
 
   ObjectMetadata _metadata() {
     return ObjectMetadata(

app/lib/package/model_properties.dart

@@ -236,8 +236,8 @@ class MinSdkVersion {
 }
 
 class PubspecProperty extends StringProperty {
-  const PubspecProperty({String? propertyName, bool required = false})
-      : super(propertyName: propertyName, required: required, indexed: false);
+  const PubspecProperty({super.propertyName, super.required = false})
+      : super(indexed: false);
 
   @override
   bool validate(ModelDB db, Object? value) =>

app/lib/package/models.dart

@@ -484,8 +484,8 @@ class AutomatedPublishing {
 /// A [db.Property] encoding [AutomatedPublishing] as JSON.
 class AutomatedPublishingProperty extends db.Property {
   const AutomatedPublishingProperty(
-      {String? propertyName, bool required = false})
-      : super(propertyName: propertyName, required: required, indexed: false);
+      {super.propertyName, super.required = false})
+      : super(indexed: false);
 
   @override
   Object? encodeValue(

app/lib/search/search_service.dart

@@ -339,10 +339,9 @@ class PackageSearchResult {
     List<SdkLibraryHit>? sdkLibraryHits,
     List<PackageHit>? packageHits,
     this.errorMessage,
-    int? statusCode,
+    this.statusCode,
   })  : packageHits = packageHits ?? <PackageHit>[],
-        sdkLibraryHits = sdkLibraryHits ?? <SdkLibraryHit>[],
-        statusCode = statusCode;
+        sdkLibraryHits = sdkLibraryHits ?? <SdkLibraryHit>[];
 
   PackageSearchResult.error({
     required this.errorMessage,

app/lib/search/token_index.dart

@@ -40,8 +40,8 @@ class TokenIndex<K> {
     List<K> ids,
     List<String?> values, {
     bool skipDocumentWeight = false,
-  }) : _ids = ids {
-    assert(ids.length == values.length);
+  })  : _ids = ids,
+        assert(ids.length == values.length) {
     final length = values.length;
     for (var i = 0; i < length; i++) {
       final text = values[i];

app/lib/service/download_counts/models.dart

@@ -16,8 +16,8 @@ class DownloadCounts extends db.ExpandoModel<String> {
 }
 
 class CountDataProperty extends db.Property {
-  const CountDataProperty({String? propertyName, bool required = false})
-      : super(propertyName: propertyName, required: required, indexed: false);
+  const CountDataProperty({super.propertyName, super.required = false})
+      : super(indexed: false);
 
   @override
   Object? decodePrimitiveValue(db.ModelDB db, Object? value) {

app/lib/service/security_advisories/models.dart

@@ -49,8 +49,8 @@ class SecurityAdvisory extends db.Model<String> {
 }
 
 class OSVProperty extends db.Property {
-  const OSVProperty({String? propertyName, bool required = false})
-      : super(propertyName: propertyName, required: required, indexed: false);
+  const OSVProperty({super.propertyName, super.required = false})
+      : super(indexed: false);
 
   @override
   Object? decodePrimitiveValue(db.ModelDB db, Object? value) {

app/lib/shared/exceptions.dart

@@ -24,14 +24,12 @@ abstract class ResponseException extends ApiResponseException {
     int status,
     String code,
     String message, {
-    Map<String, Object>? body,
-    Map<String, Object>? headers,
+    super.body,
+    super.headers,
   }) : super(
           status: status,
           code: code,
           message: message,
-          body: body,
-          headers: headers,
         );
 
   @override

app/lib/shared/model_properties.dart

@@ -10,8 +10,8 @@ class CompatibleListProperty<T> extends Property {
   final PrimitiveProperty subProperty;
 
   const CompatibleListProperty(this.subProperty,
-      {String? propertyName, bool indexed = true})
-      : super(propertyName: propertyName, required: true, indexed: indexed);
+      {super.propertyName, super.indexed})
+      : super(required: true);
 
   @override
   bool validate(ModelDB db, Object? value) {

app/lib/task/models.dart

@@ -333,8 +333,8 @@ class PackageVersionStateInfo {
 /// A [db.Property] encoding a Map from version to [PackageVersionStateInfo] as JSON.
 class PackageVersionStateMapProperty extends db.Property {
   const PackageVersionStateMapProperty(
-      {String? propertyName, bool required = false})
-      : super(propertyName: propertyName, required: required, indexed: false);
+      {super.propertyName, super.required = false})
+      : super(indexed: false);
 
   @override
   Object? encodeValue(
@@ -442,8 +442,8 @@ class AbortedTokenInfo {
 
 /// A [db.Property] encoding a List os [AbortedTokenInfo] as JSON.
 class AbortedTokenListProperty extends db.Property {
-  const AbortedTokenListProperty({String? propertyName, bool required = false})
-      : super(propertyName: propertyName, required: required, indexed: false);
+  const AbortedTokenListProperty({super.propertyName, super.required = false})
+      : super(indexed: false);
 
   @override
   Object? encodeValue(

app/lib/tool/neat_task/datastore_status_provider.dart

@@ -42,12 +42,13 @@ class NeatTaskStatus extends db.ExpandoModel<String> {
 
   NeatTaskStatus();
 
-  NeatTaskStatus.init(String name, {required bool isRuntimeVersioned}) {
-    this.name = name;
-    runtimeVersion =
-        _runtimeVersion(name, isRuntimeVersioned: isRuntimeVersioned);
+  NeatTaskStatus.init(String name, {required bool isRuntimeVersioned})
+      // ignore: prefer_initializing_formals
+      : name = name,
+        runtimeVersion =
+            _runtimeVersion(name, isRuntimeVersioned: isRuntimeVersioned),
+        updated = clock.now().toUtc() {
     id = _compositeId(name, isRuntimeVersioned: isRuntimeVersioned);
-    updated = clock.now().toUtc();
   }
 }
 

pkg/_popularity/lib/popularity.dart

@@ -70,10 +70,9 @@ class VoteData {
 
   int get score => _score(this);
 
-  VoteData(this.direct, this.dev, this.total) {
-    assert(direct >= 0 && direct <= total);
-    assert(dev >= 0 && dev <= total);
-  }
+  VoteData(this.direct, this.dev, this.total)
+      : assert(direct >= 0 && direct <= total),
+        assert(dev >= 0 && dev <= total);
 
   factory VoteData.fromJson(Map<String, dynamic> json) =>
       _$VoteDataFromJson(json);

pkg/pub_worker/lib/src/upload.dart

@@ -88,11 +88,11 @@ final class UploadException implements Exception {
 }
 
 final class IntermittentUploadException extends UploadException {
-  IntermittentUploadException(String message) : super(message);
+  IntermittentUploadException(super.message);
 }
 
 final class TaskAbortedException extends UploadException {
-  TaskAbortedException(String message) : super(message);
+  TaskAbortedException(super.message);
 }
 
 /// Extract `error.code` from JSON body in [res].