review nits

Author: dkijania

src/mina-signer/tests/mina-signer-test-app/README.md

@@ -17,8 +17,6 @@ The optional `graphql_url` flag lets you override the default target defined in
 ## Workflow
 
 1. `mina-test-signer.js` parses CLI arguments and wires the supporting services.
-2. `payment-service.js` derives the sender public key, composes a payment
-   payload, and signs it with `mina-signer`.
 3. `graphql-client.js` sends the signed payload to the Mina daemon and can check
    whether the transaction reached the pool.
 4. `utils.js` provides small helpers for GraphQL string construction and CLI
@@ -45,15 +43,6 @@ successes and failures at a glance. GraphQL errors (including malformed
 responses) cause the CLI to exit with a non-zero status so they can be surfaced
 in scripts and CI.
 
-## Customisation Tips
-
-- Update `CONFIG.DEFAULT_GRAPHQL_URL` in `config.js` to point at your daemon or
-  a hosted GraphQL endpoint.
-- Tweak `CONFIG.MINA_UNITS.DEFAULT_AMOUNT_MULTIPLIER` and
-  `DEFAULT_FEE_MULTIPLIER` to adjust the default transaction values.
-- Extend `GraphQLClient` with additional queries (e.g. account state, balances)
-  if you need richer diagnostics.
-
 ## Private key format
 
 For clarity, private key is in output format of:

src/mina-signer/tests/mina-signer-test-app/config.js

@@ -14,11 +14,11 @@ export const CONFIG = {
 };
 
 /**
- * Human-friendly CLI usage text that `test-signer.js` displays when
+ * Human-friendly CLI usage text that `mina-test-signer.js` displays when
  * the caller provides incomplete arguments.
  */
 export const USAGE_INFO = {
-  message: 'Usage: node test-signer.js <private_key> <recipient_address> [graphql_url] [nonce]',
-  example: 'Example: node test-signer.js EKErBK1KznrJJY3raJafSyxSayJ6viejaVrmjzXkSmoxXiJQsesU  B62qp4wcxoJyFFyXZ2RVw8kGPpWn6ncK4RtsTz29jFf6fY2XYN42R1v http://172.17.0.3:3085/graphql 3',
+  message: 'Usage: node mina-test-signer.js <private_key> <recipient_address> [graphql_url] [nonce]',
+  example: 'Example: node mina-test-signer.js EKErBK1KznrJJY3raJafSyxSayJ6viejaVrmjzXkSmoxXiJQsesU  B62qp4wcxoJyFFyXZ2RVw8kGPpWn6ncK4RtsTz29jFf6fY2XYN42R1v http://172.17.0.3:3085/graphql 3',
   defaultUrl: `Default GraphQL URL: ${CONFIG.DEFAULT_GRAPHQL_URL}`
 };

src/mina-signer/tests/mina-signer-test-app/graphql-client.js

@@ -26,7 +26,7 @@ export class GraphQLClient {
       const response = await fetch(this.url, {
         method: 'POST',
         headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ operationName: null, query, variables: {} }),
+        body: JSON.stringify({ query }),
       });
 
       return await this.handleResponse(response);

src/mina-signer/tests/mina-signer-test-app/mina-test-signer.js

@@ -9,7 +9,7 @@
  *  - Submit the signed payload to a Mina daemon and verify it reached the pool
  *
  * Usage:
- *    node test-signer.js --private-key <key> --recipient <address> [--url <graphql_url>] [--nonce <nonce>]
+ *    node mina-test-signer.js --private-key <key> --recipient <address> [--url <graphql_url>] [--nonce <nonce>]
  */
 import { Command } from 'commander';
 import { GraphQLClient } from './graphql-client.js';