Skip to content

Fix VM instance sharing across tasks - #6

Merged
teknium1 merged 2 commits into
mainfrom
fix-leakage
Nov 4, 2025
Merged

Fix VM instance sharing across tasks#6
teknium1 merged 2 commits into
mainfrom
fix-leakage

Conversation

@hjc-puro

@hjc-puro hjc-puro commented Nov 3, 2025

Copy link
Copy Markdown
Contributor
  • Isolates each VM to a task ID
  • Guarantees VMs will live for at most 20 minutes

Comment thread run_agent.py

# Clean up VM for this task after conversation completes
try:
cleanup_vm(effective_task_id)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@teknium1 this part is a bit hacky - I can take it out if you're ok with instances running for ~5 mins after convo ends

@hjc-puro
hjc-puro requested a review from teknium1 November 4, 2025 08:37
@teknium1
teknium1 merged commit 9573b2a into main Nov 4, 2025
h4x3rotab referenced this pull request in Clawdi-AI/hermes-agent Apr 10, 2026
h4x3rotab referenced this pull request in Clawdi-AI/hermes-agent Apr 10, 2026
#3 Cost Analytics Dashboard
- New Analytics tab with summary cards (total tokens, cost, avg/mission, today, week)
- CSS bar charts: cost by agent, cost by model, daily timeline (7d)
- No external chart libraries — pure Tailwind

#4 Export Mission as Markdown
- Download .md file with full mission report (goal, team, transcript, artifacts)
- Copy to clipboard button with visual feedback
- Wired into Mission Detail Overlay

#5 Word-by-word Streaming in Agent Chat
- Replaced polling with SSE EventSource in AgentChatPanel
- Real-time chunk streaming with fallback to polling on error
- Streaming assistant message updates in-place

#6 Remote Agents Panel
- Fetches external sessions from gateway /api/sessions
- Filters out local agent sessions — shows only remote/external
- Auto-polls every 15s, card layout with status, model, tokens, cost
- Open Chat links to ClawSuite chat tab

#7 Real-time Collaboration (Presence)
- BroadcastChannel-based cross-tab presence detection
- Shows colored avatars of other users viewing Agent Hub
- Heartbeat every 5s, stale cleanup at 30s
- Shows which tab each peer is viewing
Vex-Dravex added a commit to Vex-Dravex/hermes-agent that referenced this pull request Apr 10, 2026
Vex-Dravex added a commit to Vex-Dravex/hermes-agent that referenced this pull request Apr 11, 2026
malaiwah pushed a commit to malaiwah/hermes-agent that referenced this pull request Apr 11, 2026
…lity + make configurable' (NousResearch#6) from fix/pids-limit-cgroup-probe into main
gary-the-ai pushed a commit to gary-the-ai/hermes-web-console-gui that referenced this pull request Apr 11, 2026
…t, stop/undo honesty, json_error crash, codex validation, deep-link race

Bug #1: ChatPage loadSession reads res.items (not res.transcript) to match backend
Bug NousResearch#2: Add GET /api/gui/session-search backed by SessionDB.search_messages (FTS5)
Bug NousResearch#3: Stop button now checks res.supported before claiming run was stopped
Bug NousResearch#4: Undo button now checks res.supported before removing messages locally
Bug NousResearch#5: Fix _json_error positional calls in handle_chat_compress (was crashing 500)
Bug NousResearch#6: Codex provider validation now also guards switching TO openai-codex
Bug NousResearch#7: Deep-link hash check runs before health callback to prevent race condition
@Aecroo

Aecroo commented Apr 12, 2026

Copy link
Copy Markdown

The fix has been merged into main and verified. host.get now correctly uses selectParentTemplates to retrieve templates for hosts.

malaiwah pushed a commit to malaiwah/hermes-agent that referenced this pull request Apr 13, 2026
- connection.py: cap header read at 8KB to prevent DoS from malicious handler
- handler.py: use .find() instead of `in` + .index() to eliminate race in patch
- handler.py: add truncated field to execute response when output exceeds 50KB
- server.py: include error data field in formatted error messages
- test: add timeout to test client recv, handle TimeoutExpired in close

Fixes issues NousResearch#1, NousResearch#4, NousResearch#5, NousResearch#6, NousResearch#8, NousResearch#10 from Qwen 3.5 peer review on PR NousResearch#19.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants