Skip to content

Add PQ documentation#10514

Open
kaleb-himes wants to merge 1 commit into
wolfSSL:masterfrom
kaleb-himes:PQ-DOX
Open

Add PQ documentation#10514
kaleb-himes wants to merge 1 commit into
wolfSSL:masterfrom
kaleb-himes:PQ-DOX

Conversation

@kaleb-himes
Copy link
Copy Markdown
Contributor

Description

During the team meeting it was noted that the documents lacked PQ algorithm coverage except SLH-DSA. Adding the others and one minor update to SLH-DSA.

Fixes zd#

Testing

How did you test?

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@github-actions
Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@dgarske
Copy link
Copy Markdown
Member

dgarske commented May 22, 2026

Jenkins retest this please

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds Doxygen coverage for post-quantum wolfCrypt APIs, expanding documentation for ML-DSA, ML-KEM, LMS/HSS, and XMSS/XMSS^MT, plus group-level descriptions for PQ algorithms.

Changes:

  • Added new Doxygen header comment files for ML-DSA, ML-KEM, LMS, and XMSS APIs.
  • Added Doxygen groups and overview text for ML-DSA, ML-KEM, SLH-DSA, LMS, and XMSS.
  • Documented usage examples, parameter descriptions, and return codes for PQ APIs.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 13 comments.

Show a summary per file
File Description
doc/dox_comments/header_files/wc_xmss.h Adds Doxygen comments for XMSS/XMSS^MT APIs.
doc/dox_comments/header_files/wc_mlkem.h Adds Doxygen comments for ML-KEM APIs.
doc/dox_comments/header_files/wc_mldsa.h Adds Doxygen comments for ML-DSA APIs.
doc/dox_comments/header_files/wc_lms.h Adds Doxygen comments for LMS/HSS APIs.
doc/dox_comments/header_files/doxygen_groups.h Adds group-level PQ algorithm descriptions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread doc/dox_comments/header_files/wc_mlkem.h Outdated
Comment thread doc/dox_comments/header_files/wc_mlkem.h Outdated
Comment thread doc/dox_comments/header_files/wc_mldsa.h Outdated
Comment thread doc/dox_comments/header_files/wc_mldsa.h Outdated
Comment thread doc/dox_comments/header_files/wc_mlkem.h Outdated
Comment thread doc/dox_comments/header_files/wc_lms.h Outdated
Comment thread doc/dox_comments/header_files/wc_mlkem.h Outdated
Comment thread doc/dox_comments/header_files/wc_mlkem.h Outdated
Comment thread doc/dox_comments/header_files/wc_mlkem.h Outdated
Comment thread doc/dox_comments/header_files/wc_mlkem.h Outdated
Co-Pilot feedback
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.

5 participants