fix: address PR review comments for markdown output

- Fix Cache-Control mismatch: change from 3600s to 60s to match ISR TTL
- Add URL validation after normalizeGitUrl() to skip non-HTTP URLs
- Fix installSize check: use !== undefined instead of truthiness
- Enhance sparkline with numeric trend values for LLM readability
- Add comprehensive unit tests for markdown utils (97.97% coverage)

Author: BYK

server/routes/raw/[...slug].md.get.ts

@@ -1,11 +1,10 @@
 import { generatePackageMarkdown } from '../../utils/markdown'
 import * as v from 'valibot'
 import { PackageRouteParamsSchema } from '#shared/schemas/package'
-import {
-  CACHE_MAX_AGE_ONE_HOUR,
-  NPM_MISSING_README_SENTINEL,
-  ERROR_NPM_FETCH_FAILED,
-} from '#shared/utils/constants'
+import { NPM_MISSING_README_SENTINEL, ERROR_NPM_FETCH_FAILED } from '#shared/utils/constants'
+
+// Cache TTL matches the ISR config for /raw/** routes (60 seconds)
+const CACHE_MAX_AGE = 60
 import { parseRepositoryInfo } from '#shared/utils/git-providers'
 
 const NPM_API = 'https://api.npmjs.org'
@@ -225,11 +224,7 @@ export default defineEventHandler(async event => {
   })
 
   setHeader(event, 'Content-Type', 'text/markdown; charset=utf-8')
-  setHeader(
-    event,
-    'Cache-Control',
-    `public, max-age=${CACHE_MAX_AGE_ONE_HOUR}, stale-while-revalidate`,
-  )
+  setHeader(event, 'Cache-Control', `public, max-age=${CACHE_MAX_AGE}, stale-while-revalidate`)
 
   return markdown
 })

server/utils/markdown.ts

@@ -30,6 +30,12 @@ function formatNumber(num: number): string {
   return new Intl.NumberFormat('en-US').format(num)
 }
 
+function formatCompactNumber(num: number): string {
+  return new Intl.NumberFormat('en-US', { notation: 'compact', maximumFractionDigits: 1 }).format(
+    num,
+  )
+}
+
 function formatBytes(bytes: number): string {
   if (bytes < 1024) return `${bytes} B`
   if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(1)} kB`
@@ -65,6 +71,8 @@ function getRepositoryUrl(repository?: {
 }): string | null {
   if (!repository?.url) return null
   let url = normalizeGitUrl(repository.url)
+  // Skip non-HTTP URLs after normalization
+  if (!isHttpUrl(url)) return null
   // Append directory for monorepo packages
   if (repository.directory) {
     url = joinURL(`${url}/tree/HEAD`, repository.directory)
@@ -160,7 +168,7 @@ export function generatePackageMarkdown(options: PackageMarkdownOptions): string
   const statsSeparators: string[] = []
   const statsValues: string[] = []
 
-  // Weekly downloads with sparkline
+  // Weekly downloads with sparkline and trend numbers
   if (weeklyDownloads !== undefined) {
     statsHeaders.push('Downloads (weekly)')
     statsSeparators.push('---')
@@ -169,7 +177,9 @@ export function generatePackageMarkdown(options: PackageMarkdownOptions): string
     if (dailyDownloads && dailyDownloads.length > 0) {
       const weeklyTotals = buildWeeklyTotals(dailyDownloads)
       if (weeklyTotals.length > 1) {
-        downloadCell += ` ${generateSparkline(weeklyTotals)}`
+        // Add both numeric trend and visual sparkline for LLM + human readability
+        const compactTotals = weeklyTotals.map(formatCompactNumber).join(' ')
+        downloadCell += ` [${compactTotals}] ${generateSparkline(weeklyTotals)}`
       }
     }
     statsValues.push(downloadCell)
@@ -182,7 +192,7 @@ export function generatePackageMarkdown(options: PackageMarkdownOptions): string
   statsValues.push(String(depCount))
 
   // Install size
-  if (installSize) {
+  if (installSize !== undefined) {
     statsHeaders.push('Install Size')
     statsSeparators.push('---')
     statsValues.push(formatBytes(installSize))