Skip to content

Persist claimed pulse in note-claim registry payloads#353

Merged
kojibai merged 1 commit intomainfrom
codex/ensure-claimed-pulse-persists-after-cache-refresh
Jan 31, 2026
Merged

Persist claimed pulse in note-claim registry payloads#353
kojibai merged 1 commit intomainfrom
codex/ensure-claimed-pulse-persists-after-cache-refresh

Conversation

@kojibai
Copy link
Copy Markdown
Owner

@kojibai kojibai commented Jan 31, 2026

Motivation

  • Ensure the claimed pulse for an exhale note is preserved in the global registry payload so the claimed pulse remains visible after cache/registry refresh and is stored deterministically like the rotation seal and claimed status.
  • Fall back to receipt/receive signature pulses when registry data is missing so the verify page still displays a sensible claimed pulse.

Description

  • Add an optional claimedPulse to NoteClaimArgs and include a claimedPulse-aware pulse and claimedPulse field when building note-claim payloads/URLs in src/components/SigilExplorer/registryStore.ts so the claim URL carries the pulse data.
  • When marking a note claimed, pass the claimedPulse through to the persisted registry entry, registry payload, and inhaler enqueue path in registryStore.ts so the value is stored and broadcasted.
  • Update src/pages/VerifyPage.tsx to compute noteClaimedPulse via useMemo that prefers the registry claimedPulse and otherwise falls back to embeddedProof.receivePulse, sharedReceipt.receivePulse, or receiveSig.createdAtPulse through normalizeClaimPulse for robust display.

Testing

  • Launched the dev server with npm run dev and the Vite server started successfully and served the app at the local URL. (succeeded)
  • Ran a Playwright script that opened /verify and captured a full-page screenshot to validate render changes (screenshot produced). (succeeded)
  • No unit/integration test suite was executed as part of this change. (not run)

Codex Task

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 31, 2026

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

Project Deployment Actions Updated (UTC)
phi-network-5aii Ready Ready Preview, Comment Jan 31, 2026 10:08pm

Request Review

@kojibai kojibai merged commit 880e611 into main Jan 31, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant