Repository navigation
feat(web): static Next.js report page and touchstone serve (#9) - #28
Merged
Merged
Conversation
Closed
3 tasks
Co-authored-by: hsinatfootprintai <149764663+hsinatfootprintai@users.noreply.github.com>
…l-scanner fix (#9) The revise run rewrote the web side from a clean main: it left two copies of the report and scan-page components, broke vitest.config.ts (the golden report path, so 6 of 7 test files failed), and changed the pinned dependency versions. Restore web/, Makefile and .gitignore to the previous head, and keep the Go side of the revision: main.go and serve.go wire the real service.Scanner (serve no longer uses the stub), with the serve tests and web/embed.go that serve depends on. go vet, go test ./..., tsc and vitest (5 files, 42 tests) pass on this tree. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
5 tasks
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.
For #9:
[S9] Web page: Next.js static export with URL input and report view, served by `touchstone serve`What and why
The platform made the commits and the branch from the files the agent handed back; it does not use the agent's own git state on the box, so anything the summary says about committing, branching or pushing describes only what the agent could do there.
The agent's summary, quoted:
The agent's decisions, and where it departed from the issue or the design, quoted:
Changes
.gitignore— 6 linesMakefile— 29 linescmd/touchstone/main.go— 98 linescmd/touchstone/serve.go— 73 linescmd/touchstone/serve_test.go— 37 linesinternal/server/server.go— 30 linesinternal/server/server_test.go— 79 linesweb/embed.go— 18 linesweb/next.config.mjs— 8 linesweb/package.json— 28 linesweb/postcss.config.mjs— 7 linesweb/out/.gitkeepweb/src/app/globals.css— 1 linesweb/src/app/layout.tsx— 24 linesweb/src/app/page.tsx— 5 linesweb/src/components/component-card.tsx— 87 linesweb/src/components/report-view.test.tsx— 128 linesweb/src/components/report-view.tsx— 146 linesweb/src/components/scan-form.tsx— 53 linesweb/src/components/scan-page.test.tsx— 96 linesweb/src/components/scan-page.tsx— 54 linesweb/src/lib/api.test.ts— 63 linesweb/src/lib/api.ts— 66 linesweb/src/lib/report.ts— 52 linesweb/src/lib/url.test.ts— 18 linesweb/src/lib/url.ts— 9 linesweb/src/test/golden.ts— 22 linesweb/src/report.test.ts— 27 linesweb/tsconfig.json— 16 linesweb/vitest.config.ts— 13 lines30 file(s) in all.
Acceptance, from the issue
`web/` is a Next.js static export with Tailwind, `strict` + `noUncheckedIndexedAccess`; responses are decoded with the generated `fromJson(ScanReportSchema, …)…— the agent says:output: export, Tailwind, strict and noUncheckedIndexedAccess set; fromJson in api.ts. One cast remains in a test file.The page shows components grouped by confidence tier, ranked CVEs per component, evidence per finding, the checked-locations list, and a "lookup incomplete" ba…— the agent says:Tier grouping, ranked CVEs, evidence, checked list and banner in report-view.tsx; 42 Vitest tests pass.`touchstone serve` (default `127.0.0.1:8080`) serves the embedded page and `POST /v1/scans` through grpc-gateway— the agent says:Default 127.0.0.1:8080; /v1/ via grpc-gateway plus page at /. Covered by httptest tests, not run live.The items are the issue's own; each status is the agent's report, which the platform has not checked.
Checks
Review this like any other change. The agent ran on the member's own machine from the text of the issue; the platform has not built or tested it. CI results appear on this pull request.
passedHow this pull request was made
Opened by the platform from a coding agent's run on a member's box (run
5cd78590). The platform made the commits from the files the agent handed back at the end of its run. Nothing is merged automatically.