docs(guide/dev): remove even more (#360)

Remove Visibility, Struct Field Ordering, Macros, and `pub use ... as`
bullet — these are either standard Rust knowledge, generic best practices,
or not used in the codebase.

https://claude.ai/code/session_01VE5W68ky3UobBMongN8Dfc

Co-authored-by: Claude <noreply@anthropic.com>

Author: KSXGitHub

CONTRIBUTING.md

@@ -64,12 +64,6 @@ pub use error_report::ErrorReport;
 pub use event::Event;
 ```
 
-- Type aliases using `pub use ... as ...` are used to provide semantic alternative names:
-
-```rust
-pub use Reflection as DataTreeReflection;
-```
-
 ### Derive Macro Ordering
 
 When deriving multiple traits, use this order and split across multiple `#[derive(...)]` lines for readability:
@@ -202,12 +196,6 @@ where
     HardlinksRecorder: RecordHardlinks<Size, Report> + Sync + ?Sized,
 ```
 
-### Visibility
-
-- Use `pub` for the public API surface.
-- Use `pub(crate)` for items shared within the crate but not exposed externally.
-- Default to private for everything else.
-
 ### Error Handling
 
 - Define custom error enums with `#[derive(Debug, Display, Error)]` from `derive_more`.
@@ -233,14 +221,6 @@ ExitCode::from(match self {
 })
 ```
 
-### Struct Field Ordering
-
-Order fields logically by purpose, not alphabetically. Group related fields together. Document every public field with `///` comments.
-
-### Macros
-
-Use macros to reduce boilerplate for repetitive patterns (e.g. newtype wrappers, trait impls for multiple numeric types). Keep macros well-scoped and documented.
-
 ## Setup
 
 Install the required Rust toolchain and components before running any checks: