Correct coordinator role on all build pages#8
Merged
Conversation
The issuer and verifier build pages described a coordinator as running @bedrock/vc-delivery and hand-coding the raw VCALM exchange loop. That conflates two distinct roles. A coordinator is a client of a workflow service, not the service itself. Rewrite all six build pages (government, education, supply-chain; issuer and verifier) around the actual coordinator API surface: - Create an exchange: POST <workflowId>/exchanges with ttl + variables, receiving 204 plus a Location header (the exchange URL). - Host and share the interaction URL pointing the wallet at the exchange. - Poll GET <workflowId>/exchanges/<id> until state is complete and read results under exchange.variables.results. The workflow service (which runs @bedrock/vc-delivery) handles protocol negotiation, signing, verification, and multi-wallet interop. Request and response shapes verified against the bedrock-vc-delivery source. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Cloudflare Pages previewDeployed this PR to a live preview: https://fix-coordinator-workflow-ser.learnvcorg.pages.dev This deploy: https://82102599.learnvcorg.pages.dev Updated for commit 92aaa82. |
Drop the DMV / mobile-driver's-license framing and repurpose the slot as a digital student ID vertical: a school issues, a student holds in the wallet of their choice, and a campus service or partner verifies. Student IDs are identity and entitlement credentials, kept distinct from the existing education (diploma/transcript) vertical. The selective- disclosure anchor becomes "currently enrolled" in place of "over 21". Verticals are collection-driven, so renaming the directory, the tag, and the permalinks updates the nav, footer, home grid, sitemap, and llms.txt automatically. The issuer and verifier pages keep the corrected coordinator model (create exchange, host/share interaction URL, poll for results) and are only re-themed. - Rename src/verticals/government -> src/verticals/student-id and its data file; change the collection tag to student-id. - Rewrite index.njk, build/index.njk, and build/wallet.njk around the student-ID story; re-theme build/issuer.njk and build/verifier.njk. - Rename flowIcons gov* keys to studentId* and update embedded labels. - Rename the unused government-flow.svg diagram to student-id-flow.svg. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The build pages said a coordinator does not "speak VCALM on the wire." That implies the coordinator is outside VCALM entirely. It is not — the create-exchange / host-interaction-URL / poll API is itself the coordinator side of VCALM. Reword the six issuer/verifier intros: a coordinator speaks VCALM, just not VCALM delivery — the on-the-wire exchange loop with the wallet, which the workflow service handles. Also scope "the protocol mess" to "VCALM delivery" rather than VCALM as a whole. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
This PR contains two related changes to the build/vertical pages.
1. Correct the coordinator role (all build pages)
The issuer/verifier build pages told a coordinator to
npm install @bedrock/vc-deliveryand hand-code the raw VCALM exchange loop. That conflates two roles:@bedrock/vc-deliveryand handles all protocol work — VCALM/OID4VCI/OID4VP negotiation, signing, verification, multi-wallet interop.A coordinator does three things:
POST <workflowId>/exchangeswithttl+variables; receives204+ aLocationheader (the exchange URL).GET <workflowId>/exchanges/<id>untilstateiscomplete, then read verified results underexchange.variables.results.Request/response shapes verified against the
bedrock-vc-deliverysource.2. Replace the government vertical with a student-ID vertical
For competitive reasons, the DMV / mobile-driver's-license framing is dropped. The slot is repurposed as a digital student ID vertical: a school issues, a student holds in the wallet of their choice, and a campus service or partner verifies.
Follow-up (not in this PR)
A healthcare vertical is planned next (clinician license / patient ID — strong selective-disclosure story).
Verification
npx @11ty/eleventybuilds clean (23 files);/student-id/pages generated,/government/gone.🤖 Generated with Claude Code