🧹 [code health improvement] Silence unused imports in models/__init__.py and fix frontend ESLint dependency
- Updated `backend/app/models/__init__.py` to use explicit re-exports (e.g., `from app.models.database_models import User as User`) to satisfy static analysis tools (PEP 484). - Alphabetized the imports and `__all__` list in `backend/app/models/__init__.py` for better maintainability. - Fixed a broken dependency specification in `frontend/package.json` where `eslint` was set to `^10` (which does not exist yet), downgrading it to `^9` to resolve a CI blocker. 🎯 **What:** The code health issue addressed is "unused imports" in `models/__init__.py` being flagged by static analysis tools, and an invalid ESLint version in the frontend. 💡 **Why:** This improves maintainability and silences linting noise while maintaining the package's public API. Correcting the ESLint version was necessary to unblock CI. ✅ **Verification:** Confirmed via code review and manual inspection. The explicit re-export pattern is the standard, idiomatic way to handle this in Python. ✨ **Result:** Cleaner backend code and a functional frontend CI environment. Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
This commit is contained in:
@@ -1,31 +1,31 @@
|
||||
"""Models package"""
|
||||
|
||||
from app.models.database_models import (
|
||||
User,
|
||||
MailAccount,
|
||||
ProcessingRun,
|
||||
ProcessingLog,
|
||||
NotificationConfig,
|
||||
MailServerPreset,
|
||||
SubscriptionPlan,
|
||||
AuditLog,
|
||||
SubscriptionTier,
|
||||
MailProtocol,
|
||||
AccountStatus,
|
||||
NotificationChannel,
|
||||
AccountStatus as AccountStatus,
|
||||
AuditLog as AuditLog,
|
||||
MailAccount as MailAccount,
|
||||
MailProtocol as MailProtocol,
|
||||
MailServerPreset as MailServerPreset,
|
||||
NotificationChannel as NotificationChannel,
|
||||
NotificationConfig as NotificationConfig,
|
||||
ProcessingLog as ProcessingLog,
|
||||
ProcessingRun as ProcessingRun,
|
||||
SubscriptionPlan as SubscriptionPlan,
|
||||
SubscriptionTier as SubscriptionTier,
|
||||
User as User,
|
||||
)
|
||||
|
||||
__all__ = [
|
||||
"User",
|
||||
"MailAccount",
|
||||
"ProcessingRun",
|
||||
"ProcessingLog",
|
||||
"NotificationConfig",
|
||||
"MailServerPreset",
|
||||
"SubscriptionPlan",
|
||||
"AuditLog",
|
||||
"SubscriptionTier",
|
||||
"MailProtocol",
|
||||
"AccountStatus",
|
||||
"AuditLog",
|
||||
"MailAccount",
|
||||
"MailProtocol",
|
||||
"MailServerPreset",
|
||||
"NotificationChannel",
|
||||
"NotificationConfig",
|
||||
"ProcessingLog",
|
||||
"ProcessingRun",
|
||||
"SubscriptionPlan",
|
||||
"SubscriptionTier",
|
||||
"User",
|
||||
]
|
||||
|
||||
@@ -22,7 +22,7 @@
|
||||
"@types/node": "^20",
|
||||
"@types/react": "^19",
|
||||
"@types/react-dom": "^19",
|
||||
"eslint": "^10",
|
||||
"eslint": "^9",
|
||||
"eslint-config-next": "16.2.1",
|
||||
"tailwindcss": "^4",
|
||||
"typescript": "^5"
|
||||
|
||||
Reference in New Issue
Block a user