Skip to content
This repository was archived by the owner on Mar 17, 2026. It is now read-only.

Error codes#40

Merged
zeuslawyer merged 6 commits intosvm-mainfrom
176/error-codes
Sep 19, 2025
Merged

Error codes#40
zeuslawyer merged 6 commits intosvm-mainfrom
176/error-codes

Conversation

@andrejrakic
Copy link
Copy Markdown

This PR maps Router programs' error codes to TypeScript types. Merge it after #39

@andrejrakic andrejrakic self-assigned this Sep 18, 2025
@andrejrakic andrejrakic requested a review from Copilot September 18, 2025 14:44
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces TypeScript error code mappings for Router programs in the CCIP SVM (Solana Virtual Machine) package. It establishes a new TypeScript package structure with comprehensive error handling capabilities.

  • Adds a new @chainlink/ccip-svm package with TypeScript configuration and build setup
  • Creates custom error classes mapping Router program error codes (7000-7031) to TypeScript types
  • Implements a factory function to convert numeric error codes to corresponding error instances

Reviewed Changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/ccip-svm/package.json Package configuration for the new CCIP SVM module with dependencies and scripts
packages/ccip-svm/tsconfig.json TypeScript configuration with ES2020 target and strict type checking
packages/ccip-svm/src/index.ts Main entry point exporting custom error types
packages/ccip-svm/src/errors/custom.ts Comprehensive error class definitions and factory function for Router program error codes
packages/ccip-svm/README.md Basic package documentation header
Files not reviewed (2)
  • packages/ccip-svm/package-lock.json: Language not supported
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@andrejrakic andrejrakic marked this pull request as ready for review September 18, 2025 14:50
Copy link
Copy Markdown
Contributor

@zeuslawyer zeuslawyer left a comment

Choose a reason for hiding this comment

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

Love it. LGTM

"noUnusedLocals": true,
"noUnusedParameters": true,
"esModuleInterop": true,
"skipLibCheck": true,
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.

ive not worked with Solana Kit much - lets keep an eye on skipLibCheck in case that dep is not consistent wth typing?

@zeuslawyer zeuslawyer merged commit 1a655f4 into svm-main Sep 19, 2025
2 checks passed
@andrejrakic andrejrakic deleted the 176/error-codes branch September 19, 2025 11:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants