use anyhow::{Context, Result};
use clap::{Parser, Subcommand};
use crate::atlassian::adf::AdfDocument;
use crate::atlassian::confluence_api::{ConfluenceApi, ConfluenceComment};
use crate::atlassian::convert::{adf_to_markdown, markdown_to_adf};
use crate::atlassian::document::JfmDocument;
use crate::cli::atlassian::format::{output_as, ContentFormat, OutputFormat};
use crate::cli::atlassian::helpers::{create_client, read_input};
#[derive(Parser)]
pub struct CommentCommand {
#[command(subcommand)]
pub command: CommentSubcommands,
}
#[derive(Subcommand)]
pub enum CommentSubcommands {
List(ListCommand),
Add(AddCommand),
}
impl CommentCommand {
pub async fn execute(self) -> Result<()> {
match self.command {
CommentSubcommands::List(cmd) => cmd.execute().await,
CommentSubcommands::Add(cmd) => cmd.execute().await,
}
}
}
#[derive(Parser)]
pub struct ListCommand {
pub id: String,
#[arg(long, default_value_t = 25)]
pub limit: usize,
#[arg(short = 'o', long, value_enum, default_value_t = OutputFormat::Table)]
pub output: OutputFormat,
}
impl ListCommand {
pub async fn execute(self) -> Result<()> {
let (client, _instance_url) = create_client()?;
let api = ConfluenceApi::new(client);
run_list_comments(&api, &self.id, self.limit, &self.output).await
}
}
#[derive(Parser)]
pub struct AddCommand {
pub id: String,
pub file: Option<String>,
#[arg(long, value_enum, default_value_t = ContentFormat::Jfm)]
pub format: ContentFormat,
}
impl AddCommand {
pub async fn execute(self) -> Result<()> {
let adf = self.parse_input()?;
let (client, _instance_url) = create_client()?;
let api = ConfluenceApi::new(client);
run_add_comment(&api, &self.id, &adf).await
}
fn parse_input(&self) -> Result<AdfDocument> {
let input = read_input(self.file.as_deref())?;
match self.format {
ContentFormat::Jfm => {
if input.starts_with("---\n") {
let doc = JfmDocument::parse(&input)?;
markdown_to_adf(&doc.body)
} else {
markdown_to_adf(&input)
}
}
ContentFormat::Adf => {
serde_json::from_str(&input).context("Failed to parse ADF JSON input")
}
}
}
}
async fn run_list_comments(
api: &ConfluenceApi,
id: &str,
limit: usize,
output: &OutputFormat,
) -> Result<()> {
let mut comments = api.get_page_comments(id).await?;
comments.truncate(limit);
if output_as(&comments, output)? {
return Ok(());
}
print_comments(&comments);
Ok(())
}
async fn run_add_comment(api: &ConfluenceApi, id: &str, adf: &AdfDocument) -> Result<()> {
api.add_page_comment(id, adf).await?;
println!("Comment added to page {id}.");
Ok(())
}
fn print_comments(comments: &[ConfluenceComment]) {
if comments.is_empty() {
println!("No comments.");
return;
}
for (i, comment) in comments.iter().enumerate() {
if i > 0 {
println!();
}
let timestamp = format_timestamp(&comment.created);
println!("--- {} | {} ---", comment.author, timestamp);
println!("{}", format_comment_body(&comment.body_adf));
}
}
fn format_comment_body(body_adf: &Option<serde_json::Value>) -> String {
let Some(adf_value) = body_adf else {
return "[empty]".to_string();
};
let Ok(adf) = serde_json::from_value::<AdfDocument>(adf_value.clone()) else {
return "[ADF content]".to_string();
};
let Ok(md) = adf_to_markdown(&adf) else {
return "[ADF content]".to_string();
};
let trimmed = md.trim();
if trimmed.is_empty() {
"[empty]".to_string()
} else {
trimmed.to_string()
}
}
fn format_timestamp(ts: &str) -> &str {
ts.split('.').next().unwrap_or(ts)
}
#[cfg(test)]
#[allow(clippy::unwrap_used, clippy::expect_used)]
mod tests {
use super::*;
use std::fs;
fn sample_comment(
id: &str,
author: &str,
body_adf: Option<serde_json::Value>,
) -> ConfluenceComment {
ConfluenceComment {
id: id.to_string(),
author: author.to_string(),
body_adf,
created: "2026-04-01T10:30:00.000Z".to_string(),
}
}
#[test]
fn print_comments_empty() {
print_comments(&[]);
}
#[test]
fn print_comments_with_adf_body() {
let adf = serde_json::json!({
"version": 1,
"type": "doc",
"content": [{"type": "paragraph", "content": [{"type": "text", "text": "Hello world"}]}]
});
let comments = vec![sample_comment("1", "Alice", Some(adf))];
print_comments(&comments);
}
#[test]
fn print_comments_with_null_body() {
let comments = vec![sample_comment("1", "Bob", None)];
print_comments(&comments);
}
#[test]
fn print_comments_with_invalid_adf() {
let invalid = serde_json::json!({"not": "adf"});
let comments = vec![sample_comment("1", "Carol", Some(invalid))];
print_comments(&comments);
}
#[test]
fn print_comments_multiple() {
let adf = serde_json::json!({
"version": 1,
"type": "doc",
"content": [{"type": "paragraph", "content": [{"type": "text", "text": "First"}]}]
});
let comments = vec![
sample_comment("1", "Alice", Some(adf)),
sample_comment("2", "Bob", None),
];
print_comments(&comments);
}
#[test]
fn format_body_none() {
assert_eq!(format_comment_body(&None), "[empty]");
}
#[test]
fn format_body_valid_adf_with_text() {
let adf = serde_json::json!({
"version": 1,
"type": "doc",
"content": [{"type": "paragraph", "content": [{"type": "text", "text": "Hello"}]}]
});
let result = format_comment_body(&Some(adf));
assert_eq!(result, "Hello");
}
#[test]
fn format_body_valid_adf_empty_content() {
let adf = serde_json::json!({
"version": 1,
"type": "doc",
"content": []
});
let result = format_comment_body(&Some(adf));
assert_eq!(result, "[empty]");
}
#[test]
fn format_body_invalid_adf() {
let invalid = serde_json::json!({"not": "adf"});
assert_eq!(format_comment_body(&Some(invalid)), "[ADF content]");
}
#[test]
fn format_timestamp_with_millis() {
assert_eq!(
format_timestamp("2026-04-01T10:30:00.000Z"),
"2026-04-01T10:30:00"
);
}
#[test]
fn format_timestamp_without_millis() {
assert_eq!(
format_timestamp("2026-04-01T10:30:00"),
"2026-04-01T10:30:00"
);
}
#[test]
fn format_timestamp_empty() {
assert_eq!(format_timestamp(""), "");
}
#[test]
fn parse_input_raw_markdown() {
let temp_dir = tempfile::tempdir().unwrap();
let file_path = temp_dir.path().join("comment.md");
fs::write(&file_path, "Hello **world**\n").unwrap();
let cmd = AddCommand {
id: "12345".to_string(),
file: Some(file_path.to_str().unwrap().to_string()),
format: ContentFormat::Jfm,
};
let adf = cmd.parse_input().unwrap();
assert!(!adf.content.is_empty());
}
#[test]
fn parse_input_jfm_with_frontmatter() {
let temp_dir = tempfile::tempdir().unwrap();
let file_path = temp_dir.path().join("comment.md");
let content =
"---\ntype: confluence\ninstance: https://org.atlassian.net\nid: \"12345\"\ntitle: Test\nspace_key: ENG\n---\n\nComment body\n";
fs::write(&file_path, content).unwrap();
let cmd = AddCommand {
id: "12345".to_string(),
file: Some(file_path.to_str().unwrap().to_string()),
format: ContentFormat::Jfm,
};
let adf = cmd.parse_input().unwrap();
assert!(!adf.content.is_empty());
}
#[test]
fn parse_input_adf_format() {
let temp_dir = tempfile::tempdir().unwrap();
let file_path = temp_dir.path().join("comment.json");
let adf_json = r#"{"version":1,"type":"doc","content":[{"type":"paragraph","content":[{"type":"text","text":"Hello"}]}]}"#;
fs::write(&file_path, adf_json).unwrap();
let cmd = AddCommand {
id: "12345".to_string(),
file: Some(file_path.to_str().unwrap().to_string()),
format: ContentFormat::Adf,
};
let adf = cmd.parse_input().unwrap();
assert_eq!(adf.content.len(), 1);
}
#[test]
fn parse_input_invalid_adf() {
let temp_dir = tempfile::tempdir().unwrap();
let file_path = temp_dir.path().join("bad.json");
fs::write(&file_path, "not json").unwrap();
let cmd = AddCommand {
id: "12345".to_string(),
file: Some(file_path.to_str().unwrap().to_string()),
format: ContentFormat::Adf,
};
assert!(cmd.parse_input().is_err());
}
#[test]
fn comment_command_list_variant() {
let cmd = CommentCommand {
command: CommentSubcommands::List(ListCommand {
id: "12345".to_string(),
limit: 25,
output: OutputFormat::Table,
}),
};
assert!(matches!(cmd.command, CommentSubcommands::List(_)));
}
#[test]
fn comment_command_add_variant() {
let cmd = CommentCommand {
command: CommentSubcommands::Add(AddCommand {
id: "12345".to_string(),
file: None,
format: ContentFormat::Jfm,
}),
};
assert!(matches!(cmd.command, CommentSubcommands::Add(_)));
}
fn mock_api(server: &wiremock::MockServer) -> ConfluenceApi {
let client =
crate::atlassian::client::AtlassianClient::new(&server.uri(), "user@test.com", "token")
.unwrap();
ConfluenceApi::new(client)
}
#[tokio::test]
async fn run_list_comments_success() {
let server = wiremock::MockServer::start().await;
wiremock::Mock::given(wiremock::matchers::method("GET"))
.and(wiremock::matchers::path(
"/wiki/api/v2/pages/12345/footer-comments",
))
.respond_with(
wiremock::ResponseTemplate::new(200)
.set_body_json(serde_json::json!({"results": []})),
)
.mount(&server)
.await;
let api = mock_api(&server);
assert!(run_list_comments(&api, "12345", 25, &OutputFormat::Table)
.await
.is_ok());
}
#[tokio::test]
async fn run_list_comments_json_output() {
let server = wiremock::MockServer::start().await;
wiremock::Mock::given(wiremock::matchers::method("GET"))
.and(wiremock::matchers::path(
"/wiki/api/v2/pages/12345/footer-comments",
))
.respond_with(
wiremock::ResponseTemplate::new(200)
.set_body_json(serde_json::json!({"results": []})),
)
.mount(&server)
.await;
let api = mock_api(&server);
assert!(run_list_comments(&api, "12345", 25, &OutputFormat::Json)
.await
.is_ok());
}
#[tokio::test]
async fn run_list_comments_api_error() {
let server = wiremock::MockServer::start().await;
wiremock::Mock::given(wiremock::matchers::method("GET"))
.and(wiremock::matchers::path(
"/wiki/api/v2/pages/99999/footer-comments",
))
.respond_with(wiremock::ResponseTemplate::new(404).set_body_string("Not Found"))
.mount(&server)
.await;
let api = mock_api(&server);
let err = run_list_comments(&api, "99999", 25, &OutputFormat::Table)
.await
.unwrap_err();
assert!(err.to_string().contains("404"));
}
#[tokio::test]
async fn run_add_comment_success() {
let server = wiremock::MockServer::start().await;
wiremock::Mock::given(wiremock::matchers::method("POST"))
.and(wiremock::matchers::path("/wiki/api/v2/footer-comments"))
.respond_with(
wiremock::ResponseTemplate::new(200)
.set_body_json(serde_json::json!({"id": "200"})),
)
.mount(&server)
.await;
let api = mock_api(&server);
let adf = AdfDocument::new();
assert!(run_add_comment(&api, "12345", &adf).await.is_ok());
}
#[tokio::test]
async fn run_add_comment_api_error() {
let server = wiremock::MockServer::start().await;
wiremock::Mock::given(wiremock::matchers::method("POST"))
.and(wiremock::matchers::path("/wiki/api/v2/footer-comments"))
.respond_with(wiremock::ResponseTemplate::new(403).set_body_string("Forbidden"))
.mount(&server)
.await;
let api = mock_api(&server);
let adf = AdfDocument::new();
let err = run_add_comment(&api, "12345", &adf).await.unwrap_err();
assert!(err.to_string().contains("403"));
}
}