Fewer messages when listing.

[matrix-goal-bot.git] / matrix_goal_bot / bot.py

diff --git a/matrix_goal_bot/bot.py b/matrix_goal_bot/bot.py
index d4638558730d257a641f99f792b6544d3285f0b7..ade650a6fe154ba02033d59a525984dfd3385885 100644 (file)
--- a/matrix_goal_bot/bot.py
+++ b/matrix_goal_bot/bot.py
@@ -1,5 +1,5 @@
 '''
-Prefix: !goal
+Prefix: .goal
 
 Synax:
     .goal COMMAND ARGS
@@ -21,7 +21,7 @@ More commands:
     add.vision - add a new vision (collection of goals)
     close.vision ID - mark vision as complete by ID
     dump - dump the full data set - BEWARE - so many messages
-    later GOAL_ID - put off goal for 14 days
+    later VISION_ID - put off all goals in VISION for 14 days
     list.later - list all 'later' goals
     print.vision ID - print a vision and all goals
     set.vision GOAL_ID VISION_ID- associate GOAL_ID with VISION_ID
@@ -30,6 +30,7 @@ More commands:
     remove ID - remove a task by ID
     start GOAL_ID - un-put-off a goal (reverse of later)
     visionless - list goals with no set vision
+    wins - list recently completed goals
 '''
 
 
@@ -89,9 +90,16 @@ class GoalBot():
         if match.prefix():
             try:
                 result_lines = await self._handle_match(match)
+
+                message_lines = []
                 for line in result_lines:
-                    await self.bot.api.send_text_message(
-                        room.room_id, str(line))
+                    message_lines.append(str(line))
+
+                message = '\n'.join(message_lines)
+
+                await self.bot.api.send_text_message(
+                    room.room_id, message)
+
             except Exception as ex:
                 logger.warn(f'Caught exception: {str(ex)}')
                 dumb_msg_idx = len(str(ex)) % len(snark)
@@ -196,10 +204,13 @@ class GoalBot():
             return results
 
         if match.command('later'):
-            goal_id = int(match.args()[1])
-            vision_id = self.data.put_off_goal(goal_id)
-            vision,goals = self.data.get_vision(vision_id)
-            results = pretty_print(vision, goals)
+            vision_id = int(match.args()[1])
+            self.data.put_off_vision(vision_id)
+            # Want to see what we just cleaned up
+            goals = self.data.get_current_goals()
+            results = ['# Goals list...']
+            for goal in goals:
+                results.append(str(goal))
             return results
 
 
@@ -254,6 +265,18 @@ class GoalBot():
                 results.append(item)
             return results
 
+        if match.command('wins'):
+            items = self.data.wins()
+            results = ['# Wins']
+            if not items:
+                results.append('-- No wins yet --')
+
+            for item in items:
+                results.append(item)
+            return results
+
+
+
         if match.command('set.vision'):
             goal_id = int(match.args()[1])
             vision_id = int(match.args()[2])