feat: improvements to find and replace

[sestinj]

Description

improvements to find and replace

---

Summary by cubic

Improves the Find and Replace experience with inline diff stats, clearer messages, and a cleaner tool call header. Simplifies tool copy, always shows a scrollbar, and adds model info to tool call telemetry.

• UI Improvements

  • Find & Replace: show +/− line counts in the header, left-align the ellipsis, and display a friendly “string not found” message.
  • Always show a thin scrollbar; removed the “Show Chat Scrollbar” setting.
  • Simplified tool call header: removed args and truncate-history buttons; shorter status text (“will”, “wants to”).
  • Shortened tool messages across built-ins (Create File, Read URL/File/Range/Current, Glob, Grep, ls, Find and Replace).

• Telemetry

  • Include the selected model in PostHog events for tool call decisions (auto-accept/accept/reject) and outcomes.

[cubic-dev-ai]

2 issues found across 17 files

_{React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.}

[cubic-dev-ai]

Hardcoding showScrollbar to true ignores the ui.showChatScrollbar configuration and forces the thin-scrollbar UI regardless of user settings.

Prompt for AI agents

Address the following comment on gui/src/pages/gui/Chat.tsx at line 382:

<comment>Hardcoding showScrollbar to true ignores the ui.showChatScrollbar configuration and forces the thin-scrollbar UI regardless of user settings.</comment>

<file context>
@@ -381,7 +379,7 @@ export function Chat() {
   );
 
-  const showScrollbar = showChatScrollbar ?? window.innerHeight &gt; 5000;
+  const showScrollbar = true;
 
   return (
</file context>

---

✅ Addressed in fcbe5bc

[cubic-dev-ai]

Telemetry event may include an undefined model because the null-check happens after this capture; consider providing a fallback value or moving the check earlier.

Prompt for AI agents

Address the following comment on gui/src/redux/thunks/callToolById.ts at line 49:

<comment>Telemetry event may include an undefined model because the null-check happens after this capture; consider providing a fallback value or moving the check earlier.</comment>

<file context>
@@ -43,14 +43,15 @@ export const callToolById = createAsyncThunk&lt;
+  const selectedChatModel = selectSelectedChatModel(state);
+
   posthog.capture(&quot;gui_tool_call_decision&quot;, {
+    model: selectedChatModel,
     decision: isAutoApproved ? &quot;auto_accept&quot; : &quot;accept&quot;,
     toolName: toolCallState.toolCall.function.name,
</file context>

Suggested change

	model: selectedChatModel,
	model: selectedChatModel ?? "unknown",

[sestinj]

@continuedev does this make sense?

[github-actions]

I've started a remote session to help with your request:

@continuedev does this make sense?

[RomneyDa]

@sestinj any specific reason for removing chat scrollbar?
https://github.com/continuedev/continue/issues?q=is%3Aissue%20chat%20scrollbar

[RomneyDa]

@sestinj any specific reason for removing chat scrollbar? https://github.com/continuedev/continue/issues?q=is%3Aissue%20chat%20scrollbar

personally I find the scrollbar pretty ugly/jarring

[sestinj]

Avg chat is longer these days with agent mode it seems, and I've found myself wanting it more often. Doesn't need to change though if there's a strong opinions the other way

[sestinj]

🎉 This PR is included in version 1.14.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀