Chore: Make linter happy (#1256)

* refactor: remove unused imports

* fix: replace NotImplemented with NotImplementedError

* refactor: resolve E722 (do not use bare 'except')

* refactor: remove unused variable

* refactor: remove unused imports

* refactor: ignore unused imports that will be used in the future

* refactor: resolve W293 (blank line contains whitespace)

* refactor: resolve F541 (f-string is missing placeholders)

---------

Co-authored-by: afourney <[email protected]>

Author: t3tra-dev

packages/markitdown-mcp/src/markitdown_mcp/__main__.py

@@ -1,5 +1,4 @@
 import sys
-from typing import Any
 from mcp.server.fastmcp import FastMCP
 from starlette.applications import Starlette
 from mcp.server.sse import SseServerTransport

packages/markitdown-sample-plugin/tests/test_sample_plugin.py

@@ -1,6 +1,5 @@
 #!/usr/bin/env python3 -m pytest
 import os
-import pytest
 
 from markitdown import MarkItDown, StreamInfo
 from markitdown_sample_plugin import RtfConverter

packages/markitdown/src/markitdown/__main__.py

@@ -4,7 +4,6 @@
 import argparse
 import sys
 import codecs
-import locale
 from textwrap import dedent
 from importlib.metadata import entry_points
 from .__about__ import __version__
@@ -34,13 +33,13 @@ def main():
                 OR
 
                 markitdown < example.pdf
-                
+
                 OR to save to a file use
-    
+
                 markitdown example.pdf -o example.md
-                
+
                 OR
-                
+
                 markitdown example.pdf > example.md
             """
         ).strip(),

packages/markitdown/src/markitdown/_base_converter.py

@@ -1,7 +1,4 @@
-import os
-import tempfile
-from warnings import warn
-from typing import Any, Union, BinaryIO, Optional, List
+from typing import Any, BinaryIO, Optional
 from ._stream_info import StreamInfo
 
 

packages/markitdown/src/markitdown/_markitdown.py

@@ -1,11 +1,8 @@
-import copy
 import mimetypes
 import os
 import re
 import sys
 import shutil
-import tempfile
-import warnings
 import traceback
 import io
 from dataclasses import dataclass
@@ -547,7 +544,7 @@ def _convert(
                 # Sanity check -- make sure the cur_pos is still the same
                 assert (
                     cur_pos == file_stream.tell()
-                ), f"File stream position should NOT change between guess iterations"
+                ), "File stream position should NOT change between guess iterations"
 
                 _kwargs = {k: v for k, v in kwargs.items()}
 
@@ -614,7 +611,7 @@ def _convert(
 
         # Nothing can handle it!
         raise UnsupportedFormatException(
-            f"Could not convert stream to Markdown. No converter attempted a conversion, suggesting that the filetype is simply not supported."
+            "Could not convert stream to Markdown. No converter attempted a conversion, suggesting that the filetype is simply not supported."
         )
 
     def register_page_converter(self, converter: DocumentConverter) -> None:

packages/markitdown/src/markitdown/converter_utils/docx/math/omml.py

@@ -272,7 +272,7 @@ def do_fname(self, elm):
                 if FUNC.get(t):
                     latex_chars.append(FUNC[t])
                 else:
-                    raise NotImplemented("Not support func %s" % t)
+                    raise NotImplementedError("Not support func %s" % t)
             else:
                 latex_chars.append(t)
         t = BLANK.join(latex_chars)
@@ -316,7 +316,7 @@ def do_limlow(self, elm):
         t_dict = self.process_children_dict(elm, include=("e", "lim"))
         latex_s = LIM_FUNC.get(t_dict["e"])
         if not latex_s:
-            raise NotImplemented("Not support lim %s" % t_dict["e"])
+            raise NotImplementedError("Not support lim %s" % t_dict["e"])
         else:
             return latex_s.format(lim=t_dict.get("lim"))
 

packages/markitdown/src/markitdown/converter_utils/docx/pre_process.py

@@ -147,7 +147,7 @@ def pre_process_docx(input_docx: BinaryIO) -> BinaryIO:
                         updated_content = _pre_process_math(content)
                         # In the future, if there are more pre-processing steps, they can be added here
                         zip_output.writestr(name, updated_content)
-                    except:
+                    except Exception:
                         # If there is an error in processing the content, write the original content
                         zip_output.writestr(name, content)
                 else:

packages/markitdown/src/markitdown/converters/_audio_converter.py

@@ -1,5 +1,4 @@
-import io
-from typing import Any, BinaryIO, Optional
+from typing import Any, BinaryIO
 
 from ._exiftool import exiftool_metadata
 from ._transcribe_audio import transcribe_audio

packages/markitdown/src/markitdown/converters/_bing_serp_converter.py

@@ -1,9 +1,8 @@
-import io
 import re
 import base64
 import binascii
 from urllib.parse import parse_qs, urlparse
-from typing import Any, BinaryIO, Optional
+from typing import Any, BinaryIO
 from bs4 import BeautifulSoup
 
 from .._base_converter import DocumentConverter, DocumentConverterResult

packages/markitdown/src/markitdown/converters/_csv_converter.py

@@ -1,9 +1,7 @@
-import sys
 import csv
 import io
 from typing import BinaryIO, Any
 from charset_normalizer import from_bytes
-from ._html_converter import HtmlConverter
 from .._base_converter import DocumentConverter, DocumentConverterResult
 from .._stream_info import StreamInfo