Address PR feedback for HTML tag stripping and test improvements

Co-authored-by: dscho <[email protected]>

Author: Copilot

script/update-docs.rb

@@ -174,7 +174,8 @@ def index_l10n_doc(filter_tags, doc_list, get_content)
 
       # HTML anchor on hdlist1 (i.e. command options)
       html.gsub!(/<dt class="hdlist1">(.*?)<\/dt>/) do |_m|
-        text = $1.gsub(/<\/?(?:code|placeholder)>/, "").tr("^A-Za-z0-9-", "")
+        dt_content = $1
+        text = dt_content.gsub(/<\/?[^>]+>/, "").tr("^A-Za-z0-9-", "")
         # use txt_path for backward compatibility of already-existing
 	# deep links shared across the interwebs.
 	anchor = "#{txt_path}-#{text}"
@@ -465,7 +466,8 @@ def index_doc(filter_tags, doc_list, get_content)
         end
         # HTML anchor on hdlist1 (i.e. command options)
         html.gsub!(/<dt class="hdlist1">(.*?)<\/dt>/) do |_m|
-          text = $1.gsub(/<\/?(?:code|placeholder)>/, "").tr("^A-Za-z0-9-", "")
+          dt_content = $1
+          text = dt_content.gsub(/<\/?[^>]+>/, "").tr("^A-Za-z0-9-", "")
           # use txt_path for backward compatibility of already-existing
           # deep links shared across the interwebs.
           anchor = "#{txt_path}-#{text}"

tests/git-scm.spec.js

@@ -194,22 +194,13 @@ test('manual pages', async ({ page }) => {
 })
 
 test('anchor links in manual pages', async ({ page }) => {
-  // Test that anchor links work with different formats (with or without code tags)
-  await page.goto(`${url}docs/git-clone`)
-  
-  // Check that old-style anchor links without code tags still work
-  await page.goto(`${url}docs/git-clone#Documentation/git-clone.txt---recurse-submodulesltpathspecgt`)
+  // Test that anchor links work without HTML tags
+  const anchor = '#Documentation/git-clone.txt---recurse-submodulesltpathspecgt'
+  await page.goto(`${url}docs/git-clone${anchor}`)
 
   // Find the element that should be highlighted/anchored to
   const anchoredElement = page.locator('#Documentation-git-clone\\.txt---recurse-submodulesltpathspecgt')
   await expect(anchoredElement).toBeVisible()
-  
-  // Check that new-style anchor links with code tags also work
-  await page.goto(`${url}docs/git-clone#Documentation/git-clone.txt-code--recurse-submodulesltpathspecgtcode`)
-  
-  // Both should point to the same element (after our fix)
-  const anchoredElementAlt = page.locator('#Documentation-git-clone\\.txt---recurse-submodulesltpathspecgt')
-  await expect(anchoredElementAlt).toBeVisible()
 })
 
 test('book', async ({ page }) => {