Merge branch 'main' into michael/sou-220-bug-unable-to-clone-public-gitlab-repos-from-gitlabcom-with

msukkari · web-flow · commit d6cc93423885 · 2026-01-15T20:20:04.000-08:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,9 +7,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+- Added dialog when no authentication provider is configured. [#744](https://github.com/sourcebot-dev/sourcebot/pull/744)
+
+### Fixed
+- Fixed "Invalid line number XXX in 21-line document" error when a invalid highlight range is passed to the file viewer. [#745](https://github.com/sourcebot-dev/sourcebot/pull/745)
+- Fixed visual nit where ask sb search scope selector would render long repository names poorly. [#747](https://github.com/sourcebot-dev/sourcebot/pull/747)
+- Fixed visual nit where long search previews in ask sb would take up a lot of space. [#747](https://github.com/sourcebot-dev/sourcebot/pull/747)
+
+## [4.10.11] - 2026-01-16
+
 ### Fixed
 - Fixed issue with filtering on generic git repo indexed from http/https [#742](https://github.com/sourcebot-dev/sourcebot/pull/742)
 - Fixed auth error when trying to sync public gitlab.com project with no token [#748](https://github.com/sourcebot-dev/sourcebot/pull/748)
+- Fixed "Ranges must be added sorted by `from` position and `startSide`" error when browsing certain files. [#743](https://github.com/sourcebot-dev/sourcebot/pull/743)
 
 ## [4.10.10] - 2026-01-16
 
diff --git a/packages/shared/src/version.ts b/packages/shared/src/version.ts
@@ -1,2 +1,2 @@
 // This file is auto-generated by .github/workflows/release-sourcebot.yml
-export const SOURCEBOT_VERSION = "v4.10.10";
+export const SOURCEBOT_VERSION = "v4.10.11";
diff --git a/packages/web/src/app/[domain]/browse/[...path]/components/pureCodePreviewPanel.tsx b/packages/web/src/app/[domain]/browse/[...path]/components/pureCodePreviewPanel.tsx
@@ -114,6 +114,11 @@ export const PureCodePreviewPanel = ({
         const doc = editorRef.state.doc;
         const { start, end } = highlightRange;
 
+        if (start.lineNumber > doc.lines || end.lineNumber > doc.lines) {
+            console.warn(`Highlight range is out of bounds: start.lineNumber=${start.lineNumber}, end.lineNumber=${end.lineNumber}, doc.lines=${doc.lines}`);
+            return;
+        }
+
         const from = doc.line(start.lineNumber).from;
         const to = doc.line(end.lineNumber).to;
         const selection = EditorSelection.range(from, to);
diff --git a/packages/web/src/app/[domain]/browse/[...path]/components/rangeHighlightingExtension.ts b/packages/web/src/app/[domain]/browse/[...path]/components/rangeHighlightingExtension.ts
@@ -16,6 +16,11 @@ export const rangeHighlightingExtension = (range: BrowseHighlightRange) => State
     create(state) {
         const { start, end } = range;
 
+        if (start.lineNumber > state.doc.lines || end.lineNumber > state.doc.lines) {
+            console.warn(`Highlight range is out of bounds: start.lineNumber=${start.lineNumber}, end.lineNumber=${end.lineNumber}, doc.lines=${state.doc.lines}`);
+            return Decoration.none;
+        }
+
         if ('column' in start && 'column' in end) {
             const from = state.doc.line(start.lineNumber).from + start.column - 1;
             const to = state.doc.line(end.lineNumber).from + end.column - 1;
@@ -25,14 +30,14 @@ export const rangeHighlightingExtension = (range: BrowseHighlightRange) => State
                 decorations.push(markDecoration.range(from, to));
             }
 
-            return Decoration.set(decorations);
+            return Decoration.set(decorations, /* sort = */ true);
         } else {
             const decorations: Range<Decoration>[] = [];
             for (let line = start.lineNumber; line <= end.lineNumber; line++) {
                 decorations.push(lineDecoration.range(state.doc.line(line).from));
             }
 
-            return Decoration.set(decorations);
+            return Decoration.set(decorations, /* sort = */ true);
         }
     },
     update(deco, tr) {
diff --git a/packages/web/src/app/components/authMethodSelector.tsx b/packages/web/src/app/components/authMethodSelector.tsx
@@ -9,6 +9,7 @@ import { DividerSet } from "@/app/components/dividerSet";
 import { ProviderButton } from "@/app/components/providerButton";
 import { AuthSecurityNotice } from "@/app/components/authSecurityNotice";
 import type { IdentityProviderMetadata } from "@/lib/identityProviders";
+import Link from "next/link";
 
 interface AuthMethodSelectorProps {
     providers: IdentityProviderMetadata[];
@@ -44,6 +45,15 @@ export const AuthMethodSelector = ({
     const hasCredentials = providers.some(p => p.purpose === "sso" && p.id === "credentials");
     const hasMagicLink = providers.some(p => p.purpose === "sso" && p.id === "nodemailer");
 
+    if (oauthProviders.length === 0 && !hasCredentials && !hasMagicLink) {
+        return (
+            <div className="flex flex-col items-center justify-center w-full gap-2">
+                <p className="text-center font-medium">No authentication methods available. Please contact your administrator to configure authentication.</p>
+                <Link href="https://docs.sourcebot.dev/docs/configuration/auth/overview" target="_blank" rel="noopener" className="text-link hover:underline">Learn more</Link>
+            </div>
+        )
+    }
+
     return (
         <>
             <AuthSecurityNotice closable={securityNoticeClosable} />
diff --git a/packages/web/src/ee/features/codeNav/components/symbolHoverPopup/symbolHoverTargetsExtension.ts b/packages/web/src/ee/features/codeNav/components/symbolHoverPopup/symbolHoverTargetsExtension.ts
@@ -74,7 +74,7 @@ export const symbolHoverTargetsExtension = StateField.define<DecorationSet>({
                 }
             },
         });
-        return Decoration.set(decorations);
+        return Decoration.set(decorations, /* sort = */ true);
     },
     update(deco, tr) {
         return deco.map(tr.changes);
diff --git a/packages/web/src/features/chat/components/chatBox/searchScopeSelector.tsx b/packages/web/src/features/chat/components/chatBox/searchScopeSelector.tsx
@@ -320,23 +320,17 @@ export const SearchScopeSelector = forwardRef<
                                                         >
                                                             <CheckIcon className="h-4 w-4" />
                                                         </div>
-                                                        <div className="flex items-center gap-2 flex-1">
+                                                        <div className="flex flex-row items-center gap-2 w-full overflow-hidden">
                                                             <SearchScopeIcon searchScope={item} />
-                                                            <div className="flex flex-col flex-1">
-                                                                <div className="flex items-center gap-2">
-                                                                    <span className="font-medium">
-                                                                        {item.name}
-                                                                    </span>
-                                                                    {item.type === 'reposet' && (
-                                                                        <Badge
-                                                                            variant="default"
-                                                                            className="text-[10px] px-1.5 py-0 h-4 bg-primary text-primary-foreground"
-                                                                        >
-                                                                            {item.repoCount} repo{item.repoCount === 1 ? '' : 's'}
-                                                                        </Badge>
-                                                                    )}
-                                                                </div>
-                                                            </div>
+                                                            <p className="font-medium truncate-start">{item.name}</p>
+                                                            {item.type === 'reposet' && (
+                                                                <Badge
+                                                                    variant="default"
+                                                                    className="text-[10px] px-1.5 py-0 h-4 bg-primary text-primary-foreground"
+                                                                >
+                                                                    {item.repoCount} repo{item.repoCount === 1 ? '' : 's'}
+                                                                </Badge>
+                                                            )}
                                                         </div>
                                                     </div>
                                                 );
diff --git a/packages/web/src/features/chat/components/chatThread/codeFoldingExtension.ts b/packages/web/src/features/chat/components/chatThread/codeFoldingExtension.ts
@@ -342,10 +342,7 @@ const createDecorations = (state: EditorState, foldingState: FoldingState): Deco
         decorations.push(decoration.range(from, to));
     });
 
-    // Sort decorations by their 'from' position to ensure proper ordering
-    decorations.sort((a, b) => a.from - b.from);
-
-    return Decoration.set(decorations);
+    return Decoration.set(decorations, /* sort = */ true);
 };
 
 // Combined StateField that manages both folding state and decorations
diff --git a/packages/web/src/features/chat/components/chatThread/referencedFileSourceListItem.tsx b/packages/web/src/features/chat/components/chatThread/referencedFileSourceListItem.tsx
@@ -156,8 +156,7 @@ const ReferencedFileSourceListItem = ({
                         }
                     }
 
-                    decorations.sort((a, b) => a.from - b.from);
-                    return Decoration.set(decorations);
+                    return Decoration.set(decorations, /* sort = */ true);
                 },
                 update(deco, tr) {
                     return deco.map(tr.changes);
diff --git a/packages/web/src/features/chat/components/chatThread/tools/shared.tsx b/packages/web/src/features/chat/components/chatThread/tools/shared.tsx
@@ -90,7 +90,7 @@ export const ToolHeader = ({ isLoading, isError, isExpanded, label, Icon, onExpa
         <div
             tabIndex={0}
             className={cn(
-                "flex flex-row items-center gap-2 group w-fit select-none",
+                "flex flex-row items-center gap-2 group select-none",
                 {
                     'hover:text-foreground cursor-pointer': !isLoading,
                 },
@@ -107,16 +107,20 @@ export const ToolHeader = ({ isLoading, isError, isExpanded, label, Icon, onExpa
             }}
         >
             {isLoading ? (
-                <Loader2 className="w-4 h-4 animate-spin" />
+                <Loader2 className="w-4 h-4 animate-spin flex-shrink-0" />
             ) : (
-                <Icon className="h-4 w-4" />
+                <Icon className="h-4 w-4 flex-shrink-0" />
             )}
-            <span className={cn("text-sm font-medium",
+            <span
+                className={cn("text-sm font-medium line-clamp-2",
                 {
                     'animate-pulse': isLoading,
                     'text-destructive': isError,
                 }
-            )}>{label}</span>
+            )}
+            >
+                {label}
+            </span>
             {!isLoading && (
                 <div className="opacity-0 group-hover:opacity-100 transition-opacity">
                     {isExpanded ? (
diff --git a/packages/web/src/lib/extensions/searchResultHighlightExtension.ts b/packages/web/src/lib/extensions/searchResultHighlightExtension.ts
@@ -41,7 +41,7 @@ const matchHighlighter = StateField.define<DecorationSet>({
                     })
                     .filter((decoration) => decoration !== undefined);
 
-                highlights = Decoration.set(decorations)
+                highlights = Decoration.set(decorations, /* sort = */ true);
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -1,2 +1,2 @@`
`1`	`1`	`// This file is auto-generated by .github/workflows/release-sourcebot.yml`
`2`		`-export const SOURCEBOT_VERSION = "v4.10.10";`
	`2`	`+export const SOURCEBOT_VERSION = "v4.10.11";`
Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ export const symbolHoverTargetsExtension = StateField.define<DecorationSet>({`
`74`	`74`	`}`
`75`	`75`	`},`
`76`	`76`	`});`
`77`		`- return Decoration.set(decorations);`
	`77`	`+ return Decoration.set(decorations, /* sort = */ true);`
`78`	`78`	`},`
`79`	`79`	`update(deco, tr) {`
`80`	`80`	`return deco.map(tr.changes);`
Original file line number	Diff line number	Diff line change
`@@ -156,8 +156,7 @@ const ReferencedFileSourceListItem = ({`
`156`	`156`	`}`
`157`	`157`	`}`
`158`	`158`
`159`		`- decorations.sort((a, b) => a.from - b.from);`
`160`		`- return Decoration.set(decorations);`
	`159`	`+ return Decoration.set(decorations, /* sort = */ true);`
`161`	`160`	`},`
`162`	`161`	`update(deco, tr) {`
`163`	`162`	`return deco.map(tr.changes);`