Custom Capitalizations

[MrAlias]

The Go team currently uses the following capitalization for known acronyms and initialisms:

• ACL
• AI
• AIX
• AKS
• AMD64
• API
• ARM32
• ARM64
• ARN
• ARNs
• ASCII
• ASPNETCore
• AWS
• CICD
• CPP
• CPU
• CSS
• CosmosDB
• CouchDB
• CronJob
• DB
• DC
• DNS
• DaemonSet
• DragonflyBSD
• DynamoDB
• EC2
• ECS
• EDB
• EKS
• EOF
• FaaS
• FirstSQL
• FreeBSD
• GC
• GCP
• GRPC
• GUID
• HBase
• HPUX
• HSQLDB
• HTML
• HTTP
• HTTPS
• HanaDB
• IA64
• ID
• IP
• InProc
• InstantDB
• JDBC
• JSON
• JVM
• K8S
• LHS
• MSSQL
• MariaDB
• MaxDB
• MongoDB
• MySQL
• NetBSD
• OS
• OTel
• OpenBSD
• PHP
• PID
• PPC32
• PPC64
• PostgreSQL
• QPS
• QUIC
• RAM
• RHS
• RPC
• ReplicaSet
• SDK
• SLA
• SMTP
• SPDY
• SQL
• SSH
• StatefulSet
• TCP
• TLS
• TTL
• UDP
• UI
• UID
• URI
• URL
• UTF8
• UUID
• V8JS
• VCS
• VM
• WebEngine
• XML
• XMPP
• XSRF
• XSS
• ZOS

The current build tooling ensures these capitalizations are used in any naming generated. How can we keep this behavior when using weaver?

[lquerel]

There is a section in the Weaver config just for that and a corresponding Jinja filter :-)

Exemple:

{{ attr.brief | acronym }}

[MrAlias]

Ah, thanks. I missed that. I'll give it a try.

[MrAlias]

It looks like that is already being used:

https://github.com/open-telemetry/opentelemetry-go/pull/5793/files#diff-c27c5af791853e678c1931568a666a2a468a25035fd7412fb8907f67ccad89d1R52-R146

https://github.com/open-telemetry/opentelemetry-go/pull/5793/files#diff-f65227d0a0712ab4e3e66dbedcbac72947e6e310b0d4317c1a5d5f26fee96009R2

But it is still generating invalid names. For example:

• Aws instead of AWS: https://github.com/open-telemetry/opentelemetry-go/pull/5793/files#diff-947b803ff8dbcd43bbbfd5c66bc4a0dd826b4754ec035b29f4ec867d42acc781R381
• Db instead of DB: https://github.com/open-telemetry/opentelemetry-go/pull/5793/files#diff-947b803ff8dbcd43bbbfd5c66bc4a0dd826b4754ec035b29f4ec867d42acc781R2055

[lquerel]

That’s strange. The latest links you added to your last comment aren’t working for me. Could you point me to the file and the line where you observed the issue for AWS? I’d like to determine if it’s a flaw in the acronym filter or a usage problem.

[MrAlias]

@lquerel thanks for the response.

Those files are from this PR: open-telemetry/opentelemetry-go#5793

I'm working in that branch and able to reproduce the results generated there. I have updated to use v0.10.0 as well and still not able to remedy the naming.

[MrAlias]

FWIW, I have also tried moving the acronym function earlier in the processing:

{%- macro to_go_name(fqn) -%}
{{ fqn | acronym | title_case | replace(" ", "") }}
{%- endmacro -%}

Still no change.

[MrAlias]

Removing the title_case function resulted in things like the following to be produced:

AWS.DynamoDB.consistent_readKey

Which seems to show that the acronym function is working but not interacting well with the title_case.

Is there a way to have these work together?

[lquerel]

Could you try the following macro

{%- macro to_go_name(fqn) -%}
{{ fqn | title_case | acronym | replace(" ", "") }}
{%- endmacro -%}

Applying the title_case filter after the acronym filter destroys the effect of the acronym filter.
If that doesn't work please copy/paste the result of fqn | title_case without anything else. Thanks

[MrAlias]

{%- macro to_go_name(fqn) -%}
{{ fqn | title_case | acronym | replace(" ", "") }}
{%- endmacro -%}

AwsDynamodbConsistentReadKey

{%- macro to_go_name(fqn) -%}
{{ fqn | title_case }}
{%- endmacro -%}

AwsDynamodbConsistentReadKey

[lquerel]

Okay, I see. The acronym filter works at the word boundary level, while the title_case filter transforms, in this case, a dot notation (e.g., aws.dynamodb.consistent.readKey) into a single word (e.g., AwsDynamodbConsistentReadKey). I need to consider the best way to fix this issue. I’ll include a fix in the next release.