Pause stoppt jetzt auch Verkaufsleitern und Auto-Redeem
Behebt #570. "Trading pausieren" hielt drei Wege nicht an, die Orders am CLOB platzieren: 1. SellLadderService: Eine laufende Verkaufsleiter preiste alle 5 s weiter nach, storniert und neu platziert. Sie pruefte weder Pause noch Modus. Die Engine ignoriert bei Pause zwar alle Signale, die bereits laufende Leiter lief aber ungebremst. 2. TraderMonitorService: Der REST-Pre-Redeem feuert aus der Positions-Synchronisierung, die bewusst auch bei Pause weiterlaeuft. Er pruefte nur LiveTradingMode. 3. PolymarketWssClient: Die Pause wurde nur beim Verbindungsaufbau geprueft. Auf einer bereits offenen Verbindung loesten Kursmeldungen den Auto-Redeem weiter aus. Die Ursache ist allen gemeinsam: Jeder Dienst hat seine Sperre selbst nachgebaut, und jeder hat etwas anderes vergessen. Die Regeln liegen jetzt an einer Stelle, in TradingGate, und alle drei benutzen sie: - Verkaufsleiter: Pause oder Modus des Kontotyps "Inaktiv" friert sie ein, bei "Nur Verkauf" laeuft sie weiter. Das entspricht der Engine. - Pre-Redeem (REST und WSS): wie bisher nur bei Modus Aktiv, jetzt zusaetzlich nicht bei Pause. Eine eingefrorene Leiter storniert nichts und platziert nichts. Das gilt auch am Floor, wenn dort die ruhende Order fehlt. Die ruhende GTC-Order bleibt am Buch, genau wie beim Beenden der Anwendung. LastActionAt bleibt unveraendert, nach der Freigabe geht es an der aktuellen Stufe weiter. Protokolliert wird einmal beim Einfrieren und einmal beim Fortsetzen, nicht alle 5 s. CLOB-kritisch, deshalb: - Vor der Aenderung gesichert unter agentspace/scripts/backups/*.vor-570-*.bak. - Orderaufbau und Preislogik sind nicht angefasst. Der Diff umfasst eine Sperre und drei Bedingungen. - Die Sperre steht vor dem ersten CLOB-Zugriff. Die Harness von SellLadderServiceTests setzt jetzt Live = Aktiv. Mit dem Standard Inaktiv waere Floor_replaces_order rot geworden, und Floor_holds_... waere gruen geblieben, aber aus dem falschen Grund: keine Order, weil eingefroren, nicht weil die Order noch ruht. Neu in SellLadderFreezeTests: Pause, Inaktiv, "Nur Verkauf", Floor im eingefrorenen Zustand, Fortsetzen an der aktuellen Stufe, dazu die Entscheidungstabellen von TradingGate. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -0,0 +1,49 @@
|
||||
using PolyTraderSharp;
|
||||
|
||||
namespace PolyTrader.Modules.CopyTrading.Logic
|
||||
{
|
||||
/// <summary>
|
||||
/// Die Sperren für automatische Verkäufe an einer Stelle (#570).
|
||||
///
|
||||
/// <para>Bis zum 13.09.2026 baute jeder Dienst seine Sperre selbst nach, und jeder vergaß etwas
|
||||
/// anderes. Die Verkaufsleiter prüfte weder Pause noch Modus. Der REST-Pre-Redeem im
|
||||
/// TraderMonitorService prüfte nur den Modus, weil die Positions-Synchronisierung, aus der er
|
||||
/// feuert, bewusst auch bei Pause weiterläuft. Der WSS-Auto-Redeem prüfte die Pause nur beim
|
||||
/// Verbindungsaufbau, nicht auf einer bereits offenen Verbindung. "Trading pausieren" hielt damit
|
||||
/// keinen dieser drei Wege an, und alle drei platzieren Orders am CLOB.</para>
|
||||
///
|
||||
/// <para>Die Regeln selbst sind unverändert übernommen, ergänzt um die Pause:
|
||||
/// Pre-Redeem verlangt wie bisher den Modus Aktiv. Eine laufende Leiter läuft bei
|
||||
/// "Nur Verkauf" weiter, wie die Engine dort Verkäufe zulässt.</para>
|
||||
/// </summary>
|
||||
public static class TradingGate
|
||||
{
|
||||
/// <summary>
|
||||
/// Warum eine laufende Verkaufsleiter gerade nicht nachpreisen darf, oder <c>null</c>, wenn sie darf.
|
||||
/// Die Pause sperrt; der Modus des Kontotyps "Inaktiv" ebenfalls; "Nur Verkauf" und Aktiv lassen sie laufen.
|
||||
/// </summary>
|
||||
public static string? LadderFreezeReason(TradingState state, bool isDemo)
|
||||
{
|
||||
if (state.GlobalTradingPaused)
|
||||
return "globale Pause ist aktiv";
|
||||
|
||||
TradingMode mode = isDemo ? state.DemoTradingMode : state.LiveTradingMode;
|
||||
return mode == TradingMode.Inactive
|
||||
? $"{(isDemo ? "Demo" : "Live")}-Modus ist inaktiv"
|
||||
: null;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Ob ein Pre-Redeem-Verkauf (REST oder WSS) ausgelöst werden darf: keine Pause und der Modus des
|
||||
/// Kontotyps auf Aktiv.
|
||||
/// </summary>
|
||||
public static bool PreRedeemAllowed(TradingState state, bool isDemo)
|
||||
{
|
||||
if (state.GlobalTradingPaused)
|
||||
return false;
|
||||
|
||||
TradingMode mode = isDemo ? state.DemoTradingMode : state.LiveTradingMode;
|
||||
return mode == TradingMode.Active;
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -195,13 +195,17 @@ namespace PolyTraderSharp.Services
|
||||
if ((DateTime.UtcNow - redeemState.LastAttempt).TotalMinutes < 5) continue; // Wait 5 min between attempts
|
||||
}
|
||||
|
||||
if (!acc.IsDemo && _state.LiveTradingMode == TradingMode.Active)
|
||||
// #570: Die Pause wird in ExecuteAsync nur beim Verbindungsaufbau geprueft. Auf einer
|
||||
// bereits offenen Verbindung kommen die Kursmeldungen weiter hier an - deshalb die
|
||||
// Sperre an der Stelle, die die Order ausloest. Der Kurs selbst wird weiter
|
||||
// aktualisiert, die Anzeige bleibt also richtig.
|
||||
if (!acc.IsDemo && TradingGate.PreRedeemAllowed(_state, isDemo: false))
|
||||
{
|
||||
_logger.Trade($"🚨 [AUTO REDEEM] {acc.Name} | {pos.MarketQuestion} | Preis >= {_copyState.GetAccountSettings(acc.AccountId).PreRedeemLimit}");
|
||||
// Best effort non-blocking
|
||||
_ = Task.Run(async () => await ExecuteAutoRedeemLive(acc, pos, price));
|
||||
}
|
||||
else if (acc.IsDemo && _state.DemoTradingMode == TradingMode.Active)
|
||||
else if (acc.IsDemo && TradingGate.PreRedeemAllowed(_state, isDemo: true))
|
||||
{
|
||||
_logger.Trade($"🚨 [AUTO REDEEM DEMO] {acc.Name} | {pos.MarketQuestion} | Preis >= {_copyState.GetAccountSettings(acc.AccountId).PreRedeemLimit}");
|
||||
_ = Task.Run(() => ExecuteAutoRedeemDemo(acc, pos, price));
|
||||
|
||||
@@ -35,6 +35,9 @@ namespace PolyTraderSharp.Services
|
||||
private readonly IPositionRepository _positionRepo;
|
||||
private readonly IOrderEventLog _orderEvents;
|
||||
|
||||
/// <summary>Leitern, deren Einfrieren bereits protokolliert ist (#570) – eine Meldung je Position statt einer alle 5 s.</summary>
|
||||
private readonly System.Collections.Concurrent.ConcurrentDictionary<string, string> _frozenLadders = new();
|
||||
|
||||
public SellLadderService(
|
||||
CopyTradingState copyState,
|
||||
TradingState state,
|
||||
@@ -161,6 +164,7 @@ namespace PolyTraderSharp.Services
|
||||
|
||||
// Startorder fehlgeschlagen: Claim + ExitPending zurücknehmen, Position bleibt handelbar.
|
||||
_copyState.ExitLadders.TryRemove(key, out _);
|
||||
_frozenLadders.TryRemove(key, out _);
|
||||
pos.ExitPending = false;
|
||||
_copyState.PendingOrderTimestamps[key] = (DateTime.UtcNow.AddSeconds(-15), sourceTraderId, "SELL");
|
||||
_logger.TradeReasoning($"❌ [SELL-LEITER · {reasonTag}] Startorder fehlgeschlagen: {result}. Position bleibt; neuer Versuch beim nächsten Signal/Sync.");
|
||||
@@ -195,6 +199,7 @@ namespace PolyTraderSharp.Services
|
||||
if (!_state.Accounts.TryGetValue(ladder.AccountId, out var account))
|
||||
{
|
||||
_copyState.ExitLadders.TryRemove(key, out _);
|
||||
_frozenLadders.TryRemove(key, out _);
|
||||
return;
|
||||
}
|
||||
|
||||
@@ -202,6 +207,7 @@ namespace PolyTraderSharp.Services
|
||||
if (!account.OpenPositions.TryGetValue(ladder.TokenId, out var pos))
|
||||
{
|
||||
_copyState.ExitLadders.TryRemove(key, out _);
|
||||
_frozenLadders.TryRemove(key, out _);
|
||||
_logger.Trade($"✅ [SELL-LEITER fertig] {account.Name} | {ladder.MarketQuestion} | Position gefüllt/geschlossen (Sync).");
|
||||
return;
|
||||
}
|
||||
@@ -212,6 +218,7 @@ namespace PolyTraderSharp.Services
|
||||
if (pos.Size < CopyTradingRisk.MinShares)
|
||||
{
|
||||
_copyState.ExitLadders.TryRemove(key, out _);
|
||||
_frozenLadders.TryRemove(key, out _);
|
||||
pos.ExitPending = false;
|
||||
if (!account.IsDemo) _positionRepo.UpsertLive(account.AccountId, pos);
|
||||
_logger.Warning($"🧹 [SELL-LEITER Dust] {account.Name} | {ladder.MarketQuestion}: Restgröße {pos.Size:F2} < Minimum {CopyTradingRisk.MinShares} Shares – Leiter beendet, Rest gehalten.");
|
||||
@@ -220,6 +227,23 @@ namespace PolyTraderSharp.Services
|
||||
return;
|
||||
}
|
||||
|
||||
// #570: Pause und inaktiver Modus frieren die Leiter ein. Bis zum 13.09.2026 preiste sie auch
|
||||
// nach "Trading pausieren" weiter nach und platzierte Orders am CLOB - die Engine ignoriert bei
|
||||
// Pause alle Signale, die bereits laufende Leiter lief aber ungebremst. Die Sperre steht vor
|
||||
// dem ersten CLOB-Zugriff (Existenzpruefung am Floor, Storno, Neuplatzierung). Die ruhende
|
||||
// GTC-Order bleibt am Buch, genau wie beim Beenden der Anwendung; LastActionAt bleibt
|
||||
// unveraendert, damit es nach der Freigabe an der aktuellen Stufe weitergeht.
|
||||
string? freezeReason = TradingGate.LadderFreezeReason(_state, account.IsDemo);
|
||||
if (freezeReason != null)
|
||||
{
|
||||
if (_frozenLadders.TryAdd(key, freezeReason))
|
||||
_logger.Warning($"⏸️ [SELL-LEITER eingefroren] {account.Name} | {ladder.MarketQuestion}: {freezeReason} – " +
|
||||
$"kein Nachpreisen, die ruhende Order bei {ladder.CurrentLimit:F3} bleibt am Buch.");
|
||||
return;
|
||||
}
|
||||
if (_frozenLadders.TryRemove(key, out _))
|
||||
_logger.Info($"▶️ [SELL-LEITER fortgesetzt] {account.Name} | {ladder.MarketQuestion}: weiter ab Limit {ladder.CurrentLimit:F3}.");
|
||||
|
||||
// Noch nicht Zeit für die nächste Stufe?
|
||||
double waited = (DateTime.UtcNow - ladder.LastActionAt).TotalSeconds;
|
||||
if (waited < SellLogic.LadderIntervalSeconds(ladder.IsHf)) return;
|
||||
|
||||
@@ -741,7 +741,9 @@ namespace PolyTraderSharp.Services
|
||||
|
||||
if (allowAttempt)
|
||||
{
|
||||
if (!acc.IsDemo && _state.LiveTradingMode == TradingMode.Active)
|
||||
// #570: Die Positions-Synchronisierung laeuft auch bei Pause weiter ("Always sync");
|
||||
// der Pre-Redeem-Verkauf, der aus ihr heraus Orders platziert, darf das nicht.
|
||||
if (!acc.IsDemo && TradingGate.PreRedeemAllowed(_state, isDemo: false))
|
||||
{
|
||||
_logger.Trade($"🚨 [REST AUTO REDEEM] {acc.Name} | {existing.MarketQuestion} | Preis >= {_copyState.GetAccountSettings(acc.AccountId).PreRedeemLimit}");
|
||||
_ = Task.Run(async () => await ExecuteRestAutoRedeemLive(acc, existing));
|
||||
|
||||
Reference in New Issue
Block a user