Fix anchor links in documentation by stripping HTML tags

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

Author: Copilot

script/update-docs.rb

@@ -174,7 +174,7 @@ 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.tr("^A-Za-z0-9-", "")
+        text = $1.gsub(/<\/?(?:code|placeholder)>/, "").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 +465,7 @@ 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.tr("^A-Za-z0-9-", "")
+          text = $1.gsub(/<\/?(?:code|placeholder)>/, "").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

@@ -193,6 +193,25 @@ test('manual pages', async ({ page }) => {
   await expect(synopsis).not.toHaveText(/git remote renom.*<ancien> <nouveau>/)
 })
 
+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`)
+  
+  // 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 }) => {
   await page.goto(`${url}book/`)
   await expect(page).toHaveURL(`${url}book/en/v2`)