Add versioned docs infrastructure and fix broken site URL#34
Add versioned docs infrastructure and fix broken site URL#34AnExiledDev merged 2 commits intomainfrom
Conversation
Install starlight-versions plugin (commented out until first version archive) and configure the versions content collection. Fix site URL to codeforge.core-directive.com and remove /CodeForge base path, updating all internal links and meta tag references.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdates for the docs site: switched the site domain to https://codeforge.core-directive.com and removed the /CodeForge base path, added the starlight-versions dependency and loader to enable versioned docs, adjusted asset and internal links, and made minor component import/structure tweaks. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.devcontainer/CHANGELOG.md (1)
137-139: Merge duplicate### Fixedsections in the Unreleased block.This entry is currently under a second
### Fixedheading in the same release section; consolidating into one### Fixedblock will keep the changelog easier to scan.As per coding guidelines: “Group changelog entries under appropriate headings:
### Added,### Changed,### Fixed, or### Removed.”🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.devcontainer/CHANGELOG.md around lines 137 - 139, In the Unreleased changelog block there are two separate "### Fixed" headings; consolidate them into a single "### Fixed" section by merging the entries from the duplicate heading into the existing "### Fixed" block (remove the extra "### Fixed" header and any duplicate blank lines), ensure all fixed items are listed under that one heading and preserve their order and formatting so the changelog complies with the Added/Changed/Fixed/Removed grouping guideline.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.devcontainer/CHANGELOG.md:
- Around line 137-139: In the Unreleased changelog block there are two separate
"### Fixed" headings; consolidate them into a single "### Fixed" section by
merging the entries from the duplicate heading into the existing "### Fixed"
block (remove the extra "### Fixed" header and any duplicate blank lines),
ensure all fixed items are listed under that one heading and preserve their
order and formatting so the changelog complies with the
Added/Changed/Fixed/Removed grouping guideline.
ℹ️ Review info
Configuration used: defaults
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
docs/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (8)
.devcontainer/CHANGELOG.mddocs/astro.config.mjsdocs/package.jsondocs/src/components/Header.astrodocs/src/components/Hero.astrodocs/src/content.config.tsdocs/src/content/docs/index.mdxdocs/src/content/docs/reference/changelog.md
Merge three separate ### Fixed headings into one to comply with Keep a Changelog grouping conventions. Addresses CodeRabbit review.
Summary
codeforge.core-directive.comhas broken styling/navigation becausebase: "/CodeForge"causes all CSS, JS, and link paths to 404 on the custom domain. Removedbaseand updatedsitetohttps://codeforge.core-directive.comstarlight-versionsplugin and configured theversionscontent collection. Plugin is commented out (it requires ≥1 archived version); activation instructions are inline for when v3 development begins/CodeForge/internal paths acrossastro.config.mjshead meta,Hero.astro,Header.astro, andindex.mdxTest plan
npm run buildpasses (40 pages, 0 errors)starlight-links-validatorreports all internal links validcodeforge.core-directive.comloads with proper stylingSummary by CodeRabbit
New Features
Bug Fixes
Documentation