GKaszewski/archlens
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: remove std::path from application layer, document merge error behavior

Browse Source
This commit is contained in:
Gabriel Kaszewski
2026-06-17 13:41:28 +02:00
parent 04da26beba
commit 682a64354f
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
crates/application/src/use_cases/build_code_graph.rs
View File

@@ -53,6 +53,8 @@ where
let mut graph = result.graph().clone(); let mut graph = result.graph().clone();
if level == DiagramLevel::Module { if level == DiagramLevel::Module {
if let Some(pa) = self.project_analyzer { if let Some(pa) = self.project_analyzer {
// Propagate error: a present but malformed manifest is a user error,
// not a silent skip.
let project_cg = pa.analyze(root)?; let project_cg = pa.analyze(root)?;
graph.merge_project_edges(&project_cg); graph.merge_project_edges(&project_cg);
} }

3
crates/application/src/use_cases/generate_diagram.rs
View File

@@ -44,8 +44,7 @@ fn render_split(
let ext = overview let ext = overview
.files() .files()
.first() .first()
.and_then(|f| std::path::Path::new(f.name()).extension()) .and_then(|f| f.name().rsplit_once('.').map(|(_, e)| e))
.and_then(|e| e.to_str())
.unwrap_or("txt"); .unwrap_or("txt");
if let Some(f) = overview.files().first() { if let Some(f) = overview.files().first() {