Page MenuHomePhabricator

Make it clear that CirrusDoc is not a stable format.
Closed, ResolvedPublic2 Estimated Story Points

Description

As part of the Search action API, CirrusDoc is meant to be used only internally, for communication between the various Search components and it isn't a stable interface. To avoid other teams / use cases to start relying on that format, we need to make it clear in the response itself that this format is unstable.

Event Timeline

Gehel triaged this task as Medium priority.Mar 7 2025, 9:37 AM

I've seen other api's have an internal flag in Special:ApiSandbox, maybe we need to trigger the same interface element

Gehel set the point value for this task to 2.Mar 10 2025, 4:46 PM

The API's were marked internal in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CirrusSearch/+/1127113 . I'm not sure what else we might need to do, this is mostly a UI flag in the Special:ApiSandbox but doesn't change anything else.

I'm not clear on what isInternal() actually do. During some of our discussion, we discussed having something in the response payload making it clear that this should not be used (for example an additional field "comment" -> "The CirrusDoc format is meant for internal use by CirrusSearch or debugging of queries, it might change at any time without notice"). Should we still do this?

Change #1131998 had a related patch set uploaded (by Ebernhardson; author: Ebernhardson):

[mediawiki/extensions/CirrusSearch@master] Add a note to cirrus[build]doc output about format stability

https://gerrit.wikimedia.org/r/1131998

Change #1131998 merged by jenkins-bot:

[mediawiki/extensions/CirrusSearch@master] Add a note to cirrus[build]doc output about format stability

https://gerrit.wikimedia.org/r/1131998