From 269bfaab2c08748b05129a600a350ec53b45f067 Mon Sep 17 00:00:00 2001
From: Felix Zett <fezett@gmail.com>
Date: Sat, 9 Aug 2025 21:35:45 +0200
Subject: [PATCH] Simplify endpoint paths and enhance readability

---
 backend/main.py         |  7 +++----
 backend/routes/items.py | 12 ++++++------
 backend/routes/trips.py |  8 ++++----
 3 files changed, 13 insertions(+), 14 deletions(-)

diff --git a/backend/main.py b/backend/main.py
index 7237294..010936a 100644
--- a/backend/main.py
+++ b/backend/main.py
@@ -1,12 +1,11 @@
 from fastapi import FastAPI
-from routes.items import router as items_router
-from routes.trips import router as trips_router
+from routes import items, trips
 from models import Base
 from database import engine
 
 app = FastAPI()
-app.include_router(items_router)
-app.include_router(trips_router)
+app.include_router(items.router)
+app.include_router(trips.router)
 
 @app.on_event("startup")
 async def startup():
diff --git a/backend/routes/items.py b/backend/routes/items.py
index 55611a4..c359197 100644
--- a/backend/routes/items.py
+++ b/backend/routes/items.py
@@ -4,13 +4,13 @@ from sqlalchemy.ext.asyncio import AsyncSession
 from schemas import ItemCreate
 from models import Item, Tag
 from database import get_db
-from sqlalchemy import select, update
+from sqlalchemy import select
 from sqlalchemy.orm import selectinload
 from config import FIXED_USER_ID
 
-router = APIRouter()
+router = APIRouter(prefix="/items", tags=["Items"])
 
-@router.post("/items/")
+@router.post("/")
 async def create_item(item: ItemCreate, db: AsyncSession = Depends(get_db)):
     user_id = FIXED_USER_ID
 
@@ -30,7 +30,7 @@ async def create_item(item: ItemCreate, db: AsyncSession = Depends(get_db)):
     await db.commit()
     return {"status": "item created", "item_id": str(db_item.id)}
 
-@router.get("/items/")
+@router.get("/")
 async def read_items(db: AsyncSession = Depends(get_db)):
     user_id = FIXED_USER_ID
     result = await db.execute(select(Item).where(Item.user_id == user_id).options(selectinload(Item.tags)))
@@ -46,7 +46,7 @@ async def read_items(db: AsyncSession = Depends(get_db)):
         ]
     }
 
-@router.put("/items/{item_id}")
+@router.put("/{item_id}")
 async def update_item(item_id: str, item: ItemCreate, db: AsyncSession = Depends(get_db)):
     user_id = FIXED_USER_ID
 
@@ -71,7 +71,7 @@ async def update_item(item_id: str, item: ItemCreate, db: AsyncSession = Depends
     await db.commit()
     return {"status": "item updated", "item_id": str(db_item.id)}
 
-@router.get("/items/{item_id}")
+@router.get("/{item_id}")
 async def get_item(item_id: str, db: AsyncSession = Depends(get_db)):
     user_id = FIXED_USER_ID
     result = await db.execute(
diff --git a/backend/routes/trips.py b/backend/routes/trips.py
index a82f0a0..344e6c1 100644
--- a/backend/routes/trips.py
+++ b/backend/routes/trips.py
@@ -9,9 +9,9 @@ from sqlalchemy import select
 from sqlalchemy.orm import selectinload
 from config import FIXED_USER_ID
 
-router = APIRouter()
+router = APIRouter(prefix="/trips", tags=["Trips"])
 
-@router.post("/trips/")
+@router.post("/")
 async def create_trip(trip: TripCreate, db: AsyncSession = Depends(get_db)):
     user_id = FIXED_USER_ID
 
@@ -76,7 +76,7 @@ async def create_trip(trip: TripCreate, db: AsyncSession = Depends(get_db)):
     return {"status": "trip created", "trip_id": str(db_trip.id)}
 
 
-@router.get("/trips/{trip_id}/items")
+@router.get("/{trip_id}/items")
 async def get_trip_items(trip_id: UUID, db: AsyncSession = Depends(get_db)):
     result = await db.execute(select(TripItem).where(TripItem.trip_id == trip_id))
     items = result.scalars().all()
@@ -88,7 +88,7 @@ async def get_trip_items(trip_id: UUID, db: AsyncSession = Depends(get_db)):
         } for item in items
     ]
 
-@router.get("/trips/")
+@router.get("/")
 async def get_trips(db: AsyncSession = Depends(get_db)):
     user_id = FIXED_USER_ID
     result = await db.execute(select(Trip).where(Trip.user_id == user_id))