Remove dead code surfaced by review (pub items with no callers) #15
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
The crate compiles clean (-D warnings), but these
pub/pub(crate)items escape dead-code analysis and have zero callers:High confidence (safe to remove):
JsonRpcResponse::internal_error— src/mcp/protocol.rs:56-59. Carries the only#[allow(dead_code)]in src/; no callers (sibling constructors are used). Remove the method and the#[allow].tag_description_path— src/core/storage/mod.rs:201-204. Reorg alias; zero callers (forwards totag_meta_path).Task::is_open— src/core/domain/task.rs:256-259. Zero callers (siblingsis_hidden/is_activeused).Medium (redundant reorg re-exports):
pub use resolve::resolve_task_id;andpub use config::Config;in src/core/mod.rs — every caller uses the full path or the crate-root re-export; these module-root re-exports are unused.Leave
store_mutandcore::{Result,TaskError}re-exports (plausibly intentional API surface) unless deliberately trimming. No unused dependencies found.Found in review.