Skip to content

refactor(core): split delta module helpers#74

Merged
peters merged 1 commit intomainfrom
refactor/core-delta-phase-1
Apr 10, 2026
Merged

refactor(core): split delta module helpers#74
peters merged 1 commit intomainfrom
refactor/core-delta-phase-1

Conversation

@peters
Copy link
Copy Markdown
Contributor

@peters peters commented Apr 10, 2026

Purpose

  • split surge_core::releases::delta into focused internal modules for format detection, archive delta handling, sparse delta serialization, tree walking, and filesystem apply helpers
  • keep the public delta API stable at surge_core::releases::delta
  • remove the delta entry from the maintainability baseline

Behavior Impact

  • no intended user-visible behavior change
  • archive patch formats, sparse file-op deltas, and public delta apply/decode entrypoints keep their current behavior

Test Evidence

  • cargo check -p surge-core
  • cargo test -p surge-core releases::delta
  • cargo clippy -p surge-core --all-targets --all-features -- -D warnings -W clippy::pedantic
  • ./scripts/check-maintainability.sh
  • ./scripts/sync-surge-core-vendor.sh --check
  • ./scripts/check-version-sync.sh
  • cargo fmt --all -- --check
  • RUSTFLAGS="-D warnings" cargo test --workspace
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo clippy --workspace --lib --bins --examples -- -D warnings -D clippy::unwrap_used -D clippy::expect_used
  • cargo clippy --workspace --all-targets --all-features -- -D warnings -W clippy::pedantic
  • dotnet format dotnet/Surge.slnx --verify-no-changes
  • dotnet test dotnet/Surge.slnx --configuration Release

Migration Notes

  • none

@peters peters marked this pull request as ready for review April 10, 2026 12:23
@peters peters merged commit b7c12cf into main Apr 10, 2026
11 checks passed
@peters peters deleted the refactor/core-delta-phase-1 branch April 10, 2026 12:24
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.

1 participant