feat(message-editor): add prompt history viewer

[jonathanlab]

Added a new prompt history viewer

---

Created with PostHog Code

[greptile-apps]

Comments Outside Diff (2)

1. apps/code/src/shared/types/analytics.ts, line 3-6 (link)

   Shared file imports from a feature module

   Every other analytics property interface in this file is defined inline (e.g. TaskCreateProperties, PromptSentProperties). Importing from @features/message-editor/analytics is the only case where @shared/types/analytics.ts reaches into a feature module, inverting the typical dependency direction. This violates the OnceAndOnlyOnce principle for where analytics types live and makes the shared file harder to understand at a glance. Consider moving PromptHistoryOpenedProperties and PromptHistorySelectedProperties directly into this file and deleting apps/code/src/renderer/features/message-editor/analytics.ts.

   Prompt To Fix With AI

   This is a comment left during a code review.
   Path: apps/code/src/shared/types/analytics.ts
   Line: 3-6
   
   Comment:
   **Shared file imports from a feature module**
   
   Every other analytics property interface in this file is defined inline (e.g. `TaskCreateProperties`, `PromptSentProperties`). Importing from `@features/message-editor/analytics` is the only case where `@shared/types/analytics.ts` reaches into a feature module, inverting the typical dependency direction. This violates the OnceAndOnlyOnce principle for where analytics types live and makes the shared file harder to understand at a glance. Consider moving `PromptHistoryOpenedProperties` and `PromptHistorySelectedProperties` directly into this file and deleting `apps/code/src/renderer/features/message-editor/analytics.ts`.
   
   How can I resolve this? If you propose a fix, please make it concise.

   Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

2. apps/code/src/renderer/features/message-editor/components/PromptHistoryDialog.tsx, line 112-116 (link)

   Redundant reversed dependency in filtered memo

   fuse is already recomputed whenever reversed changes (it's fuse's only dep), so adding reversed to filtered's dependency array is superfluous. When any dep changes React re-runs the callback and closes over the latest reversed in scope, so the no-query branch (if (!q) return reversed) is always safe with just [fuse, query]. The current code is not broken, but the extra dep creates a slightly misleading hint that reversed and fuse could diverge.

   Prompt To Fix With AI

   This is a comment left during a code review.
   Path: apps/code/src/renderer/features/message-editor/components/PromptHistoryDialog.tsx
   Line: 112-116
   
   Comment:
   **Redundant `reversed` dependency in `filtered` memo**
   
   `fuse` is already recomputed whenever `reversed` changes (it's `fuse`'s only dep), so adding `reversed` to `filtered`'s dependency array is superfluous. When any dep changes React re-runs the callback and closes over the latest `reversed` in scope, so the no-query branch (`if (!q) return reversed`) is always safe with just `[fuse, query]`. The current code is not broken, but the extra dep creates a slightly misleading hint that `reversed` and `fuse` could diverge.
   
   
   
   How can I resolve this? If you propose a fix, please make it concise.

Prompt To Fix All With AI

Fix the following 2 code review issues. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 2
apps/code/src/shared/types/analytics.ts:3-6
**Shared file imports from a feature module**

Every other analytics property interface in this file is defined inline (e.g. `TaskCreateProperties`, `PromptSentProperties`). Importing from `@features/message-editor/analytics` is the only case where `@shared/types/analytics.ts` reaches into a feature module, inverting the typical dependency direction. This violates the OnceAndOnlyOnce principle for where analytics types live and makes the shared file harder to understand at a glance. Consider moving `PromptHistoryOpenedProperties` and `PromptHistorySelectedProperties` directly into this file and deleting `apps/code/src/renderer/features/message-editor/analytics.ts`.

### Issue 2 of 2
apps/code/src/renderer/features/message-editor/components/PromptHistoryDialog.tsx:112-116
**Redundant `reversed` dependency in `filtered` memo**

`fuse` is already recomputed whenever `reversed` changes (it's `fuse`'s only dep), so adding `reversed` to `filtered`'s dependency array is superfluous. When any dep changes React re-runs the callback and closes over the latest `reversed` in scope, so the no-query branch (`if (!q) return reversed`) is always safe with just `[fuse, query]`. The current code is not broken, but the extra dep creates a slightly misleading hint that `reversed` and `fuse` could diverge.

```suggestion
  }, [fuse, query]);
```

_{Reviews (1): Last reviewed commit: "feat(message-editor): add prompt history..." | Re-trigger Greptile}