fix: Dont dupe record search queries when paging results

feat: Record queries via api tokens separately

server/api/documents.js

@@ -605,13 +605,17 @@ router.post("documents.search", auth(), pagination(), async (ctx) => {
     })
   );
 
-  SearchQuery.create({
-    userId: user.id,
-    teamId: user.teamId,
-    source: "app",
-    query,
-    results: totalCount,
-  });
+  // When requesting subsequent pages of search results we don't want to record
+  // duplicate search query records
+  if (offset === 0) {
+    SearchQuery.create({
+      userId: user.id,
+      teamId: user.teamId,
+      source: ctx.state.authType,
+      query,
+      results: totalCount,
+    });
+  }
 
   const policies = presentPolicies(user, documents);
 

server/migrations/20200915010511-create-search-queries.js

@@ -20,7 +20,7 @@ module.exports = {
         },
       },
       source: {
-        type: Sequelize.ENUM("slack", "app"),
+        type: Sequelize.ENUM("slack", "app", "api"),
         allowNull: false,
       },
       query: {

server/models/SearchQuery.js

@@ -10,7 +10,7 @@ const SearchQuery = sequelize.define(
       primaryKey: true,
     },
     source: {
-      type: DataTypes.ENUM("slack", "app"),
+      type: DataTypes.ENUM("slack", "app", "api"),
       allowNull: false,
     },
     query: {