feat(penpal): persist recent files activity across restarts - #595
Merged
Conversation
The activity tracker was purely in-memory, so recent files were lost on every server restart. Now activity is saved to ~/.config/penpal/activity.json with debounced writes (5s) and a final flush on shutdown. - Add Save/Load to activity.Tracker with atomic writes and pruning (500 max) - Wire up load-on-start, debounced-save-on-change, save-on-shutdown in main.go - Shutdown drains HTTP requests before saving so in-flight activity is captured - Add 9 unit tests covering round-trip, corrupt JSON, pruning, nested paths, etc. - Update PRODUCT.md, ERD.md, TESTING.md with E-PENPAL-ACTIVITY-PERSIST Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
loganj
marked this pull request as ready for review
April 9, 2026 15:59
loganj
added a commit
that referenced
this pull request
Apr 9, 2026
…ktree-dis * origin/main: (45 commits) feat(penpal): persist recent files activity across restarts (#595) feat: persist open windows, tabs, and geometry across app restarts (#588) feat(staged): generate next prefill text when generating notes (#594) feat(staged): add "Run again" button to action output dialog (#593) feat: queue repo sessions in project MCP with async cancellation fix (#591) fix(staged): lowercase "building" subtitle for consistency (#592) fix(staged): use completedAt for commit prefill timeline ranking (#590) fix: add per-branch mutex to prevent worktree setup race condition (#589) feat: add diff commit session launcher (#585) fix: avoid error state for empty titled code reviews (#587) fix: cancel in-flight auto reviews before manual sessions (#584) feat(settings): add repo context search (#583) fix(staged): sort timeline items by completion time so queued items appear before completed ones (#568) fix: drain queued branch sessions from backend lifecycle (#579) fix: restore resume session metadata for resumed actions (#580) feat(staged): add purple diff comment theme (#578) fix(ui): hide branch actions during setup (#572) fix: validate parsed PR URLs before extraction (#577) fix: centralize staged relative time updates (#576) fix: avoid UI freezes when starting background sessions (#575) ... # Conflicts: # apps/penpal/internal/watcher/watcher.go
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d4b702eb86
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| for i := range entries { | ||
| e := &entries[i] | ||
| t.RecordAt(e.Type, e.Project, e.FilePath, e.Timestamp) |
There was a problem hiding this comment.
Loading persisted items with RecordAt means they occupy the project+path key before lazy scan seeding runs; since EnsureProjectScanned also seeds via RecordAt, files edited while the server was down keep the older persisted timestamp/type after restart. In practice, /api/recent can remain stale (wrong order and activity label) until each file is touched again in-session.
Useful? React with 👍 / 👎.
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
~/.config/penpal/activity.json.🤖 Generated with Claude Code