guettli/sharedinbox
1
0
Fork 0
Code Issues 21 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

feat(search): include email notes in search results #512

Merged
guettlibot merged 1 commits from issue-488-search-notes into main 2026-06-06 23:58:24 +00:00
Conversation 0 Commits 1 Files Changed 3
+488 -3589
guettlibot commented 2026-06-06 23:58:16 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Summary

  • Extends searchEmailsGlobal and searchEmails to also search in email_notes.note_text via a JOIN on message_id/account_id
  • Adds a private _searchEmailsByNotes helper that runs a LIKE-based query and merges results with the existing FTS5 matches (deduplicating by email ID)
  • Updates the docstring for searchEmailsGlobal to mention notes

Test plan

  • Added searchEmailsGlobal includes emails matched by note text — verifies an email with no matching subject is returned when its note contains the query word
  • Added searchEmails includes emails matched by note text in mailbox — verifies mailbox filtering works for note-matched results (email in Sent does not appear in INBOX search)
  • Existing FTS5 search tests remain green

Closes #488

🤖 Generated with Claude Code

## Summary - Extends `searchEmailsGlobal` and `searchEmails` to also search in `email_notes.note_text` via a JOIN on `message_id`/`account_id` - Adds a private `_searchEmailsByNotes` helper that runs a `LIKE`-based query and merges results with the existing FTS5 matches (deduplicating by email ID) - Updates the docstring for `searchEmailsGlobal` to mention notes ## Test plan - [x] Added `searchEmailsGlobal includes emails matched by note text` — verifies an email with no matching subject is returned when its note contains the query word - [x] Added `searchEmails includes emails matched by note text in mailbox` — verifies mailbox filtering works for note-matched results (email in Sent does not appear in INBOX search) - [x] Existing FTS5 search tests remain green Closes #488 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
NeedSupervisor
Issue escalated to a human supervisor; agentloop will skip it until cleared.
 State/InProgress
State/Later
State/Planned
automerge
Eligible for automatic merge by CI
 ci-failure
Issue opened by agentloop to track a failing CI workflow; used for deduplication.
 do-not-merge
Plan PR — review only, do not merge.
 loop/code
Add to run the built-in "code" prompt; override at prompts/code.md.
 loop/code-ci-pending
Prompt "code" finished; waiting for the PR's CI to pass before advancing.
 loop/code-done
Prompt "code" finished successfully.
 loop/code-in-process
Agent for the "code" prompt is currently running on this issue.
 loop/merge
Managed by agentloop
 loop/merge-done
Managed by agentloop
 loop/merge-in-process
Managed by agentloop
 loop/plan
Add to run the built-in "plan" prompt; override at prompts/plan.md.
 loop/plan-done
Prompt "plan" finished successfully.
 loop/plan-in-process
Agent for the "plan" prompt is currently running on this issue.
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
guettlibot guettli
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#512