src/hubgw/api/v1/homes.py

@@ -6,9 +6,8 @@ from fastapi import APIRouter, Depends
 
 from hubgw.api.deps import get_homes_service, verify_api_key
 from hubgw.core.errors import AppError, create_http_exception
-from hubgw.schemas.homes import (Home, HomeCountResponse, HomeGetRequest,
-                                 HomeGetResponse, HomeListResponse,
-                                 HomeUpsertRequest)
+from hubgw.schemas.homes import (Home, HomeGetRequest, HomeGetResponse,
+                                 HomeListResponse, HomeUpsertRequest)
 from hubgw.services.homes_service import HomesService
 
 router = APIRouter()
@@ -40,33 +39,6 @@ async def get_home(
         raise create_http_exception(e)
 
 
-@router.post("/delete")
-async def delete_home(
-    request: HomeGetRequest,
-    service: Annotated[HomesService, Depends(get_homes_service)],
-    _: Annotated[str, Depends(verify_api_key)],
-):
-    """Delete home by player UUID and name."""
-    try:
-        await service.delete_home(request)
-        return {"success": True}
-    except AppError as e:
-        raise create_http_exception(e)
-
-
-@router.get("/count/{player_uuid}", response_model=HomeCountResponse)
-async def count_homes(
-    player_uuid: str,
-    service: Annotated[HomesService, Depends(get_homes_service)],
-    _: Annotated[str, Depends(verify_api_key)],
-):
-    """Count homes for player."""
-    try:
-        return await service.count_homes(player_uuid)
-    except AppError as e:
-        raise create_http_exception(e)
-
-
 @router.get("/{player_uuid}", response_model=HomeListResponse)
 async def list_homes(
     player_uuid: str,

src/hubgw/models/teleport_history.py

@@ -1,7 +1,5 @@
 """Teleport History model."""
 
-from __future__ import annotations
-
 from sqlalchemy import Column, DateTime, Float, ForeignKey, String, Text
 from sqlalchemy.dialects.postgresql import UUID
 from sqlalchemy.sql import func
@@ -15,11 +13,7 @@ _LUCKPERMS_SCHEMA = APP_CONFIG.database.luckperms_schema
 
 
 class TeleportHistory(Base):
-    """Teleport History model.
-
-    This is an append-only audit log table, so it doesn't have updated_at column.
-    We override updated_at to None to exclude it from this model.
-    """
+    """Teleport History model."""
 
     __tablename__ = "hub_teleport_history"
     __table_args__ = {"schema": _HUB_SCHEMA}
@@ -41,11 +35,8 @@ class TeleportHistory(Base):
     to_x = Column(Float, nullable=False)  # DOUBLE PRECISION
     to_y = Column(Float, nullable=False)  # DOUBLE PRECISION
     to_z = Column(Float, nullable=False)  # DOUBLE PRECISION
-    tp_type = Column(String(50), nullable=False, index=True)
+    tp_type = Column(String(50), nullable=False)
     target_name = Column(String(255))
     created_at = Column(
-        DateTime(timezone=True), server_default=func.now(), nullable=False, index=True
+        DateTime(timezone=True), server_default=func.now(), nullable=False
     )
-
-    # Override updated_at from Base to exclude it
-    updated_at = None

src/hubgw/repositories/cooldowns_repo.py

@@ -4,7 +4,6 @@ from typing import List, Optional
 from uuid import UUID
 
 from sqlalchemy import delete, func, select
-from sqlalchemy.dialects.postgresql import insert
 from sqlalchemy.ext.asyncio import AsyncSession
 
 from hubgw.models.cooldown import Cooldown
@@ -18,30 +17,16 @@ class CooldownsRepository:
         self.session = session
 
     async def create(self, request: CooldownCreate) -> Cooldown:
-        """Create or update cooldown (UPSERT)."""
-        stmt = (
-            insert(Cooldown)
-            .values(
-                player_uuid=request.player_uuid,
-                cooldown_type=request.cooldown_type,
-                expires_at=request.expires_at,
-                cooldown_seconds=request.cooldown_seconds,
-                cooldown_metadata=request.metadata,
-            )
-            .on_conflict_do_update(
-                index_elements=["player_uuid", "cooldown_type"],
-                set_={
-                    "expires_at": request.expires_at,
-                    "cooldown_seconds": request.cooldown_seconds,
-                    "metadata": request.metadata,
-                    "updated_at": func.now(),
-                },
-            )
-            .returning(Cooldown)
+        """Create new cooldown."""
+        cooldown = Cooldown(
+            player_uuid=request.player_uuid,
+            cooldown_type=request.cooldown_type,
+            expires_at=request.expires_at,
+            cooldown_seconds=request.cooldown_seconds,
+            cooldown_metadata=request.metadata,
         )
-        result = await self.session.execute(stmt)
+        self.session.add(cooldown)
         await self.session.commit()
-        cooldown = result.scalar_one()
         await self.session.refresh(cooldown)
         return cooldown
 

src/hubgw/repositories/homes_repo.py

@@ -74,13 +74,6 @@ class HomesRepository:
         await self.session.commit()
         return result.rowcount > 0
 
-    async def delete_by_player_and_name(self, player_uuid: str, name: str) -> bool:
-        """Delete home by player UUID and name."""
-        stmt = delete(Home).where(Home.player_uuid == player_uuid, Home.name == name)
-        result = await self.session.execute(stmt)
-        await self.session.commit()
-        return result.rowcount > 0
-
     async def query(self, query: HomeQuery) -> tuple[List[Home], int]:
         """Query homes with filters and pagination."""
         stmt = select(Home)
@@ -126,9 +119,3 @@ class HomesRepository:
         stmt = select(Home).where(Home.player_uuid == player_uuid).order_by(Home.name)
         result = await self.session.execute(stmt)
         return list(result.scalars().all())
-
-    async def count_by_player(self, player_uuid: str) -> int:
-        """Count homes for a player."""
-        stmt = select(func.count(Home.id)).where(Home.player_uuid == player_uuid)
-        result = await self.session.execute(stmt)
-        return result.scalar()

src/hubgw/schemas/homes.py

@@ -73,12 +73,6 @@ class HomeListResponse(BaseModel):
     total: int
 
 
-class HomeCountResponse(BaseModel):
-    """Home count response schema."""
-
-    total: int
-
-
 class HomeQuery(PaginationParams):
     """Home query schema."""
 

src/hubgw/schemas/teleport_history.py

@@ -6,7 +6,7 @@ from uuid import UUID
 
 from pydantic import BaseModel
 
-from hubgw.schemas.common import PaginationParams
+from hubgw.schemas.common import BaseSchema, PaginationParams
 
 
 class TeleportHistoryBase(BaseModel):
@@ -30,11 +30,8 @@ class TeleportHistoryCreate(TeleportHistoryBase):
     player_uuid: str
 
 
-class TeleportHistory(TeleportHistoryBase):
-    """Teleport history response schema.
-
-    This is an append-only audit log, so it only has created_at (no updated_at).
-    """
+class TeleportHistory(TeleportHistoryBase, BaseSchema):
+    """Teleport history schema."""
 
     id: UUID
     player_uuid: str

src/hubgw/services/homes_service.py

@@ -5,9 +5,8 @@ from sqlalchemy.ext.asyncio import AsyncSession
 
 from hubgw.core.errors import NotFoundError
 from hubgw.repositories.homes_repo import HomesRepository
-from hubgw.schemas.homes import (Home, HomeCountResponse, HomeGetRequest,
-                                 HomeGetResponse, HomeListResponse,
-                                 HomeUpsertRequest)
+from hubgw.schemas.homes import (Home, HomeGetRequest, HomeGetResponse,
+                                 HomeListResponse, HomeUpsertRequest)
 
 
 class HomesService:
@@ -66,19 +65,3 @@ class HomesService:
         ]
 
         return HomeListResponse(homes=home_list, total=len(home_list))
-
-    async def count_homes(self, player_uuid: str) -> HomeCountResponse:
-        """Count homes for player."""
-        total = await self.repo.count_by_player(player_uuid)
-        return HomeCountResponse(total=total)
-
-    async def delete_home(self, request: HomeGetRequest) -> bool:
-        """Delete home by player UUID and name."""
-        deleted = await self.repo.delete_by_player_and_name(
-            request.player_uuid, request.name
-        )
-        if not deleted:
-            raise NotFoundError(
-                f"Home '{request.name}' not found for player {request.player_uuid}"
-            )
-        return deleted