Add /feedback command for users to send anonymous feedback about CLI (#293)

* Add /feedback command to open anonymous feedback form

This adds a new /feedback command that opens a Google Form in the user's
browser where they can submit anonymous feedback about the CLI.

Changes:
- Add /feedback to COMMANDS list in commands.py
- Add /feedback to show_help function
- Implement _handle_feedback_command in textual_app.py
- Add tests for the new command

Closes #241

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

* Add /feedback to splash page and add test for commands in help

- Update splash page instructions to mention /feedback for anonymous feedback
- Add test to verify all commands from COMMANDS list are included in help text
- Add test to verify /help and /feedback are mentioned in splash instructions

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

---------

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

Author: malhotra5

openhands_cli/tui/content/splash.py

@@ -59,8 +59,8 @@ def get_splash_content(conversation_id: str, *, theme: Theme) -> dict:
             "1. Ask questions, edit files, or run commands.",
             "2. Use @ to look up a file in the folder structure",
             (
-                "3. Type /help for help or / to immediately scroll through "
-                "available commands"
+                "3. Type /help for help, /feedback to leave anonymous feedback, "
+                "or / to scroll through available commands"
             ),
         ],
         "update_notice": None,

openhands_cli/tui/core/commands.py

@@ -16,6 +16,7 @@
     DropdownItem(main="/help - Display available commands"),
     DropdownItem(main="/confirm - Configure confirmation settings"),
     DropdownItem(main="/condense - Condense conversation history"),
+    DropdownItem(main="/feedback - Send anonymous feedback about CLI"),
     DropdownItem(main="/exit - Exit the application"),
 ]
 
@@ -66,6 +67,7 @@ def show_help(main_display: VerticalScroll) -> None:
   [{secondary}]/help[/{secondary}] - Display available commands
   [{secondary}]/confirm[/{secondary}] - Configure confirmation settings
   [{secondary}]/condense[/{secondary}] - Condense conversation history
+  [{secondary}]/feedback[/{secondary}] - Send anonymous feedback about CLI
   [{secondary}]/exit[/{secondary}] - Exit the application
 
 [dim]Tips:[/dim]

openhands_cli/tui/textual_app.py

@@ -405,6 +405,8 @@ def _handle_command(self, command: str) -> None:
             self._handle_confirm_command()
         elif command == "/condense":
             self._handle_condense_command()
+        elif command == "/feedback":
+            self._handle_feedback_command()
         elif command == "/exit":
             self._handle_exit()
         else:
@@ -560,6 +562,18 @@ def _handle_condense_command(self) -> None:
         # This will handle all error cases and notifications
         asyncio.create_task(self.conversation_runner.condense_async())
 
+    def _handle_feedback_command(self) -> None:
+        """Handle the /feedback command to open feedback form in browser."""
+        import webbrowser
+
+        feedback_url = "https://forms.gle/chHc5VdS3wty5DwW6"
+        webbrowser.open(feedback_url)
+        self.notify(
+            title="Feedback",
+            message="Opening feedback form in your browser...",
+            severity="information",
+        )
+
     def _handle_confirmation_request(
         self, pending_actions: list[ActionEvent]
     ) -> UserConfirmation:

tests/tui/test_commands.py

@@ -23,7 +23,7 @@ class TestCommands:
     def test_commands_list_structure(self):
         """Test that COMMANDS list has correct structure."""
         assert isinstance(COMMANDS, list)
-        assert len(COMMANDS) == 4
+        assert len(COMMANDS) == 5
 
         # Check that all items are DropdownItems
         for command in COMMANDS:
@@ -38,6 +38,7 @@ def test_commands_list_structure(self):
             ("/help", "Display available commands"),
             ("/confirm", "Configure confirmation settings"),
             ("/condense", "Condense conversation history"),
+            ("/feedback", "Send anonymous feedback about CLI"),
             ("/exit", "Exit the application"),
         ],
     )
@@ -73,10 +74,12 @@ def test_show_help_function_signature(self):
             "/help",
             "/confirm",
             "/condense",
+            "/feedback",
             "/exit",
             "Display available commands",
             "Configure confirmation settings",
             "Condense conversation history",
+            "Send anonymous feedback about CLI",
             "Exit the application",
             "Tips:",
             "Type / and press Tab",
@@ -144,6 +147,7 @@ def test_show_help_formatting(self):
             ("/help", True),
             ("/confirm", True),
             ("/condense", True),
+            ("/feedback", True),
             ("/exit", True),
             ("/help extra", False),
             ("/exit now", False),
@@ -157,6 +161,36 @@ def test_is_valid_command(self, cmd, expected):
         """Command validation is strict and argument-sensitive."""
         assert is_valid_command(cmd) is expected
 
+    def test_all_commands_included_in_help(self):
+        """Test that all commands from COMMANDS list are included in help text.
+
+        This ensures that when new commands are added to COMMANDS, they are also
+        added to the help text displayed by show_help().
+        """
+        from openhands_cli.tui.core.commands import get_valid_commands
+
+        mock_main_display = mock.MagicMock(spec=VerticalScroll)
+        show_help(mock_main_display)
+
+        # Get the help text that was mounted
+        mock_main_display.mount.assert_called_once()
+        help_widget = mock_main_display.mount.call_args[0][0]
+        help_text = help_widget.content
+
+        # Get all valid commands from COMMANDS list
+        valid_commands = get_valid_commands()
+
+        # Verify each command is present in the help text
+        missing_commands = []
+        for command in valid_commands:
+            if command not in help_text:
+                missing_commands.append(command)
+
+        assert not missing_commands, (
+            f"The following commands are defined in COMMANDS but missing from "
+            f"help text: {missing_commands}"
+        )
+
 
 class TestOpenHandsAppCommands:
     """Integration-style tests for command handling in OpenHandsApp."""
@@ -355,3 +389,40 @@ async def test_condense_command_no_runner_error_message(
                 message="No conversation available to condense",
                 severity="error",
             )
+
+    @pytest.mark.asyncio
+    async def test_feedback_command_opens_browser(
+        self,
+        monkeypatch: pytest.MonkeyPatch,
+    ) -> None:
+        """`/feedback` should open the feedback form URL in the browser."""
+        monkeypatch.setattr(
+            SettingsScreen,
+            "is_initial_setup_required",
+            lambda: False,
+        )
+
+        app = OpenHandsApp(exit_confirmation=False)
+
+        async with app.run_test() as pilot:
+            oh_app = cast(OpenHandsApp, pilot.app)
+
+            # Mock webbrowser.open to verify it's called with correct URL
+            with mock.patch("webbrowser.open") as mock_browser:
+                # Mock notify to verify notification is shown
+                notify_mock = mock.MagicMock()
+                oh_app.notify = notify_mock
+
+                oh_app._handle_command("/feedback")
+
+                # Verify browser was opened with correct URL
+                mock_browser.assert_called_once_with(
+                    "https://forms.gle/chHc5VdS3wty5DwW6"
+                )
+
+                # Verify notification was shown
+                notify_mock.assert_called_once_with(
+                    title="Feedback",
+                    message="Opening feedback form in your browser...",
+                    severity="information",
+                )

tests/tui/test_splash.py

@@ -66,6 +66,9 @@ def test_splash_content_with_conversation_id(self):
                 "2. Use @ to look up a file in the folder structure"
                 in content["instructions"][1]
             )
+            # Verify /help and /feedback are mentioned in instructions
+            assert "/help" in content["instructions"][2]
+            assert "/feedback" in content["instructions"][2]
 
             # Should contain conversation ID
             assert "conversation_text" in content