reg

Author: tsmluky

backend/core/schemas.py

@@ -170,3 +170,27 @@ class SignalCreate(BaseModel):
     rationale: Optional[str] = Field(None, description="Justificación breve")
     source: str = Field(..., description="Origen de la señal", max_length=50)
     extra: Optional[Dict[str, Any]] = Field(None, description="Metadatos adicionales")
+
+
+class UserCreate(BaseModel):
+    """
+    Schema for user registration request.
+    """
+    email: str = Field(..., description="User email (unique)")
+    password: str = Field(..., description="Password (min 8 chars)", min_length=8)
+    name: str = Field("Trader", description="User display name")
+
+
+class UserResponse(BaseModel):
+    """
+    Schema for user response (safe DTO, no password).
+    """
+    id: int
+    email: str
+    name: str
+    role: str
+    plan: str
+    created_at: datetime
+    
+    class Config:
+        from_attributes = True

backend/routers/auth.py

@@ -1,4 +1,4 @@
-from datetime import timedelta
+from datetime import timedelta, datetime
 from typing import Annotated
 from fastapi import APIRouter, Depends, HTTPException, status, Request
 from fastapi.security import OAuth2PasswordBearer, OAuth2PasswordRequestForm
@@ -7,6 +7,7 @@
 from sqlalchemy.exc import IntegrityError
 from database import get_db
 from models_db import User
+from core.schemas import UserCreate, UserResponse
 from core.security import verify_password, create_access_token, get_password_hash, ACCESS_TOKEN_EXPIRE_MINUTES, SECRET_KEY, ALGORITHM
 
 router = APIRouter(prefix="/auth", tags=["Auth"])
@@ -84,42 +85,64 @@ async def login_for_access_token(
         }
     }
 
-@router.post("/register")
+@router.post("/register", response_model=UserResponse, status_code=status.HTTP_201_CREATED)
 async def register_user(
-    email: str, 
-    password: str, 
-    name: str = "Trader", 
+    user_data: UserCreate, 
     db: Session = Depends(get_db)
 ):
-    """(Dev Helper) Registra un usuario inicial."""
-    # Verificar si existe
-    result = db.execute(select(User).where(User.email == email))
+    """
+    Registra un nuevo usuario en la plataforma.
+    
+    - Valida email único (409 Conflict)
+    - Hashing seguro de contraseña (bcrypt)
+    - Retorna el usuario creado (DTO seguro sin password)
+    """
+    # 1. Normalizar email
+    email_norm = user_data.email.strip().lower()
+    
+    # 2. Verificar duplicado
+    result = db.execute(select(User).where(User.email == email_norm))
     existing_user = result.scalars().first()
 
     if existing_user:
-        raise HTTPException(status_code=400, detail="Email already registered")
+        raise HTTPException(
+            status_code=status.HTTP_409_CONFLICT,
+            detail="Email already registered"
+        )
 
-    hashed_pwd = get_password_hash(password)
-    new_user = User(email=email, hashed_password=hashed_pwd, name=name, role="user")
+    # 3. Hash Password
+    hashed_pwd = get_password_hash(user_data.password)
+    
+    # 4. Crear Usuario
+    new_user = User(
+        email=email_norm, 
+        hashed_password=hashed_pwd, 
+        name=user_data.name, 
+        role="user",
+        plan="free",
+        plan_status="active",
+        created_at=datetime.utcnow() # Ensure model has this or defaults
+    )
 
-    db.add(new_user)
     try:
+        db.add(new_user)
         db.commit()
         db.refresh(new_user)
-        return {"id": new_user.id, "email": new_user.email, "msg": "User created"}
+        return new_user
+        
     except IntegrityError:
         db.rollback()
-        raise HTTPException(status_code=400, detail="Error creating user")
-
-    return {
-        "id": current_user.id,
-        "email": current_user.email,
-        "name": current_user.name,
-        "role": current_user.role,
-        "plan": current_user.plan,
-        "plan_status": current_user.plan_status,
-        "avatar_url": f"https://ui-avatars.com/api/?name={current_user.name}&background=10b981&color=fff"
-    }
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail="Error creating user (Integrity)"
+        )
+    except Exception as e:
+        db.rollback()
+        print(f"[AUTH ERROR] Register failed: {e}")
+        raise HTTPException(
+            status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
+            detail="Internal server error during registration"
+        )
 
 # Entitlements Endpoint (Sync DB required for core logic)
 from database import SessionLocal

web/src/App.tsx

@@ -13,6 +13,7 @@ import { PricingPage } from './pages/PricingPage';
 import { AdminPage } from './pages/AdminPage';
 import { SettingsPage } from './pages/SettingsPage';
 import { LoginPage } from './pages/LoginPage';
+import { RegisterPage } from './pages/RegisterPage';
 import { AuthProvider, useAuth } from './context/AuthContext';
 import { ThemeProvider } from './context/ThemeContext';
 import { ToastContainer } from './components/Toast';
@@ -46,6 +47,7 @@ const App: React.FC = () => {
           <ToastContainer />
           <Routes>
             <Route path="/login" element={<LoginPage />} />
+            <Route path="/register" element={<RegisterPage />} />
 
             {/* Protected Routes */}
             <Route
@@ -74,10 +76,10 @@ const App: React.FC = () => {
             <Route path="*" element={<Navigate to="/" replace />} />
           </Routes>
 
-        
 
-      </Router>
-    </ThemeProvider>
+
+        </Router>
+      </ThemeProvider>
     </AuthProvider>
   );
 };

web/src/pages/LoginPage.tsx

@@ -14,7 +14,7 @@ import {
   CheckCircle2,
   ChevronRight
 } from 'lucide-react';
-import { useNavigate } from 'react-router-dom';
+import { useNavigate, Link } from 'react-router-dom';
 
 export const LoginPage: React.FC = () => {
   const [email, setEmail] = useState('demo@tradercopilot.com');
@@ -180,6 +180,12 @@ export const LoginPage: React.FC = () => {
                 >
                   {loading ? <Loader2 className="animate-spin" size={20} /> : <>Initialize Session <ChevronRight size={18} className="group-hover/btn:translate-x-0.5 transition-transform" /></>}
                 </button>
+
+                <div className="text-center pt-2">
+                  <Link to="/register" className="text-xs text-slate-400 hover:text-emerald-400 transition-colors">
+                    Don't have an account? <span className="font-bold text-slate-300">Register Access</span>
+                  </Link>
+                </div>
               </form>