chiark / gitweb /
Many bugfixes from actual testing!
[ircbot] / bot.tcl
diff --git a/bot.tcl b/bot.tcl
index f05c738b908600196c6cda8f76db504f3ab12007..ac5714b3d4de991892eaad3b7e343aa2d7056008 100755 (executable)
--- a/bot.tcl
+++ b/bot.tcl
@@ -145,6 +145,26 @@ proc looking_whenwhere {when where} {
     return $str
 }
 
+proc tell_getcstate {} {
+    # uses nl from caller's context
+    # imports telling (as the nick_telling) and u
+    # sets stt, telling_when
+    uplevel 1 {
+       upvar #0 nick_telling($nl) telling
+       upvar #0 nick_unique($nl) u
+
+       if {[info exists telling]} {
+           manyset $telling u_last stt telling_when
+           if {![info exists u] || "$u_last" != "$u"} {
+               set stt undelivered
+           }
+       } else {
+           set stt undelivered
+           set telling_when 0
+       }
+    }
+}
+
 proc tell_event {nl event} {
     # For `act' we *haven't* yet done the 750ms delay; we implement
     # that here.  Also, here we turn `talk' into `talk' now and `act'
@@ -152,7 +172,7 @@ proc tell_event {nl event} {
     # and new msg events are handled by the command procedures, not here.
     global calling_nick
     if {[info exists calling_nick]} { set save $calling_nick }
-puts "[list tell_event $nl $event]"
+    set r DELAYED
     switch -exact $event {
        none { }
        talk {
@@ -165,48 +185,38 @@ puts "[list tell_event $nl $event]"
        ident - msgsarrive {
            tell_event_core $nl $event
        }
+       tellme {
+           set r [tell_event_core $nl TELLME]
+       }
        default {
            error "tell_event $nl $event"
        }
     }
     if {[info exists save]} { set calling_nick $save }
-global errorInfo
-puts "[list tell_event $nl $event] $errorInfo"
-}
-
-proc tell_getcstate {} {
-    # uses nl from caller's context
-    # imports telling (as the nick_telling) and u
-    # sets stt, telling_when
-    uplevel 1 {
-       upvar #0 nick_telling($nl) telling
-       upvar #0 nick_unique($nl) u
-
-       if {[info exists telling]} {
-           manyset $telling u_last stt telling_when
-           if {![info exists u] || "$u_last" != "$u"} {
-               unset telling; unset stt; unset telling_when
-           }
-       }
-
-       if {![info exists stt]} {
-           set stt norecord
-           set telling_when $now
-       }
-    }
+    return $r
 }
 
 proc tell_event_core {nl event} {
-    catch_logged { tell_event_core1 $nl $event }
+    global tell_event_teventi errorInfo
+    set tell_event_teventi "*$event* $nl"
+    if {[catch {
+       tell_event_core1 $nl $event
+    } emsg]} {
+       log_intern "tell event error" "$emsg >$errorInfo<"
+       set emsg ERROR
+    } else {
+       log_intern "tell event" "done $tell_event_teventi $emsg"
+    }
+    return $emsg
 }
 
 proc tell_event_core1 {nl event} {
-    # event is `talk', `act', `ident' or `msgsarrive'
+    # event is `talk', `act', `ident', `msgsarrive' or `TELLME'
     # When user talks we actually get talk now and act later
     global calling_nick
     set calling_nick $nl
     set iml [msgsdb_get $nl inbound]
-    if {![llength $iml]} return
+    if {![llength $iml]} { return nomsgs }
 
     set now [clock seconds]
     tell_getcstate
@@ -214,14 +224,14 @@ proc tell_event_core1 {nl event} {
 
     # Now we have the components of a telling state
     #   u     - nick_unique (unset if not visible)
-    #   stt   - state: norecord, mentioned, passede
+    #   stt   - state: undelivered, mentioned, passede
     #   ago   - how long ago since we did anything
 
     # We compute an evstate to dispatch on as follows:
 
     # evstate is string of letters
     #   current state
-    #      n   NORECORD (MESSAGES)
+    #      u   UNDELIVERED (MESSAGES)
     #      m   MENTIONED
     #      p   PASSED
     #   event
@@ -229,6 +239,7 @@ proc tell_event_core1 {nl event} {
     #      a   act
     #      i   ident
     #      m   msgsarrive
+    #      T   tellme (user command)
     #   security level and timing
     #      ii  Insecure
     #      ss  Secure and soon (before interval)
@@ -249,15 +260,15 @@ proc tell_event_core1 {nl event} {
     append evstate [string range $stt 0 0]
     append evstate [string range $event 0 0]
 
-    manyset [tell_effective_sec $nl] sec secwhen
+    manyset [nickdb_get_sec_effective $nl] sec secwhen
     switch -exact $sec {
        insecure { append evstate ii }
-       secure { append evstate [expr {$ago<$secwhen ? "sl" : "ss"}] }
+       secure { append evstate [expr {$ago<$secwhen ? "ss" : "sl"}] }
        default { append evstate "#$sec#" }
     }
 
     upvar #0 nick_username($nl) nu
-    if {[info exists nu] && "$nu" == "[nickdb_get $nl username]"} {
+    if {[info exists nu] && "$nu" == "[nickdb_get_username $nl]"} {
        append evstate i
     } else {
        append evstate u
@@ -273,37 +284,44 @@ proc tell_event_core1 {nl event} {
        default { append evstate "#$rel#" }
     }
 
+    global tell_event_teventi
+    set tell_event_teventi "$evstate $ago $nl"
     switch -glob $evstate {
        pt???rv {
            # consider delivered:
            #  (very recently passed, and the user talks)
-           tell_delete_msgs {} $nl
-           return
+           set ndel [tell_delete_msgs {} $nl]
+           sendprivmsg $nl \
+ "I'm assuming you got the $ndel message(s) I just passed on."
+           return delivered
        }
        pm????? {
            # oops, messages passed are now out of date
            catch_restoreei { unset telling }
-           return
+           return reset
        }
        ?m????? {
            # ignore new msgs if we haven't passed yet
-           return
+           return ignorenew
        }
-       nt????? - mt????? -
+       ut????? - mt????? -
        pt???uu - pt???rs - pt???rl - pt???p? {
            # ignore (any other `talk's) - act handles these
-           return
+           return ignoretalk
        }
-       ni????? - naii??? - nas?i?? - mi????? - pa????l {
+       ui????? -
+       uaii?uu - uaii??l - uas?iuu - uas?i?l -
+       mi????? - pa????l -
+       ?Tii??? - ?Ts?i?? {
            # pass and then stuff
-           if {[length $iml] == 3} {
+           if {[llength $iml] == 3} {
                manyset $iml sender sentwhen msg
                sendprivmsg $nl \
- "$sender asked me [showintervalsecs [expr {$now-$sentwhen}] 0]\
+ "$sender asked me, [showinterval [expr {$now-$sentwhen}]],\
  to tell you: $msg"
             } else {
                sendprivmsg $nl \
- "Here are the [expr {[llength $iml]/3}] messages there are for you:"
+ "I have [expr {[llength $iml]/3}] messages for you:"
                while {[llength $iml] >= 3} {
                    manyset [lrange $iml 0 2] sender sentwhen msg
                    set iml [lrange $iml 3 end]
@@ -313,33 +331,44 @@ proc tell_event_core1 {nl event} {
            }
            if {"$rel" == "unreliable"} {
                tell_delete_msgs {} $nl
-               return
+               return toldunreliable
            }
            set stt passed
+           set re passed
        }
-       nas?u?? {
+       uaslu?? {
            sendprivmsg $nl {You have messages (so identify yourself please).}
            set stt mentioned
+           set re mentioned
+       }
+       ?Ts?u?? {
+           sendprivmsg $nl {You must identify yourself to see your messages.}
+           return ignoreuitm
        }
        masl??? {
            sendprivmsg $nl {Don't forget about your messages.}
+           return remind
        }
        pi????? {
-           return
+           return ignorepi
        }
-       mass??? - pa????v - pa????s {
+       mass??? - pa????v - pa????s -
+       uaii??v - uaii??s -
+       uas?i?v - uas?i?s -
+       uassu?? {
            # too soon
-           return
+           return ignoresoon
        }
        * {
            error "tell_event_core nl=$nl evstate=$evstate ?"
        }
     }
     if {![info exists u]} {
-       catch_restoreei { unset telling }
+       set telling [list {} undelivered $now]
     } else {
        set telling [list $u $stt $now]
     }
+    return $re
 }
 
 proc recordlastseen_n {n how here} {
@@ -445,7 +474,7 @@ proc chanmode_o1 {m g p chan} {
            set chan_initialop([irctolower $chan]) $u
            sendprivmsg $n \
  "Thanks.  You can use `channel manager ...' to register this channel."
-           if {![nickdb_exists $n] || ![string length [nickdb_get $n username]]} {
+           if {![string length [nickdb_get_username $n username]]} {
                sendprivmsg $n \
  "(But to do that you must register your nick securely first.)"
            }
@@ -1147,8 +1176,7 @@ def_ucmd invite {
     }
     set invitees {}
     while {[ta_anymore]} {
-       set invitee [ta_word]
-       check_nick $invitee
+       set invitee [ta_nick]
        lappend invitees $invitee
     }
     foreach invitee $invitees {
@@ -1209,19 +1237,25 @@ def_ucmd channel {
     channel/$subcmd
 }
 
-proc tell_effective_sec {n} {
+proc nickdb_get_username {n} {
+    if {![nickdb_exists $n]} { return "" }
+    return [nickdb_get $n username]
+}
+
+proc nickdb_get_sec_effective {n} {
     set l [nickdb_get $n tellsec]
-    set u [nickdb_get $n username]
+    set u [nickdb_get_username $n]
     if {"[lindex $l 0]" == "secure" && ![string length $u]} { set l insecure }
     return $l
 }
 
 proc tell_peernicks {text} {
+    global errorInfo errorCode
     set text [irctolower [string trim $text]]
     set senders [split $text " "]
     foreach sender $senders {
        if {[catch { check_nick $sender } emsg]} {
-           error "invalid sender nick `$sender': $emsg" $errorInfo $errorCode
+           error "invalid nick `$sender': $emsg" $errorInfo $errorCode
        }
     }
     return $senders
@@ -1237,9 +1271,9 @@ proc msgsdb_set_maydelete {n key l otherkey} {
 proc tell_delete_msgs {lsenders lrecip} {
     set ninbound {}
     set ndel 0
-    foreach {s t m} [msgsdb_get $recip inbound] {
-       if {[llength $senders]} {
-           if {[lsearch -exact $senders [irctolower $s]] == -1} {
+    foreach {s t m} [msgsdb_get $lrecip inbound] {
+       if {[llength $lsenders]} {
+           if {[lsearch -exact $lsenders [irctolower $s]] == -1} {
                lappend ninbound $s $t $m
                continue
            }
@@ -1247,7 +1281,7 @@ proc tell_delete_msgs {lsenders lrecip} {
        set rsenders($s) 1
        incr ndel
     }
-    msgsdb_set_maydelete $recip inbound $ninbound outbound
+    msgsdb_set_maydelete $lrecip inbound $ninbound outbound
     if {![llength $ninbound]} {
        upvar #0 nick_telling($lrecip) telling
        catch { unset telling }
@@ -1283,7 +1317,7 @@ def_ucmd delmsg {
     prefix_nick
     set nl [irctolower $n]
     check_notonchan
-    manyset [tell_effective_sec $n] sec secwhen
+    manyset [nickdb_get_sec_effective $n] sec secwhen
     switch -exact $sec {
        insecure { }
        refuse - mailto {
@@ -1298,10 +1332,20 @@ def_ucmd delmsg {
            error "delmsg sec $sec"
        }
     }
+    if {![llength [msgsdb_get $n inbound]]} {
+       ucmdr "No incoming messages to delete." {}
+    }
     tell_getcstate
+    if {![info exists u]} {
+       usererror \
+ "I can't delete your messages unless I can see you on a channel with me.\
+  Otherwise I might delete a message I hadn't told you about yet."
+    }
     if {"$stt" != "passed"} {
+       set telling [list $u undelivered 0]
        usererror \
- "There are message(s) you've not yet seen; I'll deliver them to you now.\
+ "There are message(s) you may not yet have seen;\
+ I'll deliver them to you now.\
   If you actually want to delete them, just tell me `delmsg' again."
     }
     set senders [tell_peernicks $text]
@@ -1309,8 +1353,6 @@ def_ucmd delmsg {
     if {!$ndel} {
        if {[llength $senders]} {
            ucmdr "No relevant incoming messages to delete." {}
-       } else {
-           ucmdr "No incoming messages to delete." {}
        }
     }
     switch -exact [llength $senders] {
@@ -1323,11 +1365,22 @@ def_ucmd delmsg {
     }
 }
 
+def_ucmd tellme {
+    prefix_nick
+    ta_nomore
+    check_notonchan
+    switch -exact [tell_event [irctolower $n] tellme] {
+       ERROR - INVALID { ucmdr {} {is ill.  Help!} }
+       nomsgs { ucmdr {You have no messages.} {} }
+       default { }
+    }
+}
+
 def_ucmd tell {
     global nick_case ownmailaddr ownfullname
     
     prefix_nick
-    set target [ta_word]
+    set target [ta_nick]
     if {![string length $text]} { error "tell them what?" }
     if {[string length $text] > 400} { error "message too long" }
 
@@ -1335,7 +1388,7 @@ def_ucmd tell {
     set ctarget $target
     if {[info exists nick_case($ltarget)]} { set ctarget $nick_case($ltarget) }
 
-    manyset [tell_effective_sec $target] sec mailtoint mailwhy
+    manyset [nickdb_get_sec_effective $target] sec mailtoint mailwhy
     manyset [nickdb_get $target tellrel] rel relint relwithin
     switch -exact $sec {
        insecure - secure {
@@ -1408,7 +1461,7 @@ $n asked me[expr {[ischan $dest] ? " on $dest" : ""}] to tell you:
 
 def_ucmd who {
     if {[ta_anymore]} {
-       set target [ta_word]; ta_nomore
+       set target [ta_nick]; ta_nomore
        set myself 1
     } else {
        prefix_nick
@@ -1513,7 +1566,7 @@ proc tellme_sec_desc {v n} {
            return "I'll tell you your messages whenever I see you."
        }
        secure {
-           if {[string length [nickdb_get $n username]]} {
+           if {[string length [nickdb_get_username $n]]} {
                return \
  "I'll keep the bodies of your messages private until you identify yourself, reminding you every [showintervalsecs $mailtoint 1]."
            } else {
@@ -1535,7 +1588,7 @@ proc tellme_sec_desc {v n} {
     }
 }
 
-proc tellme_rel_desc {v} {
+proc tellme_rel_desc {v n} {
     manyset $v rel every within
     switch -exact $rel {
        unreliable {
@@ -1618,7 +1671,7 @@ proc tellme_setting_sec_simple {} {
 proc tellme_setting_neednomsgs {} {
     uplevel 1 {
        if {[llength [msgsdb_get $n inbound]]} {
-           usererror "You must delete the messages you have, first."
+           usererror "You must delete the incoming messages you have, first."
        }
     }
 }
@@ -1629,7 +1682,7 @@ def_setting tellme {
     manyset $secv sec
     switch -exact $sec {
        insecure - secure {
-           set mr [tellme_rel_desc [nickdb_get $n tellrel]]
+           set mr [tellme_rel_desc [nickdb_get $n tellrel] $n]
            return "$ms  $mr"
        }
        refuse - mailto {
@@ -1649,13 +1702,13 @@ def_setting tellme {
            set v [list secure $every]
        }
        refuse {
-           telling_setting_neednomsgs
-           telling_setting_sec_simple
+           tellme_setting_neednomsgs
+           tellme_setting_sec_simple
        }
        mailto {
-           telling_setting_neednomsgs
-            set u [nickdb_get $n username]
-           if {[string length $u]} {
+           tellme_setting_neednomsgs
+            
+           if {![string length [set u [nickdb_get_username $n]]]} {
                usererror \
  "Sorry, you must register securely to have your messages mailed\
  (to prevent the use of this feature for spamming).  See `help register'."
@@ -1675,7 +1728,7 @@ def_setting tellme {
            set sr rel
            set v $setting
            if {"$setting" != "unreliable"} {
-               set every [parse_interval_optional 300 3600]
+               set every [ta_interval_optional 300 3600]
                lappend v $every
            }
            if {"$setting" == "remind"} {
@@ -1692,14 +1745,16 @@ def_setting tellme {
        }
     }
     nickdb_set $n tell$sr $v
-    ucmdr [tellme_${sr}_desc $v] {}
+    upvar #0 nick_telling([irctolower $n]) telling
+    catch { unset telling }
+    ucmdr [tellme_${sr}_desc $v $n] {}
 }
 
 proc lnick_checktold {luser} {
     set ml [msgsdb_get $luser outbound]
     if {![llength $ml]} return
     set is1 [expr {[llength $ml]==3}]
-    set m1 "FYI, I haven't yet passed on your"
+    set m1 "FYI, I haven't yet delivered your"
     set ol {}
     set now [clock seconds]
     while {[llength $ml]} {
@@ -1772,8 +1827,8 @@ def_ucmd identpass {
            irc-identpass $n
     upvar #0 nick_username($luser) rec_username
     set rec_username $username
+    after 50 [list tell_event $luser ident]
     ucmdr "Pleased to see you, $username." {}
-    tell_event $luser ident
 }
 
 def_ucmd summon {
@@ -1883,7 +1938,7 @@ proc lnick_marktime_now {luser why mentiontold} {
     global calling_nick
     set calling_nick $luser
     sendprivmsg $luser [lnick_pingstring $why $oc ""]
-    if {$mentiontold} { lnick_checktold $luser }
+    if {$mentiontold} { after 150 [list lnick_checktold $luser] }
     lnick_marktime_reset $luser
 }    
 
@@ -1933,7 +1988,7 @@ def_ucmd ping {
        }
        if {[llength $oc]} { lnick_marktime_reset $ln }
     }
-    lnick_checktold $ln
+    after 150 [list lnick_checktold $ln]
     ucmdr {} [lnick_pingstring "Pong!" $oc $text]
 }