Skip to content

perf(why): look up a question's named files in one query - #4218

Merged
swati510 merged 1 commit into
mainfrom
perf/why-named-files-one-query
Oct 19, 2026
Merged

swati510 merged 1 commit into
mainfrom
perf/why-named-files-one-query

Conversation

@RaghavChamadiya

Copy link
Copy Markdown
Member

What

get_why resolves the files a question names with one query instead of one per path-shaped word, and matches each word against a set.

Verification

  • tests/unit/server/mcp/test_why.py passes.
  • ruff check . is clean.

@repowise-bot

repowise-bot Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

✅ Health of changed files: 10.0 (unchanged)
Average full file score (structure and history) of 1 changed code file, weighted by size before this PR; not the repository score.

Check Result
Coverage 100% of 10 changed lines covered
Tests 0 of 1 changed code file reached by a test · 1 reached by no test
Security No new findings on changed lines
Doc drift No drift in docs this PR edits
Coverage

Patch coverage 100.0% (10 of 10 changed executable lines covered)

b6d7c1d · cobertura · 1 of 1 changed files measured

Report from this run for b6d7c1d.

🧾 Evidence: 6 not known
Item Status Evidence
Components touched ok · inferred 1 component: packages
Declared owners ok · declared @RaghavChamadiya, @swati510
Suggested reviewers unknown · not known Not known: not gathered on a pull request
Tests reaching the change unknown · not known 0 of 1 changed file reached by a test; 1 not known
Contracts touched ok · measured No workspace contract touched; no breaking signature change
Consumers in other repos unknown · not known Not known: not gathered on a pull request
Decisions crossed unknown · not known Not known: decision records are read only for an AI review
Health delta ok · measured Findings: 0 introduced, 0 worsened, 0 resolved
AI provenance unknown · not known Provenance: unknown, known for 0 of 1 commit; AI share withheld: coverage 0% is below 30%
Diff size rank ok · measured Larger than 7.0% of recent commits
Outcomes since merge unknown · not known Not known: a pull request is not a merged change

AI review is paused: this repository reached its monthly spending limit. Bot settings


📊 See the full report for this PR
Blast radius, every caller of the contracts it changes, and health before and after. No sign-in.
Plain markdown for agents

Settings and updates

Updated 2026-10-19 13:11 UTC
Silence one PR with [skip repowise] in the title · Bot settings
⭐ Star Repowise · 📥 Install on another repo

@swati510
swati510 merged commit 32962f9 into main Oct 19, 2026
9 of 10 checks passed
@swati510
swati510 deleted the perf/why-named-files-one-query branch October 19, 2026 13:37
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.

2 participants