feat(1651): Token delete command extend (#1723)

Signed-off-by: matevszm <mateusz.marcinkowski@blockydevs.com>

Author: matevszm

docs/output-schemas-guide.md

@@ -552,6 +552,37 @@ Lists all tokens from all networks stored in state.
 }
 ```
 
+#### `token delete`
+
+**Output** (network delete):
+
+```json
+{
+  "transactionId": "0.0.123@1700000000.123456789",
+  "deletedToken": {
+    "name": "MyToken",
+    "tokenId": "0.0.67890"
+  },
+  "network": "testnet",
+  "removedAliases": ["my-token (testnet)"]
+}
+```
+
+**Output** (`--state-only`):
+
+```json
+{
+  "deletedToken": {
+    "name": "MyToken",
+    "tokenId": "0.0.67890"
+  },
+  "network": "testnet",
+  "removedAliases": ["my-token (testnet)"]
+}
+```
+
+`transactionId` is absent for `--state-only`. `removedAliases` is omitted when no aliases exist.
+
 #### `token allowance-nft`
 
 **Output** (specific serials):

src/__tests__/integration/token/import-token.integration.test.ts

@@ -37,6 +37,7 @@ describe('Import Token Integration Tests', () => {
       initialSupply: '10',
       supplyType: SupplyType.INFINITE,
       name: `token-import-${Date.now()}`,
+      adminKey: `${process.env.OPERATOR_ID}:${process.env.OPERATOR_KEY}`,
     };
     const createTokenResult = await tokenCreateFt({
       args: createTokenArgs,
@@ -57,11 +58,18 @@ describe('Import Token Integration Tests', () => {
   });
 
   afterEach(async () => {
-    const deleteTokenArgs: Record<string, unknown> = {
-      token: tokenId,
-    };
     await tokenDelete({
-      args: deleteTokenArgs,
+      args: { token: tokenId, stateOnly: true },
+      api: coreApi,
+      state: coreApi.state,
+      logger: coreApi.logger,
+      config: coreApi.config,
+    });
+  });
+
+  afterAll(async () => {
+    await tokenDelete({
+      args: { token: tokenId },
       api: coreApi,
       state: coreApi.state,
       logger: coreApi.logger,

src/core/services/token/__tests__/unit/mocks.ts

@@ -37,6 +37,10 @@ export const createMockTokenAssociateTransaction = () => ({
   setTokenIds: jest.fn().mockReturnThis(),
 });
 
+export const createMockTokenDeleteTransaction = () => ({
+  setTokenId: jest.fn().mockReturnThis(),
+});
+
 export const createMockCustomFixedFee = () => ({
   setHbarAmount: jest.fn().mockReturnThis(),
   setAmount: jest.fn().mockReturnThis(),

src/core/services/token/__tests__/unit/token-service.test.ts

@@ -26,6 +26,7 @@ import {
   createMockCustomFractionalFee,
   createMockTokenAssociateTransaction,
   createMockTokenCreateTransaction,
+  createMockTokenDeleteTransaction,
   createMockTransferTransaction,
 } from './mocks';
 
@@ -49,6 +50,7 @@ const TRANSFER_AMOUNT = 100n;
 const mockTransferTransaction = createMockTransferTransaction();
 const mockTokenCreateTransaction = createMockTokenCreateTransaction();
 const mockTokenAssociateTransaction = createMockTokenAssociateTransaction();
+const mockTokenDeleteTransaction = createMockTokenDeleteTransaction();
 const mockCustomFixedFee = createMockCustomFixedFee();
 const mockCustomFractionalFee = createMockCustomFractionalFee();
 
@@ -68,6 +70,7 @@ jest.mock('@hashgraph/sdk', () => ({
   TransferTransaction: jest.fn(() => mockTransferTransaction),
   TokenCreateTransaction: jest.fn(() => mockTokenCreateTransaction),
   TokenAssociateTransaction: jest.fn(() => mockTokenAssociateTransaction),
+  TokenDeleteTransaction: jest.fn(() => mockTokenDeleteTransaction),
   CustomFixedFee: jest.fn(() => mockCustomFixedFee),
   CustomFractionalFee: jest.fn(() => mockCustomFractionalFee),
   FeeAssessmentMethod: {
@@ -741,4 +744,26 @@ describe('TokenServiceImpl', () => {
       expect(AccountId.fromString).toHaveBeenCalledWith('0.0.8888');
     });
   });
+
+  describe('createDeleteTransaction', () => {
+    it('should create delete transaction with correct tokenId', () => {
+      const params = { tokenId: TOKEN_ID };
+
+      const result = tokenService.createDeleteTransaction(params);
+
+      expect(TokenId.fromString).toHaveBeenCalledWith(TOKEN_ID);
+      expect(mockTokenDeleteTransaction.setTokenId).toHaveBeenCalledWith(
+        mockTokenIdInstance,
+      );
+      expect(result).toBe(mockTokenDeleteTransaction);
+    });
+
+    it('should log debug messages during delete transaction creation', () => {
+      tokenService.createDeleteTransaction({ tokenId: TOKEN_ID });
+
+      expect(logger.debug).toHaveBeenCalledWith(
+        `[TOKEN SERVICE] Creating delete transaction for token ${TOKEN_ID}`,
+      );
+    });
+  });
 });

src/core/services/token/token-service.interface.ts

@@ -6,6 +6,7 @@ import type {
   AccountAllowanceApproveTransaction,
   TokenAssociateTransaction,
   TokenCreateTransaction,
+  TokenDeleteTransaction,
   TokenMintTransaction,
   TransferTransaction,
 } from '@hashgraph/sdk';
@@ -15,6 +16,7 @@ import type {
   TokenAllowanceFtParams,
   TokenAssociationParams,
   TokenCreateParams,
+  TokenDeleteParams,
   TokenMintParams,
   TokenTransferParams,
 } from '@/core/types/token.types';
@@ -59,4 +61,6 @@ export interface TokenService {
   createFungibleTokenAllowanceTransaction(
     params: TokenAllowanceFtParams,
   ): AccountAllowanceApproveTransaction;
+
+  createDeleteTransaction(params: TokenDeleteParams): TokenDeleteTransaction;
 }

src/core/services/token/token-service.ts

@@ -11,6 +11,7 @@ import type {
   TokenAllowanceFtParams,
   TokenAssociationParams,
   TokenCreateParams,
+  TokenDeleteParams,
   TokenMintParams,
   TokenTransferParams,
 } from '@/core/types/token.types';
@@ -26,6 +27,7 @@ import {
   NftId,
   TokenAssociateTransaction,
   TokenCreateTransaction,
+  TokenDeleteTransaction,
   TokenId,
   TokenMintTransaction,
   TokenSupplyType,
@@ -342,6 +344,15 @@ export class TokenServiceImpl implements TokenService {
     return tx;
   }
 
+  createDeleteTransaction(params: TokenDeleteParams): TokenDeleteTransaction {
+    this.logger.debug(
+      `[TOKEN SERVICE] Creating delete transaction for token ${params.tokenId}`,
+    );
+    return new TokenDeleteTransaction().setTokenId(
+      TokenId.fromString(params.tokenId),
+    );
+  }
+
   /**
    * Process custom fees and convert them to Hedera CustomFee objects
    */

src/core/types/token.types.ts

@@ -107,6 +107,10 @@ export interface NftTransferParams {
   serialNumbers: number[];
 }
 
+export interface TokenDeleteParams {
+  tokenId: string;
+}
+
 /**
  * Parameters for approving NFT allowance for specific serial numbers
  */

src/plugins/batch/manifest.ts

@@ -104,6 +104,7 @@ export const batchPluginManifest: PluginManifest = {
         'token-create-nft-batch-state',
         'token-create-nft-from-file-batch-state',
         'token-associate-batch-state',
+        'token-delete-batch-state',
       ],
       options: [
         {

src/plugins/token/README.md

@@ -67,7 +67,7 @@ src/plugins/token/
 │   │   ├── types.ts         # Internal types
 │   │   └── index.ts         # Command exports
 │   ├── delete/
-│   │   ├── handler.ts       # Token delete handler (local state only)
+│   │   ├── handler.ts       # Token delete handler (network or local state)
 │   │   ├── input.ts         # Input schema
 │   │   ├── output.ts        # Output schema and template
 │   │   └── index.ts         # Command exports
@@ -520,21 +520,30 @@ hcli token allowance-ft \
 
 ### Token Delete
 
-Delete a token from local state. This only removes the token from the local address book, not from the Hedera network.
+Delete a token from the Hedera network and remove it from local state. The token must have an admin key to be deleted from the network.
 
 ```bash
-# Delete by token alias
+# Delete by token alias (network delete)
 hcli token delete --token mytoken-alias
 
-# Delete by token ID
+# Delete by token ID (network delete)
 hcli token delete --token 0.0.123456
+
+# Provide admin key explicitly
+hcli token delete --token 0.0.123456 --admin-key <key-ref>
+
+# Remove from local state only (no network transaction)
+hcli token delete --token mytoken-alias --state-only
 ```
 
 **Parameters:**
 
 - `--token` / `-T`: Token identifier: either a token alias or token-id - **Required**
+- `--admin-key`: Admin key reference for signing (auto-resolved from key manager if omitted) - **Optional**
+- `--key-manager`: Key manager type, defaults to config setting - **Optional**
+- `--state-only`: Remove token from local state only, without a network transaction - **Optional**
 
-Any aliases associated with the token on the current network will also be removed.
+`--state-only` and `--admin-key` are mutually exclusive. Any aliases associated with the token on the current network will also be removed.
 
 ### Token List