This repository was archived by the owner on Aug 6, 2026. It is now read-only.
fix(archive): preserve archived task metadata - #3755
Merged
Conversation
Recover metadata for older archives and keep large archive lists responsive. Generated-By: PostHog Code Task-Id: 68800b78-554e-42ce-9077-b5125e27a6aa
|
😎 Merged successfully - details. |
|
React Doctor found no issues in the changed files. 🎉 Reviewed by React Doctor for commit |
Contributor
Prompt To Fix All With AIFix the following 2 code review issues. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 2
packages/ui/src/features/archive/useArchiveTask.ts:131-141
**Archive metadata misses summary cache**
When a task is archived from the default sidebar with the full task-list query disabled, this lookup excludes the task-summary cache and sends undefined title, creation date, and repository fields, causing the archive to use fallback metadata and depend on best-effort log recovery.
### Issue 2 of 2
packages/ui/src/features/archive/ArchivedTasksView.tsx:523-525
**Restore action loses task navigation**
When an archived task is available through the explicit-ID summaries but absent from the current user's filtered task list, this lookup uses the pre-restore `listedTasks` value and omits the “View task” action from the successful restore toast.
Reviews (1): Last reviewed commit: "fix(archive): preserve archived task met..." | Re-trigger Greptile |
Read archive metadata from summary caches and restore task navigation. Generated-By: PostHog Code Task-Id: 68800b78-554e-42ce-9077-b5125e27a6aa
jonathanlab
approved these changes
Jul 23, 2026
Keep each recovered archive field independent when another is unavailable. Generated-By: PostHog Code Task-Id: 68800b78-554e-42ce-9077-b5125e27a6aa
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Problem
Archived tasks can lose their title and repository metadata once they fall out of the normal task list, leaving recent work difficult to find or recover. Large archive lists also render every row and only support broad text filtering.
Changes
I persist each available archive field independently and recover older local and cloud records from retained session logs in the background. Unrecoverable records keep honest fallback titles without hiding known dates or repositories. Archive actions read both full-task and summary caches, while restored tasks fetch their full details only when opened. The archive page virtualizes rows and has a dedicated repository filter alongside title and task ID search.
How did you test this?
Automatic notifications
Created with PostHog Code