fix(search): Use fts search index for typeahead results

Summary: This improves performance of the search typeahead

Test Plan: Manual

Reviewers: bengotow

Reviewed By: bengotow

Differential Revision: https://phab.nylas.com/D3363

Author: jstejada

internal_packages/thread-search/keymaps/search-bar.json

@@ -20,7 +20,7 @@ export default class SearchBar extends React.Component {
 
   componentDidMount() {
     this._mounted = true;
-    this._unsubscribes = [
+    this._unsubscribers = [
       SearchStore.listen(this._onChange),
       WorkspaceStore.listen(() => {
         if (this.state.focused) {
@@ -30,20 +30,22 @@ export default class SearchBar extends React.Component {
     ];
   }
 
-  // It's important that every React class explicitly stops listening to
-  // N1 events before it unmounts. Thank you event-kit
-  // This can be fixed via a Reflux mixin
   componentWillUnmount() {
     this._mounted = false;
-    for (const usub of this._unsubscribes) {
-      usub();
-    }
+    this._unsubscribers.forEach((usub) => usub())
   }
 
   _onFocusSearch = () => {
     ReactDOM.findDOMNode(this.refs.searchInput).focus();
   }
 
+  _onInputKeyDown = (event) => {
+    const {key, target: {value}} = event;
+    if (value.length > 0 && key === 'Escape') {
+      this._onClearAndBlur();
+    }
+  }
+
   _onValueChange = (event) => {
     SearchActions.queryChanged(event.target.value);
     if (event.target.value === '') {
@@ -136,6 +138,7 @@ export default class SearchBar extends React.Component {
         className={inputClass}
         placeholder="Search all email"
         value={query}
+        onKeyDown={this._onInputKeyDown}
         onChange={this._onValueChange}
         onFocus={this._onFocus}
         onBlur={this._onBlur}
@@ -168,7 +171,6 @@ export default class SearchBar extends React.Component {
         className="search-bar"
         globalHandlers={{
           'core:focus-search': this._onFocusSearch,
-          'search-bar:escape-search': this._onClearAndBlur,
         }}
       >
         <div>

internal_packages/thread-search/lib/search-bar.jsx

@@ -52,7 +52,11 @@ class SearchQuerySubscription extends MutableQuerySubscription {
     if (this._accountIds.length === 1) {
       dbQuery = dbQuery.where({accountId: this._accountIds[0]})
     }
-    dbQuery = dbQuery.search(this._searchQuery).limit(30)
+    dbQuery = dbQuery
+    .search(this._searchQuery)
+    .order(Thread.attributes.lastMessageReceivedTimestamp.descending())
+    .limit(30)
+
     dbQuery.then((results) => {
       if (results.length > 0) {
         this.replaceQuery(dbQuery)

internal_packages/thread-search/lib/search-query-subscription.es6

@@ -117,17 +117,17 @@ class SearchStore extends NylasStore {
     if (this._fetchingThreadResultsVersion) { return; }
     this._fetchingThreadResultsVersion = this._searchSuggestionsVersion;
 
-    const databaseQuery = DatabaseStore.findAll(Thread)
-      .where(Thread.attributes.subject.like(this._searchQuery))
-      .order(Thread.attributes.lastMessageReceivedTimestamp.descending())
-      .limit(4);
-
     const {accountIds} = FocusedPerspectiveStore.current();
-    if (accountIds instanceof Array) {
-      databaseQuery.where(Thread.attributes.accountId.in(accountIds));
+    let dbQuery = DatabaseStore.findAll(Thread)
+    if (Array.isArray(accountIds) && accountIds.length === 1) {
+      dbQuery = dbQuery.where({accountId: accountIds[0]})
     }
+    dbQuery = dbQuery
+      .search(this._searchQuery)
+      .order(Thread.attributes.lastMessageReceivedTimestamp.descending())
+      .limit(4);
 
-    databaseQuery.then(results => {
+    dbQuery.then(results => {
       // We've fetched the latest thread results - display them!
       if (this._searchSuggestionsVersion === this._fetchingThreadResultsVersion) {
         this._fetchingThreadResultsVersion = null;