Skip to content

Commit 11ea25f

Browse files
committed
chore: remove unused codes
1 parent 0927d8b commit 11ea25f

16 files changed

+46
-1257
lines changed

.sqlx/query-a86b1bafa09dce47f7cb020c8f3b6545a3472f39fccffb78036495f2249d923e.json

Lines changed: 0 additions & 12 deletions
This file was deleted.

.sqlx/query-d8afba9d2c421e71f3ec567212228c7962f50fd9fee31e53b8d0cef48d2e6faf.json renamed to .sqlx/query-cdbbe9a12319db0b723fd9d40cf431db6dd6943a1df5bbb6096257e84602500a.json

Lines changed: 8 additions & 14 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.sqlx/query-e6df29ab91df154a77f022b2bbd7814145ed9c7aab397815843eab3612462953.json renamed to .sqlx/query-d483b01dc587a29a1df2b2572e50218583812f9cd5db544609fda8c8541933aa.json

Lines changed: 8 additions & 14 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.sqlx/query-e983e83e9bb71f42a062af6ddc6ee674bac9238e2bdb760e41f377653cadd518.json

Lines changed: 12 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/cli/analytics.rs

Lines changed: 5 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ use std::sync::Arc;
55
use crate::database::DatabaseManager;
66
use crate::env::apis as env_vars;
77
use crate::models::OperationStatus;
8-
use crate::services::analytics::formatters::{AnalyticsFormatter, OutputFormat};
98
use crate::services::{
109
google_ai::{GoogleAiClient, GoogleAiConfig},
1110
AnalyticsRequestService,
@@ -71,8 +70,6 @@ pub async fn handle_execute_command(
7170
custom_prompt: Option<String>,
7271
all: bool,
7372
background: bool,
74-
format: String,
75-
plain: bool,
7673
) -> Result<()> {
7774
let db_path = crate::database::config::get_default_db_path()?;
7875
let db_manager = Arc::new(DatabaseManager::new(&db_path).await?);
@@ -94,8 +91,6 @@ pub async fn handle_execute_command(
9491
session_id,
9592
custom_prompt,
9693
background,
97-
format,
98-
plain,
9994
)
10095
.await
10196
} else {
@@ -108,8 +103,6 @@ async fn execute_analysis_for_session(
108103
session_id: String,
109104
custom_prompt: Option<String>,
110105
background: bool,
111-
format: String,
112-
plain: bool,
113106
) -> Result<()> {
114107
println!("Starting analysis for session: {session_id}");
115108

@@ -141,20 +134,13 @@ async fn execute_analysis_for_session(
141134
.filter(|r| matches!(r.status, OperationStatus::Completed))
142135
.max_by_key(|r| r.completed_at.as_ref())
143136
{
144-
// Determine output format
145-
let output_format = if plain {
146-
OutputFormat::Plain
147-
} else {
148-
OutputFormat::parse(&format)
149-
};
150-
151137
// Get and display cached results
152138
if let Some(analysis) = service
153139
.get_analysis_result(latest_request.id.clone())
154140
.await
155141
.map_err(|e| anyhow::anyhow!("Failed to get cached analysis: {e}"))?
156142
{
157-
print_unified_analysis(&analysis, output_format).await?;
143+
print_unified_analysis(&analysis).await?;
158144
println!(
159145
"\n✓ Showing cached results from: {}",
160146
latest_request
@@ -189,20 +175,13 @@ async fn execute_analysis_for_session(
189175
Ok(_) => {
190176
println!("✓ Analysis completed successfully");
191177

192-
// Determine output format
193-
let output_format = if plain {
194-
OutputFormat::Plain
195-
} else {
196-
OutputFormat::parse(&format)
197-
};
198-
199178
// Get and display results
200179
if let Some(analysis) = service
201180
.get_analysis_result(request.id.clone())
202181
.await
203182
.map_err(|e| anyhow::anyhow!("Failed to get analysis result: {e}"))?
204183
{
205-
print_unified_analysis(&analysis, output_format).await?;
184+
print_unified_analysis(&analysis).await?;
206185
}
207186
}
208187
Err(e) => {
@@ -239,7 +218,6 @@ async fn execute_analysis_for_all_sessions(
239218
pub async fn handle_show_command(
240219
session_id: Option<String>,
241220
all: bool,
242-
_format: String,
243221
) -> Result<()> {
244222
let db_path = crate::database::config::get_default_db_path()?;
245223
let db_manager = Arc::new(DatabaseManager::new(&db_path).await?);
@@ -525,9 +503,8 @@ async fn list_cancellable_requests(service: &AnalyticsRequestService) -> Result<
525503
// =============================================================================
526504

527505
async fn print_unified_analysis(
528-
analysis: &crate::models::Analytics,
529-
output_format: OutputFormat,
506+
_analysis: &crate::models::Analytics,
530507
) -> Result<()> {
531-
let formatter = AnalyticsFormatter::new(output_format);
532-
formatter.print_analysis(analysis)
508+
println!("Good!");
509+
Ok(())
533510
}

src/cli/mod.rs

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -173,12 +173,6 @@ pub enum AnalysisCommands {
173173
/// Process in background (simplified - just shows progress)
174174
#[arg(long)]
175175
background: bool,
176-
/// Output format: enhanced (default), markdown, json, or plain
177-
#[arg(long, short = 'f', default_value = "enhanced")]
178-
format: String,
179-
/// Use plain text format (alias for --format=plain)
180-
#[arg(long)]
181-
plain: bool,
182176
},
183177

184178
/// Show analysis results
@@ -188,9 +182,6 @@ pub enum AnalysisCommands {
188182
/// Show all results
189183
#[arg(long)]
190184
all: bool,
191-
/// Output format
192-
#[arg(short, long, default_value = "text")]
193-
format: String,
194185
},
195186

196187
/// Show analysis request status
@@ -329,25 +320,20 @@ impl Cli {
329320
custom_prompt,
330321
all,
331322
background,
332-
format,
333-
plain,
334323
} => {
335324
analytics::handle_execute_command(
336325
session_id,
337326
custom_prompt,
338327
all,
339328
background,
340-
format,
341-
plain,
342329
)
343330
.await
344331
}
345332

346333
AnalysisCommands::Show {
347334
session_id,
348335
all,
349-
format,
350-
} => analytics::handle_show_command(session_id, all, format).await,
336+
} => analytics::handle_show_command(session_id, all).await,
351337

352338
AnalysisCommands::Status {
353339
all,

src/database/analytics_repo.rs

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,6 @@ impl AnalyticsRepository {
2020
let generated_at = analytics.generated_at.to_rfc3339();
2121

2222
// Serialize JSON fields
23-
let scores_json =
24-
serde_json::to_string(&analytics.scores).context("Failed to serialize scores")?;
2523
let metrics_json =
2624
serde_json::to_string(&analytics.metrics).context("Failed to serialize metrics")?;
2725
let qualitative_output_json = serde_json::to_string(&analytics.qualitative_output)
@@ -35,17 +33,16 @@ impl AnalyticsRepository {
3533
r#"
3634
INSERT INTO analytics (
3735
id, analytics_request_id, session_id, generated_at,
38-
scores_json, metrics_json,
36+
metrics_json,
3937
qualitative_output_json, processed_output_json,
4038
ai_quantitative_output_json,
4139
model_used, analysis_duration_ms
42-
) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
40+
) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
4341
"#,
4442
analytics.id,
4543
analytics.analytics_request_id,
4644
analytics.session_id,
4745
generated_at,
48-
scores_json,
4946
metrics_json,
5047
qualitative_output_json,
5148
processed_output_json,
@@ -65,7 +62,7 @@ impl AnalyticsRepository {
6562
r#"
6663
SELECT
6764
id, analytics_request_id, session_id, generated_at,
68-
scores_json, metrics_json,
65+
metrics_json,
6966
qualitative_output_json, processed_output_json,
7067
ai_quantitative_output_json,
7168
model_used, analysis_duration_ms
@@ -82,8 +79,6 @@ impl AnalyticsRepository {
8279
let generated_at = DateTime::parse_from_rfc3339(&row.generated_at)?.with_timezone(&Utc);
8380

8481
// Deserialize JSON fields
85-
let scores: crate::models::Scores =
86-
serde_json::from_str(&row.scores_json).context("Failed to deserialize scores")?;
8782
let metrics: crate::models::Metrics =
8883
serde_json::from_str(&row.metrics_json).context("Failed to deserialize metrics")?;
8984
let qualitative_output: crate::services::analytics::AIQualitativeOutput =
@@ -114,7 +109,6 @@ impl AnalyticsRepository {
114109
analytics_request_id: row.analytics_request_id,
115110
session_id,
116111
generated_at,
117-
scores,
118112
metrics,
119113
qualitative_output,
120114
processed_output,
@@ -135,7 +129,7 @@ impl AnalyticsRepository {
135129
r#"
136130
SELECT
137131
id, analytics_request_id, session_id, generated_at,
138-
scores_json, metrics_json,
132+
metrics_json,
139133
qualitative_output_json, processed_output_json,
140134
ai_quantitative_output_json,
141135
model_used, analysis_duration_ms
@@ -154,8 +148,6 @@ impl AnalyticsRepository {
154148
let generated_at = DateTime::parse_from_rfc3339(&row.generated_at)?.with_timezone(&Utc);
155149

156150
// Deserialize JSON fields
157-
let scores: crate::models::Scores =
158-
serde_json::from_str(&row.scores_json).context("Failed to deserialize scores")?;
159151
let metrics: crate::models::Metrics =
160152
serde_json::from_str(&row.metrics_json).context("Failed to deserialize metrics")?;
161153
let qualitative_output: crate::services::analytics::AIQualitativeOutput =
@@ -186,7 +178,6 @@ impl AnalyticsRepository {
186178
analytics_request_id: row.analytics_request_id,
187179
session_id,
188180
generated_at,
189-
scores,
190181
metrics,
191182
qualitative_output,
192183
processed_output,

0 commit comments

Comments
 (0)