fix(docs-site): migrate to astro v6 content collections config#6451
Conversation
Astro v6 removed legacy content collections. 'astro check'/'astro build' failed with LegacyContentConfigError because the config lived at src/content/config.ts without a collection loader. - Move src/content/config.ts -> src/content.config.ts - Add Starlight docsLoader() to the docs collection Verified: 'npm run build' (astro check && astro build) now completes.
There was a problem hiding this comment.
Pull request overview
Migrates the docs site’s content collection configuration to Astro v6 conventions.
Changes:
- Relocates the content configuration to
src/content.config.ts. - Configures Starlight’s
docsLoader()for the docs collection.
Show a summary per file
| File | Description |
|---|---|
docs-site/src/content.config.ts |
Adds the required Starlight loader to the docs collection. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Medium
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
|
@copilot merge main |
|
✅ Copilot review passed with no inline comments. @lpcox Add the |
Adds a brace-expansion >=5.0.7 override to resolve GHSA-3jxr-9vmj-r5cp (DoS via exponential-time expansion of consecutive non-expanding {} groups), which was failing the 'Audit Main Package' npm audit --audit-level=high gate. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 7ef9f0e7-1882-478d-90c0-6e4d42313685
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🔑 Smoke Copilot PAT PAT auth validated. All systems operational. ✅ |
|
✅ Smoke Gemini completed. All facets verified. 💎 |
|
✅ Build Test Suite completed successfully! |
|
📡 Smoke OTel Tracing completed. All tracing scenarios validated. ✅ |
|
❌ Smoke Copilot BYOK AOAI (Entra) reports failed. AOAI BYOK (Entra) mode investigation needed... |
|
✅ Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓 |
|
🚀 Security Guard has started processing this pull request |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
🔌 Smoke Services — All services reachable! ✅ |
|
✅ Smoke Claude passed |
|
📰 VERDICT: Smoke Docker Sbx has concluded. All systems operational. This is a developing story. 🎤 |
|
🛡️ Smoke Copilot Network Isolation confirmed the egress allowlist is enforced. ✅ |
|
Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded. |
|
✅ Contribution Check completed successfully! PR #6451 follows the relevant contribution guidelines: the PR description is clear and references #6450, the change is in the correct docs-site location, and no additional docs or tests are required for this Astro v6 content-collections migration. |
Smoke Test: Claude Engine Validation
Overall result: PASS ✅
|
🔬 Smoke Test: Copilot PAT Auth — PR #6451
Overall: Auth mode: PAT (COPILOT_GITHUB_TOKEN)
|
Smoke Test Results — Services Connectivity
Overall: FAIL —
|
|
@lpcox — Smoke test results for this PR: EGRESS_RESULT allow=pass deny=pass ✅ Allowed domain ( Overall: PASS Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "example.com"See Network Configuration for more information.
|
github-actions
Bot
added
the
smoke-copilot-network-isolation
|
Merged PRs reviewed:
Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "registry.npmjs.org"See Network Configuration for more information.
|
|
Smoke Test Results: Copilot BYOK (Direct) Mode ✅ GitHub.com connectivity (HTTP 200) Status: PASS | Direct BYOK mode (COPILOT_PROVIDER_API_KEY) operational
|
🔍 Smoke Test: API Proxy OpenTelemetry Tracing Results
All validatable scenarios pass. OTEL integration is functional with correct span lifecycle, GenAI semantic conventions support, parent context propagation via
|
|
Merged PRs:
Results:
Running in direct BYOK mode (COPILOT_PROVIDER_API_KEY + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw) Overall: FAIL cc @lpcox
|
Gemini Smoke Test Results
Overall Status: FAIL Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "localhost"See Network Configuration for more information.
|
Chroot Version Comparison Results ✅
All runtimes match between host and chroot environments.
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
🔥 Smoke Test: Docker Sbx — PASS
Overall: PASS
|
What
Completes the Astro v6 content-collections migration for the docs site.
src/content/config.ts→src/content.config.tsdocsLoader()to thedocscollectionWhy
Astro v6 removed legacy content collections. On
main,npm run build(astro check && astro build) fails with:The config file was at the old path and had no collection loader.
Verification
npm run build(astro check + astro build) now completes successfully — 9 pages built, search index generated.Split out from the docs-site npm audit fix (#6450) since it's an independent, pre-existing build issue.