refactor: clean up unused code

[tamtamchik]

Summary:

• Remove unused deep_match_bytecode.
• Remove unused exception/logger leftovers.
• Inline explorer config reads.
• Align config TypedDict with runtime fields.
• Remove deprecated CLI/config compatibility.

Tests:

• uv run pytest -q
• uv run mypy

[Copilot]

Pull request overview

Refactors the bytecode verification utilities by removing an unused legacy wrapper (deep_match_bytecode) and its associated exception type, and updates the test suite to assert directly on the structured output from analyze_bytecode_diff.

Changes:

• Removed deep_match_bytecode from diffyscan/utils/binary_verifier.py.
• Removed BinVerifierError from diffyscan/utils/custom_exceptions.py.
• Updated tests/test_binary_verifier.py to validate analyze_bytecode_diff analysis fields instead of exception behavior.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
tests/test_binary_verifier.py	Updates tests to assert on analyze_bytecode_diff outputs (length mismatch and mismatch range classification).
diffyscan/utils/custom_exceptions.py	Removes the now-unused BinVerifierError exception class.
diffyscan/utils/binary_verifier.py	Removes the unused deep_match_bytecode wrapper and its dependency on BinVerifierError.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

diffyscan/diffyscan.py:308

• explorer_chain_id is checked with truthiness here; if it were ever 0 it would be treated as “not set”. Since this value can also be a string when loaded from config, it’s safer to normalize to int | None and check is not None.

    explorer_hostname = config.get("explorer_hostname")
    explorer_chain_id = config.get("explorer_chain_id")
    logger.divider()
    logger.okay("Contract", contract_address_from_config)
    logger.okay("Blockchain explorer Hostname", explorer_hostname)
    if explorer_chain_id:
        logger.okay("Blockchain explorer Chain ID", explorer_chain_id)
    else:
        logger.warn("Blockchain explorer Chain ID isn't set")

[Copilot]

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.

[TheDZhon]

GG 🎬