Skip to content

chore: add ibc commands#2047

Closed
Zblocker64 wants to merge 2 commits intomainfrom
add-ibc-transfer
Closed

chore: add ibc commands#2047
Zblocker64 wants to merge 2 commits intomainfrom
add-ibc-transfer

Conversation

@Zblocker64
Copy link
Collaborator

@Zblocker64 Zblocker64 commented Mar 9, 2026

Add back IBC commands to CLI

@Zblocker64 Zblocker64 requested a review from a team as a code owner March 9, 2026 19:18
@github-actions github-actions bot added the C:CLI label Mar 9, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d605c465-e886-4ad7-b094-e40abace96e2

📥 Commits

Reviewing files that changed from the base of the PR and between ff9fe75 and 4c2f339.

📒 Files selected for processing (1)
  • cmd/akash/cmd/root.go
🚧 Files skipped from review as they are similar to previous changes (1)
  • cmd/akash/cmd/root.go

Walkthrough

IBC CLI commands (core and transfer) are registered into the existing root command by adding an addIBCCommands helper and importing the IBC core and transfer packages; the helper is invoked during command initialization to wire TX and Query subcommands.

Changes

Cohort / File(s) Summary
IBC Command Registration
cmd/akash/cmd/root.go
Added imports for IBC core and transfer packages and a new addIBCCommands helper that registers core and transfer TX and Query commands into the existing CLI command tree; invoked during root command init.

Sequence Diagram(s)

sequenceDiagram
    participant DevCLI as Dev CLI (root cmd)
    participant AddIBCCmds as addIBCCommands
    participant IBCCore as IBC Core CLI
    participant IBCTransfer as IBC Transfer CLI

    DevCLI->>AddIBCCmds: initialize root command
    AddIBCCmds->>IBCCore: attach core TX commands to "tx"
    AddIBCCmds->>IBCCore: attach core query commands to "query"/"q"
    AddIBCCmds->>IBCTransfer: attach transfer TX commands to "tx"
    AddIBCCmds->>IBCTransfer: attach transfer query commands to "query"/"q"
    AddIBCCmds->>DevCLI: return updated command tree
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 I hopped through code, with tiny paws,
Wired bridges, checks, and CLI laws,
Core and transfer take their stands,
Commands now travel distant lands. 🥕✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: add ibc commands' accurately summarizes the main change: adding IBC (Inter-Blockchain Communication) commands to the CLI.
Description check ✅ Passed The description 'Add back IBC commands to CLI' is directly related to the changeset, which adds IBC core and transfer commands to the root command.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch add-ibc-transfer

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Zblocker64 Zblocker64 changed the title chore: add ibc transfer commands chore: add ibc commands Mar 9, 2026
@Zblocker64
Copy link
Collaborator Author

closing as we will add to cli pkg

@Zblocker64 Zblocker64 closed this Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant