pauljd/pdfcraft
1
0
Fork 0
mirror of https://github.com/storytold/pdfcraft.git synced 2026-11-08 20:10:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

address review: report output-name exhaustion per file in create_separate

Browse Source
This commit is contained in:
Dylan M. Taylor
2026-11-04 01:49:00 +00:00
parent d2d032a76e
commit cdcb05d414
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/automation/src/assemble.rs
+8 -5
View File
@@ -227,11 +227,14 @@ impl Automation {
});
out.push(match converted {
Ok((pdfcraft_engine::SourceKind::Pdf, ..)) => json!({ "path": p, "skipped": "already a PDF" }),
Ok((_, stem, pdf)) => {
let target = unused(&dir, &stem)?;
write_atomic(&target, &pdf)?;
json!({ "path": p, "output": target.to_string_lossy(), "bytes": pdf.len() })
}
// every numbered name taken is this file's failure, not the whole call's
Ok((_, stem, pdf)) => match unused(&dir, &stem) {
Ok(target) => {
write_atomic(&target, &pdf)?;
json!({ "path": p, "output": target.to_string_lossy(), "bytes": pdf.len() })
}
Err(e) => json!({ "path": p, "error": e.to_string() }),
},
Err(e) => json!({ "path": p, "error": e }),
});
}