Skip to content

feat: add timestamp driven callbacks to indexer#1042

Closed
ryardley wants to merge 20 commits into
mainfrom
ry/1020-blockchain-time-threshold-queue
Closed

feat: add timestamp driven callbacks to indexer#1042
ryardley wants to merge 20 commits into
mainfrom
ry/1020-blockchain-time-threshold-queue

Conversation

@ryardley

@ryardley ryardley commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

References #1020

  • Marks sections to refactor
  • Add new ThresholdQueue for executing threshold callbacks. We can use this later with events in ciphernodes too.
  • Add specialized version for Indexer called CallbackQueue
  • Setup block listener functionality in EventListener
  • Ensured that on failure listening retries
  • Created dispatch_after_tijmestamp method on EnclaveIndexer
  • Colocated public methods
  • Hid non public exports
  • Extracted delayed functionliaty to handle_e3_input_deadline_expiration function
  • Needed to run TS tools with pnpm
  • Contract documentation was inaccurate compared to how we are using the expires field so I updated the docs.

@coderabbitai

coderabbitai Bot commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ry/1020-blockchain-time-threshold-queue

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@vercel

vercel Bot commented Nov 21, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
crisp Ready Ready Preview Comment Nov 22, 2025 2:32am
enclave-docs Ready Ready Preview Comment Nov 22, 2025 2:32am

@vercel vercel Bot temporarily deployed to Preview – crisp November 21, 2025 18:43 Inactive
@vercel vercel Bot temporarily deployed to Preview – enclave-docs November 21, 2025 18:43 Inactive
@ryardley ryardley changed the title add timestamp driven callbacks to indexer feat: add timestamp driven callbacks to indexer Nov 21, 2025
@vercel vercel Bot temporarily deployed to Preview – enclave-docs November 21, 2025 20:14 Inactive
@vercel vercel Bot temporarily deployed to Preview – crisp November 21, 2025 20:14 Inactive
@vercel vercel Bot temporarily deployed to Preview – enclave-docs November 21, 2025 23:32 Inactive
@vercel vercel Bot temporarily deployed to Preview – crisp November 21, 2025 23:32 Inactive
@ryardley

Copy link
Copy Markdown
Contributor Author

putting this on hold until #1043 is merged

@ryardley ryardley added blocked An issue that is blocked by something. testing labels Nov 23, 2025
@ryardley ryardley added crisp Related to the crisp example app and removed blocked An issue that is blocked by something. labels Nov 25, 2025
@ryardley

Copy link
Copy Markdown
Contributor Author

closing this to re-implement over #1043

@ryardley ryardley closed this Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

crisp Related to the crisp example app testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant