Skip to content

docs: enable missing_docs + document public APIs in 4 crates - #3469

Closed
jewoos2921 wants to merge 1 commit into
block:mainfrom
jewoos2921:docs/expand-missing-docs
Closed

docs: enable missing_docs + document public APIs in 4 crates#3469
jewoos2921 wants to merge 1 commit into
block:mainfrom
jewoos2921:docs/expand-missing-docs

Conversation

@jewoos2921

Copy link
Copy Markdown

Summary

Core crates (buzz-core, buzz-auth, buzz-db, etc.) already enforce #![warn(missing_docs)]. This PR extends the same standard to 4 auxiliary crates that were missing it, clearing all missing_docs warnings and improving rustdoc output.

Changes

Crate Items documented What was added
buzz-ws-client 4 Crate-level //! doc + 3 module docs (connection, error, message)
buzz-acp 2 Crate-level //! doc + run() function doc
buzz-dev-mcp 2 Crate-level //! doc + run() function doc
buzz-cli ~70 Crate-level //! doc + enums, variants, struct fields, and functions across lib.rs, agent_management.rs, main.rs

132 insertions, 0 deletions — all changes are doc-comment-only. No code logic was modified.

Verification

  • RUSTDOCFLAGS="-W missing_docs" cargo doc — zero missing_docs warnings across all 4 crates
  • cargo clippy -- -D warnings — clean
  • cargo fmt --check — clean

Add #![warn(missing_docs)] and crate-level //! docs to buzz-cli, buzz-acp,
buzz-dev-mcp, and buzz-ws-client, clearing all missing_docs warnings.

Core crates (buzz-core, buzz-auth, buzz-db, etc.) already enforce
missing_docs; this extends the same standard to the auxiliary crates that
were missing it, improving rustdoc output and API discoverability.

Changes per crate:
- buzz-ws-client: crate doc + 3 module docs
- buzz-acp: crate doc + run() function doc
- buzz-dev-mcp: crate doc + run() function doc
- buzz-cli: crate doc + ~70 items (enums, variants, struct fields,
  functions across lib.rs, agent_management.rs, main.rs)

All changes are doc-comment-only (132 insertions, 0 deletions).
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.

1 participant