Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

fix(ambient): remove redundant 'Await instructions' message#288

Merged
jsell-rh merged 2 commits intojsell-rh:mainfrom
tiwillia-ai-bot:ambient/session-1774274223
Mar 23, 2026
Merged

fix(ambient): remove redundant 'Await instructions' message#288
jsell-rh merged 2 commits intojsell-rh:mainfrom
tiwillia-ai-bot:ambient/session-1774274223

Conversation

@tiwillia-ai-bot
Copy link
Copy Markdown

Summary

  • Removes the redundant "You are an agent. Await instructions." message that was sent as initialPrompt when opts.Command was empty
  • This message is unnecessary because the workflow's systemPrompt and agent ignition protocol already provide all necessary context and instructions

Changes

  • Modified session_backend_ambient.go:143-148 to set initialPrompt directly to opts.Command without the fallback message
  • The initialPrompt may now be empty if the workflow handles initialization entirely via its system/startup prompts

Test plan

  • Code compiles successfully
  • Manual test: spawn ambient agent session and verify it receives workflow instructions correctly
  • Existing ambient backend tests pass

Closes TASK-005

🤖 Generated with Claude Code

Addresses confusion between interrupt, stop, kill, and delete operations
in the ambient workflow guidance.

Changes:
- protocol.md: Added "Agent Lifecycle Operations" section with clear
  definitions and distinctions between each operation
- mcp_tools.go: Updated stop_agent, spawn_agent, and restart_agent tool
  descriptions to clarify behavior and data preservation
- api-reference.md: Added lifecycle operations summary and expanded
  endpoint descriptions to clarify what is preserved vs deleted
- AgentDetail.vue: Updated tooltips and dialog text to reinforce that
  kill/stop/restart preserve agent records while delete removes all data

Key clarifications:
- Interrupt = pause current task (agent keeps running)
- Stop/Kill = end session cleanly (preserves agent record, can restart)
- Delete = permanent removal (all data deleted, cannot restart)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@tiwillia-ai-bot tiwillia-ai-bot force-pushed the ambient/session-1774274223 branch 3 times, most recently from 19558ed to df58a09 Compare March 23, 2026 14:10
@jsell-rh jsell-rh enabled auto-merge (squash) March 23, 2026 15:07
Resolved conflict in AgentDetail.vue by accepting main's dropdown menu
approach for lifecycle actions. This supersedes the flex-wrap grouping
from PR jsell-rh#288 while preserving all flex-wrap additions on other rows.

Main's dropdown menu is a better UX solution for mobile/overflow issues
and already includes all flex-wrap improvements from frontend-ui.

Changes merged from main:
- AgentDetail.vue: Dropdown menu for lifecycle actions
- AgentMessages.vue: Session active warning and input disabling
auto-merge was automatically disabled March 23, 2026 15:14

Head branch was pushed to by a user without write access

@jsell-rh jsell-rh enabled auto-merge (squash) March 23, 2026 17:53
@jsell-rh jsell-rh merged commit 777a033 into jsell-rh:main Mar 23, 2026
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants