Merge pull request #277 from Sherlouk/resilient-script

Improve SemanticConvention generation template:

At the moment, if you run the generate script, it will make changes to the project which will cause compiler or test failures.

The three failures which this protects against:

The value "int" being used as an enum value. This needs to be "Int" instead.
The keyword "internal" being used as a case value. This should be wrapped with backticks like internal.
The Swift telemetry language which we have chosen to add, outside of the specification.
While not technically a failure, this change will also revert the documentation changes made in a previous PR which changed all the Int values to "int" incorrectly. This fixes that too.

Author: Ignacio Bonafonte

Scripts/semantic-convention/templates/SemanticAttributes.swift.j2

@@ -13,6 +13,8 @@
     Bool
   {%- elif type == "number" -%}
     Int
+  {%- elif type == "int" -%}
+    Int
   {%- else -%}
     {{type}}
   {%- endif -%}
@@ -26,6 +28,7 @@ import Foundation
 public enum {{enum}}: String {
 
     {%- for attribute in attributes | unique(attribute="fqn") %}
+    {%- set class_name = attribute.fqn | to_camelcase(True) ~ "Values" %}
     /**
     {{attribute.brief | to_doc_brief}}.
     {%- if attribute.examples %}
@@ -50,7 +53,11 @@ public enum {{enum}}: String {
     - Warning: Deprecated • {{attribute.deprecated | to_doc_brief}}.
     {%- endif %}
 
+    {%- if attribute.is_enum %}
+    - Requires: Value should be one of [`{{enum}}.{{class_name}}`](x-source-tag://otel{{class_name}}) (of type `{{to_swift_return_type(attribute.attr_type|lower)}}`)
+    {% else %}
     - Requires: Value type should be `{{to_swift_return_type(attribute.attr_type|lower)}}`
+    {% endif -%}
     */
     {%- if attribute.deprecated %}
     @available(*, deprecated)
@@ -76,6 +83,7 @@ public enum {{enum}}: String {
     /**
     {{attribute.brief | to_doc_brief}}.
     */
+    /// - Tag: otel{{class_name}}
     {%- if attribute.attr_type.custom_values %}
     public struct {{class_name}}: CustomStringConvertible {
         {%- for member in attribute.attr_type.members %}
@@ -101,7 +109,7 @@ public enum {{enum}}: String {
         /**
         {{member.brief | to_doc_brief}}.
         */
-        case {{ member.member_id | replace(".", "_") }} = {{ print_value(type, member.value) }}
+        case {{ member.member_id | replace(".", "_") | replace("internal", "`internal`") }} = {{ print_value(type, member.value) }}
         {%- endfor %}
     }
     {% endif %}