-
Notifications
You must be signed in to change notification settings - Fork 14
Description
While reviewing the Trust Mark section of the spec, I’d like to suggest clarifying the naming of the trust_mark_id field.
Currently, trust_mark_id refers to the identifier of the class or definition of a Trust Mark.
However, the _id suffix commonly implies an instance identifier, which in this case can lead to confusion. For example, it’s easy to misinterpret trust_mark_id as referring to a specific, issued trust mark (the JWT), rather than the type or class it belongs to.
In a past discussion with @peppelinux, the idea came up that this value is closer to a "Trust Mark Type" identifier. Therefore, naming it trust_mark_type_id (or trust_mark_definition_id, or simply trust_mark_type) might better reflect its role as a reference to the Trust Mark's category, and not to an issued Trust Mark instance.
@peppelinux acknowledged this as a semantic point and mentioned that trust_mark_type_id could be a better fitting name.
Proposed change: Rename trust_mark_id to something more precise, such as:
trust_mark_type_idtrust_mark_definition_id- or even
trust_mark_type
This would more clearly signal that the field is identifying a class of trust mark (not an issued instance), improving developer understanding without affecting semantics.
Making this change would not only improve clarity, but also lay the groundwork for potential features like a Trust Mark definition repository, where each trust_mark_type_id could be enriched with metadata, documentation, or validation rules.
Metadata
Metadata
Assignees
Labels
Type
Projects
Status