use actix_web::{HttpResponse, web};
use uuid::Uuid;

use crate::api::response::{ApiErrorResponse, ApiResponse};
use crate::error::AppError;
use crate::models::workspaces::WorkspaceWebhook;
use crate::service::AppService;
use crate::service::workspace::webhooks::UpdateWebhookParams;
use crate::session::Session;

#[utoipa::path(
    put,
    path = "/api/v1/workspaces/{workspace_name}/webhooks/{webhook_id}",
    tag = "Workspaces",
    operation_id = "workspaceUpdateWebhook",
    summary = "Update a webhook",
    description = "Update a webhook's URL, secret, events, or active state. Requires admin role.",
    params(
        ("workspace_name" = String, Path, description = "Workspace name."),
        ("webhook_id" = Uuid, Path, description = "Webhook ID.")
    ),
    request_body(
        content = UpdateWebhookParams,
        description = "Webhook update parameters — only included fields are changed.",
        content_type = "application/json"
    ),
    responses(
        (status = 200, description = "Webhook updated.", body = ApiResponse<WorkspaceWebhook>),
        (status = 400, description = "Invalid URL.", body = ApiErrorResponse),
        (status = 401, description = "Unauthenticated or insufficient role.", body = ApiErrorResponse),
        (status = 404, description = "Webhook not found.", body = ApiErrorResponse),
        (status = 500, description = "Database transaction failed.", body = ApiErrorResponse)
    )
)]
pub async fn handle(
    service: web::Data<AppService>,
    session: Session,
    path: web::Path<(String, Uuid)>,
    params: web::Json<UpdateWebhookParams>,
) -> Result<HttpResponse, AppError> {
    let (ws_name, webhook_id) = path.into_inner();
    let ws = service.workspace.find_workspace_by_name(&ws_name).await?;
    let data = service
        .workspace
        .workspace_update_webhook(&session, &ws, webhook_id, params.into_inner())
        .await?;
    Ok(HttpResponse::Ok().json(ApiResponse::new(data)))
}