docs: correct frontend drift (Blazor is the deployed UI) + add TestSupport#52
Merged
Conversation
…pport AGENTS.md/README.md still called Blazor a "WIP scaffold, not in slnx, not built by CI" and the wwwroot SPA the production demo UI. Reality (merged #48): Paperless.slnx contains PaperlessUI.Blazor + a paperless-blazor compose service with a Dockerfile, and nginx routes / to the Blazor app (Interactive Server, SignalR). Blazor is the deployed demo UI — a vanilla port of the wwwroot SPA that replaced it at the root (wwwroot still ships via UseStaticFiles). Also add Paperless.TestSupport (#49) to the layout and fix the stale "scaffold-only, re-add to slnx" comment in ci.yml. Remove the completed superpowers plan/spec docs for the now-merged Blazor and test-infrastructure work — the merged PRs + code are the record. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Aligns repository documentation and CI commentary with the current “ground truth” after recent merges: Blazor (Interactive Server) is now the deployed demo UI behind nginx /, Paperless.TestSupport/ exists as shared test infrastructure, and the old PaperlessREST/wwwroot SPA is no longer the nginx root (but still ships via UseStaticFiles()).
Changes:
- Update
README.mdandAGENTS.mdto reflect the Blazor demo UI deployment path, solution membership, and CI build behavior; addPaperless.TestSupport/to the documented repo layout. - Refresh the CI workflow comment to remove the outdated “Blazor scaffold-only” guidance (no functional workflow changes).
- Remove now-obsolete “superpowers” plan/spec artifacts for already-merged work.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Corrects UI/CI/docs drift: Blazor is the demo UI served at / behind nginx; wwwroot is legacy and still served by REST via UseStaticFiles; adds Paperless.TestSupport to layout. |
| AGENTS.md | Updates repo guide to match current UI deployment and build/test layout (Blazor in Paperless.slnx, compiled by backend CI; React/Angular built separately). |
| .github/workflows/ci.yml | Replaces a stale comment about Blazor being scaffold-only with an accurate note that it’s built via the NUKE pipeline. |
| docs/superpowers/specs/2026-06-04-test-infrastructure-uplift-design.md | Deletes completed/obsolete design spec artifact for merged test-infra uplift work. |
| docs/superpowers/specs/2026-06-04-blazor-document-pipeline-design.md | Deletes completed/obsolete design spec artifact for merged Blazor work. |
| docs/superpowers/plans/2026-06-04-blazor-document-pipeline.md | Deletes completed/obsolete implementation plan artifact for merged Blazor work. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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.
What
Truth-drift pass on the repo docs after this session's merged work (#48 Blazor pipeline, #49–#51 test infra).
wwwroot/SPA as the production demo UI. Both are now wrong — corrected across intro, layout, build/test, CI, the mermaid diagram, the Rating-Matrix row, and the stack table.Paperless.TestSupport/(shared test lib from test: shared TestSupport infrastructure (template-method base + builders + RAII cleanup) #49).Why it's accurate now (verified against ground truth)
Paperless.slnxcontainsPaperlessUI.Blazor.csproj→ compiled by the backend NUKE/CI job (saw it in./build.sh Compileoutput).compose.yamlhas apaperless-blazorservice built fromPaperlessUI.Blazor/Dockerfile.docker/nginx.conf:location /→proxy_pass http://paperless-blazor(Interactive Server + SignalR upgrade). No nginx location serves the wwwroot SPA at/anymore.PaperlessRESTstill callsUseStaticFiles(), sowwwroot/still ships — it's the original SPA Blazor ported from, not the deployed demo root.Verification
Docs + one YAML comment + doc deletions — no code/test surface touched.
ci.ymlre-parses as valid YAML.CLAUDE.mdis a symlink toAGENTS.md, so it's updated transitively.🤖 Generated with Claude Code