This commit migrates Partial methods from taking `&mut self` and returning`Result<&mut Self, ReflectError>` to taking `self` (ownership) and returning`Result<Self, ReflectError>`.Benefits:- Simpler error handling - no poisoning logic needed- Cleaner semantics - ownership transfer makes state transitions explicit- No invalid states - can't accidentally use a Partial after an error- Rust-idiomatic - follows builder pattern conventionsChanges:- All Partial methods now take `self` instead of `&mut self`- TypedPartial updated to match the ownership API- Removed poisoning logic (require_active, poison_and_cleanup, BuildFailed)- Fuzzer updated to use Option<Partial> pattern for error handlingStill TODO:- Update all deserializers (facet-json, facet-yaml, facet-toml, etc.)- Update facet-value deserialize.rs- Update all testsSee OWNERSHIP_API_MIGRATION.md for migration patterns.

- Convert all deserializers to use the new ownership API pattern- Replace &mut self methods with self -> Result<Self, Error>- Capture path/shape before ownership-taking calls for error handling- Use match blocks instead of map_err closures to avoid borrow-after-move- Update HeapValue::materialize for final type conversion

With the ownership-based Partial API, these tests are no longer needed:- double_free_test.rs: Double-free is impossible by construction- dynamic_value_segv_test.rs: Uses old inner_mut() API- leak_repro.rs: Uses old inner_mut() APIThe type system now enforces proper ownership, making these failuremodes impossible. The fuzzer can still catch any remaining issues.Updated map_leak_test.rs for the ownership API pattern.

- Remove TypedPartial wrapper, Partial::alloc::<T>() now returns Partial directly- Update tests to use .materialize::<T>()? instead of HeapValue dereference- Fix .unwrap_err() calls that require Debug on Partial (use match instead)- Remove obsolete map_leak_test.rs which tested invalid begin_inner() operation

github-advanced-securitybot left a comment