Add SIGQUIT handler, fix setting of default value of loglevel

.perlcriticrc

@@ -14,8 +14,6 @@
 # Perl::Critic::Policy::ProhibitImplicitImport
 # Perl::Critic::Policy::CodeLayout::RequireUseUTF8
 
-severity    = 5
-verbose     = 8
 
 # minorify some of warnings that are not really critical
 [-InputOutput::RequireCheckedClose]
@@ -131,3 +129,5 @@ lib_sections    = NAME | VERSION | DESCRIPTION | AUTHOR | LICENSE AND COPYRIGHT
 max_length = 256
 
 [-Miscellanea::ProhibitUnrestrictedNoCritic]
+
+[-RegularExpressions::ProhibitFixedStringMatches]

bin/aleesa-craniac

@@ -47,7 +47,10 @@ my $runAsUid = $UID;
 
 my $piddir = '/var/run/' . basename (abs_path ($PROGRAM_NAME));
 my $C = LoadConf ();
-my $loglevel = $C->{'loglevel'} // 'notice';
+
+my $loglevel = 'notice';
+$loglevel = $C->{'loglevel'} if defined $C->{'loglevel'};
+
 my $logfile;
 
 if (defined $C->{'log'}) {
@@ -136,6 +139,7 @@ close $MYPID;                                        ## no critic (InputOutput::
 # подстраховка на случай ядерной войны.
 local $SIG{INT}  = \&__signal_handler;
 local $SIG{TERM} = \&__signal_handler;
+local $SIG{QUIT} = \&__signal_handler;
 
 # Поставим правильное имя процесса в proctitle
 $PROGRAM_NAME = abs_path $PROGRAM_NAME;              ## no critic (Variables::RequireLocalizedPunctuationVars)

cpanfile

@@ -5,7 +5,6 @@ requires 'Cwd',                  '==3.75';
 requires 'Cwd::utf8',            '==0.011';
 requires 'Data::Dumper',         '==2.183';
 requires 'Digest::SHA',          '==6.02';
-requires 'Encode',               '==3.16';
 # Part of perl
 requires 'English',              '0';
 requires 'File::Basename',       '==2.85';
@@ -18,15 +17,13 @@ requires 'Log::Any',             '==1.710';
 requires 'Log::Any::Adapter',    '==1.710';
 requires 'Log::Any::For::Std',   '==0.05';
 requires 'Math::Random::Secure', '==0.080001';
-requires 'Mojolicious',          '==9.24';
+requires 'Mojolicious',          '==9.26';
 # Part of Mojolicious
 requires 'Mojo::IOLoop',         '0';
 requires 'Mojo::IOLoop::Signal', '==0.003';
-requires 'Mojo::Redis',          '==3.27';
+requires 'Mojo::Redis',          '==3.29';
 # Part of perl
 requires 'POSIX',                '0';
-# Buggy, latest version works, as pub-sub, but fails tests
-requires 'Protocol::Redis::XS',  '0';
 requires 'Text::Fuzzy',          '==0.29';
 # Part of perl
 requires 'utf8',                 '0';

lib/Craniac.pm

@@ -19,15 +19,12 @@ use English qw ( -no_match_vars );
 # Модули для работы приложения
 use Clone qw (clone);
 use Data::Dumper qw (Dumper);
-use Digest::SHA qw (sha1_base64);
+use Digest::SHA qw (sha256_hex);
-use Encode qw (encode_utf8);
 use File::Spec ();
 use Hailo ();
 use Log::Any qw ($log);
 use Mojo::IOLoop ();
 use Mojo::IOLoop::Signal ();
-# Чтобы "уж точно" использовать hiredis-биндинги, загрузим этот модуль перед Mojo::Redis
-use Protocol::Redis::XS ();
 use Mojo::Redis ();
 use Math::Random::Secure qw (irand);
 use Text::Fuzzy qw (distance_edits);
@@ -40,13 +37,16 @@ use Exporter qw (import);
 our @EXPORT_OK = qw (RunCraniac);
 
 sub RandomCommonPhrase ();
-sub utf2sha1 ($);
 sub fmatch (@);
 sub brains (@);
 sub RunCraniac ();
 
 my $c = LoadConf ();
-my $fwd_cnt = $c->{'forward_max'} // 5;
+my $fwd_cnt = 5;
+
+if (defined $c->{'forward_max'}) {
+	$fwd_cnt = $c->{'forward_max'};
+}
 
 my $hailo;
 
@@ -69,17 +69,6 @@ sub RandomCommonPhrase () {
 	return $myphrase[irand ($#myphrase + 1)];
 }
 
-sub utf2sha1 ($) {
-	my $string = shift;
-
-	if ($string eq '') {
-		return sha1_base64 '';
-	}
-
-	my $bytes = encode_utf8 $string;
-	return sha1_base64 $bytes;
-}
-
 sub fmatch (@) {
 	my $srcphrase = shift;
 	my $answer = shift;
@@ -97,30 +86,27 @@ sub fmatch (@) {
 
 # Ленивая инициализация мозга. Создаём/открываем мозг только если нам надо "подумать" а не на каждую входящую фразу.
 sub brains (@) {
-	my $chatid = shift;
+	my $plugin = shift;
-	my $telegram = shift // 0;
+	my $cid = shift;
 
-	unless (defined $hailo->{$chatid}) {
+	unless (defined $hailo->{$cid}) {
-		my $cid = $chatid;
 		my $brainname;
 
 		# Костыль, чтобы не портировать данные из телеграммных мозгов и не заниматься переименованием
-		unless ($telegram) {
+		if ($plugin eq 'telegram') {
-			$cid = utf2sha1 ($chatid);
-			$cid =~ s/\//-/xmsg;
-			$brainname = sprintf '%s/%s.sqlite', $c->{braindir}, $cid;
-		} else {
 			$brainname = sprintf '%s/%s.brain.sqlite', $c->{braindir}, $cid;
+		} else {
+			$brainname = sprintf '%s/%s.sqlite', $c->{braindir}, $cid;
 		}
 
-		$hailo->{$chatid} = eval {
+		$hailo->{$cid} = eval {
 			Hailo->new (
 				brain => $brainname,
 				order => 3,
 			);
 		};
 
-		if (defined $hailo->{$chatid}) {
+		if (defined $hailo->{$cid}) {
 			$log->info (sprintf '[INFO] Lazy init brain: %s', $brainname);
 		} else {
 			$log->error ("[ERROR] $EVAL_ERROR");
@@ -194,11 +180,21 @@ my $parse_message = sub {
 	my $csign = $answer->{misc}->{csign};
 	$answer->{message} = '';
 
+	my $is_command = 0;
+	my $cmd = $m->{message};
+
+	if ($m->{message} =~ /^\Q$csign/u) {
+		$cmd = substr $m->{message}, length ($csign);
+		$is_command = 1;
+	}
+
 	if ($m->{misc}->{bot_nick} ne '') {
 		# Если нам сообщили ник бота, попробуем вымарать его из оригинального текста сообщения, в большинстве случаев он
-		# нам не интересн.
+		# нам не интересен.
 		$phrase =~ s/\s*\,?\s*($m->{misc}->{bot_nick})\s*\,?\s*/ /gui;
 
+		# TODO: проработать реакцию на "пьяный бред".
+
 		# В ключевых фразах можно обратиться с некоторыми командами к боту по нику, не используя символ команды
 		if ($m->{message} =~ /^\s*\Q$m->{misc}->{bot_nick}\E\s*$/ui) {
 			$reply = 'Чего?';
@@ -215,22 +211,25 @@ my $parse_message = sub {
 		}
 	} else {
 		# Если нам не сообщили ник бота, то ищем команды, используя символ команды
-		if ($m->{message} =~ /^\Q$csign\Eping\.?\s*$/ui) {
+		if ($is_command) {
+			if ($cmd =~ /^ping\.?\s*$/ui) {
 				$reply = 'Pong.';
-		} elsif ($m->{message} =~ /^\Q$csign\Epong\.?\s*$/i) {
+			} elsif ($cmd =~ /^pong\.?\s*$/i) {
 				$reply = 'Wat?';
-		} elsif ($m->{message} =~ /^\Q$csign\Eпин(г|х)\.?\s*$/ui) {
+			} elsif ($cmd=~ /^пин(г|х)\.?\s*$/ui) {
 				$reply = 'Понг.';
-		} elsif ($m->{message} =~ /^\Q$csign\Eпон(г|х)\.?\s*$/ui) {
+			} elsif ($cmd =~ /^пон(г|х)\.?\s*$/ui) {
 				$reply = 'Шта?';
-		} elsif ($m->{message} =~ /^\Q$csign\E(хэлп|halp)\s*$/ui) {
+			} elsif ($cmd=~ /^(хэлп|halp)\s*$/ui) {
 				$reply = 'HALP!!!11';
 			}
 		}
+	}
 
 	# Поищем в тексте входящего сообщения команды общего плана, с символом команды.
 	unless (defined $reply) {
-		if ($m->{message} =~ /^\Q$csign\E(kde|кде)\s*$/ui) {
+		if ($is_command) {
+			if ($cmd =~ /^(kde|кде)\s*$/ui) {
 				my @phrases = (
 					'Нет, я не буду поднимать вам плазму.',
 					'Повторяйте эту мантру по утрам не менее 5 раз: "Плазма не падает." И, возможно, она перестанет у вас падать.',
@@ -239,7 +238,8 @@ my $parse_message = sub {
 				);
 
 				$reply = $phrases[irand ($#phrases + 1)];
-		} elsif ($m->{message} =~ /^\Q$csign\E(coin|монетка)$/ui) {
+			} elsif ($cmd =~ /^(coin|монетка)$/ui) {
+				# Нам тут говорили, что есть некая вероятность, что монета упадёт ребром... И даже цифры называли.
 				if (rand (101) < 0.016) {
 					$reply = 'ребро';
 				} else {
@@ -257,14 +257,15 @@ my $parse_message = sub {
 						}
 					}
 				}
-		} elsif ($m->{message} =~ /^\Q$csign\E(roll|dice|кости)$/ui) {
+			} elsif ($cmd =~ /^(roll|dice|кости)$/ui) {
 				$reply = sprintf 'На первой кости выпало %d, а на второй — %d.', irand (6) + 1, irand (6) + 1;
-		} elsif ($m->{message} =~ /^\Q$csign\E(ver|version|версия)$/ui) {
+			} elsif ($cmd =~ /^(ver|version|версия)$/ui) {
 				$reply = 'Версия:  Четыре.Технологическое_превью';
-		} elsif ($m->{message} =~ /^\Q$csign\E(lat|лат)\s*$/ui) {
+			} elsif ($cmd =~ /^(lat|лат)\s*$/ui) {
 				$reply = Lat ();
 			}
 		}
+	}
 
 	# У нас уже есть чего ответить, нам попалась известная фраза, на которую одинаковый ответ, независимо от типа
 	# общения - публичного или приватного.
@@ -343,29 +344,31 @@ my $parse_message = sub {
 		}
 
 		# Попадаем сюда только если входящая фраза не распознана ранее как ключевая.
-		if ($m->{misc}->{answer}) {
+		my $cid;
-			# Пытаемся что-то генерировать, если фраза длиннее 3-х букв
-			if (length ($m->{message}) > 3) {
-				my $telegram = 0;
 
 		if ($m->{plugin} eq 'telegram') {
-					$telegram = 1;
+			$cid = $chatid;
+		} else {
+			$cid = sha256_hex (sprintf '%s/%s', $m->{plugin}, $chatid);
 		}
 
+		if ($m->{misc}->{answer}) {
+			# Пытаемся что-то генерировать, если фраза длиннее 3-х букв.
+			if (length ($m->{message}) > 3) {
 				# Lazy brain init
-				if (brains ($chatid, $telegram)) {
+				if (brains ($m->{plugin}, $cid)) {
-					$reply = $hailo->{$chatid}->learn_reply ($m->{message});
+					$reply = $hailo->{$cid}->learn_reply ($m->{message});
 
-					# Если из реактора вернулся undef выдаём "общую" фразу из словаря
+					# Если из реактора вернулся undef выдаём "общую" фразу из словаря.
 					unless (defined $reply) {
 						$reply = RandomCommonPhrase ();
 					} else {
-						# Если из реактора вернулась пустота выдаём "общую" фразу из словаря
+						# Если из реактора вернулась пустота выдаём "общую" фразу из словаря.
 						if ($reply eq '') {
 							$reply = RandomCommonPhrase ();
 						}
 
-						# Если сгенерированная фраза слишком похожа на начальную - выдаём "общую" фразу из словаря
+						# Если сгенерированная фраза слишком похожа на начальную - выдаём "общую" фразу из словаря.
 						if (fmatch (lc ($m->{message}), lc ($reply))) {
 							$reply = RandomCommonPhrase ();
 						}
@@ -379,16 +382,11 @@ my $parse_message = sub {
 				}
 			}
 		} else {
-			# пытаемся что-то запоминать, если фраза длиннее 3-х букв
+			# Пытаемся что-то запоминать, если фраза длиннее 3-х букв
 			if (length ($m->{message}) > 3) {
-				my $telegram = 0;
+				# Lazy brain init
-
+				if (brains ($m->{plugin}, $cid)) {
-				if ($m->{plugin} eq 'telegram') {
+					$hailo->{$cid}->learn ($m->{message});
-					$telegram = 1;
-				}
-
-				if (brains ($chatid, $telegram)) {
-					$hailo->{$chatid}->learn ($m->{message});
 				}
 			}
 		}
@@ -452,6 +450,7 @@ sub RunCraniac () {
 
 	Mojo::IOLoop::Signal->on (TERM => $__signal_handler);
 	Mojo::IOLoop::Signal->on (INT  => $__signal_handler);
+	Mojo::IOLoop::Signal->on (QUIT => $__signal_handler);
 
 	do { Mojo::IOLoop->start } until Mojo::IOLoop->is_running;
 	return;