Skip to content

feat(error-registry): implement error code lookup pagination, agent query, and stats#137

Merged
devJaja merged 4 commits into
Epta-Node:mainfrom
morelucks:feature/error-code-lookup-117
Jul 24, 2026
Merged

feat(error-registry): implement error code lookup pagination, agent query, and stats#137
devJaja merged 4 commits into
Epta-Node:mainfrom
morelucks:feature/error-code-lookup-117

Conversation

@morelucks

Copy link
Copy Markdown
Contributor

Closes #117

Summary

This PR implements the requested query, pagination, and statistical features for the on-chain error-registry contract.

Changes

  1. Added PaginationParams struct containing offset and limit.
  2. Updated get_errors_by_code to support pagination with offset and limit, capping the limit to 100 and filtering out expired errors.
  3. Implemented get_errors_by_agent to retrieve active errors reported by a specific agent.
  4. Implemented get_error_stats to return active error counts per error code.
  5. Added comprehensive unit tests in test.rs covering the new features and pagination edge cases.

praisesokoyebomfrancis and others added 3 commits July 19, 2026 00:03
…#113)

Add the error-registry Soroban contract: error reports carry an
application-level TTL (expires_at = created_at + ttl_seconds, capped at
90 days), expiry is enforced at read time, and a permissionless,
bounded cleanup_expired_errors reclaims storage in batches.

Also pin ed25519-dalek to 2.2.0 in the workspace lockfile so the
contracts' testutils dev-dependency (via soroban-env-host 22.1.3)
compiles on Rust stable, and re-enable clippy + cargo test for
error-registry and agent-registry in CI with --locked to hold the pin.
@vercel

vercel Bot commented Jul 19, 2026

Copy link
Copy Markdown

@morelucks is attempting to deploy a commit to the Jaja's projects Team on Vercel.

A member of the Team first needs to authorize it.

@devJaja
devJaja self-requested a review July 20, 2026 22:29
@devJaja

devJaja commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Nice implementation @morelucks
fix the CI/smart contracts

@devJaja devJaja left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@devJaja
devJaja merged commit d121c7d into Epta-Node:main Jul 24, 2026
10 of 11 checks passed
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.

[Smart Contracts] Add Error Code Lookup/Query Function to error-resolver

3 participants