Remove unused exported symbols and one dead component (#62047)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: heiskr

src/content-linter/lib/helpers/liquid-utils.ts

@@ -30,8 +30,6 @@ export function getLiquidTokens(
 
 export const OUTPUT_OPEN = '{%'
 export const OUTPUT_CLOSE = '%}'
-export const TAG_OPEN = '{{'
-export const TAG_CLOSE = '}}'
 
 export const conditionalTags = ['if', 'elseif', 'unless', 'case', 'ifversion']
 

src/content-linter/lib/helpers/utils.ts

@@ -70,75 +70,6 @@ export function quotePrecedesLinkOpen(text: string | undefined): boolean {
   return text.endsWith('"') || text.endsWith("'")
 }
 
-// Filters a list of tokens by token type only when they match
-// a specific token type order.
-// For example, if a list of tokens contains:
-//
-//   [
-//      { type: 'inline'},
-//      { type: 'list_item_close'},
-//      { type: 'list_item_open'},
-//      { type: 'paragraph_open'},
-//      { type: 'inline'},
-//      { type: 'paragraph_close'},
-//   ]
-//
-// And if the `tokenOrder` being looked for is:
-//
-//   [
-//      'list_item_open',
-//      'paragraph_open',
-//      'inline'
-//    ]
-//
-// Then the return value would be the items that match that sequence:
-// Index 2-4:
-//   [
-//      { type: 'inline'},            <-- Index 0 - NOT INCLUDED
-//      { type: 'list_item_close'},   <-- Index 1 - NOT INCLUDED
-//      { type: 'list_item_open'},    <-- Index 2 - INCLUDED
-//      { type: 'paragraph_open'},    <-- Index 3 - INCLUDED
-//      { type: 'inline'},            <-- Index 4 - INCLUDED
-//      { type: 'paragraph_close'},   <-- Index 5 - NOT INCLUDED
-//   ]
-//
-export function filterTokensByOrder(
-  tokens: MarkdownToken[],
-  tokenOrder: string[],
-): MarkdownToken[] {
-  const matches: MarkdownToken[] = []
-
-  // Get a list of token indexes that match the
-  // first token (root) in the tokenOrder array
-  const tokenRootIndexes: number[] = []
-  const firstTokenOrderType = tokenOrder[0]
-  for (let index = 0; index < tokens.length; index++) {
-    const token = tokens[index]
-    if (token.type === firstTokenOrderType) {
-      tokenRootIndexes.push(index)
-    }
-  }
-
-  // Loop through each root token index and check if
-  // the order matches the tokenOrder array
-  for (const tokenRootIndex of tokenRootIndexes) {
-    for (let i = 1; i < tokenOrder.length; i++) {
-      if (tokens[tokenRootIndex + i].type !== tokenOrder[i]) {
-        // This tokenRootIndex was a possible start,
-        // but doesn't match the tokenOrder perfectly, so break out
-        // of the inner loop before it reaches the end.
-        break
-      }
-      if (i === tokenOrder.length - 1) {
-        matches.push(...tokens.slice(tokenRootIndex, tokenRootIndex + i + 1))
-      }
-    }
-  }
-  return matches
-}
-
-export const docsDomains = ['docs.github.com', 'help.github.com', 'developer.github.com']
-
 // Lines is an array of strings read from a
 // Markdown file a split around new lines.
 // This is the format we get from Markdownlint.

src/frame/components/hooks/useFeatureFlags.ts

@@ -1,8 +1 @@
-import { useMainContext } from '@/frame/components/context/MainContext'
-
 export type FeatureFlags = Record<string, string>
-
-export const useFeatureFlags = (): FeatureFlags => {
-  const { featureFlags } = useMainContext()
-  return featureFlags
-}

src/landings/components/LandingSection.tsx

@@ -1,4 +1,3 @@
-import type { UIStrings } from '@/frame/components/context/MainContext'
 import { useMainContext } from '@/frame/components/context/MainContext'
 import { createTranslationFunctions } from '@/languages/lib/translation-utils'
 
@@ -37,10 +36,3 @@ export const useTranslation = (namespaces: string | Array<string>) => {
 
   return createTranslationFunctions(loadedData, namespaces)
 }
-
-/**
- * Hook for App Router contexts that don't use MainContext
- */
-export const useAppTranslation = (uiData: UIStrings, namespaces: string | Array<string>) => {
-  return createTranslationFunctions(uiData, namespaces)
-}

src/languages/components/useTranslation.ts

@@ -355,22 +355,6 @@ export async function renderAndExtractLinks(
   }
 }
 
-/**
- * Read a file and extract links
- */
-export async function extractLinksFromFile(
-  filePath: string,
-  context?: Context,
-): Promise<LinkExtractionResult> {
-  const content = fs.readFileSync(filePath, 'utf-8')
-
-  if (context) {
-    return extractLinksWithLiquid(content, context)
-  }
-
-  return extractLinksFromMarkdown(content)
-}
-
 /**
  * Get relative path from content root
  */

src/links/lib/extract-links.ts

@@ -9,7 +9,6 @@ import { sanitizeSearchQuery } from '@/search/lib/sanitize-search-query'
 // Search context values for identifying each search event
 export const GENERAL_SEARCH_CONTEXT = 'general-search'
 export const AI_SEARCH_CONTEXT = 'ai-search'
-export const COMBINED_SEARCH_CONTEXT = 'combined-search'
 
 // The logic that redirects to the /search page with the proper query params
 // The query params will be consumed in the general search middleware

src/search/components/helpers/execute-search-actions.ts

@@ -1,24 +1,3 @@
-import { useRouter } from 'next/router'
-
-type QueryInfo = {
-  query: string
-  debug: boolean
-}
-export const useQuery = (): QueryInfo => {
-  const router = useRouter()
-  const query =
-    router.query.query && Array.isArray(router.query.query)
-      ? router.query.query[0]
-      : router.query.query || ''
-
-  const debug = parseDebug(router.query.debug)
-
-  return {
-    query,
-    debug,
-  }
-}
-
 export function parseDebug(debug: string | Array<string> | undefined) {
   if (debug === '') {
     // E.g. `?query=foo&debug` should be treated as truthy

src/search/components/hooks/useQuery.ts

@@ -2,12 +2,6 @@
 import { versionToIndexVersionMap } from '../elasticsearch-versions'
 const CSE_COPILOT_DOCS_VERSIONS = ['dotcom', 'ghec', 'ghes']
 
-// Languages supported by cse-copilot
-const DOCS_LANGUAGES = ['en']
-export function supportedCSECopilotLanguages() {
-  return DOCS_LANGUAGES
-}
-
 export function getCSECopilotSource(version: (typeof CSE_COPILOT_DOCS_VERSIONS)[number]) {
   if (!version) {
     throw new Error(`Missing required key 'version' in request body`)

src/search/lib/helpers/cse-copilot-docs-versions.ts

@@ -124,20 +124,6 @@ function sanitizeUserAgent(userAgent: string | undefined): string {
   return 'other'
 }
 
-/**
- * Determines if a host should bypass client_name requirement for analytics
- * Returns true if the host ends with github.net or githubapp.com
- * (for internal staging environments)
- * Note: docs.github.com is removed since normalizedHost will always be docs.github.com in production
- * Note: localhost is NOT included here as it should send analytics with auto-set client_name
- */
-export function shouldBypassClientNameRequirement(host: string | undefined): boolean {
-  if (!host) return false
-
-  const normalizedHost = stripPort(host)
-  return normalizedHost.endsWith('.github.net') || normalizedHost.endsWith('.githubapp.com')
-}
-
 /**
  * Strips port number from host string
  */