feat(docs): implement 10 Astro feature items from PAO audit#524
Merged
bradygaster merged 1 commit intobradygaster:devfrom Mar 23, 2026
Merged
feat(docs): implement 10 Astro feature items from PAO audit#524bradygaster merged 1 commit intobradygaster:devfrom
bradygaster merged 1 commit intobradygaster:devfrom
Conversation
1. @astrojs/sitemap — add integration, sitemap-index.xml generated on build 2. Code block copy button — global JS in BaseLayout, clipboard API, Tailwind styled 3. Twitter Card — change to summary_large_image in BaseLayout 4. Edit this page — GitHub edit link in DocsLayout footer 5. robots.txt — create docs/public/robots.txt with Allow: /, Sitemap ref 6. Table of Contents — TableOfContents.astro client-side component, right column in DocsLayout 7. @astrojs/rss — install package, create src/pages/rss.xml.js from blog collection 8. Next/Prev navigation — compute from NAV_SECTIONS in [...slug].astro, render in DocsLayout 9. Richer frontmatter — add tags, author, updatedAt, status to docs + blog schemas; display updatedAt in DocsLayout 10. View Transitions — import ViewTransitions from astro:transitions in BaseLayout Tests: docs/tests/astro-features.test.mjs — 19/19 pass (source + build-output checks) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Collaborator
Author
|
@bradygaster - all 10 PAO audit items implemented and tested. 19/19 new tests pass, 13/13 existing tests still green. Build clean at 138 pages. Ready for review. |
chrislomonico
pushed a commit
to clomonico/squad
that referenced
this pull request
Mar 26, 2026
…r#546) Consolidated 15 remaining issues into a single PR. Redesigns help UX with grouped categories, surfaces all /help commands, aligns docs with actual CLI behavior, adds first-run welcome, standardizes naming. Closes bradygaster#510, Closes bradygaster#513, Closes bradygaster#514, Closes bradygaster#515, Closes bradygaster#516, Closes bradygaster#517, Closes bradygaster#518, Closes bradygaster#521, Closes bradygaster#522, Closes bradygaster#523, Closes bradygaster#524, Closes bradygaster#525, Closes bradygaster#527, Closes bradygaster#528, Closes bradygaster#529 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements all 10 items from PAO's docs site audit. ALL changes are inside docs/.
Working as EECOM (Core Dev)
Changes
Tests
docs/tests/astro-features.test.mjs - 19/19 pass
Existing build-output.test.mjs - 13/13 pass
Build: astro build succeeded, 138 pages + sitemap-index.xml + rss.xml generated.
@bradygaster - tagging you for review. Requested by Dina.