Skip to content

refactor(root): re-enable the structural clippy lints one at a time as a heatmap #1086

Description

@ScriptedAlchemy

Priority: P2 architecture instrument, not a lint chore.

crates/tracedecay/src/lib.rs allows crate-wide: too_many_lines, struct_excessive_bools, too_many_arguments, fn_params_excessive_bools, large_futures, unused_async, unnecessary_wraps, option_option, wildcard_imports, …. In the composition root this combination is a heatmap of leftover implementation. Re-enable them one at a time in the root only, in this order: too_many_arguments, fn_params_excessive_bools, struct_excessive_bools, large_futures, unnecessary_wraps, unused_async, too_many_lines. For each, record the offender inventory in the commit body and either fix the offender, move it to its destination crate (feeding the daemon-extraction issue), or place a narrow item-level allow with the invariant it protects. Do not "fix" cosmetics by splitting functions mechanically; the point is to find code that is no longer composition. Mirrors what #1037 did for tracedecay-mcp.

Acceptance: no crate-wide allow for the seven lints above in the root; each remaining allow is item-scoped with a reason.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions