Skip to content

Capitalize Python MLDSA class names from MlDsa to MLDSA#14596

Merged
reaperhulk merged 1 commit intomainfrom
claude/capitalize-mldsa-python-ydTYH
Apr 7, 2026
Merged

Capitalize Python MLDSA class names from MlDsa to MLDSA#14596
reaperhulk merged 1 commit intomainfrom
claude/capitalize-mldsa-python-ydTYH

Conversation

@alex
Copy link
Copy Markdown
Member

@alex alex commented Apr 7, 2026

Summary

  • Rename Python classes MlDsa65PrivateKeyMLDSA65PrivateKey and MlDsa65PublicKeyMLDSA65PublicKey across all Python interfaces, type stubs, tests, and documentation
  • Add name= annotations to the PyO3 #[pyclass] definitions so Rust structs keep their original names but expose the capitalized MLDSA65* names to Python
  • Update test helper classes (MlDsaVariantMLDSAVariant, TestMlDsaTestMLDSA)

Test plan

  • Verify all existing ML-DSA tests pass with the renamed classes
  • Confirm isinstance checks work correctly (PyO3 classes register under the new ABC names)
  • Verify documentation builds without errors

https://claude.ai/code/session_01N13VBJmo4GbcFX86Yu4ugZ

@reaperhulk
Copy link
Copy Markdown
Member

rebase and also do mldsa44

Rename all Python-facing MLDSA class names:
- MlDsa44PrivateKey -> MLDSA44PrivateKey
- MlDsa44PublicKey -> MLDSA44PublicKey
- MlDsa65PrivateKey -> MLDSA65PrivateKey
- MlDsa65PublicKey -> MLDSA65PublicKey

Add name= annotations to PyO3 pyclass definitions so Rust structs
keep their original names but expose the capitalized names to Python.
Update all Python interfaces, type stubs, tests, and documentation.

https://claude.ai/code/session_01N13VBJmo4GbcFX86Yu4ugZ
@alex alex force-pushed the claude/capitalize-mldsa-python-ydTYH branch from 02be983 to 7282d22 Compare April 7, 2026 13:35
@reaperhulk reaperhulk merged commit f29e264 into main Apr 7, 2026
68 checks passed
@reaperhulk reaperhulk deleted the claude/capitalize-mldsa-python-ydTYH branch April 7, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants