[reakaleek]

SearchQueryBuilder loses the API-only branches that the per-word lookup in the layer below made dead: the loose partial match, ApplyApiRanking, the api type filter in the reading parse, the prefix and children options of a reading, and the tests that covered them. Nothing changes for readers; the docs-wide search keeps its identifier field reading exactly as before.

Affects: Search

Prompt summary: The API search box ranks search retriever perfectly but returns bogus results for cluster flat_settings and nothing useful for es|ql drop_null_columns. Asked why, and then asked to design the fix properly instead of patching the rule, the work became a per-token query model delivered as a stack of three changes. This is the top layer, the deletion of what the new query replaced.

Why

After the layer below, type=api requests never reach the reading parse, so its API-only paths have no caller. Leaving them in would keep two models of the same query alive and invite the next change to patch the wrong one.

What

One lookup for the docs-wide readings

A docs-wide reading always used no prefix and no children; it now says so through one fixed DocsWideLookup, and the api branches that chose otherwise are gone. The loose 75 percent clause and the bare-word endpoint lift for API queries are deleted with the method that applied them, since the per-word lookup has both natively.

Tests follow the behaviour

The children-row result tests move to the API lookup's own result path, which is the only place children are listed now, with their assertions unchanged. Six cases that only the old reading tests covered (exact and prefix name matches with bounded scores, the qualified path, the parent key of the children option, what a trailing dot keeps) are added against the new builder before the old ones go. The remaining reading tests run docs-wide only.

Verify

dotnet test tests/Elastic.Documentation.Api.Tests/
grep -rn "ApplyApiRanking\|BuildLoosePartialMatchQuery\|IsApiLookup" src tests tests-integration
# the grep prints nothing

Out of scope: BuildApiPropertyQuery keeps its typeFilter parameter for its test callers; a later cleanup can drop it.

Stack: 3 of 3, on top of #4699, which sits on #4698.

🤖 Generated with Claude Code

[github-actions]

Docs preview (local build)

Handbook preview: https://docs-v3-preview.elastic.dev/elastic/docs-builder/pull/4700/