From 043a75acde80a1c02f236bd81360be15de600694 Mon Sep 17 00:00:00 2001 From: Luke Mino-Altherr Date: Tue, 10 Feb 2026 13:01:25 -0800 Subject: [PATCH] Fix ruff linting issues - Remove debug print statements - Remove trailing whitespace on blank lines - Remove unused pytest import Amp-Thread-ID: https://ampcode.com/threads/T-019c3a8d-3b4f-75b4-8513-1c77914782f7 Co-authored-by: Amp --- app/assets/scanner.py | 2 +- app/assets/seeder.py | 5 +---- app/assets/services/metadata_extract.py | 6 +++--- tests-unit/assets_test/services/test_bulk_ingest.py | 1 - 4 files changed, 5 insertions(+), 9 deletions(-) diff --git a/app/assets/scanner.py b/app/assets/scanner.py index 98891d5b4..b622b78f7 100644 --- a/app/assets/scanner.py +++ b/app/assets/scanner.py @@ -322,7 +322,7 @@ def build_asset_specs( mime_type = metadata.content_type if metadata else None if mime_type is None: - print(f"[build_asset_specs] no mime_type for {abs_p} (metadata={metadata is not None})") + pass specs.append( { "abs_path": abs_p, diff --git a/app/assets/seeder.py b/app/assets/seeder.py index a6594615f..19e175c69 100644 --- a/app/assets/seeder.py +++ b/app/assets/seeder.py @@ -6,7 +6,7 @@ import threading import time from dataclasses import dataclass, field from enum import Enum -from typing import TYPE_CHECKING, Callable +from typing import Callable from app.assets.scanner import ( RootType, @@ -20,9 +20,6 @@ from app.assets.scanner import ( ) from app.database.db import dependencies_available -if TYPE_CHECKING: - pass - class State(Enum): """Seeder state machine states.""" diff --git a/app/assets/services/metadata_extract.py b/app/assets/services/metadata_extract.py index b630da1e2..298385d57 100644 --- a/app/assets/services/metadata_extract.py +++ b/app/assets/services/metadata_extract.py @@ -22,7 +22,7 @@ MAX_SAFETENSORS_HEADER_SIZE = 8 * 1024 * 1024 def _register_custom_mime_types(): """Register custom MIME types for model and config files. - + Called before each use because mimetypes.init() in server.py resets the database. Uses a quick check to avoid redundant registrations. """ @@ -30,7 +30,7 @@ def _register_custom_mime_types(): test_result, _ = mimetypes.guess_type("test.safetensors") if test_result == "application/safetensors": return - + mimetypes.add_type("application/safetensors", ".safetensors") mimetypes.add_type("application/safetensors", ".sft") mimetypes.add_type("application/pytorch", ".pt") @@ -314,7 +314,7 @@ def extract_file_metadata( mime_type, _ = mimetypes.guess_type(abs_path) meta.content_type = mime_type if mime_type is None: - print(f"[extract_file_metadata] No mime_type for {abs_path}") + pass # Size from stat if stat_result is None: diff --git a/tests-unit/assets_test/services/test_bulk_ingest.py b/tests-unit/assets_test/services/test_bulk_ingest.py index a9af8ca67..4d362ce1f 100644 --- a/tests-unit/assets_test/services/test_bulk_ingest.py +++ b/tests-unit/assets_test/services/test_bulk_ingest.py @@ -2,7 +2,6 @@ from pathlib import Path -import pytest from sqlalchemy.orm import Session from app.assets.database.models import Asset