fix: polish remaining review findings (8-16)

- Update stale /bmad:core:tools:shard-doc command to /bmad-shard-doc
- Fix 100vw scrollbar overflow: banner uses width:100%, html gets overflow-x:clip
- Add :focus-visible ring to banner link for keyboard navigation
- Use file.path instead of file.history[0] in rehype-markdown-links
- Fix incomplete sentence in install-bmad.md
- Drop unnecessary fetch-depth:0 from docs deploy workflow

Co-Authored-By: Claude Opus 4.6 <[email protected]>

Author: alexeyv

.github/workflows/docs.yaml

@@ -27,8 +27,6 @@ jobs:
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
 
       - name: Setup Node.js
         uses: actions/setup-node@v4

docs/how-to/install-bmad.md

@@ -7,7 +7,7 @@ sidebar:
 
 Use the `npx bmad-method install` command to set up BMad in your project with your choice of modules and AI tools.
 
-If you want to use a non interactive installer and provide all install options on the command line, [this guide](../non-interactive-installation.md).
+If you want to use a non interactive installer and provide all install options on the command line, see [this guide](../non-interactive-installation.md).
 
 ## When to Use This
 

docs/how-to/shard-large-documents.md

@@ -39,7 +39,7 @@ docs/
 ### 1. Run the Shard-Doc Tool
 
 ```bash
-/bmad:core:tools:shard-doc
+/bmad-shard-doc
 ```
 
 ### 2. Follow the Interactive Process

test/test-rehype-plugins.mjs

@@ -53,7 +53,7 @@ function assert(condition, testName, errorMessage = '') {
 // ---------------------------------------------------------------------------
 
 const CONTENT_DIR = '/project/src/content/docs';
-const STD_FILE = { history: ['/project/src/content/docs/guide/intro.md'] };
+const STD_FILE = { path: '/project/src/content/docs/guide/intro.md' };
 const STD_OPTS = { contentDir: CONTENT_DIR };
 const BASE = '/BMAD-METHOD/';
 
@@ -350,17 +350,17 @@ function runTests() {
   }
 
   {
-    // No file history -> no processing
+    // No file path -> no processing
     const tree = makeAnchorTree('sibling.md');
-    transform(tree, { history: undefined }, STD_OPTS);
-    assert(getHref(tree) === 'sibling.md', 'No file history -> no processing', `Got ${getHref(tree)}`);
+    transform(tree, { path: undefined }, STD_OPTS);
+    assert(getHref(tree) === 'sibling.md', 'No file path -> no processing', `Got ${getHref(tree)}`);
   }
 
   {
-    // Empty file history array -> no processing
+    // Empty string path -> no processing
     const tree = makeAnchorTree('sibling.md');
-    transform(tree, { history: [] }, STD_OPTS);
-    assert(getHref(tree) === 'sibling.md', 'Empty file history array -> no processing', `Got ${getHref(tree)}`);
+    transform(tree, { path: '' }, STD_OPTS);
+    assert(getHref(tree) === 'sibling.md', 'Empty string path -> no processing', `Got ${getHref(tree)}`);
   }
 
   {
@@ -387,7 +387,7 @@ function runTests() {
   {
     // No content dir + no contentDir option -> throws
     const tree = makeAnchorTree('sibling.md');
-    const file = { history: ['/some/random/path/file.md'] };
+    const file = { path: '/some/random/path/file.md' };
     let threw = false;
     let errorMsg = '';
     try {
@@ -431,7 +431,7 @@ function runTests() {
   {
     // Use a file two levels deep so ../../ still stays inside content root
     const deepFile = {
-      history: ['/project/src/content/docs/guide/sub/intro.md'],
+      path: '/project/src/content/docs/guide/sub/intro.md',
     };
     const tree = makeAnchorTree('../../root-level.md');
     transform(tree, deepFile, STD_OPTS);
@@ -484,7 +484,7 @@ function runTests() {
   {
     // Root index.md: file at content root
     const rootFile = {
-      history: ['/project/src/content/docs/intro.md'],
+      path: '/project/src/content/docs/intro.md',
     };
     const tree = makeAnchorTree('index.md');
     transform(tree, rootFile, STD_OPTS);

website/src/components/Banner.astro

@@ -10,8 +10,7 @@ const llmsFullUrl = `${getSiteUrl()}/llms-full.txt`;
 
 <style>
   .ai-banner {
-    width: 100vw; /* Full viewport width */
-    margin-left: calc(-50vw + 50%); /* Center and break out of container */
+    width: 100%;
     height: var(--ai-banner-height, 2.75rem);
     background: #334155;
     color: #cbd5e1;
@@ -40,6 +39,11 @@ const llmsFullUrl = `${getSiteUrl()}/llms-full.txt`;
   .ai-banner a:hover {
     text-decoration: underline;
   }
+  .ai-banner a:focus-visible {
+    outline: 2px solid #B9B9FF;
+    outline-offset: 2px;
+    border-radius: 2px;
+  }
 
   /* Match navbar padding at breakpoints */
   @media (min-width: 50rem) {

website/src/rehype-markdown-links.js

@@ -26,7 +26,7 @@ export default function rehypeMarkdownLinks(options = {}) {
 
   return (tree, file) => {
     // The current file's absolute path on disk, set by Astro's markdown pipeline
-    const currentFilePath = file.history?.[0];
+    const currentFilePath = file.path;
     if (!currentFilePath) return;
 
     // Auto-detect content root: walk up from current file to find src/content/docs

website/src/styles/custom.css

@@ -339,6 +339,11 @@ button:hover {
    MISC ENHANCEMENTS
    ============================================ */
 
+/* Prevent horizontal scrollbar from full-viewport-width breakout elements (e.g. iframe) */
+html {
+  overflow-x: clip;
+}
+
 /* Smooth scrolling */
 @media (prefers-reduced-motion: no-preference) {
   html {