]> git.sesse.net Git - skvidarsync/blobdiff - bin/sync.pl
Use an ephemeral message for nagging about the spreadsheet; hopefully more visible...
[skvidarsync] / bin / sync.pl
index 2bb175bf9bdbb7809dbd4aafecb8945d16d788c2..cec919953933224f87aa4e2c4f50bfc93eb5c737 100644 (file)
@@ -7,19 +7,37 @@ use JSON::XS;
 use LWP::UserAgent;
 use DBI;
 use POSIX;
+use Time::HiRes;
 binmode STDOUT, ':utf8';
 binmode STDERR, ':utf8';
 use utf8;
 
-# TODO:
-# - detect moves between groups
-# - message people when they sign up and we can't find them in the spreadsheet
-
 require '../include/config.pm';
 
 my $dbh;
 my @log = ();
 
+my %rgb = (
+       yellow => {
+               red => 1,
+               green => 1,
+               blue => 0,
+               alpha => 1
+       },
+       blue => {
+               red => 0,
+               green => 1,
+               blue => 1,
+               alpha => 1
+       },
+       white => {
+               red => 1,
+               green => 1,
+               blue => 1,
+               alpha => 0
+       }
+);
+
 sub get_oauth_bearer_token {
        my $ua = shift;
        my $now = time();
@@ -40,7 +58,7 @@ sub get_oauth_bearer_token {
 sub get_slack_name {
        my ($ua, $userid) = @_;
        my $req = HTTP::Request->new('GET', 'https://slack.com/api/users.info?user=' . $userid, [
-              'Authorization' => 'Bearer ' . $config::oauth_token
+              'Authorization' => 'Bearer ' . $config::slack_oauth_token
        ]);
        my $response = $ua->request($req);
        die $response->status_line if !$response->is_success;
@@ -94,7 +112,7 @@ sub matches_name {
 
 sub format_cell_names_for_seen {
        my $seen = shift;
-       my @cells = map { chr(ord('A') + $_->[2]) . $_->[1] } @$seen;
+       my @cells = map { chr(ord('A') + $_->[2]) . ($_->[1] + 1) } @$seen;
        return join(', ', @cells);
 }
 
@@ -138,6 +156,71 @@ sub sheet_batch_update {
        die $response->decoded_content if !$response->is_success;
 }
 
+sub get_group_assignments {
+       my $json = shift;
+
+       my %assignments = ();
+       my $rows = $json->{'sheets'}[0]{'data'}[0]{'rowData'};
+       my @curr_groups = ();
+       for my $row (@$rows) {
+               my $col = 0;
+               for my $val (@{$row->{'values'}}) {
+                       ++$col;
+                       my $contents = $val->{'userEnteredValue'}{'stringValue'};
+                       next if !defined($contents);
+                       if ($contents =~ /Gruppe /) {
+                               @curr_groups = ();
+                               last;
+                       }
+                       next if $contents =~ /^VL:/;
+                       next if $contents =~ /^LT\b/;
+                       next if $contents =~ /^400m/;
+                       next if $contents =~ /^546m/;
+                       if ($contents =~ /^(G\d\.\d)/ || $contents =~ /^(Nye løpere.*)/) {
+                               $curr_groups[$col] = $1;
+                       } else {
+                               my $name = get_spreadsheet_name($val);
+                               next if (!defined($name));
+                               my $group = $curr_groups[$col] // $curr_groups[$col - 1];
+                               # print $group, " ", $name, "\n";
+                               if (exists($assignments{$name})) {
+                                       $assignments{$name} = "(flere grupper)";
+                               } else {
+                                       $assignments{$name} = $group;
+                               }
+                       }
+               }
+       }
+       return %assignments;
+}
+
+sub update_assignment_db {
+       my ($dbh, $channel, $ts, $assignments) = @_;
+
+       local $dbh->{AutoCommit} = 0;
+       my %db_assignments = ();
+       my $q = $dbh->prepare('SELECT name,group_name FROM current_group_membership_history WHERE channel=? AND ts=?');
+       $q->execute($channel, $ts);
+       while (my $ref = $q->fetchrow_hashref) {
+               if (defined($ref->{'group_name'})) {
+                       $db_assignments{$ref->{'name'}} = $ref->{'group_name'};
+               }
+       }
+
+       $q = $dbh->prepare('INSERT INTO group_membership_history (channel, ts, name, change_seen, group_name) VALUES (?, ?, ?, CURRENT_TIMESTAMP, ?)');
+       for my $name (keys %$assignments) {
+               if (!exists($db_assignments{$name}) || $db_assignments{$name} ne $assignments->{$name}) {
+                       $q->execute($channel, $ts, $name, $assignments->{$name});
+               }
+       }
+       for my $name (keys %db_assignments) {
+               if (!exists($assignments->{$name})) {
+                       $q->execute($channel, $ts, $name, undef);
+               }
+       }
+       $dbh->commit;
+}
+
 sub get_spreadsheet_with_title {
        my ($ua, $token, $wanted_sheet_title) = @_;
 
@@ -164,7 +247,7 @@ sub find_where_each_name_is {
 
        my %seen_names = ();
        my $rows = $json->{'sheets'}[0]{'data'}[0]{'rowData'};
-       my $rowno = 4;
+       my $rowno = 3;
        for my $row (@$rows) {
                my $colno = 0;
                for my $val (@{$row->{'values'}}) {
@@ -180,13 +263,26 @@ sub find_where_each_name_is {
        return %seen_names;
 }
 
+sub best_name_for_log {
+       my ($userid, $slack_userid_to_real_name, $slack_userid_to_slack_name) = @_;
+       if (exists($slack_userid_to_real_name->{$userid})) {
+               return $slack_userid_to_real_name->{$userid};
+       } elsif (exists($slack_userid_to_slack_name->{$userid})) {
+               return $slack_userid_to_slack_name->{$userid} . ' (fant ikke regneark-navn)';
+       } else {
+               # Should only happen if we didn't see the initial reaction_add, only reaction_remove.
+               # (TODO: Is the comment above true anymore, now that we use this from multiple contexts?)
+               return $userid . ' (fant ikke Slack-navn)';
+       }
+}
+
 # Add the reaction log. (This only takes into account the last change
 # for each user; earlier ones are irrelevant and don't count. But it
 # doesn't deduplicate across reactions. Meh.)
 sub create_reaction_log {
        my ($dbh, $invitation_ts, $slack_userid_to_real_name, $slack_userid_to_slack_name) = @_;
 
-       my $q = $dbh->prepare('select userid,event_type,reaction,to_char(event_ts,\'YYYY-mm-dd HH24:MI\') as event_ts from ( select distinct on (channel,ts,userid,reaction) userid,event_type,reaction,timestamptz \'1970-01-01 utc\' + event_ts::float * interval \'1 second\' as event_ts from reaction_log where channel=? and ts=? and reaction in (\'heart\',\'open_mouth\') order by channel,ts,userid,reaction,event_ts desc ) t1 where event_ts > current_timestamp - interval \'8 hours\' order by event_ts desc limit 50');
+       my $q = $dbh->prepare('select userid,event_type,reaction,to_char(event_ts,\'YYYY-mm-dd HH24:MI\') as event_ts from ( select distinct on (channel,ts,userid,reaction) userid,event_type,reaction,timestamptz \'1970-01-01 utc\' + event_ts::float * interval \'1 second\' as event_ts from reaction_log where channel=? and ts=? and reaction in (\'heart\',\'open_mouth\',\'blue_heart\') order by channel,ts,userid,reaction,event_ts desc ) t1 where event_ts > current_timestamp - interval \'8 hours\' order by event_ts desc limit 50');
        $q->execute($config::invitation_channel, $invitation_ts);
        my @recent_changes = ();
        while (my $ref = $q->fetchrow_hashref) {
@@ -198,18 +294,13 @@ sub create_reaction_log {
                }
                if ($ref->{'reaction'} eq 'open_mouth') {
                        $msg .= '😮';
+               } elsif ($ref->{'reaction'} eq 'blue_heart') {
+                       $msg .= '💙';
                } else {
                        $msg .= '❤️';
                }
                $msg .= ' ';
-               if (exists($slack_userid_to_real_name->{$ref->{'userid'}})) {
-                       $msg .= $slack_userid_to_real_name->{$ref->{'userid'}};
-               } elsif (exists($slack_userid_to_slack_name->{$ref->{'userid'}})) {
-                       $msg .= $slack_userid_to_slack_name->{$ref->{'userid'}} . ' (fant ikke regneark-navn)';
-               } else {
-                       # Should only happen if we didn't see the initial reaction_add, only reaction_remove.
-                       $msg .= $ref->{'userid'} . ' (fant ikke Slack-navn)';
-               }
+               $msg .= best_name_for_log($ref->{'userid'}, $slack_userid_to_real_name, $slack_userid_to_slack_name);
                push @recent_changes, { values => [{ userEnteredValue => { stringValue => $msg } }] };
        }
        while (scalar @recent_changes < 50) {
@@ -218,45 +309,78 @@ sub create_reaction_log {
        return @recent_changes;
 }
 
+sub create_move_log {
+        my ($dbh, $invitation_ts, $prev_invitation_ts) = @_;
+        my $q = $dbh->prepare(<<"EOF");
+SELECT
+  name, g_old.group_name as old_group, g_new.group_name as new_group, TO_CHAR(g_new.change_seen, \'YYYY-mm-dd HH24:MI\') AS change_seen
+FROM ( SELECT * FROM current_group_membership_history WHERE ts=? ) g_old
+  FULL OUTER JOIN ( SELECT * FROM current_group_membership_history WHERE ts=? ) g_new USING (name)
+WHERE
+  g_new.group_name IS DISTINCT FROM g_old.group_name
+  AND g_new.group_name IS NOT NULL
+ORDER BY g_new.change_seen DESC, name
+LIMIT 50
+EOF
+       $q->execute($prev_invitation_ts, $invitation_ts);
+       my @recent_moves = ();
+       while (my $ref = $q->fetchrow_hashref) {
+               my $name = $ref->{'name'};
+               my $old_group = $ref->{'old_group'};
+               my $new_group = $ref->{'new_group'};
+
+               my $msg = $ref->{'change_seen'} . " ";
+               if (!defined($old_group)) {
+                       $msg .= "$name, (ny løper) → $new_group";
+               } else {
+                       $msg .= "$name, $old_group → $new_group";
+               }
+               push @recent_moves, { values => [{ userEnteredValue => { stringValue => $msg } }] };
+       }
+       while (scalar @recent_moves < 50) {
+               push @recent_moves, { values => [{ userEnteredValue => { stringValue => '' } }] };
+       }
+       return @recent_moves;
+}
+
 # Also applies the diff to the database (a bit ugly).
 sub find_diff {
-       my ($dbh, $invitation_ts, $want_names, $have_names, $seen_names) = @_;
+       my ($dbh, $invitation_ts, $want_colors, $have_colors, $seen_names) = @_;
 
        my @diffs = ();
-       for my $real_name (keys %$want_names) {
-               next if (exists($have_names->{$real_name}));
-               skv_log("Markerer at $real_name skal på trening.");
-               push @diffs, [
-                       $real_name,
-                       {
-                               backgroundColor => {
-                                       red => 1,
-                                       green => 1,
-                                       blue => 0,
-                                       alpha => 1
-                               }
+       for my $real_name (keys %$want_colors) {
+               my $wc = $want_colors->{$real_name};
+               if (exists($have_colors->{$real_name})) {
+                       if ($have_colors->{$real_name} eq $wc) {
+                               # Already good.
+                               next;
                        }
-               ];
-               $dbh->do('INSERT INTO applied (channel, ts, name) VALUES (?, ?, ?)', undef,
-                       $config::invitation_channel, $invitation_ts, $real_name);
+                       skv_log("Markerer at $real_name har byttet treningssted.");
+                       push @diffs, [
+                               $real_name, { backgroundColor => $rgb{$wc} }
+                       ];
+                       $dbh->do('UPDATE applied SET color=? WHERE channel=? AND ts=? AND name=?', undef,
+                               $wc, $config::invitation_channel, $invitation_ts, $real_name);
+               } else {
+                       skv_log("Markerer at $real_name skal på trening.");
+                       push @diffs, [
+                               $real_name, { backgroundColor => $rgb{$wc} }
+                       ];
+                       $dbh->do('INSERT INTO applied (channel, ts, name, color) VALUES (?, ?, ?, ?)', undef,
+                               $config::invitation_channel, $invitation_ts, $real_name, $wc);
+               }
        }
-       for my $real_name (keys %$have_names) {
-               next if (exists($want_names->{$real_name}));
+       for my $real_name (keys %$have_colors) {
+               next if (exists($want_colors->{$real_name}));
                if (!exists($seen_names->{lc $real_name})) {
                        # TODO: This can somehow come if we try to add someone who's not in the sheet, too?
                        skv_log("Ønsket å fjerne at $real_name skulle på trening, men de var ikke i regnearket lenger.");
+               } elsif (scalar @{$seen_names->{lc $real_name}} > 1) {
+                       # Don't touch them.
                } else {
                        skv_log("Fjerner at $real_name skal på trening.");
                        push @diffs, [
-                               $real_name,
-                               {
-                                       backgroundColor => {
-                                               red => 1,
-                                               green => 1,
-                                               blue => 1,
-                                               alpha => 0
-                                       }
-                               }
+                               $real_name, { backgroundColor => $rgb{white} }
                        ];
                        $dbh->do('DELETE FROM applied WHERE channel=? AND ts=? AND name=?', undef,
                                $config::invitation_channel, $invitation_ts, $real_name);
@@ -265,7 +389,41 @@ sub find_diff {
        return @diffs;
 }
 
+sub possibly_nag_user {
+       my ($dbh, $ua, $userid) = @_;
+
+       # See if we've nagged this user before.
+       my $q = $dbh->prepare('SELECT * FROM users_nagged WHERE userid=?');
+       $q->execute($userid);
+       if (defined($q->fetchrow_hashref)) {
+               return;
+       }
+
+       my $msg = "Hei! Du meldte deg akkurat på trening, men vi klarer ikke å finne deg i en gruppe i regnearket. For at det skal være enklere for trenerne, ønsker vi gjerne at du går inn på https://regneark.skvidar.run/ og skriver deg inn der med samme navn som du bruker på Slack. Om du er usikker på hvilken gruppe som passer for deg, ta gjerne kontakt med en trener. Velkommen på trening og til klubben!";
+
+       skv_log("Sender melding til $userid for å spørre om gruppe.");
+       my $content = {
+               channel => $config::invitation_channel,
+               user => $userid,
+               text => $msg
+       };
+       my $response = $ua->post(
+               'https://slack.com/api/chat.postEphemeral',
+               Content => JSON::XS::encode_json($content),
+               Content_type => 'application/json;charset=UTF-8',
+               Authorization => 'Bearer ' . $config::slack_oauth_token
+       );
+       die $response->status_line if !$response->is_success;
+       my $msg_json = JSON::XS::decode_json($response->decoded_content);
+       die "Something went wrong: " . $response->decoded_content if (!defined($msg_json) || !$msg_json->{'ok'});
+
+       # Mark that we've sent the message, so it won't happen again.
+       $dbh->do('INSERT INTO users_nagged (userid, last_nag) VALUES (?, CURRENT_TIMESTAMP)', undef, $userid);
+}
+
 sub run {
+       my $start = [Time::HiRes::gettimeofday];
+
        @log = ();
        skv_log("Siste sync startet: " . POSIX::ctime(time));
 
@@ -275,9 +433,9 @@ sub run {
        my $ua = LWP::UserAgent->new('SKVidarLang/1.0');
        my $token = get_oauth_bearer_token($ua);
 
-       # Find the newest message, and what it is linked to.
+       # Find the newest message, what it is linked to, and what was the one before it (for group diffing).
        # TODO: Support more than one, and test better for errors here.
-       my $q = $dbh->prepare('select * from message_sheet_link where channel=? order by ts desc limit 1');
+       my $q = $dbh->prepare('select * from message_sheet_link where channel=? order by ts desc limit 2');
        $q->execute($config::invitation_channel);
        my $linkref = $q->fetchrow_hashref;
        my $invitation_ts = $linkref->{'ts'};
@@ -286,17 +444,40 @@ sub run {
 
        my ($tab_name, $tab_id) = get_spreadsheet_with_title($ua, $token, $wanted_sheet_title);
        if (!defined($tab_name)) {
-               skv_log("Fant ikke noen fane med “$wanted_sheet_title” i navnet; kan ikke synkronisere.\n");
+               skv_log("Fant ikke noen fane med «$wanted_sheet_title» i navnet; kan ikke synkronisere.\n");
                sheet_batch_update($ua, $token, [ serialize_skv_log_to_sheet() ]);
                die;
        }
 
+       # Store away the second-newest ID.
+       my $prev_invitation_ts = $q->fetchrow_hashref->{'ts'};
+
        # Find everyone who are marked as attending on Slack (via reactions).
-       $q = $dbh->prepare('SELECT DISTINCT userid FROM current_reactions WHERE channel=? AND ts=? AND reaction IN (\'heart\', \'open_mouth\')');
+       $q = $dbh->prepare('SELECT DISTINCT userid,reaction FROM current_reactions WHERE channel=? AND ts=? AND reaction IN (\'heart\', \'open_mouth\', \'blue_heart\')');
        $q->execute($config::invitation_channel, $invitation_ts);
        my @attending_userids = ();
+       my %colors = ();
+       my %double = ();
        while (my $ref = $q->fetchrow_hashref) {
-               push @attending_userids, $ref->{'userid'};
+               my $userid = $ref->{'userid'};
+               push @attending_userids, $userid;
+               if ($ref->{'reaction'} eq 'blue_heart') {
+                       if (exists($colors{$userid}) && $colors{$userid} eq 'yellow') {
+                               $double{$userid} = 1;
+                       }
+                       $colors{$userid} = 'blue';
+               } else {
+                       if (exists($colors{$userid}) && $colors{$userid} eq 'blue') {
+                               $double{$userid} = 1;
+                       }
+                       $colors{$userid} = 'yellow';
+               }
+       }
+
+       # Remove double-attenders (we will log them as warnings further down).
+       @attending_userids = grep { !exists($double{$_}) } @attending_userids;
+       for my $userid (keys %double) {
+               delete $colors{$userid};
        }
 
        # Get the list of all people in the sheet (we're going to need them soon anyway).
@@ -305,6 +486,10 @@ sub run {
        );
        my $main_sheet_json = JSON::XS::decode_json($response->decoded_content);
 
+       # Update the list of groups we've seen people in.
+       my %assignments = get_group_assignments($main_sheet_json);
+       update_assignment_db($dbh, $config::invitation_channel, $invitation_ts, \%assignments);
+
        my %seen_names = find_where_each_name_is($main_sheet_json);
 
        # Find duplicates.
@@ -391,6 +576,7 @@ sub run {
                        }
                        if ($#candidates == -1) {
                                skv_log("$slack_name ($userid) er påmeldt på Slack, men fant ikke et regneark-navn for dem.");
+                               possibly_nag_user($dbh, $ua, $userid);
                        } elsif ($#candidates == 0) {
                                my $name = $candidates[0];
                                $slack_userid_to_real_name{$userid} = $name;
@@ -415,8 +601,14 @@ sub run {
        );
        die $response->decoded_content if (!$response->is_success);
 
+       # Now that we have Slack names, we can log double-reacters.
+       for my $userid (keys %double) {
+               my $name = best_name_for_log($userid, \%slack_userid_to_real_name, \%slack_userid_to_slack_name);
+               skv_log("$name er påmeldt flere steder på Slack; vet ikke hvilken som skal brukes.");
+       }
+
        # Find the list of names to mark yellow.
-       my %want_names = ();
+       my %want_colors = ();
        my $main_sheet_rows = $main_sheet_json->{'sheets'}[0]{'data'}[0]{'rowData'};
        for my $userid (@attending_userids) {
                next if (!exists($slack_userid_to_real_name{$userid}));
@@ -432,22 +624,22 @@ sub run {
                        if (scalar @$seen >= 2) {
                                skv_log("$slack_name ($userid) er påmeldt på Slack, men står flere steder (se over); vet ikke hvilken celle som skal brukes.");
                        } else {
-                               $want_names{$seen->[0][0]} = 1;
+                               $want_colors{$seen->[0][0]} = $colors{$userid};
                        }
                }
        }
 
        # Find the list of names we already marked yellow.
-       my %have_names = ();
+       my %have_colors = ();
        $dbh->{AutoCommit} = 0;
        $dbh->do('SET TRANSACTION ISOLATION LEVEL SERIALIZABLE');
-       $q = $dbh->prepare('SELECT name FROM applied WHERE channel=? AND ts=?');
+       $q = $dbh->prepare('SELECT name,color FROM applied WHERE channel=? AND ts=?');
        $q->execute($config::invitation_channel, $invitation_ts);
        while (my $ref = $q->fetchrow_hashref) {
-               $have_names{$ref->{'name'}} = 1;
+               $have_colors{$ref->{'name'}} = $ref->{'color'};
        }
 
-       my @diffs = find_diff($dbh, $invitation_ts, \%want_names, \%have_names, \%seen_names);
+       my @diffs = find_diff($dbh, $invitation_ts, \%want_colors, \%have_colors, \%seen_names);
 
        my @yellow_updates = ();
        if (scalar @diffs > 0) {
@@ -455,9 +647,13 @@ sub run {
                for my $diff (@diffs) {
                        my $real_name = $diff->[0];
 
-                       # See if we can find them in the spreadsheet.
                        my $seen = $seen_names{lc $real_name};
-                       die if (!defined($seen) || scalar @$seen > 1);
+
+                       # We've already complained about these earlier, so just skip them silently.
+                       next if (scalar @$seen > 1);
+
+                       # See if we can find them in the spreadsheet.
+                       die "Could not find $real_name" if (!defined($seen));
                        my $rowno = $seen->[0][1];
                        my $colno = $seen->[0][2];
                        push @yellow_updates, {
@@ -495,14 +691,36 @@ sub run {
                }
        };
 
+       my @recent_moves = create_move_log($dbh, $invitation_ts, $prev_invitation_ts);
+       push @yellow_updates, {
+               updateCells => {
+                       rows => \@recent_moves,
+                       fields => 'userEnteredValue.stringValue',
+                       range => {
+                               sheetId => $config::log_tab_id,
+                               startRowIndex => 4,
+                               endRowIndex => 4 + scalar @recent_moves,
+                               startColumnIndex => 1,
+                               endColumnIndex => 2
+                       }
+               }
+       };
+
        # Push the final set of updates (including the log).
        skv_log("Ferdig.");
        push @yellow_updates, serialize_skv_log_to_sheet();
        sheet_batch_update($ua, $token, \@yellow_updates);
        $dbh->commit;
+
+       my $elapsed = Time::HiRes::tv_interval($start);
+       printf "Tok %.0f ms.\n", 1e3 * $elapsed;
 }
 
 if ($#ARGV >= 0 && $ARGV[0] eq '--daemon') {
+       # Start with a single, forced run.
+       unlink("/srv/skvidar-slack.sesse.net/marker");
+       run();
+
        while (1) {
                if (!unlink("/srv/skvidar-slack.sesse.net/marker")) {
                        unless ($!{ENOENT}) {