Allow filtering recent reviews by AI use

Author: kraih

assets/vue/RecentReviews.vue

@@ -14,7 +14,7 @@
               <label class="form-label">Reviews per Page</label>
             </div>
           </div>
-          <div class="col">
+          <div class="col-lg-2">
             <div class="form-check">
               <input
                 v-model="params.byUser"
@@ -25,6 +25,18 @@
               />
               <label class="form-check-label" for="cavil-pkg-by-user">Reviewed By User</label>
             </div>
+            <div class="form-check">
+              <input
+                v-model="params.aiAssisted"
+                @change="gotoPage(1)"
+                type="checkbox"
+                class="form-check-input"
+                id="cavil-pkg-ai-assisted"
+              />
+              <label class="form-check-label" for="cavil-pkg-ai-assisted">Reviewed With AI</label>
+            </div>
+          </div>
+          <div class="col">
             <div class="form-check">
               <input
                 v-model="params.unresolvedMatches"
@@ -161,14 +173,20 @@ export default {
 
       const reviews = [];
       for (const review of data.page) {
+        const login = [];
+        if (review.login) {
+          login.push(review.login);
+          if (review.ai_assisted) login.push('<i class="fas fa-robot"></i>');
+        }
+
         reviews.push({
           link: externalLink(review),
           created: moment(review.created_epoch * 1000).fromNow(),
           reviewed: moment(review.reviewed_epoch * 1000).fromNow(),
           package: packageLink(review),
           priority: review.priority,
           report: reportLink(review),
-          login: review.login,
+          login: login.join(' '),
           result: review.result,
           state: review.state
         });
@@ -183,7 +201,7 @@ export default {
     }
   },
   watch: {
-    ...genParamWatchers('limit', 'offset', 'byUser', 'unresolvedMatches'),
+    ...genParamWatchers('limit', 'offset', 'byUser', 'aiAssisted', 'unresolvedMatches'),
     filter: function (val) {
       this.params.filter = val;
       this.params.offset = 0;

lib/Cavil/Controller/Pagination.pm

@@ -145,12 +145,14 @@ sub recent_reviews ($self) {
   $v->optional('limit')->num;
   $v->optional('offset')->num;
   $v->optional('byUser');
+  $v->optional('aiAssisted');
   $v->optional('unresolvedMatches');
   $v->optional('filter');
   return $self->reply->json_validation_error if $v->has_error;
   my $limit              = $v->param('limit')             // 10;
   my $offset             = $v->param('offset')            // 0;
   my $by_user            = $v->param('byUser')            // 'false';
+  my $ai_assisted        = $v->param('aiAssisted')        // 'false';
   my $unresolved_matches = $v->param('unresolvedMatches') // 'false';
   my $search             = $v->param('filter')            // '';
 
@@ -159,6 +161,7 @@ sub recent_reviews ($self) {
       limit              => $limit,
       offset             => $offset,
       by_user            => $by_user,
+      ai_assisted        => $ai_assisted,
       unresolved_matches => $unresolved_matches,
       search             => $search
     }

lib/Cavil/Model/Packages.pm

@@ -148,7 +148,7 @@ sub has_file_stats ($self, $id) {
   return defined($self->pg->db->select('bot_packages', 'unpacked_files', {id => $id})->hash->{unpacked_files});
 }
 
-sub has_human_review ($self, $name) {
+sub has_manual_review ($self, $name) {
   return !!$self->pg->db->query('SELECT COUNT(*) FROM bot_packages WHERE name = ? AND reviewing_user IS NOT NULL',
     $name)->array->[0];
 }
@@ -330,20 +330,25 @@ sub paginate_recent_reviews ($self, $options) {
     $user = 'AND p.reviewing_user IS NOT NULL';
   }
 
+  my $ai_assisted = '';
+  if ($options->{ai_assisted} eq 'true') {
+    $ai_assisted = 'AND p.ai_assisted = true';
+  }
+
   my $unresolved = '';
   if ($options->{unresolved_matches} eq 'true') {
     $unresolved = "AND unresolved_matches > 0";
   }
 
   my $results = $db->query(
     qq{
-      SELECT p.id, p.name, u.login, p.result,  EXTRACT(EPOCH FROM p.created) AS created_epoch,
+      SELECT p.id, p.name, u.login, p.result, p.ai_assisted, EXTRACT(EPOCH FROM p.created) AS created_epoch,
         EXTRACT(EPOCH FROM p.reviewed) AS reviewed_epoch, EXTRACT(EPOCH FROM p.imported) as imported_epoch,
         EXTRACT(EPOCH FROM p.unpacked) as unpacked_epoch, EXTRACT(EPOCH FROM p.indexed) as indexed_epoch,
         external_link, priority, state, checksum, unresolved_matches, COUNT(*) OVER() AS total
        FROM bot_packages p
          LEFT JOIN bot_users u ON p.reviewing_user = u.id
-       WHERE reviewed IS NOT NULL AND reviewed > NOW() - INTERVAL '90 DAYS' $search $user $unresolved
+       WHERE reviewed IS NOT NULL AND reviewed > NOW() - INTERVAL '90 DAYS' $search $user $ai_assisted $unresolved
        ORDER BY reviewed DESC
        LIMIT ? OFFSET ?
     }, $options->{limit}, $options->{offset}

lib/Cavil/Task/Analyze.pm

@@ -112,7 +112,7 @@ sub _analyzed ($job, $id) {
 
   # Every package needs a human review before future versions can be auto-accepted, and checkouts need to be complete
   my $specfile = $reports->specfile_report($id);
-  if (!$pkgs->has_human_review($pkg->{name}) || $specfile->{incomplete_checkout}) {
+  if (!$pkgs->has_manual_review($pkg->{name}) || $specfile->{incomplete_checkout}) {
     _look_for_smallest_delta($app, $pkg, 0, 0, !!$specfile->{incomplete_checkout}) if $pkg->{state} eq 'new';
     return;
   }
@@ -149,6 +149,7 @@ sub _analyzed ($job, $id) {
       $pkg->{state}            = 'acceptable_by_lawyer';
       $pkg->{review_timestamp} = 1;
       $pkg->{reviewing_user}   = undef;
+      $pkg->{ai_assisted}      = 0;
       $pkg->{result}           = "Accepted because reviewed by lawyer under the same license ($c_id)";
       $pkgs->update($pkg);
     }
@@ -186,7 +187,7 @@ sub _analyzed ($job, $id) {
   _look_for_smallest_delta($app, $pkg, 1, 1, 0) if $pkg->{state} eq 'new';
 }
 
-sub _look_for_smallest_delta ($app, $pkg, $allow_accept, $has_human_review, $incomplete_checkout) {
+sub _look_for_smallest_delta ($app, $pkg, $allow_accept, $has_manual_review, $incomplete_checkout) {
   my $reports = $app->reports;
   my $pkgs    = $app->packages;
 
@@ -215,7 +216,7 @@ sub _look_for_smallest_delta ($app, $pkg, $allow_accept, $has_human_review, $inc
       if ($incomplete_checkout) {
         $pkg->{notice} .= ', manual review is required because the checkout might be incomplete';
       }
-      elsif (!$has_human_review) {
+      elsif (!$has_manual_review) {
         $pkg->{notice} .= ', manual review is required because previous reports are missing a reviewing user';
       }