Epic: create new CRUD-powered CLI commands

avious00 commented 7 months ago

Context

This is the CLI counterpart to the SDK epics @yorhodes is creating, starting with https://github.com/hyperlane-xyz/issues/issues/1150 for hooks/ISMs

Open questions:

what's the read interface? an address? or artifacts? or both?
where are artifacts from apply written to?
how does the existing functionality of the CLI fit in?
- e.g. do we keep the friendly but opinionated interactive interface for deploying warp / core / kurtosis agents, or generating configs, etc?
Is this necessarily blocked on https://github.com/hyperlane-xyz/issues/issues/1150

MVP steps

These are the tasks we should do to have the minimum functionality we need in the CLI

### `Core` commands
- [ ] https://github.com/hyperlane-xyz/issues/issues/1185
- [ ] https://github.com/hyperlane-xyz/issues/issues/1186
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3539
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3688
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3721

### `Warp` commands
- [ ] https://github.com/hyperlane-xyz/issues/issues/1189
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3541
- [ ] https://github.com/hyperlane-xyz/issues/issues/1190
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3479
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3690
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3691
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3667

### Initial Hook commands
- [ ] https://github.com/hyperlane-xyz/issues/issues/1183

### ISM commands
- [ ] https://github.com/hyperlane-xyz/issues/issues/1187

Next Steps

These are the tasks we should do to clean some things up and expand the capabilities of the CLI

Moves existing CLI-native config management and makes it plug into the new commands:

### Migrate existing CLI-native config management to new setup
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3535
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3537
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3534
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3536

Lower priority commands to help us directly manage ISMs or hooks:

### Icebox: Additional Hook / ISM commands - **Skip**
- [ ] https://github.com/hyperlane-xyz/issues/issues/1184
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3538
- [ ] https://github.com/hyperlane-xyz/issues/issues/1188
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3540

### Misc tasks
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3724
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3764
- [ ] https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3765

nambrot commented 7 months ago

@tkporter is the migration supposed to happen later or part of this epic? If that is the case, can we capture documentation and testing tickets to reflect the need before rolling this out?

avious00 commented 7 months ago

@tkporter are the open questions still relevant?

hyperlane-xyz / hyperlane-monorepo