extract gql_tristate_value to separate package in order to not pollut…

…e gql_exec for users who don't use tristate null

.github/workflows/dart.yml

@@ -18,6 +18,7 @@ jobs:
           - gql_http_link
           - gql_websocket_link
           - gql_transform_link
+          - gql_tristate_value
     runs-on: ubuntu-latest
     container:
       image: dart:latest

codegen/end_to_end_test/pubspec.yaml

@@ -15,3 +15,12 @@ dev_dependencies:
   build: ^2.0.0
   build_runner: ^2.0.0
   test: ^1.16.8
+dependency_overrides: 
+  gql_tristate_value: 
+    path: ../gql_tristate_value
+  gql_code_builder: 
+    path: ../gql_code_builder
+  gql_build: 
+    path: ../gql_build
+  end_to_end_test_tristate: 
+    path: ../end_to_end_test_tristate

codegen/end_to_end_test_tristate/pubspec.yaml

@@ -15,3 +15,13 @@ dev_dependencies:
   build: ^2.0.0
   build_runner: ^2.0.0
   test: ^1.16.8
+  gql_tristate_value: ^1.0.0
+dependency_overrides: 
+  gql_tristate_value: 
+    path: ../gql_tristate_value
+  gql_code_builder: 
+    path: ../gql_code_builder
+  gql_build: 
+    path: ../gql_build
+  end_to_end_test: 
+    path: ../end_to_end_test

codegen/end_to_end_test_tristate/test/operation/variables_test.dart

@@ -1,4 +1,4 @@
-import 'package:gql_exec/value.dart';
+import 'package:gql_tristate_value/gql_tristate_value.dart';
 import "package:test/test.dart";
 
 import 'package:end_to_end_test_tristate/graphql/__generated__/schema.schema.gql.dart';

codegen/end_to_end_test_tristate/test/schema/input_test.dart

@@ -1,4 +1,4 @@
-import 'package:gql_exec/value.dart';
+import 'package:gql_tristate_value/gql_tristate_value.dart';
 import "package:test/test.dart";
 
 import 'package:end_to_end_test_tristate/graphql/__generated__/schema.schema.gql.dart';

codegen/end_to_end_test_tristate/test/schema/scalars_test.dart

@@ -1,5 +1,5 @@
 import 'package:built_collection/built_collection.dart';
-import 'package:gql_exec/value.dart';
+import 'package:gql_tristate_value/gql_tristate_value.dart';
 import "package:test/test.dart";
 
 import 'package:end_to_end_test_tristate/graphql/__generated__/schema.schema.gql.dart';

codegen/gql_build/lib/src/utils/config.dart

@@ -74,7 +74,7 @@ Map<String, String> enumFallbackMap(final dynamic enumFallbacks) {
 }
 
 TriStateValueConfig triStateOptionalsConfig(Map<String, dynamic> config) {
-  final configValue = config["tristate_optionals"];
+  final Object? configValue = config["tristate_optionals"];
 
   if (configValue is bool) {
     return configValue

codegen/gql_build/pubspec.yaml

@@ -17,9 +17,19 @@ dependencies:
   gql_code_builder: ^0.8.0
   path: ^1.8.0
   yaml: ^3.1.0
+  gql_tristate_value: ^1.0.0
 dev_dependencies: 
   build_test: ^2.0.0
   gql_pedantic: ^1.0.2
+dependency_overrides: 
+  gql_tristate_value: 
+    path: ../gql_tristate_value
+  gql_code_builder: 
+    path: ../gql_code_builder
+  end_to_end_test_tristate: 
+    path: ../end_to_end_test_tristate
+  end_to_end_test: 
+    path: ../end_to_end_test
 topics: 
   - graphql
   - gql

codegen/gql_code_builder/lib/src/common.dart

@@ -3,6 +3,7 @@ import "package:code_builder/code_builder.dart";
 import "package:collection/collection.dart";
 import "package:gql/ast.dart";
 import "package:gql_code_builder/src/config/tristate_optionals_config.dart";
+import "package:gql_code_builder/src/tristate_optionals.dart";
 
 import "../source.dart";
 
@@ -213,8 +214,8 @@ Method buildOptionalGetter({
   final optionalGetter = baseGetter.rebuild((b) => b
     ..returns = TypeReference((b2) => b2
       ..isNullable = true
-      ..url = "package:gql_exec/value.dart"
-      ..symbol = "Value"
+      ..url = valueTypeUrl
+      ..symbol = valueTypeSymbol
       ..types.add((baseGetter.returns as TypeReference)
           .rebuild((b3) => b3..isNullable = false))));
   return optionalGetter;

codegen/gql_code_builder/lib/src/tristate_optionals.dart

@@ -262,7 +262,7 @@ String _generateTypeCast(TypeReference ref, Allocator allocator) {
 }
 
 const valueTypeSymbol = "Value";
-const valueTypeUrl = "package:gql_exec/value.dart";
+const valueTypeUrl = "package:gql_tristate_value/gql_tristate_value.dart";
 
 final valueTypeRef = TypeReference(
   (b) => b

codegen/gql_code_builder/pubspec.yaml

@@ -13,10 +13,20 @@ dependencies:
   gql: ^1.0.0
   gql_exec: ^1.0.0
   path: ^1.8.0
+  gql_tristate_value: ^1.0.0
 dev_dependencies: 
   build_runner: ^2.1.0
   gql_pedantic: ^1.0.2
   test: ^1.16.8
+dependency_overrides: 
+  gql_tristate_value: 
+    path: ../gql_tristate_value
+  gql_build: 
+    path: ../gql_build
+  end_to_end_test_tristate: 
+    path: ../end_to_end_test_tristate
+  end_to_end_test: 
+    path: ../end_to_end_test
 topics: 
   - graphql
   - gql

codegen/gql_tristate_value/.gitignore

@@ -0,0 +1,3 @@
+# https://dart.dev/guides/libraries/private-files
+# Created by `dart pub`
+.dart_tool/

codegen/gql_tristate_value/CHANGELOG.md

@@ -0,0 +1,3 @@
+## 1.0.0
+
+- Initial version.

codegen/gql_tristate_value/README.md

@@ -0,0 +1,9 @@
+
+A `Value` class used to represent the three possible states:
+- absence of a value
+- presence of a value that is `null`
+- presence of a non-null value
+
+This class is used by the generated code for GraphQL variables and input types that are nullable
+in order to distinguish between the absence of a value and the presence of a `null` value, typically
+used for "update" Mutations.

codegen/gql_tristate_value/analysis_options.yaml

@@ -0,0 +1 @@
+include: package:lints/recommended.yaml

codegen/gql_tristate_value/lib/gql_tristate_value.dart

@@ -0,0 +1 @@
+export 'src/value.dart';

codegen/gql_tristate_value/lib/src/value.dart

@@ -0,0 +1,54 @@
+/// A nullable value that may or may not be present.
+/// This is used to represent three possible states:
+/// - The value is absent. It will not be serialized.
+/// - The value is present and null. It will be serialized as null.
+/// - The value is present and non-null. It will be serialized as the value.
+sealed class Value<T extends Object> {
+  const Value._();
+
+  /// The value is absent. It will not be serialized.
+  const factory Value.absent() = AbsentValue;
+
+  /// The value is present. It may still be be null.
+  /// If the value is null, it will be serialized as null.
+  /// If the value is non-null, it will be serialized as the value.
+  const factory Value.present(T? value) = PresentValue<T>;
+
+  /// Returns the value if present (no matter if null or non-null), otherwise throws a [StateError].
+  T? get requireValue => switch (this) {
+        PresentValue(:final value) => value,
+        AbsentValue() => throw StateError("Value is absent"),
+      };
+
+  /// return the value if present and non-null, otherwise null.
+  /// this is useful for cases where you don't care between the value being absent or null.
+  T? get valueOrNull => switch (this) {
+        PresentValue(:final value) => value,
+        AbsentValue() => null,
+      };
+
+  bool get isPresent => switch (this) {
+        PresentValue() => true,
+        AbsentValue() => false,
+      };
+}
+
+class AbsentValue extends Value<Never> {
+  const AbsentValue() : super._();
+}
+
+class PresentValue<T extends Object> extends Value<T> {
+  final T? value;
+
+  const PresentValue(this.value) : super._();
+
+  @override
+  bool operator ==(Object other) =>
+      identical(this, other) ||
+      other is PresentValue<T> &&
+          runtimeType == other.runtimeType &&
+          value == other.value;
+
+  @override
+  int get hashCode => value.hashCode;
+}

codegen/gql_tristate_value/pubspec.yaml

@@ -0,0 +1,23 @@
+name: gql_tristate_value
+version: 1.0.0
+description: A wrapper class for representing a value that can be absent, present and null, or present and non-null.
+repository: https://github.com/gql-dart/gql
+environment: 
+  sdk: '>=3.0.0 <4.0.0'
+
+
+dev_dependencies:
+  test: ^1.16.8
+dependency_overrides: 
+  gql_code_builder: 
+    path: ../gql_code_builder
+  gql_build: 
+    path: ../gql_build
+  end_to_end_test_tristate: 
+    path: ../end_to_end_test_tristate
+  end_to_end_test: 
+    path: ../end_to_end_test
+topics: 
+  - graphql
+  - gql
+  - codegen

codegen/gql_tristate_value/test/value_test.dart

@@ -0,0 +1,34 @@
+import 'package:gql_tristate_value/gql_tristate_value.dart';
+import 'package:test/test.dart';
+
+void main() {
+  test("requireValue on absent throws", () {
+    final absent = const AbsentValue();
+
+    expect(() => absent.requireValue, throwsA(isA<StateError>()));
+  });
+
+  test("requireValue on present returns value", () {
+    final present = Value.present(42);
+
+    expect(present.requireValue, equals(42));
+  });
+
+  test("requireValue on null returns null", () {
+    final nullValue = Value<String>.present(null);
+
+    expect(nullValue.requireValue, isNull);
+  });
+
+  test("valueOrNull on absent returns null", () {
+    final absent = const AbsentValue();
+
+    expect(absent.valueOrNull, isNull);
+  });
+
+  test("valueOrNull on present returns value", () {
+    final present = Value.present(42);
+
+    expect(present.valueOrNull, equals(42));
+  });
+}

links/gql_exec/lib/src/value.dart

@@ -1,54 +1 @@
-/// A nullable value that may or may not be present.
-/// This is used to represent three possible states:
-/// - The value is absent. It will not be serialized.
-/// - The value is present and null. It will be serialized as null.
-/// - The value is present and non-null. It will be serialized as the value.
-sealed class Value<T extends Object> {
-  const Value._();
 
-  /// The value is absent. It will not be serialized.
-  const factory Value.absent() = AbsentValue;
-
-  /// The value is present. It may still be be null.
-  /// If the value is null, it will be serialized as null.
-  /// If the value is non-null, it will be serialized as the value.
-  const factory Value.present(T value) = PresentValue<T>;
-
-  /// Returns the value if present (no matter if null or non-null), otherwise throws a [StateError].
-  T? get requireValue => switch (this) {
-        PresentValue(:final value) => value,
-        AbsentValue() => throw StateError("Value is absent"),
-      };
-
-  /// return the value if present and non-null, otherwise null.
-  /// this is useful for cases where you don't care between the value being absent or null.
-  T? get valueOrNull => switch (this) {
-        PresentValue(:final value) => value,
-        AbsentValue() => null,
-      };
-
-  bool get isPresent => switch (this) {
-        PresentValue() => true,
-        AbsentValue() => false,
-      };
-}
-
-class AbsentValue extends Value<Never> {
-  const AbsentValue() : super._();
-}
-
-class PresentValue<T extends Object> extends Value<T> {
-  final T? value;
-
-  const PresentValue(this.value) : super._();
-
-  @override
-  bool operator ==(Object other) =>
-      identical(this, other) ||
-      other is PresentValue<T> &&
-          runtimeType == other.runtimeType &&
-          value == other.value;
-
-  @override
-  int get hashCode => value.hashCode;
-}

links/gql_exec/pubspec.yaml

@@ -1,5 +1,5 @@
 name: gql_exec
-version: 1.0.0
+version: 1.1.0
 description: Basis for GraphQL execution layer to support Link and Client.
 repository: https://github.com/gql-dart/gql
 environment: