fix: wiki/routes.py — add missing db parameter to all service calls
Check Cross-Plugin Imports / check (push) Has been cancelled

All wiki service functions expect db as first argument but routes
were passing tenant_id as first argument. This caused 500 on
/wiki/articles, /wiki/categories, and all wiki endpoints.
This commit is contained in:
Agent Zero
2026-08-21 10:08:33 +02:00
parent e92034f1b6
commit 72e3756c60
+72 -18
View File
@@ -12,56 +12,110 @@ router = APIRouter(prefix="/api/v1/wiki", tags=["wiki"])
@router.get("/articles") @router.get("/articles")
async def list_articles(page: int = Query(1, ge=1), page_size: int = Query(20, ge=1, le=100), category_id: str | None = None, status: str | None = None, search: str | None = None, db: AsyncSession = Depends(get_db), current_user: dict = Depends(require_permission("wiki:read"))): async def list_articles(
return await services.list_articles(uuid.UUID(current_user["tenant_id"]), page=page, page_size=page_size, category_id=category_id, status=status, search=search) page: int = Query(1, ge=1),
page_size: int = Query(20, ge=1, le=100),
category_id: str | None = None,
status: str | None = None,
search: str | None = None,
db: AsyncSession = Depends(get_db),
current_user: dict = Depends(require_permission("wiki:read")),
):
return await services.list_articles(
db, uuid.UUID(current_user["tenant_id"]),
page=page, page_size=page_size, category_id=category_id, status=status, search=search,
)
@router.post("/articles", status_code=status.HTTP_201_CREATED) @router.post("/articles", status_code=status.HTTP_201_CREATED)
async def create_article(body: ArticleCreate, db: AsyncSession = Depends(get_db), current_user: dict = Depends(require_permission("wiki:write"))): async def create_article(
return await services.create_article(uuid.UUID(current_user["tenant_id"]), uuid.UUID(current_user["user_id"]), body.model_dump()) body: ArticleCreate,
db: AsyncSession = Depends(get_db),
current_user: dict = Depends(require_permission("wiki:write")),
):
return await services.create_article(
db, uuid.UUID(current_user["tenant_id"]), uuid.UUID(current_user["user_id"]), body.model_dump(),
)
@router.get("/articles/{article_id}") @router.get("/articles/{article_id}")
async def get_article(article_id: str, db: AsyncSession = Depends(get_db), current_user: dict = Depends(require_permission("wiki:read"))): async def get_article(
result = await services.get_article(uuid.UUID(current_user["tenant_id"]), uuid.UUID(article_id)) article_id: str,
db: AsyncSession = Depends(get_db),
current_user: dict = Depends(require_permission("wiki:read")),
):
result = await services.get_article(db, uuid.UUID(current_user["tenant_id"]), uuid.UUID(article_id))
if result is None: if result is None:
raise HTTPException(status_code=404, detail={"detail": "Article not found", "code": "not_found"}) raise HTTPException(status_code=404, detail={"detail": "Article not found", "code": "not_found"})
return result return result
@router.patch("/articles/{article_id}") @router.patch("/articles/{article_id}")
async def update_article(article_id: str, body: ArticleUpdate, db: AsyncSession = Depends(get_db), current_user: dict = Depends(require_permission("wiki:write"))): async def update_article(
result = await services.update_article(uuid.UUID(current_user["tenant_id"]), uuid.UUID(current_user["user_id"]), uuid.UUID(article_id), body.model_dump(exclude_unset=True)) article_id: str,
body: ArticleUpdate,
db: AsyncSession = Depends(get_db),
current_user: dict = Depends(require_permission("wiki:write")),
):
result = await services.update_article(
db, uuid.UUID(current_user["tenant_id"]), uuid.UUID(current_user["user_id"]),
uuid.UUID(article_id), body.model_dump(exclude_unset=True),
)
if result is None: if result is None:
raise HTTPException(status_code=404, detail={"detail": "Article not found", "code": "not_found"}) raise HTTPException(status_code=404, detail={"detail": "Article not found", "code": "not_found"})
return result return result
@router.delete("/articles/{article_id}", status_code=status.HTTP_204_NO_CONTENT) @router.delete("/articles/{article_id}", status_code=status.HTTP_204_NO_CONTENT)
async def delete_article(article_id: str, db: AsyncSession = Depends(get_db), current_user: dict = Depends(require_permission("wiki:delete"))): async def delete_article(
ok = await services.delete_article(uuid.UUID(current_user["tenant_id"]), uuid.UUID(article_id)) article_id: str,
db: AsyncSession = Depends(get_db),
current_user: dict = Depends(require_permission("wiki:delete")),
):
ok = await services.delete_article(db, uuid.UUID(current_user["tenant_id"]), uuid.UUID(article_id))
if not ok: if not ok:
raise HTTPException(status_code=404, detail={"detail": "Article not found", "code": "not_found"}) raise HTTPException(status_code=404, detail={"detail": "Article not found", "code": "not_found"})
@router.get("/articles/{article_id}/versions") @router.get("/articles/{article_id}/versions")
async def list_versions(article_id: str, db: AsyncSession = Depends(get_db), current_user: dict = Depends(require_permission("wiki:read"))): async def list_versions(
return {"items": await services.list_versions(uuid.UUID(current_user["tenant_id"]), uuid.UUID(article_id))} article_id: str,
db: AsyncSession = Depends(get_db),
current_user: dict = Depends(require_permission("wiki:read")),
):
return {"items": await services.list_versions(db, uuid.UUID(current_user["tenant_id"]), uuid.UUID(article_id))}
@router.post("/articles/{article_id}/versions/{version}/restore") @router.post("/articles/{article_id}/versions/{version}/restore")
async def restore_version(article_id: str, version: int, db: AsyncSession = Depends(get_db), current_user: dict = Depends(require_permission("wiki:write"))): async def restore_version(
result = await services.restore_version(uuid.UUID(current_user["tenant_id"]), uuid.UUID(current_user["user_id"]), uuid.UUID(article_id), version) article_id: str,
version: int,
db: AsyncSession = Depends(get_db),
current_user: dict = Depends(require_permission("wiki:write")),
):
result = await services.restore_version(
db, uuid.UUID(current_user["tenant_id"]), uuid.UUID(current_user["user_id"]),
uuid.UUID(article_id), version,
)
if result is None: if result is None:
raise HTTPException(status_code=404, detail={"detail": "Version not found", "code": "not_found"}) raise HTTPException(status_code=404, detail={"detail": "Version not found", "code": "not_found"})
return result return result
@router.get("/categories") @router.get("/categories")
async def list_categories(db: AsyncSession = Depends(get_db), current_user: dict = Depends(require_permission("wiki:read"))): async def list_categories(
return {"items": await services.list_categories(uuid.UUID(current_user["tenant_id"]))} db: AsyncSession = Depends(get_db),
current_user: dict = Depends(require_permission("wiki:read")),
):
return {"items": await services.list_categories(db, uuid.UUID(current_user["tenant_id"]))}
@router.post("/categories", status_code=status.HTTP_201_CREATED) @router.post("/categories", status_code=status.HTTP_201_CREATED)
async def create_category(body: CategoryCreate, db: AsyncSession = Depends(get_db), current_user: dict = Depends(require_permission("wiki:write"))): async def create_category(
return await services.create_category(uuid.UUID(current_user["tenant_id"]), uuid.UUID(current_user["user_id"]), body.model_dump()) body: CategoryCreate,
db: AsyncSession = Depends(get_db),
current_user: dict = Depends(require_permission("wiki:write")),
):
return await services.create_category(
db, uuid.UUID(current_user["tenant_id"]), uuid.UUID(current_user["user_id"]), body.model_dump(),
)