fix: do not strip package name from description (#2274)

Author: gameroman

app/components/Package/Card.vue

@@ -37,7 +37,6 @@ const isExactMatch = computed(() => {
 const pkgDescription = useMarkdown(() => ({
   text: props.result.package.description ?? '',
   plain: true,
-  packageName: props.result.package.name,
 }))
 
 const numberFormatter = useNumberFormatter()

app/composables/useMarkdown.ts

@@ -4,8 +4,6 @@ interface UseMarkdownOptions {
   text: string
   /** When true, renders link text without the anchor tag (useful when inside another link) */
   plain?: boolean
-  /** Package name to strip from the beginning of the description (if present) */
-  packageName?: string
 }
 
 export function useMarkdown(options: MaybeRefOrGetter<UseMarkdownOptions>) {
@@ -25,7 +23,7 @@ function stripMarkdownImages(text: string): string {
 }
 
 // Strip HTML tags and escape remaining HTML to prevent XSS
-function stripAndEscapeHtml(text: string, packageName?: string): string {
+function stripAndEscapeHtml(text: string): string {
   // First decode any HTML entities in the input
   let stripped = decodeHtmlEntities(text)
 
@@ -45,18 +43,6 @@ function stripAndEscapeHtml(text: string, packageName?: string): string {
     (match, codeSpan: string | undefined) => codeSpan ?? '',
   )
 
-  if (packageName) {
-    // Trim first to handle leading/trailing whitespace from stripped HTML
-    stripped = stripped.trim()
-    // Collapse multiple whitespace into single space
-    stripped = stripped.replace(/\s+/g, ' ')
-    // Escape special regex characters in package name
-    const escapedName = packageName.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
-    // Match package name at the start, optionally followed by: space, dash, colon, hyphen, or just space
-    const namePattern = new RegExp(`^${escapedName}\\s*[-:—]?\\s*`, 'i')
-    stripped = stripped.replace(namePattern, '').trim()
-  }
-
   // Then escape any remaining HTML entities
   return stripped
     .replace(/&/g, '&amp;')
@@ -67,11 +53,11 @@ function stripAndEscapeHtml(text: string, packageName?: string): string {
 }
 
 // Parse simple inline markdown to HTML
-function parseMarkdown({ text, packageName, plain }: UseMarkdownOptions): string {
+function parseMarkdown({ text, plain }: UseMarkdownOptions): string {
   if (!text) return ''
 
   // First strip HTML tags and escape remaining HTML
-  let html = stripAndEscapeHtml(text, packageName)
+  let html = stripAndEscapeHtml(text)
 
   // Bold: **text** or __text__
   html = html.replace(/\*\*(.+?)\*\*/g, '<strong>$1</strong>')

app/pages/package/[[org]]/[name].vue

@@ -257,7 +257,6 @@ const versionSecurityMetadata = computed<PackageVersionInfo[]>(() => {
 // Process package description
 const pkgDescription = useMarkdown(() => ({
   text: pkg.value?.description ?? '',
-  packageName: pkg.value?.name,
 }))
 
 // Fetch dependency analysis (lazy, client-side)

test/nuxt/composables/use-markdown.spec.ts

@@ -196,95 +196,6 @@ describe('useMarkdown', () => {
     })
   })
 
-  describe('packageName prop', () => {
-    it('strips package name from the beginning of plain text', () => {
-      const processed = useMarkdown({
-        text: 'my-package - A great library',
-        packageName: 'my-package',
-      })
-      expect(processed.value).toBe('A great library')
-    })
-
-    it('strips package name with colon separator', () => {
-      const processed = useMarkdown({
-        text: 'my-package: A great library',
-        packageName: 'my-package',
-      })
-      expect(processed.value).toBe('A great library')
-    })
-
-    it('strips package name with em dash separator', () => {
-      const processed = useMarkdown({
-        text: 'my-package — A great library',
-        packageName: 'my-package',
-      })
-      expect(processed.value).toBe('A great library')
-    })
-
-    it('strips package name without separator', () => {
-      const processed = useMarkdown({
-        text: 'my-package A great library',
-        packageName: 'my-package',
-      })
-      expect(processed.value).toBe('A great library')
-    })
-
-    it('is case-insensitive', () => {
-      const processed = useMarkdown({
-        text: 'MY-PACKAGE - A great library',
-        packageName: 'my-package',
-      })
-      expect(processed.value).toBe('A great library')
-    })
-
-    it('does not strip package name from middle of text', () => {
-      const processed = useMarkdown({
-        text: 'A great my-package library',
-        packageName: 'my-package',
-      })
-      expect(processed.value).toBe('A great my-package library')
-    })
-
-    it('handles scoped package names', () => {
-      const processed = useMarkdown({
-        text: '@org/my-package - A great library',
-        packageName: '@org/my-package',
-      })
-      expect(processed.value).toBe('A great library')
-    })
-
-    it('handles package names with special regex characters', () => {
-      const processed = useMarkdown({
-        text: 'pkg.name+test - A great library',
-        packageName: 'pkg.name+test',
-      })
-      expect(processed.value).toBe('A great library')
-    })
-
-    it('strips package name from HTML-containing descriptions', () => {
-      const processed = useMarkdown({
-        text: '<b>my-package</b> - A great library',
-        packageName: 'my-package',
-      })
-      expect(processed.value).toBe('A great library')
-    })
-
-    it('strips package name from descriptions with markdown images', () => {
-      const processed = useMarkdown({
-        text: '![badge](https://badge.svg) my-package - A great library',
-        packageName: 'my-package',
-      })
-      expect(processed.value).toBe('A great library')
-    })
-
-    it('does nothing when packageName is not provided', () => {
-      const processed = useMarkdown({
-        text: 'my-package - A great library',
-      })
-      expect(processed.value).toBe('my-package - A great library')
-    })
-  })
-
   describe('HTML tag stripping', () => {
     it('strips simple HTML tags but keeps content', () => {
       const processed = useMarkdown({ text: '<b>bold text</b> here' })