Repository navigation
docs(submission): land the Serverpod submission document (#181) - #183
Conversation
Add docs/submission/serverpod.md on main: the rules confirmed on the official source with the date checked, a required-items checklist with evidence links, a judging-criteria -> evidence map, and the <2 min video script. Live state verified 2026-10-14: server 1.0.0+8e6f98a-dirty with the wallet and hire endpoints deployed (400/401, not 404), public app the 2bb61a4 build in server mode. Studio deploy and rating are labelled demos; only agt-006/agt-007 have manifests. Closes #181
Deploying puls3 with
|
| Latest commit: |
8534ebd
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://52c6c77a.puls3-4lw.pages.dev |
| Branch Preview URL: | https://docs-181-serverpod-submissio.puls3-4lw.pages.dev |
TOMOKI977
left a comment
There was a problem hiding this comment.
The package is the right shape, and the live checks (8e6f98a-dirty, challenge 400, hire 401, public app on 2bb61a4) match what is deployed. Two claims do not match main today, so I am not approving yet.
docs/adr/has seven files,0001through0006, including two0002files. "Five ADRs" is low.- "no feedback endpoint" and "rating is UI-only" understate the repo.
puls3_server/lib/src/feedbackis onmain. The public UI still shows the empty rating, and the live server may not serve that endpoint until the next deploy. Please say that. - "Only
agt-006andagt-007have manifests" points atdemo_manifests.dart. #180 adds the other six in the repo. On the deployed server they are still missing untilsclouddeploy. Please tie that sentence to the deployed server.
The build-and-run link is the README section in #185.
…allet-warning note (#181) Address the #183 review: - docs/adr/ has six ADRs (0001-0006) plus the #74 drop-in review, not five. - Rating/feedback: the feedback endpoint (puls3_server/lib/src/feedback) and the Reputation Registry (#14) are in the repo (#21); the public UI still shows the empty rating and the deployed server may not serve the endpoint until the next deploy. - Manifests: all eight demo agents have manifests in the repo (#180); the deployed server still has only agt-006/agt-007 until the next scloud deploy. - Add the Blockaid wallet-warning note (a Pages-domain false positive) to §2 and acknowledge it in video scene 3; tie scene 4 to the deployed server. Refs #181
|
All three points fixed in
The README live-demo note is a separate small change (README is on |
moises-cisneros
left a comment
There was a problem hiding this comment.
The three points from the earlier review are addressed: the ADR count is now six (0001-0006), the feedback caveat says the public UI is still empty and the deployed server may not serve it yet, and the manifest sentence is tied to the deployed server (only agt-006/agt-007 until the next scloud deploy). CI green.
…) (#186) Freighter may show a Blockaid warning when connecting to puls3-4lw.pages.dev (a domain-reputation false positive on the shared Cloudflare Pages domain, raised before any signature). Add a short note next to the live-demo link, as the #183 review suggested for the README. Refs #181 Co-authored-by: Moises Cisneros <110249336+moises-cisneros@users.noreply.github.com>
TOMOKI977
left a comment
There was a problem hiding this comment.
The earlier notes (six ADRs, the feedback caveat, the manifest sentence) are in the file. I am not approving a merge onto current main.
main has moved since 4c80e074 (b65c62e #188, d08b4e7 #186, 368430b #175). Branch protection requires the head to be up to date before gate and scan count, and this branch is BEHIND, so the green checks on 4c80e074 do not satisfy merge.
Re-checked the live state just now. Several sentences in docs/submission/serverpod.md no longer match it:
POST /health/checkreturns1.0.0+d52970e, not1.0.0+8e6f98a-dirty.d52970eisfeat(flutter): turn Studio on in the Pages build.- The public bundle at https://puls3-4lw.pages.dev/main.dart.js contains
Save the draft before deploying, which exists only inServerDeployGatewayand is kept by the compile-timeSTUDIO=serverdefine. Section 3 and section 5 still say Studio deploy isFakeDeployGatewayand a labelled demo (#18/#35).POST /studio/listDraftsis deployed (HTTP 401); a missing endpoint returnsEndpoint not found. POST /feedback/submitFeedbackis HTTP 401 (requireLogin), not missing. Please keep the public-UIrating: 0.0caveat and dropmay not serve the feedback endpoint until the next deploy.d52970ealready has manifests foragt-001throughagt-008inpuls3_server/lib/src/runtime/demo_manifests.dart. Theonly agt-006 and agt-007 until the next scloud deploysentence is stale. Whether the runtime is enabled is still configuration, and that part can stay.
Please refresh the live-state sentences, then update the branch so gate and scan run on the new head.
The submission doc still described the 2026-10-14 server and a demo Studio. Health is 1.0.0+d52970e, the public app includes 368430b with STUDIO=server, and feedback, Studio, and the eight demo manifests are on that server. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Updated onto
|
Closes #181
Summary
Land
docs/submission/serverpod.mdonmain(it previously existed only in PR #157): the rules confirmed on the official source with the date checked, a required-items checklist with evidence links, a judging-criteria → evidence map, and the < 2 min video script.Live state verified 2026-10-14: server
1.0.0+8e6f98a-dirty;/agent/listserves the catalog;/walletAuth/createChallenge→ 400 and/hire/createHire→ 401 (deployed, not 404); public app the2bb61a4build in server mode. Studio deploy and rating are labelled demos; onlyagt-006/agt-007have manifests.Acceptance criteria
main(this PR)8e6f98a-dirty, public app on the2bb61a4build in server modeVerification evidence
Notes for reviewers