fix: address 5 Copilot comments on PR #6

1. CEL link: separate external cel.dev link from internal expressions guide
2. status_code → status in server-guide (matches http.go output)
3. sanitizeExcerpt: recursive DOM walk preserves nested <mark> tags
4. Search race condition: requestId counter discards stale responses
5. Bottom padding: moved to global CSS @media (max-width: 1023px) body
   so all pages (including 404) clear the mobile bottom nav

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: michaelmcnees

site/src/components/SearchModal.astro

@@ -68,28 +68,28 @@
     if (modal) modal.classList.add('hidden');
   }
 
-  // Sanitize Pagefind excerpt: strip all HTML except <mark> tags
+  // Recursively sanitize Pagefind excerpt: preserve only <mark> tags, strip everything else
   function sanitizeExcerpt(html) {
-    var tmp = document.createElement('div');
-    tmp.textContent = '';
-    // Parse into a temporary element to strip tags
     var parser = new DOMParser();
     var doc = parser.parseFromString(html, 'text/html');
-    var walker = doc.body.childNodes;
-    for (var i = 0; i < walker.length; i++) {
-      var node = walker[i];
-      if (node.nodeType === 3) {
-        tmp.appendChild(document.createTextNode(node.textContent));
-      } else if (node.nodeType === 1 && node.tagName === 'MARK') {
-        var mark = document.createElement('mark');
-        mark.setAttribute('data-pagefind-highlight', '');
-        mark.textContent = node.textContent;
-        tmp.appendChild(mark);
-      } else {
-        tmp.appendChild(document.createTextNode(node.textContent || ''));
+    var container = document.createElement('div');
+    function walk(parent, target) {
+      for (var i = 0; i < parent.childNodes.length; i++) {
+        var node = parent.childNodes[i];
+        if (node.nodeType === 3) {
+          target.appendChild(document.createTextNode(node.textContent));
+        } else if (node.nodeType === 1 && node.tagName === 'MARK') {
+          var mark = document.createElement('mark');
+          mark.setAttribute('data-pagefind-highlight', '');
+          walk(node, mark);
+          target.appendChild(mark);
+        } else if (node.nodeType === 1) {
+          walk(node, target);
+        }
       }
     }
-    return tmp;
+    walk(doc.body, container);
+    return container;
   }
 
   document.addEventListener('keydown', function(e) {
@@ -109,6 +109,7 @@
   });
 
   var debounceTimer;
+  var searchRequestId = 0;
   document.addEventListener('input', function(e) {
     if (!e.target || !e.target.hasAttribute('data-search-input')) return;
     clearTimeout(debounceTimer);
@@ -123,20 +124,24 @@
         return;
       }
 
+      var thisRequestId = ++searchRequestId;
+
       loadPagefind().then(function(pf) {
         if (!pf) {
           setResultsText('Search unavailable (build required)');
           return;
         }
 
         pf.search(query).then(function(search) {
+          if (thisRequestId !== searchRequestId) return; // stale response
           if (search.results.length === 0) {
             setResultsText('No results for "' + query + '"');
             return;
           }
 
           Promise.all(search.results.slice(0, 8).map(function(r) { return r.data(); }))
             .then(function(items) {
+              if (thisRequestId !== searchRequestId) return; // stale response
               while (results.firstChild) results.removeChild(results.firstChild);
               items.forEach(function(item) {
                 var link = document.createElement('a');

site/src/content/docs/getting-started/data-passing.md

@@ -44,7 +44,7 @@ Execution b2c3d4e5-f6a7-8901-bcde-f12345678901: completed
 
 ## CEL Expression Syntax
 
-Mantle uses [CEL (Common Expression Language)](/docs/concepts/expressions) for data passing and conditional logic. The essentials:
+Mantle uses [CEL (Common Expression Language)](https://cel.dev) for data passing and conditional logic. See the [Expressions guide](/docs/concepts/expressions) for the full reference. The essentials:
 
 - **Access step output:** `steps['step-name'].output.json.field`
 - **Access inputs:** `inputs.field_name`

site/src/content/docs/server-guide/index.md

@@ -66,12 +66,12 @@ steps:
 
   - name: alert-on-failure
     action: http/request
-    if: "steps['check-api'].output.status_code != 200"
+    if: "steps['check-api'].output.status != 200"
     params:
       method: POST
       url: https://hooks.slack.com/services/T00/B00/xxx
       body:
-        text: "API health check failed with status {{ steps.check-api.output.status_code }}"
+        text: "API health check failed with status {{ steps.check-api.output.status }}"
 ```
 
 Apply the workflow and start the server:

site/src/layouts/Docs.astro

@@ -25,7 +25,7 @@ const { prev, next } = getPrevNext(currentPath);
 <Base title={`${title} - Mantle Docs`}>
   <Nav />
   <div class="mx-auto max-w-[1280px] px-4 sm:px-6 overflow-x-hidden">
-    <div class="flex min-h-screen pt-16 pb-16 md:pb-0">
+    <div class="flex min-h-screen pt-16">
       <DocsSidebar currentPath={currentPath} />
 
       <!-- Main content -->

site/src/pages/404.astro

@@ -4,7 +4,7 @@ import Nav from '../components/Nav.astro';
 ---
 <Base title="404 — Route Not Found | Mantle">
   <Nav />
-  <main class="min-h-screen flex items-center justify-center pt-16">
+  <main class="min-h-screen flex items-center justify-center pt-16" id="main-content">
     <div class="mx-auto max-w-[700px] px-6 py-24">
       <h1 class="font-headline text-4xl sm:text-5xl font-bold tracking-tight mb-8">
         404: Route Not Found

site/src/pages/index.astro

@@ -13,7 +13,7 @@ import Footer from '../components/Footer.astro';
 <Base title="Mantle — Headless AI Workflow Automation">
   <div data-pagefind-ignore>
   <Nav isHome={true} />
-  <main id="main-content" class="pb-16 md:pb-0">
+  <main id="main-content">
     <Hero />
     <Features />
     <HowItWorks />

site/src/styles/global.css

@@ -100,6 +100,13 @@ body {
   color: var(--color-on-primary);
 }
 
+/* Bottom padding for fixed mobile nav — applied globally so all pages clear it */
+@media (max-width: 1023px) {
+  body {
+    padding-bottom: calc(3.5rem + env(safe-area-inset-bottom, 0px));
+  }
+}
+
 /* Blinking cursor animation */
 @keyframes blink {
   0%, 50% { opacity: 1; }