jesse/cognix
1
0
Fork 0
Code Issues 7 Pull requests Projects Releases Packages Wiki Activity Actions

fix: harden shared-session hydration #85

Merged
jesse merged 2 commits from fix/dashboard-load-all-sessions into main 2026-05-23 16:23:27 +02:00
Conversation 0 Commits 2 Files changed 13 +397 -66
jesse commented 2026-05-19 19:09:13 +02:00
Owner
Copy link

Summary

  • adopt legacy sessions into shared-session metadata during local-owner hydration
  • make orphan adoption and snapshot loading atomic so malformed rows roll back cleanly
  • add ordering indexes and migration tests for shared-session snapshots

Tests

  • git diff --cached --check
  • git diff --check origin/main...HEAD
  • cargo test -p cognix-storage migration_019
  • cargo test -p cognix-storage shared_sessions
  • cargo test -p cognix-server session_sharing
## Summary - adopt legacy sessions into shared-session metadata during local-owner hydration - make orphan adoption and snapshot loading atomic so malformed rows roll back cleanly - add ordering indexes and migration tests for shared-session snapshots ## Tests - git diff --cached --check - git diff --check origin/main...HEAD - cargo test -p cognix-storage migration_019 - cargo test -p cognix-storage shared_sessions - cargo test -p cognix-server session_sharing
fix: harden shared-session hydration
All checks were successful
CI / Detect Changes (pull_request) Successful in 41s
Details
CI / Integration Tests (pull_request) Has been skipped
Details
CI / Benchmarks (pull_request) Has been skipped
Details
CI / Security Scan (pull_request) Successful in 25s
Details
CI / Format (pull_request) Successful in 41s
Details
CI / Check file lengths (pull_request) Successful in 41s
Details
CI / Deny (pull_request) Successful in 44s
Details
CI / Audit (CVEs) (pull_request) Successful in 44s
Details
CI / Conventional Validation (pull_request) Successful in 1m34s
Details
CI / Clean Build Sample 1 (pull_request) Has been skipped
Details
CI / Clean Build Sample 2 (pull_request) Has been skipped
Details
CI / Clean Build Sample 3 (pull_request) Has been skipped
Details
CI / Clean Build Summary (pull_request) Has been skipped
Details
CI / Documentation (pull_request) Successful in 1m47s
Details
CI / Check (linux-aarch64 compile-validation) (pull_request) Successful in 2m3s
Details
CI / Dashboard Browser (pull_request) Successful in 2m3s
Details
CI / Clippy (pull_request) Successful in 2m23s
Details
CI / RSS gate (P-15) (pull_request) Successful in 4m22s
Details
CI / Test (pull_request) Successful in 7m3s
Details
CI / Build (release) (pull_request) Successful in 8m5s
Details
CI / PR Size Check (pull_request) Successful in 15s
Details
CI / Dashboard UI Build (pull_request) Successful in 4m24s
Details
CI / D-02 Clean Build Gate (pull_request) Successful in 4m26s
Details
CI / Coverage (80% gate) (pull_request) Successful in 6m17s
Details
CI / CI Report (pull_request) Successful in 5s
Details
5e14921f98
jesse merged commit 54f97429d0 into main 2026-05-23 16:23:27 +02:00
jesse deleted branch fix/dashboard-load-all-sessions 2026-05-23 16:23:27 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
jesse/cognix!85
No description provided.