Skip to content

Conversation

@neubig
Copy link
Contributor

@neubig neubig commented Dec 26, 2025

Summary

This PR simplifies the JetBrains ACP configuration instructions and updates the version requirement.

Changes

  1. Simplified configuration: Users can now use "openhands" directly in the command field instead of needing to find and specify the full path to the executable
  2. Updated version requirement: Changed IntelliJ version requirement from 2024.3 to 25.3

Before

The instructions required users to:

  • Run which openhands to find the full path
  • Copy that path into the configuration
  • Included an example showing /Users/username/.local/bin/openhands

After

The instructions now simply use "command": "openhands" directly, reducing the setup steps from 4 to 3.

@neubig can click here to continue refining the PR

- Use 'openhands' directly instead of requiring full path
- Update IntelliJ version requirement from 2024.3 to 25.3

Co-authored-by: openhands <openhands@all-hands.dev>
@neubig neubig marked this pull request as ready for review December 26, 2025 22:21
@neubig neubig requested a review from mamoodi as a code owner December 26, 2025 22:21
@neubig
Copy link
Contributor Author

neubig commented Jan 6, 2026

@OpenHands fix the merge conflicts

@openhands-ai
Copy link

openhands-ai bot commented Jan 6, 2026

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

- Resolve merge conflict: acp.mdx was deleted in main and moved to cli/ide/jetbrains.mdx
- Apply PR changes to new jetbrains.mdx location:
  - Simplify configuration to use 'openhands' directly instead of full path
  - Update version requirement from 2024.3 to 25.3
  - Reduce setup steps from 4 to 3

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

openhands-ai bot commented Jan 6, 2026

Summary

I've successfully resolved the merge conflicts in PR #213. Here's what was done:

Conflict Resolution

The merge conflict occurred because:

  • The PR branch modified openhands/usage/run-openhands/acp.mdx to simplify JetBrains configuration
  • The main branch deleted that file and reorganized the documentation, moving JetBrains content to openhands/usage/cli/ide/jetbrains.mdx

Changes Applied

  1. ✅ Removed the deleted acp.mdx file (accepting main's deletion)
  2. ✅ Applied the PR's simplifications to the new jetbrains.mdx location:
    • Simplified configuration: Changed from requiring full path (/Users/username/.local/bin/openhands) to just "openhands" directly
    • Updated version requirement: Changed from 2024.3 to 25.3
    • Reduced setup steps: From 4 steps to 3 steps (removed "Find the OpenHands Executable Path" step)
    • Updated all examples: All JSON configuration examples now use "command": "openhands" instead of full paths
    • Updated troubleshooting: Simplified the troubleshooting section to match the new simpler configuration

Verification

  • ✅ All merge conflicts resolved
  • ✅ Changes are concise and focused only on the PR's original intent
  • ✅ Changes pushed to simplify-jetbrains-acp-config branch

The PR is now ready for review with the merge conflicts resolved.

View full conversation

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