Skip to content

Fix: feat: implement robust error mapping and error parser in @astroid/client (Auto-Generated) - #40

Merged
Cjay-Cyber-2 merged 19 commits into
ASTROIDX556:mainfrom
Michealshodipo56:driptide/issue-7-1787918756920
Sep 1, 2026
Merged

Fix: feat: implement robust error mapping and error parser in @astroid/client (Auto-Generated)#40
Cjay-Cyber-2 merged 19 commits into
ASTROIDX556:mainfrom
Michealshodipo56:driptide/issue-7-1787918756920

Conversation

@Michealshodipo56

Copy link
Copy Markdown
Contributor

Closes #7

This pull request was generated automatically and scoped strictly to issue #7.

Changes

Implemented robust error mapping and error parser in @astroid/client, defining specialized classes like AstroidSDKError, AstroidValidationError, AstroidHorizonError, and AstroidPolicyViolationError, with secure content-type inspection, cause preservation, and comprehensive unit tests.

Verification

⚠️ Not verified locally (no build system detected, or the required toolchain isn't installed on the worker). GitHub CI is the source of truth — please check the CI status on this PR before merging.

Linked with Closes #7 so the Drips Wave bot resolves the issue on merge.

@drips-wave

drips-wave Bot commented Aug 28, 2026

Copy link
Copy Markdown

@Michealshodipo56 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

@mergekeeper

mergekeeper Bot commented Aug 28, 2026

Copy link
Copy Markdown

MergeKeeper review

Scope: in scope for linked issue #7.
Verdict: clean

The pull request successfully implements robust error mapping, Horizon error parsing, and error classes within the client package in fulfillment of issue #7.

Reviewed commit: c1d7a6690d3bf4d04b041cdce08b668864e8cb44.
CI and merge eligibility are checked separately.

@mergekeeper

mergekeeper Bot commented Aug 28, 2026

Copy link
Copy Markdown

MergeKeeper merge status

Status: blocked
PR state: open
Mergeability: conflicts / not mergeable
Checked commit: c1d7a6690d3bf4d04b041cdce08b668864e8cb44.

Reason: One or more required CI checks failed.

Failing checks:

Next steps:

  1. Open the failing check details above and fix the reported error.
  2. Run the same checks locally where possible.
  3. Commit and push the fix.
  4. MergeKeeper will automatically re-review the updated PR.

@mergekeeper

mergekeeper Bot commented Aug 28, 2026

Copy link
Copy Markdown

Needs review

Linked to #7, but the diff does not match the issue scope.

The pull request introduces extensive unrelated changes and rewrites existing client and auth code outside the scope of issue #7.

Reviewed commit: 03ecb0a125eb36f676c46cf345bb7b8e68eb4d24.

@Cjay-Cyber-2
Cjay-Cyber-2 merged commit 388001c into ASTROIDX556:main Sep 1, 2026
1 check failed
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.

feat: implement robust error mapping and error parser in @astroid/client

2 participants