@tradejs/cli 3.1.29-beta.286 → 3.1.29-beta.287

package/dist/lib/mcpReplayProcess.js

@@ -62,14 +62,16 @@ const timer = setInterval(() => {
     if (result.exitCode !== 0) {
       let fd;
       try {
-        fd = fs.openSync(file + '.log', fs.constants.O_RDONLY | fs.constants.O_NOFOLLOW);
+        const logId = result.logRequestId === undefined ? id : result.logRequestId;
+        if (typeof logId !== 'string' || !/^[a-f0-9]{64}$/.test(logId)) throw new Error('Invalid replay log identity');
+        fd = fs.openSync(path.join(root, logId) + '.log', fs.constants.O_RDONLY | fs.constants.O_NOFOLLOW);
         const size = fs.fstatSync(fd).size;
         const tail = Buffer.alloc(Math.min(size, 65536));
         fs.readSync(fd, tail, 0, tail.length, Math.max(0, size - tail.length));
         // The worker redacts each line before persisting or returning it through MCP.
         for (const line of tail.toString('utf8').split(/[\\r\\n]+/).filter(Boolean).slice(-150)) console.error('Host replay: ' + line);
       } catch (error) {
-        console.error('Host replay log unavailable: ' + error.code);
+        console.error('Host replay: ' + result.message + '; log unavailable: ' + (error.code || error.message));
       } finally { if (fd !== undefined) fs.closeSync(fd); }
     }
     clearInterval(timer);

package/dist/lib/strategyResearch.js

@@ -2364,6 +2364,10 @@ var validateStrategyResearchSpec = (value) => {
   };
   if (!spec.budget || !spec.selection || !spec.diagnostics)
     throw new Error("Missing research budgets or criteria");
+  if (spec.featurePaths !== void 0 && (!Array.isArray(spec.featurePaths) || !spec.featurePaths.length || spec.featurePaths.length > 64 || new Set(spec.featurePaths).size !== spec.featurePaths.length || spec.featurePaths.some(
+    (path13) => typeof path13 !== "string" || !/^additionalIndicators\.[a-zA-Z0-9_.]+$/.test(path13) || classifyAiPocketFeaturePath(path13.split(".")) !== "eligible"
+  )))
+    throw new Error("Feature paths must be unique causal market features");
   positive(spec.budget.maxVariants, "maxVariants", 12);
   if (!Number.isInteger(spec.minQuality) || spec.minQuality < 1 || spec.minQuality > 5)
     throw new Error("minQuality must be 1 through 5");
@@ -4392,6 +4396,7 @@ var runStrategyResearch = async (parameters) => {
           parentSpec.selection,
           parentSpec.portfolio,
           parentSpec.minQuality,
+          parentSpec.featurePaths,
           parentSpec.diagnostics
         ]) !== sha256Json([
           spec.core.variants.map((item) => item.resolvedConfig),
@@ -4400,6 +4405,7 @@ var runStrategyResearch = async (parameters) => {
           spec.selection,
           spec.portfolio,
           spec.minQuality,
+          spec.featurePaths,
           spec.diagnostics
         ]) || parentState.lineage.runtimeSha256 !== state.lineage.runtimeSha256 || parentState.lineage.contextEnvironmentSha256 !== state.lineage.contextEnvironmentSha256)
           throw new Error(
@@ -5779,9 +5785,10 @@ var createNativeStrategyResearchBackend = (parameters) => {
       throw new Error(
         "Tester workers are already active; research will resume when the host is free"
       );
-    if (import_node_os2.default.freemem() < spec.budget.minimumFreeMemoryMb * 1024 * 1024)
+    const availableBytes = import_node_os2.default.platform() === "darwin" ? process.availableMemory() : import_node_os2.default.freemem();
+    if (!Number.isFinite(availableBytes) || availableBytes < spec.budget.minimumFreeMemoryMb * 1024 * 1024)
       throw new Error(
-        "Insufficient free host memory for the frozen research budget"
+        "Insufficient available host memory for the frozen research budget"
       );
   };
   return {
@@ -6234,9 +6241,14 @@ var createNativeStrategyResearchBackend = (parameters) => {
         throw new Error(
           `Native/export reconciliation is ${reconciliation.status}: ${reconciliation.reasons.join("; ")}`
         );
+      const researchSignalId = (symbol, signalId) => JSON.stringify([symbol, signalId]);
       const trades = loaded.trades.filter(
         (trade) => trade.signalTimestamp >= spec.core.window.start && trade.signalTimestamp < spec.core.window.end && trade.exitTimestamp < spec.core.window.end
-      );
+      ).map((trade) => ({
+        ...trade,
+        // Strategy-generated signal IDs need not be unique across symbols.
+        signalId: researchSignalId(trade.symbol, trade.signalId)
+      }));
       const tradeBySignal = new Map(
         trades.map((trade) => [trade.signalId, trade])
       );
@@ -6260,9 +6272,10 @@ var createNativeStrategyResearchBackend = (parameters) => {
           for await (const line of lines) {
             if (!line.trim()) continue;
             const row = JSON.parse(line);
-            const trade = tradeBySignal.get(row.signalId);
-            if (!trade || seen.has(row.signalId)) continue;
-            seen.add(row.signalId);
+            const signalId = researchSignalId(row.symbol, row.signalId);
+            const trade = tradeBySignal.get(signalId);
+            if (!trade || seen.has(signalId)) continue;
+            seen.add(signalId);
             if (discoveryOnly && (row.timestamp >= spec.testSince || trade.exitTimestamp >= spec.testSince)) {
               if (row.timestamp < spec.testSince) purgedLabels++;
               continue;
@@ -6300,7 +6313,7 @@ var createNativeStrategyResearchBackend = (parameters) => {
               featureProfile: "all"
             });
             for (const key of Object.keys(features))
-              if (!key.startsWith("additionalIndicators.") || features[key] == null)
+              if (!key.startsWith("additionalIndicators.") || features[key] == null || spec.featurePaths && !spec.featurePaths.includes(key))
                 delete features[key];
             featureValues += Object.keys(features).length;
             if (featureValues > 5e6)
@@ -6319,7 +6332,7 @@ var createNativeStrategyResearchBackend = (parameters) => {
                 pathCoverage[key]++;
             }
             rows.push({
-              signalId: row.signalId,
+              signalId,
               symbol: row.symbol,
               strategy: row.strategyName,
               timestamp: row.timestamp,
@@ -6334,7 +6347,7 @@ var createNativeStrategyResearchBackend = (parameters) => {
             });
             if (discoveryOnly) {
               eventInputs.push({
-                signalId: row.signalId,
+                signalId,
                 timestamp: row.timestamp,
                 symbol: row.symbol,
                 direction: row.direction,
@@ -6345,8 +6358,8 @@ var createNativeStrategyResearchBackend = (parameters) => {
               });
               const key = `${row.direction}:${trade.netProfit > 0}`;
               const bucket = sampleBuckets.get(key) ?? [];
-              bucket.push(row.signalId);
-              sampleInputs.set(row.signalId, row);
+              bucket.push(signalId);
+              sampleInputs.set(signalId, row);
               bucket.sort((a, b) => sha256Json(a).localeCompare(sha256Json(b)));
               while (bucket.length > spec.diagnostics.samplePerSideAndOutcome)
                 sampleInputs.delete(bucket.pop());

package/dist/scripts/mcpWorker.js

@@ -69,14 +69,16 @@ const timer = setInterval(() => {
     if (result.exitCode !== 0) {
       let fd;
       try {
-        fd = fs.openSync(file + '.log', fs.constants.O_RDONLY | fs.constants.O_NOFOLLOW);
+        const logId = result.logRequestId === undefined ? id : result.logRequestId;
+        if (typeof logId !== 'string' || !/^[a-f0-9]{64}$/.test(logId)) throw new Error('Invalid replay log identity');
+        fd = fs.openSync(path.join(root, logId) + '.log', fs.constants.O_RDONLY | fs.constants.O_NOFOLLOW);
         const size = fs.fstatSync(fd).size;
         const tail = Buffer.alloc(Math.min(size, 65536));
         fs.readSync(fd, tail, 0, tail.length, Math.max(0, size - tail.length));
         // The worker redacts each line before persisting or returning it through MCP.
         for (const line of tail.toString('utf8').split(/[\\r\\n]+/).filter(Boolean).slice(-150)) console.error('Host replay: ' + line);
       } catch (error) {
-        console.error('Host replay log unavailable: ' + error.code);
+        console.error('Host replay: ' + result.message + '; log unavailable: ' + (error.code || error.message));
       } finally { if (fd !== undefined) fs.closeSync(fd); }
     }
     clearInterval(timer);