From 96f1849752a3f67d6f66bf4ae156d198cb37aa03 Mon Sep 17 00:00:00 2001
From: alexander-yevsyukov <alexander.yevsyukov@teamdev.com>
Date: Thu, 12 Sep 2024 19:40:19 +0100
Subject: [PATCH 1/5] Consider interfaces annotated `@GeneratedMixin` used

---
 .idea/inspectionProfiles/Project_Default.xml | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/.idea/inspectionProfiles/Project_Default.xml b/.idea/inspectionProfiles/Project_Default.xml
index 35bbdff59d..b29432f4cc 100644
--- a/.idea/inspectionProfiles/Project_Default.xml
+++ b/.idea/inspectionProfiles/Project_Default.xml
@@ -338,6 +338,11 @@
     </inspection_tool>
     <inspection_tool class="InterfaceNeverImplemented" enabled="true" level="WARNING" enabled_by_default="true">
       <option name="ignoreInterfacesThatOnlyDeclareConstants" value="true" />
+      <option name="ignorableAnnotations">
+        <value>
+          <item value="io.spine.annotation.GeneratedMixin" />
+        </value>
+      </option>
     </inspection_tool>
     <inspection_tool class="IteratorNextDoesNotThrowNoSuchElementException" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="JDBCExecuteWithNonConstantString" enabled="true" level="WARNING" enabled_by_default="true" />
@@ -578,7 +583,6 @@
     </inspection_tool>
     <inspection_tool class="NonSerializableObjectBoundToHttpSession" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="NonSerializableObjectPassedToObjectStream" enabled="true" level="WARNING" enabled_by_default="true" />
-    <inspection_tool class="NonSerializableWithSerialVersionUIDField" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="NonSerializableWithSerializationMethods" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="NonShortCircuitBoolean" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="NonStaticFinalLogger" enabled="true" level="WARNING" enabled_by_default="true">
@@ -623,6 +627,9 @@
     </inspection_tool>
     <inspection_tool class="OctalAndDecimalIntegersMixed" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="OnDemandImport" enabled="true" level="WARNING" enabled_by_default="true" />
+    <inspection_tool class="OptionalOfNullableMisuse" enabled="true" level="WARNING" enabled_by_default="true">
+      <scope name="Production" level="WARNING" enabled="true" />
+    </inspection_tool>
     <inspection_tool class="OverloadedMethodsWithSameNumberOfParameters" enabled="true" level="WARNING" enabled_by_default="true">
       <option name="ignoreInconvertibleTypes" value="true" />
     </inspection_tool>
@@ -690,7 +697,6 @@
       <option name="ignoreSerializable" value="false" />
       <option name="ignoreCloneable" value="false" />
     </inspection_tool>
-    <inspection_tool class="RedundantMethodOverride" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="ResultOfObjectAllocationIgnored" enabled="true" level="WARNING" enabled_by_default="true">
       <scope name="Tests" level="WARNING" enabled="false" />
     </inspection_tool>
@@ -711,7 +717,6 @@
     <inspection_tool class="SamePackageImport" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="SameParameterValue" enabled="true" level="TYPO" enabled_by_default="true" />
     <inspection_tool class="SerialPersistentFieldsWithWrongSignature" enabled="true" level="WARNING" enabled_by_default="true" />
-    <inspection_tool class="SerialVersionUIDNotStaticFinal" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="SerializableHasSerialVersionUIDField" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="SerializableInnerClassHasSerialVersionUIDField" enabled="true" level="WARNING" enabled_by_default="true">
       <option name="ignoreAnonymousInnerClasses" value="false" />
@@ -750,13 +755,10 @@
     <inspection_tool class="StringBufferToStringInConcatenation" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="StringConcatenationInFormatCall" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="StringConcatenationInMessageFormatCall" enabled="true" level="WARNING" enabled_by_default="true" />
-    <inspection_tool class="StringEqualsEmptyString" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="StringReplaceableByStringBuffer" enabled="true" level="WARNING" enabled_by_default="true">
       <option name="onlyWarnOnLoop" value="true" />
     </inspection_tool>
     <inspection_tool class="SubtractionInCompareTo" enabled="true" level="WARNING" enabled_by_default="true" />
-    <inspection_tool class="SuspiciousIndentAfterControlStatement" enabled="true" level="WARNING" enabled_by_default="true" />
-    <inspection_tool class="SwitchStatementWithConfusingDeclaration" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="SwitchStatementsWithoutDefault" enabled="true" level="WARNING" enabled_by_default="true">
       <option name="m_ignoreFullyCoveredEnums" value="true" />
     </inspection_tool>
@@ -790,7 +792,6 @@
     <inspection_tool class="ThrowableNotThrown" enabled="true" level="WARNING" enabled_by_default="true">
       <scope name="Tests" level="WARNING" enabled="false" />
     </inspection_tool>
-    <inspection_tool class="ThrowablePrintStackTrace" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="ThrownExceptionsPerMethod" enabled="true" level="WARNING" enabled_by_default="true">
       <option name="m_limit" value="3" />
     </inspection_tool>
@@ -811,7 +812,6 @@
     <inspection_tool class="TransientFieldInNonSerializableClass" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="TransientFieldNotInitialized" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="TrivialIf" enabled="true" level="TYPO" enabled_by_default="true" />
-    <inspection_tool class="TrivialStringConcatenation" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="TsLint" enabled="true" level="WARNING" enabled_by_default="true" />
     <inspection_tool class="TypeMayBeWeakened" enabled="true" level="INFO" enabled_by_default="true">
       <scope name="Problems" level="INFO" enabled="true">

From 47b216fce9d7cbf5a562eb17f226d4efb62c0bc6 Mon Sep 17 00:00:00 2001
From: alexander-yevsyukov <alexander.yevsyukov@teamdev.com>
Date: Thu, 12 Sep 2024 19:40:27 +0100
Subject: [PATCH 2/5] Update dependency reports

---
 dependencies.md | 4 ++--
 pom.xml         | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/dependencies.md b/dependencies.md
index 490f4e648d..7b44450818 100644
--- a/dependencies.md
+++ b/dependencies.md
@@ -1,6 +1,6 @@
 
 
-# Dependencies of `io.spine:spine-base:2.0.0-SNAPSHOT.208`
+# Dependencies of `io.spine:spine-base:2.0.0-SNAPSHOT.210`
 
 ## Runtime
 1.  **Group** : com.google.code.findbugs. **Name** : jsr305. **Version** : 3.0.2.
@@ -865,4 +865,4 @@
 
 The dependencies distributed under several licenses, are used according their commercial-use-friendly license.
 
-This report was generated on **Tue Sep 10 17:37:13 CEST 2024** using [Gradle-License-Report plugin](https://github.com/jk1/Gradle-License-Report) by Evgeny Naumenko, licensed under [Apache 2.0 License](https://github.com/jk1/Gradle-License-Report/blob/master/LICENSE).
\ No newline at end of file
+This report was generated on **Thu Sep 12 19:33:08 WEST 2024** using [Gradle-License-Report plugin](https://github.com/jk1/Gradle-License-Report) by Evgeny Naumenko, licensed under [Apache 2.0 License](https://github.com/jk1/Gradle-License-Report/blob/master/LICENSE).
\ No newline at end of file
diff --git a/pom.xml b/pom.xml
index d52a0caa5b..1a108772c1 100644
--- a/pom.xml
+++ b/pom.xml
@@ -10,7 +10,7 @@ all modules and does not describe the project structure per-subproject.
  -->
 <groupId>io.spine</groupId>
 <artifactId>base</artifactId>
-<version>2.0.0-SNAPSHOT.208</version>
+<version>2.0.0-SNAPSHOT.210</version>
 
 <inceptionYear>2015</inceptionYear>
 

From 67b253818e3f85db9f7f9267471ed30a51967d12 Mon Sep 17 00:00:00 2001
From: alexander-yevsyukov <alexander.yevsyukov@teamdev.com>
Date: Thu, 12 Sep 2024 19:40:40 +0100
Subject: [PATCH 3/5] Bump version -> `2.0.0-SNAPSHOT.210`

---
 version.gradle.kts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/version.gradle.kts b/version.gradle.kts
index e8ff6f34a9..7d5a6fba60 100644
--- a/version.gradle.kts
+++ b/version.gradle.kts
@@ -24,4 +24,4 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-val versionToPublish: String by extra("2.0.0-SNAPSHOT.208")
+val versionToPublish: String by extra("2.0.0-SNAPSHOT.210")

From 579c48321519c3814aac98cc0dd4e681d62fd395 Mon Sep 17 00:00:00 2001
From: alexander-yevsyukov <alexander.yevsyukov@teamdev.com>
Date: Thu, 12 Sep 2024 19:51:02 +0100
Subject: [PATCH 4/5] Deprecate `constraint_for` option

Also:
 * Improve docs layout.
---
 src/main/proto/spine/options.proto | 100 +++++++++++++++++------------
 1 file changed, 59 insertions(+), 41 deletions(-)

diff --git a/src/main/proto/spine/options.proto b/src/main/proto/spine/options.proto
index 04dfd5773f..27ebf3b476 100644
--- a/src/main/proto/spine/options.proto
+++ b/src/main/proto/spine/options.proto
@@ -232,8 +232,8 @@ extend google.protobuf.FieldOptions {
     // An API bearing this annotation is exempt from any compatibility guarantees made by its
     // containing library. Note that the presence of this annotation implies nothing about the
     // quality of the API in question, only the fact that it is not "API-frozen."
-    // It is generally safe for applications to depend on beta APIs, at the cost of some extra work
-    // during upgrades.
+    // It is generally safe for applications to depend on beta APIs, at the cost of
+    // some extra work during upgrades.
     //
     bool beta = 73853;
 
@@ -259,9 +259,9 @@ extend google.protobuf.FieldOptions {
     //
     // All column fields are considered optional by the framework.
     //
-    // Currently, only entities of projection and process manager type are eligible for having
-    // columns (see `EntityOption`). For all other message types the column declarations are
-    // ignored.
+    // Currently, only entities of projection and process manager type are
+    // eligible for having columns (see `EntityOption`).
+    // For all other message types the column declarations are ignored.
     //
     // The `repeated` and `map` fields cannot be columns.
     //
@@ -276,8 +276,8 @@ extend google.protobuf.OneofOptions {
 
     // Marks a `oneof` group, in which one field *must* be set.
     //
-    // Alternative to `(required_field)` with all the field in the group joined with the OR
-    // operator.
+    // Alternative to `(required_field)` with all the field in the group
+    // joined with the OR operator.
     //
     bool is_required = 73891;
 
@@ -295,9 +295,10 @@ extend google.protobuf.MessageOptions {
     // The number of parameters and their types are determined by the type of field options.
     //
     // Usage of this value is deprecated. Along with the old `msg_format`s, it exists to support
-    // the old validation library. The new version of the validation library, which does not lie in
-    // the `base` repository, constructs the default error messages separately when creating
-    // language-agnostic validation rules.
+    // the old version of the Validation library.
+    // The new version of the Validation library, which does not lie in the `base` repository,
+    // constructs the default error messages separately when creating language-agnostic
+    // validation rules.
     //
     string default_message = 73901 [deprecated = true];
 
@@ -329,6 +330,8 @@ extend google.protobuf.MessageOptions {
 
     // An external validation constraint for a field.
     //
+    // WARNING: This option is deprecated and is scheduled for removal in Spine v2.0.0.
+    //
     // Allows to re-define validation constraints for a message when its usage as a field of
     // another type requires alternative constraints. This includes definition of constraints for
     // a message which does not have them defined within the type.
@@ -408,7 +411,7 @@ extend google.protobuf.MessageOptions {
     //       Spine Model Compiler does not check such an "overwriting".
     //       See the issue: https://github.com/SpineEventEngine/base/issues/318.
     //
-    string constraint_for = 73904;
+    string constraint_for = 73904 [deprecated = true];
 
     // Reserved 73905 to 73910 for future validation options.
 
@@ -456,8 +459,10 @@ extend google.protobuf.MessageOptions {
     //
     // In the example above, `CreateProject` message is a `ProjectCommand`.
     //
-    // To specify a characteristic for every message in a `.proto` file, use `(every_is)` file
-    // option. If both `(is)` and `(every_is)` options are found, both are applied.
+    // To specify a characteristic for every message in a `.proto` file,
+    // please use `(every_is)` file option.
+    //
+    // If both `(is)` and `(every_is)` options are applicable for a type, both are applied.
     //
     // When targeting Java, specify the name of a Java interface to be implemented by this
     // message via `(is).java_type`.
@@ -545,13 +550,13 @@ extend google.protobuf.FileOptions {
     // In the example above, `CreateProject`, `CreateProject.WithAssignee`, and `DeleteProject`
     // messages are `ProjectCommand`-s.
     //
-    // To specify a characteristic for a single message, use `(is)` message option. If both `(is)`
-    // and `(every_is)` options are found, both are applied.
+    // To specify a characteristic for a single message, please use `(is)` message option.
+    // If both `(is)` and `(every_is)` options are applicable for a type, both are applied.
     //
     // When targeting Java, specify the name of a Java interface to be implemented by these
     // message types via `(every_is).java_type`.
     //
-    IsOption every_is = 73946;
+    EveryIsOption every_is = 73946;
 
     // Reserved 73947 to 73970 for future use.
 }
@@ -896,53 +901,66 @@ message EntityOption {
     Visibility visibility = 2;
 }
 
-// Defines a marker for a given type or a set of types.
+// Defines a common type for message types declared in the same proto file.
 //
-// The option may be used in two modes:
-//   - with the marker code generation;
-//   - without the marker code generation.
+// The nature of the type depends on the target programming language.
+// For example, the `java_type` property defines a name of the Java interface common
+// to all message classes generated for the proto file having this option.
 //
-// When used with the code generation, language-specific markers are generated by the Protobuf
-// compiler. Otherwise, it is expected that the user creates such markers manually.
+// The option triggers creation of the common type if the `generate` property is set to true.
+// Otherwise, it is expected that the user provides the reference to an existing type.
 //
-message IsOption {
+message EveryIsOption {
 
-    // Enables the generation of marker interfaces.
+    // Enables the generation of the common type.
     //
-    // Creation of the marker interfaces is disabled by default.
+    // The default value is `false`.
     //
     bool generate = 1;
 
     // The reference to a Java top-level interface.
     //
     // The interface cannot be nested into a class or another interface.
-    // If a nested interface is provided, the code generation is likely to produce
-    // uncompilable code.
+    // If a nested interface is provided, the code generation should fail the build process.
     //
     // The value may be a fully-qualified or a simple name.
     //
-    // When a simple name is set, the package of the interface matches the
-    // package of the Java class generated for message types to which this option applies.
+    // When a simple name is set, it is assumed that the interface belongs to
+    // the package of the generated message classes.
+    //
+    // If the value of the `generate` field is set to `false` the referenced interface must exist.
+    // Otherwise, a compilation error will occur.
     //
     // If the value of the `generate` field is set to `true`, the framework will
-    // generate the interface using the given name.
+    // generate the interface using the given name and the package as described above.
     //
-    // Otherwise, the code will be generated assuming the that the referenced interface
-    // already exists. If this is not the case, compilation error will occur.
+    // The generated interface will extend `com.google.protobuf.Message` and
+    // will have no declared methods.
+    //
+    string java_type = 2;
+}
+
+// Defines additional type for a message type in which this option is declared.
+//
+// The nature of the type depends on the target programming language.
+// For example, the `java_type` property defines a name of the Java interface which
+// the generated message class will implement.
+//
+message IsOption {
+
+    // The reference to a Java top-level interface.
     //
-    // The interface will extend `com.google.protobuf.Message`.
+    // The interface cannot be nested into a class or another interface.
+    // If a nested interface is provided, the code generation should fail the build process.
     //
-    // The generated interface will have no declared methods.
+    // The value may be a fully-qualified or a simple name.
     //
-    // If the value of the option contains the Java package name separator (`.`) AND
-    // the value of the `generate` property is `true`, the Model Compiler will create
-    // a new Java file using the package of the referenced interface (either direct or
-    // implied from the options of the proto file).
+    // When a simple name is set, it is assumed that the interface belongs to
+    // the package of the generated message classes.
     //
-    // If both `(is)` and `(every_is)` options specify a Java interface, the message class
-    // implements both interfaces.
+    // The referenced interface must exist. Otherwise, a compilation error will occur.
     //
-    string java_type = 2;
+    string java_type = 1;
 }
 
 // Defines the way to compare two messages of the same type to one another.

From e0a58cfb5af0b25f6cf73a9d821e496f37b3a6d3 Mon Sep 17 00:00:00 2001
From: alexander-yevsyukov <alexander.yevsyukov@teamdev.com>
Date: Thu, 12 Sep 2024 19:51:08 +0100
Subject: [PATCH 5/5] Update build time

---
 dependencies.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dependencies.md b/dependencies.md
index 7b44450818..14fbc74009 100644
--- a/dependencies.md
+++ b/dependencies.md
@@ -865,4 +865,4 @@
 
 The dependencies distributed under several licenses, are used according their commercial-use-friendly license.
 
-This report was generated on **Thu Sep 12 19:33:08 WEST 2024** using [Gradle-License-Report plugin](https://github.com/jk1/Gradle-License-Report) by Evgeny Naumenko, licensed under [Apache 2.0 License](https://github.com/jk1/Gradle-License-Report/blob/master/LICENSE).
\ No newline at end of file
+This report was generated on **Thu Sep 12 19:50:56 WEST 2024** using [Gradle-License-Report plugin](https://github.com/jk1/Gradle-License-Report) by Evgeny Naumenko, licensed under [Apache 2.0 License](https://github.com/jk1/Gradle-License-Report/blob/master/LICENSE).
\ No newline at end of file