fix: improve link handling for absolute paths and mailto links

Author: sy-records

src/core/render/compiler/link.js

@@ -12,41 +12,37 @@ export const linkCompiler = ({
     const attrs = [];
     const text = this.parser.parseInline(tokens) || '';
     const { str, config } = getAndRemoveConfig(title);
+    const isAbsolute = isAbsolutePath(href);
+    const isNotCompilable = compiler._matchNotCompileLink(href);
+    const isMailto = href.startsWith('mailto:');
+
     linkTarget = config.target || linkTarget;
     linkRel =
       linkTarget === '_blank'
         ? compiler.config.externalLinkRel || 'noopener'
         : '';
     title = str;
 
-    if (
-      !isAbsolutePath(href) &&
-      !compiler._matchNotCompileLink(href) &&
-      !config.ignore
-    ) {
+    if (!isAbsolute && !isNotCompilable && !config.ignore) {
       if (href === compiler.config.homepage) {
         href = 'README';
       }
-
       href = router.toURL(href, null, router.getCurrentPath());
 
-      if (config.target) {
-        href.indexOf('mailto:') !== 0 && attrs.push(`target="${linkTarget}"`);
+      if (config.target && !isMailto) {
+        attrs.push(`target="${linkTarget}"`);
       }
     } else {
-      if (!isAbsolutePath(href) && href.startsWith('./')) {
-        href = router
-          .toURL(href, null, router.getCurrentPath())
-          .replace(/^#\//, '/');
+      if (!isAbsolute && !isNotCompilable && href.startsWith('./')) {
+        href = router.toURL(href, null, router.getCurrentPath()).replace(/^#\//, '/');
+      }
+
+      if (!isMailto) {
+        attrs.push(`target="${linkTarget}"`);
+        if (linkRel !== '') {
+          attrs.push(`rel="${linkRel}"`);
+        }
       }
-      attrs.push(href.indexOf('mailto:') === 0 ? '' : `target="${linkTarget}"`);
-      attrs.push(
-        href.indexOf('mailto:') === 0
-          ? ''
-          : linkRel !== ''
-            ? ` rel="${linkRel}"`
-            : '',
-      );
     }
 
     if (config.disabled) {