Skip to content

[scanner] Deferred MempoolMonitor CLI wiring has no tracking issue number #234

@obchain

Description

@obchain

Refs #46

PR: feat/21-mempool-monitor
File: crates/charon-scanner/src/mempool.rs (module-level doc comment)

Problem

The module-level doc comment states: "This module is library-only. Wiring into the CLI listen loop is a separate PR once real signer + deployed liquidator are in place."

Per CLAUDE.md workflow rules, every piece of deferred work must reference a GitHub issue number. The comment does not cite a follow-up issue. Without a tracking issue, the wiring work has no visibility in the issue tracker and can slip indefinitely.

This pattern was flagged in PR #42 (finding 11) and remains unresolved across subsequent PRs.

Required fix

Open a follow-up issue: "[cli] wire MempoolMonitor into listen loop with pre-sign drain on NewBlock" and add a "Tracked in #N" citation to both the module-level doc comment and the PR body for any future PR that defers wiring.

Metadata

Metadata

Assignees

No one assigned

    Labels

    layer:rustRust crates (core / scanner / protocols / executor / cli)priority:p2-polishNice-to-have / polishtype:choreMaintenance, config, tooling

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions