From d33d695db523e4da9fe8f9dfd44fcaf023984211 Mon Sep 17 00:00:00 2001
From: Richard <bergmann.richard@fastmail.de>
Date: Sun, 13 Sep 2026 13:51:24 +0200
Subject: [PATCH] WebUI: Trading beenden, ohne die Anwendung zu beenden

Umsetzung von #563.

Entscheidung Richard, 13.09.2026: Beenden soll ueber das WebUI gehen, aber
beschraenkt auf das Trading. Ein Ende der ganzen Anwendung naehme den Webserver
mit; danach waere das WebUI nicht mehr erreichbar.

POST /api/dashboard/stop-trading setzt Live- und Demo-Modus auf Inaktiv und
schaltet die Pause ein. Damit halten CopyTrading und ResolutionFarming (#569)
an, wie beim Ende der Anwendung. Offene Orders an der Boerse werden bewusst
nicht storniert: Das tut auch das Beenden der App nicht, und ein Eingriff in
die CLOB-Integration gehoert nicht in einen Notschalter. Der Dialog sagt das
ausdruecklich.

Die Route verlangt {"confirmed": true}. Ein Aufruf ohne Absicht, etwa aus einem
alten Skript, beendet nichts. Der Vorgang wird als Warnung protokolliert, mit
den Modi vorher.

Im Dashboard steht unter der Notfall-Pause jetzt "Trading beenden ...". Der
Dialog ist wie im Desktop 10 Sekunden gesperrt, Abbrechen und Esc gehen
jederzeit. Wieder aufgenommen wird nur ueber die Modus-Schalter. Die Pause
aufzuheben genuegt nicht, das haelt ein Test fest.

Offen bis #570: Laufende Verkaufsleitern und der REST-Pre-Redeem pruefen die
Pause nicht. Der Inaktiv-Modus, den dieser Knopf setzt, sperrt den Pre-Redeem
bereits; die Leitern friert erst #570 ein.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 .../Endpoints/DashboardEndpoints.cs           | 41 +++++++++
 src/PolyTrader.Web/wwwroot/index.html         | 37 ++++++++
 src/PolyTrader.Web/wwwroot/js/app.js          | 61 +++++++++++++
 .../WebUi/StopTradingEndpointTests.cs         | 89 +++++++++++++++++++
 4 files changed, 228 insertions(+)
 create mode 100644 tests/PolyTrader.Tests/WebUi/StopTradingEndpointTests.cs

diff --git a/src/PolyTrader.Web/Endpoints/DashboardEndpoints.cs b/src/PolyTrader.Web/Endpoints/DashboardEndpoints.cs
index 9296338..cd0fc4d 100644
--- a/src/PolyTrader.Web/Endpoints/DashboardEndpoints.cs
+++ b/src/PolyTrader.Web/Endpoints/DashboardEndpoints.cs
@@ -78,6 +78,45 @@ namespace PolyTrader.Web.Endpoints
                 return Results.BadRequest(new { error = "Ungültiger Modus oder Parametertyp." });
             });
 
+            // Trading beenden (#563) - das WebUI-Gegenstueck zu "Beenden" im Desktop, beschraenkt auf das
+            // Trading. Ein Ende der ganzen Anwendung naehme den Webserver mit; danach waere das WebUI nicht
+            // mehr erreichbar, und nur ein Zugang zur Maschine braechte es zurueck.
+            //
+            // Beide Modi auf Inaktiv und die Pause an: Damit halten CopyTrading (Engine, Monitor,
+            // Verkaufsleitern) und ResolutionFarming (#569) an, wie beim Ende der Anwendung. Offene Orders
+            // an der Boerse werden bewusst NICHT storniert. Das tut auch das Beenden der App nicht, und
+            // ein Eingriff in die CLOB-Integration gehoert nicht in einen Notschalter. Wieder
+            // aufgenommen wird nur ausdruecklich ueber die Modus-Schalter.
+            //
+            // "confirmed" ist Pflicht: Ein Aufruf ohne Absicht, etwa aus einem alten Skript oder einem
+            // doppelt abgeschickten Formular, soll das Trading nicht beenden.
+            group.MapPost("/stop-trading", (StopTradingRequest? req, TradingState state, TerminalLogger logger) =>
+            {
+                if (req?.Confirmed != true)
+                    return Results.BadRequest(new { error = "Bestätigung fehlt – das Trading wurde nicht beendet." });
+
+                string liveBefore = state.LiveTradingMode.ToString();
+                string demoBefore = state.DemoTradingMode.ToString();
+                bool pausedBefore = state.GlobalTradingPaused;
+
+                state.LiveTradingMode = TradingMode.Inactive;
+                state.DemoTradingMode = TradingMode.Inactive;
+                state.GlobalTradingPaused = true;
+
+                logger.Warning($"🛑 WebUI: Trading beendet (Live {liveBefore} → Inactive, Demo {demoBefore} → Inactive, " +
+                               $"Pause {(pausedBefore ? "war schon an" : "an")}). Anwendung und WebUI laufen weiter; " +
+                               "offene Orders an der Börse bleiben stehen.");
+
+                return Results.Ok(new
+                {
+                    success = true,
+                    liveTradingMode = state.LiveTradingMode.ToString(),
+                    demoTradingMode = state.DemoTradingMode.ToString(),
+                    globalTradingPaused = state.GlobalTradingPaused,
+                    stoppedAt = DateTime.UtcNow
+                });
+            });
+
             group.MapGet("/overview-widgets", (IServiceProvider sp) =>
             {
                 var tradeLog = sp.GetService<ITradeLogRepository>();
@@ -251,5 +290,7 @@ namespace PolyTrader.Web.Endpoints
 
             return routes;
         }
+
+        public record StopTradingRequest(bool Confirmed);
     }
 }
diff --git a/src/PolyTrader.Web/wwwroot/index.html b/src/PolyTrader.Web/wwwroot/index.html
index e31700f..b0c8db6 100644
--- a/src/PolyTrader.Web/wwwroot/index.html
+++ b/src/PolyTrader.Web/wwwroot/index.html
@@ -104,6 +104,11 @@
                     <button class="btn btn-danger" id="btnPause" style="width: 100%; margin-top: 6px;" onclick="togglePause()">
                         Trading pausieren
                     </button>
+                    <!-- #563: Gegenstueck zu "Beenden" im Desktop, beschraenkt auf das Trading. Die ganze
+                         Anwendung zu beenden, naehme den Webserver mit. -->
+                    <button class="btn btn-secondary" id="btnStopTrading" style="width: 100%; margin-top: 8px;" onclick="openStopTradingModal()">
+                        🛑 Trading beenden …
+                    </button>
                 </div>
             </div>
 
@@ -1176,6 +1181,38 @@ Wähle ein Profil und stelle mir eine Frage zu deinen Positionen, Risiken oder d
         </section>
     </main>
 
+    <!-- MODAL: TRADING BEENDEN (#563) -->
+    <!-- Dieselbe Sicherheitsabfrage wie beim Beenden im Desktop: Die Bestaetigung ist 10 Sekunden
+         gesperrt, Abbrechen geht jederzeit. -->
+    <div id="modalStopTrading" class="modal-overlay" style="display: none;" role="dialog" aria-modal="true" aria-labelledby="stopTradingTitle">
+        <div class="modal-dialog" style="max-width: 520px;">
+            <div class="modal-header">
+                <h3 id="stopTradingTitle" style="margin: 0; font-size: 16px;">Trading beenden</h3>
+                <button class="modal-close" onclick="closeStopTradingModal()" aria-label="Schließen">&times;</button>
+            </div>
+            <div class="modal-body" style="display: flex; flex-direction: column; gap: 12px; font-size: 14px; line-height: 1.55;">
+                <p style="margin: 0;">Das beendet das Trading aller Module:</p>
+                <ul style="margin: 0; padding-left: 20px;">
+                    <li>Live- und Demo-Modus werden auf <b>Inaktiv</b> gesetzt, die Pause wird eingeschaltet.</li>
+                    <li>Es werden keine neuen Positionen eröffnet und keine Verkäufe mehr nachgepreist.</li>
+                    <li><b>Anwendung und WebUI laufen weiter.</b></li>
+                </ul>
+                <div class="rate-warning" style="margin: 0;">
+                    Offene Orders an der Börse bleiben stehen – genau wie beim Beenden der Anwendung.
+                    Sie werden nicht storniert.
+                </div>
+                <p style="margin: 0; color: var(--muted); font-size: 13px;">
+                    Wieder aufgenommen wird das Trading nur ausdrücklich über die Modus-Schalter.
+                </p>
+                <div id="stopTradingError" class="rate-warning" style="margin: 0;" hidden></div>
+            </div>
+            <div style="display: flex; justify-content: flex-end; gap: 10px; padding: 14px 20px 20px;">
+                <button class="btn btn-secondary" onclick="closeStopTradingModal()">Abbrechen</button>
+                <button class="btn btn-danger" id="btnConfirmStopTrading" onclick="confirmStopTrading()" disabled>Trading beenden (10 s)</button>
+            </div>
+        </div>
+    </div>
+
     <!-- MODAL: MASTER TRADER EDIT/ADD -->
     <div id="modalTrader" class="modal-overlay" style="display: none;">
         <div class="modal-dialog">
diff --git a/src/PolyTrader.Web/wwwroot/js/app.js b/src/PolyTrader.Web/wwwroot/js/app.js
index eb05d43..1dee65b 100644
--- a/src/PolyTrader.Web/wwwroot/js/app.js
+++ b/src/PolyTrader.Web/wwwroot/js/app.js
@@ -6,6 +6,9 @@ let allLogs = [];
 
 // Init on DOM ready
 document.addEventListener('DOMContentLoaded', () => {
+    document.addEventListener('keydown', e => {
+        if (e.key === 'Escape' && document.getElementById('modalStopTrading')?.style.display === 'flex') closeStopTradingModal();
+    });
     initNavMenus();
     refreshNavModules();
     initSseLogs();
@@ -269,6 +272,64 @@ async function togglePause() {
     } catch (e) {}
 }
 
+// ==================== Trading beenden (#563) ====================
+
+const STOP_TRADING_LOCK_SECONDS = 10;
+let stopTradingTimer = null;
+
+function openStopTradingModal() {
+    const modal = document.getElementById('modalStopTrading');
+    const btn = document.getElementById('btnConfirmStopTrading');
+    const err = document.getElementById('stopTradingError');
+    if (!modal || !btn) return;
+
+    err.hidden = true;
+    let remaining = STOP_TRADING_LOCK_SECONDS;
+    btn.disabled = true;
+    btn.textContent = `Trading beenden (${remaining} s)`;
+    modal.style.display = 'flex';
+
+    clearInterval(stopTradingTimer);
+    stopTradingTimer = setInterval(() => {
+        remaining--;
+        if (remaining > 0) {
+            btn.textContent = `Trading beenden (${remaining} s)`;
+            return;
+        }
+        clearInterval(stopTradingTimer);
+        stopTradingTimer = null;
+        btn.textContent = 'Trading beenden';
+        btn.disabled = false;
+    }, 1000);
+}
+
+function closeStopTradingModal() {
+    clearInterval(stopTradingTimer);
+    stopTradingTimer = null;
+    const modal = document.getElementById('modalStopTrading');
+    if (modal) modal.style.display = 'none';
+}
+
+async function confirmStopTrading() {
+    const btn = document.getElementById('btnConfirmStopTrading');
+    const err = document.getElementById('stopTradingError');
+    btn.disabled = true;
+    try {
+        const res = await fetch('/api/dashboard/stop-trading', {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify({ confirmed: true })
+        });
+        if (!res.ok) throw new Error(await errorText(res));
+        closeStopTradingModal();
+        refreshDashboard();
+    } catch (e) {
+        err.textContent = `Trading konnte nicht beendet werden: ${e.message}`;
+        err.hidden = false;
+        btn.disabled = false;
+    }
+}
+
 // Dashboard Account-Übersicht
 async function loadDashboardAccounts() {
     try {
diff --git a/tests/PolyTrader.Tests/WebUi/StopTradingEndpointTests.cs b/tests/PolyTrader.Tests/WebUi/StopTradingEndpointTests.cs
new file mode 100644
index 0000000..7009ecd
--- /dev/null
+++ b/tests/PolyTrader.Tests/WebUi/StopTradingEndpointTests.cs
@@ -0,0 +1,89 @@
+using System;
+using System.Linq;
+using System.Threading.Tasks;
+using PolyTraderSharp;
+using PolyTraderSharp.Services;
+using Xunit;
+
+namespace PolyTrader.Tests.WebUi
+{
+    /// <summary>
+    /// Trading beenden über das WebUI, ohne die Anwendung zu beenden (#563).
+    /// </summary>
+    public class StopTradingEndpointTests
+    {
+        [Fact]
+        public async Task Beenden_setzt_beide_Modi_auf_Inaktiv_und_die_Pause_an()
+        {
+            await using var web = await WebUiTestHost.StartAsync();
+            web.State.LiveTradingMode = TradingMode.Active;
+            web.State.DemoTradingMode = TradingMode.SellOnly;
+
+            var (status, body) = await web.PostJsonAsync("/api/dashboard/stop-trading", new { confirmed = true });
+
+            Assert.Equal(200, status);
+            Assert.Equal(TradingMode.Inactive, web.State.LiveTradingMode);
+            Assert.Equal(TradingMode.Inactive, web.State.DemoTradingMode);
+            Assert.True(web.State.GlobalTradingPaused);
+            Assert.True(body.GetProperty("globalTradingPaused").GetBoolean());
+        }
+
+        [Fact]
+        public async Task Ohne_Bestaetigung_bleibt_alles_wie_es_ist()
+        {
+            await using var web = await WebUiTestHost.StartAsync();
+            web.State.LiveTradingMode = TradingMode.Active;
+
+            var (withoutBody, _) = await web.PostJsonAsync("/api/dashboard/stop-trading");
+            var (notConfirmed, _) = await web.PostJsonAsync("/api/dashboard/stop-trading", new { confirmed = false });
+
+            Assert.Equal(400, withoutBody);
+            Assert.Equal(400, notConfirmed);
+            Assert.Equal(TradingMode.Active, web.State.LiveTradingMode);
+            Assert.False(web.State.GlobalTradingPaused);
+        }
+
+        [Fact]
+        public async Task Anwendung_und_WebUI_bleiben_danach_erreichbar()
+        {
+            await using var web = await WebUiTestHost.StartAsync();
+
+            await web.PostJsonAsync("/api/dashboard/stop-trading", new { confirmed = true });
+            var summary = await web.GetJsonAsync("/api/dashboard/summary");
+
+            Assert.True(summary.GetProperty("globalTradingPaused").GetBoolean());
+            Assert.Equal("Inactive", summary.GetProperty("liveTradingMode").GetString());
+        }
+
+        [Fact]
+        public async Task Beenden_wird_als_Warnung_protokolliert()
+        {
+            await using var web = await WebUiTestHost.StartAsync();
+
+            await web.PostJsonAsync("/api/dashboard/stop-trading", new { confirmed = true });
+
+            var entry = web.Logger.GetHistory(TimeSpan.FromMinutes(1))
+                .LastOrDefault(e => e.Message.Contains("Trading beendet", StringComparison.Ordinal));
+            Assert.NotNull(entry);
+            Assert.Equal(LogLevel.Warning, entry!.Level);
+            Assert.Contains("Orders an der Börse bleiben stehen", entry.Message);
+        }
+
+        [Fact]
+        public async Task Wiederaufnahme_nur_ausdruecklich_ueber_die_Modus_Schalter()
+        {
+            await using var web = await WebUiTestHost.StartAsync();
+            await web.PostJsonAsync("/api/dashboard/stop-trading", new { confirmed = true });
+
+            // Die Pause aufzuheben genuegt nicht: Die Modi stehen weiter auf Inaktiv.
+            await web.PostJsonAsync("/api/dashboard/trading-mode", new { modeType = "pause", targetMode = "" });
+            Assert.False(web.State.GlobalTradingPaused);
+            Assert.Equal(TradingMode.Inactive, web.State.LiveTradingMode);
+            Assert.Equal(TradingMode.Inactive, web.State.DemoTradingMode);
+
+            var (status, _) = await web.PostJsonAsync("/api/dashboard/trading-mode", new { modeType = "demo", targetMode = "Active" });
+            Assert.Equal(200, status);
+            Assert.Equal(TradingMode.Active, web.State.DemoTradingMode);
+        }
+    }
+}