update

TabbyML · May 30, 2024 · 74e8f8a · 74e8f8a
1 parent 583c163
commit 74e8f8a
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 6 deletions.
diff --git a/clients/vscode/src/chat/ChatViewProvider.ts b/clients/vscode/src/chat/ChatViewProvider.ts
@@ -35,7 +35,12 @@ export class ChatViewProvider implements WebviewViewProvider {
     }
 
     this.client = createClient(webviewView, {
-      navigate: async () => {},
+      navigate: async (context: Context) => {
+        if (context?.filepath && context?.git_url) {
+          const url = `${context.git_url}/blob/main/${context.filepath}#L${context.range.start}-L${context.range.end}`;
+          await env.openExternal(Uri.parse(url));
+        }
+      },
     });
 
     this.agent.on("didChangeStatus", async (status) => {

diff --git a/ee/tabby-ui/app/chat/page.tsx b/ee/tabby-ui/app/chat/page.tsx
@@ -159,6 +159,7 @@ export default function ChatPage() {
       onLoaded={onChatLoaded}
       maxWidth={maxWidth}
       onCopyContent={from === 'vscode' ? onCopyContent : undefined}
+      isReferenceClickable={from !== 'vscode'}
     />
   )
 }
diff --git a/ee/tabby-ui/components/chat/chat.tsx b/ee/tabby-ui/components/chat/chat.tsx
@@ -34,6 +34,7 @@ type ChatContextValue = {
   onClearMessages: () => void
   container?: HTMLDivElement
   onCopyContent?: (value: string) => void
+  isReferenceClickable: boolean
 }
 
 export const ChatContext = React.createContext<ChatContextValue>(
@@ -118,6 +119,7 @@ interface ChatProps extends React.ComponentProps<'div'> {
   welcomeMessage?: string
   promptFormClassname?: string
   onCopyContent?: (value: string) => void
+  isReferenceClickable?: boolean
 }
 
 function ChatRenderer(
@@ -137,7 +139,8 @@ function ChatRenderer(
     maxWidth,
     welcomeMessage,
     promptFormClassname,
-    onCopyContent
+    onCopyContent,
+    isReferenceClickable = true
   }: ChatProps,
   ref: React.ForwardedRef<ChatRef>
 ) {
@@ -384,7 +387,8 @@ function ChatRenderer(
         handleMessageAction,
         onClearMessages,
         container,
-        onCopyContent
+        onCopyContent,
+        isReferenceClickable
       }}
     >
       <div className="flex justify-center overflow-x-hidden">

diff --git a/ee/tabby-ui/components/chat/question-answer.tsx b/ee/tabby-ui/components/chat/question-answer.tsx
@@ -391,7 +391,8 @@ interface ContextReferencesProps {
   contexts: Context[]
 }
 const CodeReferences = ({ contexts }: ContextReferencesProps) => {
-  const { onNavigateToContext } = React.useContext(ChatContext)
+  const { onNavigateToContext, isReferenceClickable } =
+    React.useContext(ChatContext)
   const isMultipleReferences = contexts?.length > 1
 
   if (!contexts?.length) return null
@@ -421,9 +422,14 @@ const CodeReferences = ({ contexts }: ContextReferencesProps) => {
               .join('/')
             return (
               <div
-                className="cursor-pointer rounded-md border p-2 hover:bg-accent"
+                className={cn('rounded-md border p-2 hover:bg-accent', {
+                  'cursor-pointer': isReferenceClickable,
+                  'cursor-default pointer-events-auto': !isReferenceClickable
+                })}
                 key={index}
-                onClick={e => onNavigateToContext?.(item)}
+                onClick={e =>
+                  isReferenceClickable && onNavigateToContext?.(item)
+                }
               >
                 <div className="flex items-center gap-1 overflow-x-auto">
                   <IconFile className="shrink-0" />