Skip to content

Fix mixed-unit session activity ordering#234

Open
ScriptedAlchemy wants to merge 1 commit into
masterfrom
codex/session-activity-normalized-latest
Open

Fix mixed-unit session activity ordering#234
ScriptedAlchemy wants to merge 1 commit into
masterfrom
codex/session-activity-normalized-latest

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Owner

Summary

  • normalize seconds and millisecond session message timestamps before choosing latest scheduler activity
  • preserve bounded indexed lookups by comparing the newest raw timestamp from each unit bucket
  • add a regression for mixed seconds/milliseconds where the newer seconds timestamp must win

Addresses review thread PRRT_kwDOSzKG2s6N1M6A from PR #218.

Tests

  • TMPDIR=/home/zack/.cache/tmp-session-activity-normalized CARGO_TARGET_DIR=/home/zack/.cache/tracedecay-target-session-activity-normalized cargo test --test automation_runner_test load_session_activity_selects_newest_after_normalizing_mixed_timestamp_units
  • TMPDIR=/home/zack/.cache/tmp-session-activity-normalized CARGO_TARGET_DIR=/home/zack/.cache/tracedecay-target-session-activity-normalized cargo test --test automation_runner_test load_session_activity
  • TMPDIR=/home/zack/.cache/tmp-session-activity-normalized CARGO_TARGET_DIR=/home/zack/.cache/tracedecay-target-session-activity-normalized cargo test --test automation_runner_test scheduler
  • TMPDIR=/home/zack/.cache/tmp-session-activity-normalized CARGO_TARGET_DIR=/home/zack/.cache/tracedecay-target-session-activity-normalized cargo check
  • cargo fmt --all -- --check

@ScriptedAlchemy ScriptedAlchemy force-pushed the codex/session-activity-normalized-latest branch from e44e7f7 to 7ee8883 Compare July 2, 2026 09:22
@changeset-bot

changeset-bot Bot commented Jul 2, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 5bbbffb

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy ScriptedAlchemy force-pushed the codex/session-activity-normalized-latest branch from 7ee8883 to 5bbbffb Compare July 2, 2026 09:43
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