A GitHub pages site to host the AIPs#662
Open
gregnazario wants to merge 19 commits intoaptos-foundation:mainfrom
Open
A GitHub pages site to host the AIPs#662gregnazario wants to merge 19 commits intoaptos-foundation:mainfrom
gregnazario wants to merge 19 commits intoaptos-foundation:mainfrom
Conversation
- Initialize Starlight/Astro documentation site - Preserve all AIPs in original /aips/ directory via symlink - Add interactive browse page with filters for status and category - Add search functionality via Pagefind - Configure GitHub Actions workflow for GitHub Pages deployment - Add custom homepage, submission guide, and theme styling - Normalize AIP metadata (status, type/category) for consistent filtering - Rewrite image paths to work with /diagrams symlink
- Convert quoted comma-separated strings to proper YAML format - Use simple string format for single requirements - Use YAML array format for multiple requirements - Fixes YAMLException during site build
- Add preserveSymlinks: true to Vite resolve config - Allow serving files from parent directories for symlinked content - Fixes UnknownContentCollectionError with symlinked AIPs
- Add .nullable() to all optional fields to handle empty YAML values - Fixes InvalidContentEntryFrontmatterError for AIPs with empty frontmatter fields - Handles cases where YAML parses empty values as null
- Remove inline YAML comment from Status field - Comments in YAML values cause parsing errors
- Change 'requires: 62' to 'requires: AIP-62' for consistency
- Change 'aip: AIP-121' to 'aip: 121' (number format) - Remove '(*optional)' text from discussions-to field name
- Add scripts/validate-frontmatter.mjs for zero-dep AIP frontmatter validation - Add CI workflow (.github/workflows/aip-frontmatter.yml) to run on PRs - Add "validate" npm script - Strip (*optional) from frontmatter keys (broke js-yaml parser) - Fix markdown link in aip-096 requires field (YAML flow sequence conflict) - Accept numeric requires values in content schema (aip-053) - Add slug frontmatter to all 142 AIPs for number-based URLs (/aips/61/) - Generate zero-padded redirects in astro config (/aips/001/ → /aips/1/) - Fix data.json.ts to use e.id instead of deprecated e.slug (Astro 6)
Contributor
There was a problem hiding this comment.
Copilot reviewed 159 out of 165 changed files in this pull request and generated no comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
- robots.txt with sitemap reference - llms.txt and llms-full.txt endpoints (llmstxt.org standard) - RSS feed at /aips/feed.xml - OpenGraph and Twitter Card meta tags via Starlight head config - PR preview deployments with rossjrw/pr-preview-action - PREVIEW_BASE env var support for preview base paths
…t link - Custom Head component override with Schema.org TechArticle per AIP page - Static OG image (1200x630 PNG) with Aptos brand colors - Auto-generated meta descriptions from first paragraph of AIP body - <link> tag for llms.txt discoverability in <head> - Canonical URLs verified correct in CI builds
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.
Add standardized formatting and frontmatter for AIPs.
Add website for GitHub pages documentation so you can see all the AIPs with a more standardized (and SEO'd) website.
Example https://gregnazario.github.io/AIPs/