chore: drop noqa comments and blob_storage.is_enabled (#65)

No linter is configured in this repo so the noqa hints were just
ceremony. Removed across app.py and lambda_handlers/.

is_enabled() existed to skip S3 when PACKAGE_DATA_BUCKET wasn't set
(legacy local-dev escape hatch). Now that /data is gone and /blob is
the only data path, S3 is required everywhere; missing the env should
fail loudly, not 501 quietly.

Author: Androz2091

src/app.py

@@ -6,7 +6,7 @@
 from flask_limiter import Limiter
 
 # Import tasks before db so dotenv loads before SQLAlchemy reads POSTGRES_URL.
-import tasks  # noqa: F401
+import tasks
 from enqueue import enqueue_package
 from db import PackageProcessStatus, SavedPackageData, Session, fetch_package_status, fetch_package_rank
 
@@ -212,10 +212,6 @@ def get_package_blob(package_id):
     import blob_storage
     from db import SavedPackageData
 
-    if not blob_storage.is_enabled():
-        # /blob requires S3. Local dev should use /data.
-        return jsonify({'errorMessageCode': 'S3_NOT_CONFIGURED'}), 501
-
     ttl = int(os.getenv('PACKAGE_DATA_PRESIGNED_URL_TTL_SECONDS', '300'))
 
     # Demo: unauthenticated, lazy-seed S3 from generate_demo_database() on the

src/blob_storage.py

@@ -5,8 +5,7 @@
 download directly from S3 — bypassing API Gateway, which is slow and
 6MB-capped for binary responses.
 
-No-op when PACKAGE_DATA_BUCKET is unset (local dev): callers fall back
-to the legacy DB-backed path.
+PACKAGE_DATA_BUCKET must be set; the /blob endpoint requires S3.
 """
 import os
 
@@ -15,9 +14,6 @@ def _bucket():
     return os.getenv("PACKAGE_DATA_BUCKET")
 
 
-def is_enabled() -> bool:
-    return bool(_bucket())
-
 
 def _key(package_id: str) -> str:
     # Flat layout. Package IDs are MD5 hex digests, so no path-traversal risk.

src/lambda_handlers/api.py

@@ -6,8 +6,8 @@
 # keep working when this module is loaded from `src/lambda_handlers/api.py`.
 sys.path.insert(0, str(Path(__file__).resolve().parent.parent))
 
-import secrets_loader  # noqa: F401, E402 — runs at import to fetch SM secrets into os.environ
-from apig_wsgi import make_lambda_handler  # noqa: E402
-from app import app  # noqa: E402
+import secrets_loader
+from apig_wsgi import make_lambda_handler
+from app import app
 
 handler = make_lambda_handler(app)

src/lambda_handlers/worker.py

@@ -14,8 +14,8 @@
 
 sys.path.insert(0, str(Path(__file__).resolve().parent.parent))
 
-import secrets_loader  # noqa: F401, E402 — runs at import to fetch SM secrets into os.environ
-from tasks import process_package  # noqa: E402
+import secrets_loader
+from tasks import process_package
 
 
 def handler(event, context):