fix: improve context menu behavior and revert tool descriptions

Author: DLhugly

clif-pad-ide/src-tauri/Cargo.lock

@@ -56,7 +56,7 @@ fn tool_definitions() -> Vec<serde_json::Value> {
             "type": "function",
             "function": {
                 "name": "read_file",
-                "description": "Read the contents of a file. Read files before editing them. Use this to inspect code, configs, and logs before making changes. For large files, use offset and limit to read specific line ranges instead of the whole file. Output includes line numbers. Prefer this over run_command for reading files.",
+                "description": "Read the contents of a file. Read files before editing them. Use this to inspect code, configs, and logs before making changes. For large files, use offset and limit to read specific line ranges instead of the whole file.",
                 "strict": true,
                 "parameters": {
                     "type": "object",
@@ -91,7 +91,7 @@ fn tool_definitions() -> Vec<serde_json::Value> {
             "type": "function",
             "function": {
                 "name": "edit_file",
-                "description": "Make a targeted edit by replacing old_string with new_string exactly once. Read the file first. Prefer this over write_file for localized changes. old_string must match the current file contents exactly, including whitespace. Include enough context to make the match unique.",
+                "description": "Make a targeted edit by replacing old_string with new_string exactly once. Read the file first. Prefer this over write_file for localized changes. old_string must match the current file contents exactly, including whitespace.",
                 "strict": true,
                 "parameters": {
                     "type": "object",
@@ -109,7 +109,7 @@ fn tool_definitions() -> Vec<serde_json::Value> {
             "type": "function",
             "function": {
                 "name": "list_files",
-                "description": "List files and directories at a path. Use this to explore the workspace before reading or editing files. Returns names and types (file/directory).",
+                "description": "List files and directories at a path. Use this to explore the workspace before reading or editing files.",
                 "strict": true,
                 "parameters": {
                     "type": "object",
@@ -125,7 +125,7 @@ fn tool_definitions() -> Vec<serde_json::Value> {
             "type": "function",
             "function": {
                 "name": "search",
-                "description": "Search for text in files within a directory. Prefer this over run_command for codebase exploration. Returns file paths and matching lines with context.",
+                "description": "Search for text in files within a directory. Prefer this over run_command for codebase exploration.",
                 "strict": true,
                 "parameters": {
                     "type": "object",
@@ -142,7 +142,7 @@ fn tool_definitions() -> Vec<serde_json::Value> {
             "type": "function",
             "function": {
                 "name": "run_command",
-                "description": "Run a shell command and return its output. Use this mainly for build, test, lint, git, or validation tasks. Prefer read/search/list tools over shell commands for basic exploration. Requires user approval before execution. Avoid for reading files - use read_file instead.",
+                "description": "Run a shell command and return its output. Use this mainly for build, test, lint, git, or validation tasks. Prefer read/search/list tools over shell commands for basic exploration. Requires user approval before execution.",
                 "strict": true,
                 "parameters": {
                     "type": "object",
@@ -159,7 +159,7 @@ fn tool_definitions() -> Vec<serde_json::Value> {
             "type": "function",
             "function": {
                 "name": "find_file",
-                "description": "Find files or directories by partial name anywhere in the workspace. Use this when you do not know the exact location of something. Faster than run_command find.",
+                "description": "Find files or directories by partial name anywhere in the workspace. Use this when you do not know the exact location of something.",
                 "strict": true,
                 "parameters": {
                     "type": "object",

clif-pad-ide/src-tauri/src/commands/agent.rs

@@ -1,4 +1,5 @@
 import { Component, For, Show, onMount, onCleanup, createSignal } from "solid-js";
+import { Portal } from "solid-js/web";
 
 export interface ContextMenuItem {
   label: string;
@@ -55,22 +56,23 @@ const ContextMenu: Component<ContextMenuProps> = (props) => {
   });
 
   return (
-    <div
-      ref={menuRef}
-      class="fixed z-50"
-      style={{
-        left: `${position().x}px`,
-        top: `${position().y}px`,
-        background: "var(--bg-surface)",
-        border: "1px solid var(--border-default)",
-        "border-radius": "6px",
-        "box-shadow": "0 4px 16px rgba(0,0,0,0.3)",
-        "min-width": "160px",
-        padding: "4px 0",
-        "font-size": "12px",
-        "font-family": "var(--font-sans)",
-      }}
-    >
+    <Portal>
+      <div
+        ref={menuRef}
+        class="fixed z-50"
+        style={{
+          left: `${position().x}px`,
+          top: `${position().y}px`,
+          background: "var(--bg-surface)",
+          border: "1px solid var(--border-default)",
+          "border-radius": "6px",
+          "box-shadow": "0 4px 16px rgba(0,0,0,0.3)",
+          "min-width": "160px",
+          padding: "4px 0",
+          "font-size": "12px",
+          "font-family": "var(--font-sans)",
+        }}
+      >
       <For each={props.items}>
         {(item) => (
           <>
@@ -119,7 +121,8 @@ const ContextMenu: Component<ContextMenuProps> = (props) => {
           </>
         )}
       </For>
-    </div>
+      </div>
+    </Portal>
   );
 };
 

clif-pad-ide/src/components/explorer/ContextMenu.tsx

@@ -158,8 +158,8 @@ const FileRow: Component<{
           color: "var(--text-primary)",
           background: hovered() ? "var(--bg-hover)" : "transparent",
         }}
-        onMouseEnter={() => setHovered(true)}
-        onMouseLeave={() => setHovered(false)}
+        onMouseEnter={() => !ctxMenu() && setHovered(true)}
+        onMouseLeave={() => !ctxMenu() && setHovered(false)}
         onContextMenu={handleContextMenu}
         onClick={() => {
           if (isRenaming()) return;