chore: Log and expose aws services error internal details

[zeljkoX]

Summary

This PR adds logging to expose more details about AWS service errors, improving debugging.

Testing Process

Checklist

• Add a reference to related issues in the PR description.
• Add unit tests if applicable.

Note

If you are using Relayer in your stack, consider adding your team or organization to our list of Relayer Users in the Wild!

Summary by CodeRabbit

• Chores
  • Enhanced error diagnostics and logging for AWS SQS and KMS operations with richer error context and classification to improve troubleshooting and visibility into operation failures

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

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.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: efca38ad-3b6c-4bff-ba56-72d66f67809f

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

Use the checkbox below for a quick retry:

• 🔍 Trigger review

Walkthrough

This PR adds AWS SDK error classification and enriched error logging throughout the relayer. A new aws_error utility module provides stable, low-cardinality error categorization and full error source chain rendering. SQS and KMS backends integrate these utilities into their failure paths to emit detailed logs and user-facing error messages.

Changes

AWS Error Classification and Enriched Logging

Layer / File(s)	Summary
Dependencies Cargo.toml	Adds aws-smithy-types and aws-smithy-runtime-api v1 to support error introspection and formatting.
Error Utilities src/utils/aws_error.rs, src/utils/mod.rs	New module exports classify_sdk_error (maps SdkError to stable &'static str tags) and re-exports DisplayErrorContext (renders full error source chain). Comprehensive unit tests cover construction, timeout, dispatch, and service error paths.
SQS Backend Integration src/queues/sqs/backend.rs	Updates send_message_to_sqs error handling to classify errors and emit warn logs with error kind, detail, and context; returns QueueBackendError message via DisplayErrorContext.
KMS Module Integration src/services/aws_kms/mod.rs	Updates error handling across secp256k1 and Ed25519 public key retrieval and signing operations to log classified warnings with operation labels and format AwsKmsError variants using DisplayErrorContext.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 AWS errors now wear their source chains bright,
Classification tags keep cardinality light,
From KMS keys to SQS queues they flow,
Each failure logged with details we should know!
Enriched diagnostics hop along the way,
Making troubleshooting brighter every day! 🌟

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	The PR description follows the template structure with a summary section, testing process section, and checklist. However, both required checklist items (issue references and unit tests) are left unchecked, and the testing process section is empty.	Complete the testing process section with details of how the changes were tested, and consider filling out the checklist items regarding issue references and unit tests if applicable to this change.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title clearly and concisely describes the main change: adding logging to expose AWS service error details for improved debugging.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch aws-error-details

---

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

[Copilot]

Pull request overview

This PR improves observability for AWS SDK failures by introducing shared utilities to classify SdkError variants and to render full nested error causes, then wiring those into AWS KMS and SQS call sites.

Changes:

• Added utils::aws_error with classify_sdk_error and a re-export of DisplayErrorContext, including unit tests.
• Updated AWS KMS and SQS integrations to emit structured tracing logs with low-cardinality error kind tags plus full error-chain details.
• Added direct dependencies on aws-smithy-runtime-api and aws-smithy-types to support the new helpers.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
src/utils/mod.rs	Exposes the new AWS error helper module via crate::utils.
src/utils/aws_error.rs	Adds shared AWS SDK error classification + full error-chain rendering utilities (with tests).
src/services/aws_kms/mod.rs	Logs structured AWS KMS failures with error kind + detailed context; updates returned error strings.
src/queues/sqs/backend.rs	Logs structured SQS send failures with error kind + detailed context; updates returned error strings.
Cargo.toml	Adds smithy runtime/type crates as direct dependencies.
Cargo.lock	Updates lockfile for the added dependencies.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/services/aws_kms/mod.rs`:
- Around line 327-336: The AWS KMS sign path currently maps all SDK errors to
AwsKmsError::PermissionError inside the map_err closure in
sign_digest_secp256k1; change that to return AwsKmsError::SignError (using the
same DisplayErrorContext(&e).to_string() message) so KMS sign() failures are
classified as signing errors rather than permission errors while keeping the
existing warn(...) call (which uses classify_sdk_error(&e) and
DisplayErrorContext(&e)) intact.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 50300784-48de-48d2-81b8-71a56cbc8c89

📥 Commits

Reviewing files that changed from the base of the PR and between 8194752 and b6762e6.

⛔ Files ignored due to path filters (1)

• Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (5)

• Cargo.toml
• src/queues/sqs/backend.rs
• src/services/aws_kms/mod.rs
• src/utils/aws_error.rs
• src/utils/mod.rs

[codecov]

Codecov Report

❌ Patch coverage is 48.32215% with 77 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.27%. Comparing base (8194752) to head (5f1db8b).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
src/services/aws_kms/mod.rs	0.00%	32 Missing ⚠️
src/queues/sqs/worker.rs	0.00%	23 Missing ⚠️
src/queues/sqs/backend.rs	0.00%	13 Missing ⚠️
src/utils/aws_error.rs	88.88%	9 Missing ⚠️

Additional details and impacted files

Flag	Coverage Δ
ai	0.00% <0.00%> (ø)
dev	90.27% <48.32%> (-0.05%)	⬇️
properties	0.01% <0.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@@            Coverage Diff             @@
##             main     #769      +/-   ##
==========================================
- Coverage   90.31%   90.27%   -0.05%     
==========================================
  Files         291      292       +1     
  Lines      124510   124642     +132     
==========================================
+ Hits       112448   112516      +68     
- Misses      12062    12126      +64     

Files with missing lines	Coverage Δ
src/utils/aws_error.rs	88.88% <88.88%> (ø)
src/queues/sqs/backend.rs	55.82% <0.00%> (-0.61%)	⬇️
src/queues/sqs/worker.rs	52.96% <0.00%> (-0.67%)	⬇️
src/services/aws_kms/mod.rs	83.92% <0.00%> (-5.60%)	⬇️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[tirumerla]

overall lgtm, added few improvements