Skip to content

fix(errors): add CANCELLED and RESPONSE_VALIDATION_ERROR to LILY_ERROR_CODES - #525

Merged
David-patrick-chuks-02 merged 2 commits into
Lilly-Protocol:mainfrom
Ranjeet2063:fix/add-cancelled-response-validation-error-codes
Sep 6, 2026
Merged

fix(errors): add CANCELLED and RESPONSE_VALIDATION_ERROR to LILY_ERROR_CODES#525
David-patrick-chuks-02 merged 2 commits into
Lilly-Protocol:mainfrom
Ranjeet2063:fix/add-cancelled-response-validation-error-codes

Conversation

@Ranjeet2063

Copy link
Copy Markdown
Contributor

Summary

Add CANCELLED and RESPONSE_VALIDATION_ERROR to LILY_ERROR_CODES constant map and replace hard-coded string literals in fetch-http-client.ts.

Changes

  • Added CANCELLED: 'CANCELLED' and RESPONSE_VALIDATION_ERROR: 'RESPONSE_VALIDATION_ERROR' to LILY_ERROR_CODES in src/errors/sdk-error.ts.
  • Replaced raw string literals with constant references across src/http/fetch-http-client.ts.
  • Added assertions in tests/error-tojson.test.ts locking the presence and values of the new error codes.

Closes #447

Copilot AI lite review requested due to automatic review settings September 4, 2026 13:19

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@David-patrick-chuks-02
David-patrick-chuks-02 merged commit 5540f32 into Lilly-Protocol:main Sep 6, 2026
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.

[Bounty: $55] Add CANCELLED and RESPONSE_VALIDATION_ERROR to the LILY_ERROR_CODES constants

4 participants