tinker-agent 1.0.33 → 1.0.35

package/agents.rb

@@ -64,6 +64,13 @@ AGENT_CONFIGS = {
       ║  • DO NOT ask "Would you like me to...", just DO IT                          ║
       ╚══════════════════════════════════════════════════════════════════════════════╝
 
+      ╔══════════════════════════════════════════════════════════════════════════════╗
+      ║  CRITICAL PRIORITY: FINISH WHAT WE START                                     ║
+      ║  • ALWAYS prioritize finishing existing tickets over starting new ones       ║
+      ║  • list_tickets returns high-attempt (rework) tickets FIRST - trust it       ║
+      ║  • If a worker is idle, check for rejected/retried tickets before new work   ║
+      ╚══════════════════════════════════════════════════════════════════════════════╝
+
       ╔══════════════════════════════════════════════════════════════════════════════╗
       ║  CRITICAL WORKFLOW: ASSIGN + MESSAGE (ALWAYS TOGETHER)                       ║
       ║                                                                               ║

package/bin/install-agent.sh

@@ -2,11 +2,14 @@
 set -e
 
 # Install dependencies
+# Handle potential clock skew in containers
+echo 'Acquire::Check-Date "false";' > /etc/apt/apt.conf.d/99no-check-date
+
 apt-get update && apt-get install -y \
     git curl tmux sudo unzip wget
 
 # Install Node.js (required for Claude CLI)
-if ! command -v node &> /dev/null; then
+if ! command -v node &> /dev/null || ! command -v npm &> /dev/null; then
   curl -fsSL https://deb.nodesource.com/setup_20.x | bash -
   apt-get install -y nodejs
 fi
@@ -32,8 +35,12 @@ if getent group ${GROUP_ID} >/dev/null 2>&1; then
   GROUP_NAME=$(getent group ${GROUP_ID} | cut -d: -f1)
 else
   # Create group
-  groupadd -g ${GROUP_ID} ${AGENT_USER}
-  GROUP_NAME=${AGENT_USER}
+  if getent group ${AGENT_USER} >/dev/null 2>&1; then
+      GROUP_NAME=${AGENT_USER}
+  else
+      groupadd -g ${GROUP_ID} ${AGENT_USER}
+      GROUP_NAME=${AGENT_USER}
+  fi
 fi
 
 # 2. Handle User

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tinker-agent",
-  "version": "1.0.33",
+  "version": "1.0.35",
   "description": "Tinker Agent Runner",
   "bin": {
     "tinker-agent": "./run-tinker-agent.rb"

package/run-tinker-agent.rb

@@ -308,13 +308,29 @@ def attach_to_agent(agent_type, config)
   puts "📎 Attaching to #{agent_type} agent..."
   
   # Determine the user to attach as
-  # First try to detect the user the container is running as
-  user = `docker exec #{container_name} whoami 2>/dev/null`.strip
+  # Robust method: find the user running the agent process (tmux or bridge)
+  user = `docker exec #{container_name} ps aux | grep "[a]gent-bridge-tmux" | awk '{print $1}' | head -n 1`.strip
   
   if user.empty?
-    # Fallback: try to match host UID
-    uid = Process.uid
-    user = `docker exec #{container_name} getent passwd #{uid} | cut -d: -f1`.strip
+    user = `docker exec #{container_name} ps aux | grep "[t]mux new-session" | awk '{print $1}' | head -n 1`.strip
+  end
+
+  if user.empty?
+    # Fallback to previous heuristic
+    detected_user = `docker exec #{container_name} whoami 2>/dev/null`.strip
+    if detected_user == "root" || detected_user.empty?
+       uid = Process.uid
+       mapped_user = `docker exec #{container_name} getent passwd #{uid} | cut -d: -f1`.strip
+       user = mapped_user unless mapped_user.empty?
+    else
+       user = detected_user
+    end
+  end
+  
+  if user.empty?
+    # Final Fallback
+    user = "rails"
+    puts "⚠️  Could not detect agent user, defaulting to '#{user}'"
   end
   
   if user.empty?