镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Add the identity data structure draft for OpenVEX v1 - #70

Open
puerco wants to merge 1 commit into
openvex:mainfrom
puerco:identity
Open

puerco wants to merge 1 commit into
openvex:mainfrom
puerco:identity

Conversation

@puerco

@puerco puerco commented Oct 30, 2026

Copy link
Copy Markdown
Member

Part of our roadmap towards openvex v1

(This is a second PR woth the sam doc after #68 was automerged without reviews)

This PR adds the definition for an identity struct that names a person, organization,tool or agent with a canonical IRI, a display name, a type and a map of identifiers in external systems. We now make authors, suppliers and tooling accept it alongside the string form they take today. Include the JSON-LD context and JSON schema changes and the compatibility rules.

This spec is intended to land as an independent document to be merged when we compile the v1 specification. Merging it now does not make it part of the openvex spec.

/hold
/cc @macedogm @cpanato

This commit adds the definition for an identity struct that names a
person, organization,tool or agent with a canonical IRI, a display
name, a type and a map of identifiers in external systems. We now
make authors, suppliers and tooling accept it alongside the string
form they take today. Include the JSON-LD context and JSON schema
changes and the compatibility rules.

Signed-off-by: Adolfo García Veytia (Puerco) <puerco@carabiner.dev>
@miniprow miniprow Bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command lgtm Indicates that a PR is ready to be merged (Looks Good To Me) approved Indicates a PR has been approved by an approver from all required OWNERS files labels Oct 30, 2026
@miniprow

miniprow Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cpanato, macedogm, puerco

The full list of commands accepted by this bot can be found here.

Details Approved by a repository administrator.

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cpanato cpanato left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@macedogm macedogm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but I find that keeping the role field could be confusing with the new author field, but removing it would be a breaking change.

@macedogm

Copy link
Copy Markdown
Member

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command lgtm Indicates that a PR is ready to be merged (Looks Good To Me)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants