Skip to content

Split ingestion profile default storage#266

Merged
yvette-carlisle merged 1 commit into
mainfrom
xy/split-ingestion-profile-defaults
Jun 29, 2026
Merged

Split ingestion profile default storage#266
yvette-carlisle merged 1 commit into
mainfrom
xy/split-ingestion-profile-defaults

Conversation

@yvette-carlisle

Copy link
Copy Markdown
Member

Summary

  • Add ingestion_profiles::storage::defaults as a private child module.
  • Move default profile pointer persistence and default profile seeding out of storage.rs.
  • Preserve existing storage::... call sites through a narrow pub(super) facade.

Validation

  • cargo make fmt-rust
  • cargo make check-rust
  • cargo make lint-vstyle
  • cargo make lint-rust
  • cargo make test-rust
  • cargo make check
  • cargo make test-rust-integration
  • cargo make check-trace-gate
  • git diff --cached --check

Review

  • Read-only scout selected this split as a low-risk storage boundary.
  • Read-only skeptic review found the Rust move behavior-preserving; its only blocking note was resolved by staging the new defaults.rs module before commit.

…lt persistence into a storage child module after strict validation.","authority":"manual"}
@yvette-carlisle yvette-carlisle merged commit 4acea34 into main Jun 29, 2026
12 checks passed
@yvette-carlisle yvette-carlisle deleted the xy/split-ingestion-profile-defaults branch June 29, 2026 14:19
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