Add strict_raw_type to ignore_for_file in generated file

[corran2go]

Is there an existing issue?

• I have searched existing issues

Use case

I'd like to get rid of the analyzer warning regarding EntityDefinition<dynamic> when using strict-raw-types analyzer option:

The generic type 'EntityDefinition<dynamic>' should have explicit type arguments but doesn't.
Use explicit type arguments for 'EntityDefinition<dynamic>'.

Proposed solution

Maybe this can be more specific or just add strict_raw_type to the already generated line:

// ignore_for_file: camel_case_types, depend_on_referenced_packages

[greenrobot-team]

Thanks for reporting. However, there should never be a EntityDefinition<dynamic> in objectbox.g.dart? The type parameter should always be an entity class type, such as EntityDefinition<TestEntity>.

Regardless, to reproduce this it would be helpful to know which version of analyzer and what rules or lints package you are using. An example entity class would also be helpful. Thanks!

[corran2go]

Hi, thanks for clarifying.

I'm using

pubspec.yaml

objectbox: 5.2.0
objectbox_generator: 5.2.0

pubspec.lock

analyzer:
  version: "9.0.0"
...
sdks:
  dart: ">=3.11.0 <4.0.0"
  flutter: ">=3.41.0"

analysis_options.yaml

include: package:flutter_lints/flutter.yaml

analyzer:
  language:
    strict-casts: true
    strict-inference: true
    strict-raw-types: true

settings_entity.dart

import 'package:objectbox/objectbox.dart';

import '/src/models/models.dart';

@Entity()
class SettingsEntity {
  SettingsEntity({
    this.id = 0,
    this.lang,
    this.version,
    this.newsVersion = '',
  });

  factory SettingsEntity.fromModel(SettingsModel model, {int id = 0}) => SettingsEntity(
    id: id,
    lang: model.lang,
    version: model.version,
    newsVersion: model.newsVersion,
  );

  @Id()
  int id;

  String? lang;
  String? version;
  String? newsVersion;

  SettingsModel toModel() => SettingsModel(
    lang: lang,
    version: version,
    newsVersion: newsVersion,
  );

  @override
  String toString() =>
      '['
      '$runtimeType '
      'id: $id, '
      'lang: $lang, '
      'version: $version, '
      'newsVersion: $newsVersion'
      ']';
}

Generated objectbox.g.dart

...
obx_int.ModelDefinition getObjectBoxModel() {
...
  final bindings = <Type, obx_int.EntityDefinition>{
    SettingsEntity: obx_int.EntityDefinition<SettingsEntity>(
...

The line with the binding is flagged by the analyzer.

[greenrobot-team]

Thanks for the helpful details! This should have been in the original report.

Based on a quick test it should be feasible to instead update the generated code so the error doesn't even appear.

Edit: as suggested, the workaround for this is to manually update the ignore_for_file comment in objectbox.g.dart like:

// ignore_for_file: camel_case_types, depend_on_referenced_packages, strict_raw_type

For our reference, the internal issue for this is objectbox-dart#178