feat: add territory SEO

Author: brymut

components/layout.js

@@ -10,11 +10,11 @@ import PullToRefresh from './pull-to-refresh'
 
 export default function Layout ({
   sub, contain = true, footer = true, footerLinks = true,
-  containClassName = '', seo = true, item, user, children
+  containClassName = '', seo = true, item, user, territory, children
 }) {
   return (
     <>
-      {seo && <Seo sub={sub} item={item} user={user} />}
+      {seo && <Seo sub={sub} item={item} user={user} territory={territory} />}
       <Navigation sub={sub} />
       {contain
         ? (

components/seo.js

@@ -2,6 +2,7 @@ import { NextSeo } from 'next-seo'
 import { useRouter } from 'next/router'
 import removeMd from 'remove-markdown'
 import { numWithUnits } from '@/lib/format'
+import { processTerritoryDescription } from '@/lib/territory'
 
 export function SeoSearch ({ sub }) {
   const router = useRouter()
@@ -36,14 +37,17 @@ export function SeoSearch ({ sub }) {
 // index page seo
 // recent page seo
 
-export default function Seo ({ sub, item, user }) {
+export default function Seo ({ sub, item, user, territory }) {
   const router = useRouter()
   const pathNoQuery = router.asPath.split('?')[0]
   const defaultTitle = pathNoQuery.slice(1)
   const snStr = `stacker news${sub ? ` ~${sub}` : ''}`
   let fullTitle = `${defaultTitle && `${defaultTitle} \\ `}stacker news`
   let desc = "It's like Hacker News but we pay you Bitcoin."
-  if (item) {
+  if (territory) {
+    fullTitle = `${territory.name} \\ ${snStr}`
+    desc = processTerritoryDescription(territory)
+  } else if (item) {
     if (item.title) {
       fullTitle = `${item.title} \\ ${snStr}`
     } else if (item.root) {
@@ -67,8 +71,7 @@ export default function Seo ({ sub, item, user }) {
     } else if (item.boost) {
       desc += ` [${item.boost} boost]`
     }
-  }
-  if (user) {
+  } else if (user) {
     desc = `@${user.name} has [${user.optional.stacked ? `${user.optional.stacked} stacked,` : ''}${numWithUnits(user.nitems, { unitSingular: 'item', unitPlural: 'items' })}]`
   }
 

lib/territory.js

@@ -1,5 +1,6 @@
 import { TERRITORY_GRACE_DAYS, TERRITORY_PERIOD_COST } from './constants'
 import { datePivot, diffDays } from './time'
+import removeMd from 'remove-markdown'
 
 export function nextBilling (relativeTo, billingType) {
   if (!relativeTo || billingType === 'ONCE') return null
@@ -28,3 +29,103 @@ export function nextBillingWithGrace (sub) {
   if (!sub) return null
   return datePivot(new Date(sub.billPaidUntil), { days: TERRITORY_GRACE_DAYS })
 }
+
+function cleanMarkdownText (text) {
+  if (!text) return ''
+
+  let cleaned = removeMd(text)
+  cleaned = cleaned
+    .replace(/\s+/g, ' ') // Replace multiple spaces with single space
+    .replace(/\[\[([^\]]+)\]\]\([^)]+\)/g, '$1') // Remove nested markdown links, keep inner text
+    .replace(/\[([^\]]+)\]\([^)]+\)/g, '$1') // Remove regular markdown links, keep text
+    .replace(/!\[([^\]]*)\]\([^)]+\)/g, '') // Remove images completely
+    .replace(/`([^`]+)`/g, '$1') // Remove code formatting
+    .replace(/\*\*([^*]+)\*\*/g, '$1') // Remove bold formatting
+    .replace(/\*([^*]+)\*/g, '$1') // Remove italic formatting
+    .trim()
+
+  return cleaned
+}
+
+function truncateAtSentenceBoundary (text, maxLength = 160) {
+  if (!text || text.length <= maxLength) return text
+  const sentences = text
+    .split(/[.!?]+/)
+    .map(s => s.trim())
+    .filter(Boolean)
+
+  let result = ''
+  let currentLength = 0
+
+  for (const sentence of sentences) {
+    const separator = result ? '. ' : ''
+    const newLength = currentLength + separator.length + sentence.length
+
+    if (newLength <= maxLength - 3) {
+      result += separator + sentence
+      currentLength = newLength
+    } else {
+      break
+    }
+  }
+  if (result) {
+    return result + '...'
+  }
+  // Fallback to simple truncation
+  return text.substring(0, maxLength - 3) + '...'
+}
+
+export function processTerritoryDescription (territory) {
+  if (!territory) return null
+
+  if (territory.desc) {
+    const processedDesc = cleanMarkdownText(territory.desc)
+
+    if (processedDesc.length >= 30 && processedDesc.length <= 160) {
+      return processedDesc
+    }
+    if (processedDesc.length > 160) {
+      return truncateAtSentenceBoundary(processedDesc, 160)
+    }
+  }
+
+  // Generate a contextual description if the original is too short or unsuitable
+  return generateTerritoryDescription(territory)
+}
+
+export function generateTerritoryDescription (territory) {
+  const parts = []
+  if (territory.desc) {
+    const processedDesc = cleanMarkdownText(territory.desc)
+    if (processedDesc) {
+      parts.push(processedDesc)
+    }
+  }
+
+  if (territory.postTypes && territory.postTypes.length > 0) {
+    const postTypeLabels = territory.postTypes.map(type => {
+      switch (type) {
+        case 'LINK': return 'links'
+        case 'DISCUSSION': return 'discussions'
+        case 'BOUNTY': return 'bounties'
+        case 'POLL': return 'polls'
+        default: return type.toLowerCase()
+      }
+    })
+    parts.push(postTypeLabels.join(', '))
+  }
+
+  if (territory.user?.name) {
+    parts.push(`created by @${territory.user.name}`)
+  }
+
+  let desc = parts.length > 1
+    ? `Stacker.news community: ${parts.join('. ')}`
+    : `Stacker.news community for ${parts.join(', ')}`
+
+  if (desc.length > 160) {
+    desc = truncateAtSentenceBoundary(desc, 160)
+  }
+
+  return desc
+}

pages/~/index.js

@@ -22,7 +22,7 @@ export default function Sub ({ ssrData }) {
   const { sub } = data || ssrData
 
   return (
-    <Layout sub={sub?.name}>
+    <Layout sub={sub?.name} territory={sub}>
       {sub
         ? <TerritoryHeader sub={sub} />
         : (