feat: migrate @value annotation (#1148)

Closes #1145.

### Summary of Changes

- add migration for `@value` annotations to another version, with the
possibilities:
- If the variant of the annotation is omitted and both parameter have
the same value, the annotation will be migrated as expected. If the type
is different, the annotation will be marked as unsure.
- If the variant of the annotation is required and the parameter from
apiv2 has a default value of the same type, the annotation will be
migrated as expected.
- If the variant is optional or constant and the type of the parameter
from apiv2 is known and the type of the parameters match, an annotation
will be migrated as expected. If the type is unknown, the annotation
will be marked as unsure.

### Testing Instructions

run the migrate command or view and run the `test_migration.py` file

Co-authored-by: Aclrian <[email protected]>

Author: Aclrian

package-parser/package_parser/processing/migration/_migrate.py

@@ -10,6 +10,7 @@
     migrate_enum_annotation,
     migrate_rename_annotation,
     migrate_todo_annotation,
+    migrate_value_annotation,
 )
 from package_parser.processing.migration.model import Mapping
 
@@ -56,4 +57,10 @@ def migrate_annotations(
             for annotation in migrate_todo_annotation(todo_annotation, mapping):
                 migrated_annotation_store.add_annotation(annotation)
 
+    for value_annotation in annotationsv1.valueAnnotations:
+        mapping = _get_mapping_from_annotation(value_annotation, mappings)
+        if mapping is not None:
+            for annotation in migrate_value_annotation(value_annotation, mapping):
+                migrated_annotation_store.add_annotation(annotation)
+
     return migrated_annotation_store

package-parser/package_parser/processing/migration/annotations/__init__.py

@@ -3,3 +3,4 @@
 from ._migrate_enum_annotation import migrate_enum_annotation
 from ._migrate_rename_annotation import migrate_rename_annotation
 from ._migrate_todo_annotation import migrate_todo_annotation
+from ._migrate_value_annotation import migrate_value_annotation

package-parser/package_parser/processing/migration/annotations/_migrate_boundary_annotation.py

@@ -103,13 +103,23 @@ def migrate_boundary_annotation(
             ) = _contains_number_and_is_discrete(parameter.type)
             if parameter.type is None:
                 boundary_annotation.reviewResult = EnumReviewResult.UNSURE
+                boundary_annotation.comment = (
+                    migrate_text
+                    if len(boundary_annotation.comment) == 0
+                    else boundary_annotation.comment + "\n" + migrate_text
+                )
                 return [boundary_annotation]
             if parameter_expects_number:
                 if (
                     parameter_type_is_discrete
                     is not boundary_annotation.interval.isDiscrete
                 ):
                     boundary_annotation.reviewResult = EnumReviewResult.UNSURE
+                    boundary_annotation.comment = (
+                        migrate_text
+                        if len(boundary_annotation.comment) == 0
+                        else boundary_annotation.comment + "\n" + migrate_text
+                    )
                     boundary_annotation.interval = (
                         migrate_interval_to_fit_parameter_type(
                             boundary_annotation.interval, parameter_type_is_discrete
@@ -154,7 +164,9 @@ def migrate_boundary_annotation(
                             parameter.id,
                             authors,
                             boundary_annotation.reviewers,
-                            boundary_annotation.comment,
+                            migrate_text
+                            if len(boundary_annotation.comment) == 0
+                            else boundary_annotation.comment + "\n" + migrate_text,
                             EnumReviewResult.UNSURE,
                             migrate_interval_to_fit_parameter_type(
                                 boundary_annotation.interval,
@@ -168,7 +180,9 @@ def migrate_boundary_annotation(
                             parameter.id,
                             authors,
                             boundary_annotation.reviewers,
-                            boundary_annotation.comment,
+                            migrate_text
+                            if len(boundary_annotation.comment) == 0
+                            else boundary_annotation.comment + "\n" + migrate_text,
                             EnumReviewResult.UNSURE,
                             boundary_annotation.interval,
                         )

package-parser/package_parser/processing/migration/annotations/_migrate_enum_annotation.py

@@ -93,6 +93,11 @@ def migrate_enum_annotation(
                     return []
             else:
                 enum_annotation.reviewResult = EnumReviewResult.UNSURE
+                enum_annotation.comment = (
+                    migrate_text
+                    if len(enum_annotation.comment) == 0
+                    else enum_annotation.comment + "\n" + migrate_text
+                )
                 enum_annotation.target = parameter.id
                 return [enum_annotation]
         return [

package-parser/package_parser/processing/migration/annotations/_migrate_rename_annotation.py

@@ -52,9 +52,11 @@ def migrate_rename_annotation(
             ):
                 rename_annotation.target = element.id
                 rename_annotation.reviewResult = EnumReviewResult.UNSURE
-                if len(rename_annotation.comment) > 0:
-                    rename_annotation.comment += "\n"
-                rename_annotation.comment += migrate_text
+                rename_annotation.comment = (
+                    migrate_text
+                    if len(rename_annotation.comment) == 0
+                    else rename_annotation.comment + "\n" + migrate_text
+                )
                 return [rename_annotation]
             todo_annotations.append(
                 TodoAnnotation(