Fix Python SDK linting errors and test failure

- Fix 69 ruff linting errors: import ordering, modern type annotations
  (Optional[X] -> X | None, Union -> |), isinstance tuple syntax
- Remove unused imports (pytest in test files)
- Fix test_verify_valid_signature: use actual derived address from
  signature.public_key instead of hardcoded incorrect address

Author: bafu

python/capture_sdk/__init__.py

@@ -11,33 +11,33 @@
     >>> print(asset.nid)
 """
 
+from . import verify
 from .client import Capture
+from .crypto import sha256, verify_signature
+from .errors import (
+    AuthenticationError,
+    CaptureError,
+    InsufficientFundsError,
+    NetworkError,
+    NotFoundError,
+    PermissionError,
+    ValidationError,
+)
 from .types import (
-    FileInput,
-    CaptureOptions,
-    RegisterOptions,
-    UpdateOptions,
-    SignOptions,
     Asset,
-    Commit,
-    AssetTree,
     AssetSearchOptions,
     AssetSearchResult,
-    SimilarMatch,
-    NftSearchResult,
+    AssetTree,
+    CaptureOptions,
+    Commit,
+    FileInput,
     NftRecord,
+    NftSearchResult,
+    RegisterOptions,
+    SignOptions,
+    SimilarMatch,
+    UpdateOptions,
 )
-from .errors import (
-    CaptureError,
-    AuthenticationError,
-    PermissionError,
-    NotFoundError,
-    InsufficientFundsError,
-    ValidationError,
-    NetworkError,
-)
-from .crypto import sha256, verify_signature
-from . import verify
 
 __version__ = "0.1.0"
 

python/capture_sdk/client.py

@@ -5,28 +5,27 @@
 import json
 import mimetypes
 from pathlib import Path
-from typing import Any, Optional, Union
+from typing import Any
 from urllib.parse import urlencode
 
 import httpx
 
+from .crypto import create_integrity_proof, sha256, sign_integrity_proof
+from .errors import CaptureError, ValidationError, create_api_error
 from .types import (
-    FileInput,
-    CaptureOptions,
-    RegisterOptions,
-    UpdateOptions,
     Asset,
-    Commit,
-    AssetTree,
     AssetSearchOptions,
     AssetSearchResult,
-    SimilarMatch,
-    NftSearchResult,
+    AssetTree,
+    CaptureOptions,
+    Commit,
+    FileInput,
     NftRecord,
+    NftSearchResult,
+    RegisterOptions,
+    SimilarMatch,
+    UpdateOptions,
 )
-from .errors import ValidationError, CaptureError, create_api_error
-from .crypto import sha256, create_integrity_proof, sign_integrity_proof
-
 
 DEFAULT_BASE_URL = "https://api.numbersprotocol.io/api/v3"
 HISTORY_API_URL = "https://e23hi68y55.execute-api.us-east-1.amazonaws.com/default/get-commits-storage-backend-jade-near"
@@ -65,7 +64,7 @@ def _get_mime_type(filename: str) -> str:
 
 def _normalize_file(
     file_input: FileInput,
-    options: Optional[RegisterOptions] = None,
+    options: RegisterOptions | None = None,
 ) -> tuple[bytes, str, str]:
     """
     Normalizes various file input types to a common format.
@@ -93,7 +92,7 @@ def _normalize_file(
         return data, filename, mime_type
 
     # 3. bytes or bytearray
-    if isinstance(file_input, (bytes, bytearray)):
+    if isinstance(file_input, bytes | bytearray):
         if not options or not options.filename:
             raise ValidationError("filename is required for binary input")
         data = bytes(file_input)
@@ -128,11 +127,11 @@ class Capture:
 
     def __init__(
         self,
-        token: Optional[str] = None,
+        token: str | None = None,
         *,
         testnet: bool = False,
-        base_url: Optional[str] = None,
-        options: Optional[CaptureOptions] = None,
+        base_url: str | None = None,
+        options: CaptureOptions | None = None,
     ):
         """
         Initialize the Capture client.
@@ -171,10 +170,10 @@ def _request(
         method: str,
         url: str,
         *,
-        data: Optional[dict[str, Any]] = None,
-        files: Optional[dict[str, Any]] = None,
-        json_body: Optional[dict[str, Any]] = None,
-        nid: Optional[str] = None,
+        data: dict[str, Any] | None = None,
+        files: dict[str, Any] | None = None,
+        json_body: dict[str, Any] | None = None,
+        nid: str | None = None,
     ) -> dict[str, Any]:
         """Makes an authenticated API request."""
         headers = {"Authorization": f"token {self._token}"}
@@ -221,12 +220,12 @@ def register(
         self,
         file: FileInput,
         *,
-        filename: Optional[str] = None,
-        caption: Optional[str] = None,
-        headline: Optional[str] = None,
+        filename: str | None = None,
+        caption: str | None = None,
+        headline: str | None = None,
         public_access: bool = True,
-        sign: Optional[dict[str, str]] = None,
-        options: Optional[RegisterOptions] = None,
+        sign: dict[str, str] | None = None,
+        options: RegisterOptions | None = None,
     ) -> Asset:
         """
         Registers a new asset.
@@ -330,11 +329,11 @@ def update(
         self,
         nid: str,
         *,
-        caption: Optional[str] = None,
-        headline: Optional[str] = None,
-        commit_message: Optional[str] = None,
-        custom_metadata: Optional[dict[str, Any]] = None,
-        options: Optional[UpdateOptions] = None,
+        caption: str | None = None,
+        headline: str | None = None,
+        commit_message: str | None = None,
+        custom_metadata: dict[str, Any] | None = None,
+        options: UpdateOptions | None = None,
     ) -> Asset:
         """
         Updates an existing asset's metadata.
@@ -565,12 +564,12 @@ def get_asset_tree(self, nid: str) -> AssetTree:
     def search_asset(
         self,
         *,
-        file_url: Optional[str] = None,
-        file: Optional[FileInput] = None,
-        nid: Optional[str] = None,
-        threshold: Optional[float] = None,
-        sample_count: Optional[int] = None,
-        options: Optional[AssetSearchOptions] = None,
+        file_url: str | None = None,
+        file: FileInput | None = None,
+        nid: str | None = None,
+        threshold: float | None = None,
+        sample_count: int | None = None,
+        options: AssetSearchOptions | None = None,
     ) -> AssetSearchResult:
         """
         Searches for similar assets using image similarity.
@@ -632,7 +631,7 @@ def search_asset(
         form_data: dict[str, Any] = {"token": self._token}
 
         # Add input source
-        files_data: Optional[dict[str, Any]] = None
+        files_data: dict[str, Any] | None = None
         if options.file_url:
             form_data["url"] = options.file_url
         elif options.nid:

python/capture_sdk/crypto.py

@@ -5,15 +5,14 @@
 import hashlib
 import json
 import time
-from typing import Union
 
 from eth_account import Account
 from eth_account.messages import encode_defunct
 
-from .types import IntegrityProof, AssetSignature
+from .types import AssetSignature, IntegrityProof
 
 
-def sha256(data: Union[bytes, bytearray]) -> str:
+def sha256(data: bytes | bytearray) -> str:
     """
     Computes SHA-256 hash of data.
 

python/capture_sdk/errors.py

@@ -2,7 +2,6 @@
 Error classes for the Capture SDK.
 """
 
-from typing import Optional
 
 
 class CaptureError(Exception):
@@ -12,7 +11,7 @@ def __init__(
         self,
         message: str,
         code: str,
-        status_code: Optional[int] = None,
+        status_code: int | None = None,
     ):
         super().__init__(message)
         self.message = message
@@ -43,7 +42,7 @@ def __init__(self, message: str = "Insufficient permissions for this operation")
 class NotFoundError(CaptureError):
     """Thrown when the requested asset is not found."""
 
-    def __init__(self, nid: Optional[str] = None):
+    def __init__(self, nid: str | None = None):
         message = f"Asset not found: {nid}" if nid else "Asset not found"
         super().__init__(message, "NOT_FOUND", 404)
         self.nid = nid
@@ -66,14 +65,14 @@ def __init__(self, message: str):
 class NetworkError(CaptureError):
     """Thrown when a network or API request fails."""
 
-    def __init__(self, message: str, status_code: Optional[int] = None):
+    def __init__(self, message: str, status_code: int | None = None):
         super().__init__(message, "NETWORK_ERROR", status_code)
 
 
 def create_api_error(
     status_code: int,
     message: str,
-    nid: Optional[str] = None,
+    nid: str | None = None,
 ) -> CaptureError:
     """Maps HTTP status codes to appropriate error classes."""
     if status_code == 400: