Skip to content

Preserve LoadOptions through lossless edits - #89

Draft
jskoiz wants to merge 1 commit into
mainfrom
c1/issue-75-load-options
Draft

Preserve LoadOptions through lossless edits#89
jskoiz wants to merge 1 commit into
mainfrom
c1/issue-75-load-options

Conversation

@jskoiz

@jskoiz jskoiz commented Aug 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • Preserve the caller's LoadOptions on LosslessStream instances.
  • Use those options for low-level LosslessEdit validation and final replay.
  • Add focused regressions for no-op, scalar, and structural edits.

Verification

  • Local: cargo fmt --all --check
  • Local: git diff --check
  • Local: cargo test --locked --test lossless_roundtrip (50 passed)
  • Local: cargo test --locked (all unit, integration, and 8 doctests passed)
  • Local: cargo clippy --locked --all-targets -- -D warnings
  • Local: scripts/check-feature-clippy.sh
  • Hosted CI/merge state: not yet observed; this is a draft PR and has not been merged.

Notes/Risks

  • The change is limited to lossless edit option propagation and regression coverage.
  • No public option schema or compatibility fallback was added.

Closes #75

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[P1] Preserve LoadOptions through low-level lossless edits

1 participant