Repository navigation
docs: the changelog records the @viewer opener's removal where a release reads it - #142
Merged
Merged
Conversation
…ase reads it What the review of #141 asked for and #141 merged without: the removal under Removed, and #141 in the release note's list of what needs the 0.9 bump, as #137 is. Claude-Session: https://claude.ai/code/session_01EToj6RJRMSSHQiqGNMU9wv
endrix
added a commit
that referenced
this pull request
Nov 9, 2026
…s test (#143) * test: sessions sorted by time are created at distinct times session-manager.test.ts expected ['Session 1', 'Session 3'] from a sort by updatedAt, most recent first, as its own comment says. That held only when all three sessions were created within one millisecond: the sort then kept insertion order. On a runner where the clock ticked between them, Session 3 came first and the test failed, as on main's CI after #142 (a changelog-only change). The test now creates each session at its own fixed time (fake Date only) and expects the most recent first. With distinct times the old expectation fails every time; the new one passes every time. Claude-Session: https://claude.ai/code/session_01EToj6RJRMSSHQiqGNMU9wv * fix: sessions with the same time list the later one first Review of #143: the sort by updatedAt had no tie-breaker, so two sessions created (or restored) in the same millisecond came back oldest first, and the chat's delete fallback took the older one as the most recent sibling. Equal times now fall back to createdAt, then to the later one added. A test pins equal times (it fails without the tie-breaker); the first test's comment says what it needs rather than its history. Claude-Session: https://claude.ai/code/session_01EToj6RJRMSSHQiqGNMU9wv
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
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.
The review of #141 asked for this changelog fix. #141 merged before it was committed, although its description said it was in.
@vieweropener is now under Removed.Changelog only.
https://claude.ai/code/session_01EToj6RJRMSSHQiqGNMU9wv