Skip to content

[CLI][Umbrella Ticket] Various pocketd improvements identified during the Morse -> Shannon migration #1368

@Olshansk

Description

@Olshansk

Objective

Improve the pocketd CLI via DevEx gaps identified in preparation for the Morse -> Shannon migration.

Improve the user experience as the network matures as well.

Origin Document

While preparing the Beta TestNet Rehersal, I:

  • Identified some missing (not ultra critical but necessary) functionality
  • Gaps in usability

Goals

  • Improve the DevEx for the Morse->Shannon migration
  • Improve the pocketd CLI for years to come
  • Create simple & constrained example PRs of how to add new tx/query functionality
  • Create example PRs of how to add new tx/query functionality

Deliverables (in order of priority)

  • [@red-0ne ] pocketd tx service update-service

    • We need to be able to update the CUTTM of existing services
    • See TODOs in x/service/keeper/msg_server_add_service.go
  • [@bryanchriswhite] pocketd tx service delete-service

    • Scaffold and add the delete-service endpoint
    • Only the service owner should be able to delete an onchain service
    • Enable the chain authority to delete other services (with a TODO to remove this functionality in the future)
  • [@Olshansk] Streamline governance parameter updates with pocketd

    • Ensure every parameter from every module has a .json here: ./tools/scripts/params/
    • One of the following:
      • Copy-pasta to easily generate json files and submit upgrades
      • Business logic that wraps it in pocketd <module> tx update-param around an authz command
    • Update autoCLI with long/short documentation on how to use update-params even thought its authz under
  • pocketd query service all-services

    • [@red-0ne ] Add the --service-owner=[owner_address] flag to avoid jq operations or pagination on the client side
  • pocketd query application list-applications

    • [@red-0ne ] Add a flag --service-id=[service_id] so we can list all applications staking for a particular service
    • [@red-0ne ] Add a flag --gateway-address=[gateway_address] so we can list all applications delegating to a particular gateway
  • Bonus: Look into how much of this can be automated with an agent.


Creator: @Olshansk
Co-Owners: @bryanchriswhite @red-0ne @fredteumer

Metadata

Metadata

Assignees

Labels

toolingTooling - CLI, scripts, helpers, off-chain, etc...

Projects

Status

🏗 In progress

Relationships

None yet

Development

No branches or pull requests

Issue actions