chore: its factory commands update

[benjamin852]

why?

ITS Factory script commands and ITS script commands are written out differently. Makes it inconsistent to run the two scripts

• ...
• Task: ?

how?

Refactored ITS Factory commands to match ITS script commands

• ...

testing

Tested each command and added an example for how to run each command in the README

• ...

Greptile Overview

Updated On: 2025-09-26 20:12:39 UTC

Summary

This PR refactors the Interchain Token Factory commands to match the existing ITS script command structure, replacing the old --action flag approach with individual commands using positional arguments. The changes include:

• Command Structure Alignment: Updated interchainTokenFactory.js to use individual commands (e.g., deploy-interchain-token) instead of the previous --action deployInterchainToken format
• Positional Arguments: Changed from option-based parameters to positional arguments, consistent with its.js patterns
• GitHub Workflow Updates: Updated test workflow to use the new command syntax
• Documentation: Added comprehensive documentation for all Interchain Token Factory commands with examples

The refactoring improves consistency across the codebase by standardizing the command-line interface pattern used throughout the project.

Confidence Score: 4/5

• This PR is generally safe to merge with minor fixes needed for syntax issues
• Score reflects a well-structured refactoring that improves consistency, but has two minor syntax issues: a duplicate argument in the GitHub workflow and a missing code block formatting in README
• Fix the link-token command in .github/workflows/test-evm.yaml and the code block formatting in evm/README.md

Important Files Changed

File Analysis

Filename	Score	Overview
.github/workflows/test-evm.yaml	3/5	Updated commands to match new CLI interface with one syntax error in link-token command
evm/README.md	4/5	Comprehensive documentation for Interchain Token Factory commands with one formatting issue
evm/interchainTokenFactory.js	5/5	Complete refactoring to match ITS command structure with proper argument parsing and command definitions

Sequence Diagram

sequenceDiagram
    participant GH as GitHub Workflow
    participant ITF as InterchainTokenFactory.js
    participant ITS as InterchainTokenService.js
    participant BC as Blockchain

    Note over GH: Test EVM Workflow Execution
    
    GH->>ITF: deploy-interchain-token "test" "TST" 18 12345 <minter> --salt "salt"
    ITF->>BC: Deploy token with metadata
    BC-->>ITF: TokenId & Token Address
    ITF-->>GH: Success
    
    GH->>ITF: deploy-remote-interchain-token remote --salt "salt"
    ITF->>BC: Deploy remote token
    BC-->>ITF: TokenId
    ITF-->>GH: Success
    
    GH->>ITS: register-token-metadata <token-address>
    ITS->>BC: Register metadata
    BC-->>ITS: Success
    ITS-->>GH: Success
    
    GH->>ITF: register-custom-token <token-address> 4 <operator> --salt "salt"
    ITF->>BC: Register custom token
    BC-->>ITF: Success
    ITF-->>GH: Success
    
    GH->>ITS: transfer-mintership <token-address> <new-minter>
    ITS->>BC: Transfer mintership
    BC-->>ITS: Success
    ITS-->>GH: Success
    
    GH->>ITF: link-token remote "0x1234" 4 "0x5678" --salt "salt"
    Note right of GH: Fixed: removed duplicate 'remote' argument
    ITF->>BC: Link tokens across chains
    BC-->>ITF: Success
    ITF-->>GH: Success

Loading

---

Note

Refactors evm/interchainTokenFactory.js from a single --action flag to kebab-case subcommands and updates workflows and docs to the new CLI.

• CLI/Script (evm/interchainTokenFactory.js):
  • Replace --action-based interface with kebab-case subcommands (e.g., deploy-interchain-token, register-custom-token, link-token, etc.).
  • Adjust command handlers and main signature to route by subcommand; pass action explicitly to handleTx.
  • Rename actions and method calls to canonical kebab-case equivalents (e.g., canonical-interchain-token-id, canonical-interchain-token-deploy-salt).
  • Use encodeITSDestination instead of encodeITSDestinationToken; validate and gas estimation flow updated per subcommand.
  • Add shared option wiring via addOptionsToCommands(program, addEvmOptions, { address: true, salt: true }).
• Docs:
  • Add comprehensive "Interchain Token Factory" section to evm/README.md with command references and examples.
  • Update release template (releases/evm/EVM-ITS-Release-Template.md) to new subcommand syntax.
• CI:
  • Update .github/workflows/test-evm.yaml steps to use new subcommand-based factory commands (deploy, register, link).

^{Written by Cursor Bugbot for commit f69020d. This will update automatically on new commits. Configure here.}

[greptile-apps]

_{3 files reviewed, 2 comments}

_{Edit Code Review Agent Settings | Greptile}

[kulikthebird]

I like that we get rid of the --action option in favor of separate commands. Though in my opinion the sequence of 5+ unnaed arguments makes it harder to read the commands.

[isi8787]

update the release template document

[kulikthebird]

I like that now we use the required options instead of pos args. I left comments related to removing the args completely, other than that it looks good to me!

[kulikthebird]

Looks good now, thank you for all the updates!