Replace CodeGlyphX website with PowerForge.Web#73
Conversation
|
Claude finished @PrzemyslawKlys's task —— View job Code Review Complete ✅I've completed a comprehensive review of the PowerForge.Web migration. This is a substantial infrastructure change that replaces the legacy website system with a modern build pipeline. Review Checklist:
🎯 SummaryOverall Assessment: This is a well-executed infrastructure modernization. The migration from legacy build scripts to PowerForge.Web significantly simplifies the build pipeline and improves maintainability. ✅ Strengths
|
There was a problem hiding this comment.
Pull request overview
This PR replaces the legacy CodeGlyphX website infrastructure with the new PowerForge.Web system, consolidating website content under a unified Website/ directory structure and routing API documentation to /api.
Changes:
- Remove generated API documentation outputs (JSON/HTML) from
CodeGlyphX.Website/wwwroot/api/types/ - Remove legacy website build and deployment scripts from
Build/directory - Clean up obsolete infrastructure supporting the old website architecture
Reviewed changes
Copilot reviewed 52 out of 961 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| CodeGlyphX.Website/wwwroot/api/types/*.json | Removal of generated API type metadata files |
| CodeGlyphX.Website/wwwroot/api/types/*.html | Removal of generated API documentation HTML pages |
| Build/website-smoke-tests.js | Removal of Node.js-based website smoke test suite |
| Build/Verify-ApiDataSources.ps1 | Removal of API data source verification script |
| Build/Update-StyleBoard.ps1 | Removal of style board asset generation script |
| Build/Update-NavFragments.ps1 | Removal of navigation fragment update script |
| Build/Test-MobileLayout.ps1 | Removal of mobile layout testing script |
| Build/Run-Website.ps1 | Removal of legacy website development server script |
| Build/Publish-WebsitePages.ps1 | Removal of static page publication script |
| Build/Generate-Sitemap.ps1 | Removal of sitemap generation script |
| Build/Generate-Llms.ps1 | Removal of LLM context file generation script |
| Build/Build-Website.ps1 | Removal of master website build orchestration script |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ca0d23bd6e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "task": "overlay", | ||
| "source": "./_temp/playground/wwwroot", | ||
| "destination": "./_site/playground", | ||
| "clean": true, | ||
| "include": "**/*, *" |
There was a problem hiding this comment.
This overlay only ships the published wwwroot into /playground, but the generated wwwroot/api output was removed in this change, so the published app no longer contains api/faq.json or api/showcase.json. The Blazor pages still request these files (e.g., Http.GetFromJsonAsync("api/faq.json") in CodeGlyphX.Website/Pages/Faq.razor and api/showcase.json in Showcase.razor), so navigating to /playground/faq or /playground/showcase will 404 the data and leave the pages stuck in their loading state. Consider copying Assets/Data into /playground/api during the pipeline or updating the app to read from the new data location.
Useful? React with 👍 / 👎.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #73 +/- ##
==========================================
- Coverage 59.75% 59.75% -0.01%
==========================================
Files 448 448
Lines 57814 57814
Branches 16563 16563
==========================================
- Hits 34547 34545 -2
- Misses 17564 17565 +1
- Partials 5703 5704 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Updates pushed:
|
|
Summary
Testing