Skip to content

added crate level documentation for smite core library#36

Open
psg-19 wants to merge 2 commits intomorehouse:masterfrom
psg-19:smite-documentation
Open

added crate level documentation for smite core library#36
psg-19 wants to merge 2 commits intomorehouse:masterfrom
psg-19:smite-documentation

Conversation

@psg-19
Copy link
Copy Markdown

@psg-19 psg-19 commented Apr 5, 2026

The smite crate's lib.rs had no documentation. This pr adds crate level doc comment explaining the library's purpose and descriptions of each module.

Copy link
Copy Markdown
Owner

@morehouse morehouse left a comment

Choose a reason for hiding this comment

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

Thanks, this is definitely an improvement.

I think it could be even better if we move the detailed descriptions of each module to the module files themselves and then have short one-line summaries of each module here. That's how rust docs are usually structured.

smite/src/lib.rs Outdated
Comment on lines +24 to +29
//! - [`runners`] - Runner abstraction for delivering fuzz inputs.
//! `NyxRunner` communicates with the Nyx hypervisor for
//! fast snapshot-based fuzzing, `LocalRunner` reads input
//! from a file or stdin for crash reproduction. `StdRunner`
//! auto-selects between them based on the `SMITE_NYX` environment
//! variable.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Also it would be helpful to mention that NyxRunner requires the nyx cargo feature to be enabled.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Sure, will do that

@psg-19
Copy link
Copy Markdown
Author

psg-19 commented Apr 7, 2026

Thanks, this is definitely an improvement.

I think it could be even better if we move the detailed descriptions of each module to the module files themselves and then have short one-line summaries of each module here. That's how rust docs are usually structured.

Okay, I'll do this way.
Thanks for the review!!

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.

2 participants