update: simpify

TabbyML · Dec 23, 2024 · be2faf1 · be2faf1
1 parent dd8608d
commit be2faf1
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 12 deletions.
diff --git a/ee/tabby-ui/app/files/components/chat-side-bar.tsx b/ee/tabby-ui/app/files/components/chat-side-bar.tsx
@@ -30,9 +30,7 @@ export const ChatSideBar: React.FC<ChatSideBarProps> = ({
   const repoMapRef = useLatest(repoMap)
   const onNavigate = async (context: Context) => {
     if (context?.filepath && context?.git_url) {
-      const lineHash = context.range
-        ? undefined
-        : formatLineHashForCodeBrowser(context?.range)
+      const lineHash = formatLineHashForCodeBrowser(context?.range)
       const repoMap = repoMapRef.current
       const matchedRepositoryKey = find(
         Object.keys(repoMap),

diff --git a/ee/tabby-ui/app/search/components/assistant-message-section.tsx b/ee/tabby-ui/app/search/components/assistant-message-section.tsx
@@ -207,12 +207,7 @@ export function AssistantMessageSection({
     searchParams.append('redirect_git_url', ctx.git_url)
     url.search = searchParams.toString()
 
-    const lineHash = ctx.range
-      ? formatLineHashForCodeBrowser({
-          start: ctx.range.start,
-          end: ctx.range.end
-        })
-      : undefined
+    const lineHash = formatLineHashForCodeBrowser(ctx.range)
     if (lineHash) {
       url.hash = lineHash
     }
@@ -240,7 +235,7 @@ export function AssistantMessageSection({
     searchParams.append('redirect_git_url', code.gitUrl)
     url.search = searchParams.toString()
 
-    const lineHash = range ? formatLineHashForCodeBrowser(range) : undefined
+    const lineHash = formatLineHashForCodeBrowser(range)
     if (lineHash) {
       url.hash = lineHash
     }

diff --git a/ee/tabby-ui/lib/utils/index.ts b/ee/tabby-ui/lib/utils/index.ts
@@ -112,7 +112,7 @@ export function getRangeFromAttachmentCode(code: {
   startLine?: Maybe<number>
   content: string
 }): LineRange | undefined {
-  if (!code?.startLine || typeof code.startLine !== 'number') return undefined
+  if (!code?.startLine) return undefined
 
   const start = code.startLine
   const lineCount = code.content.split('\n').length
@@ -126,7 +126,7 @@ export function getRangeFromAttachmentCode(code: {
 
 export function getRangeTextFromAttachmentCode(code: AttachmentCodeItem) {
   const range = getRangeFromAttachmentCode(code)
-  return range ? formatLineHashForCodeBrowser(range) : ''
+  return formatLineHashForCodeBrowser(range)
 }
 
 export function getContent(item: AttachmentDocItem) {