Skip to content

fix: wrap JSON data in object for CypherEngine graph build #64

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sonesuke merged 1 commit into from
Apr 4, 2026
Merged

fix: wrap JSON data in object for CypherEngine graph build #64

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 15 additions & 14 deletions src/mcp/mod.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -187,9 +187,10 @@ impl ArxivHandler {
})?;

// Create CypherEngine with auto-detection
let json_value: serde_json::Value = serde_json::to_value(&papers).map_err(|e| {
ErrorData::internal_error(format!("Failed to serialize papers: {}", e), None)
})?;
// Wrap in object so cypher-rs can detect a named node path
let json_value = serde_json::json!({
"papers": &papers
});

let engine = CypherEngine::from_json_auto(&json_value).map_err(|e| {
ErrorData::internal_error(format!("Failed to create query engine: {}", e), None)
Expand Down Expand Up @@ -260,12 +261,13 @@ impl ArxivHandler {
ErrorData::internal_error(format!("Failed to save PDF: {}", e), None)
})?;

// Create CypherEngine from the result (wrap in array)
let result = serde_json::json!({
"id": request.id,
"pdf_path": temp_path.display().to_string(),
// Create CypherEngine from the result (wrap in object with named key)
let json_value = serde_json::json!({
"results": [{
"id": request.id,
"pdf_path": temp_path.display().to_string(),
}]
});
let json_value = serde_json::json!([result]);
CypherEngine::from_json_auto(&json_value).map_err(|e| {
ErrorData::internal_error(format!("Failed to create query engine: {}", e), None)
})?
Expand All @@ -275,13 +277,12 @@ impl ArxivHandler {
ErrorData::internal_error(format!("Failed to fetch paper: {}", e), None)
})?;

// Create CypherEngine from the paper (wrap in array)
let json_value = serde_json::to_value(&paper).map_err(|e| {
ErrorData::internal_error(format!("Failed to serialize paper: {}", e), None)
})?;
let paper_array = serde_json::json!([json_value]);
// Create CypherEngine from the paper (wrap in object with named key)
let json_value = serde_json::json!({
"papers": [&paper]
});

CypherEngine::from_json_auto(&paper_array).map_err(|e| {
CypherEngine::from_json_auto(&json_value).map_err(|e| {
ErrorData::internal_error(format!("Failed to create query engine: {}", e), None)
})?
};
Expand Down
Loading