use axum::extract::{Path, Query, State};
use export::rest::AppResponseData;
use session::Session;

use crate::errors::{ApiError, GitApiError};
use crate::handler::resolve_reader;
use crate::service::GitService;
use crate::service::logs::LogListDto;

#[derive(Debug, Clone, serde::Deserialize)]
pub struct BetweenQuery {
    pub after: String,
    pub before: String,
    #[serde(default)]
    pub limit: Option<u64>,
}

pub async fn log_commits_between(
    State(service): State<GitService>,
    session: Session,
    Path(path): Path<crate::handler::RepoPath>,
    Query(query): Query<BetweenQuery>,
) -> Result<AppResponseData<LogListDto>, ApiError> {
    let _ = resolve_reader(&service, &session, path.space_name.clone(), path.repo_name.clone()).await?;
    let out = service
        .log_commits_between(
            path.space_name,
            path.repo_name,
            query.after,
            query.before,
            query.limit.unwrap_or(50),
        )
        .await
        .map_err(GitApiError::from)?;
    Ok(AppResponseData::ok(out))
}