Repository navigation
feat(site): open the guided walkthrough at /lab/start as a public beta - #691
DealAppSeo wants to merge 2 commits into
Conversation
Three paths through one journey, preview only: a 404 on production, not linked, not indexed. The home page and existing routes are unchanged. The chat path runs the real claim checker in the browser against two saved GitHub runs, with no key and nothing sent. The GitHub path and the developer path show commands that were run against the published 1.9.1 in an empty folder with no keys, in PowerShell and in bash. Reuses the guided page's components, which now export what this needs. NOT_CHECKED now renders neutral and COMMAND_PASSED dashed, on both pages. The shared example report is regenerated with 1.9.1. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Removes the production 404 and the preview banner, adds a public-beta notice, and limits the headline to checking claims against evidence. The route stays noindex and out of the sitemap. The existing home page is unchanged apart from one new link. The first-run walk now runs against a production build, with the preview-only /lab/test-agent as a control that must still answer 404. It can also walk a deployed site through WALK_BASE. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Strix Security Review✅ No open security findings. PR summary: Opens the guided walkthrough at `/lab/start` as a public beta (it previously answered 404 on production) and adds one link to it from the home page, plus a refactor that exports the shared report-viewing pieces out of `components/test-agent-flow.tsx` and adds an optional launcher to `inspectCommand`. The page runs TrustShell's real claim checker in the browser against two saved GitHub Actions runs; nothing under `src/` changes. Review summaryReviewed all 14 changed files of the /lab/start public-beta launch: the new route and its client-side walkthrough components, the refactored shared report loader, the new Ruled out (8)
Review 1 on this PR · Reviewed by Strix |
There was a problem hiding this comment.
Reviewed all 14 changed files of the /lab/start public-beta launch: the new route and its client-side walkthrough components, the refactored shared report loader, the new inspectCommand launcher parameter, the regenerated example records, and the home-page link. Four independent investigations (generated-shell-command safety, route/gating exposure, client bundle integrity, report-file rendering) plus lint triage (11 items in or reached by the changed files) found no exploitable weakness, and every candidate went through independent validation and was rejected: the launcher argument is a compile-time literal and visitor text stays gated by isRunUrl/isShellSafe; /lab/start exposes only already-public documentation and public GitHub Actions data, with noindex intact and the page absent from the sitemap; the client bundle carries no environment secret or server-only code (verified against a production build) and verdicts cannot be forged through the textarea; uploaded or fetched report content reaches only React text nodes, with the pre-PR size and parse guards intact. Non-blocking hardening worth considering separately: inspectCommand interpolates an unvalidated launcher (latent footgun for a future caller), parseSavedRun could assert an https://github.com scheme before its URL becomes an href, and parseTrustReport could type-check list elements so a malformed report shows the page's own error instead of a generic crash — plus a docs-only clarification that a shell-exported TRUSTSHELL_HOME is not inherited by an MCP host.
Reviewed by Strix
Configure security review settings
Opens
/lab/startas a public beta and adds one link to it from the home page. The owner authorized this website-only launch on 2026-11-08. No npm release, database change or credential change is part of it.What changed for visitors
/lab/startno longer answers 404 on production.noindexand out of the sitemap./lab/startwalks a first-time visitor through Try, Test and Report on three paths: I chat with AI, an AI builds software for me, and I build or run agents. The chat path runs TrustShell's real claim checker in the browser against two saved GitHub Actions runs. It says plainly that TrustShell cannot see AI chats.What changed outside the new page
components/test-agent-flow.tsx: shared pieces are exported and report loading moved into a hook. NOT_CHECKED is now neutral rather than amber, and COMMAND_PASSED is dashed./lab/test-agentremains preview-only.lib/trust-report-view.ts:inspectCommandtakes an optional launcher. The default is unchanged.public/examples/trust-report-example.json: regenerated with 1.9.1. It is real output.tests/site-npm-badge.test.ts: the home page may not link to/start; the rule now excludes/lab/start.src/, so no change to the published package.Evidence on commit 4cc6910
Local, on Windows:
tests/e2e/first-run-walk.mjs, desktop and 375 px phone, against a build made withVERCEL_ENV=production: 295 passed, 0 failed. That is the original 270 checks with the preview-banner check reworded, plus 25 new ones./lab/startanswers 200 while/lab/test-agentstill answers 404, which shows the production setting took effect.tests/e2e/test-agent-walk.mjs: 96 passed, 0 failed.next build: exit 0.wrapper-readinesssits outside that CI job.Not verified
/checkpage's own results. The walk confirms it answers 200 and does not submit to it.Rollback
If the launch fails, production traffic goes back to the saved deployment of commit
7d55cd2. The branch and evidence are kept.🤖 Generated with Claude Code