Skip to content

Preserve thread recency when reopening from memory - #391

Merged
Tryanks merged 1 commit into
Tryanks:mainfrom
guivieiras:pr/thread-recency
Sep 10, 2026
Merged

Tryanks merged 1 commit into
Tryanks:mainfrom
guivieiras:pr/thread-recency

Conversation

@guivieiras

Copy link
Copy Markdown
Contributor

Opening a thread that is still in memory changes its last activity time and moves it up the thread list. This fix keeps that time unchanged when reopening it, just as when opening a thread from disk.

The fix removes one unnecessary save from the existing code. The updated test checks that the activity time stays unchanged both in memory and on disk. It also keeps the existing checks that reopening the thread doesn’t stop its provider.

Implemented with GPT-6 Astra in Tcode

@guivieiras

Copy link
Copy Markdown
Contributor Author

Hey @Tryanks! Really nice project!

I was hunting for a library to start working on a PoC of an agent wrapper. I then found about GPUI, looked for projects that use it and ended up here. I was getting sick of having no other option than electron 😄

Tcode performs really well, the mobile app is lightweight and smooth. Great job!

I have some other changes and fixes that I did on my fork here, over the next few days I'll clean them and
submit some PR's if you want to have a look. Some might be too opinionated, but I'll send them just in case.

Fixes:

Improvements:

Tweaks:

  • Centerer blank thread composer + project header - TODO
  • Added drawer for chat settings below composer on mobile
  • Thread state presentation (has some settle related things) - TODO
    • unread -> completed
    • timer for time spent working
    • pulse animation
    • unsent-draf indicator
    • settle instead of archive on the hover icon action

Features:

  • Settled threads - similar to t3code
  • Thread sorting by last user message setting - instead of just based on agent updates - TODO
  • Keyboard shortcuts for navigating between threads - TODO
  • T3 Code importer - brings over t3code thread titles and some additional metadatas - TODO

@Tryanks
Tryanks enabled auto-merge (squash) September 10, 2026 05:22
@Tryanks
Tryanks merged commit ead1f3d into Tryanks:main Sep 10, 2026
6 checks passed
@Tryanks

Tryanks commented Sep 10, 2026

Copy link
Copy Markdown
Owner

Hey @guivieiras , thanks — I really appreciate that! I've gone through those branches, and they all look very helpful.

In fact, some of the issues they address are things that have bothered me in day-to-day use as well. I guess I'd just gotten so used to them over time that I stopped noticing them enough to actually fix them.

If you're interested, please feel free to open PRs for any of these improvements. I'd be very happy to take a closer look and discuss them further there. Thanks again!

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.

2 participants