Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Commit 673f9e8

Browse files
authored
remove old and unused Cody search page (#63439)
This page (at `/cody/search`) let you type in a natural-language description of what to search for, and then it used an LLM to rewrite that to Sourcegraph search query syntax. This was enabled using the `cody-web-search` feature flag, which has been disabled on dotcom for a while and was never documented or shared with customers, so it's safe to remove. It was a cool idea, but we decided to focus on making Cody's search in the editor better, and we haven't seriously improved this for ~12 months. ## Test plan n/a; removed feature
1 parent 6f9b7c5 commit 673f9e8

File tree

13 files changed

+15
-1174
lines changed

13 files changed

+15
-1174
lines changed

client/web/BUILD.bazel

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -272,9 +272,6 @@ ts_project(
272272
"src/cody/onboarding/instructions/JetBrains.tsx",
273273
"src/cody/onboarding/instructions/NeoVim.tsx",
274274
"src/cody/onboarding/instructions/VsCode.tsx",
275-
"src/cody/search/CodySearchPage.tsx",
276-
"src/cody/search/api.ts",
277-
"src/cody/search/translateToQuery.ts",
278275
"src/cody/sidebar/CodySidebar.tsx",
279276
"src/cody/sidebar/Provider.tsx",
280277
"src/cody/sidebar/index.tsx",

client/web/src/LegacyLayout.tsx

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
import { type FC, Suspense, useCallback, useLayoutEffect, useState } from 'react'
1+
import { Suspense, useCallback, useLayoutEffect, useState, type FC } from 'react'
22

33
import classNames from 'classnames'
4-
import { matchPath, useLocation, Route, Routes, Navigate, type RouteObject } from 'react-router-dom'
4+
import { matchPath, Navigate, Route, Routes, useLocation, type RouteObject } from 'react-router-dom'
55

66
import { useKeyboardShortcut } from '@sourcegraph/shared/src/keyboardShortcuts/useKeyboardShortcut'
77
import { Shortcut } from '@sourcegraph/shared/src/react-shortcuts'
88
import { useExperimentalFeatures } from '@sourcegraph/shared/src/settings/settings'
9-
import { useTheme, Theme } from '@sourcegraph/shared/src/theme'
9+
import { Theme, useTheme } from '@sourcegraph/shared/src/theme'
1010
import { lazyComponent } from '@sourcegraph/shared/src/util/lazyComponent'
1111
import { FeedbackPrompt, LoadingSpinner, useLocalStorage } from '@sourcegraph/wildcard'
1212

@@ -64,7 +64,6 @@ export const LegacyLayout: FC<LegacyLayoutProps> = props => {
6464
const isSearchHomepage = location.pathname === '/search' && !parseSearchURLQuery(location.search)
6565
const isSearchJobsPage = routeMatch?.startsWith(PageRoutes.SearchJobs)
6666
const isSearchNotebooksPage = routeMatch?.startsWith(PageRoutes.Notebooks)
67-
const isCodySearchPage = routeMatch === PageRoutes.CodySearch
6867
const isRepositoryRelatedPage = routeMatch === PageRoutes.RepoContainer ?? false
6968

7069
// Since the access token callback page is rendered in a nested route, we can't use
@@ -188,7 +187,6 @@ export const LegacyLayout: FC<LegacyLayoutProps> = props => {
188187
!isSearchHomepage &&
189188
!isCommunitySearchContextPage &&
190189
!isSearchNotebooksPage &&
191-
!isCodySearchPage &&
192190
!isSearchJobsPage
193191

194192
return (

client/web/src/cody/search/CodySearchPage.module.scss

Lines changed: 0 additions & 23 deletions
This file was deleted.

client/web/src/cody/search/CodySearchPage.tsx

Lines changed: 0 additions & 220 deletions
This file was deleted.

client/web/src/cody/search/api.ts

Lines changed: 0 additions & 46 deletions
This file was deleted.

0 commit comments

Comments
 (0)