fix(ci): cpp clang-tidy now properly reads non-build files only

.gitea/workflows/cpp.yml

@@ -40,5 +40,7 @@ jobs:
               run: |
                   sed -i 's/-mno-direct-extern-access //g' build/compile_commands.json
                   set -o pipefail
-                  run-clang-tidy -p build -j $(nproc) -header-filter="Plugins/.*" 2>&1 | \
-                    grep -v 'Suppressed\|non-user\|non-system\|unknown argument.*mno-direct\|Found compiler error\|warnings generated\.\|^Use -header-filter'
+                  run-clang-tidy -p build -j "$(nproc)" \
+                      -source-filter="^$(pwd)/Plugins/" \
+                      -header-filter=".*Plugins/.*" \
+                      -removed-arg=-mno-direct-extern-access

.gitignore

@@ -1,23 +1,35 @@
-**/__pycache__/
 ./result/
 .pyre/
-.cache/
-.venv/
-.qmlls.ini
-build/
-compile_commands.json
-testpython
 pkg/
 *.tar.*
-uv.lock
-.qtcreator/
 dist/
 **/target/
 **/test-plugins/
 **/Charts/
 network-dev/
-**/zshell.build/
 **/zshell.dist/
+
+# Qml
+.qtcreator/
+.qmlls.ini
+
+# CMake compile artifacts
+.cache/
+compile_commands.json
+build/
+
+# Python
+uv.lock
+testpython
+.venv/
+**/__pycache__/
+
+# Nuitka compile
+**/zshell.pyi
+**/zshell.cpython-*
+**/zshell.build/
+
+# Agents stuff
 .opencode/
 run-agent.sh
 AGENTS.md

CMakeLists.txt

@@ -50,6 +50,7 @@ set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_CXX_EXTENSIONS OFF)
 set(CMAKE_RUNTIME_OUTPUT_DIRECTORY "${CMAKE_BINARY_DIR}/lib")
 set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+set(QT_NO_PRIVATE_MODULE_WARNING ON)
 set(ENABLE_MODULES "plugin;shell;cli;m3shapes" CACHE STRING "Modules to build/install")
 
 set(INSTALL_LIBDIR "${CMAKE_INSTALL_LIBDIR}/ZShell" CACHE STRING "Library install dir")

Components/TextFieldBase.qml

@@ -12,7 +12,7 @@ TextField {
 	implicitHeight: contentHeight + topPadding + bottomPadding
 	implicitWidth: contentWidth + leftPadding + rightPadding
 	placeholderTextColor: Colors.palette.m3onSurfaceVariant // No anim cause placeholder is custom
-	renderType: echoMode === TextField.Password ? TextField.QtRendering : TextField.QtRendering
+	renderType: TextField.QtRendering
 	selectedTextColor: color
 	selectionColor: Qt.alpha(Colors.palette.m3primary, 0.4)
 	verticalAlignment: TextInput.AlignVCenter

Drawers/Exclusions.qml

@@ -43,13 +43,6 @@ Scope {
 		hasBar: root.geometry.barOnBottom
 	}
 
-	Timer {
-		interval: 5000
-		running: true
-
-		onTriggered: console.log("top height:", top.exclusiveZone, "left width:", left.exclusiveZone, "right width:", right.exclusiveZone, "bottom height:", bottom.exclusiveZone)
-	}
-
 	component ExclusionZone: CustomWindow {
 		property bool hasBar
 

Modules/Bar/Components/Tray/TrayItem.qml

@@ -79,6 +79,7 @@ Item {
 
 		anchors.centerIn: parent
 		antialiasing: true
+		backer.retainWhileLoading: true
 		color: root.current && !Config.bar.tray.showOnHover ? Colors.palette.m3onPrimary : Colors.palette.m3onSurface
 		implicitSize: Config.bar.tray.trayIconSize * root.dpr
 		layer.enabled: Config.bar.tray.recolorIcons

Modules/Notifications/Content.qml

@@ -28,7 +28,10 @@ Item {
 		for (let i = 0; i < count; i++)
 			height += (list.itemAtIndex(i) as NotifWrapper)?.nonAnimHeight ?? 0;
 
-		if (panels.popouts.hasCurrent && (panels.popouts.currentCenter + (panels.popouts.current?.width / 2)) > panels.notifications.x || screenState.dashboard)
+		if (panels.popouts.hasCurrent && (panels.popouts.currentCenter + (panels.popouts.current?.width / 2)) > panels.notifications.x)
+			return 0;
+
+		if (screenState.dashboard && (panels.dashboard.x + panels.dashboard.width >= panels.notifications.x))
 			return 0;
 
 		if (screenState.osd) {

Modules/Settings/Pages.qml

@@ -38,7 +38,6 @@ Item {
 		id: container
 
 		anchors.fill: parent
-		layer.enabled: opacity < 1
 		objectName: "PageContainer"
 
 		Component.onCompleted: root.loadPage(root.sState.currentPageIdx)

Modules/Sidebar/Chat/ChatContent.qml

@@ -3,6 +3,7 @@ pragma ComponentBehavior: Bound
 import QtQuick
 import Quickshell
 import QtQuick.Layouts
+import ZShell.Services
 import ZShell.Components
 import ZShell.Config
 import ZShell.Llm
@@ -90,7 +91,9 @@ Item {
 	signal requestClose
 
 	readonly property bool atNewestEnd: !listLoader.item || listLoader.item.contentY < 4
+	readonly property bool importing: importIds.length > 0
 
+	property var importIds: []
 	property var pendingAttachments: []
 
 	function attachFiles(): void {
@@ -541,7 +544,7 @@ Item {
 		bg.border.color: Colors.palette.m3outlineVariant
 		bg.color: Colors.tPalette.m3surfaceContainerLowest
 		fastGlow: LlmClient.busy
-		glow.opacity: input.activeFocus && (!LlmClient.busy || ChatState.isWindow)
+		glow.opacity: !ChatState.narrowSidebarExpanded ? (input.activeFocus && (!LlmClient.busy || ChatState.isWindow) ? 1 : 0) : 0
 		font: Tokens.font.body.small
 		implicitHeight: Math.min(root.height / 5, contentHeight + topPadding + bottomPadding)
 		focus: true
@@ -578,4 +581,52 @@ Item {
 				root.send(text);
 		}
 	}
+
+	Connections {
+		target: DropImporter
+
+		function onImportFinished(id: int, paths: var, errors: var): void {
+			if (!root.importIds.includes(id))
+				return;
+			root.importIds = root.importIds.filter(i => i !== id);
+
+			if (paths.length > 0) {
+				const added = root.chatData.addAttachments(paths);
+				root.pendingAttachments = root.pendingAttachments.concat(added);
+			}
+			for (const e of errors)
+				console.warn("Drop import:", e);
+		}
+	}
+
+	DropArea {
+		id: dropArea
+
+		anchors.fill: parent
+
+		onEntered: drag => {
+			drag.accepted = drag.hasUrls || drag.hasText || drag.hasHtml || drag.formats.some(f => f.startsWith("image/"));
+		}
+
+		onDropped: drop => {
+			const formats = drop.formats;
+			const urls = drop.hasUrls ? drop.urls : [];
+			const text = drop.hasText ? drop.text : "";
+			const html = drop.hasHtml ? drop.html : "";
+
+			const imgFormat = formats.find(f => f.startsWith("image/"));
+			const raw = imgFormat ? drop.getDataAsArrayBuffer(imgFormat) : new ArrayBuffer(0);
+
+			switch (DropImporter.decide(urls, text, html, raw.byteLength > 0)) {
+			case DropImporter.Attach:
+				root.importIds = [...root.importIds, DropImporter.importAttachments(urls, html, raw)];
+				break;
+			case DropImporter.InsertText:
+				input.insert(input.cursorPosition, text || urls.map(u => u.toString()).join("\n"));
+				root.focusInput();
+				break;
+			}
+			drop.acceptProposedAction();
+		}
+	}
 }

Modules/Sidebar/Chat/ChatDelegate.qml

@@ -91,6 +91,50 @@ CustomRect {
 
 		opacity: root.slim ? 0 : 1
 
+		state: titleText.readOnly ? "static" : "editing"
+
+		states: [
+			State {
+				name: "static"
+				PropertyChanges {
+					staticText.opacity: 1
+				}
+				PropertyChanges {
+					titleText.opacity: 0
+				}
+			},
+			State {
+				name: "editing"
+				PropertyChanges {
+					staticText.opacity: 0
+				}
+				PropertyChanges {
+					titleText.opacity: 1
+				}
+			}
+		]
+
+		transitions: [
+			// Going static: titleText fades out (animated), staticText snaps in
+			Transition {
+				to: "static"
+				Anim {
+					target: titleText
+					property: "opacity"
+					type: Anim.SlowEffects
+				}
+			},
+			// Going editing: staticText fades out (animated), titleText snaps in
+			Transition {
+				to: "editing"
+				Anim {
+					target: staticText
+					property: "opacity"
+					type: Anim.SlowEffects
+				}
+			}
+		]
+
 		Behavior on opacity {
 			Anim {
 				type: Anim.DefaultEffects
@@ -106,7 +150,7 @@ CustomRect {
 			color: Colors.layer(Colors.palette.m3surfaceContainerHighest, 1)
 			implicitHeight: titleText.implicitHeight + Tokens.padding.extraSmall * 2
 			radius: Tokens.rounding.full
-			implicitWidth: titleText.implicitWidth + Tokens.padding.large * 2
+			implicitWidth: titleText.width + Tokens.padding.large * 2
 			opacity: titleText.readOnly ? 0 : 1
 
 			Behavior on opacity {
@@ -122,11 +166,6 @@ CustomRect {
 			anchors.right: parent.right
 			implicitHeight: titleText.implicitHeight + Tokens.padding.small * 2
 
-			layer.enabled: true
-			layer.effect: OpacityMask {
-				maskSource: fadeMask
-			}
-
 			TextFieldBase {
 				id: titleText
 
@@ -134,10 +173,12 @@ CustomRect {
 				anchors.leftMargin: readOnly ? 0 : Tokens.padding.large
 				anchors.top: parent.top
 				anchors.topMargin: readOnly ? Tokens.padding.small : Tokens.padding.extraSmall * 2
+				width: Math.min(implicitWidth, parent.width - Tokens.padding.large * 2)
 				font: Tokens.font.body.medium
 				text: root.modelData.title || qsTr("New chat")
 				readOnly: true
 				color: Colors.palette.m3onSurface
+				opacity: 0
 
 				Behavior on anchors.leftMargin {
 					Anim {}
@@ -146,6 +187,11 @@ CustomRect {
 					Anim {}
 				}
 
+				function cancelTitleEdit(): void {
+					text = root.modelData.title;
+					readOnly = true;
+				}
+
 				onEditingFinished: root.modelData.title = text
 				onReadOnlyChanged: {
 					if (!readOnly) {
@@ -156,27 +202,24 @@ CustomRect {
 					}
 				}
 			}
-		}
 
-		CustomRect {
-			id: fadeMask
+			CustomText {
+				id: staticText
 
-			anchors.fill: titleWrapper
+				anchors.left: parent.left
+				anchors.leftMargin: titleText.readOnly ? 0 : Tokens.padding.large
+				anchors.top: parent.top
+				anchors.topMargin: titleText.readOnly ? Tokens.padding.small : Tokens.padding.extraSmall * 2
+				anchors.right: parent.right
+				font: titleText.font
+				text: titleText.text
+				elide: CustomText.ElideRight
 
-			layer.enabled: true
-			visible: false
-
-			gradient: Gradient {
-				orientation: Gradient.Horizontal
-
-				GradientStop {
-					color: Qt.rgba(1, 1, 1, 1.0)
-					position: 0.85
+				Behavior on anchors.leftMargin {
+					Anim {}
 				}
-
-				GradientStop {
-					color: Qt.rgba(1, 1, 1, 0)
-					position: 1.0
+				Behavior on anchors.topMargin {
+					Anim {}
 				}
 			}
 		}
@@ -323,7 +366,13 @@ CustomRect {
 			inactiveColor: Colors.layer(Colors.palette.m3surfaceContainerHighest, 3)
 			inactiveOnColor: Colors.palette.m3onSurfaceVariant
 
-			onClicked: root.remove(root.modelData)
+			onClicked: {
+				if (!titleText.readOnly) {
+					titleText.cancelTitleEdit();
+					return;
+				}
+				root.remove(root.modelData);
+			}
 		}
 
 		IconButton {
@@ -366,7 +415,10 @@ CustomRect {
 		}
 
 		StateLayer {
-			onClicked: root.expanded = !root.expanded
+			onClicked: {
+				root.expanded = !root.expanded;
+				titleText.readOnly = true;
+			}
 		}
 
 		MaterialIcon {

Modules/Sidebar/Chat/ChatList.qml

@@ -140,7 +140,7 @@ Item {
 		label.transform: Rotation {
 			origin.y: fabRoot.label.height / 2
 			origin.x: fabRoot.label.width / 2
-			angle: ChatState.fabExpanded && (ChatState.narrowSidebarExpanded && !ChatState.isWide || !ChatState.isWindow) ? 135 : 0
+			angle: ChatState.fabExpanded && (ChatState.narrowSidebarExpanded || !ChatState.isWide || !ChatState.isWindow) ? 135 : 0
 
 			Behavior on angle {
 				Anim {}
@@ -148,8 +148,9 @@ Item {
 		}
 
 		onClicked: {
-			if (root.slim && !ChatState.narrowSidebarExpanded) {
+			if ((root.slim && !ChatState.narrowSidebarExpanded) || ChatState.shrinkSidebarOverride) {
 				root.requestExpand();
+				ChatState.shrinkSidebarOverride = false;
 				return;
 			}
 
@@ -183,6 +184,10 @@ Item {
 					name: "pop out in window"
 					icon: "open_in_new"
 				}
+				ListElement {
+					name: "shrink sidebar"
+					icon: "left_panel_close"
+				}
 			}
 
 			CustomRect {
@@ -196,7 +201,18 @@ Item {
 
 				radius: sLayer.pressed ? Tokens.rounding.small : implicitHeight / 2
 				color: Colors.palette.m3primaryContainer
-				visible: !(modelData.name === "pop out in window" && ChatState.isWindow)
+				visible: {
+					const n = modelData.name;
+					const w = ChatState.isWindow;
+					const wide = ChatState.isWide;
+
+					if (n === "pop out in window" && w)
+						return false;
+					if (n === "shrink sidebar" && (!w || !wide))
+						return false;
+					else
+						return true;
+				}
 				clip: true
 
 				Behavior on radius {
@@ -297,6 +313,8 @@ Item {
 							root.newChatRequest();
 						else if (name === "pop out in window")
 							Detach.create();
+						else if (name === "shrink sidebar")
+							ChatState.shrinkSidebarOverride = true;
 
 						ChatState.fabExpanded = false;
 					}

Modules/Sidebar/Chat/ChatState.qml

@@ -19,7 +19,7 @@ Singleton {
 	property bool settledSlim: false
 	property bool settledWide: false
 	property bool narrowSidebarExpanded: false
-	property bool windowIsWide: false
+	property bool shrinkSidebarOverride: false
 	property ShellScreen screen
 	property bool sidebarVisible: ShellState.forActive().sidebar
 
@@ -28,6 +28,9 @@ Singleton {
 		root.currentIdx = -1;
 	}
 
+	onIsWideChanged: if (isWide)
+		narrowSidebarExpanded = false
+
 	onSidebarVisibleChanged: if (!sidebarVisible)
 		fabExpanded = false
 }

Modules/Sidebar/Chat/Content/CodeBlockView.qml

@@ -220,9 +220,8 @@ CustomRect {
 
 			Layout.alignment: Qt.AlignVCenter
 			color: root.codeHeaderColor
-			font: Tokens.font.label.small
+			font: Tokens.font.label.builders.small.capitalization(Font.Capitalize).build()
 			text: CodeIcons.name(root.language)
-			visible: root.language.length > 0
 		}
 
 		Item {

Modules/Sidebar/Chat/SidebarView.qml

@@ -21,12 +21,8 @@ Item {
 	property bool chatOpen: ChatState.chatSession !== null
 
 	readonly property bool isWide: root.width >= root.breakpoint
-
-	readonly property bool showList: isWide || !chatOpen
-	readonly property bool showContent: !isWide && chatOpen
-
-	function openConversation(conv: ChatSession, index: int): void {
-	}
+	readonly property bool wideExpandedSidebar: isWide && !ChatState.shrinkSidebarOverride
+	readonly property bool narrowExpandedSidebar: !isWide && ChatState.narrowSidebarExpanded
 
 	BlobGroup {
 		id: blobGroup
@@ -54,9 +50,9 @@ Item {
 		anchors.left: parent.left
 		anchors.bottom: parent.bottom
 		anchors.margins: Tokens.padding.medium
-		implicitWidth: root.isWide || ChatState.narrowSidebarExpanded ? Config.sidebar.sizes.width : root.iconSize
+		implicitWidth: root.wideExpandedSidebar || root.narrowExpandedSidebar ? Config.sidebar.sizes.width : root.iconSize
 		highlight: true
-		slim: !root.isWide && !ChatState.narrowSidebarExpanded
+		slim: !root.wideExpandedSidebar && !root.narrowExpandedSidebar
 		z: 1
 
 		Behavior on implicitWidth {
@@ -86,13 +82,13 @@ Item {
 		}
 	}
 
-	ClipRect {
+	Item {
 		anchors.fill: parent
 		anchors.leftMargin: invertedRect.borderLeft
 		anchors.topMargin: invertedRect.borderTop
 		anchors.bottomMargin: invertedRect.borderBottom
 		anchors.rightMargin: invertedRect.borderRight
-		radius: invertedRect.radius
+		clip: true
 
 		children: [contentArea, dim]
 	}
@@ -100,9 +96,10 @@ Item {
 	CustomRect {
 		id: dim
 		anchors.fill: parent
+		radius: invertedRect.radius
 
 		color: Colors.palette.m3shadow
-		opacity: ChatState.narrowSidebarExpanded && !root.isWide ? 0.3 : 0
+		opacity: root.narrowExpandedSidebar ? 0.3 : 0
 		z: 1
 
 		Behavior on opacity {
@@ -155,7 +152,7 @@ Item {
 			id: dimArea
 			anchors.fill: parent
 			preventStealing: true
-			enabled: !root.isWide && ChatState.narrowSidebarExpanded
+			enabled: root.narrowExpandedSidebar
 			cursorShape: undefined
 
 			onClicked: {
@@ -175,12 +172,6 @@ Item {
 		}
 	}
 
-	Binding {
-		target: ChatState
-		property: "windowIsWide"
-		value: root.showList
-	}
-
 	Binding {
 		target: ChatState
 		property: "isWide"

Plugins/ZShell/Config/fontbuilder.cpp

@@ -52,7 +52,7 @@ FontBuilder FontBuilder::letterSpacing(qreal spacing, bool absolute) {
 	return *this;
 }
 
-FontBuilder FontBuilder::capitalisation(QFont::Capitalization cap) {
+FontBuilder FontBuilder::capitalization(QFont::Capitalization cap) {
 	m_font.setCapitalization(cap);
 	return *this;
 }

Plugins/ZShell/Config/fontbuilder.hpp

@@ -23,7 +23,7 @@ class FontBuilder {
 	[[nodiscard]] Q_INVOKABLE FontBuilder
 	letterSpacing(qreal spacing, bool absolute = true);
 	[[nodiscard]] Q_INVOKABLE FontBuilder
-	capitalisation(QFont::Capitalization cap);
+	capitalization(QFont::Capitalization cap);
 	[[nodiscard]] Q_INVOKABLE FontBuilder vaxis(const QString& tag, float value);
 	[[nodiscard]] Q_INVOKABLE FontBuilder vaxes(const QVariantMap& axes);
 	[[nodiscard]] Q_INVOKABLE QFont build() const;

Plugins/ZShell/Services/CMakeLists.txt

@@ -22,10 +22,12 @@ qml_module(ZShell-services
 		networkusage.hpp networkusage.cpp
 		lyriccandidate.hpp lyriccandidate.cpp
 		lyrics.hpp lyrics.cpp
+		dropimporter.hpp dropimporter.cpp
 	LIBRARIES
 		Qt6::Core
 		Qt6::Qml
 		Qt6::Concurrent
+		Qt6::Network
 		PkgConfig::Pipewire
 		PkgConfig::Aubio
 		PkgConfig::Cava

Plugins/ZShell/Services/dropimporter.cpp

@@ -0,0 +1,263 @@
+#include "dropimporter.hpp"
+
+#include <QDir>
+#include <QFile>
+#include <QFileInfo>
+#include <QMetaObject>
+#include <QMimeDatabase>
+#include <QNetworkReply>
+#include <QRegularExpression>
+
+namespace {
+constexpr qint64 kMaxBytes = 100 * 1024 * 1024;
+
+bool isHttp(const QUrl& u) {
+	return u.scheme() == QLatin1String("http") ||
+		   u.scheme() == QLatin1String("https");
+}
+
+QString firstImgSrc(const QString& html) {
+	static const QRegularExpression re(
+		R"(<img\b[^>]*?\bsrc\s*=\s*["']([^"']+)["'])",
+		QRegularExpression::CaseInsensitiveOption);
+	const auto m = re.match(html);
+	if (!m.hasMatch()) return {};
+	return m.captured(1).replace(QLatin1String("&amp;"), QLatin1String("&"));
+}
+
+bool looksLikeUrl(const QString& text) {
+	const QString t = text.trimmed();
+	if (t.isEmpty() || t.contains(QRegularExpression(QStringLiteral("\\s"))))
+		return false;
+	const QUrl u(t);
+	return u.isValid() && (isHttp(u) || u.isLocalFile());
+}
+
+QString sanitize(QString name) {
+	name = QFileInfo(name).fileName();
+	name.replace(
+		QRegularExpression(QStringLiteral("[\\\\/:*?\"<>|\\x00-\\x1f]")),
+		QStringLiteral("_"));
+	return name.left(120);
+}
+
+QString fileNameFor(QNetworkReply* reply, const QUrl& url) {
+	static const QRegularExpression cd(
+		QStringLiteral("filename\\*?=(?:UTF-8'')?\"?([^\";]+)\"?"),
+		QRegularExpression::CaseInsensitiveOption);
+	const auto m =
+		cd.match(QString::fromUtf8(reply->rawHeader("Content-Disposition")));
+	QString name = m.hasMatch()
+					   ? QUrl::fromPercentEncoding(m.captured(1).toUtf8())
+					   : url.fileName();
+	name = sanitize(name);
+	return name.isEmpty() ? QStringLiteral("download") : name;
+}
+} // namespace
+
+namespace ZShell::Services {
+
+DropImporter::DropImporter(QObject* parent) : QObject(parent) {}
+
+DropImporter::Decision DropImporter::decide(
+	const QList<QUrl>& urls,
+	const QString& text,
+	const QString& html,
+	bool hasRawImage) const {
+	for (const QUrl& u : urls)
+		if (u.isLocalFile()) return Attach;
+
+	const QString src = firstImgSrc(html);
+	if (hasRawImage ||
+		(!src.isEmpty() && (text.trimmed().isEmpty() || looksLikeUrl(text))))
+		return Attach;
+
+	QMimeDatabase db;
+	for (const QUrl& u : urls) {
+		if (!isHttp(u) || QFileInfo(u.path()).suffix().isEmpty()) continue;
+		const QMimeType mt = db.mimeTypeForUrl(u);
+		if (!mt.isDefault() && !mt.inherits(QStringLiteral("text/html")))
+			return Attach;
+	}
+
+	if (!text.isEmpty() || !urls.isEmpty()) return InsertText;
+	return Ignore;
+}
+
+int DropImporter::importAttachments(
+	const QList<QUrl>& urls, const QString& html, const QByteArray& rawImage) {
+	const int id = ++m_nextId;
+	m_requests[id].pending = 1;
+
+	if (!rawImage.isEmpty()) {
+		saveBytes(id, rawImage);
+	} else if (const QString src = firstImgSrc(html); !src.isEmpty()) {
+		if (src.startsWith(QLatin1String("data:"), Qt::CaseInsensitive)) {
+			const int comma = src.indexOf(QLatin1Char(','));
+			if (comma < 0) {
+				failWith(id, tr("Malformed data URI"));
+			} else {
+				const bool b64 = src.left(comma).contains(
+					QLatin1String(";base64"), Qt::CaseInsensitive);
+				const QString payload = src.mid(comma + 1);
+				saveBytes(
+					id,
+					b64 ? QByteArray::fromBase64(payload.toLatin1())
+						: QByteArray::fromPercentEncoding(payload.toUtf8()));
+			}
+		} else {
+			const QUrl u(src);
+			if (u.isLocalFile())
+				addLocal(id, u);
+			else if (isHttp(u))
+				download(id, u);
+			else
+				failWith(id, tr("Unsupported image source"));
+		}
+	} else {
+		for (const QUrl& u : urls) {
+			if (u.isLocalFile())
+				addLocal(id, u);
+			else if (isHttp(u))
+				download(id, u);
+			else
+				failWith(id, tr("Unsupported URL: %1").arg(u.toString()));
+		}
+	}
+
+	QMetaObject::invokeMethod(
+		this, [this, id] { finishUnit(id); }, Qt::QueuedConnection);
+	return id;
+}
+
+void DropImporter::addLocal(int id, const QUrl& url) {
+	const QFileInfo fi(url.toLocalFile());
+	if (!fi.exists())
+		failWith(id, tr("File not found: %1").arg(fi.filePath()));
+	else if (fi.isDir())
+		failWith(id, tr("Folders can't be attached: %1").arg(fi.fileName()));
+	else if (fi.size() > kMaxBytes)
+		failWith(id, tr("File too large: %1").arg(fi.fileName()));
+	else
+		succeed(id, fi.absoluteFilePath());
+}
+
+void DropImporter::saveBytes(int id, const QByteArray& bytes) {
+	if (bytes.isEmpty() || bytes.size() > kMaxBytes) {
+		failWith(id, tr("Empty or oversized data"));
+		return;
+	}
+	const QString dir = newSubdir();
+	if (dir.isEmpty()) {
+		failWith(id, tr("No temp directory"));
+		return;
+	}
+
+	const QString suffix =
+		QMimeDatabase().mimeTypeForData(bytes).preferredSuffix();
+	const QString path =
+		dir + QStringLiteral("/dropped") +
+		(suffix.isEmpty() ? QString() : QLatin1Char('.') + suffix);
+
+	QFile f(path);
+	if (!f.open(QIODevice::WriteOnly) || f.write(bytes) != bytes.size()) {
+		failWith(id, tr("Could not write temp file"));
+		return;
+	}
+	f.close();
+	succeed(id, path);
+}
+
+void DropImporter::download(int id, const QUrl& url) {
+	const QString dir = newSubdir();
+	if (dir.isEmpty()) {
+		failWith(id, tr("No temp directory"));
+		return;
+	}
+
+	++m_requests[id].pending;
+
+	QNetworkRequest req(url);
+	req.setAttribute(
+		QNetworkRequest::RedirectPolicyAttribute,
+		QNetworkRequest::NoLessSafeRedirectPolicy);
+	QNetworkReply* reply = m_nam.get(req);
+	auto* file = new QFile(dir + QStringLiteral("/download.part"), reply);
+
+	connect(reply, &QNetworkReply::readyRead, this, [reply, file] {
+		if (!file->isOpen() && !file->open(QIODevice::WriteOnly)) {
+			reply->abort();
+			return;
+		}
+		file->write(reply->readAll());
+		if (file->size() > kMaxBytes) reply->abort();
+	});
+
+	connect(
+		reply,
+		&QNetworkReply::finished,
+		this,
+		[this, reply, file, id, dir, url] {
+			reply->deleteLater();
+			const bool tooBig = file->size() > kMaxBytes;
+			const bool wrote = file->isOpen();
+			file->close();
+
+			if (tooBig) {
+				failWith(id, tr("Download too large: %1").arg(url.toString()));
+			} else if (reply->error() != QNetworkReply::NoError) {
+				failWith(
+					id,
+					tr("Download failed (%1): %2")
+						.arg(url.toString(), reply->errorString()));
+			} else if (!wrote) {
+				failWith(id, tr("Empty response: %1").arg(url.toString()));
+			} else {
+				const QString name = fileNameFor(reply, url);
+				QString finalPath = dir + QLatin1Char('/') + name;
+				if (QFile::rename(file->fileName(), finalPath)) {
+					if (QFileInfo(name).suffix().isEmpty()) {
+						const QString sfx =
+							QMimeDatabase()
+								.mimeTypeForFile(
+									finalPath, QMimeDatabase::MatchContent)
+								.preferredSuffix();
+						if (!sfx.isEmpty() &&
+							QFile::rename(
+								finalPath, finalPath + QLatin1Char('.') + sfx))
+							finalPath += QLatin1Char('.') + sfx;
+					}
+					succeed(id, finalPath);
+				} else {
+					failWith(id, tr("Could not store download"));
+				}
+			}
+			finishUnit(id);
+		});
+}
+
+void DropImporter::succeed(int id, const QString& path) {
+	if (auto it = m_requests.find(id); it != m_requests.end())
+		it->files.append(path);
+}
+
+void DropImporter::failWith(int id, const QString& message) {
+	if (auto it = m_requests.find(id); it != m_requests.end())
+		it->errors.append(message);
+}
+
+void DropImporter::finishUnit(int id) {
+	auto it = m_requests.find(id);
+	if (it == m_requests.end() || --it->pending > 0) return;
+	const Request r = *it;
+	m_requests.erase(it);
+	emit importFinished(id, r.files, r.errors);
+}
+
+QString DropImporter::newSubdir() {
+	if (!m_tempDir.isValid()) return {};
+	const QString p = m_tempDir.filePath(QString::number(++m_counter));
+	return QDir().mkpath(p) ? p : QString();
+}
+
+} // namespace ZShell::Services

Plugins/ZShell/Services/dropimporter.hpp

@@ -0,0 +1,62 @@
+#pragma once
+#include <QHash>
+
+#include <QList>
+#include <QNetworkAccessManager>
+#include <QObject>
+#include <QQmlEngine>
+#include <QStringList>
+#include <QTemporaryDir>
+#include <QUrl>
+
+namespace ZShell::Services {
+
+class DropImporter : public QObject {
+	Q_OBJECT
+	QML_ELEMENT
+	QML_SINGLETON
+
+	public:
+	enum Decision { Ignore, InsertText, Attach };
+	Q_ENUM(Decision)
+
+	explicit DropImporter(QObject* parent = nullptr);
+
+	Q_INVOKABLE Decision decide(
+		const QList<QUrl>& urls,
+		const QString& text,
+		const QString& html,
+		bool hasRawImage) const;
+
+	Q_INVOKABLE int importAttachments(
+		const QList<QUrl>& urls,
+		const QString& html,
+		const QByteArray& rawImage);
+
+	signals:
+	void importFinished(
+		int id, const QStringList& paths, const QStringList& errors);
+
+	private:
+	struct Request {
+		int pending = 0;
+		QStringList files;
+		QStringList errors;
+	};
+
+	void addLocal(int id, const QUrl& url);
+	void download(int id, const QUrl& url);
+	void saveBytes(int id, const QByteArray& bytes);
+	void succeed(int id, const QString& path);
+	void failWith(int id, const QString& message);
+	void finishUnit(int id);
+	QString newSubdir();
+
+	QNetworkAccessManager m_nam;
+	QTemporaryDir m_tempDir;
+	QHash<int, Request> m_requests;
+	int m_nextId = 0;
+	int m_counter = 0;
+};
+
+} // namespace ZShell::Services

ci/Dockerfile

@@ -3,9 +3,9 @@ FROM archlinux:latest
 RUN pacman -Syu --noconfirm \
  && pacman -S --needed --noconfirm \
       base-devel cmake ninja clang lld nodejs tree-sitter \
-      jkqtplotter cmark-gfm git wayland-protocols \
+      jkqtplotter cmark-gfm git wayland-protocols ffmpeg mpv \
       qt6-base qt6-declarative qt6-shadertools qt6-5compat \
-      qt6-canvaspainter qt6-wayland qt5-wayland\
+      qt6-canvaspainter qt6-wayland qt5-wayland \
       libqalculate pipewire aubio cava lm_sensors glib2 \
  && pacman -Scc --noconfirm
 

cli/src/zshell/subcommands/shell.py

@@ -1,3 +1,4 @@
+import os
 import subprocess
 import sys
 import time
@@ -8,6 +9,15 @@ args = ["qs", "-c", "zshell"]
 
 app = typer.Typer()
 
+qt_logging_rules = (
+	"qt.qml.warning=false;"
+	"qt.qpa.wayland.*=false;"
+	"scene=false;"
+	"quickshell.service.sni.watcher=false;"
+	"default.warning=false;"
+	"default.info=false"
+)
+
 
 @app.command()
 def kill():
@@ -19,8 +29,13 @@ def kill():
 
 
 def start_instance(no_daemon: bool = False) -> None:
+	env = os.environ.copy()
+	env["QT_LOGGING_RULES"] = qt_logging_rules
+
 	result = subprocess.run(
-		args + ["-n"] + ([] if no_daemon else ["-d"]), capture_output=True
+		args + ["-n"] + ([] if no_daemon else ["-d"]),
+		capture_output=True,
+		env=env,
 	)
 	stdout = result.stdout.decode().strip()
 	if stdout and "already running" in stdout.lower():

makefile

@@ -1,9 +1,11 @@
 BUILD		?=		cmake -B build -G Ninja
+NONUITKA ?=	cmake -S . -B build -G Ninja -DCMAKE_BUILD_TYPE=Release -DENABLE_MODULES="plugin;shell;m3shapes"
 NINJA		?=		ninja -C build
 COMPILE		?=		$(BUILD) && $(NINJA)
 INSTALL		?=		$(BUILD) && $(NINJA) && sudo ninja -C build install
 
 CPPFMT ?=		find . \( -name '*.cpp' -o -name '*.h' -o -name '*.hpp' \) -not -path './build/*' -exec clang-format -i --style=file {} +
+CPPLINT ?=		run-clang-tidy -p build -j "$$(nproc)" -source-filter="^$$(pwd)/Plugins/" -header-filter=".*Plugins/.*" -removed-arg=-mno-direct-extern-access
 
 NUITKA ?=		nuitka --module --include-package=zshell cli/src/zshell/ --output-dir=cli/
 
@@ -16,7 +18,7 @@ help:
 	@echo "compile			cmake builds and compiles C++ code"
 	@echo "cpp-fmt			format C++ code with clang-format"
 	@echo "cpp-lint		lint check C++ code with clang-tidy"
-	@echo "cpp-compile		alias to make compile"
+	@echo "cpp-compile		similar to make compile, but excludes Nuitka from compiling python"
 	@echo "py-fmt			format Python code with ruff"
 	@echo "py-lint			lint check Python code with ruff"
 	@echo "py-check		lsp and typecheck all Python code using Basedpyright"
@@ -40,16 +42,15 @@ install:
 compile:
 	$(COMPILE)
 
-# TODO: Anything below is not ready
-
 # C++
 cpp-fmt:
 	$(CPPFMT)
 
-# cpp-lint needs adjustments in ci and here
+cpp-lint:
+	$(CPPLINT)
 
 cpp-compile:
-	$(COMPILE)
+	$(NONUITKA)
 
 # Python
 py-fmt: