Skip to content

Conversation

@xingyaoww
Copy link
Contributor

Summary

This PR updates the ACP documentation to reflect the newly implemented confirmation mode and slash command features.

Changes

Updated Feature List

  • Marked "Confirmation Mode" and "Slash Commands" as ✅ Implemented (previously listed as 🚧 Coming Soon)
  • Updated Prompt Turns description to mention permission support

New Section: Confirmation Modes

Added comprehensive documentation for three confirmation modes:

  1. Always Ask (Default): User confirms each action

    openhands acp  # defaults to always-ask mode
  2. Always Approve: Auto-approve all actions without confirmation

    openhands acp --always-approve
  3. LLM-Based Approval: Intelligent risk-based approval

    openhands acp --llm-approve

Slash Commands

Documented slash commands for changing modes during a session:

  • /confirm always-ask - Switch to always-ask mode
  • /confirm always-approve - Switch to always-approve mode
  • /confirm llm-approve - Switch to LLM-based approval mode
  • /help - Show all available slash commands

Zed IDE Configuration

Added a tip showing how to configure Zed with a specific confirmation mode by default:

{
  "agent_servers": {
    "OpenHands": {
      "command": "uvx",
      "args": ["openhands", "acp", "--always-approve"],
      "env": {}
    }
  }
}

Related PRs

Testing

  • Verified markdown syntax is correct
  • Checked that all code examples are properly formatted
  • Confirmed section ordering makes sense

Co-authored-by: openhands openhands@all-hands.dev

@xingyaoww can click here to continue refining the PR

- Update feature list to mark Confirmation Mode and Slash Commands as implemented
- Add comprehensive Confirmation Modes section explaining three modes:
  - Always Ask (default): user confirms each action
  - Always Approve: auto-approve all actions
  - LLM-Based Approval: intelligent risk-based approval
- Document CLI flags: --always-approve and --llm-approve
- Explain slash commands for changing modes during session (/confirm, /help)
- Add Zed IDE configuration example with mode flags
- Note that mode persists for session duration

Co-authored-by: openhands <openhands@all-hands.dev>
@xingyaoww
Copy link
Contributor Author

@OpenHands merge from main an dresolve the conflict. Did we document this already?

@openhands-ai
Copy link

openhands-ai bot commented Feb 10, 2026

I'm on it! xingyaoww can track my progress at all-hands.dev

- Resolved conflict: old acp.mdx was deleted in main (docs reorganized)
- Added detailed Confirmation Modes section to ide/overview.mdx
- Added slash commands documentation for changing modes during session

Co-authored-by: openhands <openhands@all-hands.dev>
@openhands-ai
Copy link

openhands-ai bot commented Feb 10, 2026

Summary

I've merged main into the branch and resolved the conflict. Here's what happened:

Conflict Resolution

The old openhands/usage/run-openhands/acp.mdx file (which this PR modified) was deleted in main because the documentation was reorganized. The ACP documentation is now split across multiple files in the new openhands/usage/cli/ structure.

What Was Already Documented

The command-line options (--always-approve, --llm-approve) were already documented in the new structure in:

  • openhands/usage/cli/ide/overview.mdx - ACP options table
  • openhands/usage/cli/ide/zed.mdx - Zed configuration examples
  • openhands/usage/cli/command-reference.mdx - Full command reference

What I Added

I added the detailed Confirmation Modes section to openhands/usage/cli/ide/overview.mdx which includes:

  1. Always Ask (Default) - Explanation of the default confirmation behavior
  2. Always Approve - Explanation of auto-approve mode
  3. LLM-Based Approval - Explanation of intelligent risk-based approval
  4. Changing Modes During a Session - Slash commands table (/confirm always-ask, /confirm always-approve, /confirm llm-approve, /help)

This content was from the original PR but wasn't present in the new documentation structure.

Changes Pushed

The merge commit has been pushed to the branch. The PR is now up-to-date with main and includes the confirmation mode documentation in the appropriate new location.

View full conversation

@xingyaoww xingyaoww marked this pull request as ready for review February 10, 2026 10:45
@xingyaoww xingyaoww requested a review from mamoodi as a code owner February 10, 2026 10:45
@xingyaoww xingyaoww requested review from enyst and malhotra5 February 10, 2026 10:46
@xingyaoww xingyaoww enabled auto-merge (squash) February 10, 2026 10:46
@xingyaoww
Copy link
Contributor Author

(realized that we forget to merge this PR)

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.

2 participants