site: renderer and link fixes from the entry pages review

- Deprecation banners start the reason with a capital letter.
- Bare URLs stop at bold markers and code spans, keep a closing paren
  that balances one inside them, and <url> autolinks render as links.
- A malformed percent escape in a body link is a named link error, not
  a crash; a ?query on a relative link is kept.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Wc67NX9MM964uaVGn4bpZJ

Author: TMHSDigital

site/lib/lib.test.mjs

@@ -89,7 +89,7 @@ eq("split rest starts at first h2", splitEntryBody(body).rest.split("\n")[0], "#
 eq("split restFirstLine", splitEntryBody(body).restFirstLine, 7);
 eq("sectionMarkdown Notes", sectionMarkdown(body, "Notes"), "- n1\n- Deprecated: gone");
 eq("sectionMarkdown missing", sectionMarkdown(body, "Nope"), null);
-eq("deprecationReason", deprecationReason(body), "gone");
+eq("deprecationReason starts a sentence", deprecationReason(body), "Gone");
 
 /* links -------------------------------------------------------------------- */
 import { makeLinkResolver } from "./links.mjs";
@@ -323,6 +323,15 @@ const lfs = llmsFullTxt({ entries: llmsEntries, site, categories: cats, bodies:
 has("full lists each pick", lfs, "- Tiles: Alpha (https://x.test/s/entry/a1/); CC0; commercial OK; no credit required; active");
 has("full lists credits", lfs, "Credits to ship: none");
 
+/* deferred minors, batch A ------------------------------------------------- */
+eq("deprecation reason is capitalised", deprecationReason("- Deprecated: the service is offline"), "The service is offline");
+eq("bare url in bold", renderInline("**https://a.test/x**", passLink), '<strong><a href="https://a.test/x" rel="noopener noreferrer">https://a.test/x</a></strong>');
+eq("bare url keeps a balanced paren", renderInline("see https://a.test/wiki/Foo_(bar).", passLink), 'see <a href="https://a.test/wiki/Foo_(bar)" rel="noopener noreferrer">https://a.test/wiki/Foo_(bar)</a>.');
+eq("angle-bracket autolink", renderInline("<https://a.test/x>", passLink), '<a href="https://a.test/x" rel="noopener noreferrer">https://a.test/x</a>');
+eq("bare url stops at a code span", renderInline("https://a.test/x`y`", passLink), '<a href="https://a.test/x" rel="noopener noreferrer">https://a.test/x</a><code>y</code>');
+throws("malformed escape is a link error", () => resolve("bad%E0.md"), 'link target "bad%E0.md" is malformed');
+eq("query on a relative link is kept", resolve("../../docs/provenance.md?plain=1").href, "https://github.com/o/r/blob/main/docs/provenance.md?plain=1");
+
 /* report (keep last) ------------------------------------------------------ */
 if (failures.length) {
   console.error(`lib.test failed (${failures.length}):`);

site/lib/links.mjs

@@ -8,10 +8,17 @@ export function makeLinkResolver({ entryPath, idByPath, repo, kindOf, entryBase
   return (raw) => {
     if (/^(https?:|mailto:)/i.test(raw)) return { href: raw, external: true };
     if (raw.startsWith("#")) return { href: raw, external: false };
-    const at = raw.indexOf("#");
-    const target = at === -1 ? raw : raw.slice(0, at);
-    const hash = at === -1 ? "" : raw.slice(at);
-    const repoPath = path.posix.normalize(path.posix.join(dir, decodeURI(target))).replace(/\/$/, "");
+    // Split off ?query and #hash; both are kept on the rewritten link.
+    const m = raw.match(/^([^?#]*)(\?[^#]*)?(#.*)?$/);
+    const target = m[1];
+    const hash = `${m[2] || ""}${m[3] || ""}`;
+    let decoded;
+    try {
+      decoded = decodeURI(target);
+    } catch {
+      throw new LinkError(`${entryPath}: link target "${raw}" is malformed`);
+    }
+    const repoPath = path.posix.normalize(path.posix.join(dir, decoded)).replace(/\/$/, "");
     const id = idByPath.get(repoPath);
     if (id) return { href: `${entryBase}${id}/${hash}`, external: false };
     const kind = repoPath.startsWith("..") ? null : kindOf(repoPath);

site/lib/markdown.mjs

@@ -35,9 +35,18 @@ export function renderInline(text, resolveLink) {
     const rel = external ? ' rel="noopener noreferrer"' : "";
     return keep(`<a href="${esc(out)}"${rel}>${marks(esc(label))}</a>`);
   });
-  // Bare URLs, leaving trailing sentence punctuation outside the link.
-  s = s.replace(/https?:\/\/[^\s<>"]+/g, (url) => {
-    const [, core, tail] = url.match(/^(.*?)([.,;:!?)\]]*)$/);
+  // Autolinks in angle brackets.
+  s = s.replace(/<(https?:\/\/[^\s<>]+)>/g, (_, url) => keep(`<a href="${esc(url)}" rel="noopener noreferrer">${esc(url)}</a>`));
+  // Bare URLs, leaving trailing sentence punctuation outside the link. A URL
+  // stops at `*` (bold around it) and at a slot (a code span right after it),
+  // and keeps a closing paren that balances one inside it (Foo_(bar)).
+  s = s.replace(/https?:\/\/[^\s<>"*\u0000]+/g, (url) => {
+    let [, core, tail] = url.match(/^(.*?)([.,;:!?)\]]*)$/);
+    const count = (str, ch) => str.split(ch).length - 1;
+    while (tail.startsWith(")") && count(core, "(") > count(core, ")")) {
+      core += ")";
+      tail = tail.slice(1);
+    }
     return keep(`<a href="${esc(core)}" rel="noopener noreferrer">${esc(core)}</a>`) + tail;
   });
   s = marks(esc(s));
@@ -161,5 +170,8 @@ export function sectionMarkdown(body, heading) {
 /** The text after the `- Deprecated:` marker line, or null. */
 export function deprecationReason(body) {
   const m = String(body).match(/^- Deprecated:\s*(.+)$/m);
-  return m ? m[1].trim() : null;
+  if (!m) return null;
+  // The banner reads "Deprecated. <reason>", so the reason starts a sentence.
+  const reason = m[1].trim();
+  return reason.charAt(0).toUpperCase() + reason.slice(1);
 }