Refactor the @ sign tooltip logic to only show on hover when no context is available, and standardize diff length check in commit handling

Author: robertpiosik

packages/ui/src/components/editor/panel/ChatInput/ChatInput.tsx

@@ -244,15 +244,6 @@ export const ChatInput: React.FC<ChatInputProps> = (props) => {
       document.removeEventListener('selectionchange', on_selection_change)
   }, [props.on_caret_position_change, props.value, props.context_file_paths])
 
-  useEffect(() => {
-    if (show_at_sign_tooltip) {
-      const timer = setTimeout(() => {
-        set_show_at_sign_tooltip(false)
-      }, 2000)
-      return () => clearTimeout(timer)
-    }
-  }, [show_at_sign_tooltip])
-
   const placeholder = useMemo(() => {
     const active_history = props.chat_history
 
@@ -493,10 +484,13 @@ export const ChatInput: React.FC<ChatInputProps> = (props) => {
             }}
           >
             <button
-              onClick={() => {
+              onMouseEnter={() => {
                 if (has_no_context) {
                   set_show_at_sign_tooltip(true)
-                } else {
+                }
+              }}
+              onClick={() => {
+                if (!has_no_context) {
                   props.on_at_sign_click()
                 }
               }}

packages/vscode/src/views/panel/backend/message-handlers/handle-commit-changes/handle-commit-changes.ts

@@ -26,7 +26,7 @@ async function proceed_with_commit_generation(
       cwd: repository.rootUri.fsPath
     }).toString()
 
-    if (!diff || diff.length === 0) {
+    if (!diff || diff.length == 0) {
       vscode.window.showInformationMessage(
         dictionary.information_message.NO_CHANGES_TO_COMMIT
       )