Skip to content

feat: ship ScratchNode public wiki loop#490

Merged
HomenShum merged 2 commits into
mainfrom
feat/scratchnode-public-wiki-loop
Jun 3, 2026
Merged

feat: ship ScratchNode public wiki loop#490
HomenShum merged 2 commits into
mainfrom
feat/scratchnode-public-wiki-loop

Conversation

@HomenShum
Copy link
Copy Markdown
Owner

What changed

  • Adds a published-only public ScratchNode wiki route at scratchnode.live/e/:slug/wiki.
  • Uses the existing events:getPublishedWikiBySlug({ slug }) public reader, extended to return both the flat main-branch shape and the nested route payload without exposing drafts/private notes.
  • Wires post-publish room UI to surface/copy the public wiki URL from Share and Wiki sheets.
  • Fixes answer Share to copy real addressable answer URLs with #answer-<encodedAnswerId> instead of only showing a toast.
  • Leaves an agent coordination handoff with the new query/route contract.

Verification

  • npx vitest run convex/__tests__/scratchnode.publicWikiRead.test.ts convex/__tests__/scratchnode.publicWiki.test.ts
  • npx playwright test tests/e2e/scratchnode-live-route-honesty.spec.ts --project=chromium --workers=1
  • npx tsc --noEmit --pretty false
  • npm run build
  • Local in-app browser smoke: http://127.0.0.1:5198/proto/home-v5.html loaded with no console errors.

Notes

This supersedes #488, which used the default Codex branch prefix but failed the repo branch-name style gate.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nodebench-ai Ready Ready Preview, Comment Jun 3, 2026 7:12am

Request Review

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

PR size advisory

This PR adds 598 lines of substantive change. CONTRIBUTING.md defines a soft limit of ~400 LOC.

If the PR is genuinely cohesive (e.g. an architecture map, a generated migration, a deletion of a dead module), no action is needed. Otherwise consider:

  • splitting into 2-3 PRs along independent concerns
  • pre-discussing the architecture change in a GitHub Discussion before merge

This is advisory — it does not block the merge.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

✅ Dogfood Visual QA Gate: PASSED

Check Status
Screenshots 23 captured (pass)
Walkthrough 9 chapters (pass)
Key Frames 9 extracted (pass)
Scribe Steps 8 how-to steps (pass)
Build success
Artifacts

Download the dogfood-evidence-9351a1b artifact from the Actions tab for full screenshots, frames, and walkthrough video.


Generated by Dogfood QA Gate

@HomenShum HomenShum force-pushed the feat/scratchnode-public-wiki-loop branch from 41704c0 to a447076 Compare June 3, 2026 07:11
@HomenShum HomenShum enabled auto-merge (squash) June 3, 2026 07:11
@HomenShum HomenShum merged commit 744765d into main Jun 3, 2026
17 checks passed
@HomenShum HomenShum deleted the feat/scratchnode-public-wiki-loop branch June 3, 2026 07:23
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Demo: walkthrough of the surfaces this PR changed is available as a workflow artifact (pr-demo-490) at https://github.com/HomenShum/nodebench-ai/actions/runs/26870000962

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants