cleanup

Author: tannerlinsley

src/components/RedirectVersionBanner.tsx

@@ -1,5 +1,5 @@
 import { useLocalStorage } from '~/utils/useLocalStorage'
-import { useClientOnlyRender } from '~/utils/useClientOnlyRender'
+import { useMounted } from '~/hooks/useMounted'
 import { Link, useMatches } from '@tanstack/react-router'
 
 export function RedirectVersionBanner(props: {
@@ -18,7 +18,7 @@ export function RedirectVersionBanner(props: {
     1000 * 60 * 24 * 30,
   )
 
-  if (!useClientOnlyRender()) {
+  if (!useMounted()) {
     return null
   }
 

src/components/SearchModal.tsx

@@ -355,23 +355,18 @@ const Hit = ({
             </p>
           ) : null}
         </div>
-        {(() => {
-          // Only show the badge on the right if we ARE filtered to a framework
-          if (!refinedFramework || !hitFramework) return null
-
-          return (
-            <div className="flex-none">
-              <div className="flex items-center gap-1 text-xs font-black bg-white rounded-xl px-2 py-1 dark:bg-black">
-                <img
-                  src={hitFramework.logo}
-                  alt={hitFramework.label}
-                  className="w-4"
-                />
-                {capitalize(hitFramework.label)}
-              </div>
+        {refinedFramework && hitFramework ? (
+          <div className="flex-none">
+            <div className="flex items-center gap-1 text-xs font-black bg-white rounded-xl px-2 py-1 dark:bg-black">
+              <img
+                src={hitFramework.logo}
+                alt={hitFramework.label}
+                className="w-4"
+              />
+              {capitalize(hitFramework.label)}
             </div>
-          )
-        })()}
+          </div>
+        ) : null}
       </article>
     </SafeLink>
   )

src/routes/__root.tsx

@@ -23,15 +23,11 @@ import { ToastProvider } from '~/components/ToastProvider'
 import { Spinner } from '~/components/Spinner'
 import { ThemeProvider } from '~/components/ThemeProvider'
 import { Navbar } from '~/components/Navbar'
-
-// import { LibrariesLayout } from './_libraries/route'
 import { THEME_COLORS } from '~/utils/utils'
 import { useHubSpotChat } from '~/hooks/useHubSpotChat'
-// import { getCurrentUser } from '~/utils/auth.server'
 
 export const Route = createRootRouteWithContext<{
   queryClient: QueryClient
-  // user?: Awaited<ReturnType<typeof getCurrentUser>>
 }>()({
   head: () => ({
     meta: [
@@ -129,9 +125,7 @@ export const Route = createRootRouteWithContext<{
   notFoundComponent: () => {
     return (
       <DocumentWrapper>
-        {/* <LibrariesLayout> */}
         <NotFound />
-        {/* </LibrariesLayout> */}
       </DocumentWrapper>
     )
   },

src/routes/admin/users.tsx

@@ -173,7 +173,6 @@ function UsersPage() {
   const [editingUserId, setEditingUserId] = useState<string | null>(null)
   const [editingCapabilities, setEditingCapabilities] = useState<string[]>([])
   const [editingRoleIds, setEditingRoleIds] = useState<string[]>([])
-  const [updatingAdsUserId] = useState<string | null>(null)
   const [selectedUserIds, setSelectedUserIds] = useState<Set<string>>(new Set())
   const [bulkActionRoleId, setBulkActionRoleId] = useState<string | null>(null)
   const [expandedSections, setExpandedSections] = useState<
@@ -844,7 +843,6 @@ function UsersPage() {
               onChange={(e) =>
                 handleToggleAdsDisabled(user._id, e.target.checked)
               }
-              disabled={updatingAdsUserId === user._id}
               className="h-4 w-4 accent-blue-600"
             />
           )
@@ -911,7 +909,6 @@ function UsersPage() {
       toggleCapability,
       toggleRole,
       handleToggleAdsDisabled,
-      updatingAdsUserId,
       allRoles,
       selectedUserIds,
       toggleUserSelection,

src/stores/userSettings.ts

@@ -1,31 +1,17 @@
 import { create } from 'zustand'
-// Remove persist and createJSONStorage imports since we no longer use localStorage
-// import { persist, createJSONStorage } from 'zustand/middleware'
-// Remove the useCurrentUserQuery import since we'll use the new hook
-// import { useCurrentUserQuery } from '~/hooks/useCurrentUser'
 
-// Update the export to use the new hook
 export { useAdsPreference } from '~/hooks/useAdPreference'
 
-export type UserSettings = {
-  // Other settings can be added here in the future
-}
+export type UserSettings = {}
 
 type UserSettingsState = {
   settings: UserSettings
   hasHydrated: boolean
   setHasHydrated: (value: boolean) => void
-  // Remove toggleAds since it's now handled by the new hooks
 }
 
 export const useUserSettingsStore = create<UserSettingsState>()((set) => ({
-  settings: {
-    // Remove adsDisabled initialization
-  },
-  hasHydrated: true, // No need for hydration since we're not using persistence
+  settings: {},
+  hasHydrated: true,
   setHasHydrated: (value) => set({ hasHydrated: value }),
-  // Remove toggleAds function
 }))
-
-// Remove the persist wrapper and localStorage configuration
-// The useAdsPreference function is now exported from useAdPreference.ts

src/styles/app.css

@@ -618,10 +618,6 @@ mark {
   @apply fill-gray-800/90 stroke-gray-500/20;
 }
 
-/* svg g[aria-label='crosshair text'] {
-  stroke: none;
-} */
-
 @font-face {
   font-display: swap;
   font-family: 'Inter';

src/utils/docs.ts

@@ -8,25 +8,15 @@ import { notFound } from '@tanstack/react-router'
 import { createServerFn } from '@tanstack/react-start'
 import { z } from 'zod'
 import { setResponseHeader } from '@tanstack/react-start/server'
-// import {
-//   getContributorStats,
-//   getContributorStatsForLibrary,
-//   getBatchContributorStats,
-//   type ContributorStats,
-// } from '~/server/github'
 
 export const loadDocs = async ({
   repo,
   branch,
-  // currentPath,
-  // redirectPath,
   docsPath,
 }: {
   repo: string
   branch: string
   docsPath: string
-  currentPath: string
-  redirectPath: string
 }) => {
   if (!branch || !docsPath) {
     throw notFound({
@@ -43,8 +33,6 @@ export const loadDocs = async ({
       repo,
       branch,
       filePath,
-      // currentPath,
-      // redirectPath,
     },
   })
 }
@@ -58,15 +46,6 @@ export const fetchDocs = createServerFn({ method: 'GET' })
 
     if (!file) {
       throw notFound()
-      // if (currentPath === redirectPath) {
-      //   // console.log('not found')
-      //   throw notFound()
-      // } else {
-      //   // console.log('redirect')
-      //   throw redirect({
-      //     to: redirectPath,
-      //   })
-      // }
     }
 
     const frontMatter = extractFrontMatter(file)
@@ -134,86 +113,3 @@ export const fetchRepoDirectoryContents = createServerFn({
 
     return githubContents
   })
-
-// GitHub contribution stats server functions - commented out due to performance/accuracy concerns
-/*
-export const fetchContributorStats = createServerFn({ method: 'GET' })
-  .inputValidator(z.object({ username: z.string() }))
-  .handler(async ({ data: { username } }) => {
-    const stats = await getContributorStats(username)
-
-    // Cache for 30 minutes on shared cache
-    // Revalidate in the background
-    setResponseHeader('Cache-Control', 'public, max-age=0, must-revalidate')
-    setResponseHeader(
-      'CDN-Cache-Control',
-      'max-age=1800, stale-while-revalidate=1800, durable'
-    )
-
-    return stats
-  })
-
-export const fetchContributorStatsForLibrary = createServerFn({ method: 'GET' })
-  .inputValidator(
-    z.object({
-      username: z.string(),
-      libraryRepo: z.string(),
-    })
-  )
-  .handler(async ({ data: { username, libraryRepo } }) => {
-    const stats = await getContributorStatsForLibrary(username, libraryRepo)
-
-    // Cache for 30 minutes on shared cache
-    // Revalidate in the background
-    setResponseHeader('Cache-Control', 'public, max-age=0, must-revalidate')
-    setResponseHeader(
-      'CDN-Cache-Control',
-      'max-age=1800, stale-while-revalidate=1800, durable'
-    )
-
-    return stats
-  })
-
-export const fetchBatchContributorStats = createServerFn({ method: 'GET' })
-  .inputValidator(z.object({ usernames: z.array(z.string()) }))
-  .handler(async ({ data: { usernames } }) => {
-    const stats = await getBatchContributorStats(usernames)
-
-    // Cache for 30 minutes on shared cache
-    // Revalidate in the background
-    setResponseHeader('Cache-Control', 'public, max-age=0, must-revalidate')
-    setResponseHeader(
-      'CDN-Cache-Control',
-      'max-age=1800, stale-while-revalidate=1800, durable'
-    )
-
-    return stats
-  })
-
-// Helper function to get stats for all maintainers
-export const fetchAllMaintainerStats = createServerFn({
-  method: 'GET',
-}).handler(async () => {
-  try {
-    // Import maintainers data
-    const { allMaintainers } = await import('~/libraries/maintainers')
-
-    const usernames = allMaintainers.map((maintainer) => maintainer.github)
-    const stats = await getBatchContributorStats(usernames)
-
-    // Cache for 30 minutes on shared cache
-    // Revalidate in the background
-    setResponseHeader('Cache-Control', 'public, max-age=0, must-revalidate')
-    setResponseHeader(
-      'CDN-Cache-Control',
-      'max-age=1800, stale-while-revalidate=1800, durable'
-    )
-
-    return stats
-  } catch (error) {
-    console.error('Error fetching all maintainer stats:', error)
-    // Return empty array if there's an error
-    return []
-  }
-})
-*/