feat(webview): add back/forward nav and fix dock width

Replace setMinimumWidth with sizeHint + resizeDocks so the AI dock
gets its preferred 600px on first attach without permanently clamping
the minimum, letting users resize freely afterward.

- Back/forward buttons added to WebViewWidget toolbar on all platforms
  (Mac WKWebView, Windows WebView2); virtual interface extended with
  goBack/goForward
- NewWindowRequestedHandler intercepts window.open and target=_blank,
  navigating in-place rather than spawning a detached window that
  WebView2 cannot manage

Author: nullmastermind

src/dialogs/MainWindow.cpp

@@ -3739,6 +3739,8 @@ void MainWindow::attachAiAgentDock(AiAgentDock *dock)
     DockMiddleClickCloser::install(dock);
     if (existing) {
         tabifyDockWidget(existing, dock);
+    } else {
+        resizeDocks({dock}, {600}, Qt::Horizontal);
     }
     dock->setVisible(true);
     dock->raise();

src/docks/AiAgentDock.cpp

@@ -53,7 +53,6 @@ AiAgentDock::AiAgentDock(QString sessionId,
 {
     setAttribute(Qt::WA_DeleteOnClose, true);
     setObjectName(QStringLiteral("AiAgentDock_%1").arg(m_sessionId));
-    setMinimumWidth(600);
     // Spec ("Default dock area"): dock is unrestricted — user may move it to
     // any side. defaultArea() is only consulted on first attach.
     setAllowedAreas(Qt::AllDockWidgetAreas);
@@ -201,6 +200,11 @@ AiAgentDock::~AiAgentDock()
     }
 }
 
+QSize AiAgentDock::sizeHint() const
+{
+    return QSize(600, 400);
+}
+
 void AiAgentDock::closeEvent(QCloseEvent *event)
 {
     if (m_model && m_model->isProcessing()) {

src/docks/AiAgentDock.h

@@ -92,6 +92,7 @@ class AiAgentDock : public QDockWidget
     void goalDebugLogAppended(const QString &entry);
 
 protected:
+    QSize sizeHint() const override;
     void closeEvent(QCloseEvent *event) override;
 
     // Test seam — override in tests to bypass the modal QMessageBox.

src/widgets/WebViewWidget.cpp

@@ -33,6 +33,22 @@ void WebViewWidget::setupToolbar()
     m_toolbarLayout->setContentsMargins(4, 0, 4, 0);
     m_toolbarLayout->setSpacing(4);
 
+    m_backBtn = new QToolButton(toolbarWidget);
+    m_backBtn->setAutoRaise(true);
+    m_backBtn->setIcon(style()->standardIcon(QStyle::SP_ArrowBack));
+    m_backBtn->setToolTip(tr("Back"));
+    m_backBtn->setIconSize(QSize(14, 14));
+    connect(m_backBtn, &QToolButton::clicked, this, &WebViewWidget::goBack);
+    m_toolbarLayout->addWidget(m_backBtn);
+
+    m_forwardBtn = new QToolButton(toolbarWidget);
+    m_forwardBtn->setAutoRaise(true);
+    m_forwardBtn->setIcon(style()->standardIcon(QStyle::SP_ArrowForward));
+    m_forwardBtn->setToolTip(tr("Forward"));
+    m_forwardBtn->setIconSize(QSize(14, 14));
+    connect(m_forwardBtn, &QToolButton::clicked, this, &WebViewWidget::goForward);
+    m_toolbarLayout->addWidget(m_forwardBtn);
+
     m_reloadBtn = new QToolButton(toolbarWidget);
     m_reloadBtn->setAutoRaise(true);
     m_reloadBtn->setIcon(style()->standardIcon(QStyle::SP_BrowserReload));

src/widgets/WebViewWidget.h

@@ -29,6 +29,8 @@ class WebViewWidget : public QWidget
     virtual void navigate(const QUrl &url) = 0;
     virtual void reload() = 0;
     virtual void stop() = 0;
+    virtual void goBack() = 0;
+    virtual void goForward() = 0;
     virtual void destroy() = 0;
     virtual QString debugInfo() const { return QString(); }
 
@@ -68,6 +70,8 @@ class WebViewWidget : public QWidget
     QUrl m_url;
     QVBoxLayout *m_mainLayout = nullptr;
     QHBoxLayout *m_toolbarLayout = nullptr;
+    QToolButton *m_backBtn = nullptr;
+    QToolButton *m_forwardBtn = nullptr;
     QToolButton *m_reloadBtn = nullptr;
     QLabel *m_urlLabel = nullptr;
     QToolButton *m_stopBtn = nullptr;

src/widgets/WebViewWidget_mac.mm

@@ -75,6 +75,8 @@ void navigate(const QUrl &url) override
 
     void reload() override { setLoading(true); [m_webView reload]; }
     void stop() override { [m_webView stopLoading]; setLoading(false); }
+    void goBack() override { if (m_webView) [m_webView goBack]; }
+    void goForward() override { if (m_webView) [m_webView goForward]; }
 
     void destroy() override
     {

src/widgets/WebViewWidget_win.cpp

@@ -150,6 +150,18 @@ class WebViewWidgetWin : public WebViewWidget
         setLoading(false);
     }
 
+    void goBack() override
+    {
+        if (!m_webView) return;
+        m_webView->GoBack();
+    }
+
+    void goForward() override
+    {
+        if (!m_webView) return;
+        m_webView->GoForward();
+    }
+
     void destroy() override
     {
         m_alive->store(false, std::memory_order_release);
@@ -344,6 +356,11 @@ class WebViewWidgetWin : public WebViewWidget
         m_webView->add_DocumentTitleChanged(
             new DocumentTitleChangedHandler(this), &titleToken);
 
+        // Block new windows — navigate in-place instead
+        EventRegistrationToken newWinToken;
+        m_webView->add_NewWindowRequested(
+            new NewWindowRequestedHandler(this), &newWinToken);
+
         // Navigate to initial URL
         setLoading(true);
         m_dbgNavigateCalled = true;
@@ -479,6 +496,32 @@ class WebViewWidgetWin : public WebViewWidget
         }
     };
 
+    // NewWindowRequestedHandler: intercepts window.open / target="_blank" and navigates in-place.
+    struct NewWindowRequestedHandler : ICoreWebView2NewWindowRequestedEventHandler {
+        WebViewWidgetWin *owner;
+        std::shared_ptr<std::atomic<bool>> alive;
+        ULONG refCount = 1;
+        NewWindowRequestedHandler(WebViewWidgetWin *o) : owner(o), alive(o->m_alive) {}
+        HRESULT STDMETHODCALLTYPE QueryInterface(REFIID riid, void **ppv) override {
+            if (IsEqualIID(riid, IID_IUnknown) || IsEqualIID(riid, IID_ICoreWebView2NewWindowRequestedEventHandler)) {
+                *ppv = this; AddRef(); return S_OK;
+            }
+            *ppv = nullptr; return E_NOINTERFACE;
+        }
+        ULONG STDMETHODCALLTYPE AddRef() override { return ++refCount; }
+        ULONG STDMETHODCALLTYPE Release() override { if (--refCount == 0) { delete this; return 0; } return refCount; }
+        HRESULT STDMETHODCALLTYPE Invoke(ICoreWebView2 *, ICoreWebView2NewWindowRequestedEventArgs *args) override {
+            if (!alive->load(std::memory_order_acquire)) return S_OK;
+            LPWSTR uri = nullptr;
+            if (args && SUCCEEDED(args->get_Uri(&uri)) && uri) {
+                owner->m_webView->Navigate(uri);
+                CoTaskMemFree(uri);
+            }
+            if (args) args->put_Handled(TRUE);
+            return S_OK;
+        }
+    };
+
     // --- CDP Discovery ---
 
     void startCdpDiscovery()