Skip to content

Keep similar thread cache indefinitely - #920

Merged
RhysSullivan merged 1 commit into
mainfrom
fix/permanent-similar-thread-cache
Jul 27, 2026
Merged

Keep similar thread cache indefinitely#920
RhysSullivan merged 1 commit into
mainfrom
fix/permanent-similar-thread-cache

Conversation

@RhysSullivan

Copy link
Copy Markdown
Collaborator

What changed

Removed the 30-day expiry check from stored similar-thread results. A stored result is now recomputed only when no cache record exists.

Why

The periodic expiry caused full-text searches to recur for popular threads and contributed heavily to Convex search-query usage. Existing cache timestamps remain available as metadata.

Impact

Similar-thread suggestions can remain stale until an explicit refresh mechanism is added, in exchange for avoiding automatic search recomputation.

Checks

  • bun run ci

@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
answer-overflow-main-site Ready Ready Preview, Comment Jul 27, 2026 8:51pm

Request Review

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
answeroverflow-main-site ad52026 Jul 27 2026, 08:49 PM

@RhysSullivan
RhysSullivan marked this pull request as ready for review July 27, 2026 20:54
@cursor

cursor Bot commented Jul 27, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@RhysSullivan
RhysSullivan merged commit 1b94e8f into main Jul 27, 2026
5 of 6 checks passed
@RhysSullivan
RhysSullivan deleted the fix/permanent-similar-thread-cache branch July 27, 2026 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant