feat(pyramid): integrate reasoning mechanism for Pyramid index

[LHT129]

Summary

Implement SearchWithRequest for the Pyramid index to integrate the reasoning mechanism.

Changes

• Add Pyramid::SearchWithRequest override (pyramid.h/pyramid.cpp)
• Update search_impl to accept optional QueryContext* parameter
• Integrate ReasoningContext for expected labels tracking

Behavior

When expected_labels is provided in SearchRequest, Pyramid generates a reasoning report diagnosing missed expected results:

• not_reachable, filter_rejected, quantization_error, ef_too_small, reorder_evicted

Zero overhead when expected_labels is empty.

Testing

• All 39 existing Pyramid functests pass (113187 assertions)
• All 5 reasoning functests pass

Closes #2332

[mergify]

Merge Protections

🟢 All 3 merge protections satisfied — ready to merge.

Show 3 satisfied protections

🟢 Require kind label

• label~=^kind/

🟢 Require version label

• label~=^version/

🟢 Require linked issue for feature/bug PRs

• body~=(?im)(?:^|[\s\-\*])(?:close[sd]?|fix(?:e[sd])?|resolve[sd]?)\s*:?\s+(?:#\d+|[\w.\-]+/[\w.\-]+#\d+|https?://github\.com/[\w.\-]+/[\w.\-]+/issues/\d+)

[gemini-code-assist]

Code Review

This pull request implements the SearchWithRequest method in the Pyramid class, enabling search requests with reasoning capabilities when expected labels are provided. The feedback highlights two main issues: first, a potential null pointer dereference if the query dataset or its vectors are null, which should be validated with assertions; second, a design flaw where making the QueryContext parameter optional in search_impl leads to silent statistics loss for other search methods (like KnnSearch and RangeSearch). It is recommended to make QueryContext a required parameter across all callers.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[Copilot]

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

Integrates the reasoning mechanism into Pyramid searches by adding a SearchWithRequest override and wiring ReasoningContext through the search path when expected_labels are provided.

Changes:

• Added Pyramid::SearchWithRequest implementation to support reasoning reports for missed expected labels
• Extended search_impl to accept an optional external QueryContext*
• Added reasoning context setup/diagnosis/report generation when expected_labels is non-empty

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 9 comments.

File	Description
src/algorithm/pyramid/pyramid.h	Adds SearchWithRequest API and extends search_impl with optional QueryContext*
src/algorithm/pyramid/pyramid.cpp	Implements SearchWithRequest, integrates ReasoningContext, and plumbs external QueryContext through search_impl

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

[vsag-bot]

/label S-waiting-on-review
/waiting-on reviewer
/request-review @jiaweizone
/request-review @wxyucs
/request-review @inabao