fix: implement stripUrlExceptId function to clean URLs

Author: sy-records

src/core/event/index.js

@@ -1,5 +1,6 @@
 import { isMobile, mobileBreakpoint } from '../util/env.js';
 import * as dom from '../util/dom.js';
+import { stripUrlExceptId } from '../router/util.js';
 
 /** @typedef {import('../Docsify.js').Constructor} Constructor */
 
@@ -474,6 +475,8 @@ export function Events(Base) {
         return;
       }
 
+      href = stripUrlExceptId(href);
+
       const oldActive = dom.find(sidebar, 'li.active');
       const newActive = dom
         .find(

src/core/render/compiler/heading.js

@@ -4,6 +4,7 @@ import {
   getAndRemoveDocsifyIgnoreConfig,
 } from '../utils.js';
 import { slugify } from '../slugify.js';
+import { stripUrlExceptId } from '../../router/util.js';
 
 export const headingCompiler = ({ renderer, router, compiler }) =>
   (renderer.heading = function ({ tokens, depth }) {
@@ -20,7 +21,7 @@ export const headingCompiler = ({ renderer, router, compiler }) =>
     nextToc.ignoreSubHeading = ignoreSubHeading;
     const slug = slugify(config.id || str);
     const url = router.toURL(router.getCurrentPath(), { id: slug });
-    nextToc.slug = url;
+    nextToc.slug = stripUrlExceptId(url);
     compiler.toc.push(nextToc);
 
     // Note: tabindex="-1" allows programmatically focusing on heading

src/core/router/history/base.js

@@ -82,10 +82,6 @@ export class History {
     const local = currentRoute && path[0] === '#';
     const route = this.parse(replaceSlug(path));
 
-    if (route?.query?.s) {
-      delete route.query.s;
-    }
-
     route.query = { ...route.query, ...params };
     path = route.path + stringifyQuery(route.query);
     path = path.replace(/\.md(\?)|\.md$/, '$1');

src/core/router/util.js

@@ -40,6 +40,20 @@ export function stringifyQuery(obj, ignores = []) {
   return qs.length ? `?${qs.join('&')}` : '';
 }
 
+export function stripUrlExceptId(str) {
+  const [path, queryString] = str.split('?');
+  if (!queryString) {return str;}
+
+  const params = new URLSearchParams(queryString);
+  const id = params.get('id');
+
+  if (id !== null) {
+    return `${path}?id=${id}`;
+  }
+
+  return path;
+}
+
 export const isAbsolutePath = cached(path => {
   return /(:|(\/{2}))/g.test(path);
 });