Skip to content

eboody/modum

Repository files navigation

modum logo

Modum enforces consistent module naming, import style, and public API paths across a Rust workspace.

build status crates.io docs.rs

modum

It is a lint tool. It reports diagnostics. It does not rewrite code.

Why It Exists

These API shapes are legal Rust, but they add noise:

use http::Client;
use user::UserRepository;

pub use partials::error::Error;

These read more clearly:

use http;
use user;

pub use partials::Error;

And these public paths usually read better:

user::Repository
user::error::InvalidEmail
partials::Error

instead of:

user::UserRepository
user::error::InvalidEmailError
partials::error::Error

modum enforces that style across an entire workspace.

Mental Model

modum follows three rules:

  1. Keep namespace context visible at call sites.
  2. Keep public paths meaningful without redundancy.
  3. Use modules for domain boundaries, not file organization.

Quick Usage

cargo install modum
cargo modum check --root .
cargo modum check --root . --mode warn
cargo modum check --root . --format json

cargo install modum installs both modum and the Cargo subcommand cargo-modum, so either of these is valid:

modum check --root .
cargo modum check --root .

If you are developing modum itself:

cargo run -p modum -- check --root .

Environment:

MODUM=off|warn|deny

Default mode is deny.

CI Usage

Use modum the same way you would use clippy or cargo-deny: run it as a normal command in CI, not from build.rs.

- run: cargo install modum
- run: cargo modum check --root .

Exit Behavior

  • 0: clean, or warnings allowed via --mode warn
  • 2: warning-level policy violations found in deny mode
  • 1: hard errors, including parse/configuration failures and error-level policy violations such as api_organizational_submodule_flatten

Configuration

Configure the lints in any workspace with Cargo metadata:

[workspace.metadata.modum]
generic_nouns = ["Id", "Repository", "Service", "Error", "Command", "Request", "Response"]
weak_modules = ["storage", "transport", "infra", "common", "misc", "helpers", "helper", "types", "util", "utils"]
catch_all_modules = ["common", "misc", "helpers", "helper", "types", "util", "utils"]
organizational_modules = ["error", "errors"]
namespace_preserving_modules = ["auth", "command", "components", "email", "error", "http", "page", "partials", "policy", "query", "repo", "store", "storage", "transport", "infra"]

Use [package.metadata.modum] inside a member crate to override workspace defaults for that package.

Tuning guide:

  • generic_nouns: generic leaves like Repository, Error, or Request
  • namespace_preserving_modules: modules that should stay visible at call sites, such as http, email, partials, or components
  • organizational_modules: modules that should not leak into the public API surface, such as error

Lint Categories

Import Style

These warn when imports or re-exports flatten a namespace that should stay visible.

  • namespace_flat_use Warning for flattened imports of generic nouns when there is an actionable namespace-visible call-site form that adds net context, such as storage::Repository or http::StatusCode. It skips cases where the only preserved form would still be redundant, such as error::Error or response::Response.
  • namespace_flat_use_preserve_module Warning for flattened imports from configured namespace-preserving modules when the preserved call-site form still adds net context.
  • namespace_flat_use_redundant_leaf_context Warning for flattened imports or actionable rename-heavy aliases whose leaf repeats parent context. For plain imports, this only fires when the shorter leaf would be an actionable generic noun such as Repository, Error, or Id. For rename aliases, this only fires when the qualified form would still preserve real context, such as http::StatusCode or page::Event.
  • namespace_redundant_qualified_generic Warning for qualified call-site paths whose module only repeats a generic category already named by the leaf, such as response::Response or error::Error.
  • namespace_parent_surface
  • namespace_flat_pub_use
  • namespace_flat_pub_use_preserve_module
  • namespace_flat_pub_use_redundant_leaf_context

Examples:

  • use storage::Repository;
  • use http::Client;
  • use user::UserRepository;
  • response::Response
  • use crate::error::Error; inside a crate whose root surface already exposes Error
  • pub use auth::{login, logout};

Canonical parent-surface re-exports are allowed. pub use error::{Error, Result}; is valid when that is how a module intentionally exposes module::Error and module::Result. The same applies to broader UI surfaces such as exposing both components::Button and partials::Button.

Public API Paths

These warn when public leaves are too generic for a weak parent, or when the path repeats context it already has.

  • api_missing_parent_surface_export
  • api_weak_module_generic_leaf
  • api_redundant_leaf_context
  • api_redundant_category_suffix

Examples:

  • partials::button::Button when the intended surface should also expose partials::Button
  • storage::Repository
  • user::UserRepository
  • user::error::InvalidEmailError

Private organizational child modules are allowed to flatten their family items back to the parent surface. For example, mod auth_shell; pub use auth_shell::{AuthShell, AuthShellVariant}; is treated as a valid parent-surface export shape.

Module Boundaries

These catch weak or redundant public module structure.

  • api_catch_all_module
  • api_repeated_module_segment

Examples:

  • helpers
  • error::error

Structural Errors

This rule is an error, not a warning.

  • api_organizational_submodule_flatten

Example:

  • partials::error::Error should usually be partials::Error

What It Does Not Check

Some naming-guide rules stay advisory because they are too semantic to lint reliably without compiler-grade context.

Examples:

  • choosing the best public path among several plausible domain decompositions
  • deciding when an internal long name plus pub use ... as ... is the right tradeoff
  • deciding whether a new module level adds real meaning or only mirrors the file tree in edge cases

Scope

Default discovery:

  • package root: scans <root>/src
  • workspace root: scans each member crate's src

Override discovery with --include:

modum check --root . --include crates/api/src --include crates/domain/src

False Positives And False Negatives

The broader import-style lints only inspect module-scope use items. They do not scan local block imports inside functions or tight test scopes, because those scopes often benefit from flatter imports.

To reduce false negatives:

  • extend namespace_preserving_modules for domain modules like user, billing, or tenant
  • keep generic_nouns aligned with the generic leaves your API actually uses
  • keep organizational_modules configured so partials::error::Error-style paths stay blocked

About

modum enforces consistent module naming, import style, and public API paths across a Rust workspace.

Resources

Stars

Watchers

Forks

Releases

No releases published

Packages

 
 
 

Contributors

Languages