agents_control 0.2.1 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5ca83918e01b4dc98c7b72f17da5cd31d859d863d61614f7eee74dd01e62ecb2
-  data.tar.gz: d50f30441533aded38f243800dda29a884fd01fda135506dee034115eeb63e7a
+  metadata.gz: aadeb50868f7e15b7da9346e051f268ffad1c652c1e7297d54cb328380b030f6
+  data.tar.gz: cf62c6519dfd54bb7b9ee169c5e1b58f5bcf92b54993e5a61e954547180b6ec2
 SHA512:
-  metadata.gz: 9dd870ff93dd579dd1bbbc7cffd7e9f62d7a5d25ba79d2f69adb6018ac1ae8061614b5e1d09db5bc817d65f8f6a33fc7557b8a6a035245d640e92373bcdc598d
-  data.tar.gz: 71c5a463b794b2196fa3023a0a703ab6cd02da596f2b1f003ea1365db96f9a67e8d3ab3e1ddc51bdd470d76217395907565a2d53ef65dccf6fc0b411c04dee0a
+  metadata.gz: dce0f05882260d26dddc1a65f392ef0cf8ad7351b1ccbd07e29d2378c2871077434669e4f3c7248b682fd10aa72aedbfbbfae5739703222040bd3741e9a2be4a
+  data.tar.gz: f3e61a373a6252b920e6086b9e9b01591855fecc8708ffd059e0f87d412fce749d97cc445a6553aa42d69e404c60973ab1eb6551c2db60ae3362cce2756964f7

data/README.md

@@ -126,6 +126,14 @@ shell command that finishes in a couple of seconds, so there's no
 telling Telegram when it's actually done or needs something, the same
 as replying to one of its own questions.
 
+A tab stuck inside `less`, `vim`, a REPL, or anything else that reads
+keystrokes as its own input rather than a line to submit gets a
+confirmation first instead of a blind send — the text would go to
+whatever's actually running there, not run as a command. A pager not
+on that recognized list still gets caught: a bare `:` as the entire
+last line is the one thing practically every pager agrees on for
+"waiting on you," and a real shell prompt never looks like that.
+
 The result stays a live target: replying to it — "y", "n", anything —
 types straight into that same pane and shows what came back, so
 something like `git add -p`'s hunk-by-hunk prompts works as an actual

data/lib/agents_control/channels/telegram/router.rb

@@ -14,6 +14,18 @@ module AgentsControl
         # they get a separate confirmation.
         REMOTE_COMMANDS = %w[ssh mosh].freeze
 
+        # Programs that read keystrokes as their own input, not as a
+        # line to submit to a shell. Sending a command into one of
+        # these wouldn't run it — it'd feed the letters to whatever's
+        # already there instead: pager navigation, an editor's insert
+        # mode, a REPL evaluating each character-by-character. Confirmed
+        # against a real incident: `git log`'s pager was still open,
+        # "git status" went in as keystrokes, and nothing about it ran
+        # as a command at all.
+        INTERACTIVE_COMMANDS = %w[less more most vim vi nvim emacs nano pico
+                                   man top htop irb pry python python3 node
+                                   mysql psql sqlite3].freeze
+
         # Shared list for the Telegram menu and for /help: a menu that's
         # drifted from reality is worse than no menu. The third element
         # is the argument syntax, needed only in /help; setMyCommands
@@ -196,19 +208,29 @@ module AgentsControl
 
           with_session(chat_id, number) do |session|
             next say(chat_id, "This session has no terminal — nothing to run there.") if session.terminalless?
-
-            if remote?(session)
-              confirm_remote(chat_id, session, command)
+            next confirm_remote(chat_id, session, command) if remote?(session)
+
+            # An agent isn't a shell command that finishes in a couple
+            # of seconds — it's a real task, and hooks already own
+            # telling Telegram when it's actually done or needs
+            # something. Capturing a "result" a moment after typing
+            # would just catch it mid-thought and fall back to dumping
+            # its whole transcript, which isn't a result at all — same
+            # reasoning as replying to an agent's own question
+            # (type_into_session).
+            next execute(chat_id, session, command, show_result: false) if session.agent?
+            next confirm_interactive(chat_id, session, command) if interactive?(session)
+
+            # One capture serves two purposes: it's checked for a
+            # pager's bare `:` prompt right here, and reused as
+            # execute()'s own "before" snapshot if it turns out clean —
+            # capturing twice would desync anything that reads the
+            # screen expecting to see it change between calls.
+            before = capture_screen(session, lines: run_result_lines)
+            if paused_for_input?(before)
+              confirm_interactive(chat_id, session, command)
             else
-              # An agent isn't a shell command that finishes in a
-              # couple of seconds — it's a real task, and hooks already
-              # own telling Telegram when it's actually done or needs
-              # something. Capturing a "result" a moment after typing
-              # would just catch it mid-thought and fall back to
-              # dumping its whole transcript, which isn't a result at
-              # all — same reasoning as replying to an agent's own
-              # question (type_into_session).
-              execute(chat_id, session, command, show_result: !session.agent?)
+              execute(chat_id, session, command, show_result: true, before: before)
             end
           end
         end
@@ -219,6 +241,21 @@ module AgentsControl
           REMOTE_COMMANDS.include?(session.foreground_command.to_s)
         end
 
+        def interactive?(session)
+          INTERACTIVE_COMMANDS.include?(session.foreground_command.to_s)
+        end
+
+        # Backstop for interactive? missing a pager by name — a custom
+        # $PAGER, or anything else not on that list. A bare `:` as the
+        # entire last line is the one thing practically every pager in
+        # the less/more lineage agrees on for "waiting on you"; a real
+        # shell prompt always has more on that line than a single
+        # colon. Takes the screen rather than the session so the
+        # caller's own capture can be reused instead of taking another.
+        def paused_for_input?(screen)
+          screen.to_s.rstrip.lines.last.to_s.strip == ":"
+        end
+
         def confirm_remote(chat_id, session, command)
           key = @store.put({ "action" => "run", "session_id" => session.id, "text" => command },
                            ttl: 300)
@@ -233,6 +270,21 @@ module AgentsControl
               markup: markup)
         end
 
+        def confirm_interactive(chat_id, session, command)
+          key = @store.put({ "action" => "run", "session_id" => session.id, "text" => command },
+                           ttl: 300)
+
+          markup = { inline_keyboard: [[
+            { text: "⚠️ Send anyway", callback_data: key },
+            { text: "cancel", callback_data: @store.put({ "action" => "cancel" }, ttl: 300) }
+          ]] }
+
+          say(chat_id, "#{session.label} is inside #{session.foreground_command} right now, " \
+                       "not at a shell prompt — the text would go to that, not run as a command:\n\n" \
+                       "`#{command}`",
+              markup: markup)
+        end
+
         # Enter is sent as a separate call, not tacked onto the same
         # input: a merged call can fail to send multi-line text at all.
         TYPING_PAUSE = 0.4
@@ -249,9 +301,9 @@ module AgentsControl
 
         def run_result_lines = @config.get("terminal.run_result_lines", 200)
 
-        def execute(chat_id, session, command, show_result: false)
+        def execute(chat_id, session, command, show_result: false, before: nil)
           backend = @registry.backend_for(session)
-          before = show_result ? capture_screen(session, lines: run_result_lines) : nil
+          before ||= capture_screen(session, lines: run_result_lines) if show_result
 
           ok = backend.send_text(session.id, command, newline: false) &&
                sleep(TYPING_PAUSE).then { backend.send_text(session.id, "", newline: true) }

data/lib/agents_control/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AgentsControl
-  VERSION = "0.2.1"
+  VERSION = "0.2.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: agents_control
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.3
 platform: ruby
 authors:
 - Sapar Kurmanov