Skip to content

feat: add structured json logging to keeper- #109#127

Merged
ayomideadeniran merged 2 commits intoSoroLabs:mainfrom
OperaCode:feat/json-keeper-logging
Mar 29, 2026
Merged

feat: add structured json logging to keeper- #109#127
ayomideadeniran merged 2 commits intoSoroLabs:mainfrom
OperaCode:feat/json-keeper-logging

Conversation

@OperaCode
Copy link
Copy Markdown
Contributor

What does this PR do?

This PR standardizes keeper logging on structured JSON output, adds env-driven log level and format controls, and ensures log entries carry consistent metadata such as service, module, timestamp, and level. It also updates the keeper test and coverage setup so the existing CI jobs pass against the current Node and Stellar SDK behavior.

Related Issue

Closes #109

Type of Change

  • Bug fix
  • New feature
  • Refactor (no behavior change)
  • Documentation
  • Infrastructure / CI
  • Tests

Checklist

  • I have read the Contributing Guide
  • My PR title follows Conventional Commits (e.g., feat:, fix:, refactor:)
  • I have added at least one label to this PR
  • I have added/updated tests for my changes (if applicable)
  • cargo test --workspace passes (if contracts changed)
  • pnpm build passes (if frontend changed)
  • cargo fmt --all -- --check passes (if Rust changed)
  • pnpm lint passes (if frontend changed)

Screenshots

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 28, 2026

@OperaCode Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@ayomideadeniran
Copy link
Copy Markdown
Contributor

Your pr is under review.

@ayomideadeniran ayomideadeniran merged commit db603a7 into SoroLabs:main Mar 29, 2026
4 of 6 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.

[Infra] Add Structured JSON Logging to Keeper Service

2 participants