exarch-rs 0.3.0 → 0.4.0

package/README.md

@@ -167,11 +167,13 @@ Builder-style security configuration.
 
 ```typescript
 const config = new SecurityConfig()
-  .maxFileSize(bytes)              // Max size per file
-  .maxTotalSize(bytes)             // Max total extraction size
-  .maxFileCount(count)             // Max number of files
-  .maxCompressionRatio(n)          // Max compression ratio (zip bomb detection)
-  .setAllowSolidArchives(true);    // Allow solid 7z archives (default: false)
+  .maxFileSize(bytes)                        // Max size per file
+  .maxTotalSize(bytes)                       // Max total extraction size
+  .maxFileCount(count)                       // Max number of files
+  .maxCompressionRatio(n)                    // Max compression ratio (zip bomb detection)
+  .allowedExtensions([".txt", ".md"])        // Restrict to a set of extensions
+  .bannedPathComponents(["__MACOSX"])        // Skip these path components
+  .setAllowSolidArchives(true);              // Allow solid 7z archives (default: false)
 ```
 
 ## Security Features
@@ -203,6 +205,8 @@ The library provides built-in protection against:
 
 **Note:** 7z creation is not yet supported. Solid and encrypted 7z archives are rejected for security reasons. Unix symlinks inside 7z archives are reported as regular files (sevenz-rust2 API limitation).
 
+**Note:** Since v0.4.0, partial extraction failures return the `ExtractionReport` accumulated up to the failure point without the inner error text being duplicated, and the report is now correctly delivered across the FFI boundary instead of being dropped early.
+
 ## Comparison with tar-fs
 
 ```javascript

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "exarch-rs",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "Memory-safe archive extraction library with built-in security validation",
   "main": "index.js",
   "types": "index.d.ts",
@@ -36,8 +36,8 @@
     }
   },
   "devDependencies": {
-    "@biomejs/biome": "^2.0",
-    "@napi-rs/cli": "^3.5"
+    "@biomejs/biome": "^2.4.15",
+    "@napi-rs/cli": "^3.6.2"
   },
   "engines": {
     "node": ">= 18"

package/src/error.rs

@@ -182,7 +182,18 @@ pub fn convert_error(err: CoreError) -> Error {
             msg.push_str(&reason);
             Error::new(Status::GenericFailure, msg)
         }
-        CoreError::PartialExtraction { source, .. } => convert_error(*source),
+        CoreError::PartialExtraction { source, report } => {
+            let inner = convert_error(*source);
+            let mut msg = String::with_capacity(inner.reason.len() + 64);
+            msg.push_str("PARTIAL_EXTRACTION: ");
+            msg.push_str(&inner.reason);
+            let _ = write!(
+                &mut msg,
+                " | filesExtracted={}, bytesWritten={}",
+                report.files_extracted, report.bytes_written
+            );
+            Error::new(Status::GenericFailure, msg)
+        }
     }
 }
 
@@ -349,4 +360,40 @@ mod tests {
         assert!(err_str.contains("IO_ERROR"));
         assert!(err_str.contains("file not found"));
     }
+
+    /// Regression test for #210: `convert_error` must embed `filesExtracted`
+    /// and `bytesWritten` in the error message for `PartialExtraction`.
+    #[test]
+    fn test_partial_extraction_node_message_format() {
+        use exarch_core::ExtractionReport;
+        use exarch_core::QuotaResource;
+
+        let report = ExtractionReport {
+            files_extracted: 3,
+            bytes_written: 1024,
+            ..ExtractionReport::default()
+        };
+        let source = CoreError::QuotaExceeded {
+            resource: QuotaResource::FileCount { current: 4, max: 3 },
+        };
+        let err = CoreError::PartialExtraction {
+            source: Box::new(source),
+            report,
+        };
+
+        let napi_err = convert_error(err);
+        let msg = napi_err.reason.clone();
+        assert!(
+            msg.contains("PARTIAL_EXTRACTION"),
+            "message must contain PARTIAL_EXTRACTION, got: {msg}"
+        );
+        assert!(
+            msg.contains("filesExtracted=3"),
+            "message must contain filesExtracted=3, got: {msg}"
+        );
+        assert!(
+            msg.contains("bytesWritten=1024"),
+            "message must contain bytesWritten=1024, got: {msg}"
+        );
+    }
 }

package/src/report.rs

@@ -285,7 +285,7 @@ mod tests {
         core_report.files_extracted = 100_000;
         core_report.directories_created = 50_000;
         core_report.bytes_written = 10_000_000_000; // 10 GB
-        core_report.duration = Duration::from_secs(3600); // 1 hour
+        core_report.duration = Duration::from_hours(1);
 
         let report = ExtractionReport::from(core_report);
 
@@ -312,7 +312,7 @@ mod tests {
     #[test]
     fn test_extraction_report_duration_hours() {
         let mut core_report = CoreReport::new();
-        core_report.duration = Duration::from_secs(7200); // 2 hours
+        core_report.duration = Duration::from_hours(2);
 
         let report = ExtractionReport::from(core_report);