forked from ignite/cli
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: add
chain debug
command (ignite#3221)
* feat: add support to compile chain app without optimizations * feat: add `debugger` package * chore: change `cliui` package to allow multiple `Session.End()` calls * feat: add `chain debug` command to launch a debugging session * feat: add debug server support to `chain debug` command This allows running a debug server without launching a debug client which is useful to debug using other clients like gdlv or VSCode. * chore: consistency changes * chore: add missing handling for config flag * chore: fix GRPC address * feat: add bubbletea UI to `chain debug` when `--server` flag is present * chore: remove some `chain debug` flags in favour of default values * refactor: simplify debug command by removing build functionality To keep the command simple it only starts the debug shell or server favouring the build and init commands to init/reset blockchain app data or to build the debug binary. Both build and init commands have now a `--debug` flag to build a binary for debugging. * chore: change `ignite serve` to build a debugging binary * docs: add long description for `ignite debug` command * test: add unit tests for `chain debug` server UI views * fix: change error output to be consisten between UI & non UI runs Bubbletea UI is used when the debug server is run (--server flag) while a standard Cobra command is used when running the debug console. The debug command without `--server` runs outside Bubbletea because the debugger needs control over stdout. * refactor: use a single `AddCommand` call to register commands Co-authored-by: Danilo Pantani <[email protected]> * chore: remove redundant `AddCommand` calls * chore: change chain debug command variable name to match flag name Co-authored-by: Alex Johnson <[email protected]> * chore: move disconnect channel init to debug options initialization Co-authored-by: Thomas Bruyelle <[email protected]> * docs: add docstrings to public model methods Co-authored-by: Danilo Pantani <[email protected]> Co-authored-by: Alex Johnson <[email protected]> Co-authored-by: Thomas Bruyelle <[email protected]>
- Loading branch information
1 parent
bd021d9
commit 5900959
Showing
17 changed files
with
687 additions
and
30 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.