Skip to content

[repo-assist] perf(query): normalize search terms once per query - #2928

Draft
github-actions[bot] wants to merge 1 commit into
nextfrom
repo-assist/improve-normalize-search-terms-20260722-b2a29d8a0029fe28
Draft

[repo-assist] perf(query): normalize search terms once per query#2928
github-actions[bot] wants to merge 1 commit into
nextfrom
repo-assist/improve-normalize-search-terms-20260722-b2a29d8a0029fe28

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

🤖 This is an automated change from Repo Assist.

Summary

Normalize multi-term search strings once before filtering AUR results instead of lowercasing each term for every package. This removes repeated allocations and work from the result loop while preserving matching behavior.

Rationale

AUR search filtering can process many results, so per-result term normalization scales unnecessarily with the number of packages. Precomputing the normalized terms makes the hot path perform only the package-field normalization and containment checks.

Trade-offs

The change adds one small normalized-term slice per search and keeps the existing filtering semantics unchanged.

Test Status

  • go test ./pkg/query -count=1 passed.
  • make build passed.
  • make test is blocked by the environment: missing libalpm.so, pacman-conf, and the Go covdata tool.
  • make lint is blocked because golangci-lint is unavailable.

Repo Assist leaves this draft for maintainer review.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • awmgmcpg

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "awmgmcpg"

See Network Configuration for more information.

Generated by 🌈 Repo Assist, see workflow run. Learn more.
Comment /repo-assist to run again

Add this agentic workflow to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@1c6668b751c51af8571f01204ceffb19362e0f66

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants