remove exported output file on error

[dloghin]

Description

Issue

The export command creates the output file before executing any export work, then returns early on errors, e.g., failed block reads, interrupts, or invalid ranges, without removing or rolling back the newly created file. When the export loop is skipped or aborted, it leaves behind an empty or partial file. At the same time, the command still appears to have produced an artifact, increasing the likelihood that operators will treat incomplete exports as valid backups.Consider ensuring that any early-return path removes the partially created file so that failed exports do not leave behind misleading artifacts.

Fix

We modified the execute() function to remove the output file if an error occurs. We wrap the main logic in a closure block and add cleanup logic on error (file bin/toolsexport.rs).

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactoring (no functional changes)
• Other (please describe):

Code Guidelines

Before submitting your PR, please review the relevant code guidelines in the docs/ folder:

• Building on Reth Guide - Comprehensive guide for building on top of Reth

Specific Guidelines by Component:

• Chainspec & EVM Guide - For changes to chainspec or EVM configuration
• Database Guide - For changes to database tables or storage
• Engine & Consensus Guide - For changes to consensus or engine API
• Extension Guide - For adding extensions or plugins
• Node Builder Guide - For changes to node initialization
• Payload Builder Guide - For changes to block building logic

Checklist

• I have reviewed the relevant code guidelines in the docs/ folder
• My code follows the coding standards of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

Testing

Previously:

• create a local devnet using xlayer-toolkit
• run:

xlayer-reth-tools export --datadir /home/ubuntu/git/XLayer/xlayer-toolkit/devnet/data/op-reth-seq --exported-data 
exported.rlp
...
2026-01-26T02:26:03.885564Z ERROR Error: Other(
    FileSystem(
        Open {
            source: Os {
                code: 2,
                kind: NotFound,
                message: "No such file or directory",
            },
            path: "xlayer-toolkit/devnet/data/op-reth-seq/static_files/static_file_headers_0_499999.conf",
        },
    ),
)

• but the output file is still created:
  exported.rlp 0 bytes

After fix:

• we get the same error but no file is created

[JimmyShi22]

LGTM, this code will delete the file when error or ctr-c