-
Notifications
You must be signed in to change notification settings - Fork 38
feature: generator for relayer support #704
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Enhance generator to support relayer configurations by merging defaults, building Kubernetes manifests, and integrating relayers into the builder pipeline.
- Added a
deepMergeutility and extendedDefaultsManagerto process relayer defaults - Introduced
RelayerBuilderand factory logic to render relayer manifests - Updated
BuilderManagerto includeRelayerBuilderand added tests for relayer scenarios
Reviewed Changes
Copilot reviewed 26 out of 36 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| defaults.ts | Added deepMerge, getRelayerDefaults, and processRelayer along with relayer handling in applyDefaults |
| builders/relayers/index.ts | New factory and main builder for selecting and running relayer-specific builders |
| builders/relayers/base.ts | Defined common relayer builder interfaces and helpers |
| builders/index.ts | Added RelayerBuilder to the overall builder list |
| tests/test-utils/config.ts | Added relayer configs for end-to-end testing |
| tests/defaults.test.ts | Covered processRelayer, applyDefaults, and deepMerge behaviors |
| tests/builder.test.ts | Added relayer manifest generation test |
| package.json | Added test script |
Comments suppressed due to low confidence (1)
packages/packages/generator/src/builders/relayers/index.ts:35
- There is no test for the default case in
RelayerBuilderFactory.createBuilder. Adding a test that an unsupportedrelayer.typethrows a clear error will ensure coverage for invalid inputs.
default:
Co-authored-by: Copilot <[email protected]>
No description provided.