Skip to content

Conversation

@Sukuna0007Abhi
Copy link
Contributor

@Sukuna0007Abhi Sukuna0007Abhi commented Sep 25, 2025

This PR addresses issue #65 by updating the repository guide to include repositories that have been added to the Veraison organization in recent months.

Changes Made

  • Added book repository for compiled documentation in reader-friendly format
  • Included new Rust implementations of core libraries (rust-ccatoken, rust-cmw, corim-rs)
  • Added CLI tools for testing and verification (gen-testcase, cover)
  • Added CCA realm measurements library for attestation data handling
  • Included Python implementation of EAR alongside existing Go, Rust, and C versions
  • Added RATS daemon (ratsd) for message collection workflows
  • Added keybroker demo to showcase protocol implementation
  • Updated mermaid diagrams to accurately reflect current architecture
  • Reorganized demo projects into a dedicated section for better navigation

Impact

This update makes the repository guide a reliable reference for understanding the full scope of Veraison components. Contributors and users can now easily discover all available tools and libraries across different programming languages.

The changes maintain the existing structure while adding missing pieces that have grown the project ecosystem significantly over the past months.

Closes #65

Ready for Review sir @thomas-fossati sir @yogeshbdeshpande

- Added book repository for compiled documentation
- Added new Rust implementations: rust-ccatoken, rust-cmw, corim-rs
- Added new CLI tools: gen-testcase, cover
- Added new libraries: cca-realm-measurements, python-ear
- Added new services: ratsd (RATS daemon)
- Added new demo: keybroker-demo
- Updated mermaid diagrams to reflect new repositories
- Reorganized demos into separate section

Fixes veraison#65

Signed-off-by: Sukuna0007Abhi <appsonly310@gmail.com>
@yogeshbdeshpande
Copy link
Contributor

@Sukuna0007Abhi Thank you for this change, very helpful!!

Copy link
Contributor

@yogeshbdeshpande yogeshbdeshpande left a comment

Choose a reason for hiding this comment

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

While all looks good to me, while you are at this PR, could you please remove extra \n on some of the Role boxes when Mermaid gets rendered.

@Sukuna0007Abhi
Copy link
Contributor Author

Sukuna0007Abhi commented Sep 29, 2025

Sure sir @yogeshbdeshpande removing it under 10 minutes

- Cleaned up Supply Chain, Attester, and Relying Party role boxes
- Addresses feedback from @yogeshbdeshpande about extra \n in rendered Mermaid diagrams

Signed-off-by: Sukuna0007Abhi <appsonly310@gmail.com>
@Sukuna0007Abhi
Copy link
Contributor Author

Sukuna0007Abhi commented Sep 29, 2025

Ready for Review sir @yogeshbdeshpande sir @thomas-fossati

@yogeshbdeshpande
Copy link
Contributor

Thank you for a super-fast response. Just one more change needed on the Verifier Owner Box and then it Looks Good to Me!

@Sukuna0007Abhi
Copy link
Contributor Author

Sukuna0007Abhi commented Sep 29, 2025

Sure sir @yogeshbdeshpande thanks, and updating it under 10 minutes I am actually on dinner that's why I think it will late a minute but will give it under 10-15 minutes

@yogeshbdeshpande
Copy link
Contributor

yogeshbdeshpande commented Sep 29, 2025

Author

No need to rush, please do take your time, you can do it tomorrow also, as it is late at the time zone you are in!!

@Sukuna0007Abhi
Copy link
Contributor Author

Sure sir, thanks for advising it's 8pm now in India, so it's little late but if I do it now I can work on another issue on Corim/services that's why, thanks..

Copy link
Contributor

@yogeshbdeshpande yogeshbdeshpande left a comment

Choose a reason for hiding this comment

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

Have a look at the recent comment, please!

- Clean up formatting in Verifier Owner role box
- Addresses reviewer feedback about extra \n in role boxes

Signed-off-by: Sukuna0007Abhi <appsonly310@gmail.com>
Copy link
Contributor

@yogeshbdeshpande yogeshbdeshpande left a comment

Choose a reason for hiding this comment

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

please see more comments!

@Sukuna0007Abhi
Copy link
Contributor Author

Sure sir fixing it @yogeshbdeshpande

- Remove cover repo (work in progress)
- Update cca-realm-measurements description and location
- Clean up mermaid diagram formatting

Signed-off-by: Sukuna0007Abhi <appsonly310@gmail.com>
@Sukuna0007Abhi
Copy link
Contributor Author

Ready for review sir @yogeshbdeshpande

Copy link
Contributor

@yogeshbdeshpande yogeshbdeshpande left a comment

Choose a reason for hiding this comment

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

LGTM!

@yogeshbdeshpande
Copy link
Contributor

Thank you @Sukuna0007Abhi for the work, I will await @thomas-fossati review, prior to merge!

@Sukuna0007Abhi
Copy link
Contributor Author

Sukuna0007Abhi commented Sep 29, 2025

Sure sir @yogeshbdeshpande thanks for approving 🙏, as a humble 🙏request could you kindly review if possible today veraison/corim#224 Pr, so that I can represent that on tomorrow meeting

Also pls review sir @thomas-fossati

@Sukuna0007Abhi
Copy link
Contributor Author

Thanks sir @setrofim

@yogeshbdeshpande
Copy link
Contributor

Thanks sir @setrofim

Thank you @Sukuna0007Abhi for a great work! I will merge your first contribution to Project Veraison.

@yogeshbdeshpande yogeshbdeshpande merged commit 0f68cda into veraison:main Oct 1, 2025
1 check passed
@Sukuna0007Abhi
Copy link
Contributor Author

Thanks sir @yogeshbdeshpande sir @setrofim

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Repo guide needs updation

3 participants