Repository navigation
feat(search): find a letter by its whole tracking link and by a part of a link - #1328
Merged
Merged
Conversation
…of a link Mail keeps a tracking link only as its site's origin (url rule version 2), so url:"<link>" found nothing for it. Now url: also matches a letter with a row of that origin whose body holds the link, as typed or without utm_*. The new field link: matches a part of a link, any case, in messages (stored links) and in mail (stored links, and the body of letters that hold a tracking link). No new rows or indexes. On a copy of the owner's store: url: 86 ms median, link: 504 ms, domain: 27 ms; every sampled letter found (WireCatLabs/cli-testing#88). Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ting link: and url: values must be quoted: ?, / and = are query syntax, and an unquoted value fails with invalid-syntax. LIKE folds the case of ASCII letters only. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
leemour
force-pushed
the
feat/mail-link-search
branch
from
October 15, 2026 21:44
6aee870 to
a4eb841
Compare
This was referenced Oct 15, 2026
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.
Handoff task 2 (mentions-next-handoff.md).
Mail keeps a tracking link only as its site's origin (url rule version 2, #1310), so
url:"<link>"found nothing for it.url:in mail now also matches a letter that has a row of the link's site and whose body holds the link, as typed or withoututm_*. The body is read only for those letters.link:— a part of a link, any case (LIKEwith%,_escaped). Messages: stored links. Mail: stored links plus the body of letters that hold a tracking link.src/store/sqlite/mention-links.ts(lucene.ts is at its line cap).On a copy of the owner's store (WireCatLabs/cli-testing#88, totals only):
url:86 ms median,link:504 ms,domain:27 ms; every sampled letter found.Known limit:
link:reads the whole body of such a letter, so it can match a part outside a link there.Checked locally: typecheck, biome (no errors), 4578 tests, docs:check, parity:render clean.
🤖 Generated with Claude Code