Skip to content

Conversation

@Anmol1696
Copy link
Collaborator

part of: #695

@Anmol1696 Anmol1696 requested a review from Copilot June 27, 2025 11:14
Copy link
Contributor

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 refactors the Cosmos chain generator logic by splitting the legacy code into multiple specialized components under a new directory structure (chains/cosmos). Key changes include updating import paths in the main index, removing the old cosmos.ts file, and introducing distinct modules for validator, genesis, statefulset, and configmap generation.

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/generator/src/builders/index.ts Updated the import path for CosmosBuilder to reference the new chains/cosmos location.
packages/generator/src/builders/cosmos.ts Removed the legacy Cosmos chain generator implementation.
packages/generator/src/builders/chains/cosmos/validator.ts Introduced a new validator StatefulSet generator for Cosmos chains.
packages/generator/src/builders/chains/cosmos/statefulset.ts Added a new generator to orchestrate genesis and (if applicable) validator StatefulSets.
packages/generator/src/builders/chains/cosmos/index.ts Created a new main Cosmos builder that integrates service, statefulset, and configmap generation.
packages/generator/src/builders/chains/cosmos/genesis.ts Added a dedicated genesis StatefulSet generator.
packages/generator/src/builders/chains/cosmos/configmap.ts Introduced multiple ConfigMap generators (keys, global scripts, setup scripts, genesis patch, ICS proposal) for Cosmos chains.

@Anmol1696 Anmol1696 merged commit a25d4a3 into main Jun 30, 2025
3 checks passed
@Anmol1696 Anmol1696 deleted the anmol/cosmos-refactor branch June 30, 2025 17:23
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.

2 participants