use actix_web::{HttpResponse, web};
use serde::Deserialize;
use utoipa::IntoParams;

use crate::api::response::{ApiErrorResponse, ApiResponse};
use crate::error::AppError;
use crate::models::prs::PullRequest;
use crate::service::AppService;
use crate::session::Session;

#[derive(Debug, Deserialize, IntoParams)]
pub struct PathParams {
    /// Workspace name (unique identifier)
    pub workspace_name: String,
    /// Repository name (unique within the workspace)
    pub repo_name: String,
    /// PR number (unique within the repository)
    pub number: i64,
}

/// Mark a draft PR as ready for review
///
/// Converts a draft pull request to a ready-for-review state.
/// Requires write access to the PR.
///
/// Effects:
/// - Draft flag is set to false
/// - A "DraftReady" event is logged
/// - Reviewers are notified that the PR is ready
///
/// Returns the updated PR.
#[utoipa::path(
    post,
    path = "/api/v1/workspaces/{workspace_name}/repos/{repo_name}/prs/{number}/ready",
    tag = "Pull Requests",
    operation_id = "prMarkReady",
    params(PathParams),
    responses(
        (status = 200, description = "PR marked as ready for review.", body = ApiResponse<PullRequest>),
        (status = 400, description = "PR is already ready for review", body = ApiErrorResponse),
        (status = 401, description = "Authentication required or session expired", body = ApiErrorResponse),
        (status = 403, description = "Insufficient permissions", body = ApiErrorResponse),
        (status = 404, description = "PR not found", body = ApiErrorResponse),
        (status = 500, description = "Internal server error", body = ApiErrorResponse),
    ),
    security(
        ("session_cookie" = [])
    )
)]
pub async fn mark_ready(
    service: web::Data<AppService>,
    session: Session,
    path: web::Path<PathParams>,
) -> Result<HttpResponse, AppError> {
    let pr = service
        .pr
        .pr_mark_ready(&session, &path.workspace_name, &path.repo_name, path.number)
        .await?;
    Ok(HttpResponse::Ok().json(ApiResponse::new(pr)))
}