Skip to content

land: watcher-batch-root-wait - #517

Merged
ualtinok merged 2 commits into
mainfrom
land/watcher-batch-root-wait
Oct 18, 2026
Merged

ualtinok merged 2 commits into
mainfrom
land/watcher-batch-root-wait

Conversation

@aft-alfonso

@aft-alfonso aft-alfonso Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Accepted agent delivery; branch-keyed search descriptor included.


Summary by cubic

Scopes the watcher batch refresh wait to its process root so oversized callgraph refreshes are verified more reliably in tests.

  • Replaces the budgeted flush with wait_for_process_callgraph_refresh_for_test, waiting on the specific root.
  • Adds a benchmark slice descriptor land-watcher-batch-root-wait.json for the search-quality harness.

Written for commit a231748. Summary will update on new commits.

View guided diff Turn on auto-fix

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge; no actionable issues were found.

Summary

The PR makes the watcher batch test wait for its own root’s callgraph refresh.

  • The watcher test waits for its own project root's refresh.

Reviews (2) · Last reviewed commit: "release notes and search-quality descrip..." · Reviewed by Greptile

@ualtinok
ualtinok added this pull request to the merge queue Oct 18, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 18, 2026
ualtinok and others added 2 commits October 18, 2026 08:43
Co-authored-by: alfonso-aft <289616620+alfonso-aft@users.noreply.github.com>
Co-authored-by: alfonso-aft <289616620+alfonso-aft@users.noreply.github.com>
@ualtinok
ualtinok force-pushed the land/watcher-batch-root-wait branch from e544f44 to a231748 Compare October 18, 2026 06:43
@ualtinok
ualtinok added this pull request to the merge queue Oct 18, 2026
Merged via the queue into main with commit 9b91cdf Oct 18, 2026
49 of 50 checks passed
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