Skip to content

Conversation

@xingyaoww
Copy link
Contributor

  • I have read and reviewed the documentation changes to the best of my ability.
  • If the change is significant, I have run the documentation site locally and confirmed it renders as expected.

Summary of changes

This PR improves the documentation for the critic feature by adding clear step-by-step instructions for users who want to disable it.

Changes

Added navigation instructions to the "Disabling the Critic" section:

  1. Open the command palette with Ctrl+P
  2. Select Settings
  3. Navigate to the CLI Settings tab
  4. Toggle off Enable Critic (Experimental)

Context

This addresses user feedback from issue OpenHands/OpenHands#8963 where users noted that instructions to opt-out of the critic feature were not very visible in the CLI or documentation.

References

@xingyaoww can click here to continue refining the PR

Add clear navigation instructions for users to disable the critic feature:
1. Open command palette with Ctrl+P
2. Select Settings
3. Navigate to CLI Settings tab
4. Toggle off Enable Critic (Experimental)

This improves discoverability of the opt-out option for the experimental
critic feature.

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

openhands-ai bot commented Jan 27, 2026

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • .github/workflows/sync-docs-code-blocks.yml
    • .github/workflows/sync-agent-sdk-openapi.yml

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #279 at branch `openhands/improve-critic-docs`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

@xingyaoww xingyaoww merged commit cd62953 into main Jan 28, 2026
4 checks passed
@xingyaoww xingyaoww deleted the openhands/improve-critic-docs branch January 28, 2026 02:45
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