Repository navigation
feat(staged): stamp every session-context item with its UTC date-time - #981
Merged
Merged
Conversation
…-time The context a session receives at start — the branch-level history and the project-level readout (per-repo branch timelines plus project notes) — listed commits, notes, reviews, and images without saying when they happened, except for commits, which carried git's own `Date: %ci` line: the committer date, in the author's time zone. Nothing else had a date at all, so a session could only infer ordering from position. Every item now carries a `Date:` line directly under its heading, as an absolute `YYYY-MM-DD HH:MM:SS UTC` value from one shared formatter, so items of every type and at both levels compare directly. Each item uses the same field the branch card renders for it: - commits: author time (the `%at` prefix the log already carried, now rendered in UTC before clamping so a cherry-pick keeps its real date; the `%ci` line is dropped from the git format) - notes and reviews: `completed_at ?? created_at` - images: `created_at` - project notes: `completed_at ?? created_at`, in both the project readout and a branch's history Both context preambles say the `Date:` lines are UTC. Tests cover the formatter, the commit date placement and cherry-pick case, each store item type, and project notes at both levels; existing layout tests are updated for the extra line. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Signed-off-by: Matt Toohey <contact@matttoohey.com>
…preambles Both the branch-level and project-level context preambles gained a sentence telling the session that every item carries a `Date:` line in UTC and what to use it for. Review feedback: the dates speak for themselves, so the explanation is noise. Each preamble goes back to its previous wording; the `Date:` lines on every item are unchanged. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Signed-off-by: Matt Toohey <contact@matttoohey.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The context a session gets at start — the branch-level history and the project-level readout (per-repo branch timelines plus project notes) — gave no times for notes, reviews, or images. Commits were the only items with a time, from git's
%ciline, which uses the committer's time zone. This PR adds aDate:line under the heading of every item, at both levels. Each date is a fullYYYY-MM-DD HH:MM:SS UTCvalue from one shared formatter (format_context_timestamp), so items of any type can be compared directly.Each item uses the same field the branch card shows for it, so the UI and the agent agree:
%atprefix the log already had. The date is rendered before clamping, so a cherry-pick keeps its real date. TheDate: %ciline is removed from both the local and remotegit logformats.completed_at ?? created_atcreated_atcompleted_at ?? created_at, in both the project readout and a branch's historyThe context preambles are unchanged. A UTC explanation was added at first and removed after review, because the dates are clear without it.
Tests
Date:line goes: commits with a body, a cherry-pick that gets clamped, and a commit with no body🤖 Generated with Claude Code