@MNeisler: If you do want to mark Vue.js widget-produced events in the SearchSatisfaction schema, you'd need to add such a field to its current.yaml in the modern event platform schema repository as described in wikitech:Event Platform/Schemas#Modifying schemas, not the legacy schema on meta.

Also, the extraParams field was created for exactly such situations so you could have events with extraParams: "vuejs-widget" and use that for filtering instead.

In T256100#6322636, @Ottomata wrote:

And any new instruments based on SearchSatisfaction schema

Hm. Yes you could change the SearchSatsifaction instrumentation code to use mw.eventLog.submit, but you'd have to make sure that the legacy integration logic that is currently formatting the event like EventGate wants it is run in the instrumentation code instead.

For legacy EventLogging schemas, I think it is fine to continue to run with mw.eventLog.logEvent if you prefer. Or, at least it wasn't in the migration plan to move them to mw.eventLog.submit.

@Niedzielski: okay, so unless y'all want to make a new Event Platform schema from scratch (but inspired by SearchSatisfaction, but sans the legacy event capsule stuff that the port has to have), you'll want to use mw.eventLog.logEvent()

Thank you, @mpopov!

I'm going to start by defining some terms – that this has to be done should probably be taken as a warning sign for a long comment!

• searchSuggest protocol: This is the protocol used to communicate between MediaWiki Core's mediawiki.searchSuggest module and the SearchSatisfaction instrument. It consists of payloads despatched via mw.track at certain breakpoints
• SearchSatisfaction instrument: The part of WikimediaEvents/modules/ext.wikimediaEvents/searchSatisfaction.js that listens to the searchSuggest protocol and logs SearchSatisfaction events, i.e. https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/WikimediaEvents/+/refs/heads/master/modules/ext.wikimediaEvents/searchSatisfaction.js#620 through L680.
• SearchSatisfaction supporting code: All other parts of WikimediaEvents/modules/ext.wikimediaEvents/searchSatisfaction.js.

We can safely ignore the SearchSatisfaction supporting code here.

Now, in order to use the SearchSatisfaction instrument with the Vue.js-based search widget, we'll need to speak the searchSuggest protocol, which is a little… complicated at times. What follows are my rough notes of what we'll have to send and when we'll have to send it:

1. User types a character in the search widget

mw.track( 'mediawiki.searchSuggest', {
  action: 'session-start'
} );

2. Autocomplete results are displayed

const query = 'Cult of Luna';
const request = makeRequest( query ); 

request.done( ( results ) => {
  mw.track( 'mediawiki.searchSuggest', {
      action: 'impression-results',
      numberOfResults: results.length,
      resultSetType: request.getResponseHeader( 'X-OpenSearch-Type' ) || 'unknown',
      searchId: request.getResponseHeader( 'X-Search-ID' ) || null,
      query: query,

      // The SearchSatisfaction schema already has a field to track where the search widget was placed.
      inputLocation: 'moved', // See T256100
  } );
} );

3. User clicks an autocomplete result

mw.track( 'mediawiki.searchSuggest', {
  action: 'click-result',
  index: /* ... */,
  numberOfResults: results.length
} );

4. User submits the search form

At this point, the protocol becomes more complicated. The submit-form action leads the SearchSatisfaction instrument to directly manipulate the DOM, adding a hidden input element that encodes the position of the result in the provenance parameter to the DOM element.

// If the user has highlighted a result with their keyboard, then this is the index of that result in
// the result set; -1 otherwise.
const resultIndex = /* ... */;

mw.track( 'mediawiki.searchSuggest', {
  action: 'submit-form',
  index: resultIndex,
  numberOfResults: results.length,
  $form: $( formElement )
} );

Note well that the same SearchSatisfaction event is logged by both the submit-form and click-result actions. The former only exists to facilitate the creation/maintenance of the hidden input. However, the value of the hidden input is easy to compute (see below) so we can deviate from the protocol here.

function getWprovFromResultIndex( index ) {

  // If the user hasn't highlighted an autocomplete result, NOOP.
  if ( index === -1 ) {
    return 'acrw1';
  }

  return 'acrw1' + index;
}

5. When an autocomplete result is rendered

The searchSuggest protocol also facilitates adding a provenance parameter to the URLs of autocomplete result via the render-one action. Similar to 4, the value of this parameter is easy to compute given the index of the autocomplete result in the result set so we can add it ourselves, i.e.

const BASE_URL = '/wiki/Special:Search';

function getAutocompleteResultUrl( result, index ) {
  const url = new mw.Uri( BASE_URL );

  url.query.wprov = getWprovFromResultIndex( index );

  /* ... */
}

It's worth pointing out that if we do choose to ignore the protocol for 4 and 5 above, we'll still have to reconcile the Vue.js-based search widget with the protocol or vice versa if/when it's deployed.

Change 629693 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[wvui@master] Add fundamentals of instrumentation for TypeaheadSearch component

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

This is blocked on T256041: Build the wvui-search-form component for Vue.js search.

^Followed up on the open patch with some feedback that can be addressed in the meantime

Change 631799 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[wvui@master] [search] TypeaheadSearch emits suggestion-click and submit events

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

Change 629693 abandoned by Phuedx:
[wvui@master] Add instrumentation for TypeaheadSearch component

Reason:
Abandoned in favour of Ic967b5e58768123f8561c455178a71d933f08228.

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

Change 638487 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[wvui@master] [components][typeahead-search] Further tidy onInput method definition

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

Change 638487 merged by jenkins-bot:
[wvui@master] [components][typeahead-search] Further tidy onInput method definition

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

Change 640115 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[mediawiki/skins/Vector@feat/search] [WIP] [search] Instrument Vue.js-based search widget

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

Change 643266 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[wvui@master] [components][typeahead-search] Show footer when there aren't suggestions

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

Change 643266 merged by jenkins-bot:
[wvui@master] [components][typeahead-search] Show footer when there aren't suggestions

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

Change 643355 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[mediawiki/skins/Vector@master] [WIP] [search] Instrument Vue.js-based search widget

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

Change 643355 abandoned by Phuedx:
[mediawiki/skins/Vector@master] [WIP] [search] Instrument Vue.js-based search widget

Reason:

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

Change 643355 restored by Phuedx:
[mediawiki/skins/Vector@master] [WIP] [search] Instrument Vue.js-based search widget

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

Change 640115 abandoned by Phuedx:
[mediawiki/skins/Vector@feat/search] [WIP] [search] Instrument Vue.js-based search widget

Reason:

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

Change 631799 merged by jenkins-bot:
[wvui@master] [components][typeahead-search] Emit suggestion-click and submit events

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

The Vector integration patch has been merged on the basis that the new search is disabled by default and will only work while https://gerrit.wikimedia.org/r/c/mediawiki/core/+/641052 is checked out. This unblocks review here and I've found an issue with the logging of click events on https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/643355/5

There is a bug in the instrumentation inside wvui library.

Whenever clicking a search result, the event is logged with an index of -1, this is because updateSuggestions sets this.suggestionActiveIndex to -1
https://github.com/wikimedia/wvui/blob/e32b54f3b8d1118b6a25cdc46b5638d6d048533e/src/components/typeahead-search/TypeaheadSearch.vue#L250

and onSuggestionClick calls this method here:
https://github.com/wikimedia/wvui/blob/e32b54f3b8d1118b6a25cdc46b5638d6d048533e/src/components/typeahead-search/TypeaheadSearch.vue#L350

Change 643355 merged by jenkins-bot:
[mediawiki/skins/Vector@master] [search] Instrument Vue.js-based search widget

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

Change 647322 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[wvui@master] [components][typeahead-search] Fix suggestion-click event data

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

Nice catch, @Jdlrobson!

Change 647322 merged by jenkins-bot:
[wvui@master] [components][typeahead-search] Fix suggestion-click event data

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

Change 649858 had a related patch set uploaded (by Jdrewniak; owner: Jdrewniak):
[mediawiki/extensions/WikimediaEvents@master] Update query selector for modern Vector test in searchSatisfaction

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

@phuedx Working off of WikimediaEvents & Vector master, we changed #app to #p-search in Vector, so I updated the following line in the patch above.
https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/WikimediaEvents/+/refs/heads/master/modules/ext.wikimediaEvents/searchSatisfaction.js#488

Also, I'm going to assume that the value extraParams: "{}" is OK on the initial page-load (it looks like that value is derived from more things than we are interested in).

Change 649858 merged by jenkins-bot:
[mediawiki/extensions/WikimediaEvents@master] Update query selector for modern Vector test in searchSatisfaction

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