Skip to content

fix(firestore): graduate pipeline search from preview to GA - #9700

Merged
MarkDuckworth merged 4 commits into
mainfrom
markduckworth/search-ga
Oct 23, 2026
Merged

MarkDuckworth merged 4 commits into
mainfrom
markduckworth/search-ga

Conversation

@MarkDuckworth

Copy link
Copy Markdown
Contributor
  • Remove beta/preview annotations and documentation from Pipeline.search, SearchStage/SearchStageOptions, documentMatches, score, and geoDistance.
  • Update API reports and reference documentation.

- Remove beta/preview annotations and documentation from Pipeline.search, SearchStage/SearchStageOptions, documentMatches, score, and geoDistance.
- Update API reports and reference documentation.
@MarkDuckworth
MarkDuckworth requested review from a team as code owners October 22, 2026 22:50
@product-auto-label product-auto-label Bot added the api: firestore Issues related to the Firestore API. label Oct 22, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request promotes several Firestore pipeline features, including documentMatches, geoDistance, score, and search, from beta to public by removing or updating @beta tags across the codebase. The feedback suggests improving a TSDoc link in firestore.d.ts by using standard relative link syntax and referencing the correct SDK.

/**
* @beta
*
* Options defining how a SearchStage is evaluated. See {@link @firebase/firestore/pipelines#Pipeline.(search)}.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The TSDoc link references @firebase/firestore, which is the Web SDK package, but this is the Node.js SDK (@google-cloud/firestore). Additionally, the syntax Pipeline.(search) is non-standard. Since Pipeline is defined within the same namespace, we can simplify this to a relative link {@link Pipeline.search}.

Suggested change
* Options defining how a SearchStage is evaluated. See {@link @firebase/firestore/pipelines#Pipeline.(search)}.
* Options defining how a SearchStage is evaluated. See {@link Pipeline.search}.

@MarkDuckworth
MarkDuckworth enabled auto-merge (squash) October 23, 2026 15:39
@MarkDuckworth
MarkDuckworth merged commit eb47174 into main Oct 23, 2026
45 of 46 checks passed
@MarkDuckworth
MarkDuckworth deleted the markduckworth/search-ga branch October 23, 2026 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the Firestore API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants