cardano-foundation / CIPs

https://cips.cardano.org/
Creative Commons Attribution 4.0 International
462 stars 305 forks source link

CIP-0010 | Add schema links #838

Open Godspeed-exe opened 1 month ago

Godspeed-exe commented 1 month ago

Hello guys!

As mentioned on my previous PR over the weekend I would like to add a schema attribute to the registry.json file. Initially as an optional attribute.

Problem today

Currently there is no way for anyone (or a machine / script / application) to know which CIP (actual documentation of a metadata standard) is connected to which metadata label. Or vice versa: if you look at metadata label 721, how do you know that you can find it's details in CIP-0025?

Proposed solution

Another solution would be to just point to the CIP folder and put the validators on a fixed position. (CIP-?????/schema_validators) Perhaps it's wise to make schema an array, in case there are updates / new versions which are also linked to this metadata label.

What do you think?

Crypto2099 commented 4 weeks ago

Interesting and I am in favor of the proposed change to start using CI/CD actions to validate that, particularly things like our registry CIPs, are being updated correctly by the authors.

There is an interesting point about versioning of some schemes but I think that might be overkill as generally speaking you probably just want a top-level pointer to say: This one goes to CIP-25 and going to that branch in its current state should be sufficient. Since the rest of the CIP repository is not "machine readable" in general anyway I think there will always be some amount of "human intervention" required.

Godspeed-exe commented 3 weeks ago

Hello @Crypto2099

Fully agree, my proposal is only focused on CIP-0010 metadata label entries.

Mostly trying to improve: "Where does an application that uses these metadata labels find the correct matching schema definition (in the stack of all CIP's). Currently there is no (machine readable or human) link between CIP0025 and metadata label 721) and how can it validate it."