Skip to content

perf(keywords): batch query resolution#16

Merged
hubab1 merged 1 commit into
mainfrom
06-add-keyword-query-resolution
Jul 18, 2026
Merged

perf(keywords): batch query resolution#16
hubab1 merged 1 commit into
mainfrom
06-add-keyword-query-resolution

Conversation

@hubab1

@hubab1 hubab1 commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • resolve existing keyword queries in bounded bulk fetches instead of one fetch per keyword/storefront pair
  • reconcile exact tracked-keyword identities at submit time, including saves from another SwiftData context
  • preserve normalized duplicate handling, deterministic insertion order, save rollback, analytics, and refresh behavior

Testing

  • focused AddKeywordsQueryResolverTests: 6 passed
  • full macOS suite: 193 tests in 24 suites passed
  • app build and launch verification passed

Attribution

Adapted from the query-resolution work in PR #8 by @akshaynexus. The implementation commit includes Akshay CM as a co-author.

Adapt the query-resolution approach from PR #8 while preserving submit-time duplicate reconciliation.

Co-authored-by: Akshay CM <akshaycm@hotmail.com>
@hubab1
hubab1 merged commit 7245c4c into main Jul 18, 2026
1 check passed
@hubab1
hubab1 deleted the 06-add-keyword-query-resolution branch July 18, 2026 16:33
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