hedgequantx 2.9.146 → 2.9.148

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hedgequantx",
-  "version": "2.9.146",
+  "version": "2.9.148",
   "description": "HedgeQuantX - Prop Futures Trading CLI",
   "main": "src/app.js",
   "bin": {

package/src/pages/algo/one-account.js

@@ -302,6 +302,9 @@ const oneAccountMenu = async (service) => {
 const selectMultipleSymbols = async (service, account) => {
   const spinner = ora({ text: 'Loading symbols...', color: 'yellow' }).start();
   
+  // Debug: log service type
+  const serviceType = service.constructor?.name || 'Unknown';
+  
   // Ensure we have a logged-in service
   if (!service.loginInfo && service.credentials) {
     spinner.text = 'Reconnecting to broker...';
@@ -312,19 +315,24 @@ const selectMultipleSymbols = async (service, account) => {
     }
   }
   
-  const contractsResult = await service.getContracts();
+  let contractsResult;
+  try {
+    contractsResult = await service.getContracts();
+  } catch (err) {
+    spinner.fail(`getContracts exception: ${err.message}`);
+    console.log(chalk.gray(`  Service: ${serviceType} | Tip: Try "hqx login"`));
+    return null;
+  }
   if (!contractsResult.success || !contractsResult.contracts?.length) {
     spinner.fail(`Failed to load contracts: ${contractsResult.error || 'No contracts'}`);
+    console.log(chalk.gray(`  Service: ${serviceType} | Tip: Try "hqx login"`));
     return null;
   }
   
   const contracts = sortContracts(contractsResult.contracts);
   spinner.succeed(`Found ${contracts.length} contracts`);
-  
-  console.log();
-  console.log(chalk.cyan('  Select up to 5 symbols (one at a time)'));
-  console.log(chalk.gray('  Select "Done" when finished'));
-  console.log();
+  console.log(chalk.cyan('\n  Select up to 5 symbols (one at a time)'));
+  console.log(chalk.gray('  Select "Done" when finished\n'));
   
   const selectedContracts = [];
   const maxSymbols = 5;
@@ -356,30 +364,17 @@ const selectMultipleSymbols = async (service, account) => {
       : `Select Symbol ${selectedContracts.length + 1}/${maxSymbols} (${remaining} remaining):`;
     
     const selection = await prompts.selectOption(chalk.yellow(promptText), options);
-    
-    if (selection === 'back' || selection === null) {
-      return null;
-    }
-    if (selection === 'done') {
-      break;
-    }
-    
+    if (selection === 'back' || selection === null) return null;
+    if (selection === 'done') break;
     selectedContracts.push(selection);
     console.log(chalk.green(`  ✓ Added: ${selection.symbol}`));
   }
-  
-  if (selectedContracts.length === 0) {
-    return null;
-  }
+  if (selectedContracts.length === 0) return null;
   
   // Display summary
+  console.log(chalk.cyan(`\n  Selected ${selectedContracts.length} symbol(s):`));
+  for (const c of selectedContracts) console.log(chalk.white(`    - ${c.symbol} (${c.baseSymbol || c.name})`));
   console.log();
-  console.log(chalk.cyan(`  Selected ${selectedContracts.length} symbol(s):`));
-  for (const c of selectedContracts) {
-    console.log(chalk.white(`    - ${c.symbol} (${c.baseSymbol || c.name})`));
-  }
-  console.log();
-  
   return selectedContracts;
 };
 
@@ -389,8 +384,11 @@ const selectMultipleSymbols = async (service, account) => {
 const selectSymbol = async (service, account) => {
   const spinner = ora({ text: 'Loading symbols...', color: 'yellow' }).start();
   
-  // Ensure we have a logged-in service
-  if (!service.loginInfo && service.credentials) {
+  // Debug: log service type
+  const serviceType = service.constructor?.name || 'Unknown';
+  
+  // Ensure we have a logged-in service (for direct RithmicService, not BrokerClient)
+  if (!service.loginInfo && service.credentials && typeof service.login === 'function') {
     spinner.text = 'Reconnecting to broker...';
     const loginResult = await service.login(service.credentials.username, service.credentials.password);
     if (!loginResult.success) {
@@ -399,9 +397,17 @@ const selectSymbol = async (service, account) => {
     }
   }
   
-  const contractsResult = await service.getContracts();
+  let contractsResult;
+  try {
+    contractsResult = await service.getContracts();
+  } catch (err) {
+    spinner.fail(`getContracts exception: ${err.message}`);
+    console.log(chalk.gray(`  Service: ${serviceType} | Tip: Try "hqx login"`));
+    return null;
+  }
   if (!contractsResult.success || !contractsResult.contracts?.length) {
-    spinner.fail(`Failed to load contracts: ${contractsResult.error || 'No contracts'}`);
+    spinner.fail(`Failed to load contracts: ${contractsResult.error || 'No contracts available'}`);
+    console.log(chalk.gray(`  Service: ${serviceType} | Tip: Try "hqx login"`));
     return null;
   }
   

package/src/services/rithmic-broker/client.js

@@ -211,7 +211,13 @@ class RithmicBrokerClient extends EventEmitter {
    */
   async getContracts() {
     const result = await this._request('getContracts', { propfirmKey: this.propfirmKey });
-    if (result.error) return { success: false, contracts: [] };
+    if (result.error) {
+      // Debug: show exactly what error came from daemon
+      if (process.env.HQX_DEBUG === '1') {
+        console.log('[BrokerClient] getContracts error:', result.error, 'propfirmKey:', this.propfirmKey);
+      }
+      return { success: false, contracts: [], error: result.error };
+    }
     return result.payload || { success: true, contracts: [] };
   }
 

package/src/services/rithmic-broker/daemon.js

@@ -299,8 +299,29 @@ class RithmicBrokerDaemon {
 
   async _handleGetContracts(payload, requestId) {
     const conn = this.connections.get(payload.propfirmKey);
-    if (!conn?.service) return { error: 'Not connected', requestId };
-    return { type: 'contracts', payload: await conn.service.getContracts(), requestId };
+    if (!conn?.service) {
+      log('WARN', 'getContracts: Not connected', { propfirm: payload.propfirmKey, hasConn: !!conn });
+      return { error: 'Not connected to broker', requestId };
+    }
+    
+    // Log service state for debugging
+    const hasCredentials = !!conn.service.credentials;
+    const hasTickerConn = !!conn.service.tickerConn;
+    log('DEBUG', 'getContracts request', { propfirm: payload.propfirmKey, hasCredentials, hasTickerConn });
+    
+    try {
+      const result = await conn.service.getContracts();
+      log('DEBUG', 'getContracts result', { 
+        propfirm: payload.propfirmKey, 
+        success: result.success, 
+        count: result.contracts?.length || 0,
+        error: result.error 
+      });
+      return { type: 'contracts', payload: result, requestId };
+    } catch (err) {
+      log('ERROR', 'getContracts exception', { propfirm: payload.propfirmKey, error: err.message, stack: err.stack?.split('\n')[1] });
+      return { type: 'contracts', payload: { success: false, error: err.message, contracts: [] }, requestId };
+    }
   }
 
   async _handleSearchContracts(payload, requestId) {